devrites 7.0.8 → 7.0.10

package/CHANGELOG.md

@@ -2,6 +2,18 @@
 
 All notable changes to DevRites are documented here. The format follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/) and DevRites adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html). Releases are generated automatically by [semantic-release](https://semantic-release.gitbook.io/) from Conventional Commits on `main`.
 
+## [7.0.10](https://github.com/ViktorsBaikers/DevRites/compare/v7.0.9...v7.0.10) (2026-10-23)
+
+### Fixed
+
+* **devrites:** apply-bundle refuses an unrunnable preflight row ([#166](https://github.com/ViktorsBaikers/DevRites/issues/166)) ([6bc06a4](https://github.com/ViktorsBaikers/DevRites/commit/6bc06a40530e3456b80883303e345fb25d45357a))
+
+## [7.0.9](https://github.com/ViktorsBaikers/DevRites/compare/v7.0.8...v7.0.9) (2026-10-22)
+
+### Fixed
+
+* **devrites:** keep a passed pre-dispatch row on re-dispatch ([#165](https://github.com/ViktorsBaikers/DevRites/issues/165)) ([927f5a4](https://github.com/ViktorsBaikers/DevRites/commit/927f5a482cfb70c84255a9ead467251483de7fee))
+
 ## [7.0.8](https://github.com/ViktorsBaikers/DevRites/compare/v7.0.7...v7.0.8) (2026-10-22)
 
 ### Fixed

package/README.md

@@ -31,7 +31,7 @@ history can't lose it. After you clear the conversation, switch hosts, or hand
 the feature to another engineer, the next session reads the same plan,
 decisions, and evidence before it continues.
 
-**Status:** [`v7.0.8`](https://github.com/ViktorsBaikers/DevRites/releases/tag/v7.0.8): see [`CHANGELOG.md`](CHANGELOG.md) for release notes.
+**Status:** [`v7.0.10`](https://github.com/ViktorsBaikers/DevRites/releases/tag/v7.0.10): see [`CHANGELOG.md`](CHANGELOG.md) for release notes.
 
 This is the latest published release; `main` may contain unreleased work.
 

package/docs/cli.md

@@ -128,8 +128,9 @@ self-contained updater.
 - `check readiness --emit-binding <slug>` renders the exact stable Build-input
   binding for Vet to record after semantic review.
 - `check preconditions <slug>` runs the preflight rows whose `Run at` is `vet`
-  (`--slice <SLICE>` runs that slice's `pre-dispatch` rows) and records a
-  candidate-bound transcript per row; exit 3 on failure.
+  (`--slice <SLICE>` runs that slice's `pre-dispatch` rows, keeping one that
+  already passed) and records a candidate-bound transcript per row; exit 3 on
+  failure.
 - `check seal <slug>` checks final required files, open human gates, and the
   `tasks.md` slice graph. Once those files are complete, it verifies the stable
   readiness binding; only after that aggregate check passes does it verify that

package/docs/engine/commands.md

@@ -14,7 +14,7 @@ workflow strategy.
 | `check candidate <slug>` | Validate the strict manifest and compute the content-bound project-candidate identity. Exits `3` when the manifest omits a path the feature's `WIP(<slug>):` checkpoint commits changed (net from the feature base to HEAD). Also exits `3` with `unattributed root edit: <paths>` while the feature's `tree-baseline.json` tripwire flags a product path (ADR-0038; see `dispatch`). |
 | `check readiness <slug>` | Check target-Phase files, open human gates from Clarify onward, the `tasks.md` slice graph when that artifact is required, canonical `AC-###` presence in `tasks.md`/`test-plan.md` when those artifacts are required, and the current stable Build-input binding when applicable. Leaving Vet with a `Run at` column, a slice whose `Files likely touched`/`Writer allowlist` names a dependency manifest or lockfile needs a `dependency-dry-run SLICE-###` row at `vet` with a passing transcript. Leaving Vet, every slice `state.md` does not check built (a deferred `Satisfies: none` slice excepted) must also pass the Build-entry `check slice` contract; each failure prints one `invariant: slice-check:` line naming the slice and its exact `check slice` reason, to fix in `/rite-plan` before READY. |
 | `check readiness --emit-binding <slug>` | Render the exact stable Build-input binding for Vet to record after review. |
-| `check preconditions <slug> [--slice SLICE-###] [--timeout N]` | Run the `test-plan.md` Build-entry preflight rows whose optional `Run at` cell is `vet` (with `--slice`, `pre-dispatch <SLICE>`); `action` rows and rows without the cell never run. Writes one candidate-bound transcript per row under `references/gates/preflight-*/`; a row passes on exit 0. Exit `3` on any failure or an invalid `Run at` value. |
+| `check preconditions <slug> [--slice SLICE-###] [--timeout N]` | Run the `test-plan.md` Build-entry preflight rows whose optional `Run at` cell is `vet` (with `--slice`, `pre-dispatch <SLICE>`); `action` rows and rows without the cell never run. Writes one candidate-bound transcript per row under `references/gates/preflight-*/`; a row passes on exit 0. A `pre-dispatch` row whose newest transcript passed is kept, not re-run: it is a pre-edit baseline, so a re-dispatch never runs it against the slice's own edits; changing its command or cwd retires the pass. Exit `3` on any failure or an invalid `Run at` value. |
 | `check seal <slug>` | Check files required by target Phase `seal`, open human gates, the `tasks.md` slice graph, canonical `AC-###` presence when `tasks.md`/`test-plan.md` are required, the stable Build-input binding, and exact candidate bindings. `check readiness` and `check seal` also block a `scripts/` test driver that a tasks.md Writer allowlist plans or touched-files.md lists when no gates.md `CHECK` runs it and no `Driver waiver: <path> — DEC-### <reason>` line cites a recorded decision; waived drivers print as `note: unexecuted driver (waived by DEC-###): <path>` lines, on pass or block. |
 | `check path-disjoint [--root <dir>] [<json-file> | -]` | Verify slice path sets are pairwise disjoint. |
 | `parallel select --cap <1-10> [--root <dir>] [--slug <slug>] [--from-plan \| <json-file> \| -]` | Choose the greedy path-disjoint subset ≤ cap from supplied ready slices, or with `--from-plan` (needs `--slug`) from the plan: tasks.md slices whose dependencies state.md checks `built`, with Writer allowlist (else Files likely touched) paths plus the nearest manifest/lockfile when `External dependencies:` is not none. With `--slug` and a running lease, the lease's not-yet-landed slices hold cap slots and paths, slices it names are never picked, and only new picks print. |

package/engine/internal/acceptance/approve.go

@@ -1,6 +1,7 @@
 package acceptance
 
 import (
+	"fmt"
 	"path/filepath"
 	"strings"
 
@@ -25,6 +26,9 @@ type PreflightRow struct {
 	// RunAt is the optional `Run at` cell (`vet`, `pre-dispatch SLICE-###`,
 	// `action`), whitespace-collapsed; "" when the row or table has none.
 	RunAt string
+	// Width and HeaderWidth are the row's and its table header's cell counts;
+	// they differ when a bare `|` in a command splits the row.
+	Width, HeaderWidth int
 }
 
 // ApprovedCommands extracts the approved command surface of test-plan.md: the
@@ -56,7 +60,7 @@ func PreflightRows(testPlan []byte) (rows []PreflightRow, hasRunAt bool) {
 		return nil, false
 	}
 	inSection := false
-	commandCol, cwdCol, gateCol, expectedCol, runAtCol := -1, -1, -1, -1, -1
+	commandCol, cwdCol, gateCol, expectedCol, runAtCol, headerWidth := -1, -1, -1, -1, -1, 0
 	for _, line := range strings.Split(string(structural), "\n") {
 		trimmed := strings.TrimSpace(line)
 		if strings.HasPrefix(trimmed, "##") {
@@ -77,6 +81,7 @@ func PreflightRows(testPlan []byte) (rows []PreflightRow, hasRunAt bool) {
 			gateCol = columnIndex(cells, "gate")
 			expectedCol = columnIndex(cells, "expected")
 			runAtCol = columnIndex(cells, "run at")
+			headerWidth = len(cells)
 			hasRunAt = hasRunAt || (commandCol >= 0 && runAtCol >= 0)
 			continue
 		}
@@ -87,7 +92,7 @@ func PreflightRows(testPlan []byte) (rows []PreflightRow, hasRunAt bool) {
 		if command == "" {
 			continue
 		}
-		row := PreflightRow{ApprovedCommand: ApprovedCommand{Command: command}}
+		row := PreflightRow{ApprovedCommand: ApprovedCommand{Command: command}, Width: len(cells), HeaderWidth: headerWidth}
 		if cwdCol >= 0 && cwdCol < len(cells) {
 			row.Cwd = normalizeApprovedCwd(cells[cwdCol])
 		}
@@ -99,6 +104,30 @@ func PreflightRows(testPlan []byte) (rows []PreflightRow, hasRunAt bool) {
 	return rows, hasRunAt
 }
 
+// PreflightShapeProblems names each Build-entry preflight row that cannot run
+// as written: a cell count that differs from its header (an unescaped `|`
+// in the command, which must be written `\|`) or an invalid `Run at` value.
+// `check preconditions` would otherwise find it only when Vet runs the row.
+func PreflightShapeProblems(testPlan []byte) []string {
+	rows, _ := PreflightRows(testPlan)
+	var problems []string
+	for _, row := range rows {
+		name := valueOr(row.Gate, row.Command)
+		if row.Width != row.HeaderWidth {
+			problems = append(problems, fmt.Sprintf("preflight row %q has %d cells but its header has %d; escape each `|` in the command as `\\|`", name, row.Width, row.HeaderWidth))
+		} else if !ValidRunAt(row.RunAt) {
+			problems = append(problems, fmt.Sprintf("preflight row %q has invalid Run at %q (want vet, pre-dispatch SLICE-###, or action)", name, row.RunAt))
+		}
+	}
+	return problems
+}
+
+// ValidRunAt reports whether a `Run at` cell is empty or a known value.
+func ValidRunAt(runAt string) bool {
+	lower := strings.ToLower(runAt)
+	return lower == "" || lower == "vet" || lower == "action" || preDispatchRunAtRE.MatchString(runAt)
+}
+
 // cellText returns a cell's trimmed text without code backticks, "" when the
 // column is absent or the row is short.
 func cellText(cells []string, col int) string {

package/engine/internal/acceptance/preconditions.go

@@ -77,7 +77,9 @@ func PreflightPassed(work string, row PreflightRow) bool {
 // RunPreconditions is `check preconditions <slug> [--slice S]`: it executes
 // the Build-entry preflight rows whose `Run at` is `vet` (or, with --slice,
 // `pre-dispatch <S>`), writes one candidate-bound transcript per row, and exits
-// ExitBlocked on any failure. Rows are the approved surface itself, so nothing
+// ExitBlocked on any failure. A pre-dispatch row that already passed keeps that
+// transcript: it is a pre-edit baseline, and a repair round's re-dispatch would
+// otherwise re-run it against the slice's own unfinished edits. Rows are the approved surface itself, so nothing
 // outside test-plan.md runs; `action` rows and rows without a `Run at` cell
 // never run here.
 func RunPreconditions(root string, args []string, stdout, stderr io.Writer) int {
@@ -157,12 +159,11 @@ func RunPreconditions(root string, args []string, stdout, stderr io.Writer) int
 // and whether any row ran.
 func runPreflightRows(root, slug, work, project, runAt string, rows []PreflightRow, timeout time.Duration, stdout io.Writer) (bool, bool, error) {
 	before := candidateLabel(root, slug)
-	failed := false
+	failed, kept := false, false
 	doc := &Document{}
 	var results []GateResult
 	for _, row := range rows {
-		lower := strings.ToLower(row.RunAt)
-		if lower != "" && lower != "vet" && lower != "action" && !preDispatchRunAtRE.MatchString(row.RunAt) {
+		if !ValidRunAt(row.RunAt) {
 			fmt.Fprintf(stdout, "row %s: invalid Run at %q (want vet, pre-dispatch SLICE-###, or action)\n", valueOr(row.Gate, row.Command), row.RunAt)
 			failed = true
 			continue
@@ -170,6 +171,11 @@ func runPreflightRows(root, slug, work, project, runAt string, rows []PreflightR
 		if !strings.EqualFold(row.RunAt, runAt) {
 			continue
 		}
+		if preDispatchRunAtRE.MatchString(runAt) && PreflightPassed(work, row) {
+			fmt.Fprintf(stdout, "row %s: pass (kept; a pre-dispatch row runs once, before the slice's first edit)\n", PreflightGateID(row))
+			kept = true
+			continue
+		}
 		gate := &Gate{ID: PreflightGateID(row), Title: row.Gate, Check: row.Command, Cwd: row.Cwd,
 			Expect: "exit 0 (Expected: " + valueOr(row.Expected, "unstated") + ")"}
 		cwd := project
@@ -197,5 +203,5 @@ func runPreflightRows(root, slug, work, project, runAt string, rows []PreflightR
 			return failed, true, fmt.Errorf("transcript: %w", err)
 		}
 	}
-	return failed, len(results) > 0, nil
+	return failed, kept || len(results) > 0, nil
 }

package/engine/internal/lib/apply_bundle.go

@@ -15,6 +15,7 @@ import (
 	"sort"
 	"strings"
 
+	"github.com/devrites/devrites/internal/acceptance"
 	"github.com/devrites/devrites/internal/devritespaths"
 	"github.com/devrites/devrites/internal/fsutil"
 	"github.com/devrites/devrites/internal/state"
@@ -360,7 +361,8 @@ func applyHunks(base string, hunks []candidateHunk) (string, []string) {
 
 // checkBundleResult rejects an outcome the root would otherwise discover only
 // at Vet: a tracked ID that vanished from every touched file without being
-// listed in retired_ids, a new tasks.md graph problem, or a budgeted artifact
+// listed in retired_ids, a new tasks.md graph problem, a new test-plan.md
+// preflight row that cannot run as written, or a budgeted artifact
 // that grew into material overshoot without a covering
 // `Budget override: <=N lines; <reason>` line.
 func checkBundleResult(targets []*bundleTarget, retired map[string]bool) []string {
@@ -400,6 +402,17 @@ func checkBundleResult(targets []*bundleTarget, retired map[string]bool) []strin
 				}
 			}
 		}
+		if name == "test-plan.md" && filepath.Dir(target.rel) == "." {
+			old := map[string]bool{}
+			for _, p := range acceptance.PreflightShapeProblems(target.before) {
+				old[p] = true
+			}
+			for _, p := range acceptance.PreflightShapeProblems(target.after) {
+				if !old[p] {
+					problems = append(problems, "test-plan.md: "+p)
+				}
+			}
+		}
 		if filepath.Dir(target.rel) == "." && currentStatePlanning[name] &&
 			len(datedHeadingRE.FindAllIndex(target.after, -1)) > len(datedHeadingRE.FindAllIndex(target.before, -1)) {
 			problems = append(problems, target.rel+": adds a dated delta section; rewrite the owning clause in place (rationale to decisions.md, closed narrative to history/)")

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "devrites",
-  "version": "7.0.8",
+  "version": "7.0.10",
   "description": "DevRites: a disciplined senior-engineer workflow pack for Claude Code, Codex, omp, pi, and Devin CLI",
   "license": "SEE LICENSE IN LICENSE",
   "homepage": "https://github.com/ViktorsBaikers/DevRites#readme",