Skip to content

[Fix] @powersync/shared-internals should have @powersync/common as peer dependency - #1061

Merged
Manrich121 merged 1 commit into
mainfrom
fix/shared-internals-peer-dep
Aug 5, 2026
Merged

[Fix] @powersync/shared-internals should have @powersync/common as peer dependency#1061
Manrich121 merged 1 commit into
mainfrom
fix/shared-internals-peer-dep

Conversation

@Manrich121

Copy link
Copy Markdown
Contributor

Problem

While upgrading to V2 in another repo, we ran into the following:

Rollup failed to resolve import "@powersync/common" from 
.../@powersync/shared-internals/lib/client/BasePowerSyncDatabase.js 

Which required the following entry to be added in pnpm-workspace.yaml

packageExtensions:
  '@powersync/shared-internals':
    peerDependencies:
      '@powersync/common': '^2.0.0'

Solution

Declare @powersync/common as a peer dependency so the main entry point resolves it under strict node_modules layouts.

@Manrich121
Manrich121 requested a review from simolus3 August 4, 2026 14:12
@changeset-bot

changeset-bot Bot commented Aug 4, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: c787aaf

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

This PR includes changesets to release 7 packages
Name Type
@powersync/shared-internals Patch
@powersync/adapter-sql-js Patch
@powersync/capacitor Patch
@powersync/node Patch
@powersync/react-native Patch
@powersync/web Patch
@powersync/diagnostics-app 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

@Manrich121
Manrich121 merged commit 358eef4 into main Aug 5, 2026
16 checks passed
@Manrich121
Manrich121 deleted the fix/shared-internals-peer-dep branch August 5, 2026 06:16
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