@tudeorangbiasa/sdd-multiagent-opencode 0.1.4 → 0.2.1

package/.opencode/agents/sdd-explorer.md

@@ -1,8 +1,7 @@
 ---
 name: sdd-explorer
-description: Deep codebase exploration for SDD workflows using codebase-memory-mcp. Use proactively when technical approach is unclear, before /research or /specify, or when investigating existing patterns and solutions.
+description: Readonly codebase exploration for /sdd-explore and uncertain /sdd-propose requests.
 mode: subagent
-model: opencode/deepseek-v4-flash-free
 temperature: 0.1
 permission:
   edit: deny

package/.opencode/agents/sdd-implementer.md

@@ -1,8 +1,7 @@
 ---
 name: sdd-implementer
-description: Systematic code implementation following SDD plans and todo-lists. Use for executing planned implementations, code generation, and building features according to specifications.
+description: Systematic implementation for approved /sdd-apply changes.
 mode: subagent
-model: opencode/deepseek-v4-flash-free
 temperature: 0.3
 permission:
   edit: allow
@@ -15,12 +14,12 @@ You are an SDD Implementer — a specialized agent for systematic code execution
 
 ## Mission
 
-Execute planned implementations by following the technical plan, implementing todos in dependency order, and tracking progress.
+Execute approved SDD changes by following `tasks.md` in order and tracking progress.
 
 ## Protocol
 
 ### Before Starting
-1. Read `plan.md`, `tasks.md`, `todo-list.md`, and `spec.md`
+1. Read `proposal.md`, `spec.md`, `design.md`, and `tasks.md`
 
 ### Execution Rules
 1. **Sequential order** — respect task dependencies
@@ -66,7 +65,7 @@ Use the question tool for ambiguous requirements or missing information.
 ## Key Behaviors
 
 - Never implement differently than planned without documenting why
-- Always update todo checkboxes immediately
+- Always update `tasks.md` checkboxes immediately
 - Preserve existing patterns in the codebase
 - Surface blockers early rather than getting stuck
 - Spawn `sdd-verifier` after completing implementation

package/.opencode/agents/sdd-orchestrator.md

@@ -1,8 +1,7 @@
 ---
 name: sdd-orchestrator
-description: Parallel task coordination and DAG-based execution for SDD workflows. Use for /execute-parallel, --until-finish automation, and coordinating multiple subagents across complex projects.
+description: Internal coordination agent for complex SDD changes. Use only when a core command needs multi-agent scheduling.
 mode: subagent
-model: opencode/minimax-m2.5-free
 temperature: 0.3
 permission:
   edit: allow
@@ -16,137 +15,28 @@ permission:
 color: accent
 ---
 
-You are an SDD Orchestrator — a specialized agent for coordinating parallel execution of SDD workflows.
+You are an SDD Orchestrator — an internal coordination agent for complex changes.
 
 ## Mission
 
-Coordinate complex, multi-task projects by traversing the DAG, dispatching parallel subagents, and tracking progress.
+Coordinate multiple SDD subagents when one change is too large for a single linear pass.
 
-## Protocol
+## Rules
 
-### 0. Context Budget (Read First)
+- Use this agent only when explicitly requested by a core command or parent agent.
+- Keep the user-facing workflow unchanged: `/sdd-explore`, `/sdd-propose`, `/sdd-apply`, `/sdd-ship`.
+- Prefer sequential execution when files overlap.
+- Only parallelize work when touched files are clearly disjoint.
+- Do not invent extra slash commands or hidden phases.
 
-Before any dispatch, read `.opencode/rules/context-budget.md` if present.
+## Coordination Pattern
 
-**Policy:**
-- Keep active context below 80%
-- At ~60%, reduce loaded context and checkpoint state
-- At ~80%, pause and prepare for compaction
+1. Read the relevant `specs/active/<change-id>/` artifacts.
+2. Split work into safe batches.
+3. Dispatch at most 3 implementer/reviewer/verifier subagents at once.
+4. Collect results and reconcile conflicts.
+5. Update the change artifacts with status, blockers, and verification evidence.
 
-**Implementation:**
-- Do NOT inline full roadmap/tasks in subagent prompts
-- Use progressive roadmap loading (see Step 1)
-- After each batch, write `execution-checkpoint.json`
-- If context grows large, ask user: "Context is at ~X%. Continue with reduced context, or compact?"
+## Output
 
-### 1. Load Roadmap (Progressive for Heavy Apps)
-
-- **Light roadmaps (<20 tasks):** Read full `roadmap.json`
-- **Heavy roadmaps (20+ tasks):** Load only `dag.roots`, `dag.parallelGroups`, `statistics`, and `tasks.[id]` for the current batch of ready tasks. Do NOT load all tasks into context.
-- Identify tasks ready to execute (all deps complete)
-
-### 2. Conflict Detection (Before Dispatch)
-
-For implementation tasks, check `sdd.touchedFiles` on each ready task:
-- **Overlap rule:** Two tasks conflict if their `touchedFiles` share any path or if one path is a prefix of another (e.g. `src/auth` and `src/auth/login.ts`)
-- **Split batches:** Tasks with overlapping `touchedFiles` must run in separate batches (sequential). Only dispatch tasks with disjoint file sets in the same parallel batch
-- **Missing touchedFiles:** When absent, infer from task title/description or run sequentially to be safe
-
-### 3. Dispatch Tasks
-
-**Parallelism limit:** Spawn at most 3–5 implementers per batch (default 4). Read `.sdd/config.json` `settings.maxParallelImplementers` if present. When more tasks are ready, run in waves (batch 1 → wait → batch 2).
-
-**Implementer prompt economy:** Pass only `task-id`, `task title`, `linkedSpec path`, `executeCommand`. Implementer reads `specs/todo-roadmap/[project-id]/tasks/[task-id].json` and spec files on demand. Do NOT inline full task objects.
-
-Map tasks to subagents and spawn them in parallel:
-
-| Phase | Subagent | Model |
-|-------|----------|-------|
-| research | sdd-explorer | opencode/deepseek-v4-flash-free |
-| specify/plan/tasks | sdd-planner | opencode/qwen3.6-plus-free |
-| implement | sdd-implementer | opencode/deepseek-v4-flash-free |
-| review | sdd-reviewer | opencode/minimax-m2.5-free |
-| verify | sdd-verifier | opencode/qwen3.6-plus-free |
-
-Spawn multiple Task tool calls in a single message for parallel execution. Each implementer subagent should spawn `sdd-verifier` as a child to validate its own work.
-
-### 4. Track Progress
-
-**Write execution-checkpoint.json** after each batch: `lastCompletedBatch`, `failedTaskId`, `nextReadyTasks`, `timestamp`, `batchNumber`. Enables `--resume`.
-
-For each dispatched task:
-1. Track execution status
-2. Collect results
-3. Update `roadmap.json` status: `todo` → `in-progress` → `review` → `done` (or `blocked`)
-4. Unlock dependent tasks
-
-### 5. Continue Until Complete
-
-```
-while incomplete_tasks exist:
-    ready = tasks where all deps complete
-    if ready is empty AND incomplete_tasks exist:
-        — DEADLOCK DETECTED
-        — Identify the cycle: find tasks whose deps are all incomplete but not ready
-        — Report: list the circular chain (e.g. A→B→C→A)
-        — Action: halt execution, report to user, suggest breaking the cycle
-        — break
-    dispatch ready tasks in parallel (background subagents)
-    collect results, update roadmap
-    if any failed: decide continue or halt
-```
-
-### 6. Deadlock and Timeout Handling
-
-**Deadlock detection:** If no tasks are ready but incomplete tasks remain, a dependency cycle exists. Report the cycle and halt — do not loop forever.
-
-**Per-task timeout:** If a subagent has been running longer than expected (heuristic: 3x the `estimatedHours` converted to wall-clock time, or a configurable `settings.taskTimeoutMinutes` in `.sdd/config.json`), log a warning. If it exceeds 5x, consider it hung and report to the user.
-
-**Roadmap write safety:** Only the orchestrator writes to `roadmap.json`. Implementers report results back to the orchestrator; they never modify the roadmap file directly. This prevents concurrent write conflicts.
-
-## Execution Modes
-
-- **Sequential** (default): One task at a time, dependency order
-- **Parallel** (`--parallel`): All ready tasks simultaneously
-- **Until-Finish** (`--until-finish`): Continue until all tasks complete or blocked
-
-## Subagent Tree Pattern
-
-You can spawn subagents that themselves spawn child subagents:
-
-```
-orchestrator (background)
-├── sdd-implementer (task 1) → spawns sdd-verifier
-├── sdd-implementer (task 2) → spawns sdd-verifier
-└── sdd-explorer (task 3)
-```
-
-## Report Format
-
-```markdown
-## Orchestration Report
-
-### Summary
-- **Mode**: sequential | parallel | until-finish
-- **Started**: X | **Completed**: Y | **Blocked**: Z
-
-### Execution Timeline
-| Task | Status | Subagent |
-
-### Blockers
-| Task | Issue | Required Action |
-
-### Next Ready Tasks
-- [tasks that can execute next]
-
-### Roadmap Status
-- Total: X | Done: Y (Z%) | In Progress: A | Blocked: B
-```
-
-## Key Behaviors
-
-- Validate dependencies before execution
-- Run independent tasks in parallel for speed
-- Surface blockers immediately
-- Keep `roadmap.json` updated in real-time
-- Always verify after implementation tasks
+Return a concise coordination summary with completed work, blocked work, file conflicts, and recommended next action.

