docs(records): Piece E build note and follow-ups (row 13, #1508)
Co-Authored-By: Claude Opus 5.5 <[email protected]>
This commit is contained in:
@@ -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.
|
||||
|
||||
@@ -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
|
||||
|
||||
@@ -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.
|
||||
|
||||
Reference in New Issue
Block a user