9 commits

Author	SHA1	Message	Date
Ashwin Kumar Sivakumar	c307ef1ba4	fix(ci): run ai_credits tests via buildx build, not runtime apt-get/docker run -v ... The bare bookworm runner image has no apt-get/package manager at all (the earlier build-essential install attempt failed with 'apt-get: command not found'), and a plain 'docker run -v $PWD:/workspace' wouldn't work either - DOCKER_HOST points at the sibling dind engine (same nested-container setup build.yaml's own comments describe), so a bind mount would look for the path on the wrong filesystem. Matches build.yaml's actual working pattern instead: docker buildx build with context transfer (not a volume mount), using a Dockerfile (Dockerfile.test) whose RUN step runs the test suite and fails the build on a non-zero exit. TEST_DATABASE_URL passed via buildx --secret (not --build-arg) so it never lands in image layer history. Isolated this by testing each piece of the original workflow independently (env/secrets, id+GITHUB_OUTPUT, if: conditionals all checked out fine one at a time) until apt-get was confirmed as the actual failure point. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-08-13 18:08:31 +05:30
Ashwin Kumar Sivakumar	a0a4259d50	debug: test apt-get network access in isolation	2026-08-13 18:04:31 +05:30
Ashwin Kumar Sivakumar	24e85db097	debug: add back a conditional step to isolate the failure further	2026-08-13 18:01:39 +05:30
Ashwin Kumar Sivakumar	c0a961c8ef	debug: add back the check step to isolate the failure further	2026-08-13 17:59:04 +05:30
Ashwin Kumar Sivakumar	22cbd3e285	debug: add back job-level env secret reference to isolate the failure	2026-08-13 17:55:30 +05:30
Ashwin Kumar Sivakumar	e80c58641f	debug: strip test.yaml to bare minimum to isolate why the job dies mid-run	2026-08-13 17:53:14 +05:30
Ashwin Kumar Sivakumar	3740bb81c5	fix(ci): fetch full git history in ai-credits job for diff-based change detection ... Previous job died silently right after checkout with no logs at all from the 'Check for relevant changes' step onward - shallow clone (default fetch-depth:1, unlike build.yaml which sets fetch-depth:0) may have been the cause. Matching build.yaml's checkout config. Also re-set TEST_DATABASE_URL with a properly percent-encoded password (the raw '@' in it made the postgresql:// URI ambiguous to parse). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-08-13 17:48:43 +05:30
Ashwin Kumar Sivakumar	3b4a4eac3f	fix(ci): install build-essential before cargo test in ai-credits job ... The bare bookworm runner image has no C toolchain at all, unlike build.yaml's jobs which build inside a Dockerfile-based image with one already present. proc-macro2/libc/etc build scripts need `cc` to link and failed immediately (error: linker `cc` not found) before any real test code ran. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-08-13 17:39:12 +05:30
Ashwin Kumar Sivakumar	da9be7f6a3	feat(db,ci): fix missing ai_credits tables, wire integration tests to CI ... Re-enables and corrects 20260703210000_ai_credits_wallet, which had been disabled (.up.sql.skip) in commit 0163349 because 'relation already exists' aborted the original CREATE TABLE script on its very first table (ai_plans) - silently leaving ai_credit_ledger and ai_reservation_holds never created, despite the wallet system being actively wired into apps/users/handlers/ai_credits.rs, apps/payments, apps/cron with real subscription rows already in prod. Rewrote the migration to be idempotent: ADD COLUMN IF NOT EXISTS to bring ai_plans/user_ai_subscriptions/ai_feature_costs/ai_usage_logs up to the full column set crates/db/src/models/ai_credits.rs expects (safe zero/default backfills, no data loss - verified against the 3 existing subscription rows), and CREATE TABLE IF NOT EXISTS for the 2 genuinely-missing tables. down.sql updated to only reverse what this corrected version actually adds, not drop tables that predate it. Applied directly to the live database (verified schema + existing rows intact afterward). Also wires crates/db/tests/ai_credits.rs + ai_credits_reaper.rs to Forgejo Actions CI (docs/LIVE_SERVER_RUNBOOK.md step 6): - New TEST_DATABASE_URL repo secret, pointing at a dedicated nxtgauge_test database (created fresh, schema mirrored from live prod via pg_dump --schema-only - never runs against the real nxtgauge DB). - .forgejo/workflows/test.yaml: runs on push to main/high-performance, skips when crates/db/ isn't touched, cargo test -p db --test ai_credits --test ai_credits_reaper -- --test-threads=1. Also commits docs/openapi.wallet-holds.json (hand-written minimal OpenAPI spec used for schemathesis fuzzing in step 4) and gitignores the local .schemathesis/ cache directory from that run. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-08-13 17:31:22 +05:30