gambit-admin
  • Joined on 2026-06-06
gambit-admin commented on issue gambit/gambit#416 2026-09-04 20:53:21 +00:00
Audit: rate-limit scan/pricing endpoints that spend money per call

FOLLOW-THROUGH — PR Gambit-Inc/gambit#396 extends this closed issue: a sweep of all ~323 backend routes for the same pattern, adding limiters to 33 more across 19 files, plus an `import_max_upload…

gambit-admin commented on issue gambit/gambit#421 2026-09-04 20:50:57 +00:00
Audit: remove the startup schema-healing DDL from main.py (blocked by #420)

CORRECTION to my comment above — this was already in flight when I posted it, and the "flag first, delete after" decision it records is not what shipped.

**Lucas opened PR Gambit-Inc/gambit#407…

gambit-admin commented on issue gambit/gambit#422 2026-09-04 20:50:57 +00:00
Audit: split DB users — app can no longer ALTER/DROP (blocked by startup-DDL removal)

ADDENDUM — the remaining credential work is also already in progress, by Lucas.

Evidence, since Postgres does not timestamp role creation: gambit_app and gambit_migrate hold OIDs 4168635 and…

gambit-admin commented on issue gambit/gambit#422 2026-09-04 20:46:23 +00:00
Audit: split DB users — app can no longer ALTER/DROP (blocked by startup-DDL removal)

MEASURED ON PROD (2026-09-04, read-only via railway ssh runrecon-back) — this issue is substantially smaller than scoped. The groundwork already exists; the app just isn't using it.

gambit-admin commented on issue gambit/gambit#419 2026-09-04 20:27:17 +00:00
Decide: GitHub Team plan so CI can actually block merges

DECIDED — the premise changed twice, so restating it.

  1. The GitHub Free constraint is gone: the org is on the Team plan. Required status checks are available today. main simply has no…
gambit-admin commented on issue gambit/gambit#420 2026-09-04 20:27:17 +00:00
Audit: activate the GitHub Actions deploy pipeline with a real migration step

Delivered by #587. The pipeline exists and has promoted prod: deploy.yml builds a SHA-tagged API image to GHCR on push to main, promote-prod.yml detects a pending migration, backs up (integrity-che…

gambit-admin commented on issue gambit/gambit#421 2026-09-04 20:27:17 +00:00
Audit: remove the startup schema-healing DDL from main.py (blocked by #420)

UNBLOCKED — #420 is delivered (#587), so the migrate step this was waiting on now exists.

Current measurement (2e28456): main.py still issues 147 conn.execute(text(...)) statements plus…

gambit-admin commented on issue gambit/gambit#422 2026-09-04 20:27:17 +00:00
Audit: split DB users — app can no longer ALTER/DROP (blocked by startup-DDL removal)

DECIDED — full split (all DDL out of the application). Two findings that change the scope.

FINDING 1 — runtime DDL is NOT confined to main.py. Measured on 2e28456, nine other modules issue…

gambit-admin commented on issue gambit/gambit#425 2026-09-04 20:27:17 +00:00
Audit: move uploads off the container disk into object storage

DECIDED — access control, not relocation. Object storage is deferred with the rest of the scale-out work (#423/#424/#426/#427): file LOCALITY only matters once a second instance runs, and we are…

gambit-admin closed issue gambit/gambit#420 2026-09-04 20:27:17 +00:00
Audit: activate the GitHub Actions deploy pipeline with a real migration step
gambit-admin commented on issue gambit/gambit#418 2026-09-04 20:27:16 +00:00
Audit: confirm prod DB backups are running + do one restore drill

DECIDED — scope reduced deliberately, with the residual risk recorded.

We are NOT building the off-vendor backup layers yet. Railway PITR plus the rotated pre-migration dumps are the plan for…

gambit-admin opened issue gambit/gambit#597 2026-09-04 19:50:49 +00:00
Error when printing label after adding card to inventory
gambit-admin commented on issue gambit/gambit#596 2026-09-04 18:57:40 +00:00
VPS disk: rotate promote-prod backups and prune CI docker images (runner went offline on a full disk 2026-09-04)

Worth tracing the root cause one level further back: the backup volume that filled the disk today is a direct consequence of PR #406 (merged tonight), which made promote-prod/promote-web auto-fire…

gambit-admin commented on issue gambit/gambit#587 2026-09-04 18:57:27 +00:00
Move deploys to GitHub Actions: build-once → staging → gated promote to prod

Follow-up to my earlier comment on this issue (the one specifying workflow_dispatch as the ONLY trigger for staging->production, never a side effect of a merge): PR #406, merged after this issue…

gambit-admin closed issue gambit/gambit#550 2026-09-04 14:19:37 +00:00
Card picker price differs from inventory market price for Gengar card
gambit-admin closed issue gambit/gambit#551 2026-09-04 14:19:37 +00:00
Inventory item details screen shows 'rcn' instead of 'gcn'
gambit-admin opened issue gambit/gambit#596 2026-09-04 13:41:18 +00:00
VPS disk: rotate promote-prod backups and prune CI docker images (runner went offline on a full disk 2026-09-04)
gambit-admin opened issue gambit/gambit#595 2026-09-04 13:41:09 +00:00
Card Hedge crosswalk: a demoted or rejected match keeps its old identity_link active
gambit-admin commented on issue gambit/gambit#334 2026-09-04 12:27:14 +00:00
Card condition change does not update price

GitHub PR #251 (per-condition outlier band — played conditions no longer dropped by the NM-derived band, so they stop falling back to the NM price) merged to main 2026-09-04 (74175a8), deploying…

gambit-admin commented on issue gambit/gambit#331 2026-09-04 12:27:04 +00:00
Changing card condition in inventory does not update price

GitHub PR #251 (per-condition outlier band — played conditions no longer dropped by the NM-derived band, so they stop falling back to the NM price) merged to main 2026-09-04 (74175a8), deploying…