Brutal Project Review
Review the project one subsystem at a time and persist actionable CRITICAL/MAJOR
findings through the configured work-store adapter.
Required Context
- Read
../brutal-shared/integration-resolver.md.
- Resolve the work store and load its support module before initializing state
or creating findings.
- Read repo rules from
AGENTS.md, CLAUDE.md, TARGET.md, and referenced
workflow docs.
- If
BRUTAL.md is missing or incomplete, follow the integration setup flow.
State Rules
- The local adapter uses its configured review-state path and local tasks.
- Remote adapters use
.brutal-workspace/review-state/ only for resumable local
review state. Exclude .brutal-workspace/ through .git/info/exclude before
writing it and never create local work-store artifacts.
- Preserve legacy manifest fields when resuming older runs, but write new
entries with adapter-neutral keys where practical:
work_store, work_store_project, parent_ref, findings_created.
Workflow
- Resolve or initialize the review manifest.
- Discover subsystems dynamically from behavior-defining files. Exclude
generated/vendor/build folders. Normalize ids as
01, 02, etc.
- Show the subsystem table and ask what to review: id/name/path,
all,
pending, restart, or cancel.
- For each selected subsystem:
- mark it
in_progress
- gather relevant code, tests, migrations, APIs, configs, SQL, TOML, and docs
- write temporary context under the resolved review-state directory
- launch five reviewers when available: core correctness, reliability/tests,
maintainability, performance/security, and simplification
- synthesize, dedupe, validate, and severity-check findings
- write the subsystem report
- delete temporary context before stopping or reporting
- Only create persisted work for validated
CRITICAL and MAJOR findings by
default.
Finding Persistence
Use deterministic fingerprints:
<subsystem-id>|<canonical-file>|<line-or-symbol>|<severity>|<normalized-title>
Before creating a finding, search existing state and work-store artifacts for the
same fingerprint and these legacy source markers:
Source: brutal-project-review
Legacy sources: brutal-project-review, linear-brutal-project-review, gitlear-brutal-project-review
Create/update type:review-finding artifacts through the work-store adapter in
its logical todo state, add progress to the parent when one exists, and append
stable refs to the review manifest. Follow provider-specific read fallbacks and
prepare-before-create rules from the support module.
Finding bodies must include severity, subsystem, file/line, fingerprint,
confidence, issue explanation, suggested fix, acceptance criteria, implementation
notes, dependencies, and verification commands.
Final Response
Lead with findings by severity. Include work store, created/updated finding refs,
manifest path, parent ref, reviewed subsystem count, remaining subsystem count,
and recommend $brutal-swarm for parallel fixes or $brutal-worker for one
exact finding.
1---2name: brutal-project-review3description: Systematically review a project subsystem-by-subsystem with resumable state tracking. Creates CRITICAL/MAJOR finding tasks through the work store resolved from BRUTAL.md.4---56# Brutal Project Review78Review the project one subsystem at a time and persist actionable CRITICAL/MAJOR9findings through the configured work-store adapter.1011## Required Context12131. Read `../brutal-shared/integration-resolver.md`.142. Resolve the work store and load its support module before initializing state15 or creating findings.163. Read repo rules from `AGENTS.md`, `CLAUDE.md`, `TARGET.md`, and referenced17 workflow docs.184. If `BRUTAL.md` is missing or incomplete, follow the integration setup flow.1920## State Rules2122- The local adapter uses its configured review-state path and local tasks.23- Remote adapters use `.brutal-workspace/review-state/` only for resumable local24 review state. Exclude `.brutal-workspace/` through `.git/info/exclude` before25 writing it and never create local work-store artifacts.26- Preserve legacy manifest fields when resuming older runs, but write new27 entries with adapter-neutral keys where practical:28 `work_store`, `work_store_project`, `parent_ref`, `findings_created`.2930## Workflow31321. Resolve or initialize the review manifest.332. Discover subsystems dynamically from behavior-defining files. Exclude34 generated/vendor/build folders. Normalize ids as `01`, `02`, etc.353. Show the subsystem table and ask what to review: id/name/path, `all`,36 `pending`, `restart`, or `cancel`.374. For each selected subsystem:38 - mark it `in_progress`39 - gather relevant code, tests, migrations, APIs, configs, SQL, TOML, and docs40 - write temporary context under the resolved review-state directory41 - launch five reviewers when available: core correctness, reliability/tests,42 maintainability, performance/security, and simplification43 - synthesize, dedupe, validate, and severity-check findings44 - write the subsystem report45 - delete temporary context before stopping or reporting465. Only create persisted work for validated `CRITICAL` and `MAJOR` findings by47 default.4849## Finding Persistence5051Use deterministic fingerprints:5253```text54<subsystem-id>|<canonical-file>|<line-or-symbol>|<severity>|<normalized-title>55```5657Before creating a finding, search existing state and work-store artifacts for the58same fingerprint and these legacy source markers:5960```markdown61Source: brutal-project-review62Legacy sources: brutal-project-review, linear-brutal-project-review, gitlear-brutal-project-review63```6465Create/update `type:review-finding` artifacts through the work-store adapter in66its logical `todo` state, add progress to the parent when one exists, and append67stable refs to the review manifest. Follow provider-specific read fallbacks and68prepare-before-create rules from the support module.6970Finding bodies must include severity, subsystem, file/line, fingerprint,71confidence, issue explanation, suggested fix, acceptance criteria, implementation72notes, dependencies, and verification commands.7374## Final Response7576Lead with findings by severity. Include work store, created/updated finding refs,77manifest path, parent ref, reviewed subsystem count, remaining subsystem count,78and recommend `$brutal-swarm` for parallel fixes or `$brutal-worker` for one79exact finding.