fix: resolve failed CI runs (release-please manifest + docs deploy CLI)#121
Conversation
The manifest tracked 0.2.0 while package.json and the CHANGELOG top entry are both 0.4.0 (no git tags exist), causing release-please to compute a bogus next version (0.4.0 -> 0.2.1). Align the manifest with the actual released version so release-please bases the next release on 0.4.0. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01XJPAa4e88wbH9CHw8LZ1Tm
WalkthroughUpdated the release manifest root component version from 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
amondnet/vercel-action@v25 pins Vercel CLI 25.1.0, which Vercel's API now rejects with "Your Vercel CLI version is outdated. This endpoint requires version 47.2.2 or later." Pass vercel-version: latest so the action installs a current CLI. This was the deploy step's failure once the VERCEL_* secrets were configured. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01XJPAa4e88wbH9CHw8LZ1Tm
The Vercel project's Root Directory is set to `website`. The deploy step also set working-directory: website, so the CLI ran inside website/ and Vercel resolved the root as website/website, failing with "The provided path .../website/website does not exist." Drop working-directory so the CLI runs from the repo root and Vercel's own root-directory setting applies once. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01XJPAa4e88wbH9CHw8LZ1Tm
There was a problem hiding this comment.
🧹 Nitpick comments (2)
.github/workflows/deploy-website.yml (2)
50-50: 🩺 Stability & Availability | 🔵 Trivial | ⚡ Quick winPin
vercel-versioninstead of usinglatest.The fix is well-reasoned and documented, but
vercel-version: latestis a floating target:feat: add vercel-version option by@michalak111in#183`` shows this input exists specifically so a fixed CLI version can be selected. Since the underlying motivation for this change was Vercel's API rejecting an old pinned CLI without warning, relying onlatestreintroduces the same class of risk in the opposite direction — a future Vercel CLI release could silently break `vercel-args: '--prod'` behavior or introduce required new flags with no corresponding change in this repo.Consider pinning to a known-good, currently supported CLI version and bumping it deliberately (e.g., via Renovate/Dependabot) rather than tracking
latest.♻️ Suggested pin
- vercel-version: latest + vercel-version: 47.2.2Also applies to: 55-57
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In @.github/workflows/deploy-website.yml at line 50, Update the Vercel deployment steps using amondnet/vercel-action@v25 to set vercel-version to a specific known-good supported CLI version instead of latest. Apply the same fixed version consistently to all occurrences, including the additional deployment step, and leave the existing vercel-args configuration unchanged.
50-50: 🩺 Stability & Availability | 🔵 Trivial | 🏗️ Heavy liftConsider upgrading
amondnet/vercel-actionitself instead of overriding its pinned CLI.Line 50 still uses
amondnet/vercel-action@v25, which is several years old and ships withPin Vercel CLI to 25.1.0 by@vitorbalin#171`` baked in — hence the need for thevercel-version: latestworkaround. Current README examples and Marketplace docs for this action now reference `uses: amondnet/vercel-action@v42`, a much newer major that may already track a current CLI baseline and avoid the double-path/root-directory quirks being worked around here.Worth verifying whether upgrading to a current major resolves the CLI-version issue more cleanly (and picks up any Node 20 runtime fixes), rather than layering a version override on an outdated action pin.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In @.github/workflows/deploy-website.yml at line 50, Upgrade the deployment step using the amondnet/vercel-action reference from `@v25` to the current documented major, `@v42`, then verify whether the existing vercel-version override and related path workarounds remain necessary; remove them if the upgraded action handles the CLI and root-directory behavior correctly.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Nitpick comments:
In @.github/workflows/deploy-website.yml:
- Line 50: Update the Vercel deployment steps using amondnet/vercel-action@v25
to set vercel-version to a specific known-good supported CLI version instead of
latest. Apply the same fixed version consistently to all occurrences, including
the additional deployment step, and leave the existing vercel-args configuration
unchanged.
- Line 50: Upgrade the deployment step using the amondnet/vercel-action
reference from `@v25` to the current documented major, `@v42`, then verify whether
the existing vercel-version override and related path workarounds remain
necessary; remove them if the upgraded action handles the CLI and root-directory
behavior correctly.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 81169ac7-d2d7-4baf-9c68-67155c7ca635
📒 Files selected for processing (1)
.github/workflows/deploy-website.yml
Summary
Resolves the failed workflow runs from the last
maincommit/PR cycle (merge of #119). Two independent workflows were red; this PR fixes the in-code causes of both. Two remaining causes are repo settings and are noted at the bottom.Changes
.release-please-manifest.jsontracked0.2.0whilepackage.jsonand the topCHANGELOG.mdentry are both0.4.0(no git tags exist). release-please therefore computed a bogus next version (0.4.0 → 0.2.1). Bumped the manifest to0.4.0so releases are based on the real current version.Your Vercel CLI version is outdated. This endpoint requires version 47.2.2 or later.becauseamondnet/vercel-action@v25pins Vercel CLI25.1.0. Addedvercel-version: latestso a current CLI is installed.Type of Change
Checklist
pnpm typecheckwith no errors (fullpnpm verify:fastpassed on the manifest change: 444 tests, lint, security, tool-sync)pnpm dev(n/a — CI config only)Related Issues
Closes #
Requires repo settings (cannot be fixed in code)
GitHub Actions is not permitted to create or approve pull requests.VERCEL_TOKEN/VERCEL_ORG_ID/VERCEL_PROJECT_IDrepository Actions secrets — now configured (auth confirmed working in the latest run; only the CLI-version error remained, which this PR fixes).🤖 Generated with Claude Code