diff --git a/.agents/skills/adhoc-implement/SKILL.md b/.agents/skills/adhoc-implement/SKILL.md index 14879c9..282ab5a 100644 --- a/.agents/skills/adhoc-implement/SKILL.md +++ b/.agents/skills/adhoc-implement/SKILL.md @@ -44,7 +44,11 @@ Run in order before implementation: 4. `git checkout -b codex/adhoc-` Rules: -- If worktree is dirty before step 1, stop and report blocker +- If worktree is dirty before step 1: +- Allow only the plan-handoff case where all modified files are planning outputs and include `plan_path`. +- Planning-output allowlist: `./product/PLAN_NEXT.md`, `./product/PLAN_v1.0.md`, and selected `plan_path`. +- In the allowlist case, require current branch is already `main`, run step 1, skip steps 2-3, and run step 4 to branch with plan edits preserved. +- Otherwise stop and report blocker. - If unexpected unrelated changes appear during execution, stop immediately and ask how to proceed - Do not auto-commit or auto-push unless explicitly requested by the user diff --git a/.agents/skills/adhoc-plan/SKILL.md b/.agents/skills/adhoc-plan/SKILL.md index 315a9f6..f21c420 100644 --- a/.agents/skills/adhoc-plan/SKILL.md +++ b/.agents/skills/adhoc-plan/SKILL.md @@ -68,6 +68,12 @@ Validation rules: 11. Verify quality gates. 12. Overwrite `output_plan_path` with the final plan. +## Handoff Contract (Planning -> Implementation) + +- This skill intentionally leaves `output_plan_path` modified in the working tree. +- Expected follow-up is `adhoc-implement` with the same `plan_path` on a new branch. +- If additional dirty files exist beyond the generated plan file, stop and scope/clean before implementation. + ## Command Contract (JSON Required) Use `wrkr` commands with `--json` whenever the plan needs machine-readable evidence, for example: diff --git a/.agents/skills/backlog-implement/SKILL.md b/.agents/skills/backlog-implement/SKILL.md index a1e0bd9..0343906 100644 --- a/.agents/skills/backlog-implement/SKILL.md +++ b/.agents/skills/backlog-implement/SKILL.md @@ -57,7 +57,11 @@ Before implementation starts, run in order: 4. `git checkout -b codex/` Rules: -- If working tree is dirty before step 1, stop and report blocker for user decision. +- If working tree is dirty before step 1: +- Allow only the plan-handoff case where all modified files are planning outputs and include selected `plan_path`. +- Planning-output allowlist: `./product/PLAN_NEXT.md`, `./product/PLAN_v1.0.md`, and selected `plan_path`. +- In the allowlist case, require current branch is already `main`, run step 1, skip steps 2-3, and run step 4 to branch with plan edits preserved. +- Otherwise stop and report blocker for user decision. - If unexpected changes appear during implementation, stop immediately and ask how to proceed. - Do not switch to other branches unless user explicitly requests it. diff --git a/.agents/skills/backlog-plan/SKILL.md b/.agents/skills/backlog-plan/SKILL.md index 4bfb592..c6849a8 100644 --- a/.agents/skills/backlog-plan/SKILL.md +++ b/.agents/skills/backlog-plan/SKILL.md @@ -16,7 +16,8 @@ Execute this workflow when asked to convert strategic feature recommendations in - `./product/dev_guides.md` - `./product/architecture_guides.md` - Structure references (match level of detail and style): -- `gait/product/PLAN_v1.md` (reference only, if available in your environment) +- `./product/PLAN_v1.md` (preferred local reference, if available) +- `../gait/product/PLAN_v1.md` (fallback reference, if available in your environment) - Output file: `./product/PLAN_NEXT.md` (unless user specifies a different target) - Planning only. Do not implement code or docs outside the target plan file. @@ -55,6 +56,12 @@ If these are missing, stop and output a gap note instead of inventing details. 12. Add definition of done and release/exit gate criteria. 13. Write full plan to target file, overwriting prior contents. +## Handoff Contract (Planning -> Implementation) + +- This skill intentionally leaves the generated plan file modified in the working tree. +- Expected follow-up is `backlog-implement` using that plan file on a new branch. +- If additional dirty files exist beyond the plan output, stop and scope/clean before implementation. + ## Non-Negotiables - Preserve Wrkr core contracts: determinism, offline-first defaults, fail-closed policy posture, schema stability, and exit code stability. diff --git a/.agents/skills/initial-plan/SKILL.md b/.agents/skills/initial-plan/SKILL.md index 6c668dd..4347323 100644 --- a/.agents/skills/initial-plan/SKILL.md +++ b/.agents/skills/initial-plan/SKILL.md @@ -1,6 +1,7 @@ --- name: initial-plan -description: Transform the Wrkr PRD in product/wrkr.md into a world-class, zero-ambiguity execution plan that mirrors the detail level of gait/product/PLAN_v1.md, while enforcing product/dev_guides.md and product/architecture_guides.md standards for coding, testing, CI, determinism, architecture governance, and contracts. Use when the user asks for an initial build plan from the PRD (not from ideas/recommendations). +description: Transform the Wrkr PRD in product/wrkr.md into a world-class, zero-ambiguity execution plan that mirrors the detail level of product/PLAN_v1.md (or gait/product/PLAN_v1.md when needed), while enforcing product/dev_guides.md and product/architecture_guides.md standards for coding, testing, CI, determinism, architecture governance, and contracts. Use when the user asks for an initial build plan from the PRD (not from ideas/recommendations). +disable-model-invocation: true --- # PRD to Initial Execution Plan (Wrkr) @@ -61,7 +62,7 @@ If these are missing, stop and output a gap note instead of inventing policy. - frugal architecture/cost impact requirements - chaos/hardening/perf lane triggers and failure-hypothesis expectations -4. Read `gait/product/PLAN_v1.md` to mirror structure depth and story-level specificity. +4. Read `./product/PLAN_v1.md` (or `../gait/product/PLAN_v1.md` if local reference is unavailable) to mirror structure depth and story-level specificity. 5. Inspect current repository baseline and convert observed reality into a `Current Baseline (Observed)` section: - existing directories and key files @@ -90,6 +91,12 @@ If these are missing, stop and output a gap note instead of inventing policy. 11. Write the plan to the target file, replacing prior contents. +## Handoff Contract (Planning -> Implementation) + +- This skill intentionally leaves the generated plan file modified in the working tree. +- Expected follow-up is an implementation skill on a new branch with that plan as input. +- If additional dirty files exist beyond the generated plan file, stop and scope/clean before implementation. + ## Non-Negotiables - Preserve Wrkr core contracts: deterministic execution, zero data exfiltration by default, fail-closed policy posture, stable schema contracts, stable exit code contracts.