fix: close release validation gaps - #110
Conversation
|
Important Review skippedNo new commits to review since the last review. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
📝 WalkthroughWalkthroughThis PR documents release-gate closure, updates project-status indexing, hardens credential-shaped test fixtures, and changes MCP flock smoke to use the orchestrator ID returned by the spawned roster. ChangesRelease gate closure
Estimated code review effort: 3 (Moderate) | ~25 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches 💡 1📝 Generate docstrings 💡
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
@coderabbitai review |
|
There was a problem hiding this comment.
Actionable comments posted: 4
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docs/analysis/12-anvil-project-process-status-review-2026-08-13.md`:
- Around line 55-57: 복구된 lifecycle contract 문서와 일치하도록 참조 경로를 수정하세요. 먼저
docs/governance/codex-lifecycle-control-plane.md 문서와 디렉터리를 복구하고, 문서의 공정 단계 설명에서
codex-lifecycle-control-plane.md 링크를
../governance/codex-lifecycle-control-plane.md로 변경해 깨진 상대 링크를 제거하세요.
In `@docs/superpowers/plans/2026-08-13-release-gate-closure.md`:
- Around line 181-186: Update the “완료 조건” section to require evidence for
acceptance criteria 1–11, including criterion 11 covering the roster-author
post/history/delete MCP authorship path. Preserve the existing requirements for
reviewing diffs, classifying CI/KVM results, and avoiding tag or release
creation.
- Around line 127-135: Update Task 3 in
docs/superpowers/plans/2026-08-13-release-gate-closure.md (lines 127-135) to
list the three named Go builds: go build -o anvil-daemon ./cmd/goose-daemon/, go
build ./cmd/anvil-mcp, and go build ./cmd/anvil-scheduler; retain the aggregate
checks, but do not treat go build ./... or syntax checks as substitutes. Update
docs/operations/2026-08-13-release-gate-closure-handoff.md (lines 52-60) to
record each named build result and the sudo bash e2e_test.sh result.
- Around line 149-162: The Task 5 plan at
docs/superpowers/plans/2026-08-13-release-gate-closure.md:149-162 must add a
forced VM-deletion-failure probe, verify the full cleanup sequence including
TAP/IP resources, bind mounts, loop devices, dm-snapshots, and sparse .cow
exception-store files, and account for TeardownDMSnapshot continuing cleanup
after dmsetup remove --retry fails; update
docs/operations/2026-08-13-release-gate-closure-handoff.md:131-136 to record the
result of every resource check, and keep the release gate open whenever any
resource remains.
🪄 Autofix
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro Plus
Run ID: b33d0675-1fee-4264-9caa-fce140bc1c35
📒 Files selected for processing (8)
cmd/goose-daemon/config_api_anvil_test.godocs/analysis/12-anvil-project-process-status-review-2026-08-13.mddocs/analysis/README.mddocs/operations/2026-08-13-release-gate-closure-handoff.mddocs/superpowers/grill-me/2026-08-13-release-gate-closure.mddocs/superpowers/plans/2026-08-13-release-gate-closure.mddocs/superpowers/specs/2026-08-13-release-gate-closure-design.mdscripts/anvil-mcp-smoke.go
변경 내용
orchestrator대신 spawn response의 실제 roster member ID를 사용하도록 교정했습니다.v0.7.0baseline으로 갱신했습니다.원인
Secret scanner는 의도된 credential-shaped test fixture와 실제 secret을 구분할 수 없었고, flock smoke는 roster authorship guard 도입 전의 author ID를 계속 사용하고 있었습니다. Scanner나 daemon guard를 완화하지 않고 fixture/caller만 수정했습니다.
영향
Runtime API, MCP schema, daemon security policy에는 변화가 없습니다. 이 PR은 release gate와 검증 코드·증적을 정합화합니다. 정식 tag/release publish는 포함하지 않습니다.
검증
go test ./... -count=1go test -race ./... -count=1go build ./...go vet ./...go mod verifygovulncheck ./...— reachable vulnerability 0bash scripts/secret-scan.sh— tracked tree PASSnpm run check— 0 errors, existing warnings 10npm run buildsudo ... bash e2e_test.sh— All test steps passedgit diff --check— 통과남은 release blocker
allow_hostsremoval contract 이행상세 증적은
docs/operations/2026-08-13-release-gate-closure-handoff.md에 있습니다.Summary by CodeRabbit
Bug Fixes
Documentation