Skip to content

Replace npm with pnpm#5091

Draft
Copilot wants to merge 19 commits into
mainfrom
copilot/migrate-to-pnpm
Draft

Replace npm with pnpm#5091
Copilot wants to merge 19 commits into
mainfrom
copilot/migrate-to-pnpm

Conversation

Copilot AI commented May 29, 2026

Copy link
Copy Markdown

After switching to pnpm, the playground build started failing in Vite/Rolldown on unresolved imports (e.g. @fluentui/react-component-ref) that were previously masked by hoisting. This PR aligns package boundaries with pnpm’s stricter resolution and updates playground module resolution behavior accordingly.

  • Root cause surfaced by pnpm strictness

    • Workspace packages and playground code were relying on transitive/hoisted dependencies not declared where imported.
    • preserveSymlinks in playground Vite config amplified unresolved imports across linked workspace packages.
  • Dependency declaration fixes (make imports explicit)

    • packages/semantic-ui/package.json: add @fluentui/react-component-ref
    • packages/chakra-ui/package.json: add @ark-ui/react, @zag-js/react
    • packages/fluentui-rc/package.json: add @griffel/react
    • packages/playground/package.json: add @ant-design/cssinjs, primereact
  • Playground resolver adjustment

    • packages/playground/vite.config.ts: switch resolve.preserveSymlinks from true to false so Vite resolves dependencies via real pnpm install topology instead of preserving symlink boundaries.
  • Lockfile update

    • Regenerated pnpm-lock.yaml to reflect explicit dependency ownership and version graph.
// packages/playground/vite.config.ts
resolve: {
  preserveSymlinks: false,
  alias: {
    '@rjsf/semantic-ui': path.resolve(__dirname, '../semantic-ui/src'),
    '@rjsf/chakra-ui': path.resolve(__dirname, '../chakra-ui/src'),
    // ...
  },
}

Copilot AI changed the title Migrate monorepo package management and automation from npm to pnpm Ensure pnpm is explicitly installed in GitHub Actions before workflow execution May 29, 2026
Copilot AI requested a review from nickgros May 29, 2026 21:08
Copilot AI changed the title Ensure pnpm is explicitly installed in GitHub Actions before workflow execution Downgrade CI/workspace to pnpm 10 and fix strict dependency breakages under Node 20 May 30, 2026
Copilot AI changed the title Downgrade CI/workspace to pnpm 10 and fix strict dependency breakages under Node 20 Unify pnpm version source in CI and release workflows May 30, 2026
Copilot AI changed the title Unify pnpm version source in CI and release workflows Fix pnpm-induced playground build failures by correcting dependency resolution boundaries May 30, 2026
@nickgros nickgros changed the title Fix pnpm-induced playground build failures by correcting dependency resolution boundaries Replace npm with pnpm Jun 5, 2026
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