Skip to content

chore(deps): Bump prettier-plugin-apex#43

Merged
keelerm84 merged 2 commits intomainfrom
bump-prettier
Apr 10, 2026
Merged

chore(deps): Bump prettier-plugin-apex#43
keelerm84 merged 2 commits intomainfrom
bump-prettier

Conversation

@keelerm84
Copy link
Copy Markdown
Member

@keelerm84 keelerm84 commented Apr 10, 2026

Note

Medium Risk
Primarily dev-tooling upgrades, but the updated dependency tree (notably joi) now requires Node >=20, which could break installs/CI environments on older Node versions.

Overview
Renames the npm package to launchdarkly-apex-server-sdk, marks it private, and bumps dev formatting tooling (prettier to 3.8.2, prettier-plugin-apex to 2.2.6).

Regenerates package-lock.json, pulling in new/updated transitive deps (including platform-specific optional @prettier-apex serializer binaries) and updating engine constraints (e.g., joi now requires Node >=20).

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

@keelerm84 keelerm84 requested a review from a team as a code owner April 10, 2026 11:22
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 3eb94b9. Configure here.

Comment thread package-lock.json
"integrity": "sha512-8c3mgTe0ASwWAJK+78dpviD+A8EqhndQPUBpNUIPt6+xWlIigCwfN01lWr9MAede4uqXGTEKeQWTvzb3vjia0Q==",
"dev": true,
"license": "MIT",
"peer": true,
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Lockfile incorrectly marks prettier as peer-only dependency

Medium Severity

The prettier entry in package-lock.json has a new "peer": true flag, but prettier is a direct devDependency in package.json. This flag indicates prettier exists only as a peer dependency of prettier-plugin-apex, which is incorrect. This lockfile inconsistency could cause prettier to be skipped during npm ci --omit=peer or pruned unexpectedly, breaking the prettier:format and prettier:check:ci scripts. The lockfile likely needs to be regenerated cleanly.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 3eb94b9. Configure here.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I cleared and regenerated this lock file and it was exactly the same. I think this is a false positive.

Comment thread package-lock.json
@keelerm84 keelerm84 merged commit fd122ea into main Apr 10, 2026
6 checks passed
@keelerm84 keelerm84 deleted the bump-prettier branch April 10, 2026 16:52
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