Skip to content

fix(deps): remove unused @lwrjs/api and move lwc to devDependencies @W-24460137@ - #691

Merged
rax-it merged 2 commits into
mainfrom
rv/W-24460137-remove-unused-lwr
Oct 9, 2026
Merged

rax-it merged 2 commits into
mainfrom
rv/W-24460137-remove-unused-lwr

Conversation

@rax-it

@rax-it rax-it commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor

What does this PR do?

Shrinks installed plugin size by ~208 MB (−23%).

  • Removes @lwrjs/api — no remaining imports since its last usage was removed in W-21188143: chore: remove --ssr, --get-latest, and --guest flags from site command #635. Not part of the plugin's exports, so no consumer can depend on it through us.
  • Moves lwc to devDependencies — it was only in dependencies as a peer of @lwrjs/api. Runtime compilation goes through the bundled @lwc/sfdx-local-dev-dist-* aliases; lwc is still needed for tests.
  • Removes the stale link-lwr / unlink-lwr scripts and the matching README linking steps.
  • yarn.lock diff is removal-only.

Measured (npm install --omit dev from a registry, shrinkwrap honored, i.e. what sf plugins install does):

Before (published 6.2.19) After (this branch)
Installed node_modules 886 MB 678 MB
Installed packages 1,796 1,312
Tarball 510 KB 392 KB

"After" was published to a local Verdaccio registry. Installing from a local file: tarball ignores the bundled shrinkwrap, so it under-reports size.

The savings are one-time, not per API version. 66.0, 67.0 and 68.0 already share the LWR stack, and each version adds ~55–65 MB.

Verification

  • yarn build, yarn test: pass.
  • LWC dev server websocket compile smoke on 66.0 / 67.0 / 68.0 × both test projects: 6/6 pass, with byte-identical output before vs after.
  • lightning dev component|app|site: commands load, --help OK.
  • NUTs: run in CI only (they need Dev Hub credentials).

What issues does this PR fix or reference?

@W-24460137@

🤖 Generated with Claude Code

…W-24460137@

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@rax-it
rax-it requested review from a team as code owners October 8, 2026 23:01
…4460137@

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@rax-it
rax-it merged commit eaf5483 into main Oct 9, 2026
20 checks passed
@rax-it
rax-it deleted the rv/W-24460137-remove-unused-lwr branch October 9, 2026 16:49
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