Skip to content

Conversation

@KMKoushik
Copy link
Member

@KMKoushik KMKoushik commented Jan 18, 2026

Summary

  • Fix SDK build by moving tsup CLI options to a config file (--noExternal is not a valid CLI option)
  • Add tsup.config.ts for proper build configuration
  • Bump SDK version from 1.5.7 to 1.6.0

Summary by cubic

Fixes the SDK build by moving tsup options into tsup.config.ts and bundling @usesend/lib correctly. Bumps the SDK version to 1.6.0.

  • Bug Fixes
    • Use noExternal: ["@usesend/lib"] in tsup.config.ts to prevent externalization issues.
    • Enable clean builds and replace CLI flags with a config; build script is now just "tsup".

Written for commit ca22d09. Summary will update on new commits.

Summary by CodeRabbit

Release Notes

  • Chores
    • Released SDK version 1.6.0
    • Optimized build process configuration

✏️ Tip: You can customize this high-level summary in your review settings.

@vercel
Copy link

vercel bot commented Jan 18, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
unsend-marketing Ready Ready Preview, Comment Jan 18, 2026 10:02am

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 18, 2026

Walkthrough

The change involves refactoring the SDK package's build setup. The version is bumped from 1.5.7 to 1.6.0. The build script in package.json is simplified from a multi-step command with explicit tsup parameters to a single "tsup" invocation. This is accompanied by the creation of a new tsup.config.ts configuration file that specifies the build parameters previously included in the script, including the entry point, output formats (esm and cjs), declaration file generation, and dependency handling.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes both main changes: adding a tsup config file for the SDK and bumping the version to 1.6.0, which aligns with the changeset and PR objectives.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


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.

@cloudflare-workers-and-pages
Copy link

Deploying usesend with  Cloudflare Pages  Cloudflare Pages

Latest commit: ca22d09
Status: ✅  Deploy successful!
Preview URL: https://d15c7a3b.usesend.pages.dev
Branch Preview URL: https://fix-sdk-tsup-config.usesend.pages.dev

View logs

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 2 files

@KMKoushik KMKoushik merged commit d3b6ba7 into main Jan 18, 2026
6 of 7 checks passed
@KMKoushik KMKoushik deleted the fix/sdk-tsup-config branch January 18, 2026 10:04
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