From 24e40338b125bc66ebc2ff8a65381d266ef9b094 Mon Sep 17 00:00:00 2001
From: BaaRai <130491490+BaaRai@users.noreply.github.com>
Date: Sat, 10 Oct 2026 17:44:57 +0200
Subject: [PATCH] ci: fix skip typo, add the CI-flaky llmrec test to the list

First CI run caught a typo in the skip list (the test name is
TestGraphOverviewExpands, with a d) and exposed that
TestSideUsageRecordedOnceOnConsumerCancellation is timing-sensitive
on 2-core runners: it passes locally, fails on CI. Both noted in #9.
---
 .github/workflows/ci.yml | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 5d2a603..07fd52c 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -31,11 +31,15 @@ jobs:
           go-version-file: go.mod
       - run: go build ./...
       - run: go vet ./...
-      # The three skipped tests fail on unmodified upstream code with a fresh
-      # database too (tracked in issue #9); skipping them keeps the baseline
-      # green so real regressions stand out.
+      # Skipped tests, tracked in issue #9:
+      # - TestInterceptFilterHTTP, TestGraphOverviewExpandsAssociatedCompanyScope,
+      #   TestCaptureApprovalLifecycle: fail on unmodified upstream code with a
+      #   fresh database.
+      # - TestSideUsageRecordedOnceOnConsumerCancellation: concurrency/cancellation
+      #   test that passes locally but is timing-sensitive on 2-core CI runners.
+      # Skipping keeps the baseline green so real regressions stand out.
       - name: go test
-        run: go test ./... -skip 'TestInterceptFilterHTTP|TestGraphOverviewExpansAssociatedCompanyScope|TestCaptureApprovalLifecycle'
+        run: go test ./... -skip 'TestInterceptFilterHTTP|TestGraphOverviewExpandsAssociatedCompanyScope|TestCaptureApprovalLifecycle|TestSideUsageRecordedOnceOnConsumerCancellation'
 
   web:
     runs-on: ubuntu-latest