Skip to content

chore(cms): update alt-text and vercel-deployments plugins#23

Merged
jhb-dev merged 1 commit intomainfrom
claude/update-vercel-plugin-Xx6Yo
Apr 9, 2026
Merged

chore(cms): update alt-text and vercel-deployments plugins#23
jhb-dev merged 1 commit intomainfrom
claude/update-vercel-plugin-Xx6Yo

Conversation

@jhb-dev
Copy link
Copy Markdown
Contributor

@jhb-dev jhb-dev commented Apr 9, 2026

Update @jhb.software/payload-alt-text-plugin from 0.4.2 to 0.4.3 and
@jhb.software/payload-vercel-deployments from ^0.1.0 to ^0.2.0. Pass
the website URL (NEXT_PUBLIC_FRONTEND_URL) to the vercel deployments
widget config via the new websiteUrl option.

https://claude.ai/code/session_01LqFjyiwxKEG7ZXiaZFVUnM

Update @jhb.software/payload-alt-text-plugin from 0.4.2 to 0.4.3 and
@jhb.software/payload-vercel-deployments from ^0.1.0 to ^0.2.0. Pass
the website URL (NEXT_PUBLIC_FRONTEND_URL) to the vercel deployments
widget config via the new websiteUrl option.

https://claude.ai/code/session_01LqFjyiwxKEG7ZXiaZFVUnM
@vercel
Copy link
Copy Markdown

vercel bot commented Apr 9, 2026

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

2 Skipped Deployments
Project Deployment Actions Updated (UTC)
website Ignored Ignored Apr 9, 2026 7:55am
website-cms Ignored Ignored Apr 9, 2026 7:55am

Request Review

@claude
Copy link
Copy Markdown

claude bot commented Apr 9, 2026

PR Review

This is a straightforward dependency update. A few observations:

Potential Issues

websiteUrl type mismatch (cms/src/payload.config.ts):
process.env.NEXT_PUBLIC_FRONTEND_URL returns string | undefined, while projectId above it uses the non-null assertion (!). If the plugin's widget.websiteUrl expects a string (not string | undefined), this could cause a TypeScript error or unexpected runtime behavior. Verify whether the option is optional in @jhb.software/payload-vercel-deployments@0.2.0's types — if not, add ! or a fallback.

Unintended lock file change (pnpm-lock.yaml):
The mcp-handler snapshot changed its @modelcontextprotocol/sdk peer resolution from zod@3.25.76 to zod@4.1.12. This appears to be an unintentional side-effect of running pnpm install. While likely harmless, it's worth verifying this doesn't introduce a zod version conflict at runtime.

Security

No concerns — NEXT_PUBLIC_FRONTEND_URL is already a public environment variable (browser-exposed), so passing it to the widget config doesn't expose anything sensitive.

Performance

No concerns — these are minor plugin version bumps with no expected performance impact.


Overall the PR looks good. The main thing to double-check is the websiteUrl: string | undefined typing.

@jhb-dev jhb-dev merged commit 180ba7b into main Apr 9, 2026
7 of 8 checks passed
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