Skip to content

fix(create-email): validate npm registry response and exit on invalid version - #3741

Merged
felipefreitag merged 2 commits into
resend:canaryfrom
Tyagiquamar:fix/create-email-validate-npm-response
Sep 22, 2026
Merged

felipefreitag merged 2 commits into
resend:canaryfrom
Tyagiquamar:fix/create-email-validate-npm-response

Conversation

@Tyagiquamar

@Tyagiquamar Tyagiquamar commented Sep 12, 2026 •

Copy link
Copy Markdown
Contributor

Check response.ok when querying npm registry in getLatestVersionOfTag inside create-email. Immediately exit with error status if the npm HTTP request fails, tag is not found, or the version format is invalid/empty, preventing create-email from silently writing undefined versions into starter package.json.


Summary by cubic

Fixes create-email to validate the npm registry response and exit with an error when the request fails or the version is empty/invalid, instead of silently writing undefined into the starter package.json. Missing tags (HTTP 404) keep their specific error message before the generic failure check.

Written for commit e05fc73. Summary will update on new commits.

Review in cubic

… version

Signed-off-by: Tyagiquamar <mohdquamartyagi@gmail.com>
@vercel

vercel Bot commented Sep 12, 2026

Copy link
Copy Markdown
Contributor

@Tyagiquamar is attempting to deploy a commit to the resend Team on Vercel.

A member of the Team first needs to authorize it.

@changeset-bot

changeset-bot Bot commented Sep 12, 2026 •

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: e05fc73

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
create-email Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions github-actions Bot added the linear-synced PR has been synced to Linear label Sep 13, 2026
The npm registry answers a missing tag with HTTP 404, so the generic
ok check hid the specific "Tag X does not exist" message. Check for
404 first, then for any other non-OK status, then parse the body.
@pkg-pr-new

pkg-pr-new Bot commented Sep 22, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/create-email@3741

commit: e05fc73

@felipefreitag
felipefreitag merged commit ac4d9df into resend:canary Sep 22, 2026
9 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

linear-synced PR has been synced to Linear

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants