script/lint runs the linter directly when it is already inside a container and otherwise builds Dockerfile.lint, so the linter never runs on a developer host. That closes three host-only mechanisms: the result cache golangci-lint keys on file content rather than location, which produced a confirmed false green and findings reported against other checkouts; the host-global $TMPDIR/golangci-lint.lock, which fails a run in a way no caller can distinguish from findings; and host/container version skew, which hid thirteen findings on one repo. Detection is on LINT_IN_CONTAINER=1, set by every Dockerfile, and on nothing else. The two directions are not symmetric: a false negative inside a container attempts a nested docker build, finds no daemon and fails loudly, while a false positive on a host silently lints there, which is the defect this issue exists to kill. /.dockerenv is therefore rejected even as a fallback -- measured absent inside BuildKit RUN steps and present on any host that is itself a container, so it fails in both directions and one of them is the dangerous one. Nothing else changes shape. The Dockerfile still runs make check, script/check still runs test, lint and fmt-check, script/cibuild is still a single docker build with CHECK_EPOCH and VERSION, and the Go multistage lint stage and its COPY --from=lint ordering dependency survive with ENV LINT_IN_CONTAINER=1 added. Dockerfile.lint is the standalone developer-host path and carries the same CHECK_EPOCH guard, with the ARG below the dependency layer so only the lint re-runs. The script/bootstrap golangci-lint install and the per-checkout GOLANGCI_LINT_CACHE/TMPDIR wrapper are deleted as superseded. Neither has a caller left. A JS repo's yarn install stays: the rule is that no lint verdict may come from a host invocation, not that no linter binary may exist there, and in a repo whose formatter is its linter the formatter necessarily runs on the host. golangci-lint config verify is kept, on measurement. Under the pinned v2.12.2 a bogus top-level key and a bogus key under linters.settings.lll both pass `golangci-lint run` with exit 0 and `0 issues` while config verify exits 3 and names them; an unknown linter name fails run and passes config verify. It needs no network: every case reproduced byte-identically under `docker run --network none`, in a container where `getent hosts golangci-lint.run` exits 2. Comment blocks were cut hard across every file this unit touches. .dockerignore drops from 67 comment lines to 28, script/cibuild from 17 to 12, script/docker from 18 to 12, and prompts/REPO_POLICIES.md from 1182 lines to 907. What remains says why a line is load-bearing; the discovery narratives are gone. config verify lives in script/lint's native branch rather than in a Dockerfile, so every path that lints inherits it: the lint stage of the main image, which is what CI runs, as well as Dockerfile.lint. Putting it in one Dockerfile is how the other path silently loses it.
51 KiB
title, last_modified
| title | last_modified |
|---|---|
| Repository Policies | 2026-08-10 |
This document covers repository structure, tooling, and workflow standards. Code style conventions are in separate documents:
- Code Styleguide (general, bash, Docker)
- Go
- JavaScript
- Python
- Go HTTP Server Conventions
-
Cross-project documentation (such as this file) must include
last_modified: YYYY-MM-DDin the YAML front matter so it can be kept in sync with the authoritative source as policies evolve. -
ALL external references must be pinned by cryptographic hash. This includes Docker base images, Go modules, npm packages, GitHub Actions, and anything else fetched from a remote source. Version tags (
@v4,@latest,:3.21, etc.) are server-mutable and therefore remote code execution vulnerabilities. The ONLY acceptable way to reference an external dependency is by its content hash (Docker@sha256:..., Go module hash ingo.sum, npm integrity hash in lockfile, GitHub Actions@<commit-sha>). No exceptions. This also means nevercurl | bashto install tools like pyenv, nvm, rustup, etc. Instead, download a specific release archive from GitHub, verify its hash (hardcoded in the Dockerfile or script), and only then install. Unverified install scripts are arbitrary remote code execution. This is the single most important rule in this document. Double-check every external reference in every file before committing. There are zero exceptions to this rule. -
Every repo with software must have a root
Makefilewith these targets:make bootstrap,make setup,make test,make lint,make fmt(writes),make fmt-check(read-only),make check(runstest,lint,fmt-check),make docker, andmake hooks(installs pre-commit hook). A model Makefile is athttps://git.eeqj.de/sneak/prompts/raw/branch/main/Makefile. -
Repos follow the Scripts to Rule Them All pattern: the implementation of each Makefile target lives in an executable script in
script/(script/bootstrap,script/setup,script/test,script/lint,script/fmt,script/fmt-check,script/check,script/docker), and the Makefile targets are thin shims that call them. The scripts must be POSIX sh (#!/bin/sh,set -eu, no bashisms) so they run in minimal containers (e.g. alpine images have no bash); locate the repo root with$(cd "$(dirname "$0")/.." && pwd -P)andcdthere before acting. From the standard's canonical set we usebootstrap,setup(make the repo ready for development after a fresh clone: runsbootstrap, theninstall-precommit, plus any repo-specific initialization),test, andcibuild.script/bootstrapinstalls all dependencies idempotently and assumes nothing is present: base tools come from nix, apt, brew, or apk (detected in that order; apt runs noninteractive). For node it uses the installed node if present; otherwise it installs a PINNED node version via nvm, first installing nvm itself if missing — from a hash-verified GitHub release archive (nevercurl | sh), with bash installed as an explicit prerequisite since nvm requires bash. yarn is then pinned viacorepack prepare yarn@<version> --activate. Never install "latest" or "lts"; always exact versions.script/cibuildruns the CI build: it changes to the repo root and runsdocker build --build-arg CHECK_EPOCH="$epoch" --build-arg VERSION="$version" ., whereepochis a per-invocation nonce (see theCHECK_EPOCHrule below) andversionis computed on the host because.gitis not in the build context (see the git-describe rule below); the Gitea workflow calls it. Four further scripts are our own extensions to the standard:script/checkrunsscript/test,script/lint, andscript/fmt-check;script/precommitis what the git pre-commit hook runs, and it callsscript/check;script/install-precommitinstalls the git pre-commit hook (themake hookstarget shims to it); andscript/projectname(literally that filename) simply outputs the project's name. Scripts that need the name callscript/projectname— e.g.script/dockerassembles its image tag from it — so those scripts stay byte-identical across all repos. Repo-type-specific pre-commit extras (e.g.go mod tidyverification in Go repos) belong inscript/precommit, not in the hook itself. Model scripts are athttps://git.eeqj.de/sneak/prompts/raw/branch/main/script/<name>. The README must document the provided scripts in an Entrypoints section (see the README requirements below). -
Always use Makefile targets (
make fmt,make test,make lint, etc.) instead of invoking the underlying tools directly. The Makefile is the single source of truth for how these operations are run. -
The Makefile is authoritative documentation for how the repo is used. Beyond the required targets above, it should have targets for every common operation: running a local development server (
make run,make dev), re-initializing or migrating the database (make db-reset,make migrate), building artifacts (make build), generating code, seeding data, or anything else a developer would do regularly. If someone checks out the repo and typesmake<tab>, they should see every meaningful operation available. A new contributor should be able to understand the entire development workflow by reading the Makefile. -
Every repo should have a
Dockerfile. All Dockerfiles must runmake checkas a build step so the build fails if the branch is not green — the one exception beingDockerfile.lint, which runsmake lintalone because that is its entire purpose — which requiresARG CHECK_EPOCHand its guard in every stage containing a check-runningRUN, per theCHECK_EPOCHrule below. Without them a Dockerfile satisfies this criterion while its check layers are served from cache, so the build cannot fail on a branch that is not green.Every Dockerfile must also set
ENV LINT_IN_CONTAINER=1, above the checks.script/lintbuildsDockerfile.lintwhen it is not already in a container; without the marker it would try that from inside a build step, where there is no daemon. See the containerised-lint rule below.For non-server repos, the Dockerfile should bring up a development environment and run
make check. For server repos,make checkshould run as an early build stage before the final image is assembled. Dockerfiles install development prerequisites by runningscript/bootstraprather than duplicating installs inline; COPYscript/and the dependency manifests (package.json+yarn.lock,go.mod+go.sum, etc.) before running it so the bootstrap layer stays cached until dependencies change. -
Every check-running
RUNmust be cache-busted withCHECK_EPOCH. Docker invalidates aCOPYlayer only when the copied content changes, so on an unchanged tree the check layer is served from cache, the suite never runs, and the build still exits 0. A sub-seconddocker buildreporting success is a cache hit, not a result. This applies to every file that runs checks in a build step —DockerfileandDockerfile.lintalike; aDockerfile.lintwithout the cache-bust is a lint that never ran, reported as a pass. The canonical form, in every stage containing a check-runningRUN, placed after the dependency-install layer so that layer stays cached:ENV LINT_IN_CONTAINER=1 ARG CHECK_EPOCH RUN [ -n "$CHECK_EPOCH" ] || exit 1 RUN echo "check epoch: ${CHECK_EPOCH}" && make checkENV LINT_IN_CONTAINER=1belongs in every such stage too, and is the line most often missed: without itmake checkreachesscript/lint, which tries to buildDockerfile.lintfrom inside a build step where there is no daemon. See the containerised-lint rule below.and in both
script/cibuildandscript/docker:epoch="$(date +%s%N)$$" version="$(git describe --tags --always --dirty 2>/dev/null || true)" [ -n "$version" ] || version="unknown" docker build \ --build-arg CHECK_EPOCH="$epoch" \ --build-arg VERSION="$version" \ .script/lintneeds the same nonce but is not this command: it builds a different file with-f Dockerfile.lintand passes no version. Copy its form from the containerised-lint rule below, not this block — adocker buildwith no-fbuilds the main image and lints nothing.The
VERSIONlines are there for a different reason, covered by the git-describe rule below; they are shown here so the two rules do not each document half a command. All fourCHECK_EPOCHelements are load-bearing; none is optional, and each guards a failure mode that otherwise fails green:ARGis stage-scoped, so a single declaration leaves the other check stages frozen while the fix reviews as complete. Declare it in every stage that runs checks, immediately above the first suchRUN.- Expand the value into the command. This makes the cache miss contractual
rather than dependent on BuildKit's handling of an unreferenced
ARG, and it puts the epoch in the build log. The guard is itself value-keyed, for the same reason: it references$CHECK_EPOCH, so BuildKit renders the epoch into that layer's description (rendered asRUN [ -n "<epoch>" ] || exit 1) and re-runs it whenever the value changes. Each stage therefore has two independent invalidation points, and the guard always precedes the checkRUN. Keep both: the expansion is defence in depth, and it is what makes the epoch visible in the build output. - The
[ -n ... ]guard is required: an unsetARGis empty, and empty is a stable cache key, so without it a baredocker build .still produces the false green. Failed steps are never cached, so the guard fails on every such invocation, loudly. A baredocker build .failing is by design. - Assign
epoch=on its own line, never inline in the--build-argargument: a failing command substitution inside an argument does not tripset -e, so the inline form silently degrades to an empty constant. The$$suffix is required because busyboxdatedrops%Nand exits 0, so on an alpine host the epoch would degrade to second granularity and concurrent invocations would collide.
This invalidates the check layers and everything after them while leaving
go mod downloadandscript/bootstrapcached, so it does not push against the five-minute Docker build ceiling. Blanket--no-cacheis not an acceptable substitute: it also busts the dependency layer, so every run reinstalls dependencies over the network instead of only the first and those after a manifest change. Never reach fordocker builder prune— the build cache is shared with every other build on the host. -
Every lint run happens in a container.
script/lintruns the linter directly when it is already inside one, and otherwise buildsDockerfile.lintso that it is. Either way the linter never runs on a developer host, where its answer is not trustworthy:- Confirmed false green. golangci-lint keys cached results on file
content, not location, so a second checkout of the same commit serves
its findings. One implementer reported
0 issueson a branch genuinely red with agoconstfinding. Own-clones-instead-of-worktrees does not help; two clones are byte-identical exactly as two worktrees were. - False reds: findings reported against other checkouts and against worktrees already deleted; in one case 399 issues returned to a clean clone that genuinely lints 0.
- Lock contention indistinguishable from findings. golangci-lint flocks
$TMPDIR/golangci-lint.lock(pkg/commands/run.go,acquireFileLock()), host-global and independent ofGOLANGCI_LINT_CACHE, 5-second timeout. It printsparallel golangci-lint is running, analyzes nothing, exits non-zero. Not fixed by per-cache isolation — measured. - Version skew: a host linter differing from the pinned one, with the container surfacing thirteen findings the host missed.
A container has its own cache, its own
TMPDIRand a binary pinned by digest, so none of it is reachable. This supersedes the per-checkoutGOLANGCI_LINT_CACHE/TMPDIRwrapper, which existed only to make a host run trustworthy; delete it on adoption.The canonical
script/lint, whose executable lines are the same in every repo apart from the native lint command:#!/bin/sh # script/lint: run the linter. Inside a container, run it directly; on a # host, build Dockerfile.lint so it runs in one anyway. # # LINT_IN_CONTAINER is set by this repo's Dockerfiles and is the ONLY # accepted signal. Do not add a /.dockerenv fallback: it is absent inside # BuildKit RUN steps and present on hosts that are themselves containers, # so it both misses and false-positives — and a false positive silently # restores host linting. set -eu ROOT="$(cd "$(dirname "$0")/.." && pwd -P)" main() { cd "$ROOT" if [ "${LINT_IN_CONTAINER:-}" = "1" ]; then # config verify lives here, not in a Dockerfile, so every path # that lints inherits it — the lint stage of the main image as # well as Dockerfile.lint. Duplicating it into each Dockerfile # is how one of them silently loses it. golangci-lint config verify --config .golangci.yml exec golangci-lint run --config .golangci.yml ./... fi # Own line, and `$$` because busybox `date` drops %N silently. # Without a fresh nonce the lint layer is cached and this exits 0 # having linted nothing. epoch="$(date +%s%N)$$" docker build \ --build-arg CHECK_EPOCH="$epoch" \ -f Dockerfile.lint \ . } main "$@"and
Dockerfile.lint, the standalone path for a developer host:# Lint-only image, built by script/lint when not already in a container. # golangci/golangci-lint:v2.12.2 (Debian-based), 2026-08-07 FROM golangci/golangci-lint@sha256:5cceeef04e53efe1470638d4b4b4f5ceefd574955ab3941b2d9a68a8c9ad5240 WORKDIR /src ENV LINT_IN_CONTAINER=1 COPY go.mod go.sum ./ RUN go mod download COPY . . # ARG after the dependency layer so only the lint re-runs. ARG CHECK_EPOCH RUN [ -n "$CHECK_EPOCH" ] || exit 1 RUN echo "lint epoch: ${CHECK_EPOCH}" && make lintLoad-bearing properties:
- Detection rests on
LINT_IN_CONTAINER=1and nothing else. Every Dockerfile in the repo sets it; a host does not. The asymmetry is the whole design: a false negative inside a container tries a nesteddocker build, finds no daemon and fails loudly, while a false positive on a host silently lints there — the exact defect this rule exists to kill. So the signal must be one only our own images can produce./.dockerenvis not such a signal and must not be used, even as a fallback: measured, it is absent inside BuildKitRUNsteps and present on any host that is itself a container, which is the common case for CI runners and agent sandboxes. It fails in both directions, and one of them is the dangerous one. CHECK_EPOCH, not--no-cache.docker build -f Dockerfile.lint .on an unchanged tree returns a sub-second cached success having linted nothing. TheARGgoes after the dependency layer so only the lint re-runs;--no-cachewould also reinstall dependencies on every lint.- Non-Go repos get the same pattern around their own linter —
eslint,ruff,prettier,shellcheck. Only the base image and the native lint command change. - Keep
golangci-lint config verify, put it inscript/lint, and it costs no network. It goes in the native branch, not in a Dockerfile, so the lint stage of the main image inherits it along withDockerfile.lint; putting it in one Dockerfile leaves the other path unverified. The two commands catch disjoint classes, measured under the pinned v2.12.2: a bogus top-level key and a bogus key underlinters.settings.lllboth passgolangci-lint runwith exit 0 and0 issueswhileconfig verifyexits 3 and names them; an invalid value type fails both; an unknown linter name failsrunand passesconfig verify. Sorunalone silently ignores an unknown key — the mode where a threshold reads as configured and is not applied. It needs no network: every case reproduced byte-identically underdocker run --network none, in a container wheregetent hosts golangci-lint.runexits 2. The schema is embedded in the pinned binary. Re-run that control when bumping the pin. - A failed
script/lintthat names no finding is not a lint result. On the host pathdocker buildexits 1 both for findings and for a build that never got there (daemon down, image unpullable, disk full). BuildKit names the failing step; read it, fix the environment, re-run. Do not record a verdict from a run that did not lint.
Scope: this rule is about linters, and a formatter is not one.
script/fmtwrites your working tree, so it can only run on the host, andscript/fmt-checkis its read-only twin. In a repo whose formatter is its linter (prettier over markdown; this repo),script/bootstraptherefore installs the linter on the host as an ordinary dependency andscript/fmt-checkruns it there. That is accepted: the version is pinned inpackage.jsonand installed into the repo's ownnode_modules, so there is no shared content-keyed cache, no host-global lock and nothing to skew against. What is forbidden is taking a lint verdict from it —script/lintstays the only source of one. A repo auditing itself will see those hits and should leave them; anything else the grep finds is a real second path to the linter and goes.What a consuming repo does to adopt this, in order:
- Add
Dockerfile.lint. - Replace
script/lintwith the form above, with its own native lint command. - Add
ENV LINT_IN_CONTAINER=1to every stage of every Dockerfile that runs checks — the lint stage and the build stage both. - Delete any golangci-lint install from
script/bootstrap, with its version and ref variables and its call site. No lint verdict comes from the host any more, so it can only reintroduce version skew. A JS repo'syarn installstays. - Delete the per-checkout lint state:
GOLANGCI_LINT_CACHEandTMPDIRexports,--allow-serial-runners, the retry/VOID wrapper, and.lint-cache/from both.gitignoreand.dockerignore. - Verify by running
make linttwice on an unchanged tree: the lint layer must beDONEboth times, neverCACHED. Then plant a violation, confirm it fails naming the finding, revert. A baredocker build -f Dockerfile.lint .must fail on the guard.
script/check,script/cibuild,script/dockerand theDockerfileare unchanged by this:make checkstill runs inside the image, andscript/lintthere takes the native path. - Confirmed false green. golangci-lint keys cached results on file
content, not location, so a second checkout of the same commit serves
its findings. One implementer reported
-
Dockerfiles must use a separate lint stage for fail-fast feedback. Go repos use a multistage build where linting runs in an independent stage based on the
golangci/golangci-lintimage (pinned by hash), so lint failures surface in seconds rather than after a full compile. The build stage declares an explicit dependency on it viaCOPY --from=lint /src/go.sum /dev/null, which forces BuildKit — which runs stages in parallel by default — to finish linting first. The canonical Go repoDockerfile:# Lint stage — fast feedback on formatting and lint issues # golangci/golangci-lint:v2.x.x, YYYY-MM-DD FROM golangci/golangci-lint@sha256:... AS lint WORKDIR /src ENV LINT_IN_CONTAINER=1 COPY go.mod go.sum ./ RUN go mod download COPY . . ARG CHECK_EPOCH RUN [ -n "$CHECK_EPOCH" ] || exit 1 RUN echo "check epoch: ${CHECK_EPOCH}" && make fmt-check RUN make lint # Build stage # golang:1.x-alpine, YYYY-MM-DD FROM golang@sha256:... AS builder WORKDIR /src ENV LINT_IN_CONTAINER=1 # Force BuildKit to run the lint stage before proceeding COPY --from=lint /src/go.sum /dev/null COPY go.mod go.sum ./ RUN go mod download COPY . . ARG CHECK_EPOCH RUN [ -n "$CHECK_EPOCH" ] || exit 1 RUN echo "check epoch: ${CHECK_EPOCH}" && make test # VERSION comes from the host via --build-arg; see the git-describe rule # below. Never run `git describe` here: .dockerignore excludes .git, so # it yields an empty version without failing the build. ARG VERSION=dev RUN CGO_ENABLED=0 go build -trimpath \ -ldflags="-s -w -X main.Version=${VERSION}" \ -o /app ./cmd/app/ # Runtime stage FROM alpine@sha256:... COPY --from=builder /app /usr/local/bin/app ENTRYPOINT ["app"]Key points:
- The lint stage uses the
golangci/golangci-lintimage directly (it has both Go and the linter), so nothing needs installing.make lintthere runsscript/lint, which seesLINT_IN_CONTAINER=1and invokesgolangci-lintnatively instead of buildingDockerfile.lint. Without thatENVthe stage would attempt a nested build and fail. COPY --from=lint /src/go.sum /dev/nullis a no-op copy that exists only to create the stage dependency; without it a lint failure might not fail the overall build.- Re-prove that ordering on a warm cache after adopting
CHECK_EPOCH. The cache-bust turns the no-opCOPYinto a content-cache hit, so an ordering guarantee established cold does not automatically carry over. It was re-proved in another org repo using the same trick and held, but that result does not transfer by assumption — re-check it warm. - If the project uses
//go:embedreferencing build artifacts, the lint stage must create placeholders so the directives resolve:RUN mkdir -p web/dist && touch web/dist/index.html. - If linting needs CGO or system libraries (e.g.
vips-dev),apk addthem in the lint stage. - Tests run in the build stage, not the lint stage: they may need compiled artifacts or heavier dependencies.
ARG CHECK_EPOCHappears in both stages, becauseARGis stage-scoped: declaring it only in the lint stage leavesmake testfrozen at its last cached result. In each stage the guard sits immediately below theARGand the value is expanded into the first checkRUN. LaterRUNs in the same stage need no expansion; their parent layer is already busted.ARG VERSION=devis declared in the build stage and supplied byscript/dockerandscript/cibuild. No stage may callgit describe:.dockerignoreexcludes.git, so it yields an empty version without failing. See the git-describe rule further down.
- The lint stage uses the
-
Every repo should have a Gitea Actions workflow (
.gitea/workflows/) that runsscript/cibuild(which runsdocker build --build-arg CHECK_EPOCH="$epoch" --build-arg VERSION="$version" .) on push. The Dockerfile runsmake check, so a successful build implies all checks pass — but that implication holds only because of theCHECK_EPOCHcache-bust described above. Without it, an unchanged tree serves the check layer from cache and the build reports a green it never earned. A baredocker build .fails closed by design, on the[ -n "$CHECK_EPOCH" ]guard; always go throughscript/cibuildorscript/docker. Never accept a pass as evidence without confirming it ran: a sub-second wall time, orCACHEDon the check layer, means nothing was executed. -
Use platform-standard formatters:
blackfor Python,prettierfor JS/CSS/Markdown/HTML,go fmtfor Go. Always use default configuration with two exceptions: four-space indents (except Go), andproseWrap: alwaysfor Markdown (hard-wrap at 80 columns). Documentation and writing repos (Markdown, HTML, CSS) should also have.prettierrcand.prettierignore. -
Pre-commit hook: runs
script/precommit, which callsscript/check. If local testing is not possible in the repo,script/precommitmay skipscript/testand run onlyscript/lintandscript/fmt-check. The hook is installed byscript/install-precommit; the Makefile must provide amake hookstarget that shims to it. -
All repos with software must have tests that run via the platform-standard test framework (
go test,pytest,jest/vitest, etc.). If no meaningful tests exist yet, add the most minimal test possible — e.g. importing the module under test to verify it compiles/parses. There is no excuse formake testto be a no-op. -
make testmust complete in under 20 seconds. Add a 30-second timeout in the Makefile. -
make testshould use the conditional verbose rerun pattern. Run tests without-v(verbose) first. If tests fail, automatically rerun with-vto show full output. This keeps CI logs anddocker buildoutput clean on success (just package/suite summaries) while providing full diagnostic detail on failure (every test case, every assertion). The general shell pattern:test: @<test-command> || \ { echo "--- Rerunning with -v for details ---"; \ <test-command-with-v>; exit 1; }Go example:
test: @go test -timeout 30s -race -cover ./... || \ { echo "--- Rerunning with -v for details ---"; \ go test -timeout 30s -race -v ./...; exit 1; }Python example:
test: @python -m pytest || \ { echo "--- Rerunning with -v for details ---"; \ python -m pytest -v; exit 1; }The
exit 1ensures the target always fails after a rerun — the first run already proved the tests are broken, so the build must not pass even if a flaky test happens to succeed on the second attempt. The rerun exists solely for diagnostic output. -
Docker builds must complete in under 5 minutes.
-
make checkmust not modify any files in the repo. Tests may use temporary directories. -
mainmust always passmake check, no exceptions. -
Never commit secrets.
.envfiles, credentials, API keys, and private keys must be in.gitignore. No exceptions. -
.gitignoreshould be comprehensive from the start: OS files (.DS_Store), editor files (.swp,*~), in-repo agent scratch directories (.claude/, which holds one worktree — an entire additional checkout of the repo — per in-flight agent), language build artifacts, andnode_modules/. Fetch the standard.gitignorefromhttps://git.eeqj.de/sneak/prompts/raw/branch/main/.gitignorewhen setting up a new repo. These patterns are written to.gitignore's own semantics, in which an unanchored pattern already matches at every depth. They are not a.dockerignoreand must not be transplanted into one unmodified — see the next rule. -
.dockerignoredoes not use.gitignoresemantics, and copying patterns across unmodified leaves secrets in the build context. Docker matches withmoby/patternmatcher: Gofilepath.Matchsemantics plus a**extension, compiled to a regexp — plainfilepath.Matchhas no**at all. So*does not cross/, and a pattern without a leading**/is anchored at the build-context root. A.dockerignorelisting.env,*.pemand*.keytherefore excludes only the copies at the repository root;config/.envandcerts/server.keystill reach the context and can land in an image layer. That file is more dangerous than a short one with no secret patterns at all, because it reads as solved and stops anyone looking. Give every depth-independent pattern the**/prefix —**/node_modules,**/.DS_Store, and the secret patterns in the canonical file, which are additionally case-folded per the rule below — and leave only genuinely root-anchored entries unprefixed:.git, the in-repo agent scratch directory.claude, and the repo's own host-built binary. The inverse move is equally wrong: never apply**/to.gitignore, where it is redundant and produces a file that is wrong in a way that looks careful. Each file is written to its own semantics; neither is derived from the other. Fetch the standard.dockerignorefromhttps://git.eeqj.de/sneak/prompts/raw/branch/main/.dockerignoreand extend it with the repo's own host-built artifacts — a hostmake buildthat leaves a compiled binary in the repo root puts that binary in the build context, where.gitignorehides it from every git-based check. Write that binary anchored,/myappand never**/myapp: the prefixed form also matchescmd/myapp/and deletes the package directory from the context. -
In-repo agent scratch belongs in both files, written to each file's own semantics.
.claude/holds one worktree per in-flight agent — an entire additional checkout of the repo — so withCOPY . .the build context inflates by a multiple of the repo, and another session's unreviewed, sometimes uncommitted work can be copied into an image layer. The directory is also created and destroyed constantly, so it invalidatesCOPY . .for reasons that have nothing to do with the repo's own content. In.gitignorethe entry is.claude/, unanchored, which already matches at every depth. In.dockerignoreit is.claude, anchored and with no**/prefix: the directory occurs exactly once where agents run at the repo root, and the prefixed form would also match any nested directory of that name and delete it from the build. It is not case-folded the way the secret patterns are, because tooling creates it in exactly one spelling, so a folded pattern would add no coverage.Known gap that comes with the anchored form. The directory is created in the agent's working directory, so the "exactly once, at the root" premise is a property of how agents are run and not of the tooling. Where agents run in subdirectories — a monorepo with a per-service agent is the ordinary case —
services/api/.claude/is not excluded by the canonical entry and still reaches the build context and the image, which is the exposure the entry exists to close. A repo in that shape adds its own anchored entries (/services/api/.claude), or**/.claudeonce it has confirmed no legitimately named nested directory would be caught. This is stated in the canonical.dockerignoreitself, since that file is what consuming repos receive. -
.dockerignorematching is case-sensitive, so cover capitalisation with character classes rather than by doubling patterns.**/*.keydoes not matchcerts/SERVER.KEY, which is reachable on the case-insensitive filesystems most laptops use. Adding an ALL-CAPS twin for each pattern is not the fix: it still missesServer.KeyandCa.Pemwhile reading as though case were handled — the same manufactured confidence as the root-anchored form. The matcher supports character ranges, so one line covers every spelling:**/*.[kK][eE][yY],**/*.[pP][eE][mM]. Apply this to every secret name, not only to extensions: the extensionless SSH keys and.envrcneed it for the same reason, since on the very filesystems that makeSERVER.KEYreachable, direnv reads.ENVRCand ssh readsID_RSA. Note that*matches the empty string, so**/*.[eE][nN][vV]already covers a bare.ENVand no separate literal.enventry is needed. -
A pattern that also catches something the build needs is re-included with a negation, not deleted. The canonical
**/*.[eE][nN][vV]excludes a committed env template such asexample.env; a repo whose build genuinely reads one adds!docs/example.envafter the pattern. Deleting the pattern instead reopens the exposure for every other file it covers. -
Verify
.dockerignoreby enumerating the image, not by reading the patterns. Plant files at the root and at least two directories deep, build a probe image that doesCOPY . ., and list what actually landed (docker run --rm --entrypoint find IMAGE /app). Reading the patterns and agreeing they look right is exactly what lets the root-only form through. Thetransferring contextsize is not a substitute: a nested secret is a few bytes, and BuildKit transfers only the delta from the previous build, so the reported size describes the transfer and not the contents of the image. -
Excluding
.gitmeansgit describecannot run inside any build stage, and it fails quietly there. TheGOLDFLAGSversion-embedding pattern assumes.gitis present; in a build stage there is no repository, sogit describewrites nothing to stdout and the-X main.Version=value comes out empty rather than erroring. The binary then reports no version at all and the build still exits 0. Compute the version on the host and thread it in as a build arg.script/dockerandscript/cibuilddo this, byte-identically across repos:# Assign on its own line: a failing command substitution inside an # argument does not trip `set -e`, so the inline form degrades to an # empty constant — the same silent-empty failure this rule is about. version="$(git describe --tags --always --dirty 2>/dev/null || true)" [ -n "$version" ] || version="unknown" docker build \ --build-arg CHECK_EPOCH="$epoch" \ --build-arg VERSION="$version" \ .--alwaysmakes an untagged repo yield the abbreviated commit hash instead of failing.|| truekeeps a failinggit describefrom trippingset -eand leaves the value empty, so the[ -n "$version" ]line is the single place the fallback is applied — and it is a live check, not defence in depth: it fires on a build from an export with no.git, and on a repository with no commits yet. Do not fold the fallback into the substitution as|| echo unknown; that makes the guard unreachable, and a guard that cannot fire is indistinguishable from one that works to everyone who copies it. The result is non-empty by construction either way, which is the point: an empty version reads as a successful one, whileunknownis visibly wrong. The Dockerfile's side isARG VERSION=devin the stage that compiles, declared there and not inherited, becauseARGis stage-scoped exactly asCHECK_EPOCHis. PassingVERSIONto a repo whose Dockerfile declares no suchARGis silently ignored by BuildKit and costs nothing, which is why the scripts stay byte-identical rather than growing a per-repo variant.One consequence for CI: the standard checkout action clones shallow and fetches no tags, so
git describe --tagsthere falls back to a bare commit hash. A repo that embeds a tag-derived version must setfetch-depth: 0on its checkout step; a repo that does not embed a version needs no change. -
No build artifacts in version control. Code-derived data (compiled bundles, minified output, generated assets) must never be committed to the repository if it can be avoided. The build process (e.g. Dockerfile, Makefile) should generate these at build time. Notable exception: Go protobuf generated files (
.pb.go) ARE committed because repos need to work withgo get, which downloads code but does not execute code generation. -
Never use
git add -Aorgit add .. Always stage files explicitly by name. -
Never force-push to
main. -
Make all changes on a feature branch. You can do whatever you want on a feature branch.
-
.golangci.ymlis standardized and must NEVER be modified by an agent, only manually by the user. Fetch fromhttps://git.eeqj.de/sneak/prompts/raw/branch/main/.golangci.yml. The canonical golangci-lint version is v2.12.2 (released 2026-05-06), pinned as the image digest inDockerfile.lint(golangci/golangci-lint@sha256:5cceeef04e53efe1470638d4b4b4f5ceefd574955ab3941b2d9a68a8c9ad5240, which reportsgolangci-lint has version 2.12.2 built with go1.26.2 from c0d3ddc9). That digest is the only pin: golangci-lint is not installed on the host by any repo. Bumping the version means changing that one digest. -
script/bootstrapmust not install golangci-lint. This supersedes the pinned host install that used to be canonical here.script/lintnever runs it on the host — it either buildsDockerfile.lintor is already in a container that ships the binary — so a host install has no caller, and its only remaining effect is to put a second, independently-versioned linter where somebody eventually runs it by hand and believes the result. Delete the block, its version and ref variables, and its call site.This is not a ban on host dependency installs generally. A JS or docs repo's
script/bootstraprunsyarn install, which brings its linter along with every other dependency; that is unavoidable and fine. The rule is about a dedicated linter install, and about where a verdict may come from.The version-enforcement principle it established still applies to any other tool a repo pins and installs on the host, and it is the part worth keeping, because each of its four properties guards a failure that otherwise reports success:
- Compare the installed version against the pin, never test presence. A
if missing <tool>; then install; figuard testsPATHpresence and never version, so on any already-provisioned machine the pin is inert and a version bump is a silent no-op. Compare the whole version token, exactly: a parser that stops at the first-reports2.12.2for a host running2.12.2-rc1and skips the install — the original defect, reintroduced through the comparison meant to fix it. - After installing, re-resolve the binary the way callers resolve it —
through
PATH, not the directory the installer wrote to — and assert the reported version is the pin. An installer that writes toGOBINwhile a different binary shadows it earlier inPATHgenuinely succeeds and changes nothing any caller sees, which is worse than no fix: it converts a known-stale tool into one everyone believes is pinned. Runhash -rfirst so the shell does not answer from its own lookup cache, and when the assertion fails, name the pathcommand -vfound, the version it reports, and the directory the install wrote to. Diagnose from the resolved path rather than asserting a cause: only a path outside the install directory is shadowing. - A mis-parse must fall through to reinstall, never to a false match. Absent binary, non-zero exit, empty output and unrecognised output should all yield an empty string, which compares unequal to the pin. The failure direction is always a redundant install, never a skipped one.
- Call it, and say so on success. A function defined and never called is a silent no-op indistinguishable from success: exit 0, nothing installed, no output. Both success branches must print a line naming the version.
Verifying such logic requires a negative control in an environment where a shadowing binary exists earlier in
PATHthan the install target — without it the control passes against the naive compare-then-install form too and proves nothing — plus a mis-parse control that feeds unparseable--versionoutput and confirms a reinstall. Run those controls against the block as a consuming repo would adopt it: pasted into ascript/bootstrap-shaped file that is then executed, never by sourcing it and invoking the function yourself. Driving the function directly tests something the artifact does not do, and it is exactly how a missing call site passes every control while the adopted snippet does nothing.Keep it POSIX sh: no bashisms, no arrays, no
[[, nogrep -P. - Compare the installed version against the pin, never test presence. A
-
Superseded: the per-checkout
GOLANGCI_LINT_CACHE/TMPDIRwrapper forscript/lint. It existed only to make a host lint run trustworthy, and the containerised-lint rule above removes the host run. Delete the wrapper, the--allow-serial-runnersflag, and.lint-cache/from both.gitignoreand.dockerignore. Two of its conclusions outlive it:GOCACHEdoes not need isolating (measured — content-addressed, no foreign paths in its entries, no global lock), and verifying lint plumbing requires paired controls run against the artifact as a consuming repo would adopt it, since a control that passes against the broken form proves nothing. -
Interim rule for reading a lint result produced on the host, in a repo that has not yet adopted the containerised lint above. A lint run is VOID unless both hold:
- the output contains no
parallel golangci-lint is running, and - no reported file path begins with
../, and none is an absolute path outside the tree the run was launched from.
Do not record a verdict from a void run, and do not "fix" findings in files the change does not touch — chasing phantom findings across untouched files puts unrelated edits into a reviewed diff, which is more expensive than the wasted rework.
The
../clause is the one that actually bites, and it is why a filter keyed on/tmpor on absolute prefixes is not enough: golangci-lint reports paths relative to its own resolved root rather than yours, and three of the org's reported sightings had relative paths and would have passed such a filter. Both clauses are needed and neither alone is sufficient — one reproduction exited non-zero with the lock error and no foreign paths at all, and another reported 34 well-formed findings, every one of them against another checkout.State the limit of these tests rather than treating them as a guarantee. They catch contamination that names foreign files. They cannot catch contamination that suppresses findings through a poisoned entry for colliding content, which has no wall-clock tell either — no evidence of that mode has been observed, and nobody should go chasing it; the point is the reach of the tests, not a claim that the mode exists. They are a filter for the loud mode, not a proof of soundness — which is the whole argument for containerising the linter instead of documenting a discipline that depends on every agent remembering to apply it. Adopt the rule above and this one stops applying to the repo entirely.
- the output contains no
-
When pinning images or packages by hash, add a comment above the reference with the version and date (YYYY-MM-DD).
-
Use
yarn, notnpm. -
Write all dates as YYYY-MM-DD (ISO 8601).
-
Simple projects should be configured with environment variables.
-
Dockerized web services listen on port 8080 by default, overridable with
PORT. -
HTTP/web services must be hardened for production internet exposure before tagging 1.0. This means full compliance with security best practices including, without limitation, all of the following:
- Security headers on every response:
Strict-Transport-Security(HSTS) withmax-ageof at least one year andincludeSubDomains.Content-Security-Policy(CSP) with a restrictive default policy (default-src 'self'as a baseline, tightened per-resource as needed). Never useunsafe-inlineorunsafe-evalunless unavoidable, and document the reason.X-Frame-Options: DENY(orSAMEORIGINif framing is required). Prefer theframe-ancestorsCSP directive as the primary control.X-Content-Type-Options: nosniff.Referrer-Policy: strict-origin-when-cross-origin(or stricter).Permissions-Policyrestricting access to browser features the application does not use (camera, microphone, geolocation, etc.).
- Request and response limits:
- Maximum request body size enforced on all endpoints (e.g. Go
http.MaxBytesReader). Choose a sane default per-route; never accept unbounded input. - Maximum response body size where applicable (e.g. paginated APIs).
ReadTimeoutandReadHeaderTimeouton thehttp.Serverto defend against slowloris attacks.WriteTimeouton thehttp.Server.IdleTimeouton thehttp.Server.- Per-handler execution time limits via
context.WithTimeoutor chi/stdlibmiddleware.Timeout.
- Maximum request body size enforced on all endpoints (e.g. Go
- Authentication and session security:
- Rate limiting on password-based authentication endpoints. API keys are high-entropy and not susceptible to brute force, so they are exempt.
- CSRF tokens on all state-mutating HTML forms. API endpoints
authenticated via
Authorizationheader (Bearer token, API key) are exempt because the browser does not attach these automatically. - Passwords stored using bcrypt, scrypt, or argon2 — never plain-text, MD5, or SHA.
- Session cookies set with
HttpOnly,Secure, andSameSite=Lax(orStrict) attributes.
- Reverse proxy awareness:
- True client IP detection when behind a reverse proxy
(
X-Forwarded-For,X-Real-IP). The application must accept forwarded headers only from a configured set of trusted proxy addresses — never trustX-Forwarded-Forunconditionally.
- True client IP detection when behind a reverse proxy
(
- CORS:
- Authenticated endpoints must restrict
Access-Control-Allow-Originto an explicit allowlist of known origins. Wildcard (*) is acceptable only for public, unauthenticated read-only APIs.
- Authenticated endpoints must restrict
- Error handling:
- Internal errors must never leak stack traces, SQL queries, file paths,
or other implementation details to the client. Return generic error
messages in production; detailed errors only when
DEBUGis enabled.
- Internal errors must never leak stack traces, SQL queries, file paths,
or other implementation details to the client. Return generic error
messages in production; detailed errors only when
- TLS:
- Services never terminate TLS directly. They are always deployed behind
a TLS-terminating reverse proxy. The service itself listens on plain
HTTP. However, HSTS headers and
Securecookie flags must still be set by the application so that the browser enforces HTTPS end-to-end.
- Services never terminate TLS directly. They are always deployed behind
a TLS-terminating reverse proxy. The service itself listens on plain
HTTP. However, HSTS headers and
This list is non-exhaustive. Apply defense-in-depth: if a standard security hardening measure exists for HTTP services and is not listed here, it is still expected. When in doubt, harden.
- Security headers on every response:
-
README.mdis the primary documentation. Required sections:- Description: First line must include the project name, purpose, category (web server, SPA, CLI tool, etc.), license, and author. Example: "µPaaS is an MIT-licensed Go web application by @sneak that receives git-frontend webhooks and deploys applications via Docker in realtime."
- Getting Started: Copy-pasteable install/usage code block.
- Entrypoints: Opens by stating that the repo adheres to the
Scripts to Rule Them All
standard (with that link), then documents each provided
script/entrypoint and its purpose. - Rationale: Why does this exist?
- Design: How is the program structured?
- TODO: Update meticulously, even between commits. When planning, put the todo list in the README so a new agent can pick up where the last one left off.
- License: MIT, GPL, or WTFPL. Ask the user for new projects. Include a
LICENSEfile in the repo root and a License section in the README. - Author: @sneak.
-
First commit of a new repo should contain only
README.md. -
Go module root:
sneak.berlin/go/<name>. Always rungo mod tidybefore committing. -
Use SemVer.
-
Database migrations live in
internal/db/migrations/and must be embedded in the binary.000_migration.sql— contains ONLY the creation of the migrations tracking table itself. Nothing else.001_schema.sql— the full application schema.- Pre-1.0.0: never add additional migration files (002, 003, etc.).
There is no installed base to migrate. Edit
001_schema.sqldirectly. - Post-1.0.0: add new numbered migration files for each schema change. Never edit existing migrations after release.
-
All repos should have an
.editorconfigenforcing the project's indentation settings. -
Avoid putting files in the repo root unless necessary. Root should contain only project-level config files (
README.md,Makefile,Dockerfile,LICENSE,.gitignore,.editorconfig,REPO_POLICIES.md, and language-specific config). Everything else goes in a subdirectory. Canonical subdirectory names:bin/— executable scripts and toolscmd/— Go command entrypointsconfigs/— configuration templates and examplesdeploy/— deployment manifests (k8s, compose, terraform)docs/— documentation and markdown (README.md stays in root)internal/— Go internal packagesinternal/db/migrations/— database migrationspkg/— Go library packagesshare/— systemd units, data filesstatic/— static assets (images, fonts, etc.)web/— web frontend source
-
When setting up a new repo, files from the
promptsrepo may be used as templates. Fetch them fromhttps://git.eeqj.de/sneak/prompts/raw/branch/main/<path>. -
New repos must contain at minimum:
README.md,.git,.gitignore,.editorconfigLICENSE,REPO_POLICIES.md(copy from thepromptsrepo)Makefilescript/entrypoints (bootstrap,setup,projectname,test,lint,fmt,fmt-check,check,docker,cibuild,precommit,install-precommit)Dockerfile,.dockerignore.gitea/workflows/check.yml- Go:
go.mod,go.sum,.golangci.yml - JS:
package.json,yarn.lock,.prettierrc,.prettierignore - Python:
pyproject.toml