[7ttp]

TL;DR

brings the apps/cli-e2e test files and their fixtures under the effect lint

whats introduced?

• run and runNoProjectId return effects, every test body runs through one runE2e edge, and control calls go through one postControl helper instead of fetch
• a test that asks for a runner but passes without running a CLI command now fails, since vitest would otherwise pass a body that returned an unrun effect
• the same tests run with the same results

ref

• related: CLI-2879

[github-actions]

🤖 AI Review

Verified Claude’s single finding as a consistency nit; Codex reported no findings. No functional defect was established. Lint and tests were not run because dependencies are absent.

Findings

Severity	Location	Category	Sources	Claim
⚪ NIT	apps/cli-e2e/src/tests/telemetry.e2e.test.ts:99	consistency	claude	The migrated telemetry test retains a JSON.parse assertion inside Effect.gen, while the newly added workspace guidance says Effect lint rejects JSON.parse in Effect code.

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.

[7ttp]

passed the flake check here: https://github.com/supabase/cli/actions/runs/40980043982

[github-actions]

CLI benchmark: no changes found

Commit 7301531 against 7ttp/cli-2878-harness-coverage at de0210b · 10 runs each · report only, never blocks merging

Command	Before	After	Change
supabase --version	447 ms	305 ms	no real change (-32%, within normal variation)
supabase --help	228 ms	257 ms	no real change (+13%, within normal variation)
supabase init	204 ms	221 ms	no real change (+9%, within normal variation)
supabase migration new bench	236 ms	211 ms	no real change (-11%, within normal variation)
supabase gen types --lang typescript --project-id replayfixtureprojref	259 ms	251 ms	no real change
supabase projects list	213 ms	205 ms	no real change
supabase functions list --project-ref replayfixtureprojref	187 ms	181 ms	no real change

API calls: unchanged for all 7 commands.

Binary size: 134.0 MB, unchanged.

Details: timing ranges, API calls per command, slowest phases

Timing ranges

Command	Before range	After range	Change	Verdict
supabase --version	222-584 ms	214-596 ms	-143 ms (-32%)	within noise
supabase --help	191-334 ms	206-313 ms	+29 ms (+13%)	within noise
supabase init	186-282 ms	184-300 ms	+17 ms (+9%)	within noise
supabase migration new bench	195-373 ms	184-308 ms	-25 ms (-11%)	within noise
supabase gen types --lang typescript --project-id replayfixtureprojref	241-334 ms	229-298 ms	-8 ms (-3%)	within noise
supabase projects list	197-308 ms	187-220 ms	-8 ms (-4%)	within noise
supabase functions list --project-ref replayfixtureprojref	170-257 ms	173-214 ms	-6 ms (-3%)	within noise

API calls per command

Command	Before	After	Endpoint changes
supabase --version	0	0
supabase --help	0	0
supabase init	0	0
supabase migration new bench	0	0
supabase gen types --lang typescript --project-id replayfixtureprojref	2	2
supabase projects list	1	1
supabase functions list --project-ref replayfixtureprojref	2	2

Slowest phases

No command moved beyond noise, so there are no phase deltas.

Comment /bench to run it again on the latest commit.

Workflow run

[Coly010]

LGTM, thanks!