Skip to content

Conversation

@kylewalke
Copy link

What issues does this PR fix or reference?

@W-19736546@

Summary

Migrate build system from tsup to esbuild and consolidate build configurations into a reusable npm package.

Changes

  • Created @salesforce/esbuild-presets package containing shared esbuild configurations for Node.js, browser, and web worker targets
  • Replaced tsup with esbuild across all packages:
    • apex-ls
    • apex-lsp-shared
    • apex-lsp-vscode-extension
    • apex-parser-ast
    • custom-services
    • lsp-compliant-services
  • Removed all tsup.config.ts files and replaced with esbuild.config.ts files
  • Updated package.json build scripts and dependencies across all affected packages
  • Updated tsconfig.json files to reference correct build outputs
  • Added esbuild preset package to turbo.json pipeline
  • Renamed test file from tsup-resources.test.ts to esbuild-config-resources.test.ts with updated test logic
  • Cleaned up gitignore entries related to old build artifacts
  • Updated documentation in README files to reflect new build system

Testing

All existing tests pass with the new build system. Build artifact tests updated to validate esbuild output instead of tsup output.

@kylewalke kylewalke requested a review from a team as a code owner December 12, 2025 00:25
@kylewalke kylewalke requested a review from jonnyhork December 12, 2025 00:25
@kylewalke kylewalke changed the base branch from main to tdx26/main December 12, 2025 00:26
@kylewalke kylewalke merged commit 777fafa into tdx26/main Dec 16, 2025
13 of 14 checks passed
@kylewalke kylewalke deleted the kdev/esbuildMigration branch December 16, 2025 02:01
peternhale pushed a commit that referenced this pull request Dec 16, 2025
… npm package (copy of #175) (#178)

* feat: migrate to esbuild

* feat: putting build configs into npm utility pkg

* fix: adding to turbo new esbuild preset pkg

* fix: web worker bundling fix

* fix: apply some more fixes

* fix: one more for vsix bundling
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.

2 participants