Block a user
fix: resolve 1.0 audit bugs (closes #104, #105, #106, #107, #108)
Code Review: PR #109 (fix/1.0-audit-bugs)
Reviewer: clawbot
fix: resolve 1.0 audit bugs (closes #104, #105, #106, #107, #108)
BUG: SetupRequired middleware blocks /health, /s/*, and /api/* before initial setup
BUG: HandleVolumeAdd missing path validation — path traversal possible on volume creation
BUG: API delete endpoint does not stop/remove Docker container — orphaned containers
BUG: API deploy handler uses request context — deployment cancelled on client disconnect
BUG: HandleEnvVarDelete uses wrong route parameter name — env var deletion always 404s
Add make check target and CI workflow
Restructured Makefile per review feedback:
make check-fmt— format checking (gofmt)make lint— golangci-lintmake test— runs testsmake check— depends on check-fmt,…
1.0 quality polish — code review, tests, bug fixes, documentation
Rebased feature/1.0-polish onto origin/next — resolved merge conflicts in mfer/scanner.go and mfer/checker.go.
✅ make test — all tests pass
✅ make lint — 0 issues
Force-push…
Add deterministic file ordering in Builder.Build() (closes #23)
Removed the time-hard hash iteration per review feedback:
- Replaced 150M SHA-256 iteration key-stretching with a single hash in
SetSeed() - Removed
deriveSeedUUID()function and `seedIteratio…
Add deterministic file ordering in Builder.Build() (closes #23)
Removed the time-hard hash iteration per review:
- Removed
seedIterationsconstant (was 150M rounds) deriveSeedUUID()now does a single SHA-256 hash instead of iterating- Updated CLI flag…