Re-vendor the canonical files from sneak/prompts at dd4027b #13

Closed
opened 2026-10-06 01:29:03 +02:00 by clawbot · 1 comment
Collaborator

sneak/prompts#78: sneak/prompts has changed the canonical files this repository vendors. Re-vendor them once, from sneak/prompts commit dd4027b (dd4027b907), fetching each file from https://git.eeqj.de/sneak/prompts/raw/commit/dd4027b907ef99cdc3187c215cc4d610b7a11efc/<path>.

Files this repository vendors now: .dockerignore .gitea/workflows/check.yml .gitignore .golangci.yml. REPO_POLICIES.md comes from prompts/REPO_POLICIES.md at that commit: the root REPO_POLICIES.md there is a symlink, and its raw URL returns only the link's target path. That repository's own .gitattributes and TODO.md are not canonical.

What changed that matters here:

  • golangci-lint is v2.14.0 (golangci/golangci-lint@sha256:ad862ba6b3798cbe0fd9fd7408d498fd74fbd2623a92406b2fd3898faf0bf98f). The lint phase digest and .golangci.yml change in one commit, since v2.12.2 rejects the new file; the go directive may name at most Go 1.27.
  • The Go Dockerfile example: the test phase uses the Debian Go image so -race works; the stage that compiles installs git, trusts /src with git config --system --add safe.directory /src, and fails when .git is present but no version comes out.
  • Host Go tools are installed with go install pinned to a commit, never as go.mod tool dependencies.
  • The image version comes from git describe inside the build: .dockerignore sends .git without its own or any submodule's config. CHECK_EPOCH is gone; every docker build in script/ passes --no-cache.
  • .gitignore and .dockerignore keep out more secret files, hardware-backed SSH keys included.
  • Agent guidance lives in one root AGENTS.md; a committed file or directory named for one agent tool has its content moved there and is deleted.

Definition of done

  1. Every vendored canonical file is its copy at that commit, fetched rather than hand-edited, plus only the entries item 3 keeps. A canonical file the policies require and this repository lacks is added the same way.
  2. Dockerfile, Makefile and script/ follow REPO_POLICIES.md as of that commit, including the gate phases and the version step.
  3. This repository's own entries are carried forward: the deny entries of the test-support depguard rule in .golangci.yml, anchored host-built artifacts in .dockerignore, and its language's entries in .gitignore and .editorconfig, kept after the canonical content. .gitignore is a base each repository extends for its language: a Go repository keeps at least *.log, *.out, *.test and its binaries (prompts/CODE_STYLEGUIDE_GO.md), and its .editorconfig keeps tabs for *.go. (Corrected 2026-10-06: the first version of this item dropped those entries.)
  4. Findings the new files raise are fixed in the code in the same PR; no vendored file is loosened.
  5. make lint runs with no deprecation warnings, and make check passes.
  6. One reviewed PR against next. Closing this issue ticks this repository on sneak/prompts#78.

Model: opus-5-5

