diff --git a/BUILD-LOG.md b/BUILD-LOG.md index cefcc2e5..36abf0b6 100644 --- a/BUILD-LOG.md +++ b/BUILD-LOG.md @@ -3387,3 +3387,35 @@ commit. That commit's queue-commit ran between Filbert's record and Filbert's own queue-commit, and its message doesn't mention rev 17. Filbert's queue-commit then refused with a misleading message, now on DEFERRED. + +## 2026-09-27: Queue Piece E, the ledger's queue section (#1508, Darkwing) + +fd72d268 adds a queue section to the weekly ledger report. It checks four +things: +- open issues named by done rows; +- owner registrations for active rows; +- closed issues for done rows; +- the age of required rows. + +The result is `fail`, `incomplete` or `reduced pass`. There is never a +full pass, because registrations carry no process identity. The Gitea +budget is the open list plus at most 10 lookups, separate from the metric +call. + +Lead decision 40 relaxed plan 8.10 before the commit. The repository has +50 or more open issues, so every run was `incomplete`. A full open page +now counts only while an issue in some row's `closes` has no known state. +The same decision kept row 7 until Jason closes it, amending Q9, and kept +the 14-day age rule. + +Filbert's round 1 (81f26f2e) found C1: a full ISO `requiredSince`, the +form `set required` and `add --required` write, never aged. Round 2 +(ce8ce150) approved build.patch ab1f12ca. Mutants: 11 of 13 caught. One +survivor is equivalent, the other only changes a refusal's wording. + +Sage applied the patch at 2333d837 and the manifest (0b20bbca) checked +5/5. Index export: ledger 78/78, config 24, task 90, foundation 44, +conductor 17, release 14, auth 15, discord 64, extension-package 18, +queue 27. Canonical queue suite 29/29. Row 7's brief then drifted as +expected, and 0a6f58dd re-pins it to the README's weekly routine +(rev 20). Row 13 goes to in-review with fd72d268 as its candidate. diff --git a/docs/SESSIONS.md b/docs/SESSIONS.md index 324052d2..8e1f8409 100644 --- a/docs/SESSIONS.md +++ b/docs/SESSIONS.md @@ -442,3 +442,6 @@ are never rewritten or removed; corrections are new entries. 2026-09-27T15:13:21Z | Filbert (T3 Claude Code, thread 9cb9731e) | row 12 live review round (#1508) | approve posted as comment 26579 with filbert's own token (GET user filbert), recorded rev 17 (filbert-12-record-1); f539466f passes verify-commit; rev 17 committed inside f3f48cfd by a concurrent queue commit; my queue-commit.sh refused at the step7 canary because HEAD moved (misleading message, follow-up); nothing under docs/plans/reviews/ 2026-09-27T15:14:35Z | Sage (T3 Claude Code, thread 1ef1e4f8) | row 12 closed (#1508) | pushed 3c9f4244 after verify --current; Filbert approved via comment 26579 (rev 17); row 12 done rev 19 ff61532a; DEFERRED queue-commit HEAD-moved message 2026-09-27T16:04:28Z | Sage (T3 Claude Code, thread 1ef1e4f8) | Piece E questions (#1508, row 13) | lead decision 40: full open page relaxed when every wanted issue is known; row 7 kept until Jason closes it (Q9 amended); age rule kept; T3 seats exempt per run +2026-09-27T16:15:30Z | Filbert (T3 Claude Code, thread 9cb9731e) | queue Piece E review round 1 (#1508, row 13) | changes requested (C1: age check skips ISO requiredSince, NaN) on amended patch 02a01c29 at 2333d837; review 81f26f2e; decision-40 full-page change correct (F1-F5 killed); plan amended for decision 40 (68a25ffe, uncommitted); verdict sent to Darkwing, copy to Sage +2026-09-27T16:30:11Z | Filbert (T3 Claude Code, thread 9cb9731e) | queue Piece E review round 2 (#1508, row 13) | approved patch ab1f12ca at 2333d837 (manifest 5/5, ledger 78/78, queue+seat 161/161); C1 and n1-n3 fixed; 13 mutants, 11 killed, R9 equivalent, R13 message-only; review ce8ce150; verdict to Darkwing, copy to Sage; nothing committed +2026-09-27T16:35:21Z | Sage (T3 Claude Code, thread 1ef1e4f8) | queue Piece E commit (#1508, row 13) | patch ab1f12ca, manifest 5/5, Filbert r2 approved ce8ce150; suites green on export; fd72d268; row 7 re-pinned rev 20 0a6f58dd; DEFERRED three E follow-ups diff --git a/docs/plans/DEFERRED.md b/docs/plans/DEFERRED.md index 8e7a9af0..17f67475 100644 --- a/docs/plans/DEFERRED.md +++ b/docs/plans/DEFERRED.md @@ -161,6 +161,21 @@ at every gate. Started 2026-09-12 during the control board MVP. start again". Filbert found this during the row 12 live round. (2026-09-27, #1508) +- **Queue validator checks date shape, not the calendar.** `ISO_RE` and + `DATE_RE` accept month 13, and V8 turns 2026-02-30 into March 2 without + an error. The CLI can't write such a value today; a hand edit or a + future verb could. Fix: round-trip the parsed date. Filbert and + Darkwing, Piece E round 2. (2026-09-27, #1508) +- **Ledger metric call can orphan curl.** The metric call in + `packages/ledger/src/ledger.mjs` still relies on execFileSync's + timeout, which kills the helper but can leave curl running. E's queue + calls already use `timeout -s KILL`; the metric call should match. + (2026-09-27, #1508) +- **Ledger: a missing `timeout` command reads as a Gitea failure.** With + no `timeout` on PATH the queue issue calls report "credential or Gitea + request failure". Optional test and clearer message. Filbert, E r2. + (2026-09-27, #1508) + ## Queue Moved to `docs/plans/QUEUE.md` on 2026-09-13. This file holds only gaps.