Skip to content

chore: retire validate_project / warden_audit / pre_review_scan trio (audit Phase F)#122

Merged
manager-identity[bot] merged 1 commit into
mainfrom
chore/phase-f-retire-trio
Apr 28, 2026
Merged

chore: retire validate_project / warden_audit / pre_review_scan trio (audit Phase F)#122
manager-identity[bot] merged 1 commit into
mainfrom
chore/phase-f-retire-trio

Conversation

@manager-identity
Copy link
Copy Markdown
Contributor

Summary

  • Deletes the trio (scripts/validate_project.py, scripts/warden_audit.py, scripts/pre_review_scan.sh) per Phase E.11/E.12 portfolio-wide decision.
  • Updates 4 doc files that referenced the deleted scripts: REVIEWS_AND_GOVERNANCE_PROTOCOL.md (root + .agent/rules/ copy), SCAFFOLDING_TRANSFER_GUIDE.md, MISSING_TESTS.md.
  • scripts/utils/validation.py::validate_project_exists() is unrelated (different name) — kept.

Test plan

  • pytest tests/test_discovery.py tests/test_api_portfolio.py — 4 pass.
  • git grep validate_project|warden_audit|pre_review_scan — only intentional retirement notes remain.

Related

  • Audit doc: $HOME/AGENT_MANAGEMENT_AUDIT.md Phase F
  • Decision artifacts: agent-runtime-config/.cleanup-log/E.{11,12}-*.md

…(audit Phase F)

Per Phase E.11 + E.12 decisions, the validate_project.py + warden_audit.py +
pre_review_scan.sh trio is being deleted across the portfolio. This is
project-tracker's per-project execution.

Deleted:
- scripts/validate_project.py
- scripts/warden_audit.py
- scripts/pre_review_scan.sh

Doc references updated:
- REVIEWS_AND_GOVERNANCE_PROTOCOL.md (root + .agent/rules/ copy):
  - Layer 1 description rewired to point at user-scope hooks + code-reviewer
  - Gate 2 placeholder check rewired to a `grep` invocation
  - M4 row evidence column rewired to grep output
  - Task 1 marked retired
- .agent/rules/SCAFFOLDING_TRANSFER_GUIDE.md: drop WARDEN_AUDIT bullet
- tests/MISSING_TESTS.md: collapse the two TODO blocks into one retirement note

`scripts/utils/validation.py` keeps its `validate_project_exists()`
function — different name, unrelated to the deleted file.

Tests: pytest tests/test_discovery.py tests/test_api_portfolio.py — pass.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@manager-identity manager-identity Bot added the chore Maintenance and cleanup label Apr 28, 2026
@manager-identity manager-identity Bot merged commit b44fd2d into main Apr 28, 2026
2 checks passed
@manager-identity manager-identity Bot deleted the chore/phase-f-retire-trio branch April 28, 2026 05:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Maintenance and cleanup

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants