Stefano Fiorini
437b2024cd
M1 canonical specs (opencode reviewer research, task-plan template, Review Loop subroutine, Phase 1-10 prose, secret-scan regex list) are embedded in the M2 claude-code SKILL.md as the canonical reference; later variants (M3 codex, M4 cursor, M5 opencode) will fork from this file. Reviewer: codex / gpt-5.4. Approved after 3 rounds: - Round 1: 2 P1 + 3 P2 → REVISE - Round 2: 1 P2 → REVISE - Round 3: 0 P0/P1/P2, 1 P3 (non-blocking) → APPROVED Key design properties: - Plan-review payload strips Runtime State and Review History to prevent reviewer session-ID leakage across rounds. - Secret-scan step 1a redacts matched text to [REDACTED:<pattern-label>:<match-length>-chars] before any user surfacing; never echoes raw match content. - Brainstorming required for any behavior-changing task; auto-skip limited to pure-documentation and pure-comment-whitespace-rename. - Phase 3 reviewer config defaults to codex / gpt-5.4 with MAX_ROUNDS=10 when user opts for defaults; explicit interactive default also gpt-5.4 for internal consistency. - Template Metadata records Branch Name and Worktree Path. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
4.3 KiB
4.3 KiB
Task Plan: [Short Title]
Variant guardrail (Claude Code): When generating or updating this file, the agent MUST be out of plan mode. Sub-skills (
brainstorming,test-driven-development,verification-before-completion,finishing-a-development-branch,using-git-worktrees) MUST be invoked through theSkilltool explicitly — no shell wrappers.
Metadata
| Field | Value |
|---|---|
| Created | YYYY-MM-DD |
| Slug | YYYY-MM-DD- |
| Runtime | claude-code |
| Reviewer CLI | codex | claude | cursor | opencode |
| Reviewer Model | |
| MAX_ROUNDS | 10 |
| Branch Strategy | current-branch | worktree |
| Branch Name | <current branch name, or new branch name when worktree is used> |
| Worktree Path | <absolute path to worktree dir; blank when Branch Strategy = current-branch> |
| Status | draft |
Status Enum (authoritative)
| Value | Meaning |
|---|---|
draft |
Newly created; plan review not yet started |
plan-approved |
Plan review loop returned APPROVED |
implementation-in-progress |
Phase 6 executing |
implementation-approved |
Phase 8 review loop returned APPROVED; awaiting commit |
pushed |
Committed + pushed to remote |
local-only |
Committed locally; user declined push |
aborted-plan-review |
MAX_ROUNDS reached in Phase 5; user aborted |
aborted-impl-review |
MAX_ROUNDS reached in Phase 8; user aborted |
aborted-verification |
Phase 7 retries exhausted; user aborted |
failed |
Hard tooling failure |
Prompt
Interpretation
Assumptions
Files
| Action | Path | Why |
|---|---|---|
Approach
TDD Approach
Acceptance Criteria
- <criterion 1>
- <criterion 2>
Verification
Rollback
Runtime State
plan_review_round: 0
implementation_review_round: 0
CODEX_PLAN_SESSION_ID:
CODEX_IMPL_SESSION_ID:
CURSOR_PLAN_SESSION_ID:
CURSOR_IMPL_SESSION_ID:
OPENCODE_PLAN_SESSION_ID:
OPENCODE_IMPL_SESSION_ID:
last_phase_entered:
last_round_ts:
last_scan_outcome_plan:
last_scan_outcome_impl:
verification_attempts: 0
tests_added_count: 0
tdd_used: false
Review History
| Timestamp (ISO-8601) | Loop | Round | Verdict | Summary |
|---|---|---|---|---|
Final Status
Guardrails (do NOT remove)
- This file is the single persistent artifact for
do-task. Do not split it or delete it on success. Statusmust always match one of the 10 enum values.Runtime Stateis updated by the skill, not by the user.- Review History is append-only.
last_scan_outcome_planandlast_scan_outcome_implrecord the most recent secret-scan result for each loop. They are informational; the scan itself runs per-payload with no caching.