Readiness and handoff at 01:48: round twenty-seven both accept, the merge round; 2b0abef1 (CI passed) gating before the merge
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
2 files changed,
+4
−4
.claude/HANDOFF.md+3 −3implementation/carloku-paid-launch/READINESS.md+1 −1
CI passed — run details
diff --git a/.claude/HANDOFF.md b/.claude/HANDOFF.md| index 5fbaae0bf..f3caab572 100644 |
| --- a/.claude/HANDOFF.md |
| +++ b/.claude/HANDOFF.md |
| @@ -7,7 +7,7 @@ |
| Get Carloku's paid plans live with Stripe Tax and the complete metered model; have the pricing page and signup flow reviewable in the morning. |
| **STATUS:** |
| -Written 01:42 Dublin on 2026-10-03 (resumed 21:40 on 2026-10-02 after a provider limit; nothing was lost), during the second overnight run (brief: "continue overnight until we're ready to enable payments"; ready means reviewed, tested, integrated code and a scoped enablement plan, not payments switched on). Billing is OFF everywhere. No live Stripe write, no AWS change, no charge, no merge to main. Every payment anyone can see is a labelled simulation. |
| +Written 01:48 Dublin on 2026-10-03 (resumed 21:40 on 2026-10-02 after a provider limit; nothing was lost), during the second overnight run (brief: "continue overnight until we're ready to enable payments"; ready means reviewed, tested, integrated code and a scoped enablement plan, not payments switched on). Billing is OFF everywhere. No live Stripe write, no AWS change, no charge, no merge to main. Every payment anyone can see is a labelled simulation. |
| Review servers, all answering 200: pricing page `http://100.119.180.57:38009` (static, rebuilt from site `2fc6388`), signup and billing demo `http://100.119.180.57:41881` (**still the build from `68f6ee23`**; it exited on its own at 00:10 on 2026-10-03 after its fourteen-hour limit and was restarted at 00:16 with the same binary, so it runs until about 14:15 unless restarted again, same recipe: `layout-review/STATE-demo.txt`, before the retirement protocol; not the code under review, and not Stripe proof; price book configured, USD/EUR/GBP end to end; Keymail and Stripe simulated), console billing copy `http://100.119.180.57:45033` if still up. |
| @@ -16,7 +16,7 @@ The durable records are on this branch under `implementation/carloku-paid-launch |
| Decisions received 2026-10-01 evening, applied: the annual bases (28/24/22, 6/5/4.50, 60/50/45 USD/EUR/GBP, team minimum ten) confirmed for implementation only; the currency rulings (USD default, EUR for EU member states, GBP for the UK, visible override, one stored currency for subscription and overages, bands by billing country only); the five currency strings and the regrouped pricing page copy, approved verbatim and applied. Still a draft: the book `2026-10-01-draft1` and the USD/GBP overage rates and floors. |
| Branches (all on the forge, none merged; the local gate is not CI): |
| -- `feat/carloku-multi-currency-20261001` `9b872e38` pushed at 01:34 (round twenty-six's fixes; gate queued; before it `35356ad0`, gate and CI passed): **round twenty-six was the first with both reviewers accepting** (Opus accept, Astra accept with fixes; both say review eighteen's P2 is closed on this branch); round twenty-seven, a confirming review of the fixes, running since 01:34; if it ends with both accepting and nothing left to fix, and CI is green on `9b872e38`, the branch merges into this one as a merge commit: the price book and one stored currency per purchase end to end, one form with a server-signed quote, ownership on every path, the retirement protocol's stage one, the evidence invariant made structural, the named-subscription rule, and a point of no return for an account's retirement. **Not accepted: rounds twenty-one to twenty-four are each Opus accept with fixes, Astra reject; round twenty-five's Opus report accepts with P3s and says review eighteen's P2 is closed for this binary; Astra's is pending.** The branch keeps the P2 recorded open until a round ends with both accepting. Next: read Astra's round-25 report, commit, gate alone, round 26. |
| +- `feat/carloku-multi-currency-20261001` `2b0abef1` pushed at 01:43 (CI passed; gate running alone; before it `9b872e38`, gate passed alone and CI passed): **rounds twenty-six and twenty-seven both ended with both reviewers accepting** (twenty-six: Opus accept, Astra accept with fixes; twenty-seven, confirming: both accept, nothing to fix on the branch; both say review eighteen's P2 is closed on this branch); `2b0abef1` is the one comment correction Opus allowed before the merge without another review, plus the review-27 records; **it merges into this branch as a merge commit once its gate passes**: the price book and one stored currency per purchase end to end, one form with a server-signed quote, ownership on every path, the retirement protocol's stage one, the evidence invariant made structural, the named-subscription rule, and a point of no return for an account's retirement. **Not accepted: rounds twenty-one to twenty-four are each Opus accept with fixes, Astra reject; round twenty-five's Opus report accepts with P3s and says review eighteen's P2 is closed for this binary; Astra's is pending.** The branch keeps the P2 recorded open until a round ends with both accepting. Next: read Astra's round-25 report, commit, gate alone, round 26. |
| - `feat/carloku-metered-replicator-20261001` `6249af63`: eighteenth build. Reviews ten to fourteen (Opus accept with fixes, Astra reject) each found the previous build's own addition wanting; the fourteenth build made the link entry one to one and the fifteenth quotes every entry in the inventory's rendering (the drift key), classifies a link by the spelling its entry names, and makes the Accept test tell as-written from cleaned. Review fifteen (Opus accept with fixes, Astra reject on the deep items) found a regression the fifteenth build made: a relative link target was joined and so cleaned, which is not how the kernel resolves a `..` after a link, so a link to a directory outside the tree passed in silence; the sixteenth build concatenates uncleaned, with the fixture both asked for. Review sixteen (both) found that fixed and one bound left (the concatenated spelling could outgrow PATH_MAX); the seventeenth build `bffc76b5` stat'ed the target by itself against the link's directory, and **failed the gate and CI on its own fixture** (sized for a long temp root only), which review seventeen (Opus accept with fixes, Astra reject) had worked out, together with a worse defect in the same fix: the directory was opened by path with Go's default flags, so a FIFO a tenant swaps in would block the watch and Close. The eighteenth build (`e8211fc6`, `6249af63`) opens the directory as a directory and nothing else, sizes the fixture from the temp root, compares the rendering across sections, and corrects the comments, the test name and the outstanding item Astra listed; three mutants killed. Gate (exit 0, three cross builds) and CI passed on `6249af63`. Review eighteen: Opus accept with three P3s, Astra reject on the deep items. The nineteenth build `53e3dcd8` answered them (gate exit 0, three cross builds; CI passed). Review nineteen: Opus accept, Astra reject on the deep items. The twentieth build `c9ec11eb` makes the bounded changes (a tree claim refuses what a file claim refuses; the fixture's guard reachable; the no-recorder refusal quoted) and separates them in its dispositions from the deferred items; `c9ec11eb`: gate exit 0 with three cross builds (09:52Z), CI passed. Review twenty: Opus accept with fixes, Astra reject on the deep items. `e1280811` answers it with comments, a fixture and dispositions only (no behaviour change; gate running; not sent for another round): the two refusals Opus named as widening what stops a whole box are listed as decisions (11 and 12 in the decisions file), not made as changes. The deep items stay open and are listed in `READINESS.md`; none is minor, and the branch cannot be wired until they are closed or decided (the walk by directory descriptor, litestream's opens by path, durable transition state, the uid binding under approval H, the box-wide refusal decision). |
| - `feat/carloku-storage-sole-holder-20261001` `eaf6e63e`: round 9 both accept; CI passed; needs approval K and a cutover date. |
| - `fix/carloku-observe-routes-cleanup-20261001` `023e106c`: both accept; CI passed; merged into this branch at `47c97fa6`; the gate on the launch tip `b953fae4` passed (exit 0, three cross builds). |
| @@ -45,6 +45,6 @@ Branches (all on the forge, none merged; the local gate is not CI): |
| - Then, each on its own branch: quantity reduction at renewal, the evidence door for a box's first minutes, the AWS principal inventory and the bucket ownership attestation door, Free-tier enforcement, dormancy and pool accounting. |
| **SESSION:** |
| -Running at 01:42: round twenty-seven on the currency tip `9b872e38` (Opus and Astra, since 01:34); the gate on the currency tip `9b872e38` (since 01:38). Queued: the gate on this branch's merged tip `c4ee8807` (after the currency's; commits after it are records only). Then: read round twenty-seven and, if both accept with nothing left and CI is green on `9b872e38`, merge the currency branch into this branch and gate that merged tip; merge `9b872e38` into the operator CLI branch and gate it. Full package runs need `GOFLAGS=-buildvcs=false`, as the gate sets it. Detached `setsid` processes. A gate must be started with `logs/run-gate.sh`, which uses the default build cache; full runs take the gate's `-timeout 30m`. |
| +Running at 01:48: the gate on the currency tip `2b0abef1` (alone). Done: the gate on this branch's tip `0491ff76` (the inventory merge plus records; `gate-launch-0491ff76.txt`), the gate on `9b872e38` (00:42Z). Then: merge the currency branch into this branch as a merge commit and gate that merged tip; merge `2b0abef1` into the operator CLI branch and gate it; records. Full package runs need `GOFLAGS=-buildvcs=false`, as the gate sets it. Detached `setsid` processes. A gate must be started with `logs/run-gate.sh`, which uses the default build cache; full runs take the gate's `-timeout 30m`. |
| Scratch on the box: `~/.cache/tmp/claude-1001/{currency-wt,replicator-wt,fencing-wt,evidence-wt,opcli-wt,inventory-wt,flush-wt,reduction-wt}` (git worktrees), the demo binaries, `/var/tmp/ck`, `~/.cache/carloku-gate`. The void gate logs are named `*VOID*`. |
diff --git a/implementation/carloku-paid-launch/READINESS.md b/implementation/carloku-paid-launch/READINESS.md| index e42fc2ae2..1b14a4910 100644 |
| --- a/implementation/carloku-paid-launch/READINESS.md |
| +++ b/implementation/carloku-paid-launch/READINESS.md |
| @@ -14,7 +14,7 @@ Log files are under `/home/paulca/.local/share/carloku-paid-launch-20260930/logs |
| | Branch | Tip | Gate | CI | Reviews | State | |
| | --- | --- | --- | --- | --- | --- | |
| -| `feat/carloku-multi-currency-20261001` | `9b872e38` (round twenty-six's fixes: the two-console test's losing-writer ordering requires `SQLITE_BUSY_SNAPSHOT` and its first ordering asserts the evidence's withdrawal; three comments, two design sentences and two dispositions claims corrected; no production code changed apart from comments); before it `35356ad0` (round twenty-five answered: the commit column's migration one transaction, with a regression that aborts the marking and sees the column roll back; the unconditional release moved to a `_test.go` file so no handler can release a committed reservation; two consoles at the point of no return tested on two connections to one file; the no-bucket preflight's release asserted); before it `1f1368b0` (round twenty-four answered: the committed retirement durable against every handler cleanup, the handler's commit tested, old reservations committed; round twenty-five's Opus P3s are in the worktree, uncommitted); before it `e152df30`, round twenty-three answered: a point of no return for an account's retirement; before it `2618403f` (round twenty-two answered: the rule's edges; before it `05ccec07`, round twenty-one answered as one rule: a subscription that names an intent is recorded from the first read, and nothing deletes an intent past one; before it `b260ac8e` (round twenty answered by making the evidence invariant structural: every provider read of a session or subscription goes through three recording methods, a parser test forbids any other; before it `43d326b5` (round nineteen answered as one evidence invariant; before it `59666bab`, stage one of the retirement protocol; before it `15041ca2`: round eighteen's bounded findings fixed; round eighteen's and the three design reviews' reports in the tree) (`68f6ee23`: the seventeenth review's findings: the review filed before the expiry on both paths, the `keep` door's replay, nil billing and purchase check, the rejected listing's account, order and bound, two new tests; reviews ten to seventeen's reports in the tree) | `9b872e38`: queued behind the inventory gate (the full package run before the commit passed, console 549 s, with `-buildvcs=false` as the gate has it). `35356ad0`: **passed** (23:57Z, `gate-currency-35356ad0.txt`, exit 0, three cross builds, alone; the full package run before the commit passed, console 538 s). `1f1368b0`: **passed** (23:33Z, `gate-currency-1f1368b0.txt`, exit 0, three cross builds, alone). `2618403f`: **failed** (22:28Z, `gate-currency-2618403f.txt`: one existing test whose expectation round twenty-two's own fix made out of date; corrected in `ef8d9440`); `e152df30`: **passed** (22:54Z, `gate-currency-e152df30.txt`, exit 0, three cross builds, alone). `05ccec07`: **passed** (22:04Z, `gate-currency-05ccec07.txt`, exit 0, three cross builds, alone, default cache); `2618403f`: started 22:18Z (`gate-currency-2618403f.txt`). passed on every code tip through `db130cfc` (`gate-currency-db130cfc.txt`, exit 0, three cross builds, clean tree); `68f6ee23`: passed (09:17Z, `gate-currency-68f6ee23.txt`, exit 0, three cross builds, clean tree); the console, plans and billingstripe packages passed in full on the same tree before the commit (console 525 s). `15041ca2`: passed (10:06Z, `gate-currency-15041ca2.txt`, exit 0, three cross builds); `59666bab`: passed (13:20Z, `gate-currency-59666bab.txt`, exit 0, three cross builds); `43d326b5`: **failed** (exit 2, `gate-currency-43d326b5-FAILED-e2e-build-toolchain-error.txt`): three root-package tests that build the binary failed with "is not in std" and "open …/gocache/…: no such file". Cause established: that gate inherited a scratch build cache (`~/.cache/tmp/gocache`) from the shell that started it, and the directory was deleted and recreated while it ran (its birth time is 17:15:07, after the 17:06 failures). Not a defect of the tree; `run-gate.sh` now unsets `GOCACHE` and prints the cache it uses. Not rerun on that tip. `b260ac8e`: **passed** (21:13Z, `gate-currency-b260ac8e.txt`, exit 0, three cross builds, default cache, run alone); the console, plans and billingstripe packages passed in full on the same tree before the commit (console 596 s, `currency-stage1-run3.txt`). Void or superseded logs, not evidence: `gate-currency-a9304f16.txt`, `gate-currency-f269365a-VOID-dirty-tree.txt`, `gate-currency-25487506-ended-on-28dc2629-reports-added-during-run.txt`, `gate-currency-db130cfc-VOID-dirty-tree-second-run.txt` (started by mistake on staged, uncommitted changes and stopped) | passed on every pushed tip through `35356ad0`; `9b872e38` not yet read | **Round 27 (on `9b872e38`): running since 01:34 (Opus and Astra), a confirming review of round twenty-six's fixes.** **Round 26 (on `35356ad0`): the first round in which both reviewers accept.** Opus accept (no P1 or P2; "review eighteen's P2 is closed on this branch"; three P3s); Astra accept with fixes (no new P1 or P2; "review eighteen's P2 is closed on this branch" under the roll rule; one P3 and seven claims), all answered in `9b872e38`. Review eighteen's P2 is recorded closed at that round. Round 25 answered in `35356ad0` (both reviewers' additions: the two-connection test and the release out of production code; Astra's P2, the migration). **Round 25 (on `1f1368b0`): Opus accept with fixes (no P1 or P2; three P3s; "for this binary, review eighteen's P2 is closed", wanting a two-connection test and the unconditional release out of production code, both now in the worktree); Astra's report pending at 00:38.** **Rounds 23 (on `2618403f`) and 24 (on `e152df30`): Opus accept with fixes, Astra reject, each time. Not accepted.** Opus found no P1 or P2 in either and said twice it would count review eighteen's P2 closed; Astra found one P2 each time at the account-retirement boundary (23: a dispute committed between the reservation and the tombstone; 24: a losing request's cleanup removing another's committed reservation), each real, each answered (a point of no return in the database; a release that touches only a pending reservation; the handler tested with a hook on the bucket). The round-24 answer is uncommitted at 00:05 and goes to round 25. **Round 22 (on `05ccec07`): Opus accept with fixes, Astra reject. Not accepted.** Opus: the named-subscription rule holds, review eighteen's P2 is closed for deletion and consumption; one P2 on what the rule costs an honest buyer whose provider left a dead subscription (answered as a test-mode gate, `ENABLEMENT.md` section 6 step 2, and decision 15, not as code); four P3s. Astra: three P2s at the rule's edges (account retirement under an open checkout dispute; a session-only refusal filed after another console recorded the purchase; a click racing a filing), one P3; all answered in `2618403f` with tests and nine mutants. Round 23 running on `2618403f`. Round 21 (on `b260ac8e`): Opus accept with fixes, Astra reject; answered in `05ccec07` (27 mutants; one defect found there by neither reviewer, a later event of a recorded subscription consuming the standing intent its metadata named). **Round 21 (on `b260ac8e`): Opus accept with fixes (one P2, three P3), Astra reject (three P2, two P3). Not accepted.** Both closed every round-twenty finding and both found the same new P2, opened by round twenty's own fix: a refusal can now be filed on an intent that never had a session, the 25-hour cleanup of such an intent ignores the review, and `retire` then passes with a subscription attached. Astra also: a subscription seen before any refusal vanishes when an ordinary purchase consumes the intent; the held currency and customer are not rechecked in the transaction that creates an intent (two consoles); a listing's first page is dropped when a later page fails; refusal retries are not fair beyond the budget. Opus also: a bind can move an intent under review onto another session; a hand-made subscription naming an intent whose session is open is retried, never reviewed; the store keys on the answer's id. Reports `logs/opus-currency-review21.md`, `astra-currency-review21.md`. **Review eighteen's P2 stays open.** Round 20 (on `43d326b5`): Opus accept with fixes, Astra reject; answered in `b260ac8e` (43 mutants in a tree copy, four survived the first run and have tests now). Rounds 1 to 9: Opus accept with fixes, Astra reject. Rounds 10 to 13: both accept with fixes (P3s). Round 14: Astra reject on a P2 (a reconciliation paused on the provider read while the operator resolved the review recorded the retired purchase), fixed in `abd1b76e`. Rounds 15 and 16: both accept with fixes, each closing what the previous fix left open in the intent's lifecycle (15: nothing records under an open review, the resolution retires or keeps; 16: `keep` only for an intent whose bound session is complete with a subscription, the outcome written, replay-compared and audited). Each fix has a test that fails without it and a killed mutant (dispositions, one section per round). Round 17 at `db130cfc`: Opus accept with fixes (four P3s on the `keep` door and the rejected listing), Astra reject on a P2: both sites that expire a bound session expired first and filed the review second, so a pass reading the expired session in between, or a crash between the two, retired the intent with no record; fixed in `68f6ee23` by filing first (a two-path race test runs the store's retirement inside the fake's expiry and wants the conflict; it fails with either site's order put back), with the P3s, two new tests (an older resolutions table migrates with `retire`; the listing's 200-row bound, newest first, account on each row) and the comments and design sentence both named. The first form of the listing's account fix looked the account up under open rows and deadlocked the store's one connection in the test; rows are read to the end first. Round 18 at `68f6ee23`: Opus accept with fixes, Astra reject, on the same P2: with the review now filed before the expiry, an operator's `retire` can delete the intent while its bound session is still open and payable (the expiry failed; nothing retries it; the resume path still offers the URL). The supervisor ruled that this is not fixed by moving calls again but by a durable protocol (recorded refusal authoritative; the session's fate proven from provider evidence and recorded; atomic operator completion; a paid session settled by a separate disposition) with one bounded design review before another implementation cycle. Revision 1 of that design (`8c72e88b`) was rejected by Astra (four P2s), and revision 2 (`b4ff6b72`) too (five P2s: the hand-made subscription's own obligations at `retire`; the settlement predicate; observation times against cached answers; which evidence a dispute's resolution makes authoritative; the migration barrier). Revision 3 (`15041ca2`) was rejected a third time (four P2s, each with a conservative hold as its smallest change); revision 4 applies those holds and is built as **stage one** in `59666bab` (`billingretirement.go`): the operator's `retire` is accepted only for a session proven expired and unpaid by a provider read with nothing attached and nothing contradictory; `keep` only for a completed, paid session with no subscription attached but its own; a paid session that is not kept, any case a hand-made subscription touched, any contradicted case and every review from before the build are held; the door requests the expiry and answers 503 while it is unproven; the pass retries on a schedule; a click under review gets no URL; contradictions are disputes; a delete barrier stops an older binary. Eight tests, one per interleaving; eleven existing tests reworked; seventeen mutants killed. **Round nineteen did not accept it**: Opus accept with fixes (one P2, six P3s), Astra reject (three P2s): a subscription on the purchase's own terms was never attached, so `keep` could release the review beside it; the refusal and its subscription were two transactions; the keep gate's fresh read was not recorded as evidence; contradictions arriving after a retirement or a kept purchase's record, or read by an actor with a stale view, were dropped. These are answered in `43d326b5` as one invariant (every provider observation about a refused purchase classified and recorded in one store transaction against the purchase as it stands, retired, or kept and consumed; a refusal and its subscription one transaction; a caller records before it answers; nothing dropped), with seven counterexample tests, four reworked, and twenty mutants killed. Round twenty (Opus and Astra) is reading it. **Review eighteen's P2 stays open and blocks the merge until a round accepts this code.** No test runs two console processes against one database; the interleavings are driven through the store and provider hooks, and the dispositions say so. The bounded findings of round eighteen are fixed there with tests and four killed mutants (attribution of a retired session's completion beside a replacement; a `keep` decided during the gate answered by the replay comparison; the nil-billing and tie-break tests). The findings since round ten were not minor: each was a real gap in the lifecycle of an intent under review, and the rounds have converged on the operator's door rather than the money path | integrated end to end; not merged; **the merge (a merge commit onto this branch) follows round twenty-seven if it ends with both accepting and nothing left to fix, and CI green on `9b872e38`**; before: integrated end to end; not merged; the merge waits on a round ending with both accepting and nothing left to fix, and CI on the tip | |
| +| `feat/carloku-multi-currency-20261001` | `2b0abef1` (review twenty-seven's one comment correction, made before the merge as Opus allowed, without another review; the review-27 dispositions and reports); before it `9b872e38` (round twenty-six's fixes: the two-console test's losing-writer ordering requires `SQLITE_BUSY_SNAPSHOT` and its first ordering asserts the evidence's withdrawal; three comments, two design sentences and two dispositions claims corrected; no production code changed apart from comments); before it `35356ad0` (round twenty-five answered: the commit column's migration one transaction, with a regression that aborts the marking and sees the column roll back; the unconditional release moved to a `_test.go` file so no handler can release a committed reservation; two consoles at the point of no return tested on two connections to one file; the no-bucket preflight's release asserted); before it `1f1368b0` (round twenty-four answered: the committed retirement durable against every handler cleanup, the handler's commit tested, old reservations committed; round twenty-five's Opus P3s are in the worktree, uncommitted); before it `e152df30`, round twenty-three answered: a point of no return for an account's retirement; before it `2618403f` (round twenty-two answered: the rule's edges; before it `05ccec07`, round twenty-one answered as one rule: a subscription that names an intent is recorded from the first read, and nothing deletes an intent past one; before it `b260ac8e` (round twenty answered by making the evidence invariant structural: every provider read of a session or subscription goes through three recording methods, a parser test forbids any other; before it `43d326b5` (round nineteen answered as one evidence invariant; before it `59666bab`, stage one of the retirement protocol; before it `15041ca2`: round eighteen's bounded findings fixed; round eighteen's and the three design reviews' reports in the tree) (`68f6ee23`: the seventeenth review's findings: the review filed before the expiry on both paths, the `keep` door's replay, nil billing and purchase check, the rejected listing's account, order and bound, two new tests; reviews ten to seventeen's reports in the tree) | `2b0abef1`: running, alone (`gate-currency-2b0abef1.txt`). `9b872e38`: **passed** (00:42Z, `gate-currency-9b872e38.txt`, exit 0, three cross builds, alone). `35356ad0`: **passed** (23:57Z, `gate-currency-35356ad0.txt`, exit 0, three cross builds, alone; the full package run before the commit passed, console 538 s). `1f1368b0`: **passed** (23:33Z, `gate-currency-1f1368b0.txt`, exit 0, three cross builds, alone). `2618403f`: **failed** (22:28Z, `gate-currency-2618403f.txt`: one existing test whose expectation round twenty-two's own fix made out of date; corrected in `ef8d9440`); `e152df30`: **passed** (22:54Z, `gate-currency-e152df30.txt`, exit 0, three cross builds, alone). `05ccec07`: **passed** (22:04Z, `gate-currency-05ccec07.txt`, exit 0, three cross builds, alone, default cache); `2618403f`: started 22:18Z (`gate-currency-2618403f.txt`). passed on every code tip through `db130cfc` (`gate-currency-db130cfc.txt`, exit 0, three cross builds, clean tree); `68f6ee23`: passed (09:17Z, `gate-currency-68f6ee23.txt`, exit 0, three cross builds, clean tree); the console, plans and billingstripe packages passed in full on the same tree before the commit (console 525 s). `15041ca2`: passed (10:06Z, `gate-currency-15041ca2.txt`, exit 0, three cross builds); `59666bab`: passed (13:20Z, `gate-currency-59666bab.txt`, exit 0, three cross builds); `43d326b5`: **failed** (exit 2, `gate-currency-43d326b5-FAILED-e2e-build-toolchain-error.txt`): three root-package tests that build the binary failed with "is not in std" and "open …/gocache/…: no such file". Cause established: that gate inherited a scratch build cache (`~/.cache/tmp/gocache`) from the shell that started it, and the directory was deleted and recreated while it ran (its birth time is 17:15:07, after the 17:06 failures). Not a defect of the tree; `run-gate.sh` now unsets `GOCACHE` and prints the cache it uses. Not rerun on that tip. `b260ac8e`: **passed** (21:13Z, `gate-currency-b260ac8e.txt`, exit 0, three cross builds, default cache, run alone); the console, plans and billingstripe packages passed in full on the same tree before the commit (console 596 s, `currency-stage1-run3.txt`). Void or superseded logs, not evidence: `gate-currency-a9304f16.txt`, `gate-currency-f269365a-VOID-dirty-tree.txt`, `gate-currency-25487506-ended-on-28dc2629-reports-added-during-run.txt`, `gate-currency-db130cfc-VOID-dirty-tree-second-run.txt` (started by mistake on staged, uncommitted changes and stopped) | passed on every pushed tip through `2b0abef1` | **Round 27 (on `9b872e38`): both accept.** Opus accept (no P1 or P2; every review-26 finding closed; both strengthened assertions fail without their code; review eighteen's P2 remains closed; one P3, a comment, correctable before the merge without another review); Astra accept (no new P1, P2 or P3; nothing to fix on the branch; accepts the tip for the merge commit). **This is the merge round.** **Round 26 (on `35356ad0`): the first round in which both reviewers accept.** Opus accept (no P1 or P2; "review eighteen's P2 is closed on this branch"; three P3s); Astra accept with fixes (no new P1 or P2; "review eighteen's P2 is closed on this branch" under the roll rule; one P3 and seven claims), all answered in `9b872e38`. Review eighteen's P2 is recorded closed at that round. Round 25 answered in `35356ad0` (both reviewers' additions: the two-connection test and the release out of production code; Astra's P2, the migration). **Round 25 (on `1f1368b0`): Opus accept with fixes (no P1 or P2; three P3s; "for this binary, review eighteen's P2 is closed", wanting a two-connection test and the unconditional release out of production code, both now in the worktree); Astra's report pending at 00:38.** **Rounds 23 (on `2618403f`) and 24 (on `e152df30`): Opus accept with fixes, Astra reject, each time. Not accepted.** Opus found no P1 or P2 in either and said twice it would count review eighteen's P2 closed; Astra found one P2 each time at the account-retirement boundary (23: a dispute committed between the reservation and the tombstone; 24: a losing request's cleanup removing another's committed reservation), each real, each answered (a point of no return in the database; a release that touches only a pending reservation; the handler tested with a hook on the bucket). The round-24 answer is uncommitted at 00:05 and goes to round 25. **Round 22 (on `05ccec07`): Opus accept with fixes, Astra reject. Not accepted.** Opus: the named-subscription rule holds, review eighteen's P2 is closed for deletion and consumption; one P2 on what the rule costs an honest buyer whose provider left a dead subscription (answered as a test-mode gate, `ENABLEMENT.md` section 6 step 2, and decision 15, not as code); four P3s. Astra: three P2s at the rule's edges (account retirement under an open checkout dispute; a session-only refusal filed after another console recorded the purchase; a click racing a filing), one P3; all answered in `2618403f` with tests and nine mutants. Round 23 running on `2618403f`. Round 21 (on `b260ac8e`): Opus accept with fixes, Astra reject; answered in `05ccec07` (27 mutants; one defect found there by neither reviewer, a later event of a recorded subscription consuming the standing intent its metadata named). **Round 21 (on `b260ac8e`): Opus accept with fixes (one P2, three P3), Astra reject (three P2, two P3). Not accepted.** Both closed every round-twenty finding and both found the same new P2, opened by round twenty's own fix: a refusal can now be filed on an intent that never had a session, the 25-hour cleanup of such an intent ignores the review, and `retire` then passes with a subscription attached. Astra also: a subscription seen before any refusal vanishes when an ordinary purchase consumes the intent; the held currency and customer are not rechecked in the transaction that creates an intent (two consoles); a listing's first page is dropped when a later page fails; refusal retries are not fair beyond the budget. Opus also: a bind can move an intent under review onto another session; a hand-made subscription naming an intent whose session is open is retried, never reviewed; the store keys on the answer's id. Reports `logs/opus-currency-review21.md`, `astra-currency-review21.md`. **Review eighteen's P2 stays open.** Round 20 (on `43d326b5`): Opus accept with fixes, Astra reject; answered in `b260ac8e` (43 mutants in a tree copy, four survived the first run and have tests now). Rounds 1 to 9: Opus accept with fixes, Astra reject. Rounds 10 to 13: both accept with fixes (P3s). Round 14: Astra reject on a P2 (a reconciliation paused on the provider read while the operator resolved the review recorded the retired purchase), fixed in `abd1b76e`. Rounds 15 and 16: both accept with fixes, each closing what the previous fix left open in the intent's lifecycle (15: nothing records under an open review, the resolution retires or keeps; 16: `keep` only for an intent whose bound session is complete with a subscription, the outcome written, replay-compared and audited). Each fix has a test that fails without it and a killed mutant (dispositions, one section per round). Round 17 at `db130cfc`: Opus accept with fixes (four P3s on the `keep` door and the rejected listing), Astra reject on a P2: both sites that expire a bound session expired first and filed the review second, so a pass reading the expired session in between, or a crash between the two, retired the intent with no record; fixed in `68f6ee23` by filing first (a two-path race test runs the store's retirement inside the fake's expiry and wants the conflict; it fails with either site's order put back), with the P3s, two new tests (an older resolutions table migrates with `retire`; the listing's 200-row bound, newest first, account on each row) and the comments and design sentence both named. The first form of the listing's account fix looked the account up under open rows and deadlocked the store's one connection in the test; rows are read to the end first. Round 18 at `68f6ee23`: Opus accept with fixes, Astra reject, on the same P2: with the review now filed before the expiry, an operator's `retire` can delete the intent while its bound session is still open and payable (the expiry failed; nothing retries it; the resume path still offers the URL). The supervisor ruled that this is not fixed by moving calls again but by a durable protocol (recorded refusal authoritative; the session's fate proven from provider evidence and recorded; atomic operator completion; a paid session settled by a separate disposition) with one bounded design review before another implementation cycle. Revision 1 of that design (`8c72e88b`) was rejected by Astra (four P2s), and revision 2 (`b4ff6b72`) too (five P2s: the hand-made subscription's own obligations at `retire`; the settlement predicate; observation times against cached answers; which evidence a dispute's resolution makes authoritative; the migration barrier). Revision 3 (`15041ca2`) was rejected a third time (four P2s, each with a conservative hold as its smallest change); revision 4 applies those holds and is built as **stage one** in `59666bab` (`billingretirement.go`): the operator's `retire` is accepted only for a session proven expired and unpaid by a provider read with nothing attached and nothing contradictory; `keep` only for a completed, paid session with no subscription attached but its own; a paid session that is not kept, any case a hand-made subscription touched, any contradicted case and every review from before the build are held; the door requests the expiry and answers 503 while it is unproven; the pass retries on a schedule; a click under review gets no URL; contradictions are disputes; a delete barrier stops an older binary. Eight tests, one per interleaving; eleven existing tests reworked; seventeen mutants killed. **Round nineteen did not accept it**: Opus accept with fixes (one P2, six P3s), Astra reject (three P2s): a subscription on the purchase's own terms was never attached, so `keep` could release the review beside it; the refusal and its subscription were two transactions; the keep gate's fresh read was not recorded as evidence; contradictions arriving after a retirement or a kept purchase's record, or read by an actor with a stale view, were dropped. These are answered in `43d326b5` as one invariant (every provider observation about a refused purchase classified and recorded in one store transaction against the purchase as it stands, retired, or kept and consumed; a refusal and its subscription one transaction; a caller records before it answers; nothing dropped), with seven counterexample tests, four reworked, and twenty mutants killed. Round twenty (Opus and Astra) is reading it. **Review eighteen's P2 stays open and blocks the merge until a round accepts this code.** No test runs two console processes against one database; the interleavings are driven through the store and provider hooks, and the dispositions say so. The bounded findings of round eighteen are fixed there with tests and four killed mutants (attribution of a retired session's completion beside a replacement; a `keep` decided during the gate answered by the replay comparison; the nil-billing and tie-break tests). The findings since round ten were not minor: each was a real gap in the lifecycle of an intent under review, and the rounds have converged on the operator's door rather than the money path | integrated end to end; **merges into this branch as a merge commit once the gate passes on `2b0abef1` (CI has passed on it)**; before: integrated end to end; not merged; the merge waits on a round ending with both accepting and nothing left to fix, and CI on the tip | |
| | `feat/carloku-billing-layout-20261001` | `c1ae91a0` | (templates only; gated inside the currency branch) | passed | none of its own | merged into the currency branch at `5047494b`; its two-form layout was replaced by one form in `8958392b` | |
| | `feat/carloku-storage-sole-holder-20261001` | `eaf6e63e` | passed on `3af02a65`; `eaf6e63e` is wording and reports | passed | 9 rounds; round 9 both accept (omitted-holder class closed) | needs approval K and a recorded cutover date; not merged | |
| | `feat/carloku-metered-replicator-20261001` | `e1280811` (review twenty answered with comments, a fixture and dispositions only; the code is the twentieth build `c9ec11eb`: a tree claim refuses a tenant's file in a platform directory too, the fixture's guard reachable, the no-recorder refusal quoted; after `53e3dcd8`, the nineteenth build: a tenant's file in a platform directory refused in both layouts, the long-target fixture within the pathname budget and under XFS's link limit, refusal messages quoted; after `6249af63`, the eighteenth; reviews eight to eighteen's reports in the tree) | passed on every code tip through `f306222c`, then **failed on `bffc76b5`** (`gate-replicator-bffc76b5.txt`, exit 2: the seventeenth build's long-target fixture held only under a temp root of about 34 bytes or more; the gate's `/var/tmp/ck` is shorter), passed on `6249af63` (09:04Z, `gate-replicator-6249af63.txt`, exit 0, three cross builds, clean tree); `53e3dcd8`: passed (09:38Z, `gate-replicator-53e3dcd8.txt`, exit 0, three cross builds); `c9ec11eb`: passed (09:52Z, `gate-replicator-c9ec11eb.txt`, exit 0, three cross builds); `e1280811`: passed (10:10Z, `gate-replicator-e1280811.txt`, exit 0, three cross builds). Void: `gate-replicator-e8211fc6-VOID-stopped-for-astra17-followup.txt` (stopped so Astra's report could be answered in the same build) | passed through `f306222c`; **failed on `bffc76b5`** (the same fixture); passed on `6249af63`, `53e3dcd8`, `c9ec11eb` and `e1280811` | 4 to 7: each Opus accept with fixes, Astra reject; 8: both reject on one P1 the eighth build made (fixed in the ninth); 9 to 15: Opus accept with fixes, Astra reject, each round finding the previous build's own addition wanting (15 at `03b19014`: the fifteenth build joined a relative link target and so cleaned a `..` the kernel resolves physically, a regression that let a link to a directory outside the tree pass in silence; fixed in the sixteenth build by concatenating uncleaned, with the fixture both asked for); 16 at `f306222c`: Opus accept with fixes, Astra reject; both found the fifteenth's regression closed and one bound Astra named (the concatenated spelling could outgrow PATH_MAX), fixed in the seventeenth build `bffc76b5` with `fstatat` against the link's directory. 17 at `bffc76b5`: Opus accept with fixes, Astra reject; both named the same two new defects in that fix: the directory was opened by path with Go's default flags (a FIFO a tenant swaps in blocks the watch and Close; a device would have its driver's open run) and the long-target fixture failed under any short temp root (which is what failed the gate and CI). The eighteenth build opens the directory with `O_DIRECTORY|O_NONBLOCK|O_NOCTTY|O_CLOEXEC` (anything but a directory refused by the open itself; three mutants killed: default flags, `O_DIRECTORY` alone dropped, the fixed fixture size under `/var/tmp/ck`), sizes the fixture from the temp root, compares `render` across sections, and corrects the comments and names Astra listed; the false "package tests pass under /tmp" claim of the seventeenth build is marked false in its dispositions. 18 at `6249af63`: Opus accept (three P3s: the fixture's 3 KB link target, which XFS refuses; a tenant's file in a platform directory accepted under the tenant's tree and refused under the platform's; wording), Astra reject on the deep items (and the fixture's fixed depth). The nineteenth build `53e3dcd8` answers them (two mutants killed). 19 at `53e3dcd8`: Opus accept (two P3s), Astra reject on the deep items; the twentieth build `c9ec11eb` makes the bounded changes (one mutant killed) and its dispositions list the deferred items separately, each as blocking wiring and needing a design or a decision. 20 at `c9ec11eb`: Opus accept with fixes, Astra reject on the deep items; `e1280811` changes no behaviour and is not sent for another round; the two refusals Opus named as decisions are in `DECISIONS-FOR-PAUL.md` (11, 12). Each build's dispositions section lists its killed mutants. The deep items stay open and are listed there: litestream's own opens by path (every reopen), protected endpoints under tenant-replaceable ancestry, a delayed admitted request past the margin, requests across a crash or restart, the header open through a swapped ancestor (a walk by directory descriptor, required before wiring), a tenant sharing the platform's uid, root-run units and a route naming another user's coherent tree (approval H), hibernating aliases and claims on skipped files, required coverage that can vanish without an entry, whether one tenant's unaccepted links refuse the whole box (a decision before wiring), one tenant's deletion holding every account's days once (a stated cost). None of these is minor: the branch cannot be wired until they are closed or decided | not merged; unwired side work gated on approval H | |