From 90a07debb5770a599b6a686c961f0d6c5fc6e883 Mon Sep 17 00:00:00 2001 From: nntrivi2001 Date: Tue, 25 Aug 2026 09:49:18 +0700 Subject: [PATCH] docs(lanes): fix Stage 13/14 consistency gaps found in review --- docs/LANES.md | 2 +- skills/ship-feature-lane/SKILL.md | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/docs/LANES.md b/docs/LANES.md index a9b1bff5..7220e039 100644 --- a/docs/LANES.md +++ b/docs/LANES.md @@ -1097,7 +1097,7 @@ Copies `skills/ship-feature-lane/` into `~/.claude/skills/ship-feature-lane/` (o The skill frontloads all clarifying questions once, then runs unattended. The only interactive touchpoints are **Stage 0 (Q&A intake)** and **merging the PR on GitHub**; everything else runs to completion or escalates with a `blocked` status visible on the dashboard. Progress is declared via `ccam stage` commands at every step. -### Current status: stages 0–8, 10–12, 14 work; stages 9, 13 deferred; stages 6–7 agent-gated +### Current status: stages 0–8, 10–12, 14 work; stage 9 deferred, stage 13 partially wired (CI-wait works, dev-QC doesn't); stages 6–7 agent-gated **Stages implemented now:** - **0-8**: Intake & frontloaded Q&A → plan → implementation → independent unit tests (1b) → gates+preflight → e2e on feature branch → code review → QC plan → QC → senior gate → publish PR diff --git a/skills/ship-feature-lane/SKILL.md b/skills/ship-feature-lane/SKILL.md index b6ad6876..2c027673 100644 --- a/skills/ship-feature-lane/SKILL.md +++ b/skills/ship-feature-lane/SKILL.md @@ -210,8 +210,8 @@ The e2e hook doesn't run migrations itself — it tests the already-running stac - If `ccam lanes integration ci_wait` reads off for this profile, skip the loop entirely and go straight to Stage 14 with `--note "ci_wait integration off for this profile"` — the toggle means the profile doesn't want this check, not that the command is unavailable. ### 14 — Done -- Post the final report: PR merged, that post-merge verification is unavailable pending F. -- `ccam stage done --status passed --note "merged; post-merge verification unavailable (F not built)"` → STOP (leave the lane for the human to clear from the dashboard whenever). +- Post the final report: PR merged, that dev-QC (deployed-site QC) is unavailable pending F; CI-wait already ran in Stage 13. +- `ccam stage done --status passed --note "merged; dev-QC unavailable (F not built)"` → STOP (leave the lane for the human to clear from the dashboard whenever). ## Escalation Whenever you STOP early (an ambiguous merge conflict, an unexpected/unrecoverable failure, a missing agent this skill depends on), set `ccam stage --status blocked --note ""` — the dashboard surfaces it. Then summarize for the human and wait.