Skip to content

CLI: update tsup dependency#305

Merged
cnharrison merged 1 commit into
mainfrom
chore/update-cli-tsup
Jun 1, 2026
Merged

CLI: update tsup dependency#305
cnharrison merged 1 commit into
mainfrom
chore/update-cli-tsup

Conversation

@cnharrison

@cnharrison cnharrison commented May 21, 2026

Copy link
Copy Markdown
Contributor

Updates the Flatfile CLI's tsup dependency and adds a patch changeset so new installs resolve versions outside the reported tsup and esbuild advisory ranges.

Links

@obvious-autobuild-staging obvious-autobuild-staging Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Obvious Code Review

Verdict: ✅ CLEAN — no blocking findings

Summary

  • Blocker: 0
  • High: 0
  • Medium: 0
  • Suggestion: 2
  • Nit: 1

Suggestions

  • Advisory citations: The changeset mentions "reported tsup and esbuild advisory ranges" without citing specific CVE or advisory IDs. Worth linking the actual advisory for the audit trail.
  • Two-major-version jump (6→8): tsup 8.x has several breaking changes vs 6.x. No updates appear in the diff — confirm a build smoke test was run in to verify identical output.

Nit

  • shows bumping . Confirm this is expected and not a stray version from a parallel branch.

View full review in Obvious

@cnharrison cnharrison merged commit 43f4071 into main Jun 1, 2026
3 checks passed
@cnharrison cnharrison deleted the chore/update-cli-tsup branch June 1, 2026 16:55
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