Skip to content

chore: remove rolldown override#363

Merged
toiroakr merged 1 commit into
mainfrom
remove-rolldown-override
Jan 6, 2026
Merged

chore: remove rolldown override#363
toiroakr merged 1 commit into
mainfrom
remove-rolldown-override

Conversation

@toiroakr
Copy link
Copy Markdown
Contributor

@toiroakr toiroakr commented Jan 6, 2026

Summary

  • Remove rolldown override and catalog from pnpm-workspace.yaml
  • Update packages/sdk/package.json to use direct version instead of catalog reference

The upstream issues that required this override have been resolved:

tsdown now specifies a fixed rolldown version (1.0.0-beta.58) instead of latest.

Test plan

  • pnpm install succeeds
  • pnpm exec turbo run build succeeds
  • pnpm exec turbo run test passes (649 tests)

🤖 Generated with Claude Code

@toiroakr toiroakr requested a review from remiposo as a code owner January 6, 2026 00:38
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Jan 6, 2026

🦋 Changeset detected

Latest commit: 282271e

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@tailor-platform/sdk Patch
@tailor-platform/create-sdk Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Jan 6, 2026

Open in StackBlitz

npm i https://pkg.pr.new/tailor-platform/sdk/@tailor-platform/create-sdk@363

commit: 282271e

@github-actions

This comment has been minimized.

@toiroakr toiroakr force-pushed the remove-rolldown-override branch from d281887 to 282271e Compare January 6, 2026 00:45
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jan 6, 2026

Code Metrics Report (packages/sdk)

main (2f5e2bf) #363 (3b11310) +/-
Coverage 48.6% 48.6% 0.0%
Code to Test Ratio 1:0.3 1:0.3 0.0
Details
  |                    | main (2f5e2bf) | #363 (3b11310) | +/-  |
  |--------------------|----------------|----------------|------|
  | Coverage           |          48.6% |          48.6% | 0.0% |
  |   Files            |            169 |            169 |    0 |
  |   Lines            |           4897 |           4897 |    0 |
  |   Covered          |           2383 |           2383 |    0 |
  | Code to Test Ratio |          1:0.3 |          1:0.3 |  0.0 |
  |   Code             |          26893 |          26893 |    0 |
  |   Test             |           9261 |           9261 |    0 |

SDK Configure Bundle Size

main (2f5e2bf) #363 (3b11310) +/-
configure-index-size 2.91KB 2.91KB 0KB
dependency-chunks-size 17.78KB 17.78KB 0KB
total-bundle-size 20.69KB 20.69KB 0KB

Runtime Performance

main (2f5e2bf) #363 (3b11310) +/-
Generate Median 2,471ms 2,272ms -199ms
Generate Max 2,488ms 2,381ms -107ms
Apply Build Median 2,784ms 2,541ms -243ms
Apply Build Max 2,809ms 2,608ms -201ms

Type Performance (instantiations)

main (2f5e2bf) #363 (3b11310) +/-
tailordb-basic 162,843 162,843 0
tailordb-optional 4,686 4,686 0
tailordb-relation 5,632 5,632 0
tailordb-validate 3,830 3,830 0
tailordb-hooks 6,846 6,846 0
tailordb-object 12,326 12,326 0
tailordb-enum 3,670 3,670 0
resolver-basic 14,196 14,196 0
resolver-nested 30,620 30,620 0
resolver-array 22,868 22,868 0
executor-schedule 11,178 11,178 0
executor-webhook 7,791 7,791 0
executor-record 13,331 13,331 0
executor-resolver 15,397 15,397 0
executor-operation-function 7,769 7,769 0
executor-operation-gql 7,772 7,772 0
executor-operation-webhook 7,785 7,785 0
executor-operation-workflow 9,032 9,032 0

Reported by octocov

Copy link
Copy Markdown
Contributor

@remiposo remiposo left a comment

Choose a reason for hiding this comment

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

LGTM

@toiroakr toiroakr merged commit 71fdaa9 into main Jan 6, 2026
17 checks passed
@toiroakr toiroakr deleted the remove-rolldown-override branch January 6, 2026 01:06
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