@garygentry/feature-forge 0.2.3 → 0.2.4
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/README.md +6 -1
- package/adapters/GENERATION-REPORT.md +5 -1
- package/adapters/claude/references/forge-config-schema.json +25 -3
- package/adapters/claude/references/pipeline-state-schema.json +3 -2
- package/adapters/claude/references/portable-root.md +2 -2
- package/adapters/claude/references/process-overview.md +10 -0
- package/adapters/claude/references/shared-conventions.md +14 -9
- package/adapters/claude/references/stage-exit-protocol.md +99 -0
- package/adapters/claude/scripts/epic-manifest.py +10 -0
- package/adapters/claude/scripts/forge-bootstrap.py +94 -16
- package/adapters/claude/scripts/forge-init.sh +7 -1
- package/adapters/claude/scripts/forge-session.py +175 -30
- package/adapters/claude/skills/forge/SKILL.md +28 -14
- package/adapters/claude/skills/forge-0-epic/SKILL.md +20 -15
- package/adapters/claude/skills/forge-0-epic/references/edit-mode.md +6 -4
- package/adapters/claude/skills/forge-0-epic/references/epic-manifest-subcommands.md +1 -1
- package/adapters/claude/skills/forge-1-prd/SKILL.md +14 -4
- package/adapters/claude/skills/forge-2-tech/SKILL.md +14 -3
- package/adapters/claude/skills/forge-3-specs/SKILL.md +14 -3
- package/adapters/claude/skills/forge-4-backlog/SKILL.md +16 -5
- package/adapters/claude/skills/forge-5-loop/SKILL.md +19 -21
- package/adapters/claude/skills/forge-5-loop/references/result-reporting.md +10 -5
- package/adapters/claude/skills/forge-6-docs/SKILL.md +6 -6
- package/adapters/claude/skills/forge-bootstrap/SKILL.md +4 -4
- package/adapters/claude/skills/forge-fix/SKILL.md +27 -6
- package/adapters/claude/skills/forge-guide/SKILL.md +179 -0
- package/adapters/claude/skills/forge-init/SKILL.md +28 -1
- package/adapters/claude/skills/forge-verify/SKILL.md +46 -15
- package/adapters/claude/skills/forge-verify/references/verification-checklists.md +1 -1
- package/adapters/codex/references/forge-config-schema.json +25 -3
- package/adapters/codex/references/pipeline-state-schema.json +3 -2
- package/adapters/codex/references/portable-root.md +2 -2
- package/adapters/codex/references/process-overview.md +10 -0
- package/adapters/codex/references/shared-conventions.md +14 -9
- package/adapters/codex/references/stage-exit-protocol.md +99 -0
- package/adapters/codex/scripts/epic-manifest.py +10 -0
- package/adapters/codex/scripts/forge-bootstrap.py +94 -16
- package/adapters/codex/scripts/forge-init.sh +7 -1
- package/adapters/codex/scripts/forge-session.py +175 -30
- package/adapters/codex/skills/forge/SKILL.md +33 -19
- package/adapters/codex/skills/forge-0-epic/SKILL.md +21 -16
- package/adapters/codex/skills/forge-0-epic/references/edit-mode.md +6 -4
- package/adapters/codex/skills/forge-0-epic/references/epic-manifest-subcommands.md +1 -1
- package/adapters/codex/skills/forge-1-prd/SKILL.md +14 -4
- package/adapters/codex/skills/forge-2-tech/SKILL.md +15 -4
- package/adapters/codex/skills/forge-3-specs/SKILL.md +14 -3
- package/adapters/codex/skills/forge-4-backlog/SKILL.md +16 -5
- package/adapters/codex/skills/forge-5-loop/SKILL.md +21 -23
- package/adapters/codex/skills/forge-5-loop/references/result-reporting.md +10 -5
- package/adapters/codex/skills/forge-6-docs/SKILL.md +6 -6
- package/adapters/codex/skills/forge-bootstrap/SKILL.md +4 -4
- package/adapters/codex/skills/forge-fix/SKILL.md +27 -6
- package/adapters/codex/skills/forge-guide/SKILL.md +188 -0
- package/adapters/codex/skills/forge-init/SKILL.md +28 -1
- package/adapters/codex/skills/forge-verify/SKILL.md +45 -14
- package/adapters/codex/skills/forge-verify/references/verification-checklists.md +1 -1
- package/adapters/copilot/references/forge-config-schema.json +25 -3
- package/adapters/copilot/references/pipeline-state-schema.json +3 -2
- package/adapters/copilot/references/portable-root.md +2 -2
- package/adapters/copilot/references/process-overview.md +10 -0
- package/adapters/copilot/references/shared-conventions.md +14 -9
- package/adapters/copilot/references/stage-exit-protocol.md +99 -0
- package/adapters/copilot/scripts/epic-manifest.py +10 -0
- package/adapters/copilot/scripts/forge-bootstrap.py +94 -16
- package/adapters/copilot/scripts/forge-init.sh +7 -1
- package/adapters/copilot/scripts/forge-session.py +175 -30
- package/adapters/copilot/skills/forge/forge.md +33 -19
- package/adapters/copilot/skills/forge-0-epic/forge-0-epic.md +21 -16
- package/adapters/copilot/skills/forge-0-epic/references/edit-mode.md +6 -4
- package/adapters/copilot/skills/forge-0-epic/references/epic-manifest-subcommands.md +1 -1
- package/adapters/copilot/skills/forge-1-prd/forge-1-prd.md +14 -4
- package/adapters/copilot/skills/forge-2-tech/forge-2-tech.md +15 -4
- package/adapters/copilot/skills/forge-3-specs/forge-3-specs.md +14 -3
- package/adapters/copilot/skills/forge-4-backlog/forge-4-backlog.md +16 -5
- package/adapters/copilot/skills/forge-5-loop/forge-5-loop.md +21 -23
- package/adapters/copilot/skills/forge-5-loop/references/result-reporting.md +10 -5
- package/adapters/copilot/skills/forge-6-docs/forge-6-docs.md +6 -6
- package/adapters/copilot/skills/forge-bootstrap/forge-bootstrap.md +4 -4
- package/adapters/copilot/skills/forge-fix/forge-fix.md +27 -6
- package/adapters/copilot/skills/forge-guide/forge-guide.md +188 -0
- package/adapters/copilot/skills/forge-init/forge-init.md +28 -1
- package/adapters/copilot/skills/forge-verify/forge-verify.md +45 -14
- package/adapters/copilot/skills/forge-verify/references/verification-checklists.md +1 -1
- package/adapters/cursor/references/forge-config-schema.json +25 -3
- package/adapters/cursor/references/pipeline-state-schema.json +3 -2
- package/adapters/cursor/references/portable-root.md +2 -2
- package/adapters/cursor/references/process-overview.md +10 -0
- package/adapters/cursor/references/shared-conventions.md +14 -9
- package/adapters/cursor/references/stage-exit-protocol.md +99 -0
- package/adapters/cursor/scripts/epic-manifest.py +10 -0
- package/adapters/cursor/scripts/forge-bootstrap.py +94 -16
- package/adapters/cursor/scripts/forge-init.sh +7 -1
- package/adapters/cursor/scripts/forge-session.py +175 -30
- package/adapters/cursor/skills/forge/forge.mdc +33 -19
- package/adapters/cursor/skills/forge-0-epic/forge-0-epic.mdc +21 -16
- package/adapters/cursor/skills/forge-0-epic/references/edit-mode.md +6 -4
- package/adapters/cursor/skills/forge-0-epic/references/epic-manifest-subcommands.md +1 -1
- package/adapters/cursor/skills/forge-1-prd/forge-1-prd.mdc +14 -4
- package/adapters/cursor/skills/forge-2-tech/forge-2-tech.mdc +15 -4
- package/adapters/cursor/skills/forge-3-specs/forge-3-specs.mdc +14 -3
- package/adapters/cursor/skills/forge-4-backlog/forge-4-backlog.mdc +16 -5
- package/adapters/cursor/skills/forge-5-loop/forge-5-loop.mdc +21 -23
- package/adapters/cursor/skills/forge-5-loop/references/result-reporting.md +10 -5
- package/adapters/cursor/skills/forge-6-docs/forge-6-docs.mdc +6 -6
- package/adapters/cursor/skills/forge-bootstrap/forge-bootstrap.mdc +4 -4
- package/adapters/cursor/skills/forge-fix/forge-fix.mdc +27 -6
- package/adapters/cursor/skills/forge-guide/forge-guide.mdc +189 -0
- package/adapters/cursor/skills/forge-init/forge-init.mdc +28 -1
- package/adapters/cursor/skills/forge-verify/forge-verify.mdc +45 -14
- package/adapters/cursor/skills/forge-verify/references/verification-checklists.md +1 -1
- package/adapters/gemini/gemini-extension.json +4 -0
- package/adapters/gemini/references/forge-config-schema.json +25 -3
- package/adapters/gemini/references/pipeline-state-schema.json +3 -2
- package/adapters/gemini/references/portable-root.md +2 -2
- package/adapters/gemini/references/process-overview.md +10 -0
- package/adapters/gemini/references/shared-conventions.md +14 -9
- package/adapters/gemini/references/stage-exit-protocol.md +99 -0
- package/adapters/gemini/scripts/epic-manifest.py +10 -0
- package/adapters/gemini/scripts/forge-bootstrap.py +94 -16
- package/adapters/gemini/scripts/forge-init.sh +7 -1
- package/adapters/gemini/scripts/forge-session.py +175 -30
- package/adapters/gemini/skills/forge/forge.md +33 -19
- package/adapters/gemini/skills/forge-0-epic/forge-0-epic.md +21 -16
- package/adapters/gemini/skills/forge-0-epic/references/edit-mode.md +6 -4
- package/adapters/gemini/skills/forge-0-epic/references/epic-manifest-subcommands.md +1 -1
- package/adapters/gemini/skills/forge-1-prd/forge-1-prd.md +14 -4
- package/adapters/gemini/skills/forge-2-tech/forge-2-tech.md +15 -4
- package/adapters/gemini/skills/forge-3-specs/forge-3-specs.md +14 -3
- package/adapters/gemini/skills/forge-4-backlog/forge-4-backlog.md +16 -5
- package/adapters/gemini/skills/forge-5-loop/forge-5-loop.md +21 -23
- package/adapters/gemini/skills/forge-5-loop/references/result-reporting.md +10 -5
- package/adapters/gemini/skills/forge-6-docs/forge-6-docs.md +6 -6
- package/adapters/gemini/skills/forge-bootstrap/forge-bootstrap.md +4 -4
- package/adapters/gemini/skills/forge-fix/forge-fix.md +27 -6
- package/adapters/gemini/skills/forge-guide/forge-guide.md +188 -0
- package/adapters/gemini/skills/forge-init/forge-init.md +28 -1
- package/adapters/gemini/skills/forge-verify/forge-verify.md +45 -14
- package/adapters/gemini/skills/forge-verify/references/verification-checklists.md +1 -1
- package/dist/apply.js +34 -8
- package/dist/cli.js +40 -4
- package/dist/fsutil.d.ts +0 -12
- package/dist/fsutil.js +10 -1
- package/dist/manifest.d.ts +1 -1
- package/dist/plan.js +22 -2
- package/dist/rauf.d.ts +4 -4
- package/dist/rauf.js +3 -3
- package/dist/report.js +1 -1
- package/dist/types.d.ts +1 -1
- package/package.json +1 -1
|
@@ -0,0 +1,188 @@
|
|
|
1
|
+
---
|
|
2
|
+
# GENERATED — DO NOT EDIT. Source: skills/forge-guide/SKILL.md. Regenerate: python3 scripts/build-adapters.py
|
|
3
|
+
name: forge-guide
|
|
4
|
+
description: Explain what feature-forge is, when to use it, how to configure it, and its best practices — advisory guidance, not stage execution. Use when the user or another agent asks what feature-forge is, whether/when to adopt it, how the pipeline works conceptually, how to set up or configure forge.config.json, or for usage tips and best practices. Do NOT trigger to RUN a pipeline stage (use forge-1-prd … forge-6-docs), to show a specific feature's status (use forge), or for general software questions unrelated to feature-forge.
|
|
5
|
+
---
|
|
6
|
+
|
|
7
|
+
# Feature Forge — Usage & Best-Practices Guide
|
|
8
|
+
|
|
9
|
+
You are the **guide** for feature-forge: an advisor, not an operator. Your job is to
|
|
10
|
+
explain *what* forge is, *when* to use it, *how* to configure it, and *what the best
|
|
11
|
+
practices are* — in plain language, grounded in the repo's own docs. You do **not**
|
|
12
|
+
run pipeline stages; when the user is ready to act, point them at the right skill.
|
|
13
|
+
|
|
14
|
+
## How to answer
|
|
15
|
+
|
|
16
|
+
1. Identify the topic (the argument, or infer from the question).
|
|
17
|
+
2. **Ground yourself in the canonical source before answering** — read the mapped
|
|
18
|
+
reference file(s) below rather than answering from memory. These are the source
|
|
19
|
+
of truth and stay current as the pipeline evolves.
|
|
20
|
+
3. Answer concisely, then end with the concrete next command (`/feature-forge:forge-*`)
|
|
21
|
+
or doc pointer the user should go to.
|
|
22
|
+
|
|
23
|
+
| Topic | Read first |
|
|
24
|
+
|-------|-----------|
|
|
25
|
+
| Pipeline architecture, stage-by-stage flow | `references/process-overview.md` |
|
|
26
|
+
| Cross-stage conventions (naming, state, git, branch, epic injection) | `references/shared-conventions.md` |
|
|
27
|
+
| Config keys + defaults | `references/forge-config-schema.json` |
|
|
28
|
+
| Stack detection / language profiles | `references/stack-resolution.md`, `references/stacks/*.md` |
|
|
29
|
+
| Loop runner interface, signals & version gate | `references/ralph-loop-contract.md` |
|
|
30
|
+
| Deep dives, glossary, troubleshooting | `references/process-overview.md`, `references/shared-conventions.md` |
|
|
31
|
+
|
|
32
|
+
Those `references/` files are the guaranteed grounding path — they ship in every install.
|
|
33
|
+
The project's `README.md`, `COMPATIBILITY.md`, and the hosted docs site are richer but are
|
|
34
|
+
**not** part of an installed bundle, so treat them as optional enrichment (offer the docs-site
|
|
35
|
+
URL to a human) and never block an answer on opening them — fall back to the `references/`
|
|
36
|
+
files and your own knowledge.
|
|
37
|
+
|
|
38
|
+
Do NOT actually invoke stage skills or write files — this skill only explains and directs.
|
|
39
|
+
|
|
40
|
+
## What feature-forge is
|
|
41
|
+
|
|
42
|
+
A **feature development pipeline** that refines a vague idea into shipped code through
|
|
43
|
+
discrete, auditable stages — like a compiler for features. Each stage narrows scope and
|
|
44
|
+
adds structure, reading the previous stage's artifacts as standalone contracts:
|
|
45
|
+
|
|
46
|
+
- **PRD** — *what* to build (requirements only, no technology).
|
|
47
|
+
- **Tech spec** — *how* to build it (design, grounded in the codebase).
|
|
48
|
+
- **Implementation specs** — build-ready detail (types, signatures, contracts, tests).
|
|
49
|
+
- **Backlog** — self-contained work items for autonomous execution.
|
|
50
|
+
- **Loop** — a fresh-context runner implements each item, tests, and commits.
|
|
51
|
+
- **Docs** — architecture reference generated from the real implementation.
|
|
52
|
+
|
|
53
|
+
`REQ-XXX-NN` requirement IDs form a traceability spine from PRD through implementation.
|
|
54
|
+
**Verification gates** are available after any stage and run clean-room in a fresh subagent.
|
|
55
|
+
|
|
56
|
+
## When to use it — and when not
|
|
57
|
+
|
|
58
|
+
**Use forge when:** you have a well-defined feature or small epic to ship; you want
|
|
59
|
+
requirements captured cleanly before coding; you value traceability, thorough spec
|
|
60
|
+
verification, and autonomous loop execution with fresh context per item; you want
|
|
61
|
+
reproducible, auditable artifacts.
|
|
62
|
+
|
|
63
|
+
**Skip forge when:** the work is an exploratory spike with still-fluid requirements
|
|
64
|
+
(though Stage 1's interview can help clarify them); it's a one-line bug fix or trivial
|
|
65
|
+
patch where pipeline overhead isn't justified; or you're only extending mature code
|
|
66
|
+
along established patterns, where new specs become noise.
|
|
67
|
+
|
|
68
|
+
**Anti-patterns to warn against:** skipping the PRD and jumping to tech spec (the value
|
|
69
|
+
comes from separating *what* from *how*); treating specs as living contracts (they're
|
|
70
|
+
pre-implementation — drop an `AGENTS.md`/`CLAUDE.md` in the specs dir telling agents to
|
|
71
|
+
ignore drift); forcing an epic for a single feature; relying on conversation memory to
|
|
72
|
+
carry context across stages instead of reading upstream artifacts.
|
|
73
|
+
|
|
74
|
+
## The pipeline at a glance
|
|
75
|
+
|
|
76
|
+
| Stage | Skill | Produces |
|
|
77
|
+
|-------|-------|----------|
|
|
78
|
+
| 0 (optional) | `forge-0-epic` | `epic-manifest.json` + `EPIC.md` (members, deps, `exposes`/`consumes` contracts) |
|
|
79
|
+
| 1 | `forge-1-prd` | `PRD.md` with `REQ-*` IDs |
|
|
80
|
+
| 2 | `forge-2-tech` | `tech-spec.md`; detects stack + test/typecheck commands |
|
|
81
|
+
| 3 | `forge-3-specs` | numbered spec suite + `TRACEABILITY.md` |
|
|
82
|
+
| 4 | `forge-4-backlog` | `backlog.json` (self-contained items) |
|
|
83
|
+
| 5 | `forge-5-loop` | implemented code, tested + committed per item |
|
|
84
|
+
| 6 | `forge-6-docs` | architecture docs from the real code |
|
|
85
|
+
| any | `forge-verify` → `forge-fix` | findings report → applied fixes |
|
|
86
|
+
| any | `forge` | status dashboard / navigator |
|
|
87
|
+
|
|
88
|
+
Drive the whole thing with the **navigator**: `/feature-forge:forge <feature>` shows the
|
|
89
|
+
current stage and offers the next; with `autoInvokeNextStage` it launches it directly.
|
|
90
|
+
|
|
91
|
+
## Setup & configuration
|
|
92
|
+
|
|
93
|
+
**First-time setup:** `/feature-forge:forge-init` (existing repo) creates `forge.config.json`
|
|
94
|
+
with defaults. `/feature-forge:forge-bootstrap` scaffolds a *greenfield* (empty) repo to a
|
|
95
|
+
green baseline. On non-Claude agents, install via `npx @garygentry/feature-forge install`.
|
|
96
|
+
|
|
97
|
+
**Key `forge.config.json` knobs** (authoritative list: `references/forge-config-schema.json`):
|
|
98
|
+
|
|
99
|
+
- **Paths** — `specsDir` (`./specs`), `docsDir` (`./docs/architecture`), `backlogDir`.
|
|
100
|
+
- **Git** — `gitCommitAfterStage` (true), `commitPrefix` (`forge`); commits use a two-commit
|
|
101
|
+
protocol so the stage's commit hash is recorded in state without `--amend`.
|
|
102
|
+
- **Branch** — `branchPerFeature` (true), `branchPrefix` (`forge/`): isolate each feature.
|
|
103
|
+
- **Stack** — `stack`, `typeCheckCommand`, `testCommand`: null until Stage 2 auto-detects them.
|
|
104
|
+
- **Context** — `contextWindowTokens`, `contextWarnThreshold` (0.7): the navigator warns to
|
|
105
|
+
clear your session / start a fresh session past this fullness. On 1M-context models set `contextWindowTokens` explicitly.
|
|
106
|
+
- **Verification** — `autoVerify` (false), `autoVerifyStages`, `autoFix` (false).
|
|
107
|
+
- **Stage flow** — `autoInvokeNextStage` (true on Claude, print-only elsewhere).
|
|
108
|
+
- **Loop** — `loopRunner` block (binary, command templates, version gate, agent selection);
|
|
109
|
+
defaults to **rauf** when absent. `workspaces` supports monorepos.
|
|
110
|
+
|
|
111
|
+
## Verification gates
|
|
112
|
+
|
|
113
|
+
`forge-verify <feature>` dispatches the read-only `forge-verifier` subagent to find gaps,
|
|
114
|
+
inconsistencies, and quality issues; it writes a findings doc, and `forge-fix` applies them.
|
|
115
|
+
Because verification runs in a **fresh subagent**, it's clean-room by construction — it never
|
|
116
|
+
needs a clear your session / start a fresh session, and it's safe to automate with `autoVerify: true` (fixing stays human-gated
|
|
117
|
+
unless `autoFix: true`). **Always verify before Stage 5 (the loop)** — catching errors in
|
|
118
|
+
specs/backlog is far cheaper than mid-loop. Verifying after PRD and after backlog is also
|
|
119
|
+
recommended. A findings pass is fresh only while the artifact `version` matches what was
|
|
120
|
+
verified; revise upstream and downstream re-verifies.
|
|
121
|
+
|
|
122
|
+
## Context management
|
|
123
|
+
|
|
124
|
+
Each stage reads upstream artifacts as standalone contracts, so you can (and usually should)
|
|
125
|
+
clear your session / start a fresh session between them:
|
|
126
|
+
|
|
127
|
+
- **Clear** between PRD → tech, tech → specs, specs → backlog, backlog → loop.
|
|
128
|
+
- **Stay warm** mid-interview (PRD, tech spec) — the interview needs a continuous thread.
|
|
129
|
+
- **No clear needed** for any → verify (runs in a fresh subagent).
|
|
130
|
+
|
|
131
|
+
The navigator warns when the session passes `contextWarnThreshold` (default 70% full).
|
|
132
|
+
|
|
133
|
+
## Epics (large changes)
|
|
134
|
+
|
|
135
|
+
Use Stage 0 only when a change naturally splits into **several interdependent features** that
|
|
136
|
+
must agree on interfaces. `forge-0-epic` produces `epic-manifest.json` + `EPIC.md` with a
|
|
137
|
+
per-member charter, `exposes`/`consumes` contracts, and `dependsOn` edges. Each member then
|
|
138
|
+
runs the normal pipeline with epic context injected. At Stage 5 a **dependency gate** warns if
|
|
139
|
+
a member's dependencies are unmet. Epic support is purely additive — single-feature flows are
|
|
140
|
+
unchanged. Re-run `forge-0-epic` on an existing epic to enter edit mode.
|
|
141
|
+
|
|
142
|
+
## The loop
|
|
143
|
+
|
|
144
|
+
Stage 5 runs a configurable runner (**rauf** by default) that gives each backlog item a fresh
|
|
145
|
+
agent session — implement → run the verification command → commit on pass. This is why items
|
|
146
|
+
must be truly **self-contained**; context bleed breaks the model. Per-item signals:
|
|
147
|
+
|
|
148
|
+
- `RAUF_DONE` — item passed; loop continues.
|
|
149
|
+
- `RAUF_BLOCKED` — missing dependency / unclear requirement; set aside, loop continues others.
|
|
150
|
+
- `RAUF_NEEDS_HUMAN` — decision or secret needed; set aside, loop continues.
|
|
151
|
+
|
|
152
|
+
The loop doesn't pause on blocked items. Supply what's missing, then `rauf resume <path>` to
|
|
153
|
+
retry set-aside items. The runner refuses to start with a **dirty working tree** and enforces
|
|
154
|
+
a minimum runner version (the version gate is described in `references/ralph-loop-contract.md`).
|
|
155
|
+
|
|
156
|
+
## Best practices & gotchas
|
|
157
|
+
|
|
158
|
+
- Feature name is **required** for every stage command — never guess or infer it.
|
|
159
|
+
- Verify **before the loop**; a bad spec is cheap to fix now, expensive mid-loop.
|
|
160
|
+
- Keep backlog items self-contained — the loop has zero memory across items.
|
|
161
|
+
- Let stages commit for you; don't hand-edit `.pipeline-state.json` or backlog status.
|
|
162
|
+
- Re-running an upstream stage marks downstream stages **stale** — re-run them rather than
|
|
163
|
+
reaching for `--force`, which skips prerequisite checks and should be rare.
|
|
164
|
+
- Specs are pre-implementation artifacts, not living docs — don't cite them from generated code.
|
|
165
|
+
- Use the navigator (`/feature-forge:forge <feature>`) to orient; use `forge-verify` to inspect.
|
|
166
|
+
|
|
167
|
+
## Troubleshooting starters
|
|
168
|
+
|
|
169
|
+
- **Stage 5 won't start:** backlog exists and is verified? runner installed and ≥ min version?
|
|
170
|
+
working tree clean? See `references/ralph-loop-contract.md` for the runner contract and version gate.
|
|
171
|
+
- **Loop stopped mid-run:** check the signal — `BLOCKED`/`NEEDS_HUMAN` items are set aside, not
|
|
172
|
+
failures; the loop keeps going.
|
|
173
|
+
- **Downstream flagged stale:** an upstream stage was revised; re-run the downstream stage.
|
|
174
|
+
- **Where am I?** `/feature-forge:forge <feature>` renders the full pipeline status.
|
|
175
|
+
|
|
176
|
+
For anything deeper, ground yourself in `references/process-overview.md` and
|
|
177
|
+
`references/shared-conventions.md`, and point the *user* at the hosted docs site —
|
|
178
|
+
<https://garygentry.github.io/feature-forge/> — for the full guides and glossary.
|
|
179
|
+
|
|
180
|
+
---
|
|
181
|
+
|
|
182
|
+
## Host execution notes (Codex)
|
|
183
|
+
|
|
184
|
+
This skill was authored Claude-first; the body above refers to "the host's question mechanism", "the host's subagent mechanism", and "the host's background-execution mechanism". On Codex:
|
|
185
|
+
|
|
186
|
+
- **User input:** Codex has no structured question tool — ask the question directly and wait for the user's reply before proceeding. Never skip a required question or assume an answer.
|
|
187
|
+
- **Subagents:** spawn a Codex subagent using the named custom agent under `.codex/agents/<name>.toml`. Codex spawns a subagent only when explicitly asked; if the custom agent is unavailable, run that step inline yourself.
|
|
188
|
+
- **Background / monitoring:** run long-lived runner commands in your shell session and report progress as it arrives — there is no Claude-style background or monitoring tool to arm.
|
|
@@ -9,7 +9,7 @@ description: Initialize feature-forge configuration in the current project. Use
|
|
|
9
9
|
Run the initialization script to create `forge.config.json` with default settings:
|
|
10
10
|
|
|
11
11
|
```bash
|
|
12
|
-
R="$(for d in "$HOME"/.claude/skills/feature-forge "$HOME"/.claude/plugins/*/feature-forge "$HOME"/.agents/skills/feature-forge ./.agents/skills/feature-forge; do [ -x "$d/scripts/forge-root.sh" ] && exec "$d/scripts/forge-root.sh"; done)"
|
|
12
|
+
R="$(bash -c 'for d in "$HOME"/.claude/skills/feature-forge "$HOME"/.claude/plugins/*/feature-forge "$HOME"/.agents/skills/feature-forge ./.agents/skills/feature-forge; do [ -x "$d/scripts/forge-root.sh" ] && exec "$d/scripts/forge-root.sh"; done')"
|
|
13
13
|
[ -n "$R" ] || { echo "feature-forge: cannot locate plugin root" >&2; exit 1; }
|
|
14
14
|
bash "$R/scripts/forge-init.sh"
|
|
15
15
|
```
|
|
@@ -26,9 +26,36 @@ After initialization, the config file will contain defaults for:
|
|
|
26
26
|
- `autoInvokeNextStage`: `true` (the navigator auto-starts the next stage after you confirm; set `false` to only print the command)
|
|
27
27
|
- `contextWindowTokens`: `null` (the navigator infers the context window; set to your model's window, e.g. `1000000` for a 1M-context model, for accurate context-usage advice)
|
|
28
28
|
- `contextWarnThreshold`: `0.7` (fraction of the window past which the navigator suggests a clean session)
|
|
29
|
+
- `autoVerify`: `false` (set `true` to run `forge-verify` automatically after each stage completes)
|
|
30
|
+
- `autoVerifyStages`: `{}` (per-stage overrides for `autoVerify`)
|
|
31
|
+
- `autoFix`: `false` (set `true` to chain `forge-fix` after an auto-verify finds issues)
|
|
29
32
|
|
|
30
33
|
If `forge.config.json` already exists, the script will not overwrite it.
|
|
31
34
|
|
|
35
|
+
## Offer auto-verify
|
|
36
|
+
|
|
37
|
+
The template writes `autoVerify: false`. After the config is created (and only when the script
|
|
38
|
+
actually created it — skip this if it reported the file already exists), offer to turn
|
|
39
|
+
auto-verify on, then write the choice back into `forge.config.json`.
|
|
40
|
+
|
|
41
|
+
If the host's question mechanism is available, ask exactly one question:
|
|
42
|
+
|
|
43
|
+
> **Enable auto-verify?** Verification runs in a clean-room subagent after each stage
|
|
44
|
+
> completes — it never needs a clear your session / start a fresh session and only returns a compact digest to your session.
|
|
45
|
+
> **Recommended: on.** (Change later by editing `autoVerify` in `forge.config.json`.)
|
|
46
|
+
|
|
47
|
+
Options: **Enable (recommended)** / **Leave off**.
|
|
48
|
+
|
|
49
|
+
- On **Enable**: patch `"autoVerify": false` → `"autoVerify": true` in the generated
|
|
50
|
+
`forge.config.json` in place, preserving formatting and every other key.
|
|
51
|
+
- On **Leave off**: leave the config as written (`autoVerify: false`).
|
|
52
|
+
|
|
53
|
+
If the host lacks a structured question tool but can still prompt the user (e.g. Codex asks in
|
|
54
|
+
plain text), use that — ask the one question directly and wait for the reply; it is the same
|
|
55
|
+
choice, just rendered differently. Only when the host has **no** way to ask at all (a fully
|
|
56
|
+
non-interactive / headless run) do you skip the prompt: leave `autoVerify: false` and print the
|
|
57
|
+
one-line note `Set "autoVerify": true in forge.config.json to verify automatically after each stage.`
|
|
58
|
+
|
|
32
59
|
After initialization, start the pipeline with `/feature-forge:forge-1-prd <feature-name>`.
|
|
33
60
|
|
|
34
61
|
---
|
|
@@ -47,7 +47,7 @@ Pick based on how many checks the mode carries (see the per-mode totals in Step
|
|
|
47
47
|
|
|
48
48
|
The verifier(s) are read-only — they return findings as their response; **you** (the
|
|
49
49
|
parent) assemble and write the single document to
|
|
50
|
-
`{
|
|
50
|
+
`{resolvedFeatureDir}/.verification/VERIFY-{mode}-{YYYY-MM-DD}.md`. When you fanned out:
|
|
51
51
|
1. Concatenate all instances' findings and **renumber `V-NNN` IDs uniquely** across the
|
|
52
52
|
merged set.
|
|
53
53
|
2. **Dedup** overlaps — when two instances flag the same file+location+issue (e.g. a
|
|
@@ -72,15 +72,39 @@ without subagents), fall back to running verification inline in the current sess
|
|
|
72
72
|
|
|
73
73
|
**Inline execution guidance:** If running inline (not as subagent), process verification checklists one category at a time to manage context pressure. Load only the artifacts needed for each category, verify, summarize findings, then move to the next category.
|
|
74
74
|
|
|
75
|
+
### Require-clean (`auto`) mode — unattended auto-verify
|
|
76
|
+
|
|
77
|
+
When the navigator auto-invokes this skill (its `autoVerify` path), it passes a
|
|
78
|
+
**require-clean** signal (e.g. args include `--require-clean`, or the invocation is
|
|
79
|
+
described as auto-verify). In this mode the clean-room guarantee is load-bearing: the
|
|
80
|
+
whole reason auto-verify is safe to run without a clear your session / start a fresh session is that the `forge-verifier`
|
|
81
|
+
subagent inherits none of the dispatching session's context. Running inline would break
|
|
82
|
+
that — it would consume the dispatching session's context and invalidate the no-clear
|
|
83
|
+
justification.
|
|
84
|
+
|
|
85
|
+
So in require-clean mode, **do NOT fall back to inline execution**. If the host's subagent mechanism
|
|
86
|
+
or `forge-verifier` subagent is not dispatchable, return a **sentinel** instead of doing
|
|
87
|
+
any work:
|
|
88
|
+
|
|
89
|
+
> `CLEAN_ROOM_UNAVAILABLE: forge-verifier subagent not dispatchable — verify not run.`
|
|
90
|
+
|
|
91
|
+
Do not analyze artifacts, do not write a findings document, and do not touch pipeline
|
|
92
|
+
state. The navigator detects this sentinel and degrades to its manual verify gate (Tier
|
|
93
|
+
2/3), so verify state stays outstanding and the stage is never marked verified on false
|
|
94
|
+
assurance. **Manual / interactive invocation** (the normal `/feature-forge:forge-verify`
|
|
95
|
+
path, no require-clean signal) keeps the inline fallback above unchanged.
|
|
96
|
+
|
|
75
97
|
## Prerequisites
|
|
76
98
|
|
|
77
99
|
Read and follow `references/shared-conventions.md` for feature name validation, configuration reading, and force mode handling before proceeding.
|
|
78
100
|
|
|
101
|
+
Resolve the feature directory via the **Feature Directory Resolution** block in `references/shared-conventions.md` (so a standalone feature resolves to its flat `{specsDir}/{feature}/` path exactly as today, and an epic member resolves to its nested `{specsDir}/{epic}/{feature}/` path). Use the resulting `{resolvedFeatureDir}` everywhere this skill reads or writes a per-feature artifact or state file — the `{specsDir}/{feature}/…` forms below are shorthand for the resolved path, not a literal flat layout. This does not apply to **epic mode**, whose paths are epic-scoped (`{specsDir}/{epic}/…`) by design.
|
|
102
|
+
|
|
79
103
|
**Turn structure reminder:** Output analysis/context as text, then route ALL questions through the host's question mechanism. Never embed questions in text output — the user will not be prompted and the session will stall.
|
|
80
104
|
|
|
81
105
|
## Step 1: Read Configuration and Determine Mode
|
|
82
106
|
|
|
83
|
-
Read `{
|
|
107
|
+
Read `{resolvedFeatureDir}/.pipeline-state.json` to understand current pipeline state.
|
|
84
108
|
|
|
85
109
|
### Mode Selection
|
|
86
110
|
|
|
@@ -100,16 +124,16 @@ If ambiguous, use the host's question mechanism to ask which stage to verify.
|
|
|
100
124
|
Load into context ALL artifacts for this feature based on mode:
|
|
101
125
|
|
|
102
126
|
**For prd mode:**
|
|
103
|
-
- `{
|
|
127
|
+
- `{resolvedFeatureDir}/PRD.md`
|
|
104
128
|
|
|
105
129
|
**For tech mode:**
|
|
106
|
-
- `{
|
|
107
|
-
- `{
|
|
130
|
+
- `{resolvedFeatureDir}/PRD.md`
|
|
131
|
+
- `{resolvedFeatureDir}/tech-spec.md`
|
|
108
132
|
|
|
109
133
|
**For specs mode:**
|
|
110
|
-
- `{
|
|
111
|
-
- `{
|
|
112
|
-
- `{
|
|
134
|
+
- `{resolvedFeatureDir}/PRD.md`
|
|
135
|
+
- `{resolvedFeatureDir}/tech-spec.md`
|
|
136
|
+
- `{resolvedFeatureDir}/##-*.md` (all implementation specs)
|
|
113
137
|
|
|
114
138
|
**For backlog mode:**
|
|
115
139
|
- All of the above, PLUS
|
|
@@ -150,7 +174,7 @@ Every finding must include:
|
|
|
150
174
|
|
|
151
175
|
## Step 4: Write Findings Document
|
|
152
176
|
|
|
153
|
-
Ensure the `.verification/` subdirectory exists, then write findings to `{
|
|
177
|
+
Ensure the `.verification/` subdirectory exists, then write findings to `{resolvedFeatureDir}/.verification/VERIFY-{mode}-{YYYY-MM-DD}.md`.
|
|
154
178
|
|
|
155
179
|
**For epic mode**, the target is `{specsDir}/{epic}/.verification/VERIFY-epic-{YYYY-MM-DD}.md` (the same format, with `{mode}=epic`).
|
|
156
180
|
|
|
@@ -186,9 +210,16 @@ Do NOT embed this question in your text output.
|
|
|
186
210
|
|
|
187
211
|
Write pipeline state conforming to `references/pipeline-state-schema.json`.
|
|
188
212
|
|
|
189
|
-
Update `{
|
|
190
|
-
- Set the relevant verify entry status to `findings-reported`
|
|
213
|
+
Update `{resolvedFeatureDir}/.pipeline-state.json`:
|
|
214
|
+
- Set the relevant verify entry status to `findings-reported` (or `passed` when there
|
|
215
|
+
are zero findings)
|
|
191
216
|
- Record `findingsFile`, `findingsCount`, `verifiedAt`
|
|
217
|
+
- Record `verifiedStageVersion` = the current `version` of the production stage entry
|
|
218
|
+
this verify covers (e.g. verifying `tech` → `stages["forge-2-tech"].version`). This
|
|
219
|
+
feeds the navigator's freshness ledger: a later revision to that artifact bumps its
|
|
220
|
+
`version`, so the recorded value no longer matches and auto-verify re-fires. Omitting
|
|
221
|
+
this leaves the verify looking stale (safe: the navigator re-verifies rather than
|
|
222
|
+
skips).
|
|
192
223
|
|
|
193
224
|
Do NOT mark as `findings-applied` — that happens after the fix pass.
|
|
194
225
|
|
|
@@ -213,13 +244,13 @@ Do NOT mark as `findings-applied` — that happens after the fix pass.
|
|
|
213
244
|
- Don't verify things that are intentionally left open (check the PRD's "Open Questions" section).
|
|
214
245
|
- If you find zero issues, say so honestly. Don't manufacture findings to seem thorough. But zero findings on a complex feature is suspicious — double-check.
|
|
215
246
|
- The findings document must be self-contained. A fresh agent reading it should be able to apply every fix without needing conversational context from this session.
|
|
216
|
-
- For backlog verification, also run the loop runner's validate command (resolve `loopRunner` from `forge.config.json`, default rauf: `rauf backlog validate . --backlog {backlogDir} --specs-dir {
|
|
247
|
+
- For backlog verification, also run the loop runner's validate command (resolve `loopRunner` from `forge.config.json`, default rauf: `rauf backlog validate . --backlog {backlogDir} --specs-dir {resolvedFeatureDir} --json`). Include any findings it reports (exit 1) as verification findings; if the runner isn't installed yet (command missing), note that backlog validation was skipped rather than failing.
|
|
217
248
|
- For specs verification, also run the deterministic traceability validator to supplement agent-driven traceability checks. Include any uncovered requirements or orphaned references as findings:
|
|
218
249
|
|
|
219
250
|
```bash
|
|
220
|
-
R="$(for d in "$HOME"/.claude/skills/feature-forge "$HOME"/.claude/plugins/*/feature-forge "$HOME"/.agents/skills/feature-forge ./.agents/skills/feature-forge; do [ -x "$d/scripts/forge-root.sh" ] && exec "$d/scripts/forge-root.sh"; done)"
|
|
251
|
+
R="$(bash -c 'for d in "$HOME"/.claude/skills/feature-forge "$HOME"/.claude/plugins/*/feature-forge "$HOME"/.agents/skills/feature-forge ./.agents/skills/feature-forge; do [ -x "$d/scripts/forge-root.sh" ] && exec "$d/scripts/forge-root.sh"; done')"
|
|
221
252
|
[ -n "$R" ] || { echo "feature-forge: cannot locate plugin root" >&2; exit 1; }
|
|
222
|
-
python3 "$R/scripts/validate-traceability.py" {
|
|
253
|
+
python3 "$R/scripts/validate-traceability.py" {resolvedFeatureDir}/PRD.md {resolvedFeatureDir}/ --json
|
|
223
254
|
```
|
|
224
255
|
|
|
225
256
|
---
|
|
@@ -192,7 +192,7 @@ findings to E01/E02/E03/E08. Then perform the judgment checks E04–E07 by readi
|
|
|
192
192
|
manifest, EPIC.md, and completed members' specs.
|
|
193
193
|
|
|
194
194
|
```bash
|
|
195
|
-
R="$(for d in "$HOME"/.claude/skills/feature-forge "$HOME"/.claude/plugins/*/feature-forge "$HOME"/.agents/skills/feature-forge ./.agents/skills/feature-forge; do [ -x "$d/scripts/forge-root.sh" ] && exec "$d/scripts/forge-root.sh"; done)"
|
|
195
|
+
R="$(bash -c 'for d in "$HOME"/.claude/skills/feature-forge "$HOME"/.claude/plugins/*/feature-forge "$HOME"/.agents/skills/feature-forge ./.agents/skills/feature-forge; do [ -x "$d/scripts/forge-root.sh" ] && exec "$d/scripts/forge-root.sh"; done')"
|
|
196
196
|
[ -n "$R" ] || { echo "feature-forge: cannot locate plugin root" >&2; exit 1; }
|
|
197
197
|
python3 "$R/scripts/epic-manifest.py" validate "{epic}" --specs-dir "{specsDir}" --json
|
|
198
198
|
```
|
|
@@ -16,7 +16,8 @@
|
|
|
16
16
|
},
|
|
17
17
|
"backlogDir": {
|
|
18
18
|
"type": ["string", "null"],
|
|
19
|
-
"
|
|
19
|
+
"default": null,
|
|
20
|
+
"description": "Optional override for the backlog root. If set, the backlog is written to {backlogDir}/{feature}/backlog.json — the {feature} subdirectory is always composed in so multi-feature epics never collide. Default behavior (null): backlog.json is written to {specsDir}/{feature}/backlog.json with the feature specs."
|
|
20
21
|
},
|
|
21
22
|
"gitCommitAfterStage": {
|
|
22
23
|
"type": "boolean",
|
|
@@ -61,6 +62,27 @@
|
|
|
61
62
|
"default": true,
|
|
62
63
|
"description": "When true (default), the /feature-forge:forge navigator auto-invokes the next pipeline stage via the Skill tool after the user confirms it, instead of only printing the command to copy. Set false to keep the old copy-paste behavior (the navigator suggests the command but never launches it). Ignored on non-Claude hosts, which always fall back to printing the command."
|
|
63
64
|
},
|
|
65
|
+
"autoVerify": {
|
|
66
|
+
"type": "boolean",
|
|
67
|
+
"default": false,
|
|
68
|
+
"description": "When true, the /feature-forge:forge navigator automatically runs forge-verify after a stage completes, with no prompt. forge-verify runs in a fresh forge-verifier subagent (clean-room), so it never needs a context clear and costs the current session only a compact findings digest. Default false preserves today's manual-gate behavior. Ignored on non-Claude hosts, which always fall back to printing the verify command. Per-stage overrides in autoVerifyStages take precedence."
|
|
69
|
+
},
|
|
70
|
+
"autoVerifyStages": {
|
|
71
|
+
"type": "object",
|
|
72
|
+
"default": {},
|
|
73
|
+
"description": "Per-stage overrides for autoVerify. Maps a production stage id to a boolean; the effective value for a stage is autoVerifyStages[stage] if present, else autoVerify. Keys are constrained to the five verify-capable stages so a typo (e.g. 'forge-1-prod') is a schema error, not a silent no-op. forge-6-docs has no verify step and is not a valid key.",
|
|
74
|
+
"propertyNames": {
|
|
75
|
+
"enum": ["forge-1-prd", "forge-2-tech", "forge-3-specs", "forge-4-backlog", "forge-5-loop"]
|
|
76
|
+
},
|
|
77
|
+
"additionalProperties": {
|
|
78
|
+
"type": "boolean"
|
|
79
|
+
}
|
|
80
|
+
},
|
|
81
|
+
"autoFix": {
|
|
82
|
+
"type": "boolean",
|
|
83
|
+
"default": false,
|
|
84
|
+
"description": "When true, the navigator chains forge-fix automatically after an auto-verify that finds issues. Honored ONLY when auto-verify is effectively on for the stage, and ONLY when preconditions hold (findings doc has zero unresolved decision points, working tree is clean, and a mandatory re-verify passes) — otherwise it falls back to surfacing the findings digest and prompting. Default false keeps fixing human-gated."
|
|
85
|
+
},
|
|
64
86
|
"contextWindowTokens": {
|
|
65
87
|
"type": ["integer", "null"],
|
|
66
88
|
"default": null,
|
|
@@ -190,8 +212,8 @@
|
|
|
190
212
|
},
|
|
191
213
|
"installHint": {
|
|
192
214
|
"type": "string",
|
|
193
|
-
"default": "Provision rauf for a multi-agent setup with the cross-agent installer: `npx @garygentry/feature-forge install` (records the pinned @garygentry/rauf@0.
|
|
194
|
-
"description": "Shown when the runner BINARY is missing or too old (version gate fails, minRunnerVersion floor) — how to obtain/upgrade the CLI itself. Names two distinct binary-provisioning paths: (1) the cross-agent installer (`npx @garygentry/feature-forge install`, the multi-agent provisioning path that pins @garygentry/rauf@0.
|
|
215
|
+
"default": "Provision rauf for a multi-agent setup with the cross-agent installer: `npx @garygentry/feature-forge install` (records the pinned @garygentry/rauf@0.12.0 default). Or install/upgrade just the rauf CLI: `npx @garygentry/rauf@0.12.0 --version`, or `curl -fsSL https://raw.githubusercontent.com/garygentry/rauf/main/scripts/install-binary.sh | bash`.",
|
|
216
|
+
"description": "Shown when the runner BINARY is missing or too old (version gate fails, minRunnerVersion floor) — how to obtain/upgrade the CLI itself. Names two distinct binary-provisioning paths: (1) the cross-agent installer (`npx @garygentry/feature-forge install`, the multi-agent provisioning path that pins @garygentry/rauf@0.12.0), and (2) the direct rauf-CLI install/upgrade one-liner. Distinct from setupHint (which installs per-project artifacts); a version-gate failure is ALWAYS this hint, never setupHint."
|
|
195
217
|
},
|
|
196
218
|
"schemaVersion": {
|
|
197
219
|
"type": "string",
|
|
@@ -80,7 +80,7 @@
|
|
|
80
80
|
"completedAt": { "type": ["string", "null"], "format": "date-time" },
|
|
81
81
|
"commitHash": {
|
|
82
82
|
"type": ["string", "null"],
|
|
83
|
-
"description": "Git commit SHA
|
|
83
|
+
"description": "Git commit SHA of this stage's artifact commit (Commit 1 of the two-commit Git Commit Protocol in shared-conventions.md). Written by a small follow-up commit so it points at the artifact commit itself, never at an orphaned amend or the hash-recording commit. null until recorded (or when there was no new artifact commit)."
|
|
84
84
|
},
|
|
85
85
|
"basedOnVersions": {
|
|
86
86
|
"type": "object",
|
|
@@ -107,7 +107,8 @@
|
|
|
107
107
|
},
|
|
108
108
|
"verifiedAt": { "type": ["string", "null"], "format": "date-time" },
|
|
109
109
|
"fixedAt": { "type": ["string", "null"], "format": "date-time" },
|
|
110
|
-
"commitHash": { "type": ["string", "null"] }
|
|
110
|
+
"commitHash": { "type": ["string", "null"], "description": "Git commit SHA of the verify/fix artifact commit. Recorded via the two-commit Git Commit Protocol (shared-conventions.md) so it points at the artifact commit, never an orphaned amend." },
|
|
111
|
+
"verifiedStageVersion": { "type": ["integer", "null"], "description": "The production stage's `version` at the moment this verify was resolved. The navigator's freshness ledger compares it to the stage's current `version`: equal means the verify is fresh; a mismatch (artifact revised since) or an absent field (legacy state) means stale, so auto-verify re-fires. Recorded by forge-verify/forge-fix when writing a passed/findings-applied status." }
|
|
111
112
|
}
|
|
112
113
|
}
|
|
113
114
|
}
|
|
@@ -12,7 +12,7 @@ against the fenced block here, byte-for-byte.
|
|
|
12
12
|
## Canonical bootstrap prelude
|
|
13
13
|
|
|
14
14
|
```bash
|
|
15
|
-
R="$(for d in "$HOME"/.claude/skills/feature-forge "$HOME"/.claude/plugins/*/feature-forge "$HOME"/.agents/skills/feature-forge ./.agents/skills/feature-forge; do [ -x "$d/scripts/forge-root.sh" ] && exec "$d/scripts/forge-root.sh"; done)"
|
|
15
|
+
R="$(bash -c 'for d in "$HOME"/.claude/skills/feature-forge "$HOME"/.claude/plugins/*/feature-forge "$HOME"/.agents/skills/feature-forge ./.agents/skills/feature-forge; do [ -x "$d/scripts/forge-root.sh" ] && exec "$d/scripts/forge-root.sh"; done')"
|
|
16
16
|
[ -n "$R" ] || { echo "feature-forge: cannot locate plugin root" >&2; exit 1; }
|
|
17
17
|
```
|
|
18
18
|
|
|
@@ -24,7 +24,7 @@ makes several calls, add the prelude once and reuse `$R` for each. A fresh block
|
|
|
24
24
|
prelude (per-block re-resolution). Worked example:
|
|
25
25
|
|
|
26
26
|
```bash
|
|
27
|
-
R="$(for d in "$HOME"/.claude/skills/feature-forge "$HOME"/.claude/plugins/*/feature-forge "$HOME"/.agents/skills/feature-forge ./.agents/skills/feature-forge; do [ -x "$d/scripts/forge-root.sh" ] && exec "$d/scripts/forge-root.sh"; done)"
|
|
27
|
+
R="$(bash -c 'for d in "$HOME"/.claude/skills/feature-forge "$HOME"/.claude/plugins/*/feature-forge "$HOME"/.agents/skills/feature-forge ./.agents/skills/feature-forge; do [ -x "$d/scripts/forge-root.sh" ] && exec "$d/scripts/forge-root.sh"; done')"
|
|
28
28
|
[ -n "$R" ] || { echo "feature-forge: cannot locate plugin root" >&2; exit 1; }
|
|
29
29
|
python3 "$R/scripts/epic-manifest.py" render-status "{epic}" --specs-dir "{specsDir}" --json
|
|
30
30
|
```
|
|
@@ -36,6 +36,16 @@ The pipeline compiles a fuzzy feature idea into a machine-executable `backlog.js
|
|
|
36
36
|
**Output:** `{specsDir}/{feature}/.verification/VERIFY-<stage>-<timestamp>.md` (includes both findings and a Fix Execution Plan)
|
|
37
37
|
**Method:** Clean-context analysis producing actionable findings with an ordered fix plan.
|
|
38
38
|
|
|
39
|
+
**Manual or automatic.** By default the navigator offers verification after each stage. Because
|
|
40
|
+
it runs in a fresh, read-only subagent (clean-room by construction, never needs a `/clear`), it is
|
|
41
|
+
safe to automate: set `autoVerify: true` (or per-stage via `autoVerifyStages`) in
|
|
42
|
+
`forge.config.json` and the navigator runs `forge-verify` automatically once a stage completes,
|
|
43
|
+
returning only a compact digest to the session. The freshness ledger (verify entries record the
|
|
44
|
+
artifact `version` they ran against) means a stage is re-verified only when its artifact changes,
|
|
45
|
+
and an explicitly `skipped` verify is respected. Fixing stays human-gated unless `autoFix: true` is
|
|
46
|
+
also set, which chains `forge-fix` only when preconditions hold (zero unresolved decisions, clean
|
|
47
|
+
tree, passing re-verify). See `references/shared-conventions.md` for the config keys.
|
|
48
|
+
|
|
39
49
|
After verification, fixes can be applied via:
|
|
40
50
|
- `/feature-forge:forge-fix <feature>` — reads the Fix Execution Plan from the findings document and applies changes (works in any session)
|
|
41
51
|
- Plan mode workflow — enter plan mode, run verify, review plan, exit and execute
|
|
@@ -71,6 +71,9 @@ Extract these config values (use defaults if not present):
|
|
|
71
71
|
- `autoInvokeNextStage` (default: `true` — the `/feature-forge:forge` navigator auto-invokes the next stage via the `Skill` tool after the user confirms; `false` keeps copy-paste behavior. Navigator-only.)
|
|
72
72
|
- `contextWindowTokens` (default: `null` — context window used by the navigator's context-usage check; `null` infers from the session model and falls back to 200000. Set to the model's window, e.g. `1000000` on a 1M model. Navigator-only.)
|
|
73
73
|
- `contextWarnThreshold` (default: `0.7` — fraction of the window past which the navigator recommends a clean session. Navigator-only.)
|
|
74
|
+
- `autoVerify` (default: `false` — when `true`, the navigator auto-runs `forge-verify` after a stage completes, no prompt; runs in a fresh clean-room subagent so it never needs a `/clear` and costs only a compact digest. Navigator-only.)
|
|
75
|
+
- `autoVerifyStages` (default: `{}` — per-stage overrides for `autoVerify`, e.g. `{"forge-1-prd": false}`. Effective value = `autoVerifyStages[stage]` if present, else `autoVerify`. Keys are constrained to the five verify-capable stages; a typo is a config error surfaced as `invalidAutoVerifyKeys`. Navigator-only.)
|
|
76
|
+
- `autoFix` (default: `false` — when `true`, the navigator chains `forge-fix` after an auto-verify that finds issues, but only when auto-verify is on for that stage AND preconditions hold (zero unresolved decisions, clean tree, passing re-verify); otherwise it surfaces a digest and prompts. Navigator-only.)
|
|
74
77
|
- `loopRunner` (optional object — the loop runner to drive; **defaults to rauf** when absent, with every command templated. See `references/forge-config-schema.json` and `references/ralph-loop-contract.md`.)
|
|
75
78
|
|
|
76
79
|
## Feature Directory Resolution
|
|
@@ -78,7 +81,7 @@ Extract these config values (use defaults if not present):
|
|
|
78
81
|
Before any file I/O against a feature's artifacts, resolve its directory through the deterministic helper rather than hardcoding `{specsDir}/{feature}/`. This makes flat (`{specsDir}/{feature}/`) and nested (`{specsDir}/{epic}/{feature}/`) layouts both resolve from a bare feature name (REQ-DIR-03), with standalone features behaving exactly as today.
|
|
79
82
|
|
|
80
83
|
```bash
|
|
81
|
-
R="$(for d in "$HOME"/.claude/skills/feature-forge "$HOME"/.claude/plugins/*/feature-forge "$HOME"/.agents/skills/feature-forge ./.agents/skills/feature-forge; do [ -x "$d/scripts/forge-root.sh" ] && exec "$d/scripts/forge-root.sh"; done)"
|
|
84
|
+
R="$(bash -c 'for d in "$HOME"/.claude/skills/feature-forge "$HOME"/.claude/plugins/*/feature-forge "$HOME"/.agents/skills/feature-forge ./.agents/skills/feature-forge; do [ -x "$d/scripts/forge-root.sh" ] && exec "$d/scripts/forge-root.sh"; done')"
|
|
82
85
|
[ -n "$R" ] || { echo "feature-forge: cannot locate plugin root" >&2; exit 1; }
|
|
83
86
|
resolvedFeatureDir=$(python3 "$R/scripts/epic-manifest.py" \
|
|
84
87
|
resolve "<feature>" --specs-dir "<specsDir>")
|
|
@@ -108,7 +111,7 @@ Whenever a stage creates the specs tree for the first time (the first PRD or epi
|
|
|
108
111
|
Run this after creating the feature/epic directory, before the stage's git commit:
|
|
109
112
|
|
|
110
113
|
```bash
|
|
111
|
-
R="$(for d in "$HOME"/.claude/skills/feature-forge "$HOME"/.claude/plugins/*/feature-forge "$HOME"/.agents/skills/feature-forge ./.agents/skills/feature-forge; do [ -x "$d/scripts/forge-root.sh" ] && exec "$d/scripts/forge-root.sh"; done)"
|
|
114
|
+
R="$(bash -c 'for d in "$HOME"/.claude/skills/feature-forge "$HOME"/.claude/plugins/*/feature-forge "$HOME"/.agents/skills/feature-forge ./.agents/skills/feature-forge; do [ -x "$d/scripts/forge-root.sh" ] && exec "$d/scripts/forge-root.sh"; done')"
|
|
112
115
|
[ -n "$R" ] || { echo "feature-forge: cannot locate plugin root" >&2; exit 1; }
|
|
113
116
|
mkdir -p "<specsDir>"
|
|
114
117
|
[ -f "<specsDir>/AGENTS.md" ] || cp "$R/references/templates/specs-hygiene/AGENTS.md" "<specsDir>/AGENTS.md"
|
|
@@ -135,7 +138,7 @@ After resolving the feature directory, check the feature's `.pipeline-state.json
|
|
|
135
138
|
To obtain the manifest contracts and the live completion status of each dependency in one deterministic call, run `render-status` and read the per-feature `status` and the `consumes`/`exposes` arrays rather than re-deriving them:
|
|
136
139
|
|
|
137
140
|
```bash
|
|
138
|
-
R="$(for d in "$HOME"/.claude/skills/feature-forge "$HOME"/.claude/plugins/*/feature-forge "$HOME"/.agents/skills/feature-forge ./.agents/skills/feature-forge; do [ -x "$d/scripts/forge-root.sh" ] && exec "$d/scripts/forge-root.sh"; done)"
|
|
141
|
+
R="$(bash -c 'for d in "$HOME"/.claude/skills/feature-forge "$HOME"/.claude/plugins/*/feature-forge "$HOME"/.agents/skills/feature-forge ./.agents/skills/feature-forge; do [ -x "$d/scripts/forge-root.sh" ] && exec "$d/scripts/forge-root.sh"; done')"
|
|
139
142
|
[ -n "$R" ] || { echo "feature-forge: cannot locate plugin root" >&2; exit 1; }
|
|
140
143
|
python3 "$R/scripts/epic-manifest.py" \
|
|
141
144
|
render-status "<epic>" --specs-dir "<specsDir>" --json
|
|
@@ -180,16 +183,18 @@ Invoke this block at the **very start** of a pipeline entry point — `forge-1-p
|
|
|
180
183
|
|
|
181
184
|
## Git Commit Protocol
|
|
182
185
|
|
|
183
|
-
When `gitCommitAfterStage` is true, follow this exact order to avoid state inconsistency
|
|
186
|
+
When `gitCommitAfterStage` is true, follow this exact order to avoid state inconsistency.
|
|
187
|
+
|
|
188
|
+
**Why two commits.** The stage's `.pipeline-state.json` is itself part of the staged commit, but the stage's `commitHash` cannot be known until *after* that commit is made. Recording it *inside* the same commit is a chicken-and-egg with no single-commit solution. Resolve it with a **deterministic two-commit sequence**, and **never** with `git commit --amend`: amending rewrites HEAD, so a hash captured before the amend points at an orphaned commit that is not in the final history (the exact defect this protocol exists to prevent).
|
|
184
189
|
|
|
185
190
|
1. **Stage specific files only:** `git add {specsDir}/{feature}/` — never use `git add -A` or `git add .`
|
|
186
|
-
2. **
|
|
187
|
-
3. **If
|
|
188
|
-
4. **If
|
|
191
|
+
2. **Commit 1 — artifacts + state, hash not yet known:** In `.pipeline-state.json`, set this stage's `status: "complete"` and `commitHash: null`, then `git commit -m "{commitPrefix}({feature}): <action>"`. This is the stage's **artifact commit**; its hash is the provenance hash callers rely on.
|
|
192
|
+
3. **If Commit 1 succeeds — Commit 2 records the hash:** Capture the hash of Commit 1 (`git rev-parse HEAD`). Write it into this stage's `commitHash` in `.pipeline-state.json`, then commit only that one-line change: `git add {specsDir}/{feature}/.pipeline-state.json && git commit -m "{commitPrefix}({feature}): record stage commit hash"`. The stored `commitHash` now points at the artifact commit (Commit 1) — never at Commit 2, and never at an orphaned amend. The working tree is clean afterward, so the next stage's dirty-tree check passes.
|
|
193
|
+
4. **If Commit 1 fails:** do NOT update pipeline state to complete. Report the error to the user and leave state as `in-progress` so the stage can be resumed. Common failure causes:
|
|
189
194
|
- **Pre-commit hook failure:** Report the hook output. Never use `--no-verify` to bypass. Help the user fix the underlying issue.
|
|
190
195
|
- **Merge conflicts:** Report conflicting files. Suggest resolution steps appropriate to the conflict.
|
|
191
|
-
- **Nothing to commit:** If all artifacts were already committed, this is fine —
|
|
192
|
-
5. **Never** use `git add -A`, `--no-verify`, or `--force` flags
|
|
196
|
+
- **Nothing to commit:** If all artifacts were already committed, this is fine — mark the stage `complete`, leave `commitHash` at its existing value (or `null` if there was never an artifact commit), and skip Commit 2. There is no new artifact commit to record.
|
|
197
|
+
5. **Never** use `git add -A`, `--amend`, `--no-verify`, or `--force` flags
|
|
193
198
|
|
|
194
199
|
## Crash Recovery
|
|
195
200
|
|