devrites 6.1.5 → 6.3.0
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/CHANGELOG.md +12 -0
- package/README.md +1 -1
- package/docs/engine/commands.md +2 -2
- package/engine/help.go +1 -1
- package/engine/internal/lib/dispatch.go +12 -5
- package/engine/internal/lib/dispatch_contract.go +51 -0
- package/engine/internal/lib/drift.go +14 -2
- package/engine/internal/lib/proportion.go +130 -0
- package/engine/internal/lib/slicecheck.go +1 -0
- package/pack/.claude/agents/devrites-code-reviewer.md +1 -1
- package/pack/.claude/agents/devrites-plan-drafter.md +3 -1
- package/pack/.claude/agents/devrites-plan-reviewer.md +4 -1
- package/pack/.claude/skills/devrites-lib/reference/standards/agents.md +8 -1
- package/pack/.claude/skills/devrites-lib/reference/standards/loop-operations.md +4 -0
- package/pack/.claude/skills/rite-build/SKILL.md +2 -2
- package/pack/.claude/skills/rite-build/reference/one-slice-cycle.md +3 -1
- package/pack/.claude/skills/rite-build/reference/wright-dispatch.md +8 -4
- package/pack/.claude/skills/rite-define/SKILL.md +3 -1
- package/pack/.claude/skills/rite-vet/SKILL.md +3 -1
- package/pack/.claude/skills/rite-vet/reference/artifacts.md +14 -3
- package/pack/generated/claude/agents/devrites-code-reviewer.md +1 -1
- package/pack/generated/claude/agents/devrites-plan-drafter.md +3 -1
- package/pack/generated/claude/agents/devrites-plan-reviewer.md +4 -1
- package/pack/generated/claude/skills/devrites-lib/reference/standards/agents.md +8 -1
- package/pack/generated/claude/skills/devrites-lib/reference/standards/loop-operations.md +4 -0
- package/pack/generated/claude/skills/rite-build/SKILL.md +2 -2
- package/pack/generated/claude/skills/rite-build/reference/one-slice-cycle.md +3 -1
- package/pack/generated/claude/skills/rite-build/reference/wright-dispatch.md +8 -4
- package/pack/generated/claude/skills/rite-define/SKILL.md +3 -1
- package/pack/generated/claude/skills/rite-vet/SKILL.md +3 -1
- package/pack/generated/claude/skills/rite-vet/reference/artifacts.md +14 -3
- package/pack/generated/codex/agents/devrites-code-reviewer.toml +1 -1
- package/pack/generated/codex/agents/devrites-plan-drafter.toml +3 -1
- package/pack/generated/codex/agents/devrites-plan-reviewer.toml +4 -1
- package/pack/generated/codex/skills/devrites-lib/reference/standards/agents.md +8 -1
- package/pack/generated/codex/skills/devrites-lib/reference/standards/loop-operations.md +4 -0
- package/pack/generated/codex/skills/rite-build/SKILL.md +2 -2
- package/pack/generated/codex/skills/rite-build/reference/one-slice-cycle.md +3 -1
- package/pack/generated/codex/skills/rite-build/reference/wright-dispatch.md +8 -4
- package/pack/generated/codex/skills/rite-define/SKILL.md +3 -1
- package/pack/generated/codex/skills/rite-vet/SKILL.md +3 -1
- package/pack/generated/codex/skills/rite-vet/reference/artifacts.md +14 -3
- package/pack/generated/devin/agents/devrites-code-reviewer.md +1 -1
- package/pack/generated/devin/agents/devrites-plan-drafter.md +3 -1
- package/pack/generated/devin/agents/devrites-plan-reviewer.md +4 -1
- package/pack/generated/devin/skills/devrites-lib/reference/standards/agents.md +8 -1
- package/pack/generated/devin/skills/devrites-lib/reference/standards/loop-operations.md +4 -0
- package/pack/generated/devin/skills/rite-build/SKILL.md +2 -2
- package/pack/generated/devin/skills/rite-build/reference/one-slice-cycle.md +3 -1
- package/pack/generated/devin/skills/rite-build/reference/wright-dispatch.md +8 -4
- package/pack/generated/devin/skills/rite-define/SKILL.md +3 -1
- package/pack/generated/devin/skills/rite-vet/SKILL.md +3 -1
- package/pack/generated/devin/skills/rite-vet/reference/artifacts.md +14 -3
- package/pack/generated/omp/agents/devrites-code-reviewer.md +1 -1
- package/pack/generated/omp/agents/devrites-plan-drafter.md +3 -1
- package/pack/generated/omp/agents/devrites-plan-reviewer.md +4 -1
- package/pack/generated/omp/skills/devrites-lib/reference/standards/agents.md +8 -1
- package/pack/generated/omp/skills/devrites-lib/reference/standards/loop-operations.md +4 -0
- package/pack/generated/omp/skills/rite-build/SKILL.md +2 -2
- package/pack/generated/omp/skills/rite-build/reference/one-slice-cycle.md +3 -1
- package/pack/generated/omp/skills/rite-build/reference/wright-dispatch.md +8 -4
- package/pack/generated/omp/skills/rite-define/SKILL.md +3 -1
- package/pack/generated/omp/skills/rite-vet/SKILL.md +3 -1
- package/pack/generated/omp/skills/rite-vet/reference/artifacts.md +14 -3
- package/pack/generated/pi/agents/devrites-code-reviewer.md +1 -1
- package/pack/generated/pi/agents/devrites-plan-drafter.md +3 -1
- package/pack/generated/pi/agents/devrites-plan-reviewer.md +4 -1
- package/pack/generated/pi/skills/devrites-lib/reference/standards/agents.md +8 -1
- package/pack/generated/pi/skills/devrites-lib/reference/standards/loop-operations.md +4 -0
- package/pack/generated/pi/skills/rite-build/SKILL.md +2 -2
- package/pack/generated/pi/skills/rite-build/reference/one-slice-cycle.md +3 -1
- package/pack/generated/pi/skills/rite-build/reference/wright-dispatch.md +8 -4
- package/pack/generated/pi/skills/rite-define/SKILL.md +3 -1
- package/pack/generated/pi/skills/rite-vet/SKILL.md +3 -1
- package/pack/generated/pi/skills/rite-vet/reference/artifacts.md +14 -3
- package/package.json +1 -1
|
@@ -94,10 +94,14 @@ claim add --session <id> <task-paths>`; retain its returned `<claim-id>`, then r
|
|
|
94
94
|
live foreign session releases the retained claim and stops the dispatch, per
|
|
95
95
|
[`agents.md`](../../devrites-lib/reference/standards/agents.md). `.devrites/**`
|
|
96
96
|
is never a task path; see [Workflow Artifact admission](../../devrites-lib/reference/standards/workflow-artifacts.md).
|
|
97
|
-
|
|
98
|
-
|
|
99
|
-
|
|
100
|
-
|
|
97
|
+
Write the exact contract, with its `Writer-safe checks:` line, to
|
|
98
|
+
`packets/<wave>.contract.md` and hand the wright that text verbatim. Ask the host
|
|
99
|
+
for the exact writer in fresh context; a repair round on the same slice whose
|
|
100
|
+
returned wright the host can still continue goes to that wright on its existing
|
|
101
|
+
handle instead, so it does not re-read the slice from scratch. Wait, and track it
|
|
102
|
+
through the launch barrier — `devrites-engine dispatch <slug> open --phase build --wave
|
|
103
|
+
<slice-id> --role slice-wright --slice <SLICE-ID> --contract <file>` (it refuses a
|
|
104
|
+
contract that forbids writer-safe checks or names none), then `start` with the host handle, `seal`, and
|
|
101
105
|
`return` on completion (auto-records dispatch/return metrics). Every 5 wright
|
|
102
106
|
rounds on one slice, `open` blocks for a reslice review: propose an
|
|
103
107
|
acceptance-preserving split (`root_no_change_analysis` → `FOLD` per
|
|
@@ -131,7 +131,9 @@ Pull these via `Read` when shaping the plan:
|
|
|
131
131
|
expand this into repository-wide coverage.
|
|
132
132
|
4. **Map coverage and wiring:** every `AC-###` spec acceptance criterion maps to ≥1 `SLICE-###`
|
|
133
133
|
([`acceptance-criteria.md`](../rite-spec/reference/acceptance-criteria.md)); no orphaned criteria, no slice without a
|
|
134
|
-
criterion.
|
|
134
|
+
criterion. Every proof row quotes the spec line it proves (`Proves`); proof that proves
|
|
135
|
+
none goes to `test-plan.md` `## Deferred hardening`, never into a slice
|
|
136
|
+
([artifacts.md](../rite-vet/reference/artifacts.md)). Lift covered/backstop `Edge Coverage` rows and resolved `Prohibitions (must-NOT)`
|
|
135
137
|
rows into `traceability.md` and `test-plan.md`; unresolved rows get a gate/owner. Each
|
|
136
138
|
`Existing behavior to preserve` row becomes a regression row whose current evidence is
|
|
137
139
|
observed green on the unchanged baseline as a prerequisite of the first slice touching that
|
|
@@ -209,7 +209,9 @@ plan declares a root-authored executable workflow file, read
|
|
|
209
209
|
`devrites-engine check readiness --emit-binding <slug>`; normal readiness check
|
|
210
210
|
must pass. Record the per-input drift baseline alongside it:
|
|
211
211
|
`devrites-engine check drift <slug> --record` — a later stale binding then
|
|
212
|
-
names the exact artifact that moved instead of "something changed".
|
|
212
|
+
names the exact artifact that moved instead of "something changed". An `ADVISORY
|
|
213
|
+
proportion:` line names a slice whose proof outgrew its ACs: move its rows without a
|
|
214
|
+
`Proves` spec line to `Deferred hardening` before READY, never deepen them. Technical failure records reproduction, not qid. Human gap awaits
|
|
213
215
|
Clarify. Optional cross-model follows `reference/cross-model.md` (trigger `cross-model`).
|
|
214
216
|
|
|
215
217
|
With READY, no pending remediation, and a valid technical return cursor,
|
|
@@ -128,9 +128,20 @@ prove only their named static criterion.
|
|
|
128
128
|
<the ASCII code-paths + user-flows diagram from review-axes.md, with COVERAGE / GAPS / REGRESSIONS line>
|
|
129
129
|
|
|
130
130
|
## Per-gap test requirements
|
|
131
|
-
| ID | Path / flow | Test file (match conventions) | Asserts (input → expected) | Kind | Slice | Priority |
|
|
132
|
-
|
|
133
|
-
| T1 | <path> | <path/to.test> | <empty list → 200 + []> | unit / E2E / eval | <slice> | P1 / **Regression-Critical** |
|
|
131
|
+
| ID | Path / flow | Test file (match conventions) | Asserts (input → expected) | Kind | Slice | Priority | Proves |
|
|
132
|
+
|---|---|---|---|---|---|---|---|
|
|
133
|
+
| T1 | <path> | <path/to.test> | <empty list → 200 + []> | unit / E2E / eval | <slice> | P1 / **Regression-Critical** | <AC/REQ/INV/PROH/DEC/EDGE ID or standard § + quoted line> |
|
|
134
|
+
|
|
135
|
+
Every row's `Proves` quotes the spec line or standard rule it proves. A row that proves
|
|
136
|
+
none is plan-invented hardening: move it to `Deferred hardening`.
|
|
137
|
+
|
|
138
|
+
## Deferred hardening
|
|
139
|
+
| ID | Hardening | Why no spec line needs it | Promote by |
|
|
140
|
+
|---|---|---|---|
|
|
141
|
+
| D1 | <proof or mechanism> | <reason> | <the spec line an owner would add> |
|
|
142
|
+
|
|
143
|
+
Deferred rows are an optional follow-up slice: not built, proven, or gating in this
|
|
144
|
+
feature; this table is their follow-up record. Nothing is deleted.
|
|
134
145
|
|
|
135
146
|
## Interaction inventory (UI slices — every element + flow gets an asserting test)
|
|
136
147
|
Enumerate every interactive element and user flow the feature exposes; one row each, none
|
|
@@ -159,7 +159,7 @@ Outcome: <findings | no-findings | gap>
|
|
|
159
159
|
Counts: <n per severity or kind used in the rows below>
|
|
160
160
|
Account: <admitted findings | No-findings | Gap per Result admission>
|
|
161
161
|
Coverage: <files read fully | hunk-level | skipped: name — reason>
|
|
162
|
-
Finding: <severity> conf <n> | <file:line> | <observed> | when <input/state/sequence> → <effect> | <minimum fix>
|
|
162
|
+
Finding: <severity> conf <n> | <file:line> | <observed> | when <input/state/sequence> → <effect> | breaks <ID or standard §> "<quoted line>" | <minimum fix>
|
|
163
163
|
Basis: <files read · commands run to reach this finding>
|
|
164
164
|
Contract: <applicable | not-applicable> — <basis: both sides read · crossing test>
|
|
165
165
|
Tests: <adequate? gaps>
|
|
@@ -21,7 +21,9 @@ seams supplied by the orchestrator.
|
|
|
21
21
|
Reuse existing seams and dependencies before proposing new ones. Build vertical
|
|
22
22
|
slices in dependency order; put a hub file's shared edit in one seam slice
|
|
23
23
|
([slicing.md](.agents/skills/rite-plan/reference/slicing.md#slice-independence)). Map every REQ and AC to a slice and executable proof,
|
|
24
|
-
name rollback and failure paths; keep acceptance-changing proposals separate.
|
|
24
|
+
name rollback and failure paths; keep acceptance-changing proposals separate. Each proof
|
|
25
|
+
row quotes the spec line or standard rule it proves (`Proves`); proof that proves none
|
|
26
|
+
goes to `test-plan.md` `## Deferred hardening`, never into a slice. Durable
|
|
25
27
|
proof commands must be portable repository commands, never host-local wrappers,
|
|
26
28
|
user-specific absolute paths, or temporary proof trees.
|
|
27
29
|
Reconcile `spec.md`'s applicability map against live evidence. When triggered, apply
|
|
@@ -116,7 +116,10 @@ criterion, slice, or proof.
|
|
|
116
116
|
`convergence_pressure` eligibility mechanical. Tag every Critical/Important
|
|
117
117
|
`kind: contract` (an implementer could build it differently and still pass the
|
|
118
118
|
planned tests) or `kind: mechanism` (a named test can discriminate the wrong
|
|
119
|
-
implementation; propose that case as the fix, not new plan prose).
|
|
119
|
+
implementation; propose that case as the fix, not new plan prose). Each
|
|
120
|
+
Critical/Important also quotes the spec line (`AC`/`REQ`/`INV`/`PROH`/`DEC`/`EDGE`)
|
|
121
|
+
or standard rule it breaks; hardening no spec line asks for is a Suggestion
|
|
122
|
+
([agents.md](.agents/skills/devrites-lib/reference/standards/agents.md) § Result admission, Proportionality).
|
|
120
123
|
- In a recheck, your verdict covers the listed fingerprints, the correction diff, and
|
|
121
124
|
the dependents of each changed clause. Return an explicit correction-created
|
|
122
125
|
regression verdict over the touched invariants and those dependents as
|
|
@@ -98,7 +98,7 @@ Each required reviewer/analyst/auditor starts with exactly one:
|
|
|
98
98
|
```text
|
|
99
99
|
Outcome: <findings | no-findings | gap>
|
|
100
100
|
Counts: <n per severity or kind used in the rows below>
|
|
101
|
-
Finding: <severity> conf <n> | <file:line or artifact section> | <observed quote/result> | when <input/state/sequence> → <effect> | <minimum fix>
|
|
101
|
+
Finding: <severity> conf <n> | <file:line or artifact section> | <observed quote/result> | when <input/state/sequence> → <effect> | breaks <ID or standard §> "<quoted line>" | <minimum fix>
|
|
102
102
|
Basis: <files read · commands run to reach this finding>
|
|
103
103
|
```
|
|
104
104
|
|
|
@@ -108,6 +108,13 @@ Basis: <files read · commands run to reach this finding>
|
|
|
108
108
|
A Critical/Important row carries `conf <n>` in its severity cell and writes impact
|
|
109
109
|
as `when <input/state/sequence> → <effect>`; missing either, root admits it as
|
|
110
110
|
`Unverified: <claim> — missing trigger` (stays `gap`).
|
|
111
|
+
- **Proportionality:** a Critical/Important row's `breaks` quotes the line it violates:
|
|
112
|
+
a spec ID (`AC`/`REQ`/`INV`/`PROH`/`DEC`/`EDGE`) or a shipped standard rule
|
|
113
|
+
(`<standard>.md § <section>`: security, regression, principles). Root anchors the
|
|
114
|
+
quote like the `file:line` quote. No anchor, or one the defect does not break,
|
|
115
|
+
admits the row as Suggestion logged `Deferred: <claim> — no spec anchor` for an
|
|
116
|
+
optional follow-up slice; it never drives a repair round. Hardening no spec line
|
|
117
|
+
asks for is a Suggestion, however plausible.
|
|
111
118
|
- **`no-findings`:** `No-findings:` names checks and inspected evidence. Bare
|
|
112
119
|
pass, empty list, or “looks good” is malformed.
|
|
113
120
|
- **`gap`:** names missing/unreadable/stale input; skipped/failed required check;
|
|
@@ -112,6 +112,10 @@ armed AFK workspace whose exact scope, gates, and budgets permit it.
|
|
|
112
112
|
run `devrites-engine check drift <slug>` before re-vetting: it names the
|
|
113
113
|
exact input artifact that changed (or `missing`/`added`) since the recorded
|
|
114
114
|
baseline, so the Spec Drift Guard or `$rite-vet` starts at the right file.
|
|
115
|
+
- `check drift` and `check slice` print `ADVISORY proportion:` when a slice's proof
|
|
116
|
+
grew over 2x since the Vet baseline or carries over 3x the median proof per AC.
|
|
117
|
+
Repair by moving rows without a `Proves` spec line to `Deferred hardening`
|
|
118
|
+
([agents.md](agents.md) § Result admission, Proportionality), not by adding proof.
|
|
115
119
|
- Native notifications fire only after evidence and stop state are written.
|
|
116
120
|
|
|
117
121
|
Use [`afk-hitl.md`](afk-hitl.md) for unattended authority and resource budgets,
|
|
@@ -29,8 +29,8 @@ Execute [`reference/phase-contract.md`](reference/phase-contract.md)
|
|
|
29
29
|
[`reference/wright-dispatch.md`](reference/wright-dispatch.md).
|
|
30
30
|
Every wright contract names the writer-safe checks the wright runs (syntax/type/lint,
|
|
31
31
|
unit and self-check suites, a narrow local smoke) and never forbids them; only
|
|
32
|
-
artifact-producing, consumptive, or full canonical gates stay root-owned
|
|
33
|
-
Prior contracts in `evidence.md` are not a template for this. **Failing case:** the
|
|
32
|
+
artifact-producing, consumptive, or full canonical gates stay root-owned;
|
|
33
|
+
`dispatch open --contract` refuses a contract that breaks this. Prior contracts in `evidence.md` are not a template for this. **Failing case:** the
|
|
34
34
|
contract says "skip tests mid-flight; root runs them after return", and the wright
|
|
35
35
|
returns three unit failures that a 20 s `go test` would have caught in-round.
|
|
36
36
|
|
|
@@ -23,7 +23,9 @@ DISPATCH → hand the slice contract to devrites-slice-wright (fresh context).
|
|
|
23
23
|
(independent claims, fresh contexts); wright fixes stay serial — writers share files
|
|
24
24
|
PROVE → root, concurrent with the reviews on the same frozen candidate:
|
|
25
25
|
required build/browser/E2E (narrow self-check/smoke first, full gate only
|
|
26
|
-
after it is green, long runs in the background
|
|
26
|
+
after it is green, long runs in the background; cancel one still running once
|
|
27
|
+
review or proof sends the candidate back, keeping its partial output as
|
|
28
|
+
diagnostics), then returned-diff/path recheck; fail-on-red
|
|
27
29
|
(red/gap → repair wright, then re-review and re-prove; do not commit yet)
|
|
28
30
|
RECORD → orchestrator: after green verification, update state.md/evidence.md and
|
|
29
31
|
upsert the authoritative candidate manifest from the actual scoped diff
|
|
@@ -94,10 +94,14 @@ claim add --session <id> <task-paths>`; retain its returned `<claim-id>`, then r
|
|
|
94
94
|
live foreign session releases the retained claim and stops the dispatch, per
|
|
95
95
|
[`agents.md`](../../devrites-lib/reference/standards/agents.md). `.devrites/**`
|
|
96
96
|
is never a task path; see [Workflow Artifact admission](../../devrites-lib/reference/standards/workflow-artifacts.md).
|
|
97
|
-
|
|
98
|
-
|
|
99
|
-
|
|
100
|
-
|
|
97
|
+
Write the exact contract, with its `Writer-safe checks:` line, to
|
|
98
|
+
`packets/<wave>.contract.md` and hand the wright that text verbatim. Ask the host
|
|
99
|
+
for the exact writer in fresh context; a repair round on the same slice whose
|
|
100
|
+
returned wright the host can still continue goes to that wright on its existing
|
|
101
|
+
handle instead, so it does not re-read the slice from scratch. Wait, and track it
|
|
102
|
+
through the launch barrier — `devrites-engine dispatch <slug> open --phase build --wave
|
|
103
|
+
<slice-id> --role slice-wright --slice <SLICE-ID> --contract <file>` (it refuses a
|
|
104
|
+
contract that forbids writer-safe checks or names none), then `start` with the host handle, `seal`, and
|
|
101
105
|
`return` on completion (auto-records dispatch/return metrics). Every 5 wright
|
|
102
106
|
rounds on one slice, `open` blocks for a reslice review: propose an
|
|
103
107
|
acceptance-preserving split (`root_no_change_analysis` → `FOLD` per
|
|
@@ -131,7 +131,9 @@ Pull these via `Read` when shaping the plan:
|
|
|
131
131
|
expand this into repository-wide coverage.
|
|
132
132
|
4. **Map coverage and wiring:** every `AC-###` spec acceptance criterion maps to ≥1 `SLICE-###`
|
|
133
133
|
([`acceptance-criteria.md`](../rite-spec/reference/acceptance-criteria.md)); no orphaned criteria, no slice without a
|
|
134
|
-
criterion.
|
|
134
|
+
criterion. Every proof row quotes the spec line it proves (`Proves`); proof that proves
|
|
135
|
+
none goes to `test-plan.md` `## Deferred hardening`, never into a slice
|
|
136
|
+
([artifacts.md](../rite-vet/reference/artifacts.md)). Lift covered/backstop `Edge Coverage` rows and resolved `Prohibitions (must-NOT)`
|
|
135
137
|
rows into `traceability.md` and `test-plan.md`; unresolved rows get a gate/owner. Each
|
|
136
138
|
`Existing behavior to preserve` row becomes a regression row whose current evidence is
|
|
137
139
|
observed green on the unchanged baseline as a prerequisite of the first slice touching that
|
|
@@ -209,7 +209,9 @@ plan declares a root-authored executable workflow file, read
|
|
|
209
209
|
`devrites-engine check readiness --emit-binding <slug>`; normal readiness check
|
|
210
210
|
must pass. Record the per-input drift baseline alongside it:
|
|
211
211
|
`devrites-engine check drift <slug> --record` — a later stale binding then
|
|
212
|
-
names the exact artifact that moved instead of "something changed".
|
|
212
|
+
names the exact artifact that moved instead of "something changed". An `ADVISORY
|
|
213
|
+
proportion:` line names a slice whose proof outgrew its ACs: move its rows without a
|
|
214
|
+
`Proves` spec line to `Deferred hardening` before READY, never deepen them. Technical failure records reproduction, not qid. Human gap awaits
|
|
213
215
|
Clarify. Optional cross-model follows `reference/cross-model.md` (trigger `cross-model`).
|
|
214
216
|
|
|
215
217
|
With READY, no pending remediation, and a valid technical return cursor,
|
|
@@ -128,9 +128,20 @@ prove only their named static criterion.
|
|
|
128
128
|
<the ASCII code-paths + user-flows diagram from review-axes.md, with COVERAGE / GAPS / REGRESSIONS line>
|
|
129
129
|
|
|
130
130
|
## Per-gap test requirements
|
|
131
|
-
| ID | Path / flow | Test file (match conventions) | Asserts (input → expected) | Kind | Slice | Priority |
|
|
132
|
-
|
|
133
|
-
| T1 | <path> | <path/to.test> | <empty list → 200 + []> | unit / E2E / eval | <slice> | P1 / **Regression-Critical** |
|
|
131
|
+
| ID | Path / flow | Test file (match conventions) | Asserts (input → expected) | Kind | Slice | Priority | Proves |
|
|
132
|
+
|---|---|---|---|---|---|---|---|
|
|
133
|
+
| T1 | <path> | <path/to.test> | <empty list → 200 + []> | unit / E2E / eval | <slice> | P1 / **Regression-Critical** | <AC/REQ/INV/PROH/DEC/EDGE ID or standard § + quoted line> |
|
|
134
|
+
|
|
135
|
+
Every row's `Proves` quotes the spec line or standard rule it proves. A row that proves
|
|
136
|
+
none is plan-invented hardening: move it to `Deferred hardening`.
|
|
137
|
+
|
|
138
|
+
## Deferred hardening
|
|
139
|
+
| ID | Hardening | Why no spec line needs it | Promote by |
|
|
140
|
+
|---|---|---|---|
|
|
141
|
+
| D1 | <proof or mechanism> | <reason> | <the spec line an owner would add> |
|
|
142
|
+
|
|
143
|
+
Deferred rows are an optional follow-up slice: not built, proven, or gating in this
|
|
144
|
+
feature; this table is their follow-up record. Nothing is deleted.
|
|
134
145
|
|
|
135
146
|
## Interaction inventory (UI slices — every element + flow gets an asserting test)
|
|
136
147
|
Enumerate every interactive element and user flow the feature exposes; one row each, none
|
|
@@ -163,7 +163,7 @@ Outcome: <findings | no-findings | gap>
|
|
|
163
163
|
Counts: <n per severity or kind used in the rows below>
|
|
164
164
|
Account: <admitted findings | No-findings | Gap per Result admission>
|
|
165
165
|
Coverage: <files read fully | hunk-level | skipped: name — reason>
|
|
166
|
-
Finding: <severity> conf <n> | <file:line> | <observed> | when <input/state/sequence> → <effect> | <minimum fix>
|
|
166
|
+
Finding: <severity> conf <n> | <file:line> | <observed> | when <input/state/sequence> → <effect> | breaks <ID or standard §> "<quoted line>" | <minimum fix>
|
|
167
167
|
Basis: <files read · commands run to reach this finding>
|
|
168
168
|
Contract: <applicable | not-applicable> — <basis: both sides read · crossing test>
|
|
169
169
|
Tests: <adequate? gaps>
|
|
@@ -26,7 +26,9 @@ seams supplied by the orchestrator.
|
|
|
26
26
|
Reuse existing seams and dependencies before proposing new ones. Build vertical
|
|
27
27
|
slices in dependency order; put a hub file's shared edit in one seam slice
|
|
28
28
|
([slicing.md](../skills/rite-plan/reference/slicing.md#slice-independence)). Map every REQ and AC to a slice and executable proof,
|
|
29
|
-
name rollback and failure paths; keep acceptance-changing proposals separate.
|
|
29
|
+
name rollback and failure paths; keep acceptance-changing proposals separate. Each proof
|
|
30
|
+
row quotes the spec line or standard rule it proves (`Proves`); proof that proves none
|
|
31
|
+
goes to `test-plan.md` `## Deferred hardening`, never into a slice. Durable
|
|
30
32
|
proof commands must be portable repository commands, never host-local wrappers,
|
|
31
33
|
user-specific absolute paths, or temporary proof trees.
|
|
32
34
|
Reconcile `spec.md`'s applicability map against live evidence. When triggered, apply
|
|
@@ -120,7 +120,10 @@ criterion, slice, or proof.
|
|
|
120
120
|
`convergence_pressure` eligibility mechanical. Tag every Critical/Important
|
|
121
121
|
`kind: contract` (an implementer could build it differently and still pass the
|
|
122
122
|
planned tests) or `kind: mechanism` (a named test can discriminate the wrong
|
|
123
|
-
implementation; propose that case as the fix, not new plan prose).
|
|
123
|
+
implementation; propose that case as the fix, not new plan prose). Each
|
|
124
|
+
Critical/Important also quotes the spec line (`AC`/`REQ`/`INV`/`PROH`/`DEC`/`EDGE`)
|
|
125
|
+
or standard rule it breaks; hardening no spec line asks for is a Suggestion
|
|
126
|
+
([agents.md](../skills/devrites-lib/reference/standards/agents.md) § Result admission, Proportionality).
|
|
124
127
|
- In a recheck, your verdict covers the listed fingerprints, the correction diff, and
|
|
125
128
|
the dependents of each changed clause. Return an explicit correction-created
|
|
126
129
|
regression verdict over the touched invariants and those dependents as
|
|
@@ -98,7 +98,7 @@ Each required reviewer/analyst/auditor starts with exactly one:
|
|
|
98
98
|
```text
|
|
99
99
|
Outcome: <findings | no-findings | gap>
|
|
100
100
|
Counts: <n per severity or kind used in the rows below>
|
|
101
|
-
Finding: <severity> conf <n> | <file:line or artifact section> | <observed quote/result> | when <input/state/sequence> → <effect> | <minimum fix>
|
|
101
|
+
Finding: <severity> conf <n> | <file:line or artifact section> | <observed quote/result> | when <input/state/sequence> → <effect> | breaks <ID or standard §> "<quoted line>" | <minimum fix>
|
|
102
102
|
Basis: <files read · commands run to reach this finding>
|
|
103
103
|
```
|
|
104
104
|
|
|
@@ -108,6 +108,13 @@ Basis: <files read · commands run to reach this finding>
|
|
|
108
108
|
A Critical/Important row carries `conf <n>` in its severity cell and writes impact
|
|
109
109
|
as `when <input/state/sequence> → <effect>`; missing either, root admits it as
|
|
110
110
|
`Unverified: <claim> — missing trigger` (stays `gap`).
|
|
111
|
+
- **Proportionality:** a Critical/Important row's `breaks` quotes the line it violates:
|
|
112
|
+
a spec ID (`AC`/`REQ`/`INV`/`PROH`/`DEC`/`EDGE`) or a shipped standard rule
|
|
113
|
+
(`<standard>.md § <section>`: security, regression, principles). Root anchors the
|
|
114
|
+
quote like the `file:line` quote. No anchor, or one the defect does not break,
|
|
115
|
+
admits the row as Suggestion logged `Deferred: <claim> — no spec anchor` for an
|
|
116
|
+
optional follow-up slice; it never drives a repair round. Hardening no spec line
|
|
117
|
+
asks for is a Suggestion, however plausible.
|
|
111
118
|
- **`no-findings`:** `No-findings:` names checks and inspected evidence. Bare
|
|
112
119
|
pass, empty list, or “looks good” is malformed.
|
|
113
120
|
- **`gap`:** names missing/unreadable/stale input; skipped/failed required check;
|
|
@@ -112,6 +112,10 @@ armed AFK workspace whose exact scope, gates, and budgets permit it.
|
|
|
112
112
|
run `devrites-engine check drift <slug>` before re-vetting: it names the
|
|
113
113
|
exact input artifact that changed (or `missing`/`added`) since the recorded
|
|
114
114
|
baseline, so the Spec Drift Guard or `/rite-vet` starts at the right file.
|
|
115
|
+
- `check drift` and `check slice` print `ADVISORY proportion:` when a slice's proof
|
|
116
|
+
grew over 2x since the Vet baseline or carries over 3x the median proof per AC.
|
|
117
|
+
Repair by moving rows without a `Proves` spec line to `Deferred hardening`
|
|
118
|
+
([agents.md](agents.md) § Result admission, Proportionality), not by adding proof.
|
|
115
119
|
- Native notifications fire only after evidence and stop state are written.
|
|
116
120
|
|
|
117
121
|
Use [`afk-hitl.md`](afk-hitl.md) for unattended authority and resource budgets,
|
|
@@ -28,8 +28,8 @@ Execute [`reference/phase-contract.md`](reference/phase-contract.md)
|
|
|
28
28
|
[`reference/wright-dispatch.md`](reference/wright-dispatch.md).
|
|
29
29
|
Every wright contract names the writer-safe checks the wright runs (syntax/type/lint,
|
|
30
30
|
unit and self-check suites, a narrow local smoke) and never forbids them; only
|
|
31
|
-
artifact-producing, consumptive, or full canonical gates stay root-owned
|
|
32
|
-
Prior contracts in `evidence.md` are not a template for this. **Failing case:** the
|
|
31
|
+
artifact-producing, consumptive, or full canonical gates stay root-owned;
|
|
32
|
+
`dispatch open --contract` refuses a contract that breaks this. Prior contracts in `evidence.md` are not a template for this. **Failing case:** the
|
|
33
33
|
contract says "skip tests mid-flight; root runs them after return", and the wright
|
|
34
34
|
returns three unit failures that a 20 s `go test` would have caught in-round.
|
|
35
35
|
|
|
@@ -23,7 +23,9 @@ DISPATCH → hand the slice contract to devrites-slice-wright (fresh context).
|
|
|
23
23
|
(independent claims, fresh contexts); wright fixes stay serial — writers share files
|
|
24
24
|
PROVE → root, concurrent with the reviews on the same frozen candidate:
|
|
25
25
|
required build/browser/E2E (narrow self-check/smoke first, full gate only
|
|
26
|
-
after it is green, long runs in the background
|
|
26
|
+
after it is green, long runs in the background; cancel one still running once
|
|
27
|
+
review or proof sends the candidate back, keeping its partial output as
|
|
28
|
+
diagnostics), then returned-diff/path recheck; fail-on-red
|
|
27
29
|
(red/gap → repair wright, then re-review and re-prove; do not commit yet)
|
|
28
30
|
RECORD → orchestrator: after green verification, update state.md/evidence.md and
|
|
29
31
|
upsert the authoritative candidate manifest from the actual scoped diff
|
|
@@ -94,10 +94,14 @@ claim add --session <id> <task-paths>`; retain its returned `<claim-id>`, then r
|
|
|
94
94
|
live foreign session releases the retained claim and stops the dispatch, per
|
|
95
95
|
[`agents.md`](../../devrites-lib/reference/standards/agents.md). `.devrites/**`
|
|
96
96
|
is never a task path; see [Workflow Artifact admission](../../devrites-lib/reference/standards/workflow-artifacts.md).
|
|
97
|
-
|
|
98
|
-
|
|
99
|
-
|
|
100
|
-
|
|
97
|
+
Write the exact contract, with its `Writer-safe checks:` line, to
|
|
98
|
+
`packets/<wave>.contract.md` and hand the wright that text verbatim. Ask the host
|
|
99
|
+
for the exact writer in fresh context; a repair round on the same slice whose
|
|
100
|
+
returned wright the host can still continue goes to that wright on its existing
|
|
101
|
+
handle instead, so it does not re-read the slice from scratch. Wait, and track it
|
|
102
|
+
through the launch barrier — `devrites-engine dispatch <slug> open --phase build --wave
|
|
103
|
+
<slice-id> --role slice-wright --slice <SLICE-ID> --contract <file>` (it refuses a
|
|
104
|
+
contract that forbids writer-safe checks or names none), then `start` with the host handle, `seal`, and
|
|
101
105
|
`return` on completion (auto-records dispatch/return metrics). Every 5 wright
|
|
102
106
|
rounds on one slice, `open` blocks for a reslice review: propose an
|
|
103
107
|
acceptance-preserving split (`root_no_change_analysis` → `FOLD` per
|
|
@@ -130,7 +130,9 @@ Pull these via `Read` when shaping the plan:
|
|
|
130
130
|
expand this into repository-wide coverage.
|
|
131
131
|
4. **Map coverage and wiring:** every `AC-###` spec acceptance criterion maps to ≥1 `SLICE-###`
|
|
132
132
|
([`acceptance-criteria.md`](../rite-spec/reference/acceptance-criteria.md)); no orphaned criteria, no slice without a
|
|
133
|
-
criterion.
|
|
133
|
+
criterion. Every proof row quotes the spec line it proves (`Proves`); proof that proves
|
|
134
|
+
none goes to `test-plan.md` `## Deferred hardening`, never into a slice
|
|
135
|
+
([artifacts.md](../rite-vet/reference/artifacts.md)). Lift covered/backstop `Edge Coverage` rows and resolved `Prohibitions (must-NOT)`
|
|
134
136
|
rows into `traceability.md` and `test-plan.md`; unresolved rows get a gate/owner. Each
|
|
135
137
|
`Existing behavior to preserve` row becomes a regression row whose current evidence is
|
|
136
138
|
observed green on the unchanged baseline as a prerequisite of the first slice touching that
|
|
@@ -208,7 +208,9 @@ plan declares a root-authored executable workflow file, read
|
|
|
208
208
|
`devrites-engine check readiness --emit-binding <slug>`; normal readiness check
|
|
209
209
|
must pass. Record the per-input drift baseline alongside it:
|
|
210
210
|
`devrites-engine check drift <slug> --record` — a later stale binding then
|
|
211
|
-
names the exact artifact that moved instead of "something changed".
|
|
211
|
+
names the exact artifact that moved instead of "something changed". An `ADVISORY
|
|
212
|
+
proportion:` line names a slice whose proof outgrew its ACs: move its rows without a
|
|
213
|
+
`Proves` spec line to `Deferred hardening` before READY, never deepen them. Technical failure records reproduction, not qid. Human gap awaits
|
|
212
214
|
Clarify. Optional cross-model follows `reference/cross-model.md` (trigger `cross-model`).
|
|
213
215
|
|
|
214
216
|
With READY, no pending remediation, and a valid technical return cursor,
|
|
@@ -128,9 +128,20 @@ prove only their named static criterion.
|
|
|
128
128
|
<the ASCII code-paths + user-flows diagram from review-axes.md, with COVERAGE / GAPS / REGRESSIONS line>
|
|
129
129
|
|
|
130
130
|
## Per-gap test requirements
|
|
131
|
-
| ID | Path / flow | Test file (match conventions) | Asserts (input → expected) | Kind | Slice | Priority |
|
|
132
|
-
|
|
133
|
-
| T1 | <path> | <path/to.test> | <empty list → 200 + []> | unit / E2E / eval | <slice> | P1 / **Regression-Critical** |
|
|
131
|
+
| ID | Path / flow | Test file (match conventions) | Asserts (input → expected) | Kind | Slice | Priority | Proves |
|
|
132
|
+
|---|---|---|---|---|---|---|---|
|
|
133
|
+
| T1 | <path> | <path/to.test> | <empty list → 200 + []> | unit / E2E / eval | <slice> | P1 / **Regression-Critical** | <AC/REQ/INV/PROH/DEC/EDGE ID or standard § + quoted line> |
|
|
134
|
+
|
|
135
|
+
Every row's `Proves` quotes the spec line or standard rule it proves. A row that proves
|
|
136
|
+
none is plan-invented hardening: move it to `Deferred hardening`.
|
|
137
|
+
|
|
138
|
+
## Deferred hardening
|
|
139
|
+
| ID | Hardening | Why no spec line needs it | Promote by |
|
|
140
|
+
|---|---|---|---|
|
|
141
|
+
| D1 | <proof or mechanism> | <reason> | <the spec line an owner would add> |
|
|
142
|
+
|
|
143
|
+
Deferred rows are an optional follow-up slice: not built, proven, or gating in this
|
|
144
|
+
feature; this table is their follow-up record. Nothing is deleted.
|
|
134
145
|
|
|
135
146
|
## Interaction inventory (UI slices — every element + flow gets an asserting test)
|
|
136
147
|
Enumerate every interactive element and user flow the feature exposes; one row each, none
|
|
@@ -159,7 +159,7 @@ Outcome: <findings | no-findings | gap>
|
|
|
159
159
|
Counts: <n per severity or kind used in the rows below>
|
|
160
160
|
Account: <admitted findings | No-findings | Gap per Result admission>
|
|
161
161
|
Coverage: <files read fully | hunk-level | skipped: name — reason>
|
|
162
|
-
Finding: <severity> conf <n> | <file:line> | <observed> | when <input/state/sequence> → <effect> | <minimum fix>
|
|
162
|
+
Finding: <severity> conf <n> | <file:line> | <observed> | when <input/state/sequence> → <effect> | breaks <ID or standard §> "<quoted line>" | <minimum fix>
|
|
163
163
|
Basis: <files read · commands run to reach this finding>
|
|
164
164
|
Contract: <applicable | not-applicable> — <basis: both sides read · crossing test>
|
|
165
165
|
Tests: <adequate? gaps>
|
|
@@ -22,7 +22,9 @@ seams supplied by the orchestrator.
|
|
|
22
22
|
Reuse existing seams and dependencies before proposing new ones. Build vertical
|
|
23
23
|
slices in dependency order; put a hub file's shared edit in one seam slice
|
|
24
24
|
([slicing.md](skill://rite-plan/reference/slicing.md#slice-independence)). Map every REQ and AC to a slice and executable proof,
|
|
25
|
-
name rollback and failure paths; keep acceptance-changing proposals separate.
|
|
25
|
+
name rollback and failure paths; keep acceptance-changing proposals separate. Each proof
|
|
26
|
+
row quotes the spec line or standard rule it proves (`Proves`); proof that proves none
|
|
27
|
+
goes to `test-plan.md` `## Deferred hardening`, never into a slice. Durable
|
|
26
28
|
proof commands must be portable repository commands, never host-local wrappers,
|
|
27
29
|
user-specific absolute paths, or temporary proof trees.
|
|
28
30
|
Reconcile `spec.md`'s applicability map against live evidence. When triggered, apply
|
|
@@ -117,7 +117,10 @@ criterion, slice, or proof.
|
|
|
117
117
|
`convergence_pressure` eligibility mechanical. Tag every Critical/Important
|
|
118
118
|
`kind: contract` (an implementer could build it differently and still pass the
|
|
119
119
|
planned tests) or `kind: mechanism` (a named test can discriminate the wrong
|
|
120
|
-
implementation; propose that case as the fix, not new plan prose).
|
|
120
|
+
implementation; propose that case as the fix, not new plan prose). Each
|
|
121
|
+
Critical/Important also quotes the spec line (`AC`/`REQ`/`INV`/`PROH`/`DEC`/`EDGE`)
|
|
122
|
+
or standard rule it breaks; hardening no spec line asks for is a Suggestion
|
|
123
|
+
([agents.md](skill://devrites-lib/reference/standards/agents.md) § Result admission, Proportionality).
|
|
121
124
|
- In a recheck, your verdict covers the listed fingerprints, the correction diff, and
|
|
122
125
|
the dependents of each changed clause. Return an explicit correction-created
|
|
123
126
|
regression verdict over the touched invariants and those dependents as
|
|
@@ -98,7 +98,7 @@ Each required reviewer/analyst/auditor starts with exactly one:
|
|
|
98
98
|
```text
|
|
99
99
|
Outcome: <findings | no-findings | gap>
|
|
100
100
|
Counts: <n per severity or kind used in the rows below>
|
|
101
|
-
Finding: <severity> conf <n> | <file:line or artifact section> | <observed quote/result> | when <input/state/sequence> → <effect> | <minimum fix>
|
|
101
|
+
Finding: <severity> conf <n> | <file:line or artifact section> | <observed quote/result> | when <input/state/sequence> → <effect> | breaks <ID or standard §> "<quoted line>" | <minimum fix>
|
|
102
102
|
Basis: <files read · commands run to reach this finding>
|
|
103
103
|
```
|
|
104
104
|
|
|
@@ -108,6 +108,13 @@ Basis: <files read · commands run to reach this finding>
|
|
|
108
108
|
A Critical/Important row carries `conf <n>` in its severity cell and writes impact
|
|
109
109
|
as `when <input/state/sequence> → <effect>`; missing either, root admits it as
|
|
110
110
|
`Unverified: <claim> — missing trigger` (stays `gap`).
|
|
111
|
+
- **Proportionality:** a Critical/Important row's `breaks` quotes the line it violates:
|
|
112
|
+
a spec ID (`AC`/`REQ`/`INV`/`PROH`/`DEC`/`EDGE`) or a shipped standard rule
|
|
113
|
+
(`<standard>.md § <section>`: security, regression, principles). Root anchors the
|
|
114
|
+
quote like the `file:line` quote. No anchor, or one the defect does not break,
|
|
115
|
+
admits the row as Suggestion logged `Deferred: <claim> — no spec anchor` for an
|
|
116
|
+
optional follow-up slice; it never drives a repair round. Hardening no spec line
|
|
117
|
+
asks for is a Suggestion, however plausible.
|
|
111
118
|
- **`no-findings`:** `No-findings:` names checks and inspected evidence. Bare
|
|
112
119
|
pass, empty list, or “looks good” is malformed.
|
|
113
120
|
- **`gap`:** names missing/unreadable/stale input; skipped/failed required check;
|
|
@@ -112,6 +112,10 @@ armed AFK workspace whose exact scope, gates, and budgets permit it.
|
|
|
112
112
|
run `devrites-engine check drift <slug>` before re-vetting: it names the
|
|
113
113
|
exact input artifact that changed (or `missing`/`added`) since the recorded
|
|
114
114
|
baseline, so the Spec Drift Guard or `/rite-vet` starts at the right file.
|
|
115
|
+
- `check drift` and `check slice` print `ADVISORY proportion:` when a slice's proof
|
|
116
|
+
grew over 2x since the Vet baseline or carries over 3x the median proof per AC.
|
|
117
|
+
Repair by moving rows without a `Proves` spec line to `Deferred hardening`
|
|
118
|
+
([agents.md](agents.md) § Result admission, Proportionality), not by adding proof.
|
|
115
119
|
- Native notifications fire only after evidence and stop state are written.
|
|
116
120
|
|
|
117
121
|
Use [`afk-hitl.md`](afk-hitl.md) for unattended authority and resource budgets,
|
|
@@ -29,8 +29,8 @@ Execute [`reference/phase-contract.md`](reference/phase-contract.md)
|
|
|
29
29
|
[`reference/wright-dispatch.md`](reference/wright-dispatch.md).
|
|
30
30
|
Every wright contract names the writer-safe checks the wright runs (syntax/type/lint,
|
|
31
31
|
unit and self-check suites, a narrow local smoke) and never forbids them; only
|
|
32
|
-
artifact-producing, consumptive, or full canonical gates stay root-owned
|
|
33
|
-
Prior contracts in `evidence.md` are not a template for this. **Failing case:** the
|
|
32
|
+
artifact-producing, consumptive, or full canonical gates stay root-owned;
|
|
33
|
+
`dispatch open --contract` refuses a contract that breaks this. Prior contracts in `evidence.md` are not a template for this. **Failing case:** the
|
|
34
34
|
contract says "skip tests mid-flight; root runs them after return", and the wright
|
|
35
35
|
returns three unit failures that a 20 s `go test` would have caught in-round.
|
|
36
36
|
|
|
@@ -23,7 +23,9 @@ DISPATCH → hand the slice contract to devrites-slice-wright (fresh context).
|
|
|
23
23
|
(independent claims, fresh contexts); wright fixes stay serial — writers share files
|
|
24
24
|
PROVE → root, concurrent with the reviews on the same frozen candidate:
|
|
25
25
|
required build/browser/E2E (narrow self-check/smoke first, full gate only
|
|
26
|
-
after it is green, long runs in the background
|
|
26
|
+
after it is green, long runs in the background; cancel one still running once
|
|
27
|
+
review or proof sends the candidate back, keeping its partial output as
|
|
28
|
+
diagnostics), then returned-diff/path recheck; fail-on-red
|
|
27
29
|
(red/gap → repair wright, then re-review and re-prove; do not commit yet)
|
|
28
30
|
RECORD → orchestrator: after green verification, update state.md/evidence.md and
|
|
29
31
|
upsert the authoritative candidate manifest from the actual scoped diff
|
|
@@ -94,10 +94,14 @@ claim add --session <id> <task-paths>`; retain its returned `<claim-id>`, then r
|
|
|
94
94
|
live foreign session releases the retained claim and stops the dispatch, per
|
|
95
95
|
[`agents.md`](skill://devrites-lib/reference/standards/agents.md). `.devrites/**`
|
|
96
96
|
is never a task path; see [Workflow Artifact admission](skill://devrites-lib/reference/standards/workflow-artifacts.md).
|
|
97
|
-
|
|
98
|
-
|
|
99
|
-
|
|
100
|
-
|
|
97
|
+
Write the exact contract, with its `Writer-safe checks:` line, to
|
|
98
|
+
`packets/<wave>.contract.md` and hand the wright that text verbatim. Ask the host
|
|
99
|
+
for the exact writer in fresh context; a repair round on the same slice whose
|
|
100
|
+
returned wright the host can still continue goes to that wright on its existing
|
|
101
|
+
handle instead, so it does not re-read the slice from scratch. Wait, and track it
|
|
102
|
+
through the launch barrier — `devrites-engine dispatch <slug> open --phase build --wave
|
|
103
|
+
<slice-id> --role slice-wright --slice <SLICE-ID> --contract <file>` (it refuses a
|
|
104
|
+
contract that forbids writer-safe checks or names none), then `start` with the host handle, `seal`, and
|
|
101
105
|
`return` on completion (auto-records dispatch/return metrics). Every 5 wright
|
|
102
106
|
rounds on one slice, `open` blocks for a reslice review: propose an
|
|
103
107
|
acceptance-preserving split (`root_no_change_analysis` → `FOLD` per
|
|
@@ -131,7 +131,9 @@ Pull these via `Read` when shaping the plan:
|
|
|
131
131
|
expand this into repository-wide coverage.
|
|
132
132
|
4. **Map coverage and wiring:** every `AC-###` spec acceptance criterion maps to ≥1 `SLICE-###`
|
|
133
133
|
([`acceptance-criteria.md`](skill://rite-spec/reference/acceptance-criteria.md)); no orphaned criteria, no slice without a
|
|
134
|
-
criterion.
|
|
134
|
+
criterion. Every proof row quotes the spec line it proves (`Proves`); proof that proves
|
|
135
|
+
none goes to `test-plan.md` `## Deferred hardening`, never into a slice
|
|
136
|
+
([artifacts.md](skill://rite-vet/reference/artifacts.md)). Lift covered/backstop `Edge Coverage` rows and resolved `Prohibitions (must-NOT)`
|
|
135
137
|
rows into `traceability.md` and `test-plan.md`; unresolved rows get a gate/owner. Each
|
|
136
138
|
`Existing behavior to preserve` row becomes a regression row whose current evidence is
|
|
137
139
|
observed green on the unchanged baseline as a prerequisite of the first slice touching that
|
|
@@ -209,7 +209,9 @@ plan declares a root-authored executable workflow file, read
|
|
|
209
209
|
`devrites-engine check readiness --emit-binding <slug>`; normal readiness check
|
|
210
210
|
must pass. Record the per-input drift baseline alongside it:
|
|
211
211
|
`devrites-engine check drift <slug> --record` — a later stale binding then
|
|
212
|
-
names the exact artifact that moved instead of "something changed".
|
|
212
|
+
names the exact artifact that moved instead of "something changed". An `ADVISORY
|
|
213
|
+
proportion:` line names a slice whose proof outgrew its ACs: move its rows without a
|
|
214
|
+
`Proves` spec line to `Deferred hardening` before READY, never deepen them. Technical failure records reproduction, not qid. Human gap awaits
|
|
213
215
|
Clarify. Optional cross-model follows `reference/cross-model.md` (trigger `cross-model`).
|
|
214
216
|
|
|
215
217
|
With READY, no pending remediation, and a valid technical return cursor,
|
|
@@ -128,9 +128,20 @@ prove only their named static criterion.
|
|
|
128
128
|
<the ASCII code-paths + user-flows diagram from review-axes.md, with COVERAGE / GAPS / REGRESSIONS line>
|
|
129
129
|
|
|
130
130
|
## Per-gap test requirements
|
|
131
|
-
| ID | Path / flow | Test file (match conventions) | Asserts (input → expected) | Kind | Slice | Priority |
|
|
132
|
-
|
|
133
|
-
| T1 | <path> | <path/to.test> | <empty list → 200 + []> | unit / E2E / eval | <slice> | P1 / **Regression-Critical** |
|
|
131
|
+
| ID | Path / flow | Test file (match conventions) | Asserts (input → expected) | Kind | Slice | Priority | Proves |
|
|
132
|
+
|---|---|---|---|---|---|---|---|
|
|
133
|
+
| T1 | <path> | <path/to.test> | <empty list → 200 + []> | unit / E2E / eval | <slice> | P1 / **Regression-Critical** | <AC/REQ/INV/PROH/DEC/EDGE ID or standard § + quoted line> |
|
|
134
|
+
|
|
135
|
+
Every row's `Proves` quotes the spec line or standard rule it proves. A row that proves
|
|
136
|
+
none is plan-invented hardening: move it to `Deferred hardening`.
|
|
137
|
+
|
|
138
|
+
## Deferred hardening
|
|
139
|
+
| ID | Hardening | Why no spec line needs it | Promote by |
|
|
140
|
+
|---|---|---|---|
|
|
141
|
+
| D1 | <proof or mechanism> | <reason> | <the spec line an owner would add> |
|
|
142
|
+
|
|
143
|
+
Deferred rows are an optional follow-up slice: not built, proven, or gating in this
|
|
144
|
+
feature; this table is their follow-up record. Nothing is deleted.
|
|
134
145
|
|
|
135
146
|
## Interaction inventory (UI slices — every element + flow gets an asserting test)
|
|
136
147
|
Enumerate every interactive element and user flow the feature exposes; one row each, none
|
|
@@ -160,7 +160,7 @@ Outcome: <findings | no-findings | gap>
|
|
|
160
160
|
Counts: <n per severity or kind used in the rows below>
|
|
161
161
|
Account: <admitted findings | No-findings | Gap per Result admission>
|
|
162
162
|
Coverage: <files read fully | hunk-level | skipped: name — reason>
|
|
163
|
-
Finding: <severity> conf <n> | <file:line> | <observed> | when <input/state/sequence> → <effect> | <minimum fix>
|
|
163
|
+
Finding: <severity> conf <n> | <file:line> | <observed> | when <input/state/sequence> → <effect> | breaks <ID or standard §> "<quoted line>" | <minimum fix>
|
|
164
164
|
Basis: <files read · commands run to reach this finding>
|
|
165
165
|
Contract: <applicable | not-applicable> — <basis: both sides read · crossing test>
|
|
166
166
|
Tests: <adequate? gaps>
|