Skip to content
cccskills
FreePublished on GitHub

adr-review

Multi-agent debate orchestration for Architecture Decision Records. Automatically triggers on ADR create/edit/delete. Coordinates architect, critic, independent-thinker, security, analyst, and high-level-advisor agents in structured debate rounds until consensus.

The English summary is being prepared. Showing the original description.

See how to install

Files included (2)

  • SKILL.md6.9 KB
  • source.json761 B

SKILL.md (original)

Review the instructions the agent will receive before you install.

ADR Review

Multi-agent debate pattern for rigorous ADR validation. Orchestrates 6 specialized agents through structured review rounds until consensus or 10 rounds maximum.

Quick Start

# Manual triggers:
/adr-review .agents/architecture/ADR-005-api-versioning.md
"review this ADR"
"validate ADR-005"

Automatic Detection: A Claude Code hook (Invoke-ADRChangeDetection.ps1) runs at session start and detects ADR changes, prompting you to invoke this skill. The pre-commit hook also detects staged ADR files and displays a reminder.

InputOutputConsensus Required
ADR file pathDebate log + Updated ADR6/6 Accept or D&C

File Triggers

PatternLocationEvents
ADR-*.md.agents/architecture/create, update, delete
ADR-*.mddocs/architecture/create, update, delete

Detection: .claude/skills/adr-review/scripts/Detect-ADRChanges.ps1

When to Use

MANDATORY Triggers (automatic):

  • Architect creates or updates an ADR
  • ANY agent modifies .agents/architecture/ADR-*.md

User-Initiated Triggers (manual):

  • User requests ADR review ("review this ADR", "validate this decision")
  • User requests multi-perspective validation for strategic decisions

Agent Roles

AgentFocusTie-Breaker Role
architectStructure, governance, coherence, ADR complianceStructural questions
criticGaps, risks, alignment, completenessNone
independent-thinkerChallenge assumptions, surface contrarian viewsNone
securityThreat models, security trade-offsNone
analystRoot cause, evidence, feasibilityNone
high-level-advisorPriority, resolve conflicts, break tiesDecision paralysis

Debate Protocol

PhasePurposeDetails
Phase 0Related work researchSearch issues/PRs for context
Phase 1Independent reviewEach agent reviews ADR
Phase 2ConsolidationIdentify consensus and conflicts
Phase 3ResolutionPropose updates for P0/P1 issues
Phase 4Convergence checkAgents vote: Accept/D&C/Block

Consensus: All 6 agents Accept OR Disagree-and-Commit. Max 10 rounds.

See references/debate-protocol.md for full phase details.

Deletion Workflow

PhasePurpose
D1Detection - identify deleted ADR
D2Impact assessment - find dependencies
D3Archival decision - archive accepted ADRs
D4Cleanup - update references

See references/deletion-workflow.md for full workflow.

Issue Resolution

PriorityRequirementGate
P0Must resolveBLOCKING
P1Resolve OR defer with issueBLOCKING
P2DocumentNon-blocking

See references/issue-resolution.md for deferral protocol.

Phase 4: Strategic Review (Principal-Level Validation)

After structural and technical review, apply strategic lenses:

Strategic Validation Checklist

Chesterton's Fence (Change Justification)

  • If removing/changing existing patterns: Original purpose documented
  • Investigation evidence provided (git archaeology, interviews, documentation)
  • Confirmation original problem no longer exists
  • Assessment: [PASS | FAIL | N/A]

Path Dependence (Irreversibility Recognition)

  • Historical constraints identified and documented
  • Reversibility assessment complete (rollback capability, vendor lock-in)
  • Migration/exit strategy defined if adding dependencies
  • Irreversible decisions explicitly flagged and justified
  • Assessment: [PASS | FAIL | N/A]

Core vs Context (Investment Prioritization)

  • Capability classified as Core (differentiating) or Context (commodity)
  • If building Context: Justification for not buying/outsourcing
  • If Core: Competitive differentiation explained
  • Assessment: [PASS | FAIL | N/A]

Second-System Effect (Over-Engineering Detection)

  • If replacing existing system: Scope boundaries explicit
  • Feature list justified (not "everything we didn't do last time")
  • Simplicity preservation strategy documented
  • Assessment: [PASS | FAIL | N/A]

Strategic Review Verdict

Overall Strategic Assessment: [APPROVED | CONCERNS | REJECTED]

Blocking Issues:

  • [Strategic issue 1 with required mitigation]
  • [Strategic issue 2 with required mitigation]

Recommendations:

  • [Strategic improvement 1]
  • [Strategic improvement 2]

Scripts

