Skip to content

feat: Migrate workflow discovery from TypeScript to Oxc#7

Draft
eersnington wants to merge 1 commit into
mainfrom
feat/oxc-parser-migration
Draft

feat: Migrate workflow discovery from TypeScript to Oxc#7
eersnington wants to merge 1 commit into
mainfrom
feat/oxc-parser-migration

Conversation

@eersnington

Copy link
Copy Markdown
Owner

Summary

  • Replaces TypeScript parser-based workflow discovery with an Oxc parser adapter.
  • Uses Oxc raw-transfer when supported and falls back to standard Oxc parsing otherwise.
  • Parses Oxc output into Sideffect-owned workflow AST types before analysis.
  • Models discovery failures with Effect tagged errors and translates them to Vite build errors at the boundary.
  • Keeps collectWorkflowEntries() synchronous and avoids leaking Vite diagnostics callbacks into the public API.
  • Adds Vite logging for the selected workflow discovery parser.
  • Adds regression coverage for empty re-exports so export {} from "./x" is not treated as export *.

Validation

  • bun run --cwd packages/sideffect check
  • bun run --cwd packages/sideffect test
  • bun run check
  • bun run test
  • bun run build:sideffect

@eersnington eersnington changed the title F feat: Migrate workflow discovery from TypeScript to Oxc Jul 2, 2026
@eersnington eersnington force-pushed the feat/oxc-parser-migration branch 4 times, most recently from 0226e1b to dca1c80 Compare July 2, 2026 23:04
@eersnington eersnington force-pushed the feat/oxc-parser-migration branch from dca1c80 to c49933c Compare July 8, 2026 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant