Skip to content

ci: use actions/setup-node - #747

Merged
ocavue merged 2 commits into
masterfrom
ci/restore-pnpm-action-setup
Oct 28, 2026
Merged

ocavue merged 2 commits into
masterfrom
ci/restore-pnpm-action-setup

Conversation

@ocavue

@ocavue ocavue commented Oct 28, 2026 •

Copy link
Copy Markdown
Member

Replace pnpm/setup with pnpm/action-setup + actions/setup-node in the shared setup action, since pnpm/setup has been causing CI failures. This reverts the approach from #538, using the latest pinned versions (pnpm/action-setup v6.1.0, actions/setup-node v7.1.0).

Summary by CodeRabbit

  • Chores
    • Updated automated build setup to configure the Node.js environment and package installation in separate steps. The setup now uses the configured Node.js version, enables package caching, and installs packages through Bash. These changes improve consistency in automated workflows and do not introduce changes to user-facing features.

@vercel

vercel Bot commented Oct 28, 2026 •

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated
meowdown Ignored Ignored Oct 28, 2026 12:20pm UTC

@coderabbitai

coderabbitai Bot commented Oct 28, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 52 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 41bc2b9a-83aa-466d-af1b-4a87d0b4c4f7
Commits

Reviewing files that changed from the base of the PR and between 3bfbaa4 and f04dd89.

Files selected for processing: 1
  • .github/actions/setup/action.yml

No actionable comments were generated in the recent review.

Recent review info
Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: f9be92eb-ba22-4774-906b-7b961f821758
Commits

Reviewing files that changed from the base of the PR and between e63ad78 and 3bfbaa4.

Files selected for processing: 1
  • .github/actions/setup/action.yml

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

Walkthrough

The composite setup action replaces the combined pnpm and Node setup step with separate pinned setup actions. It configures Node caching and the npm registry URL, then runs pnpm install.

Changes: Composite Setup Action

Layer / File(s) Summary
Configure tools and install dependencies
.github/actions/setup/action.yml
The action uses separate pinned pnpm and Node setup steps. Node setup uses the configured version, enables pnpm caching, and sets the npm registry URL. A Bash step runs pnpm install.

Priority: Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Bug fix

Possibly related PRs

  • prosekit/meowdown#733: This PR updates the same setup action. The current change replaces the pnpm/setup implementation updated by that PR.
Merge Risk: Minimal · previous review 3bfba

The setup action resolves pnpm 11.28.6 before enabling Node caching and installs dependencies from the repository root.

Pre-merge checks: Passed 4
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title accurately identifies the main CI change: replacing the previous setup with actions/setup-node. It is concise and specific.
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.
Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

Copy link
Copy Markdown

@ocavue ocavue changed the title ci: migrate from pnpm/setup back to pnpm/action-setup and actions/setup-node ci: use actions/setup-node Oct 28, 2026
@pkg-pr-new

pkg-pr-new Bot commented Oct 28, 2026 •

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/@meowdown/core@747
npm i https://pkg.pr.new/@meowdown/embed@747
npm i https://pkg.pr.new/@meowdown/markdown@747
npm i https://pkg.pr.new/@meowdown/react@747

commit: f04dd89

@github-actions

github-actions Bot commented Oct 28, 2026 •

Copy link
Copy Markdown

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 93.36% 6573 / 7040
🔵 Statements 92.1% 8047 / 8737
🔵 Functions 89.61% 1450 / 1618
🔵 Branches 88.29% 5425 / 6144
File CoverageNo changed files found.
Generated in workflow #2958 for commit f04dd89 by the Vitest Coverage Report Action

@vercel

vercel Bot commented Oct 28, 2026

Copy link
Copy Markdown

Deployment failed for project meowdown with the following error:

Resource is limited - try again in 24 hours (more than 100, code: "api-deployments-free-per-day").

Learn More: https://vercel.com/ocavues-projects?upgradeToPro=build-rate-limit

@ocavue
ocavue merged commit ddd007f into master Oct 28, 2026
15 checks passed
@ocavue
ocavue deleted the ci/restore-pnpm-action-setup branch October 28, 2026 12:44
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.

1 participant