Skip to content

fix(profile-sync-controller): remove leftover cjs pointers - #10218

Open
mathieuartu wants to merge 3 commits into
mainfrom
fix/psc-subpath-esm-mains
Open

fix(profile-sync-controller): remove leftover cjs pointers#10218
mathieuartu wants to merge 3 commits into
mainfrom
fix/psc-subpath-esm-mains

Conversation

@mathieuartu

@mathieuartu mathieuartu commented Sep 13, 2026

Copy link
Copy Markdown
Contributor

Explanation

References

Checklist

  • I've updated the test suite for new or updated code as appropriate
  • I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
  • I've communicated my changes to consumers by updating changelogs for packages I've changed
  • I've introduced breaking changes in this PR and have prepared draft pull requests for clients and consumer packages to resolve them

Note

Low Risk
Packaging-only path updates with no runtime logic changes; aligns subpath resolution with the existing ESM-only build.

Overview
After the package went ESM-only, several nested package.json files under sdk, auth, and user-storage (including their mocks folders) still pointed main and types at .cjs / .d.cts artifacts.

This PR retargets those entries to .js / .d.ts so tooling that resolves through those subpaths (e.g. Jest’s module map) loads the actual ESM output instead of stale CommonJS paths. The [Unreleased] changelog adds a Fixed note for the change.

Reviewed by Cursor Bugbot for commit 3834dbb. Bugbot is set up for automated code reviews on this repo. Configure here.

@mathieuartu mathieuartu self-assigned this Sep 13, 2026
@mathieuartu
mathieuartu requested review from a team as code owners September 13, 2026 18:39
@mathieuartu
mathieuartu deployed to default-branch September 13, 2026 18:39 — with GitHub Actions Active
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant