docs(docker-development): trim generic CI scaffolding, bump stale pins#39
Conversation
ci-testing.md carried tutorial-level Problem/Solution walkthroughs for entrypoint bypass, DNS mocking, compose validation, secret-scan exclusions, generic health-check verification, multi-platform build setup, and GH Actions service-container integration testing -- all already one-line-summarized in SKILL.md's Quick Reference / CI Testing Gotchas, and reproducible from a one-line prompt. Cut those plus the Complete CI Workflow Example (which only restated the same patterns). Kept every retro-born gotcha: worker/sidecar HEALTHCHECK inheritance (pgrep self-match), GitLab CI entrypoint-must-be-shell, restricted runner egress, built-image vs editable-install testing, bake target metadata-action inheritance, and local boot-test pitfalls. Renumbered the surviving patterns 1-5 for a contiguous sequence. SKILL.md Quick Reference pinned node:20-alpine and golang:1.22-alpine; bumped to node:24-alpine (current Active LTS, Krypton) and golang:1.26-alpine (current stable; 1.27 is still RC). Signed-off-by: Sebastian Mendel <info@sebastianmendel.de>
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Scanned FilesNone |
There was a problem hiding this comment.
Code Review
This pull request updates the Node.js and Go base image versions in the multi-stage build examples in SKILL.md, and refactors ci-testing.md by removing several testing patterns and referencing them in SKILL.md instead. The review feedback correctly identifies that the proposed image tags (node:24-alpine and golang:1.26-alpine) refer to unreleased, non-existent versions, and recommends pinning to the current stable releases (node:22-alpine and golang:1.24-alpine) to avoid build failures.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
After trimming the generic tutorials, ci-testing.md holds retro-born gotchas, not comprehensive patterns. Update the two catalog blurbs (README, ARCHITECTURE) to match its new framing. Signed-off-by: Sebastian Mendel <info@sebastianmendel.de>
|



Part of netresearch/skill-repo-skill#157. Ticks the
docker-development-skillcheckbox.Rubric:
netresearch/skill-repo-skillskills/skill-repo/references/skill-quality.md§ "Body rules" / "When to split toreferences/" — content a one-line prompt regenerates (generic tutorials, restated public best practice, boilerplate) is a cut target; retro-born failure patterns, version/ecosystem facts, and inference-suppression rules are hard-protected.Re-verification against current tree
Both audit-named targets still existed at
origin/main(950264d) before this change:skills/docker-development/SKILL.mdQuick Reference pinnednode:20-alpineandgolang:1.22-alpine.skills/docker-development/references/ci-testing.md(446 lines / 1897 words) carried multiple Problem/Solution tutorial sections restating public Docker/GH-Actions knowledge already summarized as one-liners in SKILL.md's own "CI Testing Gotchas" list.What was cut (ci-testing.md, rubric test: one-line-prompt-regenerable)
docker/setup-qemu-action/services:copy-paste boilerplate, no incident behind it.Net: 446 → 159 lines, 1897 → 1112 words (-785 words, -41%).
What was kept (retro-born, hard-protected)
pgrep [m]essenger:consumeself-match trap (renumbered Pattern 1; commitbbe9274).sh -cgotcha (Pattern 2; commit4b7e314).4b7e314).__file__-relative data-path failure (Pattern 4; commit4b7e314).docker-metadata-actioninheritance (Pattern 5; commit6979edf).b3aa6f7) — untouched.docker-via-wslskill content untouched — no changes needed there.Stale pins (SKILL.md Quick Reference)
Verified via
mcp__package-version__check_docker_tags:node:20-alpine→node:24-alpine— Node 24 ("Krypton") is the current Active LTS (Docker Hublts/lts-kryptontags resolve to 24.18.0); Node 26 iscurrentonly, not yet LTS.golang:1.22-alpine→golang:1.26-alpine— 1.26.5 is latest stable; 1.27 is still RC (1.27rc2/1.27-rctags only).This is a version bump, not a content cut — the illustrative Dockerfile examples stay.
Kept, borderline
None — every remaining section in
ci-testing.mdtraces to a specific commit documenting a real incident/gotcha (verified viagit log --oneline -- skills/docker-development/references/ci-testing.md), so no borderline calls were needed.Verification
DO NOT MERGE.