Skip to content

LTRAC-989: List custom domain statuses#3071

Open
jorgemoya wants to merge 4 commits into
alphafrom
jorgemoya/ltrac-989-cli-list-custom-domain-statuses
Open

LTRAC-989: List custom domain statuses#3071
jorgemoya wants to merge 4 commits into
alphafrom
jorgemoya/ltrac-989-cli-list-custom-domain-statuses

Conversation

@jorgemoya

Copy link
Copy Markdown
Contributor

What/Why?

Refs LTRAC-989

Adds catalyst domains list on top of the shared domain command/API client from #3070. The command resolves existing CLI credentials/project config, supports optional --domain and --status filters, and renders color-coded domain statuses with an explicit empty state.

This PR is stacked after #3070 and should be reviewed/merged after that PR.

Testing

  • node_modules/.bin/vitest run src/cli/commands/domains.spec.ts
  • node_modules/.bin/tsc --noEmit
  • node_modules/.bin/eslint src/cli/commands/domains.ts src/cli/commands/domains.spec.ts src/cli/lib/domains.ts tests/mocks/handlers.ts --max-warnings 0

Full node_modules/.bin/vitest run --coverage was attempted on #3070, but this local environment fails unrelated existing tests that require api.github.com network access and npm package env vars.

Migration

None.

Add a domains command group with a create-domain API client and wait support for pending domain verification.

Refs LTRAC-988
@vercel

vercel Bot commented Jun 29, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
catalyst Ready Ready Preview, Comment Jun 29, 2026 8:03pm

Request Review

@changeset-bot

changeset-bot Bot commented Jun 29, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 6e7b435

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

Bundle Size Report

Comparing against baseline from 81b88df (2026-06-29).

No bundle size changes detected.

Include the HTTP status and correlation ID when domain API requests fail with a server-side response so 502 Bad Gateway errors are actionable.

Refs LTRAC-988
@jorgemoya jorgemoya force-pushed the jorgemoya/ltrac-989-cli-list-custom-domain-statuses branch from d4fb25a to 075a3b1 Compare June 29, 2026 20:00
Keep domain API server error messages concise because the CLI fatal error handler already prints the correlation ID for support.

Refs LTRAC-988
Add domain listing support with optional domain and verification-status filters so users can scan configured Native Hosting domains from the CLI.

Refs LTRAC-989
@jorgemoya jorgemoya force-pushed the jorgemoya/ltrac-989-cli-list-custom-domain-statuses branch from 075a3b1 to 6e7b435 Compare June 29, 2026 20:02
@jorgemoya

Copy link
Copy Markdown
Contributor Author

Tested locally:

~/dev/catalyst/core on  jorgemoya/ltrac-988-cli-add-custom-domain-command [✘!?] took 3s
❯ node ../packages/catalyst/dist/cli.js domains list  --api-host api.staging.zone
◢ @bigcommerce/catalyst v1.0.0-alpha.6                                                                                                                   3:34:47 PM

◐ Fetching domains...                                                                                                                                    3:34:47 PM
✔ Domains fetched.                                                                                                                                      3:34:49 PM
jm-staging-test-v2.catalyst-sandbox-staging.store active                                                                                                 3:34:49 PM
www.example-bc-jm-staging.com unknown   

@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 90 92 92 95

Category Scores

Category Prod Desktop Prod Mobile Preview Desktop Preview Mobile
Performance 72 80 77 79
Accessibility 95 98 95 92
Best Practices 100 100 100 100
SEO 88 88 100 100

Core Web Vitals

Metric Prod Desktop Prod Mobile Preview Desktop Preview Mobile
LCP 4.8 s 5.5 s 3.4 s 5.5 s
CLS 0.039 0 0.001 0
FCP 1.1 s 1.2 s 1.2 s 1.1 s
TBT 0 ms 0 ms 0 ms 0 ms
Max Potential FID 50 ms 40 ms 50 ms 50 ms
Time to Interactive 4.8 s 5.5 s 3.4 s 6.6 s

Full Unlighthouse report →

@jorgemoya jorgemoya marked this pull request as ready for review June 29, 2026 21:26
@jorgemoya jorgemoya requested a review from a team as a code owner June 29, 2026 21:26
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