Implementation plan
Branch issue-80-config-fail-loud off main @ 4f5ecb1, single commit
titled with (closes #80).
internal/config/config.go: add `envPositiveInt(key string, …
Answering @sneak's question (issuecomment-45166). Read the code on the branch to answer this rather than quoting the PR body.
What reconciliation is for
Size accounting in this PR is a…
Manager note — picking this back up.
Found chunk 2 of the remediation sitting only in a local checkout,
never pushed: branch golangci-v2.12.2, commits 6cf9211, 7ae470e,
and a merge of…
Implementation requirements
Baseline: main @ 4f5ecb1.
Confirmed root cause
internal/server/routes.go. chi runs Use middleware in registration order, and every group registers…
Manager note.
Filed #30 retroactively as the tracking issue for this work, and put both under the new 1.0.0 milestone. Labeled needs-review and assigned to clawbot; this PR was open with no…
Implementation requirements
Baseline: main @ 4f5ecb1. All work in internal/config unless the audit in step 5 finds offenders elsewhere.
Policy being encoded
A set-but-invalid…
Manager note: this PR stays merge-ready and assigned to sneak — no action needed here — but it should be merged BEFORE #54.
Re-verified independently just now: `feature/cache-size-evicti…
Manager note: pulling this back from merge-ready — cross-PR collision with #55 that neither PR's isolated review could have caught.
Both #54 and #55 are based on main at 61f42e6 and…