devrites 6.1.1 → 6.1.3
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/cli.md +4 -1
- package/docs/command-map.md +1 -1
- package/docs/engine/commands.md +4 -4
- package/docs/orchestration.md +4 -3
- package/docs/usage.md +1 -1
- package/engine/commands.go +1 -1
- package/engine/help.go +5 -1
- package/engine/internal/acceptance/cli.go +14 -5
- package/engine/internal/acceptance/run.go +20 -5
- package/engine/internal/lib/apply_bundle.go +474 -0
- package/engine/internal/lib/claims.go +12 -2
- package/engine/internal/lib/claims_holder.go +124 -0
- package/engine/internal/lib/cli_observe.go +14 -0
- package/engine/internal/lib/diffscope.go +2 -1
- package/engine/internal/lib/taskgraph.go +124 -0
- package/engine/internal/parallel/cli.go +111 -9
- package/engine/internal/parallel/lease.go +58 -4
- package/engine/internal/parallel/ops.go +244 -75
- package/engine/internal/parallel/select.go +31 -6
- package/engine/main.go +4 -0
- package/pack/.claude/agents/devrites-plan-drafter.md +13 -3
- package/pack/.claude/agents/devrites-slice-wright.md +5 -1
- package/pack/.claude/skills/devrites-lib/reference/parallel-dispatch.md +14 -5
- package/pack/.claude/skills/devrites-lib/reference/standards/afk-hitl.md +24 -9
- package/pack/.claude/skills/devrites-lib/reference/standards/agents.md +6 -2
- package/pack/.claude/skills/devrites-lib/reference/standards/core.md +3 -0
- package/pack/.claude/skills/devrites-lib/reference/standards/gates.md +3 -0
- package/pack/.claude/skills/devrites-lib/reference/standards/testing.md +7 -0
- package/pack/.claude/skills/devrites-lib/reference/standards/workflow-artifacts.md +10 -6
- package/pack/.claude/skills/devrites-lib/reference/workspace-artifact-schema.md +1 -1
- package/pack/.claude/skills/rite-autocomplete/SKILL.md +2 -2
- package/pack/.claude/skills/rite-autocomplete/reference/loop.md +1 -1
- package/pack/.claude/skills/rite-autocomplete/reference/stop-conditions.md +6 -2
- package/pack/.claude/skills/rite-build/SKILL.md +7 -7
- package/pack/.claude/skills/rite-build/reference/afk-discipline.md +2 -2
- package/pack/.claude/skills/rite-build/reference/checkpoint.md +1 -1
- package/pack/.claude/skills/rite-build/reference/one-slice-cycle.md +3 -2
- package/pack/.claude/skills/rite-build/reference/parallel-batch.md +28 -21
- package/pack/.claude/skills/rite-build/reference/phase-contract.md +10 -1
- package/pack/.claude/skills/rite-build/reference/spec-drift-guard.md +3 -2
- package/pack/.claude/skills/rite-build/reference/wright-dispatch.md +9 -2
- package/pack/.claude/skills/rite-define/SKILL.md +11 -2
- package/pack/.claude/skills/rite-handoff/SKILL.md +1 -1
- package/pack/.claude/skills/rite-plan/SKILL.md +20 -9
- package/pack/.claude/skills/rite-plan/reference/slicing.md +5 -0
- package/pack/.claude/skills/rite-polish/SKILL.md +6 -4
- package/pack/.claude/skills/rite-prove/SKILL.md +3 -2
- package/pack/.claude/skills/rite-review/SKILL.md +6 -1
- package/pack/.claude/skills/rite-seal/SKILL.md +2 -2
- package/pack/.claude/skills/rite-seal/reference/phase-contract.md +7 -1
- package/pack/.claude/skills/rite-vet/SKILL.md +9 -5
- package/pack/.claude/skills/rite-vet/reference/artifacts.md +8 -0
- package/pack/.claude/skills/rite-vet/reference/review-axes.md +1 -1
- package/pack/generated/claude/agents/devrites-plan-drafter.md +13 -3
- package/pack/generated/claude/agents/devrites-slice-wright.md +5 -1
- package/pack/generated/claude/skills/devrites-lib/reference/parallel-dispatch.md +14 -5
- package/pack/generated/claude/skills/devrites-lib/reference/standards/afk-hitl.md +24 -9
- package/pack/generated/claude/skills/devrites-lib/reference/standards/agents.md +6 -2
- package/pack/generated/claude/skills/devrites-lib/reference/standards/core.md +3 -0
- package/pack/generated/claude/skills/devrites-lib/reference/standards/gates.md +3 -0
- package/pack/generated/claude/skills/devrites-lib/reference/standards/testing.md +7 -0
- package/pack/generated/claude/skills/devrites-lib/reference/standards/workflow-artifacts.md +10 -6
- package/pack/generated/claude/skills/devrites-lib/reference/workspace-artifact-schema.md +1 -1
- package/pack/generated/claude/skills/rite-autocomplete/SKILL.md +2 -2
- package/pack/generated/claude/skills/rite-autocomplete/reference/loop.md +1 -1
- package/pack/generated/claude/skills/rite-autocomplete/reference/stop-conditions.md +6 -2
- package/pack/generated/claude/skills/rite-build/SKILL.md +7 -7
- package/pack/generated/claude/skills/rite-build/reference/afk-discipline.md +2 -2
- package/pack/generated/claude/skills/rite-build/reference/checkpoint.md +1 -1
- package/pack/generated/claude/skills/rite-build/reference/one-slice-cycle.md +3 -2
- package/pack/generated/claude/skills/rite-build/reference/parallel-batch.md +28 -21
- package/pack/generated/claude/skills/rite-build/reference/phase-contract.md +10 -1
- package/pack/generated/claude/skills/rite-build/reference/spec-drift-guard.md +3 -2
- package/pack/generated/claude/skills/rite-build/reference/wright-dispatch.md +9 -2
- package/pack/generated/claude/skills/rite-define/SKILL.md +11 -2
- package/pack/generated/claude/skills/rite-handoff/SKILL.md +1 -1
- package/pack/generated/claude/skills/rite-plan/SKILL.md +20 -9
- package/pack/generated/claude/skills/rite-plan/reference/slicing.md +5 -0
- package/pack/generated/claude/skills/rite-polish/SKILL.md +6 -4
- package/pack/generated/claude/skills/rite-prove/SKILL.md +3 -2
- package/pack/generated/claude/skills/rite-review/SKILL.md +6 -1
- package/pack/generated/claude/skills/rite-seal/SKILL.md +2 -2
- package/pack/generated/claude/skills/rite-seal/reference/phase-contract.md +7 -1
- package/pack/generated/claude/skills/rite-vet/SKILL.md +9 -5
- package/pack/generated/claude/skills/rite-vet/reference/artifacts.md +8 -0
- package/pack/generated/claude/skills/rite-vet/reference/review-axes.md +1 -1
- package/pack/generated/codex/agents/devrites-plan-drafter.toml +13 -3
- package/pack/generated/codex/agents/devrites-slice-wright.toml +5 -1
- package/pack/generated/codex/skills/devrites-lib/reference/parallel-dispatch.md +14 -5
- package/pack/generated/codex/skills/devrites-lib/reference/standards/afk-hitl.md +24 -9
- package/pack/generated/codex/skills/devrites-lib/reference/standards/agents.md +6 -2
- package/pack/generated/codex/skills/devrites-lib/reference/standards/core.md +3 -0
- package/pack/generated/codex/skills/devrites-lib/reference/standards/gates.md +3 -0
- package/pack/generated/codex/skills/devrites-lib/reference/standards/testing.md +7 -0
- package/pack/generated/codex/skills/devrites-lib/reference/standards/workflow-artifacts.md +10 -6
- package/pack/generated/codex/skills/devrites-lib/reference/workspace-artifact-schema.md +1 -1
- package/pack/generated/codex/skills/rite-autocomplete/SKILL.md +2 -2
- package/pack/generated/codex/skills/rite-autocomplete/reference/loop.md +1 -1
- package/pack/generated/codex/skills/rite-autocomplete/reference/stop-conditions.md +6 -2
- package/pack/generated/codex/skills/rite-build/SKILL.md +7 -7
- package/pack/generated/codex/skills/rite-build/reference/afk-discipline.md +2 -2
- package/pack/generated/codex/skills/rite-build/reference/checkpoint.md +1 -1
- package/pack/generated/codex/skills/rite-build/reference/one-slice-cycle.md +3 -2
- package/pack/generated/codex/skills/rite-build/reference/parallel-batch.md +28 -21
- package/pack/generated/codex/skills/rite-build/reference/phase-contract.md +10 -1
- package/pack/generated/codex/skills/rite-build/reference/spec-drift-guard.md +3 -2
- package/pack/generated/codex/skills/rite-build/reference/wright-dispatch.md +9 -2
- package/pack/generated/codex/skills/rite-define/SKILL.md +11 -2
- package/pack/generated/codex/skills/rite-handoff/SKILL.md +1 -1
- package/pack/generated/codex/skills/rite-plan/SKILL.md +20 -9
- package/pack/generated/codex/skills/rite-plan/reference/slicing.md +5 -0
- package/pack/generated/codex/skills/rite-polish/SKILL.md +6 -4
- package/pack/generated/codex/skills/rite-prove/SKILL.md +3 -2
- package/pack/generated/codex/skills/rite-review/SKILL.md +6 -1
- package/pack/generated/codex/skills/rite-seal/SKILL.md +2 -2
- package/pack/generated/codex/skills/rite-seal/reference/phase-contract.md +7 -1
- package/pack/generated/codex/skills/rite-vet/SKILL.md +9 -5
- package/pack/generated/codex/skills/rite-vet/reference/artifacts.md +8 -0
- package/pack/generated/codex/skills/rite-vet/reference/review-axes.md +1 -1
- package/pack/generated/devin/agents/devrites-plan-drafter.md +13 -3
- package/pack/generated/devin/agents/devrites-slice-wright.md +5 -1
- package/pack/generated/devin/skills/devrites-lib/reference/parallel-dispatch.md +14 -5
- package/pack/generated/devin/skills/devrites-lib/reference/standards/afk-hitl.md +24 -9
- package/pack/generated/devin/skills/devrites-lib/reference/standards/agents.md +6 -2
- package/pack/generated/devin/skills/devrites-lib/reference/standards/core.md +3 -0
- package/pack/generated/devin/skills/devrites-lib/reference/standards/gates.md +3 -0
- package/pack/generated/devin/skills/devrites-lib/reference/standards/testing.md +7 -0
- package/pack/generated/devin/skills/devrites-lib/reference/standards/workflow-artifacts.md +10 -6
- package/pack/generated/devin/skills/devrites-lib/reference/workspace-artifact-schema.md +1 -1
- package/pack/generated/devin/skills/rite-autocomplete/SKILL.md +2 -2
- package/pack/generated/devin/skills/rite-autocomplete/reference/loop.md +1 -1
- package/pack/generated/devin/skills/rite-autocomplete/reference/stop-conditions.md +6 -2
- package/pack/generated/devin/skills/rite-build/SKILL.md +7 -7
- package/pack/generated/devin/skills/rite-build/reference/afk-discipline.md +2 -2
- package/pack/generated/devin/skills/rite-build/reference/checkpoint.md +1 -1
- package/pack/generated/devin/skills/rite-build/reference/one-slice-cycle.md +3 -2
- package/pack/generated/devin/skills/rite-build/reference/parallel-batch.md +28 -21
- package/pack/generated/devin/skills/rite-build/reference/phase-contract.md +10 -1
- package/pack/generated/devin/skills/rite-build/reference/spec-drift-guard.md +3 -2
- package/pack/generated/devin/skills/rite-build/reference/wright-dispatch.md +9 -2
- package/pack/generated/devin/skills/rite-define/SKILL.md +11 -2
- package/pack/generated/devin/skills/rite-handoff/SKILL.md +1 -1
- package/pack/generated/devin/skills/rite-plan/SKILL.md +20 -9
- package/pack/generated/devin/skills/rite-plan/reference/slicing.md +5 -0
- package/pack/generated/devin/skills/rite-polish/SKILL.md +6 -4
- package/pack/generated/devin/skills/rite-prove/SKILL.md +3 -2
- package/pack/generated/devin/skills/rite-review/SKILL.md +6 -1
- package/pack/generated/devin/skills/rite-seal/SKILL.md +2 -2
- package/pack/generated/devin/skills/rite-seal/reference/phase-contract.md +7 -1
- package/pack/generated/devin/skills/rite-vet/SKILL.md +9 -5
- package/pack/generated/devin/skills/rite-vet/reference/artifacts.md +8 -0
- package/pack/generated/devin/skills/rite-vet/reference/review-axes.md +1 -1
- package/pack/generated/omp/agents/devrites-plan-drafter.md +13 -3
- package/pack/generated/omp/agents/devrites-slice-wright.md +5 -1
- package/pack/generated/omp/skills/devrites-lib/reference/parallel-dispatch.md +14 -5
- package/pack/generated/omp/skills/devrites-lib/reference/standards/afk-hitl.md +24 -9
- package/pack/generated/omp/skills/devrites-lib/reference/standards/agents.md +6 -2
- package/pack/generated/omp/skills/devrites-lib/reference/standards/core.md +3 -0
- package/pack/generated/omp/skills/devrites-lib/reference/standards/gates.md +3 -0
- package/pack/generated/omp/skills/devrites-lib/reference/standards/testing.md +7 -0
- package/pack/generated/omp/skills/devrites-lib/reference/standards/workflow-artifacts.md +10 -6
- package/pack/generated/omp/skills/devrites-lib/reference/workspace-artifact-schema.md +1 -1
- package/pack/generated/omp/skills/rite-autocomplete/SKILL.md +2 -2
- package/pack/generated/omp/skills/rite-autocomplete/reference/loop.md +1 -1
- package/pack/generated/omp/skills/rite-autocomplete/reference/stop-conditions.md +6 -2
- package/pack/generated/omp/skills/rite-build/SKILL.md +7 -7
- package/pack/generated/omp/skills/rite-build/reference/afk-discipline.md +2 -2
- package/pack/generated/omp/skills/rite-build/reference/checkpoint.md +1 -1
- package/pack/generated/omp/skills/rite-build/reference/one-slice-cycle.md +3 -2
- package/pack/generated/omp/skills/rite-build/reference/parallel-batch.md +28 -21
- package/pack/generated/omp/skills/rite-build/reference/phase-contract.md +10 -1
- package/pack/generated/omp/skills/rite-build/reference/spec-drift-guard.md +3 -2
- package/pack/generated/omp/skills/rite-build/reference/wright-dispatch.md +9 -2
- package/pack/generated/omp/skills/rite-define/SKILL.md +11 -2
- package/pack/generated/omp/skills/rite-handoff/SKILL.md +1 -1
- package/pack/generated/omp/skills/rite-plan/SKILL.md +20 -9
- package/pack/generated/omp/skills/rite-plan/reference/slicing.md +5 -0
- package/pack/generated/omp/skills/rite-polish/SKILL.md +6 -4
- package/pack/generated/omp/skills/rite-prove/SKILL.md +3 -2
- package/pack/generated/omp/skills/rite-review/SKILL.md +6 -1
- package/pack/generated/omp/skills/rite-seal/SKILL.md +2 -2
- package/pack/generated/omp/skills/rite-seal/reference/phase-contract.md +7 -1
- package/pack/generated/omp/skills/rite-vet/SKILL.md +9 -5
- package/pack/generated/omp/skills/rite-vet/reference/artifacts.md +8 -0
- package/pack/generated/omp/skills/rite-vet/reference/review-axes.md +1 -1
- package/pack/generated/pi/agents/devrites-plan-drafter.md +13 -3
- package/pack/generated/pi/agents/devrites-slice-wright.md +5 -1
- package/pack/generated/pi/skills/devrites-lib/reference/parallel-dispatch.md +14 -5
- package/pack/generated/pi/skills/devrites-lib/reference/standards/afk-hitl.md +24 -9
- package/pack/generated/pi/skills/devrites-lib/reference/standards/agents.md +6 -2
- package/pack/generated/pi/skills/devrites-lib/reference/standards/core.md +3 -0
- package/pack/generated/pi/skills/devrites-lib/reference/standards/gates.md +3 -0
- package/pack/generated/pi/skills/devrites-lib/reference/standards/testing.md +7 -0
- package/pack/generated/pi/skills/devrites-lib/reference/standards/workflow-artifacts.md +10 -6
- package/pack/generated/pi/skills/devrites-lib/reference/workspace-artifact-schema.md +1 -1
- package/pack/generated/pi/skills/rite-autocomplete/SKILL.md +2 -2
- package/pack/generated/pi/skills/rite-autocomplete/reference/loop.md +1 -1
- package/pack/generated/pi/skills/rite-autocomplete/reference/stop-conditions.md +6 -2
- package/pack/generated/pi/skills/rite-build/SKILL.md +7 -7
- package/pack/generated/pi/skills/rite-build/reference/afk-discipline.md +2 -2
- package/pack/generated/pi/skills/rite-build/reference/checkpoint.md +1 -1
- package/pack/generated/pi/skills/rite-build/reference/one-slice-cycle.md +3 -2
- package/pack/generated/pi/skills/rite-build/reference/parallel-batch.md +28 -21
- package/pack/generated/pi/skills/rite-build/reference/phase-contract.md +10 -1
- package/pack/generated/pi/skills/rite-build/reference/spec-drift-guard.md +3 -2
- package/pack/generated/pi/skills/rite-build/reference/wright-dispatch.md +9 -2
- package/pack/generated/pi/skills/rite-define/SKILL.md +11 -2
- package/pack/generated/pi/skills/rite-handoff/SKILL.md +1 -1
- package/pack/generated/pi/skills/rite-plan/SKILL.md +20 -9
- package/pack/generated/pi/skills/rite-plan/reference/slicing.md +5 -0
- package/pack/generated/pi/skills/rite-polish/SKILL.md +6 -4
- package/pack/generated/pi/skills/rite-prove/SKILL.md +3 -2
- package/pack/generated/pi/skills/rite-review/SKILL.md +6 -1
- package/pack/generated/pi/skills/rite-seal/SKILL.md +2 -2
- package/pack/generated/pi/skills/rite-seal/reference/phase-contract.md +7 -1
- package/pack/generated/pi/skills/rite-vet/SKILL.md +9 -5
- package/pack/generated/pi/skills/rite-vet/reference/artifacts.md +8 -0
- package/pack/generated/pi/skills/rite-vet/reference/review-axes.md +1 -1
- package/package.json +1 -1
|
@@ -76,7 +76,8 @@ test changes.
|
|
|
76
76
|
command, cwd, prerequisites, exit, and sanitized
|
|
77
77
|
decisive output, plus the test selector, runner counts
|
|
78
78
|
(collected/ran/passed/failed/skipped), and attempt number with first-attempt
|
|
79
|
-
outcome; output and exit come from one execution. Run relevant suite plus build/typecheck/lint
|
|
79
|
+
outcome; output and exit come from one execution. Run relevant suite plus build/typecheck/lint,
|
|
80
|
+
concurrently when they share no build directory, port, or lock. Recheck the
|
|
80
81
|
candidate and require identical digest/no source mutation. Reject substituted
|
|
81
82
|
commands, malformed manifests, zero-test/skipped/filtered behavioral claims,
|
|
82
83
|
exit-status-only claims, status-masking forms, and source drift. A retry-only
|
|
@@ -109,7 +110,7 @@ test changes.
|
|
|
109
110
|
described screenshots, console/network, interaction, target comparisons, and
|
|
110
111
|
every required state. Material mismatch fails.
|
|
111
112
|
7. **Independent validation.** Fresh read-only `devrites-proof-runner` and
|
|
112
|
-
`devrites-spec-reviewer
|
|
113
|
+
`devrites-spec-reviewer`, in one parallel wave, judge the same candidate/commands/evidence/acceptance map.
|
|
113
114
|
Neither runs commands; reject missing/stale, invented-ID, label-only or self-attested reports.
|
|
114
115
|
8. **Map proof.** Reconcile verdicts under [acceptance-proof.md](reference/acceptance-proof.md).
|
|
115
116
|
Every criterion/scenario/interaction/key link needs discriminating evidence or a
|
|
@@ -79,7 +79,8 @@ Pull these via `Read` when the diff demands them:
|
|
|
79
79
|
reviewer's brief without severity) and ends in `review.md` with its own label and
|
|
80
80
|
action tag — never silently dropped. Hand each lane reviewer the
|
|
81
81
|
`devrites-engine recall --paths <its touched paths>` pointers as prior context, not findings.
|
|
82
|
-
2. **Review tests first
|
|
82
|
+
2. **Review tests first** while the step-3 cohort runs — launch it first; this
|
|
83
|
+
root pass needs none of its output: do they prove the acceptance criteria? Missing,
|
|
83
84
|
weak, or wrong tests are the first findings.
|
|
84
85
|
**Completion:** every acceptance criterion maps to a proven test or a labeled finding.
|
|
85
86
|
3. **Review spec and code separately in parallel.** A change can pass
|
|
@@ -100,6 +101,10 @@ Pull these via `Read` when the diff demands them:
|
|
|
100
101
|
(scope creep / drift), (c) criteria implemented incorrectly. Quote the spec
|
|
101
102
|
line per finding. If the spec is missing or unreadable, report `Review gap:
|
|
102
103
|
missing spec` and do not infer requirements from the diff."
|
|
104
|
+
- **Seal roles** → `devrites-test-analyst`, plus `devrites-doubt-reviewer` per
|
|
105
|
+
stood decision and `devrites-devex-reviewer` when triggered, join this cohort on
|
|
106
|
+
the same digest. Their findings join step 7's single correction, and Seal cites
|
|
107
|
+
their unchanged same-candidate accounts instead of re-dispatching them.
|
|
103
108
|
- **Code-review axis** → `devrites-code-reviewer`: "Apply your full documented
|
|
104
109
|
discipline (tests-first, correctness, readability, architecture, maintainability,
|
|
105
110
|
standards) on the feature workspace + diff. Cite file:line; skip tooling-enforced
|
|
@@ -61,8 +61,8 @@ Follow [`reference/phase-contract.md`](reference/phase-contract.md)
|
|
|
61
61
|
|
|
62
62
|
1. read workspace and final diff;
|
|
63
63
|
2. recheck the canonical candidate/bindings and approved repository/CI proof;
|
|
64
|
-
3. admit valid
|
|
65
|
-
`../devrites-lib/reference/parallel-dispatch.md` on the same frozen candidate,
|
|
64
|
+
3. admit every valid same-candidate Review account; dispatch remaining required exact roles under
|
|
65
|
+
`../devrites-lib/reference/parallel-dispatch.md` in one wave, proof-runner included, on the same frozen candidate,
|
|
66
66
|
code review through its § Engineering lanes;
|
|
67
67
|
4. reconcile their read-only verdicts against source and immutable evidence;
|
|
68
68
|
5. draft GO/NO-GO `seal.md` with acceptance, tests, decisions, and seven roster accounts plus proof-runner and doubt-reviewer
|
|
@@ -13,12 +13,18 @@
|
|
|
13
13
|
candidate check afterward and require the identical digest and no source
|
|
14
14
|
mutation. Capture command, cwd, exit, decisive output, and identity.
|
|
15
15
|
Missing/red/mismatched blocks; unavailable is `cannot_verify`, never pass.
|
|
16
|
+
Steps 3–6 share one dispatch wave: launch every role they still need
|
|
17
|
+
(proof-runner, test-analyst, doubt-reviewer, roster) together, admit valid
|
|
18
|
+
same-candidate Review accounts instead, and do the root work of steps 4–6 while
|
|
19
|
+
the wave runs.
|
|
20
|
+
|
|
16
21
|
3. **Judge acceptance.** Dispatch exact `devrites-proof-runner` on immutable
|
|
17
22
|
proof; admit valid same-candidate `devrites-spec-reviewer` evidence from Review
|
|
18
23
|
or dispatch that exact reviewer. Reconcile every
|
|
19
24
|
AC/REQ/scenario/key link by ID and meaning; missing/stale/self-attested/mismatched is NO-GO.
|
|
20
25
|
4. **Judge tests.** Diff baseline/candidate tests for deletion, skip/focus,
|
|
21
|
-
tautology, or weakening;
|
|
26
|
+
tautology, or weakening; admit the same-candidate `devrites-test-analyst` account
|
|
27
|
+
from Review or dispatch that exact analyst. Missing or
|
|
22
28
|
adverse account, or green tests not asserting mapped behavior, is Critical.
|
|
23
29
|
5. **Resolve gates.** Open blocking/escalating/validating questions and drift
|
|
24
30
|
block. Each stood decision needs a reconciled exact `devrites-doubt-reviewer` verdict.
|
|
@@ -5,7 +5,7 @@ argument-hint: "[slug] [--cross-model] [--full]"
|
|
|
5
5
|
user-invocable: true
|
|
6
6
|
---
|
|
7
7
|
|
|
8
|
-
<!-- loads: {"always":["devrites-lib/reference/standards/core.md","devrites-lib/reference/standards/code-navigation.md","devrites-lib/reference/orchestration-profiles.md","devrites-lib/reference/parallel-dispatch.md","devrites-lib/reference/standards/gates.md","rite-vet/reference/review-axes.md","rite-vet/reference/artifacts.md","rite-vet/reference/depth.md","rite-vet/reference/eng-lenses.md","rite-vet/reference/anti-patterns.md"],"triggers":{"afk":["devrites-lib/reference/standards/afk-hitl.md"],"applicability":["devrites-lib/reference/standards/repository-topology.md","devrites-lib/reference/standards/data-integrity.md","devrites-lib/reference/standards/integration-reliability.md"],"assumption-delta":["devrites-lib/reference/standards/assumption-checkpoints.md"],"cross-model":["rite-vet/reference/cross-model.md"],"devex":["devrites-lib/reference/standards/developer-experience.md"],"dod":["devrites-lib/reference/standards/definition-of-done.md"],"elicitation":["devrites-lib/reference/standards/elicitation.md"],"errors":["devrites-lib/reference/standards/error-handling.md"],"grammar":["devrites-lib/reference/standards/spec-grammar.md"],"one-shot":["devrites-lib/reference/standards/one-shot-actions.md"],"performance":["devrites-lib/reference/standards/performance.md"],"principles":["devrites-lib/reference/standards/principles.md"],"reslice":["devrites-lib/reference/standards/acceptance-preserving-reslice.md"],"testing":["devrites-lib/reference/standards/testing.md"],"workflow":["devrites-lib/reference/standards/development-workflow.md"],"workflow-artifacts":["devrites-lib/reference/standards/workflow-artifacts.md"],"yagni":["devrites-lib/reference/standards/patterns.md","devrites-lib/reference/standards/coding-style.md"]},"workspace":["brief.md","spec.md","state.md","decisions.md","assumptions.md","questions.md","decision-coverage.md","strategy.md","architecture.md","plan.md","tasks.md","traceability.md","eng-review.md","test-plan.md","gates.md"],"workspaceByRole":{"devex-reviewer":["spec.md","plan.md","tasks.md","decisions.md","state.md"],"plan-reviewer":["spec.md","plan.md","tasks.md","test-plan.md","decision-coverage.md","architecture.md","decisions.md","state.md"],"strategy-reviewer":["brief.md","spec.md","plan.md","decisions.md","decision-coverage.md","strategy.md","state.md"]}} -->
|
|
8
|
+
<!-- loads: {"always":["devrites-lib/reference/standards/core.md","devrites-lib/reference/standards/code-navigation.md","devrites-lib/reference/orchestration-profiles.md","devrites-lib/reference/parallel-dispatch.md","devrites-lib/reference/standards/gates.md","rite-vet/reference/review-axes.md","rite-vet/reference/artifacts.md","rite-vet/reference/depth.md","rite-vet/reference/eng-lenses.md","rite-vet/reference/anti-patterns.md"],"triggers":{"afk":["devrites-lib/reference/standards/afk-hitl.md"],"applicability":["devrites-lib/reference/standards/repository-topology.md","devrites-lib/reference/standards/data-integrity.md","devrites-lib/reference/standards/integration-reliability.md"],"assumption-delta":["devrites-lib/reference/standards/assumption-checkpoints.md"],"cross-model":["rite-vet/reference/cross-model.md"],"devex":["devrites-lib/reference/standards/developer-experience.md"],"dod":["devrites-lib/reference/standards/definition-of-done.md"],"elicitation":["devrites-lib/reference/standards/elicitation.md"],"errors":["devrites-lib/reference/standards/error-handling.md"],"grammar":["devrites-lib/reference/standards/spec-grammar.md"],"one-shot":["devrites-lib/reference/standards/one-shot-actions.md"],"performance":["devrites-lib/reference/standards/performance.md"],"principles":["devrites-lib/reference/standards/principles.md"],"reslice":["devrites-lib/reference/standards/acceptance-preserving-reslice.md"],"testing":["devrites-lib/reference/standards/testing.md"],"workflow":["devrites-lib/reference/standards/development-workflow.md"],"workflow-artifacts":["devrites-lib/reference/standards/workflow-artifacts.md"],"yagni":["devrites-lib/reference/standards/patterns.md","devrites-lib/reference/standards/coding-style.md"]},"workspace":["brief.md","spec.md","state.md","decisions.md","assumptions.md","questions.md","decision-coverage.md","strategy.md","architecture.md","plan.md","tasks.md","traceability.md","eng-review.md","test-plan.md","gates.md"],"rootOnly":["devrites-lib/reference/standards/afk-hitl.md","devrites-lib/reference/standards/elicitation.md"],"workspaceByRole":{"doubt-reviewer":["spec.md","architecture.md","plan.md","tasks.md","test-plan.md","decisions.md","state.md"],"devex-reviewer":["spec.md","plan.md","tasks.md","decisions.md","state.md"],"plan-reviewer":["spec.md","plan.md","tasks.md","test-plan.md","decision-coverage.md","architecture.md","decisions.md","state.md"],"strategy-reviewer":["brief.md","spec.md","plan.md","decisions.md","decision-coverage.md","strategy.md","state.md"]}} -->
|
|
9
9
|
> Read-set manifest: `devrites-engine context <slug> --phase vet` bundles every file named below into one deduplicated read. Trigger names map to the conditional rules in the sections that follow.
|
|
10
10
|
|
|
11
11
|
|
|
@@ -83,7 +83,9 @@ plan declares a root-authored executable workflow file, read
|
|
|
83
83
|
Temper — in
|
|
84
84
|
parallel under `../devrites-lib/reference/parallel-dispatch.md`; they are
|
|
85
85
|
independent read-only passes on one frozen candidate. Missing
|
|
86
|
-
required account blocks.
|
|
86
|
+
required account blocks. Run root steps 2–4, including step 3's preflight
|
|
87
|
+
commands, on the same frozen candidate while the wave is out; reconcile the
|
|
88
|
+
accounts when they return.
|
|
87
89
|
1b. **Recovery recheck.** Require valid return cursor, accepted prior finding,
|
|
88
90
|
exact fingerprint/reproduction, repaired candidate identity, changed
|
|
89
91
|
paths/criteria, affected drift/evidence, and the recorded delta self-check.
|
|
@@ -106,8 +108,8 @@ plan declares a root-authored executable workflow file, read
|
|
|
106
108
|
marked as a refinement of an already-pinned clause; the recheck may re-raise
|
|
107
109
|
it only by naming a concrete failure path the declaration omits. A
|
|
108
110
|
`new_failure_mode` finding is never residual-eligible. A fold whose
|
|
109
|
-
findings are all Suggestion-or-lower rechecks mechanically at the
|
|
110
|
-
materialization gate — anchors applied, mirrors consistent, invariants
|
|
111
|
+
findings are all `mechanism` kind or Suggestion-or-lower rechecks mechanically at the
|
|
112
|
+
materialization gate — anchors applied (a clean `state apply-bundle`), mirrors consistent, invariants
|
|
111
113
|
pass — without a dispatched reviewer; sub-Important findings already
|
|
112
114
|
cannot keep recovery open. Reconcile
|
|
113
115
|
shared artifact/readiness gates, then enter step 8 or the next repair.
|
|
@@ -140,7 +142,9 @@ plan declares a root-authored executable workflow file, read
|
|
|
140
142
|
ownership. The plan's `Shared contract proof` names one reused boundary
|
|
141
143
|
artifact plus two consuming tests for every changed API/event/schema/provider-
|
|
142
144
|
consumer seam, or an explicit no-impact statement. Missing, one-sided,
|
|
143
|
-
duplicated-contract, vague, or non-consuming proof fails closed.
|
|
145
|
+
duplicated-contract, vague, or non-consuming proof fails closed. Each slice
|
|
146
|
+
names a fast writer-safe check its wright can run; proof reachable only through
|
|
147
|
+
root-owned full gates is a `mechanism` finding.
|
|
144
148
|
|
|
145
149
|
Technical gaps are `NEEDS REPLAN` and Plan repair — each finding marks
|
|
146
150
|
itself `new_failure_mode` or names the already-pinned clause/fingerprint
|
|
@@ -101,6 +101,14 @@ From /rite-vet on <iso>. Runner + conventions: <detected framework + command>.
|
|
|
101
101
|
Link any disposable parser fixture. Preflight values are planning evidence, not post-build proof;
|
|
102
102
|
Prove recaptures the observed command, candidate identity, and result.
|
|
103
103
|
|
|
104
|
+
## Feature gates
|
|
105
|
+
| Gate | Command | Cwd | Expected |
|
|
106
|
+
|---|---|---|---|
|
|
107
|
+
| <full suite / workspace lint / audit> | <exact command> | <path> | <exit/output> |
|
|
108
|
+
|
|
109
|
+
Whole-repository rows only: Build runs them per landing
|
|
110
|
+
([tiers](../../rite-build/reference/phase-contract.md#proof-tiers)), Prove on the final candidate.
|
|
111
|
+
|
|
104
112
|
Commands in this durable artifact are portable repository commands: no RTK or local shell
|
|
105
113
|
aliases, user-specific absolute paths, or temporary proof trees. Evidence records
|
|
106
114
|
the command actually executed.
|
|
@@ -28,7 +28,7 @@ Before implementation review, check for work beyond the settled spec.
|
|
|
28
28
|
(over-budget `tasks.md`/`plan.md` from slice or AC count) is a scope finding, not a
|
|
29
29
|
`Budget override`: route to `/rite-plan course-correct` (`MVP cut`) with the
|
|
30
30
|
continuation sequence named; do not vet the whole epic. An `ADVISORY size:`
|
|
31
|
-
|
|
31
|
+
or `hub:` line without its rationale is a non-blocking Suggestion to split or seam it.
|
|
32
32
|
4. **Built-in check.** For each new pattern/infra/concurrency approach, invoke
|
|
33
33
|
`devrites-source-driven` inline to verify current framework/runtime support and cite it. Custom
|
|
34
34
|
work where a built-in suffices is a scope-reduction finding.
|
|
@@ -21,7 +21,8 @@ seams supplied by the orchestrator.
|
|
|
21
21
|
planning artifacts while preserving settled intent.
|
|
22
22
|
|
|
23
23
|
Reuse existing seams and dependencies before proposing new ones. Build vertical
|
|
24
|
-
slices in dependency order
|
|
24
|
+
slices in dependency order; put a hub file's shared edit in one seam slice
|
|
25
|
+
([slicing.md](../skills/rite-plan/reference/slicing.md#slice-independence)). Map every REQ and AC to a slice and executable proof,
|
|
25
26
|
name rollback and failure paths; keep acceptance-changing proposals separate. Durable
|
|
26
27
|
proof commands must be portable repository commands, never host-local wrappers,
|
|
27
28
|
user-specific absolute paths, or temporary proof trees.
|
|
@@ -75,10 +76,13 @@ Return this result:
|
|
|
75
76
|
|
|
76
77
|
```yaml
|
|
77
78
|
mode: define | repair
|
|
79
|
+
base_hashes: {} # <path>: sha256 of the bytes you read, for every replace/patch target
|
|
78
80
|
candidate_files:
|
|
79
81
|
- path: <requested canonical target>
|
|
80
82
|
operation: create | replace | patch
|
|
81
|
-
content: <complete
|
|
83
|
+
content: <complete file; create/replace only>
|
|
84
|
+
hunks: [] # patch only: {id, old_text, new_text}; old_text byte-exact, unique in the pinned file
|
|
85
|
+
retired_ids: [] # tracked IDs (AC/REQ/HZN/SLICE/DEC/...) the delta removes on purpose
|
|
82
86
|
acceptance_map: []
|
|
83
87
|
dependency_order: []
|
|
84
88
|
technical_decisions: []
|
|
@@ -88,7 +92,13 @@ applicability_disposition: []
|
|
|
88
92
|
self_attack_exposures: []
|
|
89
93
|
```
|
|
90
94
|
|
|
91
|
-
The root
|
|
95
|
+
The root saves the bundle as JSON and applies it with `devrites-engine state
|
|
96
|
+
apply-bundle <slug> <bundle.json>`, which refuses a stale pin, a missing,
|
|
97
|
+
duplicate, or overlapping `old_text`, an unlisted vanished ID, a new task-graph
|
|
98
|
+
problem, new budget overshoot, or a new dated delta section, and writes nothing on refusal. Pin each
|
|
99
|
+
`base_hashes` entry with `shasum -a 256` (or `sha256sum`) on the exact file you
|
|
100
|
+
read, and prefer short anchored hunks to whole-file `replace`. Patch the owning
|
|
101
|
+
clause in place; never add a dated delta section beside superseded text.
|
|
92
102
|
|
|
93
103
|
## Tools / read-write mode
|
|
94
104
|
|
|
@@ -103,7 +103,11 @@ unclear boundary is underspecified: return `Escalation` without proceeding.
|
|
|
103
103
|
that source in the result. Never invent an API.
|
|
104
104
|
- **No process residue.** Do not put `SLICE-###`, slice numbers, or "this slice"
|
|
105
105
|
in product source, tests, comments, or filenames.
|
|
106
|
-
4. **VERIFY (fail-on-red).** Run writer-safe tests/types/lint
|
|
106
|
+
4. **VERIFY (fail-on-red).** Run writer-safe tests/types/lint: `Tests/proof` plus
|
|
107
|
+
tests of changed code; `## Feature gates` rows are root-owned. Writer-safe means
|
|
108
|
+
local and non-consumptive — syntax/type/lint, unit and self-check suites, and a
|
|
109
|
+
narrow smoke of the changed path — never only a syntax check; contract text
|
|
110
|
+
cannot strip it. Report required
|
|
107
111
|
build/browser/E2E as `not-run` (root-owned, artifact-producing gates), and only
|
|
108
112
|
when the exact command, cwd, and prerequisites already appear in the unchanged
|
|
109
113
|
vetted `test-plan.md`; never synthesize or rewrite a root command. Root runs
|
|
@@ -4,7 +4,9 @@ Host dispatch; DevRites governs.
|
|
|
4
4
|
|
|
5
5
|
## Roster
|
|
6
6
|
|
|
7
|
-
Review: Spec/Code
|
|
7
|
+
Review: Spec/Code, triggered frontend/security/performance, and the seal roles that can
|
|
8
|
+
judge the frozen digest (test-analyst, doubt, devex); Seal: all applicable roles, citing
|
|
9
|
+
same-candidate Review accounts.
|
|
8
10
|
|
|
9
11
|
| Reviewer | Fires | Trigger |
|
|
10
12
|
|---|---|---|
|
|
@@ -74,8 +76,10 @@ If the candidate diff has no matching surfaces, record
|
|
|
74
76
|
and dedupe by evidence/root cause. Account for all seven (and, in `seal.md`, `devrites-proof-runner` and `devrites-doubt-reviewer`) in
|
|
75
77
|
`review.md`/`seal.md`, each [engineering lane](#engineering-lanes) separately:
|
|
76
78
|
`Outcome:` plus admitted account, or
|
|
77
|
-
`Not-applicable: <inspected-scope reason>`.
|
|
78
|
-
`review.md`
|
|
79
|
+
`Not-applicable: <inspected-scope reason>`. Seal cites, rather than re-dispatches, any
|
|
80
|
+
role `review.md` admitted on the identical candidate under
|
|
81
|
+
[evidence validity](candidate-integrity.md#evidence-validity); a changed digest or newly
|
|
82
|
+
triggered surface needs a fresh account. Persist no dispatch telemetry.
|
|
79
83
|
7. Missing profile, null/failure, or malformed output is a `gap`; never
|
|
80
84
|
infer approval or substitute inline review.
|
|
81
85
|
|
|
@@ -127,7 +131,8 @@ child's handle, path/resource ownership and capacity reservation until the
|
|
|
127
131
|
native host confirms a terminal state and returned work has been reconciled.
|
|
128
132
|
Query or wait on that handle; do not replace the child, release its slot, or
|
|
129
133
|
reuse its resources while it may still run. If terminal state cannot be
|
|
130
|
-
established, record a gap and retain ownership
|
|
134
|
+
established, record a gap and retain ownership; for a child launched by an earlier
|
|
135
|
+
session, the claim proof below settles it. A result classification alone
|
|
131
136
|
does not prove host termination.
|
|
132
137
|
|
|
133
138
|
On resume (compaction or a new session), run `devrites-engine dispatch <slug>
|
|
@@ -135,7 +140,11 @@ status`, `parallel status`, and `claim list --all` before any dispatch. A read-o
|
|
|
135
140
|
wave whose handles this session cannot query: `abandon` it, record each unreturned
|
|
136
141
|
role `gap`, and re-dispatch fresh on the current digest; its late returns are
|
|
137
142
|
`stale`. A writer keeps its claim and lease — no relaunch on those paths until the
|
|
138
|
-
claim is released or its holder is proven terminal.
|
|
143
|
+
claim is released or its holder is proven terminal. For an unqueryable writer,
|
|
144
|
+
`claim list --all` showing its claim `released` or `dead` (the engine proved the
|
|
145
|
+
holding session exited on this host) is that proof: `abandon` the wave, record the
|
|
146
|
+
role `gap`, claim the paths, and re-dispatch without waiting on the launching
|
|
147
|
+
session. `live` or `expired` stays held. A wright that never returned an
|
|
139
148
|
admitted result leaves unadmitted bytes: task-path changes since its pre-dispatch
|
|
140
149
|
`git diff --name-only` baseline are never cited as built nor discarded, and are
|
|
141
150
|
named in the re-dispatch contract; changes outside task paths stop for the human
|
|
@@ -223,8 +223,9 @@ project/decision docs, and authoritative dependency sources; make and record rev
|
|
|
223
223
|
implementation/test choices. Ask only about product, scope, acceptance, architecture policy,
|
|
224
224
|
irreversible risk, or human-only access/action.
|
|
225
225
|
|
|
226
|
-
Objective test/build/tool failure runs bounded `devrites-debug-recovery`; fix it or
|
|
227
|
-
|
|
226
|
+
Objective test/build/tool failure runs bounded `devrites-debug-recovery`; fix it, or escalate and
|
|
227
|
+
park it under the [retry contract](#retry-cap-no-progress-loops-and-self-resolve). Never ask
|
|
228
|
+
permission for another attempt, test, parser repair, or probe.
|
|
228
229
|
Close decisions at the earliest informed phase: product in spec, coverage in clarify,
|
|
229
230
|
scope/risk in temper, architecture/dependencies in define, and proof/toolchain in vet. Build
|
|
230
231
|
keeps only unavailable-pre-code or mandatory action-time checkpoints.
|
|
@@ -375,6 +376,14 @@ access, safety and irreversible-action boundaries independently stop work.
|
|
|
375
376
|
closes every deferred row with a labeled verdict; Seal blocks on any without one.
|
|
376
377
|
**Failing case:** round 4 finds three Important items on clauses untouched since
|
|
377
378
|
round 1 and dispatches another repair.
|
|
379
|
+
- **Recovery findings need a discriminator; recovery scope stays frozen.** While a
|
|
380
|
+
fingerprint is in recovery, a new Important finding keeps it open or opens another
|
|
381
|
+
only with an executed reproduction or a quoted AC/INV/PROH/accepted-DEC line whose
|
|
382
|
+
observable outcome fails; root turns it into a control the next wright must pass.
|
|
383
|
+
A static counterexample against plan-invented hardening goes to the Review trail or
|
|
384
|
+
`## Deferred findings` (Review gives it a verdict; Seal blocks without one). A
|
|
385
|
+
Critical with a concrete failure path always opens a fingerprint. A correction adds
|
|
386
|
+
no proof site or mechanism its fingerprint's closure does not need. No reviewer is skipped.
|
|
378
387
|
- **Separate consumptive authority from recovery.** Spent one-shot authorization
|
|
379
388
|
blocks another execution, not offline recovery. Retained evidence starts caller-owned
|
|
380
389
|
diagnosis/correction immediately; the next consumptive execution needs fresh authority.
|
|
@@ -388,13 +397,19 @@ access, safety and irreversible-action boundaries independently stop work.
|
|
|
388
397
|
the fingerprint's current *strategy*, not the work. Record that strategy and its
|
|
389
398
|
attempts under `## Dead ends`, then the controlling caller escalates inline in the
|
|
390
399
|
same invocation: re-diagnose from the reproduction and dead ends, take the first
|
|
391
|
-
class below that still has an untried strategy,
|
|
392
|
-
|
|
393
|
-
(
|
|
394
|
-
|
|
395
|
-
|
|
396
|
-
|
|
397
|
-
|
|
400
|
+
class below that still has an untried strategy, and resume with a fresh
|
|
401
|
+
three-attempt budget. Re-plan only when the strategy must change a planning
|
|
402
|
+
input (writer allowlist, interface or contract, slice topology, or a
|
|
403
|
+
`test-plan.md` proof row): run `/rite-plan repair` plus narrow Vet for it. A
|
|
404
|
+
strategy that fits the vetted plan, with `devrites-engine check readiness <slug>`
|
|
405
|
+
still passing, goes straight to the wright with the dead ends in its contract;
|
|
406
|
+
Build's review and proof still gate it. Classes, in order, cheapest evidence
|
|
407
|
+
before costliest change: (1) **shared root cause**: fix the owner every failing
|
|
408
|
+
path routes through and enforce it across all callers instead of another local
|
|
409
|
+
guard; (2) **diagnostic amplification**: add discriminating probes or fixtures,
|
|
410
|
+
then correct from that evidence (last for a consumptive action, whose attempt
|
|
411
|
+
needs fresh authorization); (3) **boundary redesign**: acceptance-preserving
|
|
412
|
+
reslice or redesign of the failing seam, built on that evidence. A recorded strategy, or a rewording of one, never
|
|
398
413
|
repeats. Escalation never weakens tests, validators or acceptance. Each
|
|
399
414
|
fingerprint gets at most three escalations per activation; record each in
|
|
400
415
|
`drift.md` with its class and strategy.
|
|
@@ -62,7 +62,9 @@ a human plus an agent) can share one working tree. Before a same-tree writer
|
|
|
62
62
|
starts, it records an advisory claim — `devrites-engine claim add --session
|
|
63
63
|
<id> <paths>` — retains the returned `<claim-id>`, then checks the same paths
|
|
64
64
|
with `devrites-engine claim check --session <id> <paths>`; a held claim from a
|
|
65
|
-
live foreign session is
|
|
65
|
+
live foreign session is never merged later or overridden. Unattended, skip only
|
|
66
|
+
work that needs those paths, run independent work, and re-check `claim list --all`
|
|
67
|
+
before stopping; stop and report the holder only when nothing independent remains. On every
|
|
66
68
|
host-confirmed terminal path after `add` ([unknown outcomes keep it](../parallel-dispatch.md#cancellation-and-terminal-reconciliation)) — return,
|
|
67
69
|
rejected result, stop, or a launch refused with no handle — run
|
|
68
70
|
`devrites-engine claim release --session <id> --id <claim-id>`. Claims expire
|
|
@@ -73,7 +75,9 @@ Claims cover product paths only; root-authored `.devrites/**` files use
|
|
|
73
75
|
Expiry is not proof the holder stopped: size `claim add --ttl <minutes>` to the
|
|
74
76
|
expected write (engine max 240) and re-add before it lapses; before writing, also
|
|
75
77
|
read `claim list --all` — a foreign `expired` claim on overlapping paths whose
|
|
76
|
-
session is not proven terminal counts as held (stop and report).
|
|
78
|
+
session is not proven terminal counts as held (stop and report). `dead` (engine-proven
|
|
79
|
+
exited holder) or `released` is terminal: reconcile per
|
|
80
|
+
[§ Cancellation](../parallel-dispatch.md#cancellation-and-terminal-reconciliation). **Failing case:**
|
|
77
81
|
session B's `claim check` passes at minute 35 over session A's still-running wright.
|
|
78
82
|
|
|
79
83
|
## Independence
|
|
@@ -163,6 +163,9 @@ A skill that stops without doing this leaves the workspace incomplete.
|
|
|
163
163
|
|
|
164
164
|
Long contexts degrade reasoning quality. Act on context at **50-70% used, not 95%**.
|
|
165
165
|
|
|
166
|
+
Read the engine `context` bundle once per phase instead of re-reading the files it
|
|
167
|
+
contains; after compaction re-read the bundle, not its sources.
|
|
168
|
+
|
|
166
169
|
Every rite names the single resume command on its action line; a `↻ Hygiene:` footer
|
|
167
170
|
advises `/clear`, `/compact`, or `/rite-handoff` and never a competing command. Full phase-by-phase guidance:
|
|
168
171
|
[`context-hygiene.md`](context-hygiene.md).
|
|
@@ -124,6 +124,9 @@ an error because the runner can never execute it.
|
|
|
124
124
|
(an invariant the output must always satisfy) over example (one concrete
|
|
125
125
|
input/output case). An example-shaped oracle on a property-shaped outcome is
|
|
126
126
|
a weak gate — reviewers flag it even when it passes.
|
|
127
|
+
- One CHECK, one oracle: never chain another artifact's self-test before the
|
|
128
|
+
AC's oracle with `&&` — give the self-test its own gate, so a red self-test
|
|
129
|
+
never hides the AC's own result.
|
|
127
130
|
- When an outcome becomes impossible, `gates abandon` it and route the
|
|
128
131
|
handoff — never delete the row to make the ledger pass.
|
|
129
132
|
- The gate set is the coverage denominator: it never shrinks to reach
|
|
@@ -250,6 +250,13 @@ Dismiss an irrelevant case with a reason; silently omitting an applicable case i
|
|
|
250
250
|
- A flaky test is a broken test. Isolate and fix it immediately; don't paper over it with
|
|
251
251
|
retries or `sleep`. **Failing case:** a known-flaky test is left with retries so the
|
|
252
252
|
suite is paper-green at Prove/Seal → NO-GO.
|
|
253
|
+
- **A diagnostic rerun is allowed, never proof.** Rerun a timing-shaped failure on
|
|
254
|
+
the unchanged candidate to classify it flaky or deterministic before a repair;
|
|
255
|
+
record it as diagnostic, never as a pass.
|
|
256
|
+
- **Attributable RED.** A negative control or mutant counts RED only on a positive
|
|
257
|
+
marker from the targeted assertion; a setup, construction, or observation failure
|
|
258
|
+
is never credited as RED. Prove the classifier with an injected setup failure
|
|
259
|
+
that must stay uncredited.
|
|
253
260
|
- **A retry-only pass is `flaky`, never green.** A pass earned by rerun on an unchanged
|
|
254
261
|
candidate digest (no admitted correction between attempts), or by a runner retry
|
|
255
262
|
setting, is `flaky`: record every attempt with the first attempt's outcome. It never
|
|
@@ -326,12 +326,16 @@ Epoch starts 1. `FAILED` records preimages/cleanup/epoch/fingerprint/reason/
|
|
|
326
326
|
boundary/route/no-progress count. Resolved fingerprints close; a different
|
|
327
327
|
invariant gets a distinct fingerprint. Handoff death resumes its retry epoch with
|
|
328
328
|
prior rows immutable. `WA-OP-011/012` enforce independent fingerprint/epoch caps
|
|
329
|
-
and terminal source cleanup. Exhaustion
|
|
329
|
+
and terminal source cleanup. Exhaustion closes this admission, not the work:
|
|
330
|
+
the controlling caller escalates it under the
|
|
331
|
+
[retry contract](afk-hitl.md#retry-cap-no-progress-loops-and-self-resolve)
|
|
332
|
+
through `PLAN_VET_REPAIR`, and a re-vetted changed design re-admits under a new
|
|
333
|
+
identity with fresh epochs. A fully escalated fingerprint parks. Exhaustion records:
|
|
330
334
|
|
|
331
335
|
```text
|
|
332
|
-
status=
|
|
336
|
+
status=running
|
|
333
337
|
exhaustion_cause=<same-fingerprint-count|total-epoch-limit>
|
|
334
|
-
next_action
|
|
338
|
+
next_action=/rite-plan repair <slug> — escalate exhausted workflow attempt
|
|
335
339
|
```
|
|
336
340
|
|
|
337
341
|
The journal, evidence section, and observer retain that cause; epoch exhaustion
|
|
@@ -343,7 +347,7 @@ Precedence: busy owner; safety/access/approval gate; completed historical; activ
|
|
|
343
347
|
journal; `CLEANED`; missing/stale authority; stale writer evidence; current
|
|
344
348
|
admission. `PROVED|CLEANING` resumes cleanup; install-through-`PROVING`, rollback,
|
|
345
349
|
or failure cleanup routes offline; retryable `FAILED` routes correction/retry;
|
|
346
|
-
`EXHAUSTED`
|
|
350
|
+
`EXHAUSTED` escalates; `PREPARING|PREPARED` resumes only with current authority.
|
|
347
351
|
There is no actor-history migration or backfill.
|
|
348
352
|
|
|
349
353
|
|Route|Owner|Exact action|Durable state/status/next action|Cursor/output|
|
|
@@ -356,7 +360,7 @@ There is no actor-history migration or backfill.
|
|
|
356
360
|
| `VERIFY_EXISTING`|controlling root|verify `CLEANED`, targets, proof, and product separation; do not reinstall|restore saved return phase/action|no retry/slice/action budget|
|
|
357
361
|
| `NO_BACKFILL`|controlling root|no action|completed state unchanged|no new output|
|
|
358
362
|
| `WAIT_ACTIVE_OWNER`|non-owner|no write; stop invocation|shared state unchanged|fixed owner-busy diagnostic|
|
|
359
|
-
| `BLOCKED_EXHAUSTED`|controlling root|no attempt 4
|
|
363
|
+
| `BLOCKED_EXHAUSTED`|controlling root|no attempt 4 on this admission; escalate through `PLAN_VET_REPAIR` or park per retry contract|`status=running`, exact exhausted `next_action`; parked keeps the parked-recovery action|fixed exhausted diagnostic; no intermediate reply|
|
|
360
364
|
| `BLOCKED_GATE`|controlling root|follow existing safety/access/approval gate|gate-owned state|gate output; never reinterpret as retry|
|
|
361
365
|
|
|
362
366
|
|Scenario ID|Trigger|Exact route / action|Durable consequence|Forbidden behavior|
|
|
@@ -422,7 +426,7 @@ pairs remain injective for actionable seams.
|
|
|
422
426
|
| `devrites-debug-recovery/SKILL.md`|durable active failure or ambiguous admitted state|`OFFLINE_RECOVERY`; correct offline, re-preflight, narrow Vet, retry only under cap|saved caller or exact Plan/Vet route|
|
|
423
427
|
| `rite-autocomplete/SKILL.md`|lifecycle cursor encounters admitted set or resumable journal|invoke classifier; execute returned route internally|saved phase/action; zero intermediate reply|
|
|
424
428
|
| `rite-autocomplete/reference/loop.md`|loop tick sees Workflow Artifact trigger/state|invoke classifier once under owner lock; no actor-history migration|same loop cursor; no budget charge for verify/rerun|
|
|
425
|
-
| `rite-autocomplete/reference/stop-conditions.md`|classifier returns owner-busy, exhausted, or existing hard gate|
|
|
429
|
+
| `rite-autocomplete/reference/stop-conditions.md`|classifier returns owner-busy, exhausted, or existing hard gate|escalate `BLOCKED_EXHAUSTED`; on `WAIT_ACTIVE_OWNER` run independent work and recheck; stop on exact `BLOCKED_GATE` result|unchanged cursor plus fixed route-owned output|
|
|
426
430
|
| `rite-build/SKILL.md`|Vet-ready admitted bytes require root authorship outside product wright|`ROOT_TRANSACTION`; root writes only admitted `.devrites/**` targets|saved Build slice cursor; wright product allowlist unchanged|
|
|
427
431
|
| `rite-build/reference/phase-contract.md`|Build gate enters or resumes transaction|invoke canonical operation table; reconcile exact result|same slice/checkpoint cursor or Plan/Vet route|
|
|
428
432
|
| `rite-prove/SKILL.md`|Prove consumes installed Workflow Artifact or `CLEANED` rerun|`VERIFY_EXISTING` or admitted proof path ending `PROVE_AND_RETURN`|saved Prove cursor; stop before real action|
|
|
@@ -237,7 +237,7 @@ slices; use `none` when they do not apply. Complexity above 3 triggers reslicing
|
|
|
237
237
|
unless the stated reason makes the boundary irreducible. A Writer allowlist of 12+
|
|
238
238
|
files prints an engine `ADVISORY size:` line (`check task-graph`, `check slice`;
|
|
239
239
|
never blocking): split it acceptance-preserving, or add `Size rationale:` naming
|
|
240
|
-
the inseparable capability
|
|
240
|
+
the inseparable capability; `ADVISORY hub:` takes a seam slice or `Serial rationale:`.
|
|
241
241
|
|
|
242
242
|
## Read next by phase
|
|
243
243
|
|
|
@@ -108,8 +108,8 @@ executable controller/harness/bundle bytes or a missing writer, read
|
|
|
108
108
|
`/rite-review` → `/rite-seal`. Read and execute each skill; durable files, not
|
|
109
109
|
chat, carry state. Apply the mutable post-vet budget before first Build.
|
|
110
110
|
Build runs the largest eligible path-disjoint batch (cap: this invocation's
|
|
111
|
-
`--parallel N`, else sentinel `max_parallel`, else 10) and
|
|
112
|
-
|
|
111
|
+
`--parallel N`, else sentinel `max_parallel`, else 10) and refills a slot
|
|
112
|
+
per landing until no pending slice remains; a one-slice round is serial for that round only, then
|
|
113
113
|
selection runs again (host isolation still forces serial when concurrent writers
|
|
114
114
|
cannot be separated). When `--parallel N`
|
|
115
115
|
is present, pass it into `/rite-build` and do not consult leftover sentinel
|
|
@@ -67,7 +67,7 @@ Workspace files carry state; chat does not. Read and execute each phase skill:
|
|
|
67
67
|
| 3 | `/rite-temper` | harden or reduce; `expand` and extra acceptance stop for a human; irreversible risk still pauses |
|
|
68
68
|
| 4 | `/rite-define` | approved plan/tasks/traceability |
|
|
69
69
|
| 5 | `/rite-vet` | every plan; derive mutable budget after READY |
|
|
70
|
-
| 6 | `/rite-build` batch loop | largest eligible path-disjoint set (cap: this invocation's `--parallel N`, else sentinel `max_parallel`, else 10; one-slice round when <2 eligible);
|
|
70
|
+
| 6 | `/rite-build` batch loop | largest eligible path-disjoint set (cap: this invocation's `--parallel N`, else sentinel `max_parallel`, else 10; one-slice round when <2 eligible); refill per landing; charge once per landed slice |
|
|
71
71
|
| 7 | `/rite-prove` | all slices built; approved proof; recovery on red |
|
|
72
72
|
| 8 | `/rite-polish` | re-prove after code edits |
|
|
73
73
|
| 9 | `/rite-review` | in-scope correction then fresh proof |
|
|
@@ -38,7 +38,7 @@ Agent-owned backtracking is not a stop condition while the exact causal
|
|
|
38
38
|
fingerprint has budget. Invoke the earlier phase, affected Vet, remediation, and
|
|
39
39
|
proof; then resume the origin. Three no-progress corrections of that exact
|
|
40
40
|
fingerprint spend only its current strategy: escalate to the next correction
|
|
41
|
-
class inline (shared root cause,
|
|
41
|
+
class inline (shared root cause, diagnostic amplification, boundary redesign)
|
|
42
42
|
with a fresh budget, never asking permission. After three escalations in this
|
|
43
43
|
activation the fingerprint parks; finish all independent work, then stop once.
|
|
44
44
|
Preserve reproduction, attempts, and dead ends; never offer `/rite-plan unblock`
|
|
@@ -58,7 +58,7 @@ amplification Plan repair and narrow Vet inline, then pause for fresh GO before
|
|
|
58
58
|
one evidence-acquisition attempt. Without a safe seam, escalate to the next
|
|
59
59
|
class or park; only a real human/risk/scope gate stops for a human.
|
|
60
60
|
|
|
61
|
-
<!-- workflow-artifact-adapter: {"module":"devrites-lib/reference/standards/workflow-artifacts.md","entry":"classifier returns owner-busy, exhausted, or existing hard gate","action":"
|
|
61
|
+
<!-- workflow-artifact-adapter: {"module":"devrites-lib/reference/standards/workflow-artifacts.md","entry":"classifier returns owner-busy, exhausted, or existing hard gate","action":"escalate BLOCKED_EXHAUSTED; on WAIT_ACTIVE_OWNER run independent work and recheck; stop on exact BLOCKED_GATE result","return":"unchanged cursor plus fixed route-owned output"} -->
|
|
62
62
|
## Other stop classes
|
|
63
63
|
|
|
64
64
|
- **Gate severity:** escalating always stops. Blocking with a ranked recommended
|
|
@@ -85,6 +85,10 @@ class or park; only a real human/risk/scope gate stops for a human.
|
|
|
85
85
|
pre-existing remaining value, explicit flag, sentinel cap, or post-vet pending count
|
|
86
86
|
does not stop. Zero with no pending slices is normal completion.
|
|
87
87
|
Malformed flag stops before any write.
|
|
88
|
+
- **Foreign writer:** a `live`/`expired` claim or `WAIT_ACTIVE_OWNER` held by
|
|
89
|
+
another session blocks only work on those paths; run independent work, re-check,
|
|
90
|
+
and stop only when nothing independent remains. `released`/`dead` claims are
|
|
91
|
+
reconciled per [§ Cancellation](../../devrites-lib/reference/parallel-dispatch.md#cancellation-and-terminal-reconciliation).
|
|
88
92
|
- **Resource envelope:** overlapping run or missing AFK after arm still stops.
|
|
89
93
|
Do not stop on `--max-slices`, `max_agents`, `max_minutes`, or `max_review_queue`.
|
|
90
94
|
- **Confidence:** intent still cannot become testable acceptance after interview.
|
|
@@ -75,15 +75,15 @@ Wright applies anti-slop; root verifies returns and never patches source. Refuse
|
|
|
75
75
|
|
|
76
76
|
Omitted/`1` ≡ serial; `2`–`10` is a **cap** — the root runs the largest eligible
|
|
77
77
|
path-disjoint set it can start now (`N_eff`), takes fewer when the cap cannot be
|
|
78
|
-
filled, and
|
|
79
|
-
|
|
78
|
+
filled, and refills a slot per landing (serial slice or one sibling) until
|
|
79
|
+
no pending slice remains
|
|
80
80
|
([`parallel-batch.md` § Dynamic selection](reference/parallel-batch.md#dynamic-selection-and-re-batching)).
|
|
81
|
-
Unattended runs (`/rite-autocomplete`, `.devrites/AFK` `max_parallel`)
|
|
82
|
-
inside the same run; HITL stops after each
|
|
81
|
+
Unattended runs (`/rite-autocomplete`, `.devrites/AFK` `max_parallel`) refill
|
|
82
|
+
inside the same run; HITL stops after each landing. This invocation's `--parallel N`
|
|
83
83
|
is the cap: leftover sentinel `max_parallel` is not consulted. Autocomplete with
|
|
84
84
|
that flag also writes or replaces only `max_parallel: N`. Non-integer/`N≤0`/`N>10` hard refuse.
|
|
85
|
-
|
|
86
|
-
|
|
85
|
+
Each sibling lands alone when its review + proof are green (`integrate --slice`):
|
|
86
|
+
one local `WIP(<slug>):` commit onto the control branch (never pushed). Do not
|
|
87
87
|
integrate a sibling because the wright returned. A red/gap sibling gets a bounded repair
|
|
88
88
|
round in its own worktree — never rebuilt from scratch while budget remains.
|
|
89
89
|
Post-writer inventory and affected rechecks follow `phase-contract.md` § Independent
|
|
@@ -94,7 +94,7 @@ a human `Fix`. Exhausted repair blocks and preserves everything;
|
|
|
94
94
|
`cleanup --force` salvages slice branches before removing worktrees —
|
|
95
95
|
human-gated to abandon a batch, orchestrator-emitted only as the salvage
|
|
96
96
|
step of an automatic frozen-lease re-batch.
|
|
97
|
-
AFK charges
|
|
97
|
+
AFK charges per landed sibling only. Running lease blocks another
|
|
98
98
|
`/rite-build`. Details: `parallel-batch.md`.
|
|
99
99
|
|
|
100
100
|
## Execute and reply
|
|
@@ -44,8 +44,8 @@ The controlling root owns the cap:
|
|
|
44
44
|
before the next dispatch.
|
|
45
45
|
- **Serial:** charge when fail-on-red is green and the built record is written
|
|
46
46
|
(same rewrite as step 2).
|
|
47
|
-
- **Parallel `--parallel`:** charge only after **successful
|
|
48
|
-
— once per
|
|
47
|
+
- **Parallel `--parallel`:** charge only after **successful integrate**
|
|
48
|
+
— once per landed green sibling. Abort / integrate-failed → charge **0**.
|
|
49
49
|
Do not charge on worktree-green before integrate. See
|
|
50
50
|
[`parallel-batch.md`](parallel-batch.md).
|
|
51
51
|
|
|
@@ -18,7 +18,7 @@ the orchestrator pauses for approval before landing it.
|
|
|
18
18
|
and fail-on-red proof are green, with no open Critical/Important and no repair
|
|
19
19
|
wright still in flight. Serial: git recipe below (skip empty index) or isolated
|
|
20
20
|
FF of a matching `transfer_commit`. Parallel: `devrites-engine parallel
|
|
21
|
-
integrate --apply-to-control` (no second control `git commit`).
|
|
21
|
+
integrate --slice <id> --apply-to-control` (no second control `git commit`).
|
|
22
22
|
- **Do not:** when a wright returns; before review/proof; with red/gap findings;
|
|
23
23
|
fast-forward an isolated transfer onto control before those checks.
|
|
24
24
|
|
|
@@ -22,13 +22,14 @@ DISPATCH → hand the slice contract to devrites-slice-wright (fresh context).
|
|
|
22
22
|
— dispatch the initial read-only reviews of distinct decisions in parallel
|
|
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
|
-
required build/browser/E2E
|
|
25
|
+
required build/browser/E2E (narrow self-check/smoke first, full gate only
|
|
26
|
+
after it is green, long runs in the background), then returned-diff/path recheck; fail-on-red
|
|
26
27
|
(red/gap → repair wright, then re-review and re-prove; do not commit yet)
|
|
27
28
|
RECORD → orchestrator: after green verification, update state.md/evidence.md and
|
|
28
29
|
upsert the authoritative candidate manifest from the actual scoped diff
|
|
29
30
|
(CHECKPOINT) → only now, with review+proof green: local WIP(<slug>): <imperative summary>
|
|
30
31
|
on the control branch (no slice id) → checkpoint.md
|
|
31
|
-
NEXT → HITL root reports and stops. AFK with parallel cap>1: return to parallel-batch selection
|
|
32
|
+
NEXT → HITL root reports and stops. AFK with parallel cap>1: return to parallel-batch selection to refill; otherwise repeat under afk-discipline.md
|
|
32
33
|
```
|
|
33
34
|
|
|
34
35
|
## Why the boundary matters
|