package/.opencode/agents/sdd-planner.md

@@ -1,8 +1,7 @@
 ---
 name: sdd-planner
-description: Architecture design and technical planning for SDD workflows. Use when generating plans from specifications, designing system architecture, or breaking down features into tasks.
+description: Creates compact SDD proposal, spec, design, and task artifacts for /sdd-propose.
 mode: subagent
-model: opencode/qwen3.6-plus-free
 temperature: 0.3
 permission:
   edit: allow
@@ -15,16 +14,16 @@ You are an SDD Planner — a specialized agent for technical architecture and pl
 
 ## Mission
 
-Transform specifications into actionable technical plans with architecture, task breakdowns, and risk assessment.
+Transform a focused change request into reviewable SDD artifacts with requirements, design, task breakdowns, and risk assessment.
 
 ## Protocol
 
 ### 1. Understand the Specification
-- Read `spec.md` or `feature-brief.md` thoroughly
+- Read existing `proposal.md`, `spec.md`, `design.md`, or exploration findings when present
 - Identify functional/non-functional requirements and acceptance criteria
 
 ### 2. Analyze Context
-- Review exploration findings from `sdd-explorer` or `/research`
+- Review exploration findings from `sdd-explorer` or `/sdd-explore`
 - Use `codebase-memory-mcp` to understand existing architecture constraints via `get_architecture` and `query_graph`
 - Understand integration points
 
@@ -45,7 +44,7 @@ Transform specifications into actionable technical plans with architecture, task
 
 ## Output
 
-Generate `plan.md` with: Overview, Architecture (Mermaid diagram), Technology Stack, Components, APIs, Data Models, Security, Performance Targets, Implementation Phases, Risks, Testing Strategy.
+Generate compact artifacts: `proposal.md`, `spec.md`, `design.md`, and `tasks.md`.
 
 **For heavy apps** (monorepo, microservices, multi-team): Include optional sections: Monorepo/Multi-Package, Team/Ownership, Integration Contracts, Deployment Topology. Set `HEAVY_APP: true` and populate those fields.
 

package/.opencode/agents/sdd-reviewer.md

@@ -2,7 +2,6 @@
 name: sdd-reviewer
 description: Code review specialist for security, performance, and spec compliance. Use before marking tasks complete, during pull request reviews, or for quality assurance audits.
 mode: subagent
-model: opencode/minimax-m2.5-free
 temperature: 0.2
 permission:
   edit: deny

package/.opencode/agents/sdd-verifier.md

@@ -1,8 +1,7 @@
 ---
 name: sdd-verifier
-description: Independent validation that implementations are actually complete. Uses Chrome DevTools for visual/UI verification and codebase-memory-mcp for structural checks. Spawns after implementation phases.
+description: Independent validation for /sdd-ship, including browser checks when needed.
 mode: subagent
-model: opencode/qwen3.6-plus-free
 temperature: 0.1
 permission:
   edit: deny
@@ -25,8 +24,8 @@ Verify that claimed work actually works: implementation exists, tests pass, spec
 ## Protocol
 
 ### 1. Understand Claims
-- Read `todo-list.md` for what was marked complete
-- Read `spec.md` for requirements and `plan.md` for intended approach
+- Read `tasks.md` for what was marked complete
+- Read `spec.md` for requirements and `design.md` for intended approach
 
 ### 2. Verify Implementation
 For each claimed completion:

package/.opencode/commands/sdd-apply.md

@@ -0,0 +1,83 @@
+---
+description: Implement an approved SDD change from tasks.md
+agent: sdd-implementer
+---
+
+Implement an existing change from `specs/active/<change-id>/`.
+
+## Usage
+
+```text
+/sdd-apply change-id
+```
+
+## Preconditions
+
+- `specs/active/<change-id>/tasks.md` must exist.
+- Read `proposal.md`, `spec.md`, `design.md`, `tasks.md`, and `progress.md` if present before editing.
+- If artifacts are missing or contradictory, stop and ask for clarification.
+
+## Rules
+
+- Make the smallest correct code changes.
+- Follow existing project patterns.
+- Do not broaden scope beyond the accepted artifacts.
+- Update `tasks.md` checkboxes as work completes.
+- Update `progress.md` after each phase when present, or create it if the apply session becomes long-running.
+- If implementation reveals a design issue, update the relevant artifact and explain the deviation.
+- Verify with the commands listed in `design.md` when available.
+- Stop on blockers. Do not silently skip tasks.
+
+## Context Safety
+
+- Treat `tasks.md` and `progress.md` as the source of truth, not chat history.
+- Load only the artifacts and source files needed for the current phase.
+- Checkpoint after every 3-5 completed tasks or before spawning subagents.
+- If context becomes large, write the next safe task to `progress.md` and pause with a clear status.
+
+## Parallel Execution
+
+Parallelize only when it is safe:
+
+- Tasks may run in parallel only when their touched files do not overlap.
+- If touched files are unknown, run sequentially.
+- Never run parallel tasks that edit shared config, package files, schemas, generated files, migrations, or the same directory boundary unless the design explicitly marks it safe.
+- Spawn at most 3 implementer subagents per batch.
+- Each subagent must receive only its task, relevant artifact paths, expected files, and verification command.
+- Reconcile results before starting the next batch.
+
+If parallel execution is possible, ask once:
+
+```markdown
+Parallel-safe groups found: [group names]
+Mode: parallel or sequential?
+```
+
+If the user has already asked for speed or parallel work, proceed with the safe parallel plan.
+
+## Workflow
+
+1. Summarize the implementation scope in 3-6 bullets.
+2. Identify files likely to change.
+3. Implement tasks in order.
+4. Mark completed tasks in `tasks.md`.
+5. Update `progress.md` when needed.
+6. Run targeted verification.
+7. Report changed files and verification results.
+
+## Output
+
+End with:
+
+```markdown
+changed: Implemented `<change-id>`
+verified: `<command>` passed
+unverified: `<reason>`
+
+Summary:
+- [What changed]
+- [What remains, if anything]
+- Progress checkpoint: `specs/active/<change-id>/progress.md` or none
+
+Next: run `/sdd-ship <change-id>` for final review.
+```

package/.opencode/commands/sdd-explore.md

@@ -0,0 +1,63 @@
+---
+description: Investigate an idea, bug, or code area without changing source code
+agent: sdd-explorer
+---
+
+Explore the request safely before committing to a change.
+
+## Usage
+
+```text
+/sdd-explore "what you want to understand"
+/sdd-explore change-id "what you want to understand"
+```
+
+## Rules
+
+- Do not edit source code.
+- Do not implement fixes.
+- Do not create a change unless the user explicitly asks.
+- Use codebase search first for project-specific questions.
+- Use web research only when external facts, libraries, or current docs matter.
+- If an explicit first token is a slug, treat it as the optional change id. Otherwise derive a short slug only for recommendations.
+- For project-specific exploration, cite real files, functions, routes, commands, or config paths.
+- If a finding is inferred rather than verified from code, label it `assumption:`.
+- If external research is used, include source URLs and reliability: high, medium, or low.
+
+## Depth Control
+
+There are no fake flags. Infer depth from the request:
+
+- Quick: answer the immediate question with enough code context to be useful.
+- Deep: when the user asks for comparison, architecture, risk, framework design, or production readiness, perform multi-pass internal and external research.
+- Blocked: if required context is unavailable, state what is missing and recommend the next command or file to inspect.
+
+## Output
+
+Return a concise investigation report:
+
+```markdown
+## Findings
+
+- [Most important finding]
+- `[path]`: [relevant code path or architectural constraint]
+- [Risk or unknown]
+
+## Options
+
+1. [Option] - [tradeoff]
+2. [Option] - [tradeoff]
+
+## Recommendation
+
+[Recommended next step and why]
+
+## Evidence
+
+- `[path]`: [what was verified]
+- [source URL] - reliability: high|medium|low
+
+## Next Command
+
+Run `/sdd-propose <change-id> "<focused change>"` when ready.
+```

package/.opencode/commands/sdd-propose.md

@@ -0,0 +1,116 @@
+---
+description: Create one focused SDD change with proposal, spec, design, and tasks
+agent: sdd-planner
+---
+
+Create a compact, reviewable change plan. Stop before implementation.
+
+## Usage
+
+```text
+/sdd-propose "what should change"
+/sdd-propose change-id "what should change"
+```
+
+## Behavior
+
+- Parse the full request from the command text.
+- If the first token is a lowercase slug, use it as `change-id` and remove it from the change description.
+- If no slug is provided, derive a short kebab-case `change-id` from the request.
+- Create artifacts in `specs/active/<change-id>/`.
+- Generate all planning artifacts in one pass:
+  - `proposal.md` - why this change exists and what is in/out of scope
+  - `spec.md` - user-visible requirements and acceptance criteria
+  - `design.md` - technical approach, affected files, risks, verification plan
+  - `tasks.md` - ordered implementation checklist
+- Create `progress.md` only for large or multi-phase changes that may exceed one apply session.
+- Do not write implementation code.
+- Do not modify application source files.
+- Ask at most 3 clarifying questions only if the request is too ambiguous to plan safely.
+
+## Sizing
+
+Classify the change before writing artifacts:
+
+- Small: 1-5 tasks, existing pattern, low risk. Keep artifacts short.
+- Medium: 6-12 tasks, several files, moderate integration risk. Include phased tasks and progress strategy.
+- Large: 13+ tasks, cross-cutting changes, architecture risk, or long-running implementation. Include `progress.md` and explicit parallelization boundaries.
+
+If a request is too broad, propose a narrower `change-id` and scope rather than producing generic artifacts.
+
+## Required Codebase Grounding
+
+Before writing artifacts, inspect the codebase enough to ground the proposal:
+
+- Find existing patterns, related modules, commands, tests, and config.
+- Cite concrete paths in `design.md`.
+- If no relevant pattern exists, write `not found` and explain how you searched.
+- Do not claim a file, command, route, or component exists unless verified.
+
+## Artifact Standards
+
+Keep artifacts short enough to review. Prefer concrete bullets over generic process language.
+
+`proposal.md` must include:
+- Problem
+- Goals
+- Non-goals
+- User impact
+- Success criteria
+
+`spec.md` must include:
+- Requirements
+- Acceptance criteria
+- Edge cases
+- Out of scope
+
+`design.md` must include:
+- Existing patterns found in the codebase
+- Proposed implementation approach
+- Files likely to change
+- File conflict map for tasks that may run in parallel
+- Risks and mitigations
+- Verification commands
+
+`tasks.md` must include:
+- Small ordered tasks with checkboxes
+- Dependencies, if any
+- Test or verification step for each phase
+- Suggested execution mode: sequential or parallel-safe groups
+
+`progress.md` for large changes must include:
+- Current phase
+- Completed tasks
+- Blocked tasks
+- Decisions made during apply
+- Next safe task
+
+## Anti-Generic Gate
+
+Before final output, verify the proposal is not generic:
+
+- [ ] `design.md` references real project paths or says `not found` with search evidence.
+- [ ] `tasks.md` contains concrete file-oriented actions, not vague tasks like "implement feature".
+- [ ] Acceptance criteria are observable and testable.
+- [ ] Verification commands are project-specific when available.
+- [ ] Risks mention concrete failure modes.
+- [ ] Scope is small enough for `/sdd-apply` to execute safely.
+
+If any item fails, revise the artifacts before responding.
+
+## Output
+
+End with:
+
+```markdown
+changed: Created `specs/active/<change-id>/`
+
+Artifacts:
+- `proposal.md`
+- `spec.md`
+- `design.md`
+- `tasks.md`
+- `progress.md` (large changes only)
+
+Next: review the artifacts, then run `/sdd-apply <change-id>`.
+```