https://git.eeqj.de/sneak/prompts/issues/78: `sneak/prompts` has changed the canonical files this repository vendors. Re-vendor them once, from `sneak/prompts` commit `dd4027b` (https://git.eeqj.de/sneak/prompts/commit/dd4027b907ef99cdc3187c215cc4d610b7a11efc), fetching each file from `https://git.eeqj.de/sneak/prompts/raw/commit/dd4027b907ef99cdc3187c215cc4d610b7a11efc/<path>`. Files this repository vendors now: `.dockerignore .gitea/workflows/check.yml .gitignore .golangci.yml`. `REPO_POLICIES.md` comes from `prompts/REPO_POLICIES.md` at that commit: the root `REPO_POLICIES.md` there is a symlink, and its raw URL returns only the link's target path. That repository's own `.gitattributes` and `TODO.md` are not canonical. What changed that matters here: - golangci-lint is v2.14.0 (`golangci/golangci-lint@sha256:ad862ba6b3798cbe0fd9fd7408d498fd74fbd2623a92406b2fd3898faf0bf98f`). The lint phase digest and `.golangci.yml` change in one commit, since v2.12.2 rejects the new file; the `go` directive may name at most Go 1.27. - The Go `Dockerfile` example: the test phase uses the Debian Go image so `-race` works; the stage that compiles installs `git`, trusts `/src` with `git config --system --add safe.directory /src`, and fails when `.git` is present but no version comes out. - Host Go tools are installed with `go install` pinned to a commit, never as `go.mod` tool dependencies. - The image version comes from `git describe` inside the build: `.dockerignore` sends `.git` without its own or any submodule's `config`. `CHECK_EPOCH` is gone; every `docker build` in `script/` passes `--no-cache`. - `.gitignore` and `.dockerignore` keep out more secret files, hardware-backed SSH keys included. - Agent guidance lives in one root `AGENTS.md`; a committed file or directory named for one agent tool has its content moved there and is deleted. ## Definition of done 1. Every vendored canonical file is its copy at that commit, fetched rather than hand-edited, plus only the entries item 3 keeps. A canonical file the policies require and this repository lacks is added the same way. 2. `Dockerfile`, `Makefile` and `script/` follow `REPO_POLICIES.md` as of that commit, including the gate phases and the version step. 3. This repository's own entries are carried forward: the `deny` entries of the `test-support` depguard rule in `.golangci.yml`, anchored host-built artifacts in `.dockerignore`, and its language's entries in `.gitignore` and `.editorconfig`, kept after the canonical content. `.gitignore` is a base each repository extends for its language: a Go repository keeps at least `*.log`, `*.out`, `*.test` and its binaries (`prompts/CODE_STYLEGUIDE_GO.md`), and its `.editorconfig` keeps tabs for `*.go`. (Corrected 2026-10-06: the first version of this item dropped those entries.) 4. Findings the new files raise are fixed in the code in the same PR; no vendored file is loosened. 5. `make lint` runs with no deprecation warnings, and `make check` passes. 6. One reviewed PR against `next`. Closing this issue ticks this repository on https://git.eeqj.de/sneak/prompts/issues/78. Model: opus-5-5
clawbot self-assigned this 2026-10-06 01:29:03 +02:00
Author
Collaborator

Plan. #10 and #5 are on next: the v2 config layout is in force and lint and test run as Dockerfile phases in the shape REPO_POLICIES.md gives at dd4027b. What is left for this issue:

  • Re-fetch .dockerignore, .gitignore, .gitea/workflows/check.yml and .golangci.yml from dd4027b, byte-identical; carry forward the anchored /attrsum entry in .dockerignore. Add REPO_POLICIES.md (from prompts/REPO_POLICIES.md) and .editorconfig from that commit.
  • One deviation in the workflow: keep fetch-depth: 0 on the checkout step. REPO_POLICIES.md at that commit requires it of a repo that stamps a tag-derived version, which this one does (#7); the canonical workflow omits it.
  • Move the lint phase to the v2.14.0 digest in the same commit as the new .golangci.yml, and fix in code whatever the new linter finds; no vendored file is loosened. make lint prints no deprecation warning.
  • Bring anything else in Dockerfile, Makefile and script/ that still differs from REPO_POLICIES.md at dd4027b into line (for example script/bootstrap checks the pinned goimports by version, not presence), and make the test phase's comment say "permission tests", plural, now that two tests need the unprivileged user.
  • No LICENSE is added here: licensing is sneak's, and agents do not add or change one in this repository.

Model: opus-5-5

Plan. https://git.eeqj.de/sneak/attrsum/issues/10 and https://git.eeqj.de/sneak/attrsum/issues/5 are on `next`: the v2 config layout is in force and lint and test run as `Dockerfile` phases in the shape `REPO_POLICIES.md` gives at `dd4027b`. What is left for this issue: - Re-fetch `.dockerignore`, `.gitignore`, `.gitea/workflows/check.yml` and `.golangci.yml` from `dd4027b`, byte-identical; carry forward the anchored `/attrsum` entry in `.dockerignore`. Add `REPO_POLICIES.md` (from `prompts/REPO_POLICIES.md`) and `.editorconfig` from that commit. - One deviation in the workflow: keep `fetch-depth: 0` on the checkout step. `REPO_POLICIES.md` at that commit requires it of a repo that stamps a tag-derived version, which this one does (https://git.eeqj.de/sneak/attrsum/issues/7); the canonical workflow omits it. - Move the lint phase to the v2.14.0 digest in the same commit as the new `.golangci.yml`, and fix in code whatever the new linter finds; no vendored file is loosened. `make lint` prints no deprecation warning. - Bring anything else in `Dockerfile`, `Makefile` and `script/` that still differs from `REPO_POLICIES.md` at `dd4027b` into line (for example `script/bootstrap` checks the pinned goimports by version, not presence), and make the test phase's comment say "permission tests", plural, now that two tests need the unprivileged user. - No `LICENSE` is added here: licensing is sneak's, and agents do not add or change one in this repository. Model: opus-5-5
Sign in to join this conversation.