refactor(cli): cover functions delete, list, new with effect lint (CLI-2459)

[7ttp]

TL;DR

brings supabase functions delete, list and new under the effect lint

whats introduced?

effect lint applied to functions delete, list and new:

• four allow list entries in .oxlintrc.effect.json, the family root plus the three subcommands
• new yields its two tagged errors directly instead of wrapping them in Effect.fail
• list timestamps format through DateTime while invalid millis still render NaN-NaN-NaN NaN:NaN:NaN like native Date, pinned by a new unit test
• integration tests move onto the FileSystem/Path services, the env case runs through withEnvVar, cause assertions use Cause.pretty
• live tests run through the effect native cliEffect fixture

ref:

• related: CLI-2459

[github-actions]

🤖 AI Review

No production correctness bugs were found. Claude reported four quality/polish issues; all are confirmed, with the missing live-test cancellation signal raised to minor severity. Codex completed its review and reported no findings, and the adjudication pass found no additional issues.

Findings

Severity	Location	Category	Sources	Claim
🟡 MINOR	apps/cli/src/commands/functions/delete/delete.live.test.ts:30	test-quality	claude	The live test does not pass Vitest's abort signal to Effect.runPromise, so test cancellation cannot promptly interrupt its deploy, delete, and cleanup fibers.
⚪ NIT	apps/cli/src/commands/functions/list/list.integration.test.ts:207	naming	claude	Seven assertions assign Cause.pretty output to a variable named json even though the value is human-readable cause text, not JSON.
⚪ NIT	apps/cli/src/commands/functions/list/list.format.ts:12	readability	claude	The invalid-input fallback creates six NaN values and sends them through the normal formatting pipeline merely to reproduce a constant legacy string.
⚪ NIT	apps/cli/src/commands/functions/new/new.integration.test.ts:232	test-quality	claude	Effect.provide wraps withEnvVar, so the test layer is acquired outside the SUPABASE_YES scope, making the ordering fragile if a layer later reads that variable.

Stats

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

---

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

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