[7ttp]

TL;DR

stops the unit and focused flaky checks failing before any test runs, by dropping the filter on the removed cli-go package

ref

• spotted while working on: test(cli): cover the remaining test files with effect lint (CLI-2898) #7414

[github-actions]

🤖 AI Review

Both independent reviews were available. The changed unit invocation and test expectation agree; no regression was identified. Claude's stale environment-variable finding is confirmed as a harmless, pre-existing cleanup nit outside the diff.

Findings

Severity	Location	Category	Sources	Claim
⚪ NIT	.github/scripts/flaky-check/cli.ts:115	dead-code	claude	The flaky-check runner still sets the unused SUPABASE_GO_BINARY variable to the removed apps/cli-go/supabase-go path for e2e suites.

Findings outside the diff

• ⚪ NIT .github/scripts/flaky-check/cli.ts:115 — The flaky-check runner still sets the unused SUPABASE_GO_BINARY variable to the removed apps/cli-go/supabase-go path for e2e suites.

Stats

Claude findings: 1 · Codex findings: 0 · Confirmed: 1 · Refuted: 0 · Uncertain: 0

---

Models: claude-opus-5-5 + gpt-6.1-sol · Trigger: auto · Workflow run

This review runs once per PR. A maintainer can request another with a /ai-review comment.

[github-actions]

Flaky test check: ✅ no flaky tests

Commit 09960d4f4459 · focused ×10 (no tests ran) · integration ×10, 6664 tests · unit ×10, 9267 tests · workflow run

_{Each execution is a separate Vitest process; Where lists failing runs, as run.iteration for suites executed more than once per run. The full data is in the flaky-check-report artifact.}