ScriptPurpose
Detect-ADRChanges.ps1Detect ADR file changes for auto-trigger
# Basic detection
& .claude/skills/adr-review/scripts/Detect-ADRChanges.ps1

# Compare to specific commit
& .claude/skills/adr-review/scripts/Detect-ADRChanges.ps1 -SinceCommit "abc123"

Verification Checklist

After skill invocation:

  • Debate log exists at .agents/critique/ADR-NNN-debate-log.md
  • ADR status updated (proposed/accepted/needs-revision)
  • All P0 issues addressed or documented
  • Dissent captured for Disagree-and-Commit positions
  • Recommendations provided to orchestrator

Anti-Patterns

AvoidWhyInstead
Single-agent ADR reviewMisses domain expertiseUse full 6-agent debate
Skipping Phase 0Duplicates existing workAlways research first
Ignoring D&C dissentLoses important contextDocument all reservations
Manual ADR monitoringError-proneUse Detect-ADRChanges.ps1
Deleting accepted ADRs without archiveLoses knowledgeAlways archive accepted ADRs

References

DocumentContent
debate-protocol.mdFull Phases 0-4 workflow
deletion-workflow.mdPhases D1-D4 workflow
issue-resolution.mdP0/P1/P2 handling and deferral
artifacts.mdOutput formats and templates
agent-prompts.mdDetailed agent prompt templates

Reviews

No reviews yet. Tell others what you thought.

Comments

No login needed. Share how it worked for you, ask a question, or note where it did not run.

No comments yet. Be the first to write one.

Comments are published right away. Do not include personal information such as email addresses or phone numbers. Up to 2 URLs.

0 / 1,000 characters

This form is protected by reCAPTCHA, and the Google Privacy Policy and Terms of Service apply.

More skills from this repository

007

Free

Security audit, hardening, threat modeling (STRIDE/PASTA), Red/Blue Team, OWASP checks, code review, incident response, and infrastructure security for any project.

The English summary is being prepared. Showing the original description.

aibot88/sec_skill_store5Updated May 27, 2026

Guides the creation of agile user stories and Gherkin feature files. Use when the user wants to create a user story, write acceptance criteria, define Gherkin scenarios, or author BDD feature files. This should trigger for requests such as Create a user story; Write a user story; I need to write a user story. Part of cursor-rules-java project

The English summary is being prepared. Showing the original description.

aibot88/sec_skill_store5Updated May 27, 2026

Facilitates conversational discovery to create Architectural Decision Records (ADRs) for non-functional requirements using the ISO/IEC 25010:2023 quality model. Use when the user wants to document quality attributes, NFR decisions, security/performance/scalability architecture, or design systems with measurable quality criteria. This should trigger for requests such as Create ADR for Non-functional requirements; Document Non-functional requirements; Capture Non-functional requirements; Generate Non-functional requirements in an ADR. Part of cursor-rules-java project

The English summary is being prepared. Showing the original description.

aibot88/sec_skill_store5Updated May 27, 2026

Run a health check on an existing project: dependency audit, security scan, test runner detection, CI/CD evaluation, and missing configuration analysis. Maps the three execution gates (pre/in/post) from /10x-bootstrapper to an assessment framework for existing codebases. Reads optional context/foundation/stack-assessment.md from /10x-stack-assess to focus checks on identified gaps. Writes context/foundation/health-check.md with findings, prioritized fixes, and an agent-readiness verdict. Use when the user has an existing project and wants to verify its health before working with an agent. Trigger phrases: "health check", "check my project", "audit my project", "is my project healthy", "sprawdź projekt", "audyt projektu", "health-check", "project health". Use AFTER /10x-stack-assess (brownfield chain), BEFORE agent onboarding (m1-l4).

The English summary is being prepared. Showing the original description.

aibot88/sec_skill_store5Updated May 27, 2026

You MUST use this when building projects end-to-end. Orchestrates all 12 team roles — automatically switches between CTO, architect, PM, engineers, SRE, security, DBA, QA, and EM based on the current phase of work. Starts with brainstorming before any implementation.

The English summary is being prepared. Showing the original description.

aibot88/sec_skill_store5Updated May 27, 2026

Use when you need to add or configure Maven plugins in your pom.xml — including quality tools (enforcer, surefire, failsafe, jacoco, pitest, spotbugs, pmd), security scanning (OWASP), code formatting (Spotless), version management, container image build (Jib), build information tracking, and benchmarking (JMH) — through a consultative, modular step-by-step approach that only adds what you actually need. This should trigger for requests such as Add Maven plugins in pom.xml; Improve Maven plugins in pom.xml. Part of cursor-rules-java project

The English summary is being prepared. Showing the original description.

aibot88/sec_skill_store5Updated May 27, 2026

See all skills by aibot88

Report a problem with this skill