LTRAC-991: Remove custom domain command#3073
Conversation
Add a domains command group with a create-domain API client and wait support for pending domain verification. Refs LTRAC-988
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Bundle Size ReportComparing against baseline from 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
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
Add a domains status command that fetches one domain, renders its color-coded state, and can wait for pending verification to finish. Refs LTRAC-990
Add a domains remove command that checks current status, confirms active domain removal, and deletes the domain through the Native Hosting API. Refs LTRAC-991
9418037 to
8b5053c
Compare
|
|
Tested locally: |
Unlighthouse Performance Comparison — VercelComparing PR preview deployment Unlighthouse scores vs production Unlighthouse scores. Summary ScoreAggregate score across all categories as reported by Unlighthouse.
Category Scores
Core Web Vitals
|
What/Why?
Refs LTRAC-991
Adds
catalyst domains remove <domain>on top of the stacked domain command work. The command fetches the current domain status before deleting, prompts before removing an active domain, supports--forceto skip the prompt, and calls the Native Hosting domain delete endpoint.This PR is stacked after #3072 and should be reviewed/merged after that PR.
Testing
node_modules/.bin/vitest run src/cli/commands/domains.spec.tsnode_modules/.bin/tsc --noEmitnode_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 0Full
node_modules/.bin/vitest run --coveragewas attempted on #3070, but this local environment fails unrelated existing tests that requireapi.github.comnetwork access and npm package env vars.Migration
None.