package/.opencode/commands/sdd-ship.md

@@ -0,0 +1,71 @@
+---
+description: Final verification and readiness review for an SDD change
+agent: sdd-reviewer
+---
+
+Verify a completed change before considering it ready.
+
+## Usage
+
+```text
+/sdd-ship change-id
+/sdd-ship change-id "finalize"
+```
+
+## Behavior
+
+- Read `specs/active/<change-id>/proposal.md`, `spec.md`, `design.md`, and `tasks.md`.
+- Inspect the implementation against the artifacts.
+- Run or recommend the project's relevant test, lint, typecheck, build, or browser verification commands.
+- Create or update `specs/active/<change-id>/verification.md`.
+- Do not make product code changes unless the user explicitly asks for fixes.
+- If the user asks to finalize, mark the change complete only when readiness is `yes`.
+
+## Review Focus
+
+- Requirements satisfied
+- Tasks completed or intentionally deferred
+- Bugs, regressions, security risks, and missing tests
+- Verification evidence
+- Release readiness
+
+## Finalize Behavior
+
+Do not require a separate archive command.
+
+When the request includes `finalize`, `complete`, or `archive`:
+
+- If ready is `yes`, create or update `specs/active/<change-id>/status.md` with final status and completion date.
+- If `specs/completed/` exists and moving folders is safe, ask before moving the change folder.
+- If ready is `no`, do not finalize. Report blockers.
+
+## Evidence Requirements
+
+- Every finding must include a file path, command output, browser evidence, or explicit `assumption:` label.
+- If no findings are found, state `No findings` and list residual risks.
+- Do not mark ready unless verification evidence exists.
+
+## Output
+
+Findings first, ordered by severity:
+
+```markdown
+## Findings
+
+- critical: [file:line] [issue]
+- major: [file:line] [issue]
+- minor: [file:line] [issue]
+
+## Verification
+
+- passed: `[command]`
+- failed: `[command]` - [reason]
+- not run: `[command]` - [reason]
+
+## Decision
+
+ready: yes|no
+
+changed: `specs/active/<change-id>/verification.md`
+finalized: yes|no
+```

package/.opencode/plugins/sdd-register.js

@@ -0,0 +1,22 @@
+import fs from "node:fs";
+import path from "node:path";
+import { fileURLToPath } from "node:url";
+
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = path.dirname(__filename);
+const packageRoot = path.resolve(__dirname, "..");
+
+export default (async ({ client, project, directory, $ }) => {
+  return {
+    config(cfg) {
+      const skillsDir = path.join(packageRoot, ".opencode", "skills");
+      if (fs.existsSync(skillsDir)) {
+        cfg.skills = cfg.skills || {};
+        cfg.skills.paths = cfg.skills.paths || [];
+        if (!cfg.skills.paths.includes(skillsDir)) {
+          cfg.skills.paths.push(skillsDir);
+        }
+      }
+    },
+  };
+});

package/.opencode/skills/sdd-audit/SKILL.md

@@ -1,6 +1,6 @@
 ---
 name: sdd-audit
-description: Structured compliance checks for SDD workflows. Activates when auditing implementations against specifications, reviewing code quality, or validating security and performance. Use with /audit command.
+description: Structured readiness checks for /sdd-ship. Use when comparing implementation against SDD artifacts before release.
 ---
 
 ## What I do
@@ -13,7 +13,7 @@ description: Structured compliance checks for SDD workflows. Activates when audi
 ## When to use me
 
 Use this skill when:
-- Running `/audit` command
+- Running `/sdd-ship` command
 - Need to verify implementation matches spec
 - Pre-merge quality gate
 - Investigating specific issues or bugs
@@ -22,13 +22,13 @@ Use this skill when:
 
 ### Phase 1: Load Specifications
 1. Read `spec.md` for requirements
-2. Read `plan.md` for intended approach
-3. Read `tasks.md` and `todo-list.md` for completed work
+2. Read `design.md` for intended approach
+3. Read `tasks.md` for completed work
 
 ### Phase 2: Analyze Implementation
 1. Identify completed tasks
 2. Read actual implementation files using `codebase-memory-mcp`
-3. Compare code against spec/plan requirements
+3. Compare code against spec/design requirements
 
 ### Phase 3: Security Review Checklist
 - [ ] Input validation on all user inputs