Skip to content

LTRAC-2163: feat(cli) - Ask for feedback after catalyst deploy - #3276

Merged
parthshahp merged 1 commit into
canaryfrom
parthshah/ltrac-2163-4-deploy-survey
Oct 19, 2026
Merged

parthshahp merged 1 commit into
canaryfrom
parthshah/ltrac-2163-4-deploy-survey

Conversation

@parthshahp

@parthshahp parthshahp commented Oct 13, 2026 •

Copy link
Copy Markdown
Contributor

What/Why?

Asks one optional question after a successful catalyst deploy. Part of a stack that lets Storefront Hosting users send feedback from the CLI. Stacked under #3264.

Help us improve Catalyst. One question, and you can skip it.
? How was your experience using the Catalyst CLI with Storefront Hosting? (0 = very poor, 10 = excellent, Enter to skip)
? What went well, or what could be better? (optional, Enter to skip)
✔ Thank you for your feedback!
  • Runs in a postAction hook (hooks/survey.ts), only after deploy succeeds.
  • Shows at most once every 90 days on each machine (survey.lastShownAt in the user config). The date is saved before the first question, so a skip, Ctrl+C, or a failed send also counts.
  • Never shows in CI, without a TTY, with the new global --no-hints option, or when CATALYST_NO_HINTS is set.
  • Sends only the score, the comment, the CLI version, and the command name (source catalyst-cli-survey).
  • Cannot fail or block the command: all errors are caught, each prompt closes after 2 minutes, and the request stops after 5 seconds.
  • lib/prompt-context.ts holds helpers that LTRAC-2163: feat(cli) - Offer to send an error report after a CLI crash #3264 also uses (opt-outs, credential lookup, quiet prompt errors, timeouts).

The feedback endpoint does not accept CLI tokens yet. A backend change for this is in progress.

Testing

  • pnpm lint, pnpm typecheck, and pnpm test pass (876 tests).
  • catalyst deploy --help shows the global --no-hints option.
  • End to end (after the backend change): remove survey.lastShownAt from the user config, run catalyst deploy in a terminal, and answer the survey.

Migration

None.

@vercel

vercel Bot commented Oct 13, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
catalyst Ready Ready Preview Oct 19, 2026 5:09pm UTC

Request Review

@changeset-bot

changeset-bot Bot commented Oct 13, 2026 •

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 483a9ff

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 changesets to release 1 package
Name Type
@bigcommerce/catalyst Minor

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

@jorgemoya jorgemoya left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@parthshahp
parthshahp added this pull request to the merge queue Oct 19, 2026
Merged via the queue into canary with commit 6872510 Oct 19, 2026
16 checks passed
@parthshahp
parthshahp deleted the parthshah/ltrac-2163-4-deploy-survey branch October 19, 2026 23:15

This branch was successfully deployed

1 active deployment
Preview — 483a9ff9 Deployed Oct 19, 2026 by vercel[bot]
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