Skip to content

fix(ci): pin Rollup to 4.62.4 - #398

Merged
SonyLeo merged 1 commit into
opentiny:developfrom
gene9831:codex/pin-rollup-4.62.4
Aug 27, 2026
Merged

fix(ci): pin Rollup to 4.62.4#398
SonyLeo merged 1 commit into
opentiny:developfrom
gene9831:codex/pin-rollup-4.62.4

Conversation

@gene9831

@gene9831 gene9831 commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Pin Rollup to 4.62.4 through a pnpm workspace override.
  • Keep the existing Node 24 runtime and the 4 GB VitePress heap limit unchanged.
  • Use this as a narrow, temporary mitigation for the recent CI out-of-memory failure.

Why

CI currently runs pnpm i --no-frozen-lockfile, while pnpm-lock.yaml is ignored. A fresh install now resolves Rollup 4.63.0, so transitive dependencies can change without any repository diff.

The recent upstream failure occurred after this dependency drift. In an A/B verification, changing only Rollup from the current 4.63.0 resolution to 4.62.4 made the complete workflow pass. Increasing the V8 heap to 8 GB and switching between Node 24 and Node 22 did not resolve the same failure.

Verification

  • Dispatch Publish with Rollup 4.62.4 passed.
  • Component build passed.
  • Playground production build passed in 23.65 seconds.
  • VitePress build passed with the existing --max-old-space-size=4096 setting in 67.62 seconds.
  • Build output verification, artifact upload, and docs deployment passed.
  • A fresh pnpm 10.34.5 lockfile-only resolution contained Rollup 4.62.4 and no Rollup 4.63.0 entry.

Follow-up

This override is intentionally temporary. The durable fix is to track pnpm-lock.yaml and use a frozen install in CI so transitive dependency updates are explicit and reproducible. PR #396 contains a broader lockfile-based approach.

Summary by CodeRabbit

  • Chores
    • Updated the application’s build tooling configuration to use a fixed Rollup version for more consistent builds.

@coderabbitai

coderabbitai Bot commented Aug 27, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 00071252-89e5-4051-96f6-26986e91747c

📥 Commits

Reviewing files that changed from the base of the PR and between 9aefb35 and 6a30cac.

📒 Files selected for processing (1)
  • pnpm-workspace.yaml

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


Walkthrough

The workspace configuration now overrides rollup to version 4.62.4.

Changes

Rollup dependency pin

Layer / File(s) Summary
Workspace dependency override
pnpm-workspace.yaml
Adds an overrides entry that pins rollup to version 4.62.4.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to 6a30c

This narrowly pins Rollup to 4.62.4 to stabilize CI dependency resolution, with the existing runtime and memory settings unchanged; no actionable merge-blocking risk remains after normal checks and review.

Poem

A rabbit checked the workspace with care
Rollup now rests at four-six-two-four there
The override sits neat and small
One version governs them all
Then the rabbit hops past the YAML lair

🚥 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 title clearly and concisely describes the main change: pinning Rollup to version 4.62.4 for CI.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@github-actions

github-actions Bot commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

✅ Preview build completed successfully!

Click the image above to preview.
Preview will be automatically removed when this PR is closed.

@github-actions

Copy link
Copy Markdown
Contributor

@SonyLeo
SonyLeo merged commit 21bf154 into opentiny:develop Aug 27, 2026
4 checks passed
@github-actions

Copy link
Copy Markdown
Contributor

🧹 Preview Cleaned Up

The preview deployment has been removed.

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.

3 participants