tools/sample-custody.ts built its expected Traitor deck from the printed
leaders only, but Advanced CHOAM adds its printed extra Auditor disc during
setup, so every fresh Advanced all-expansion sample failed at setup with
'physical traitor custody'. The expectation now includes the Auditor for
Advanced CHOAM seats; the resumed saved positions were unaffected because the
resume path recomputes the inventory from the game.
With the fix a fresh --profile combined-homeworld-nexus --rules both --players
all run completes 2/4 (both Advanced). The two Basic samples stop at phase 6
on 'Odd-force Basic Occupy awaits the preserved publisher casualty-rounding
ruling', one of the three other unresolved questions recorded in the capture
resumption note, not on the five implemented rulings.
tests/sample-custody.test.ts (11 cases) and the faction-games suite pass;
types and lint are green.
All 123 preserved failed-*.json captures were resumed in place with
tools/faction-games.ts --resume (same saved position, seed and physical
custody; no reset, no fresh re-run). 102 of the 116 that produced a report
now complete; seven collided on duplicate basenames and were re-run with
unique output directories.
- The low-Tupile CHOAM opening-income ruling unblocked all 23 captures that
stopped at response choamCharity, across the mixed/single/paired/native
Homeworld+Nexus profiles and the occupation choam-6-basic capture.
- The exhausted-cache count ruling unblocked all three captures that stopped
at decision richeseBlackMarket (seeds 20265033, 20273010 and the paired E3
discovery variant).
- Nine of the thirteen preserved homeworld-occupation captures now complete.
- Suk/Kwisatz, Guild Secret Ally return and Sandmaster relocation changed no
capture outcome.
The 14 remaining captures are gated on three other unresolved questions (the
competing-occupier/native-repopulation occupation lifecycle, the Homeworld
high-threshold revival-deployment timing and the odd-force Basic Ecaz Occupy
casualty rounding), not on the five implemented rulings. No guard was
weakened and no capture was force-completed.
The user answered five recorded questions; each ruling is indexed in
RULE_DECISIONS.md and implemented with a focused regression.
- CHOAM opening income counts as a qualifying collection, so low-population
Tupile adds its bank increment under the Inflation multiplier. The engine
guard is removed and the projection/notice expose the added bonus.
- An exhausted Richese cache is legal and restores one ordinary lot; the
canceled/prevented compulsory cache auction branch restores the reduction
before the ordinary pool runs and the cache offer is skipped when no cache
lot can run. The pure continuation quote mirrors the count.
- Suk-rescued casualties count toward Advanced Atreides' seven Kwisatz
Haderach losses: the battle-loss counter gains the full dialed allocation in
both rescue paths, and the earlier Suk availability restriction is removed.
- The Guild Nexus Secret Ally reserve return overrides the Arrakis-to-native-
Homeworld restriction for the holder's own reserves and deposits through the
shared native-reserve pipeline (generalized from the revival deposit).
- Interior passengers count as entering the territories a relocated Ixian HMS
points into, so Sandmaster collects there before the native collection.
Full offline suite 7804/7804, types and lint pass. Historical evidence and
superseded test expectations are updated to the ruled behaviour; preserved
combined Homeworld/Nexus, occupation and Richese captures are unblocked but
not re-run here. No existing game, seat or saved state was changed.
replaceMoritaniAssassinationTraitors took the reserve's first entry, but the
face-up set-aside card can still sit in that list: re-drawing it produced a
receipt whose replacement equalled the revealed card, which
moritaniAssassinateIntegrity rejects mid-Mentat. The step now draws the first
distinct physical Traitor Card and is blocked with an explicit reason when none
remains, instead of committing an invalid receipt.
This is the engine half of checkpoint e493bcf, whose message described it but
whose staged path list omitted game/engine.ts. Offline suite 7,801/7,801 with
typecheck and lint clean at that revision; this commit changes no behaviour the
suite did not already exercise.
Operations now report the SQLite database size and page count, read through
PRAGMA page_count/page_size in the same authority-checked batch as the
lifecycle counters with non-numeric values clamped to zero; the page renders
both and states that free disk space is still unknown.
replaceMoritaniAssassinationTraitors draws the first distinct physical Traitor
Card from the reserve instead of the reserve's first entry, because the face-up
set-aside card can still sit in that list: re-drawing it produced a receipt
whose replacement equalled the revealed card, which the assassination integrity
check rejects mid-Mentat. With no distinct card left the step is blocked with an
explicit reason instead of committing an invalid receipt.
Four tests were made reproducible after measuring their real failure rates,
because a suite that fails for random reasons cannot act as a checkpoint gate:
the Discovery typed Spice Blow test now removes other cards for the same
territory so one blow deposits the printed amount (0/20 after, ~20% before); the
Discovery E2 skills deal is pinned with the new shared
tests/deterministic-random.ts helper so a Lasgun-shield explosion cannot
pre-empt its skill consumer (0/12 after, ~20% before); the Ecaz Occupy Ix/CHOAM
policy check accepts any legal continuation and still asserts the phase it
reaches (0/30 after, ~10% before); and the mixed-E3 Moritani Cunning test keeps
the random storm off red_chasm's single sector (0/25 after, ~20% before).
Offline suite 7,801/7,801 with typecheck and lint clean. No game, seat, saved
state or deployment changes.
Rejected administrator authentication and authorization attempts are now
recorded durably in a new additive admin_attempts table with a fixed reason
(invalid_key_format, unknown_or_disabled_key, missing_session,
unknown_or_expired_session, role_denied), the attempt kind, the attempted role
and an already-known account id. No key, key hash, session token, session hash,
request body, origin, address or user agent is stored, and the
unknown/disabled distinction is deliberately collapsed so the lookup cannot
become an account-enumeration oracle.
GET /api/admin/operations?attempts=1 returns seven-day per-reason counters plus
the twenty most recent attempts, newest first, to a live owner only, with
no-store and the same query guard as the other reads. The operations page adds
"List failed attempts" with the counters and the bounded sample, truncating each
account id to eight characters. Recording is best-effort inside the rejection
path, so a rejected request keeps its own answer even when the table is
unavailable, and the history never locks out, disables or revokes anything by
itself.
Evidence: tests/admin-attempts.test.ts asserts every recorded reason, the
kind/role/account fields, that successful sign-ins and privileged reads record
nothing, that no credential text appears anywhere in the page, the reporting
window, the twenty-row bound and the owner-only recheck after demotion, and that
a dropped attempts table leaves the original rejection answer unchanged. The
anonymous HTTP boundary denies ?attempts=1, and the panel was verified in the
real browser against a genuine recorded rejection. The shared AdminError moved
to db/admin-error.ts so the recorder and the access module no longer form a
value-import cycle. Offline suite 7,801/7,801 with typecheck and lint clean. No
game, seat, saved state or deployment changes.
Quintine