dev-playbooks 1.7.2 → 1.7.3

package/README.md

@@ -164,7 +164,7 @@ Run devbooks-coder skill for change add-oauth2
 **3) Review stage**
 
 ```
-Run devbooks-code-review skill for change add-oauth2
+Run devbooks-reviewer skill for change add-oauth2
 ```
 
 **4) Archive stage**
@@ -202,7 +202,7 @@ Run devbooks-archiver skill for change add-oauth2
 
 | Skill | Description |
 |-------|-------------|
-| devbooks-code-review | Code review (readability/consistency) |
+| devbooks-reviewer | Code review (readability/consistency) |
 | devbooks-test-reviewer | Test quality and coverage review |
 
 ### Archive stage

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dev-playbooks",
-  "version": "1.7.2",
+  "version": "1.7.3",
   "description": "AI-powered spec-driven development workflow",
   "keywords": [
     "devbooks",

package/skills/Skill-Development-Guide.md

@@ -18,7 +18,7 @@ This document defines the design principles and constraints you must follow when
 
 | Skill type | Idempotency requirement | Examples |
 |---|---|---|
-| **Verification / checks** | Must be idempotent (must not modify files) | `change-check.sh`, `guardrail-check.sh`, `devbooks-code-review` |
+| **Verification / checks** | Must be idempotent (must not modify files) | `change-check.sh`, `guardrail-check.sh`, `devbooks-reviewer` |
 | **Generators** | Must explicitly define “overwrite vs incremental” behavior | `change-scaffold.sh`, `devbooks-design-doc`, `devbooks-proposal-author` |
 | **Editors** | Must be safe to rerun | `devbooks-spec-gardener`, `devbooks-design-backport` |
 

package/skills/Skills-Usage-Guide.md

@@ -220,7 +220,7 @@ In SKILL.md prompts, the `~/.claude/skills/_shared/` path should be replaced wit
 
 ---
 
-## `devbooks-code-review` (Reviewer)
+## `devbooks-reviewer` (Reviewer)
 
 - Purpose: review readability/consistency/dependency health/code smells; output actionable suggestions only; do not judge business correctness.
 - Hotspot-first review: call `mcp__ckb__getHotspots` before reviewing; prioritize by risk:
@@ -232,7 +232,7 @@ In SKILL.md prompts, the `~/.claude/skills/_shared/` path should be replaced wit
   - You want to discover coupling/dependency direction/complexity/smells
 - Copy-paste prompt:
   ```text
-  You are Reviewer. Explicitly use `devbooks-code-review`.
+  You are Reviewer. Explicitly use `devbooks-reviewer`.
   Review only readability/consistency/dependency health/code smells; do not discuss business correctness; do not modify `tests/` or design.
   Inputs: code changes + `dev-playbooks/specs/**` (as needed for profile/glossary/anti-patterns).
   Output: severe issues / maintainability risks / consistency suggestions / suggested quality gates to add.

package/skills/_shared/mcp-enhancement-template-mcp-enhancement.md

@@ -84,7 +84,7 @@ The following Skills depend on MCP and require the full MCP Enhancement section:
 | Skill | MCP Dependencies | Enhanced Features |
 |-------|------------------|-------------------|
 | devbooks-coder | mcp__ckb__getHotspots | Hotspot warnings |
-| devbooks-code-review | mcp__ckb__getHotspots | Hotspot highlighting |
+| devbooks-reviewer | mcp__ckb__getHotspots | Hotspot highlighting |
 | devbooks-impact-analysis | mcp__ckb__analyzeImpact, findReferences | Precise impact analysis |
 | devbooks-brownfield-bootstrap | mcp__ckb__* | COD model generation |
 | devbooks-router | mcp__ckb__getStatus | Index availability detection |

package/skills/_shared/references/deviation-detection-routing-protocol.md

@@ -124,7 +124,7 @@ deviation-log.md is a **persistent file** unaffected by compact. Even if convers
 | Current Skill | Completion Status | Next Step |
 |---------------|-------------------|-----------|
 | Any | COMPLETED_WITH_DEVIATION | `devbooks-design-backport` |
-| coder | COMPLETED | `devbooks-code-review` |
+| coder | COMPLETED | `devbooks-reviewer` |
 | coder | HANDOFF (test issue) | `devbooks-test-owner` |
 | test-owner | COMPLETED | `devbooks-coder` |
 | test-owner | HANDOFF (design issue) | `devbooks-design-backport` |

package/skills/_shared/workflow-next-steps-workflow-next-steps.md

@@ -44,9 +44,9 @@ This document defines the canonical next step recommendations for each skill. Al
 | `devbooks-spec-contract` | `devbooks-implementation-plan` | Always (NEVER recommend test-owner/coder directly) |
 | `devbooks-implementation-plan` | `devbooks-test-owner` | Always (must be separate chat) |
 | `devbooks-test-owner` | `devbooks-coder` | After Red baseline (must be separate chat) |
-| `devbooks-coder` | `devbooks-code-review` | After all tasks completed |
-| `devbooks-code-review` | `devbooks-archiver` | If spec deltas exist |
-| `devbooks-code-review` | Archive complete | If no spec deltas |
+| `devbooks-coder` | `devbooks-reviewer` | After all tasks completed |
+| `devbooks-reviewer` | `devbooks-archiver` | If spec deltas exist |
+| `devbooks-reviewer` | Archive complete | If no spec deltas |
 | `devbooks-test-reviewer` | `devbooks-coder` | If test issues found, hand back |
 | `devbooks-design-backport` | `devbooks-implementation-plan` | Rerun plan after design update |
 | `devbooks-archiver` | Archive complete | Always |

package/skills/devbooks-delivery-workflow/SKILL.md

@@ -163,7 +163,7 @@ Before execution, **must** search for configuration in the following order (stop
 | 7 | Test-Red | `devbooks-test-owner` | devbooks-test-owner | verification.md + tests/ | ✅ |
 | 8 | Code | `devbooks-coder` | devbooks-coder | src/ implementation | ✅ |
 | 9 | Test-Review | `devbooks-reviewer` | devbooks-test-reviewer | Test review opinions | ✅ |
-| 10 | Code-Review | `devbooks-reviewer` | devbooks-code-review | Code review opinions | ✅ |
+| 10 | Code-Review | `devbooks-reviewer` | devbooks-reviewer | Code review opinions | ✅ |
 | 11 | Green-Verify | `devbooks-test-owner` | devbooks-test-owner | evidence/green-final/ | ✅ |
 | 12 | Archive | `devbooks-archiver` | devbooks-archiver | Archived to archive/ | ✅ |
 

package/skills/devbooks-delivery-workflow/references/delivery-acceptance-workflow.md

@@ -156,7 +156,7 @@ Checklist:
 - Run all A-class anchors in scope (tests + static checks + build)
 - Prefer machine-readable outputs (json/xml) for traceability and tooling
 - Execute B/C anchors with evidence
-- Optional review: use `devbooks-code-review` for maintainability feedback
+- Optional review: use `devbooks-reviewer` for maintainability feedback
 
 ### Step 7: Close-out
 

package/skills/devbooks-delivery-workflow/references/subagent-invocation-spec.md

@@ -27,7 +27,7 @@ Please execute the following task:
 | 7 | devbooks-test-owner | "Use devbooks-test-owner skill to write tests and establish Red baseline for change [change-id]" |
 | 8 | devbooks-coder | "Use devbooks-coder skill to implement functionality for change [change-id]" |
 | 9 | devbooks-reviewer | "Use devbooks-test-reviewer skill to review tests for change [change-id]" |
-| 10 | devbooks-reviewer | "Use devbooks-code-review skill to review code for change [change-id]" |
+| 10 | devbooks-reviewer | "Use devbooks-reviewer skill to review code for change [change-id]" |
 | 11 | devbooks-test-owner | "Use devbooks-test-owner skill to run all tests and collect Green evidence for change [change-id]" |
 | 12 | devbooks-archiver | "Use devbooks-archiver skill to archive change [change-id]" |
 

package/skills/devbooks-entropy-monitor/SKILL.md

@@ -105,7 +105,7 @@ Goals of entropy measurement:
 
 | Skill | Relationship |
 |-------|--------------|
-| devbooks-code-review | Entropy measurement is **not** embedded in every review, runs as independent task |
+| devbooks-reviewer | Entropy measurement is **not** embedded in every review, runs as independent task |
 | devbooks-proposal-author | Entropy reports can serve as data support for refactoring proposals |
 | devbooks-impact-analysis | Changes in high-entropy areas require more careful impact analysis |
 

package/skills/{devbooks-code-review → devbooks-reviewer}/SKILL.md

@@ -1,6 +1,6 @@
 ---
-name: devbooks-code-review
-description: devbooks-code-review: Perform readability/consistency/dependency health/code smell reviews as a Reviewer role, outputting only review comments and actionable suggestions without discussing business correctness. Use when user says "help me do code review/review maintainability/code smells/dependency risks/consistency suggestions", or when executing as reviewer during DevBooks apply phase.
+name: devbooks-reviewer
+description: devbooks-reviewer: Perform readability/consistency/dependency health/code smell reviews as a Reviewer role, outputting only review comments and actionable suggestions without discussing business correctness. Use when user says "help me do code review/review maintainability/code smells/dependency risks/consistency suggestions", or when executing as reviewer during DevBooks apply phase.
 allowed-tools:
   - Glob
   - Grep

package/skills/{devbooks-code-review → devbooks-reviewer}/references/code-smell-cheatsheet.md

@@ -402,7 +402,7 @@ class PaymentService:
 - *Refactoring: Improving the Design of Existing Code* (2nd Edition) — Martin Fowler
 - *Clean Architecture* — Robert C. Martin (Chapter: Component Coupling)
 - Dev Playbooks debate-revised evaluation notes
-- `devbooks-code-review` checklists
+- `devbooks-reviewer` checklists
 
 ---
 

package/skills/devbooks-router/SKILL.md

@@ -68,7 +68,7 @@ fi
 
 **Degraded Mode Notes**:
 - Without index, `devbooks-impact-analysis` degrades to Grep text search (reduced accuracy)
-- Without index, `devbooks-code-review` cannot obtain call graph context
+- Without index, `devbooks-reviewer` cannot obtain call graph context
 - Recommend completing index generation before the Apply phase
 
 ## Your Task
@@ -228,7 +228,7 @@ LSC (Large-Scale Codemods) recommendations:
 Trigger signals: User says "review/code smells/maintainability/dependency risks/consistency" etc.
 
 Default routing:
-- `devbooks-code-review` (output actionable suggestions; do not change business conclusions or tests)
+- `devbooks-reviewer` (output actionable suggestions; do not change business conclusions or tests)
 - `devbooks-test-reviewer` (review test quality, coverage, edge cases)
 
 ### D) Docs Sync (Documentation Sync)

package/skills/devbooks-test-owner/SKILL.md

@@ -296,7 +296,7 @@ Detection rules reference: `~/.claude/skills/_shared/context-detection-template.
 
 | Code | Status | Determination Criteria | Next Step |
 |:----:|--------|------------------------|-----------|
-| ✅ | PHASE2_VERIFIED | Evidence audit passed, AC matrix checked | `devbooks-code-review` |
+| ✅ | PHASE2_VERIFIED | Evidence audit passed, AC matrix checked | `devbooks-reviewer` |
 | ⏳ | PHASE2_WAITING | @full tests still running | Wait for CI to complete |
 | ❌ | PHASE2_FAILED | @full tests not passing | Notify Coder to fix |
 | 🔄 | PHASE2_HANDOFF | Found issues with tests themselves | Fix tests then re-verify |

package/skills/devbooks-test-owner/references/phase2-evidence-audit-checklist.md

@@ -101,7 +101,7 @@ Update status at top of verification.md:
 
 | Code | Status | Determination Criteria | Next Step |
 |:----:|--------|------------------------|-----------|
-| ✅ | PHASE2_VERIFIED | Evidence audit passed, AC matrix checked | `devbooks-code-review` |
+| ✅ | PHASE2_VERIFIED | Evidence audit passed, AC matrix checked | `devbooks-reviewer` |
 | ⏳ | PHASE2_WAITING | @full tests still running | Wait for CI to complete |
 | ❌ | PHASE2_FAILED | @full tests not passing | Notify Coder to fix |
 | 🔄 | PHASE2_HANDOFF | Found issues with tests themselves | Fix tests then re-verify |

package/templates/claude-agents/devbooks-reviewer.md

@@ -1,7 +1,7 @@
 ---
 name: devbooks-reviewer
 description: DevBooks Reviewer subagent: Executes code review and test review, outputs actionable suggestions.
-skills: devbooks-code-review, devbooks-test-reviewer
+skills: devbooks-reviewer, devbooks-test-reviewer
 ---
 
 # DevBooks Reviewer Subagent

package/templates/dev-playbooks/README.md

@@ -140,7 +140,7 @@ Run devbooks-coder skill for change add-oauth2
 **3) Review stage**
 
 ```
-Run devbooks-code-review skill for change add-oauth2
+Run devbooks-reviewer skill for change add-oauth2
 ```
 
 **4) Archive stage**
@@ -178,7 +178,7 @@ Run devbooks-archiver skill for change add-oauth2
 
 | Skill | Description |
 |-------|-------------|
-| devbooks-code-review | Code review (readability/consistency) |
+| devbooks-reviewer | Code review (readability/consistency) |
 | devbooks-test-reviewer | Test quality and coverage review |
 
 ### Archive stage

package/templates/dev-playbooks/docs/devbooks-setup-guide.md

@@ -131,7 +131,7 @@ Each change must declare which gates are covered; missing items require reasons:
 | Planner | `devbooks-implementation-plan` | `tasks.md` |
 | Test Owner | `devbooks-test-owner` | `verification.md` + `tests/` |
 | Coder | `devbooks-coder` | Implement per tasks (no tests) |
-| Reviewer | `devbooks-code-review` | Review feedback |
+| Reviewer | `devbooks-reviewer` | Review feedback |
 | Archiver | `devbooks-archiver` | Archive pruning + C4 merge |
 | Design Backport | `devbooks-design-backport` | Backport design gaps |
 
@@ -185,6 +185,6 @@ The AI can automatically select Skills based on user intent:
 | User Intent | Auto Route |
 |-------------|------------|
 | "Fix bug", "find issue" | `devbooks-impact-analysis` → `devbooks-coder` |
-| "Refactor", "optimize code" | `devbooks-code-review` → `devbooks-coder` |
+| "Refactor", "optimize code" | `devbooks-reviewer` → `devbooks-coder` |
 | "New feature", "implement XX" | `devbooks-router` → full loop |
 | "Write tests", "add tests" | `devbooks-test-owner` |