Block a user
feat: validate configuration on startup, fail fast on bad config (closes #52)
Manager note: independent adversarial review returned FAIL (see review comment above). Relabeling needs-rework and dispatching rework limited to the two blocking findings:
- Fix the CI-authorit…
feat: validate configuration on startup, fail fast on bad config (closes #52)
FAIL — needs-rework
Adversarial review of head 2fb0801 against the definition of done in #52 (issue body plus the plan comment) and the repo's no-silent-fallback rule.
Blocking
- **CI is…
lint: fix all 47 noctx findings (httptest.NewRequest without context)
Done in PR #183 (branch fix-noctx-lint, commit 2164290): all 47
noctx findings resolved, make lint total 94 → 47, make test and
make fmt-check green. Awaiting independent review…
make fmt rewrites vendored static/js/alpine.min.js (missing .prettierignore)
script/bootstrap does not install goimports, which script/fmt requires
fix: resolve all 47 noctx lint findings in tests (closes #175)
Update golangci-lint to v2.12.2 with canonical config
Migrate canonical .golangci.yml to golangci-lint v2 config layout
lint: fix all 47 noctx findings (httptest.NewRequest without context)
Starting this now on a branch off current main (291f85f).
Definition of done for the PR:
- all 47
noctxfindings gone: everyhttptest.NewRequestcall in the six listed test files…