Skip to content

fix: bump synckit for yarn PnP ESM issue#776

Merged
JounQin merged 2 commits intoprettier:mainfrom
aswils:fix/restore-diagnostics-w-yarn
Jan 14, 2026
Merged

fix: bump synckit for yarn PnP ESM issue#776
JounQin merged 2 commits intoprettier:mainfrom
aswils:fix/restore-diagnostics-w-yarn

Conversation

@aswils
Copy link
Contributor

@aswils aswils commented Jan 14, 2026

Synckit was unable to properly load es modules, and would instead hang when eslint attempted to generate diagnostics. A patch was recently introduced upstream 1 so we update to bring in the fix.

fix #775


Important

Update synckit to ^0.11.12 in package.json to fix ESLint diagnostics hanging issue with Yarn PnP.

  • Dependencies:
    • Update synckit version from ^0.11.7 to ^0.11.12 in package.json to fix ES module loading issue causing ESLint to hang.
  • Issue:

This description was created by Ellipsis for 3317a9d. You can customize this summary. It will automatically update as commits are pushed.

Summary by CodeRabbit

  • Chores

    • Updated an internal dependency ("synckit") to a newer patch version to maintain stability.
  • Documentation

    • Added a changeset documenting a patch release and noting the dependency update.

✏️ Tip: You can customize this high-level summary in your review settings.

@changeset-bot
Copy link

changeset-bot bot commented Jan 14, 2026

🦋 Changeset detected

Latest commit: 3cd0048

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

This PR includes changesets to release 1 package
Name Type
eslint-plugin-prettier 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

@coderabbitai
Copy link

coderabbitai bot commented Jan 14, 2026

📝 Walkthrough

Walkthrough

Updates dependency synckit in package.json from ^0.11.7 to ^0.11.12 and adds a changeset file documenting the related bump.

Changes

Cohort / File(s) Change Summary
Dependency update
package.json
synckit bumped from ^0.11.7 to ^0.11.12 under dependencies
Changeset
.changeset/cool-taxes-grin.md
Adds changeset noting the synckit bump and a patch for eslint-plugin-prettier

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested labels

bug, dependencies

Poem

🐰 I hopped through package.json with care,
Bumped synckit tiny, tidy and fair.
A changeset scribbled, a carrot in hand,
Patching the path to make builds stand.
Hooray for clean deps across the land! 🥕

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The PR title clearly and accurately summarizes the main change: bumping the synckit dependency to fix a Yarn PnP ESM issue affecting ESLint diagnostics.
Linked Issues check ✅ Passed The PR directly addresses issue #775 by updating synckit to ^0.11.12, which includes the upstream patch that fixes the ES module loading issue causing diagnostics to hang in Yarn PnP environments.
Out of Scope Changes check ✅ Passed All changes are focused on updating the synckit dependency version and documenting the fix in the changeset file, directly related to the stated objective of resolving the Yarn PnP ESM issue.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.



📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 3317a9d and 3cd0048.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (2)
  • .changeset/cool-taxes-grin.md
  • package.json
🔇 Additional comments (1)
.changeset/cool-taxes-grin.md (1)

1-5: LGTM!

The changeset correctly documents this as a patch release with an accurate description of the fix. The format follows changesets conventions properly.

✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

Important

Looks good to me! 👍

Reviewed everything up to 3317a9d in 22 seconds. Click for details.
  • Reviewed 13 lines of code in 1 files
  • Skipped 1 files when reviewing.
  • Skipped posting 1 draft comments. View those below.
  • Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. package.json:67
  • Draft comment:
    Bump synckit to ^0.11.12 per upstream patch for yarn PnP issues. Confirm tests cover this change.
  • Reason this comment was not posted:
    Confidence changes required: 0% <= threshold 50% None

Workflow ID: wflow_LtuW2pHcgW21ldm6

You can customize Ellipsis by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In `@package.json`:
- Line 67: Update the package.json dependency "synckit" to the patched version
^0.11.12 to address CVE-2025-54313 and Yarn PnP ESM handling; after changing the
dependency, run your package manager install and test to ensure no breakages. If
your repo uses Yarn PnP and you encounter ESM issues, enable the experimental
ESM loader by adding pnpEnableEsmLoader: true in your .yarnrc.yml, then
reinstall and re-run CI; finally, re-run a security audit (e.g., npm audit or
yarn audit) to confirm the vulnerability is resolved.
📜 Review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 7264ed0 and 3317a9d.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (1)
  • package.json

✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.

Synckit was unable to properly load es modules, and would instead hang
when eslint attempted to generate diagnostics. A patch was recently
introduced upstream [1] so we update to bring in the fix.

[1]: un-ts/synckit@4fb10bd

Issue: 775
@aswils aswils force-pushed the fix/restore-diagnostics-w-yarn branch from 3317a9d to 0a54bec Compare January 14, 2026 09:16
@JounQin JounQin changed the title fix: restore diagnostics with yarn pnp fix: bump synckit for yarn PnP ESM issue Jan 14, 2026
@JounQin JounQin requested a review from Copilot January 14, 2026 09:21
@pkg-pr-new
Copy link

pkg-pr-new bot commented Jan 14, 2026

Open in StackBlitz

npm i https://pkg.pr.new/eslint-plugin-prettier@776

commit: 3cd0048

@JounQin JounQin merged commit 77651a3 into prettier:main Jan 14, 2026
16 checks passed
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the synckit dependency from ^0.11.7 to ^0.11.12 to resolve an issue where ESLint diagnostics would hang when using Yarn Plug'n'Play (PnP) with ES modules. The fix brings in an upstream patch that properly handles ES module loading.

Changes:

  • Update synckit dependency version to include upstream fix for Yarn PnP ESM loading
  • Add changeset documenting the patch release

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File Description
package.json Updates synckit dependency from ^0.11.7 to ^0.11.12
pnpm-lock.yaml Updates lock file with new synckit version and its transitive dependency @pkgr/core
.changeset/cool-taxes-grin.md Documents the patch release with description of the fix
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

renovate bot added a commit to andrei-picus-tink/auto-renovate that referenced this pull request Jan 15, 2026
| datasource | package                | from  | to    |
| ---------- | ---------------------- | ----- | ----- |
| npm        | eslint-plugin-prettier | 5.5.4 | 5.5.5 |


## [v5.5.5](https://github.com/prettier/eslint-plugin-prettier/blob/HEAD/CHANGELOG.md#555)

##### Patch Changes

- [#772](prettier/eslint-plugin-prettier#772) [`7264ed0`](prettier/eslint-plugin-prettier@7264ed0) Thanks [@BPScott](https://github.com/BPScott)! - Bump prettier-linter-helpers dependency to v1.0.1

- [#776](prettier/eslint-plugin-prettier#776) [`77651a3`](prettier/eslint-plugin-prettier@77651a3) Thanks [@aswils](https://github.com/aswils)! - fix: bump synckit for yarn PnP ESM issue
renovate bot added a commit to andrei-picus-tink/auto-renovate that referenced this pull request Jan 17, 2026
| datasource | package                | from  | to    |
| ---------- | ---------------------- | ----- | ----- |
| npm        | eslint-plugin-prettier | 5.5.4 | 5.5.5 |


## [v5.5.5](https://github.com/prettier/eslint-plugin-prettier/blob/HEAD/CHANGELOG.md#555)

##### Patch Changes

- [#772](prettier/eslint-plugin-prettier#772) [`7264ed0`](prettier/eslint-plugin-prettier@7264ed0) Thanks [@BPScott](https://github.com/BPScott)! - Bump prettier-linter-helpers dependency to v1.0.1

- [#776](prettier/eslint-plugin-prettier#776) [`77651a3`](prettier/eslint-plugin-prettier@77651a3) Thanks [@aswils](https://github.com/aswils)! - fix: bump synckit for yarn PnP ESM issue
renovate bot added a commit to andrei-picus-tink/auto-renovate that referenced this pull request Jan 18, 2026
| datasource | package                | from  | to    |
| ---------- | ---------------------- | ----- | ----- |
| npm        | eslint-plugin-prettier | 5.5.4 | 5.5.5 |


## [v5.5.5](https://github.com/prettier/eslint-plugin-prettier/blob/HEAD/CHANGELOG.md#555)

##### Patch Changes

- [#772](prettier/eslint-plugin-prettier#772) [`7264ed0`](prettier/eslint-plugin-prettier@7264ed0) Thanks [@BPScott](https://github.com/BPScott)! - Bump prettier-linter-helpers dependency to v1.0.1

- [#776](prettier/eslint-plugin-prettier#776) [`77651a3`](prettier/eslint-plugin-prettier@77651a3) Thanks [@aswils](https://github.com/aswils)! - fix: bump synckit for yarn PnP ESM issue
736-c41-2c1-e464fc974 pushed a commit to Swiss-Armed-Forces/Loom that referenced this pull request Jan 18, 2026
This MR contains the following updates:

| Package | Type | Update | Change | OpenSSF |
|---|---|---|---|---|
| [@types/node](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/master/types/node) ([source](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/node)) | devDependencies | patch | [`24.10.7` → `24.10.9`](https://renovatebot.com/diffs/npm/@types%2fnode/24.10.7/24.10.9) | [![OpenSSF Scorecard](https://api.securityscorecards.dev/projects/github.com/DefinitelyTyped/DefinitelyTyped/badge)](https://securityscorecards.dev/viewer/?uri=github.com/DefinitelyTyped/DefinitelyTyped) |
| [eslint-plugin-prettier](https://github.com/prettier/eslint-plugin-prettier) | devDependencies | patch | [`5.5.4` → `5.5.5`](https://renovatebot.com/diffs/npm/eslint-plugin-prettier/5.5.4/5.5.5) | [![OpenSSF Scorecard](https://api.securityscorecards.dev/projects/github.com/prettier/eslint-plugin-prettier/badge)](https://securityscorecards.dev/viewer/?uri=github.com/prettier/eslint-plugin-prettier) |

---

### Release Notes

<details>
<summary>prettier/eslint-plugin-prettier (eslint-plugin-prettier)</summary>

### [`v5.5.5`](https://github.com/prettier/eslint-plugin-prettier/blob/HEAD/CHANGELOG.md#555)

[Compare Source](prettier/eslint-plugin-prettier@v5.5.4...v5.5.5)

##### Patch Changes

- [#&#8203;772](prettier/eslint-plugin-prettier#772) [`7264ed0`](prettier/eslint-plugin-prettier@7264ed0) Thanks [@&#8203;BPScott](https://github.com/BPScott)! - Bump prettier-linter-helpers dependency to v1.0.1

- [#&#8203;776](prettier/eslint-plugin-prettier#776) [`77651a3`](prettier/eslint-plugin-prettier@77651a3) Thanks [@&#8203;aswils](https://github.com/aswils)! - fix: bump synckit for yarn PnP ESM issue

</details>

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this MR, check this box

---

This MR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0Mi44NC4yIiwidXBkYXRlZEluVmVyIjoiNDIuODQuMiIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOlsiZGVwZW5kZW5jaWVzIiwicmVub3ZhdGUiXX0=-->

See merge request swiss-armed-forces/cyber-command/cea/loom!252

Co-authored-by: Loom MR Pipeline Trigger <group_103951964_bot_9504bb8dead6d4e406ad817a607f24be@noreply.gitlab.com>
736-c41-2c1-e464fc974 added a commit to Swiss-Armed-Forces/Loom that referenced this pull request Jan 18, 2026
chore(deps): update frontend dependencies (patch) (patch)

This MR contains the following updates:

| Package | Type | Update | Change | OpenSSF |
|---|---|---|---|---|
| [@types/node](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/master/types/node) ([source](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/node)) | devDependencies | patch | [`24.10.7` → `24.10.9`](https://renovatebot.com/diffs/npm/@types%2fnode/24.10.7/24.10.9) | [![OpenSSF Scorecard](https://api.securityscorecards.dev/projects/github.com/DefinitelyTyped/DefinitelyTyped/badge)](https://securityscorecards.dev/viewer/?uri=github.com/DefinitelyTyped/DefinitelyTyped) |
| [eslint-plugin-prettier](https://github.com/prettier/eslint-plugin-prettier) | devDependencies | patch | [`5.5.4` → `5.5.5`](https://renovatebot.com/diffs/npm/eslint-plugin-prettier/5.5.4/5.5.5) | [![OpenSSF Scorecard](https://api.securityscorecards.dev/projects/github.com/prettier/eslint-plugin-prettier/badge)](https://securityscorecards.dev/viewer/?uri=github.com/prettier/eslint-plugin-prettier) |

---

### Release Notes

<details>
<summary>prettier/eslint-plugin-prettier (eslint-plugin-prettier)</summary>

### [`v5.5.5`](https://github.com/prettier/eslint-plugin-prettier/blob/HEAD/CHANGELOG.md#555)

[Compare Source](prettier/eslint-plugin-prettier@v5.5.4...v5.5.5)

##### Patch Changes

- [#&#8203;772](prettier/eslint-plugin-prettier#772) [`7264ed0`](prettier/eslint-plugin-prettier@7264ed0) Thanks [@&#8203;BPScott](https://github.com/BPScott)! - Bump prettier-linter-helpers dependency to v1.0.1

- [#&#8203;776](prettier/eslint-plugin-prettier#776) [`77651a3`](prettier/eslint-plugin-prettier@77651a3) Thanks [@&#8203;aswils](https://github.com/aswils)! - fix: bump synckit for yarn PnP ESM issue

</details>

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this MR, check this box

---

This MR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0Mi44NC4yIiwidXBkYXRlZEluVmVyIjoiNDIuODQuMiIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOlsiZGVwZW5kZW5jaWVzIiwicmVub3ZhdGUiXX0=-->

See merge request swiss-armed-forces/cyber-command/cea/loom!252

Co-authored-by: Loom MR Pipeline Trigger <group_103951964_bot_9504bb8dead6d4e406ad817a607f24be@noreply.gitlab.com>
Co-authored-by: open-source Pipeline <group_90701827_bot_ed04ae348bc5f40af9966fb8b6867e99@noreply.gitlab.com>
renovate bot added a commit to andrei-picus-tink/auto-renovate that referenced this pull request Jan 20, 2026
| datasource | package                | from  | to    |
| ---------- | ---------------------- | ----- | ----- |
| npm        | eslint-plugin-prettier | 5.5.4 | 5.5.5 |


## [v5.5.5](https://github.com/prettier/eslint-plugin-prettier/blob/HEAD/CHANGELOG.md#555)

##### Patch Changes

- [#772](prettier/eslint-plugin-prettier#772) [`7264ed0`](prettier/eslint-plugin-prettier@7264ed0) Thanks [@BPScott](https://github.com/BPScott)! - Bump prettier-linter-helpers dependency to v1.0.1

- [#776](prettier/eslint-plugin-prettier#776) [`77651a3`](prettier/eslint-plugin-prettier@77651a3) Thanks [@aswils](https://github.com/aswils)! - fix: bump synckit for yarn PnP ESM issue
renovate bot added a commit to andrei-picus-tink/auto-renovate that referenced this pull request Jan 20, 2026
| datasource | package                | from  | to    |
| ---------- | ---------------------- | ----- | ----- |
| npm        | eslint-plugin-prettier | 5.5.4 | 5.5.5 |


## [v5.5.5](https://github.com/prettier/eslint-plugin-prettier/blob/HEAD/CHANGELOG.md#555)

##### Patch Changes

- [#772](prettier/eslint-plugin-prettier#772) [`7264ed0`](prettier/eslint-plugin-prettier@7264ed0) Thanks [@BPScott](https://github.com/BPScott)! - Bump prettier-linter-helpers dependency to v1.0.1

- [#776](prettier/eslint-plugin-prettier#776) [`77651a3`](prettier/eslint-plugin-prettier@77651a3) Thanks [@aswils](https://github.com/aswils)! - fix: bump synckit for yarn PnP ESM issue
renovate bot added a commit to andrei-picus-tink/auto-renovate that referenced this pull request Jan 21, 2026
| datasource | package                | from  | to    |
| ---------- | ---------------------- | ----- | ----- |
| npm        | eslint-plugin-prettier | 5.5.4 | 5.5.5 |


## [v5.5.5](https://github.com/prettier/eslint-plugin-prettier/blob/HEAD/CHANGELOG.md#555)

##### Patch Changes

- [#772](prettier/eslint-plugin-prettier#772) [`7264ed0`](prettier/eslint-plugin-prettier@7264ed0) Thanks [@BPScott](https://github.com/BPScott)! - Bump prettier-linter-helpers dependency to v1.0.1

- [#776](prettier/eslint-plugin-prettier#776) [`77651a3`](prettier/eslint-plugin-prettier@77651a3) Thanks [@aswils](https://github.com/aswils)! - fix: bump synckit for yarn PnP ESM issue
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.

No diagnostics with modern yarn

3 participants