@cluesmith/codev 3.0.0-rc.0 → 3.0.0-rc.10

package/skeleton/protocols/tick/builder-prompt.md

@@ -1,65 +0,0 @@
-# {{protocol_name}} Builder ({{mode}} mode)
-
-You are implementing {{input_description}}.
-
-{{#if mode_soft}}
-## Mode: SOFT
-You are running in SOFT mode. This means:
-- You follow the TICK protocol yourself (no porch orchestration)
-- The architect monitors your work and verifies you're adhering to the protocol
-- Run consultations manually when the protocol calls for them
-- You have flexibility in execution, but must stay compliant with the protocol
-{{/if}}
-
-{{#if mode_strict}}
-## Mode: STRICT
-You are running in STRICT mode. This means:
-- Porch orchestrates your work
-- Run: `porch next` to get your next tasks
-- Follow porch signals and gate approvals
-
-### ABSOLUTE RESTRICTIONS (STRICT MODE)
-- **NEVER edit `status.yaml` directly** — only porch commands may modify project state
-- **NEVER call `porch approve` without explicit human approval** — only run it after the architect says to
-- **NEVER skip the 3-way review** — always follow porch next → porch done cycle
-{{/if}}
-
-## Protocol
-Follow the TICK protocol: `codev/protocols/tick/protocol.md`
-
-TICK is for amendments to existing SPIR specifications. You will:
-1. Identify the target spec to amend
-2. Update the spec with the amendment
-3. Update the plan
-4. Implement the changes
-5. Defend with tests
-6. Create review
-
-{{#if spec}}
-## Target Spec
-The spec to amend is at: `{{spec.path}}`
-{{/if}}
-
-{{#if plan}}
-## Target Plan
-The plan to amend is at: `{{plan.path}}`
-{{/if}}
-
-{{#if task}}
-## Amendment Description
-{{task_text}}
-{{/if}}
-
-## Handling Flaky Tests
-
-If you encounter **pre-existing flaky tests** (intermittent failures unrelated to your changes):
-1. **DO NOT** edit `status.yaml` to bypass checks
-2. **DO NOT** skip porch checks or use any workaround to avoid the failure
-3. **DO** mark the test as skipped with a clear annotation (e.g., `it.skip('...') // FLAKY: skipped pending investigation`)
-4. **DO** document each skipped flaky test in your review under a `## Flaky Tests` section
-5. Commit the skip and continue with your work
-
-## Getting Started
-1. Read the TICK protocol thoroughly
-2. Identify what needs to change in the existing spec
-3. Follow the amendment workflow

package/skeleton/protocols/tick/consult-types/impl-review.md

@@ -1,72 +0,0 @@
-# Implementation Review Prompt
-
-## Context
-You are reviewing implementation work during the Implement phase. A builder has completed a plan phase and needs feedback before proceeding. Your job is to verify the implementation matches the spec and plan.
-
-## CRITICAL: Verify Before Flagging
-
-Before requesting changes for missing configuration, incorrect patterns, or framework issues:
-1. **Check `package.json`** for actual dependency versions — framework conventions change between major versions
-2. **Read the actual config files** (or confirm their deliberate absence) before flagging missing configs
-3. **Do not assume** your training data reflects the version in use — verify against project files
-4. If "Previous Iteration Context" is provided, read it carefully before re-raising concerns that were already disputed
-
-## Focus Areas
-
-1. **Spec Adherence**
-   - Does the implementation fulfill the spec requirements for this phase?
-   - Are acceptance criteria met?
-
-2. **Code Quality**
-   - Is the code readable and maintainable?
-   - Are there obvious bugs or issues?
-   - Are error cases handled appropriately?
-
-3. **Test Coverage**
-   - Are the tests adequate for this phase?
-   - Do tests cover the main paths AND edge cases?
-
-4. **Plan Alignment**
-   - Does the implementation follow the plan?
-   - Are there plan items skipped or partially completed?
-
-5. **UX Verification** (if spec has UX requirements)
-   - Does the actual user experience match what the spec describes?
-   - If spec says "async" or "non-blocking", is it actually async?
-
-## Verdict Format
-
-After your review, provide your verdict in exactly this format:
-
-```
----
-VERDICT: [APPROVE | REQUEST_CHANGES | COMMENT]
-SUMMARY: [One-line summary of your assessment]
-CONFIDENCE: [HIGH | MEDIUM | LOW]
----
-KEY_ISSUES:
-- [Issue 1 or "None"]
-- [Issue 2]
-...
-```
-
-**Verdict meanings:**
-- `APPROVE`: Phase is complete, builder can proceed
-- `REQUEST_CHANGES`: Issues that must be fixed before proceeding
-- `COMMENT`: Minor suggestions, can proceed but note feedback
-
-## Scoping (Multi-Phase Plans)
-
-When the implementation plan has multiple phases (e.g., scaffolding, landing, media_rtl):
-- **ONLY review work belonging to the current plan phase**
-- The query will specify which phase you are reviewing
-- Do NOT request changes for functionality scheduled in later phases
-- Do NOT flag missing features that are out of scope for this phase
-- If unsure whether something belongs to this phase, check the plan file
-
-## Notes
-
-- This is a phase-level review, not the final PR review
-- Focus on "does this phase work" not "is the whole feature done"
-- If referencing line numbers, use `file:line` format
-- The builder needs actionable feedback to continue

package/skeleton/protocols/tick/consult-types/plan-review.md

@@ -1,59 +0,0 @@
-# Plan Review Prompt
-
-## Context
-You are reviewing an implementation plan during the Plan phase. The spec has been approved - now you must evaluate whether the plan adequately describes HOW to implement it.
-
-## Focus Areas
-
-1. **Spec Coverage**
-   - Does the plan address all requirements in the spec?
-   - Are there spec requirements not covered by any phase?
-   - Are there phases that go beyond the spec scope?
-
-2. **Phase Breakdown**
-   - Are phases appropriately sized (not too large or too small)?
-   - Is the sequence logical (dependencies respected)?
-   - Can each phase be completed and committed independently?
-
-3. **Technical Approach**
-   - Is the implementation approach sound?
-   - Are the right files/modules being modified?
-   - Are there obvious better approaches being missed?
-
-4. **Testability**
-   - Does each phase have clear test criteria?
-   - Will the Defend step (writing tests) be feasible?
-   - Are edge cases from the spec addressable?
-
-5. **Risk Assessment**
-   - Are there potential blockers not addressed?
-   - Are dependencies on other systems identified?
-   - Is the plan realistic given constraints?
-
-## Verdict Format
-
-After your review, provide your verdict in exactly this format:
-
-```
----
-VERDICT: [APPROVE | REQUEST_CHANGES | COMMENT]
-SUMMARY: [One-line summary of your assessment]
-CONFIDENCE: [HIGH | MEDIUM | LOW]
----
-KEY_ISSUES:
-- [Issue 1 or "None"]
-- [Issue 2]
-...
-```
-
-**Verdict meanings:**
-- `APPROVE`: Plan is ready for human review
-- `REQUEST_CHANGES`: Significant issues with approach or coverage
-- `COMMENT`: Minor suggestions, plan is workable but could improve
-
-## Notes
-
-- The spec has already been approved - don't re-litigate spec decisions
-- Focus on the quality of the plan as a guide for builders
-- Consider: Would a builder be able to follow this plan successfully?
-- If referencing existing code, verify file paths seem accurate

package/skeleton/protocols/tick/consult-types/pr-review.md

@@ -1,72 +0,0 @@
-# PR Ready Review Prompt
-
-## Context
-You are performing a final self-check during the Review phase. The builder has completed all implementation phases and is about to create a PR. This is the last check before the work goes to the architect for integration review.
-
-## Focus Areas
-
-1. **Completeness**
-   - Are all spec requirements implemented?
-   - Are all plan phases complete?
-   - Is the review document written (`codev/reviews/XXXX-name.md`)?
-   - Are all commits properly formatted (`[Spec XXXX][Phase]`)?
-
-2. **Test Status**
-   - Do all tests pass?
-   - Is test coverage adequate for the changes?
-   - Are there any skipped or flaky tests?
-
-3. **Code Cleanliness**
-   - Is there any debug code left in?
-   - Are there any TODO comments that should be resolved?
-   - Are there any `// REVIEW:` comments that weren't addressed?
-   - Is the code properly formatted?
-
-4. **Documentation**
-   - Are inline comments clear where needed?
-   - Is the review document comprehensive?
-   - Are any new APIs documented?
-
-5. **PR Readiness**
-   - Is the branch up to date with main?
-   - Are commits atomic and well-described?
-   - Is the change diff reasonable in size?
-
-## Verdict Format
-
-After your review, provide your verdict in exactly this format:
-
-```
----
-VERDICT: [APPROVE | REQUEST_CHANGES | COMMENT]
-SUMMARY: [One-line summary of your assessment]
-CONFIDENCE: [HIGH | MEDIUM | LOW]
----
-KEY_ISSUES:
-- [Issue 1 or "None"]
-- [Issue 2]
-...
-
-PR_SUMMARY: |
-  ## Summary
-  [2-3 sentences describing what this PR does]
-
-  ## Key Changes
-  - [Change 1]
-  - [Change 2]
-
-  ## Test Plan
-  - [How to test]
-```
-
-**Verdict meanings:**
-- `APPROVE`: Ready to create PR
-- `REQUEST_CHANGES`: Issues to fix before PR creation
-- `COMMENT`: Minor items, can create PR but note feedback
-
-## Notes
-
-- This is the builder's final self-review before hand-off
-- The PR_SUMMARY in your output can be used as the PR description
-- Focus on "is this ready for someone else to review" not "is this perfect"
-- Any issues found here are cheaper to fix than during integration review

package/skeleton/protocols/tick/consult-types/spec-review.md

@@ -1,55 +0,0 @@
-# Specification Review Prompt
-
-## Context
-You are reviewing a feature specification during the Specify phase. Your role is to ensure the spec is complete, correct, and feasible before it moves to human approval.
-
-## Focus Areas
-
-1. **Completeness**
-   - Are all requirements clearly stated?
-   - Are success criteria defined?
-   - Are edge cases considered?
-   - Is scope well-bounded (not too broad or vague)?
-
-2. **Correctness**
-   - Do requirements make sense technically?
-   - Are there contradictions?
-   - Is the problem statement accurate?
-
-3. **Feasibility**
-   - Can this be implemented with available tools/constraints?
-   - Are there obvious technical blockers?
-   - Is the scope realistic for a single spec?
-
-4. **Clarity**
-   - Would a builder understand what to build?
-   - Are acceptance criteria testable?
-   - Is terminology consistent?
-
-## Verdict Format
-
-After your review, provide your verdict in exactly this format:
-
-```
----
-VERDICT: [APPROVE | REQUEST_CHANGES | COMMENT]
-SUMMARY: [One-line summary of your assessment]
-CONFIDENCE: [HIGH | MEDIUM | LOW]
----
-KEY_ISSUES:
-- [Issue 1 or "None"]
-- [Issue 2]
-...
-```
-
-**Verdict meanings:**
-- `APPROVE`: Spec is ready for human review
-- `REQUEST_CHANGES`: Significant issues must be fixed before proceeding
-- `COMMENT`: Minor suggestions, can proceed but consider feedback
-
-## Notes
-
-- You are NOT reviewing code - you are reviewing the specification document
-- Focus on WHAT is being built, not HOW it will be implemented (that's for plan review)
-- Be constructive - identify issues AND suggest solutions
-- If the spec references other specs, note if context seems missing

package/skeleton/protocols/tick/protocol.json

@@ -1,151 +0,0 @@
-{
-  "$schema": "../../protocol-schema.json",
-  "name": "tick",
-  "version": "1.1.0",
-  "description": "Amendment workflow for existing SPIR specifications",
-  "input": {
-    "type": "spec",
-    "required": false
-  },
-  "phases": [
-    {
-      "id": "identify",
-      "name": "Identify Target",
-      "description": "Find the existing spec to amend",
-      "type": "once",
-      "steps": [
-        "analyze_requirements",
-        "find_target_spec",
-        "verify_spec_integrated",
-        "determine_tick_number"
-      ],
-      "transition": {
-        "on_complete": "amend_spec"
-      }
-    },
-    {
-      "id": "amend_spec",
-      "name": "Amend Specification",
-      "description": "Update the existing specification",
-      "type": "once",
-      "steps": [
-        "analyze_changes_needed",
-        "update_spec_sections",
-        "add_amendment_entry",
-        "commit_spec_changes"
-      ],
-      "transition": {
-        "on_complete": "amend_plan"
-      }
-    },
-    {
-      "id": "amend_plan",
-      "name": "Amend Plan",
-      "description": "Update the existing plan",
-      "type": "once",
-      "steps": [
-        "update_plan_phases",
-        "add_amendment_history",
-        "commit_plan_changes"
-      ],
-      "transition": {
-        "on_complete": "implement"
-      }
-    },
-    {
-      "id": "implement",
-      "name": "Implement",
-      "description": "Implement the amendment",
-      "type": "once",
-      "steps": [
-        "implement_changes",
-        "self_review",
-        "commit"
-      ],
-      "checks": {
-        "build": {
-          "command": "npm run build",
-          "on_fail": "retry",
-          "max_retries": 2
-        }
-      },
-      "transition": {
-        "on_complete": "defend"
-      }
-    },
-    {
-      "id": "defend",
-      "name": "Defend",
-      "description": "Test the amendment",
-      "type": "once",
-      "steps": [
-        "write_tests",
-        "run_tests",
-        "fix_failures"
-      ],
-      "checks": {
-        "tests": {
-          "command": "npm test",
-          "on_fail": "implement",
-          "max_retries": 1
-        }
-      },
-      "transition": {
-        "on_complete": "evaluate"
-      }
-    },
-    {
-      "id": "evaluate",
-      "name": "Evaluate",
-      "description": "Verify amendment meets requirements",
-      "type": "once",
-      "steps": [
-        "verify_requirements",
-        "check_regressions"
-      ],
-      "transition": {
-        "on_complete": "review"
-      }
-    },
-    {
-      "id": "review",
-      "name": "Review",
-      "description": "Create review document and PR",
-      "type": "once",
-      "steps": [
-        "create_tick_review",
-        "create_pr"
-      ],
-      "consultation": {
-        "on": "review",
-        "models": ["gemini", "codex"],
-        "type": "impl",
-        "parallel": true,
-        "max_rounds": 1
-      },
-      "gate": "pr"
-    }
-  ],
-  "signals": {
-    "PHASE_COMPLETE": {
-      "description": "Signal current phase is complete",
-      "transitions_to": "next_phase"
-    },
-    "BLOCKED": {
-      "description": "Signal implementation is blocked",
-      "requires": "reason"
-    }
-  },
-  "defaults": {
-    "mode": "strict",
-    "consultation": {
-      "enabled": true,
-      "models": ["gemini", "codex"],
-      "parallel": true
-    },
-    "checks": {
-      "build": "npm run build",
-      "test": "npm test"
-    }
-  }
-}