Skip to content

fix: resync pnpm-lock to match installed tsup variant#45

Merged
zaewc merged 1 commit into
developfrom
fix/missing-tsup-builds
May 19, 2026
Merged

fix: resync pnpm-lock to match installed tsup variant#45
zaewc merged 1 commit into
developfrom
fix/missing-tsup-builds

Conversation

@zaewc
Copy link
Copy Markdown
Owner

@zaewc zaewc commented May 19, 2026

The lockfile referenced tsup 8.5.0 without the @microsoft/api-extractor peer, but the variant materialized on disk includes it. The mismatch left preact/svelte/vue/react-native node_modules/tsup symlinks pointing to a non-existent .pnpm directory, breaking 'tsup' invocation during build.

The lockfile referenced tsup 8.5.0 without the @microsoft/api-extractor
peer, but the variant materialized on disk includes it. The mismatch left
preact/svelte/vue/react-native node_modules/tsup symlinks pointing to a
non-existent .pnpm directory, breaking 'tsup' invocation during build.
@zaewc zaewc self-assigned this May 19, 2026
@github-actions
Copy link
Copy Markdown

📊 Test Coverage Report (vitest)

Package Statements Branches Functions Lines
@scrolloop/core 218/218 (100%) 53/53 (100%) 25/25 (100%) 218/218 (100%)
@scrolloop/react 420/715 (58.74%) 73/89 (82.02%) 5/17 (29.41%) 420/715 (58.74%)
@scrolloop/react-native 0/238 (0%) 1/4 (25%) 1/4 (25%) 0/238 (0%)
@scrolloop/shared 176/176 (100%) 54/54 (100%) 14/14 (100%) 176/176 (100%)

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the pnpm-lock.yaml file, specifically modifying the tsup dependency version to include @microsoft/api-extractor. I have no feedback to provide as there are no review comments.

@zaewc zaewc merged commit b8797d5 into develop May 19, 2026
2 checks passed
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