sneak/prompts
1
0
Fork 0
Code Issues 7 Pull Requests 1 Actions 6 Packages Projects Releases Wiki Activity

Canonical script comments about the lint and test phases and node 22 alpine are not true of every repository #141

New Issue
Open
opened 2026-11-03 17:29:32 +01:00 by clawbot · 1 comment
clawbot commented 2026-11-03 17:29:32 +01:00
Collaborator
Copy Link
Copy Source

The canonical scripts at next (430964c) carry comments that are not true of every repository that takes them:

  • script/lint: "The phase is not the last stage in the file, so it is built only when --target names it." script/test says the same: "a phase that is not the last stage is built only when named". The last stage of the canonical Dockerfile copies a file from lint and from test, so a plain docker build builds both phases as well.
  • script/bootstrap: "the Dockerfile stages start from a node 22 alpine image". That holds for this repository's own Dockerfile. A Go repository's stages start from the golangci-lint and Go images.

Found in the cattbox re-vendor review, sneak/cattbox#56 (comment). That PR keeps its copies unchanged and would take the fix at its next re-vendor.

Definition of done

  • Each comment is true of every repository that takes the script, or is removed.
  • Every copy of those scripts in this repository stays byte-identical.

Model: opus-5-5

The canonical scripts at `next` (`430964c`) carry comments that are not true of every repository that takes them: - `script/lint`: "The phase is not the last stage in the file, so it is built only when --target names it." `script/test` says the same: "a phase that is not the last stage is built only when named". The last stage of the canonical `Dockerfile` copies a file from `lint` and from `test`, so a plain `docker build` builds both phases as well. - `script/bootstrap`: "the Dockerfile stages start from a node 22 alpine image". That holds for this repository's own `Dockerfile`. A Go repository's stages start from the golangci-lint and Go images. Found in the cattbox re-vendor review, https://git.eeqj.de/sneak/cattbox/pulls/56#issuecomment-161673. That PR keeps its copies unchanged and would take the fix at its next re-vendor. ## Definition of done - Each comment is true of every repository that takes the script, or is removed. - Every copy of those scripts in this repository stays byte-identical. Model: opus-5-5
clawbot changed title from Canonical script comments about the lint phase and node 22 alpine are not true of every repository to Canonical script comments about the lint and test phases and node 22 alpine are not true of every repository 2026-11-03 17:29:41 +01:00
clawbot commented 2026-11-04 09:56:33 +01:00
Author
Collaborator
Copy Link
Copy Source

The script/bootstrap half of this is gone: #148 removed the node 22 alpine comment. The script/lint and script/test comments remain.

Model: opus-5-5

The `script/bootstrap` half of this is gone: https://git.eeqj.de/sneak/prompts/pulls/148 removed the node 22 alpine comment. The `script/lint` and `script/test` comments remain. Model: opus-5-5
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
critical
Extremely important: data loss, a security hole, production down, or basic use impossible
 merge-ready
PR approved and ready to merge
 needs-checks
A check is red
 needs-rebase
PR conflicts with its base branch and must be rebased
 needs-review
PR needs code review
 needs-rework
PR needs changes based on review
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
clawbot sneak (Jeffrey Paul)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sneak/prompts#141