@izkac/forgekit 0.3.3 → 0.3.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/package.json
CHANGED
|
@@ -25,8 +25,11 @@ forge brief stamp # records specs hash + opens it in the operator's brows
|
|
|
25
25
|
forge phase implement --tasks-total <N> # hard-gated on a fresh stamped brief
|
|
26
26
|
```
|
|
27
27
|
|
|
28
|
-
If any spec file changes later, the brief goes **stale** —
|
|
29
|
-
|
|
28
|
+
If any spec file changes later, the brief goes **stale** — the gate cannot
|
|
29
|
+
tell meaning changes from typo fixes, so it flags both. Triage cheaply: if the
|
|
30
|
+
edit doesn't change what the operator approved (typo, task renumber,
|
|
31
|
+
formatting), just re-run `forge brief stamp` — no rewrite. If it does, update
|
|
32
|
+
the affected sections first, then stamp. `forge brief check` reports status.
|
|
30
33
|
Genuine edge case (operator explicitly waives it):
|
|
31
34
|
`forge phase implement --allow-incomplete "<reason>"`.
|
|
32
35
|
|
|
@@ -80,6 +80,29 @@ driven by any command (e.g. requires a physical device). Reviewers police the
|
|
|
80
80
|
reason; "no time" or "covered by unit tests" is a REJECT. Keep a short loop
|
|
81
81
|
narrative under `## Product loop` in `verify-evidence.md` as reviewer context.
|
|
82
82
|
|
|
83
|
+
### Keeping the loop cheap (cost policy)
|
|
84
|
+
|
|
85
|
+
"Too complex to test" is usually a signal to test it, not to wave it through —
|
|
86
|
+
a loop is complex exactly when the feature has many wiring seams, which is
|
|
87
|
+
when wiring silently breaks. Never skip the loop for complexity; control its
|
|
88
|
+
cost instead:
|
|
89
|
+
|
|
90
|
+
- **Ask before building new harness infrastructure, not before testing.** If
|
|
91
|
+
a harness already exists (test server, isolated ports, scratch DB), the loop
|
|
92
|
+
is cheap — author it. If the loop would require building a harness from
|
|
93
|
+
scratch or driving a third-party service, stop and ask the user: that is a
|
|
94
|
+
project, not a test.
|
|
95
|
+
- **One executed loop per capability, not per assertion.** Push edge cases
|
|
96
|
+
into unit tests; e2e proves the wiring exists.
|
|
97
|
+
- **Set `timeoutMs` from the step count up front.** A 10-step loop is never a
|
|
98
|
+
30s test — a too-small budget fails at whatever line the clock runs out on,
|
|
99
|
+
which reads as a phantom regression.
|
|
100
|
+
- **Run it once, at the end, serially** — never alongside unit suites; load
|
|
101
|
+
makes real assertions look flaky.
|
|
102
|
+
- **Prefer a production/preview build over the dev server** when the loop
|
|
103
|
+
drives a web app — compile-on-demand and dev-mode socket noise dominate the
|
|
104
|
+
per-run cost and bury the actual assertion.
|
|
105
|
+
|
|
83
106
|
## 6. Job-kind closure
|
|
84
107
|
|
|
85
108
|
Every job kind / entry point on the product surface (enums, API, UI) is, before
|