test(cli): cover functions deploy unit tests with effect lint (CLI-2592)

[7ttp]

TL;DR

brings the deploy unit tests under the effect lint

whats introduced?

• deploy.unit.test.ts joins the allow list
• the tests keep every existing case and run on @effect/vitest with BunServices, building fixtures through FileSystem and Path in scoped temp dirs and encoding JSON through Schema
• failure assertions read the exit cause instead of catching a rejection, and pin both import walk failures as defects, the FunctionImportNotDirectoryError and the native ENOENT host error
• a new test pins which entries discoverFunctionSlugs counts as functions, keeping symlinked directories and skipping files, links to files, broken links and directories without an entrypoint

ref

• related: CLI-2592

[github-actions]

🤖 AI Review

Both independent reviews completed and reported no findings. Reading the checked-out changes and implementation revealed no additional actionable issues: the Effect test migration preserves the assertions, scoped cleanup is appropriate, and the new symlink-discovery expectations match the implementation. Tests were not executed because dependencies are unavailable.

Findings

No issues found.

Stats

Claude findings: 0 · Codex findings: 0 · Confirmed: 0 · 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.