Skip to content

feat: Update API build script and TypeScript configuration#4

Merged
BuckyMcYolo merged 1 commit intomainfrom
dev
Feb 16, 2026
Merged

feat: Update API build script and TypeScript configuration#4
BuckyMcYolo merged 1 commit intomainfrom
dev

Conversation

@BuckyMcYolo
Copy link
Copy Markdown
Owner

@BuckyMcYolo BuckyMcYolo commented Feb 16, 2026

Summary by CodeRabbit

  • Chores
    • Updated build configuration to enhance TypeScript declaration file generation through a refined multi-stage build process.
    • Adjusted module resolution and compilation settings for improved bundler compatibility.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Feb 16, 2026

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

The API build process shifts from single-stage (tsup with declarations) to two-stage (tsup then tsc for declarations). The API client removes a path alias mapping for the external API package and updates module/resolution settings to ESNext and Bundler, respectively.

Changes

Cohort / File(s) Summary
API Build Configuration
apps/api/package.json, apps/api/tsup.config.ts
Build script now chains tsup with tsc to emit TypeScript declarations; dts: true removed from tsup config, deferring declaration generation to the TypeScript compiler instead.
API Client Configuration
packages/api-client/tsconfig.json
Added module: "ESNext" and moduleResolution: "Bundler"; removed @repo/api/* path alias mapping and the entire paths configuration block.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 Declarations dance in two sweet steps,
No aliases to make the paths less direct—
TSC and tsup now pirouette together,
Building bundles lighter than a feather!

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch dev

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@BuckyMcYolo BuckyMcYolo merged commit a78fef2 into main Feb 16, 2026
1 check was pending
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