Skip to content

fix(ci): force npm in compressed-size action - #726

Merged
LucasSantana-Dev merged 1 commit into
mainfrom
fix/bundle-size-use-npm
Apr 19, 2026
Merged

LucasSantana-Dev merged 1 commit into
mainfrom
fix/bundle-size-use-npm

Conversation

@LucasSantana-Dev

Copy link
Copy Markdown
Owner

Summary

  • The bundle-size workflow's compressed-size step was failing on every PR with `Unable to locate executable file: pnpm`.
  • Root cause: `preactjs/compressed-size-action` autodetects pnpm when both `package-lock.json` and `pnpm-lock.yaml` exist. This project is npm-managed (all `package.json` scripts use npm), but a stale `pnpm-lock.yaml` triggered the wrong path.
  • Fix: pass `install-script: 'npm ci'` to force npm. Single-line change to `.github/workflows/bundle-size.yml`.

This unblocks dependency-bump PRs (#723 prod-deps, #705 dev-deps) that were red on this check despite having no functional regression.

Test plan

The preactjs/compressed-size-action defaults to pnpm when both
package-lock.json and pnpm-lock.yaml are present. This project is npm-
managed (all package.json scripts use npm), but a stale pnpm-lock.yaml
was making the action try `pnpm install --frozen-lockfile`, which fails
on the runner with "Unable to locate executable file: pnpm".

Pin install-script: 'npm ci' to override the autodetection. Unblocks
dependency PRs (#723, #705) that were failing this check despite having
no functional regression.
@vercel

vercel Bot commented Apr 19, 2026 •

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
lucky Ready Ready Preview, Comment Apr 19, 2026 5:57pm

Request Review

@coderabbitai

coderabbitai Bot commented Apr 19, 2026

Copy link
Copy Markdown

Warning

Rate limit exceeded

@LucasSantana-Dev has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 39 minutes and 33 seconds before requesting another review.

Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 39 minutes and 33 seconds.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 60369f2f-e08e-4d2c-a355-79ccc3c3131d

📥 Commits

Reviewing files that changed from the base of the PR and between 587056a and 8b84f5d.

📒 Files selected for processing (1)
  • .github/workflows/bundle-size.yml
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/bundle-size-use-npm

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.

@sonarqubecloud

Copy link
Copy Markdown

@LucasSantana-Dev
LucasSantana-Dev merged commit a89593e into main Apr 19, 2026
12 checks passed
LucasSantana-Dev added a commit that referenced this pull request Apr 20, 2026
Five commits landed on main since 2.6.147 but Unreleased was empty.
Harvesting them now so the next version bump captures the right notes
and users scanning the changelog see what shipped.

Covered: #726 (CI pnpm→npm), #731 (/social), #733 (/leaderboard),
#738 (birthday schema + /birthday set|clear), #741 (cached-guild log
level).

Queued PRs not yet on main (top.gg webhook stack, Phase 0 docs,
dashboard vote badge, birthday scheduler + channel + list + role) will
be added when they actually merge.
LucasSantana-Dev added a commit that referenced this pull request Apr 20, 2026
Five commits landed on main since 2.6.147 but Unreleased was empty.
Harvesting them now so the next version bump captures the right notes
and users scanning the changelog see what shipped.

Covered: #726 (CI pnpm→npm), #731 (/social), #733 (/leaderboard),
#738 (birthday schema + /birthday set|clear), #741 (cached-guild log
level).

Queued PRs not yet on main (top.gg webhook stack, Phase 0 docs,
dashboard vote badge, birthday scheduler + channel + list + role) will
be added when they actually merge.
LucasSantana-Dev added a commit that referenced this pull request May 13, 2026
The preactjs/compressed-size-action defaults to pnpm when both
package-lock.json and pnpm-lock.yaml are present. This project is npm-
managed (all package.json scripts use npm), but a stale pnpm-lock.yaml
was making the action try `pnpm install --frozen-lockfile`, which fails
on the runner with "Unable to locate executable file: pnpm".

Pin install-script: 'npm ci' to override the autodetection. Unblocks
dependency PRs (#723, #705) that were failing this check despite having
no functional regression.
LucasSantana-Dev added a commit that referenced this pull request May 13, 2026
Five commits landed on main since 2.6.147 but Unreleased was empty.
Harvesting them now so the next version bump captures the right notes
and users scanning the changelog see what shipped.

Covered: #726 (CI pnpm→npm), #731 (/social), #733 (/leaderboard),
#738 (birthday schema + /birthday set|clear), #741 (cached-guild log
level).

Queued PRs not yet on main (top.gg webhook stack, Phase 0 docs,
dashboard vote badge, birthday scheduler + channel + list + role) will
be added when they actually merge.
@LucasSantana-Dev
LucasSantana-Dev deleted the fix/bundle-size-use-npm branch May 23, 2026 02:21

This branch was successfully deployed

1 active deployment
Preview — 8b84f5d1 Deployed Apr 19, 2026 by vercel[bot]
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