docs(decisions): 83, row 54 may add a rows() read to packages/queue
Dewey's row 54 stop: store.show costs ~450 ms a row. Row 54 adds export function rows(opts) plus a test; queue suites join its gate. Path redaction and Dewey's recorded view choices accepted. Co-Authored-By: Claude Opus 5.5 <[email protected]>
This commit is contained in:
@@ -1734,3 +1734,35 @@ which stay with him. Each item names who decided it and what happened.
|
|||||||
Sage's one-off `agents/sage/work/vikunja-setup/` scripts, which
|
Sage's one-off `agents/sage/work/vikunja-setup/` scripts, which
|
||||||
don't run during the window. Seats run no live tracker probes from
|
don't run during the window. Seats run no live tracker probes from
|
||||||
2026-10-11T15:00Z until the hold ends.
|
2026-10-11T15:00Z until the hold ends.
|
||||||
|
83. **Row 54 may add one read to `packages/queue` (2026-10-10).** The
|
||||||
|
row 54 brief says the views change nothing in `packages/queue`,
|
||||||
|
`packages/business` or `packages/cli`, and to stop and ask if a
|
||||||
|
module lacks a read. Dewey stopped. `store.list` returns only id,
|
||||||
|
state, owner and piece, and `store.show` replays the log and spawns
|
||||||
|
git on every call: about 450 ms a row, 19.8 s for all 61 rows.
|
||||||
|
- Row 54 adds `export function rows(opts)` to
|
||||||
|
`packages/queue/src/store.mjs`. It returns
|
||||||
|
`rowsArray(readState(opts).state)` with the notes, writes nothing
|
||||||
|
and takes the same lock-free read path as `list`. A test goes in
|
||||||
|
`packages/queue/tests`, and `packages/queue/README.md` names the
|
||||||
|
export. The `packages/queue` node suite and `scripts/test-queue.sh`
|
||||||
|
join row 54's gate. Nothing else in `packages/queue` changes, and
|
||||||
|
`packages/business` and `packages/cli` stay untouched.
|
||||||
|
- Why here and not a separate row: three lines and a test, no other
|
||||||
|
open row changes `packages/queue/src` (row 13's candidate
|
||||||
|
fd72d268 touches only `packages/ledger`), and the package is not
|
||||||
|
under the Q14 freeze. Dropping reviewers and the brief link from
|
||||||
|
the list view (Dewey's option b) is the fallback if review finds
|
||||||
|
a problem with the export.
|
||||||
|
- The brief file is not edited, since rows 53 to 61 pin its blob.
|
||||||
|
This decision is the amendment, and row 54's packet cites it.
|
||||||
|
- Refusal text that reaches the browser has absolute paths replaced
|
||||||
|
by `<config>/`, `<dataRoot>/` and `<path>`, and a test proves the
|
||||||
|
test's tmp directory never appears in a response. The brief's
|
||||||
|
no-path rule stands.
|
||||||
|
- Dewey's recorded choices stand: the Business view shows project
|
||||||
|
ids without roots; no bot ids or `human.discordUserId`;
|
||||||
|
credentials show service, account, role instance and expiry,
|
||||||
|
never a file path or env name; the business is `--business`, else
|
||||||
|
the live bus host's business; `rolesDir` is the checkout's
|
||||||
|
`roles/`.
|
||||||
|
|||||||
Reference in New Issue
Block a user