Skip to content

Add review organization - #93

Open
THOClabs wants to merge 13 commits into
mainfrom
claude/install-review-org-37y4ck
Open

Add review organization#93
THOClabs wants to merge 13 commits into
mainfrom
claude/install-review-org-37y4ck

Conversation

@THOClabs

@THOClabs THOClabs commented Jul 12, 2026

Copy link
Copy Markdown
Owner

Summary

Installs a standing 7-level code-review organization into the repository: seven subagent definitions under .claude/agents/ and a /full-review command under .claude/commands/ that orchestrates them as a pipeline.

The seven agents:

  • repo-cartographer — L1 Recon. Maps the repository territory - file tree, languages, LOC, dependencies, build/test commands, entry points, config surface. Use as the first phase of any full-repo review.
  • git-historian — L2 Forensics. Analyzes git history for churn hotspots, bus factor, abandoned areas, commit conventions, and recent activity. Use after repo-cartographer in a full-repo review.
  • domain-analyst — L3 Deep dive. Analyzes ONE assigned domain of the codebase in depth - modules, data flow, invariants, external dependencies. Spawn one instance per domain, in parallel, during a full-repo review. The task prompt must name the assigned domain and its directories.
  • security-auditor — L4 Cross-cutting security audit. Sweeps the entire codebase for vulnerabilities after domain analysts finish, cross-checking their flagged concerns. Produces a severity-ranked findings report.
  • quality-auditor — L4 Cross-cutting quality audit. Assesses tests, CI, lint, error handling, and maintainability across all domains after the analysts finish. Runs the test suite when feasible.
  • chief-architect — L5-L6 Synthesis. Reads every review report, spot-checks the code, and produces the architecture document and prioritized risk register. Use after all analysts and auditors complete.
  • executive-scribe — L7 Board report. Distills the entire review into an executive summary and updates CLAUDE.md so every future session inherits the understanding. Use as the final phase of a full-repo review.

Running /full-review in Claude Code (or asking Claude to run .claude/commands/full-review.md) executes the pipeline: recon → git forensics → parallel per-domain deep dives → parallel security/quality audits → architecture synthesis and risk register → executive summary plus a generated CLAUDE.md section. Reports land under docs/review/; the organization is read-only toward source code.

Related Issues

None.

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Refactoring (no functional changes)
  • Test addition/update

(Tooling/configuration addition: Claude Code agent and command definitions only.)

Changes Made

  • Added .claude/agents/repo-cartographer.md, git-historian.md, domain-analyst.md, security-auditor.md, quality-auditor.md, chief-architect.md, executive-scribe.md
  • Added .claude/commands/full-review.md defining the phased review pipeline and its rules of engagement
  • No source code, tests, or other repository files were modified

Safety Checklist

Not applicable — no telescope/enclosure control or safety-system code is touched. The added review organization is explicitly read-only toward source code.

Testing

  • Unit tests pass (pytest tests/unit/)
  • Integration tests pass (pytest tests/integration/)
  • Manually tested with simulators
  • Tested with actual hardware (if applicable)

Not applicable — the change adds Markdown configuration for Claude Code agents; there is no runtime code to test.

Voice Commands

Not applicable — no voice commands added or modified.

Documentation

  • Code comments added/updated
  • README updated (if needed)
  • API documentation updated (if needed)
  • Configuration examples updated (if needed)

The added files are self-documenting agent/command definitions.

Screenshots / Logs

Not applicable.

Additional Notes

The task specified branch name chore/add-review-org; this session's push access is scoped to claude/install-review-org-37y4ck, so the work ships on that branch instead. File contents are otherwise installed verbatim as specified.

🤖 Generated with Claude Code

https://claude.ai/code/session_01FjTbUXge6kcW3TZMfAdz2J


Generated by Claude Code


Note

Low Risk
Documentation and Claude tooling only; no changes to observatory control, safety, or deploy/runtime paths.

Overview
Adds a standing Claude Code review organization: seven role definitions under .claude/agents/ (L1 recon through L7 executive scribe) plus .claude/commands/full-review.md, which runs a phased, read-only pipeline—inventory → git forensics → parallel domain deep dives → security/quality audits → architecture/risk synthesis → executive summary and a generated CLAUDE.md section. Writes are limited to docs/review/, .claude/agent-memory/, and the delimited CLAUDE.md block.

This PR also ships the first full review run on the branch: docs/review/00-inventory.md through multiple 20-domain-*.md reports, durable notes in .claude/agent-memory/ (e.g. chief-architect, voice/NLP analyst), and a generated repository review summary in CLAUDE.md (verified commands, danger zones, conventions). No application source under nightwatch/, services/, or voice/ is modified.

Reviewed by Cursor Bugbot for commit cc61aa2. Bugbot is set up for automated code reviews on this repo. Configure here.

@cursor

cursor Bot commented Jul 12, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_00413fe2-37dc-404f-b11f-2d7c6b24e6e8)

@cursor

cursor Bot commented Jul 12, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_0aeda91f-607b-46f4-96e1-9ea47a0ce5ba)

@cursor

cursor Bot commented Jul 12, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_2f764230-ed80-4a44-a516-e5a70680775d)

@cursor

cursor Bot commented Jul 12, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_06aef9ea-7a83-4506-b73c-cf13c54ea4a7)

@cursor

cursor Bot commented Jul 12, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_94276eb3-c944-4b74-8ade-58041a785a96)

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01FjTbUXge6kcW3TZMfAdz2J
@cursor

cursor Bot commented Jul 12, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_d305b2de-aaf0-4c07-ac12-bbbbf74c5cb5)

@cursor

cursor Bot commented Jul 12, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_ebfb5f03-b3f3-43c5-9a6d-c1c939818c3f)

@cursor

cursor Bot commented Jul 12, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_129b5718-7e4d-418e-a14f-f610651b76c6)

@cursor

cursor Bot commented Jul 12, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_355852eb-0cc7-47dd-b642-fc175f23444f)

@cursor

cursor Bot commented Jul 12, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_c9536553-b8e7-4dce-8ca0-f25da4b145a0)

@cursor

cursor Bot commented Jul 12, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_717717c7-4024-448f-b9f6-fdaf569ebb35)

claude added 2 commits July 12, 2026 23:08
…chitect memory

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01FjTbUXge6kcW3TZMfAdz2J
@cursor

cursor Bot commented Jul 12, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_a7f319b5-7e0d-4176-86a1-75bae93d4610)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants