Skip to content

fix(cli): clean up deployment url output#2960

Merged
matthewvolk merged 2 commits intoalphafrom
TRAC-310-deployment-url
Mar 30, 2026
Merged

fix(cli): clean up deployment url output#2960
matthewvolk merged 2 commits intoalphafrom
TRAC-310-deployment-url

Conversation

@matthewvolk
Copy link
Copy Markdown
Contributor

@matthewvolk matthewvolk commented Mar 30, 2026

What/Why?

The catalyst deploy command outputs the deployment URL twice — once as a raw console.log and again formatted via consola.success(). The spinner success message also has a trailing \n that produces an extra blank line. Additionally, the URL is printed without an https:// scheme, so it isn't clickable in terminal emulators.

This PR:

  • Removes the redundant console.log(data.deployment_url) call
  • Removes the trailing \n from the spinner success message
  • Prepends https:// to the displayed URL (with a guard against double-prefixing) and colors it blue

Testing

Run catalyst deploy and verify the output shows:

✔ Deployment completed successfully.
✔ View your deployment at: https://<project>.catalyst-sandbox.store

Example:
clickable

  • URL appears once (not twice)
  • No extra blank line between messages
  • URL is blue and CMD+clickable / Ctrl+clickable in terminal

Migration

N/A

Remove redundant console.log, trailing \n on spinner message, and
prepend https:// scheme so the url is clickable in terminal emulators.
@matthewvolk matthewvolk requested a review from a team as a code owner March 30, 2026 17:53
@vercel
Copy link
Copy Markdown

vercel bot commented Mar 30, 2026

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

Project Deployment Actions Updated (UTC)
catalyst Ready Ready Preview, Comment Mar 30, 2026 6:48pm

Request Review

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Mar 30, 2026

⚠️ No Changeset found

Latest commit: b736348

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 30, 2026

Bundle Size Report

Comparing against baseline from 4870221 (2026-03-30).

No bundle size changes detected.

@github-actions
Copy link
Copy Markdown
Contributor

Unlighthouse Performance Comparison — Vercel

Comparing PR preview deployment Unlighthouse scores vs production Unlighthouse scores.

Summary Score

Aggregate score across all categories as reported by Unlighthouse.

Prod Desktop Prod Mobile Preview Desktop Preview Mobile
Score 92 94 91 94

Category Scores

Category Prod Desktop Prod Mobile Preview Desktop Preview Mobile
Performance 77 81 70 72
Accessibility 95 91 95 95
Best Practices 100 100 95 100
SEO 100 100 100 100

Core Web Vitals

Metric Prod Desktop Prod Mobile Preview Desktop Preview Mobile
LCP 3.5 s 5.2 s 5.3 s 4.5 s
CLS 0 0.039 0.037 0.245
FCP 1.2 s 1.1 s 1.2 s 1.2 s
TBT 0 ms 40 ms 10 ms 0 ms
Max Potential FID 40 ms 90 ms 70 ms 60 ms
Time to Interactive 3.6 s 5.3 s 5.5 s 4.6 s

Full Unlighthouse report →

@matthewvolk matthewvolk merged commit 99c1d4c into alpha Mar 30, 2026
14 of 16 checks passed
@matthewvolk matthewvolk deleted the TRAC-310-deployment-url branch March 30, 2026 19:57
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