Skip to content

feat(feedback): add optional startup campaigns - #33

Merged
ImPedro29 merged 1 commit into
mainfrom
feat/startup-feedback-campaigns
Aug 16, 2026
Merged

ImPedro29 merged 1 commit into
mainfrom
feat/startup-feedback-campaigns

Conversation

@ImPedro29

Copy link
Copy Markdown

Summary

  • add the authenticated feedback campaign client and validate the API contract
  • show at most one optional inline survey during an eligible interactive CLI startup
  • dismiss the survey as soon as normal work begins, without changing or delaying the prompt
  • persist bounded local receipts and a durable retry outbox containing identifiers and option IDs only
  • respect TTY, CLI entrypoint, remote mode, CI, telemetry, and organization policy gates
  • remove inherited automatic mid-session surveys from the REPL

Impact

  • user-facing impact: eligible users may see one optional PT/EN survey at startup; typing normally, submitting work, or choosing 0 dismisses it
  • developer/maintainer impact: PromptInput now exposes an optional pre-submit interceptor for inline startup UI

Testing

  • bun run build
  • bun run smoke
  • bun test src/components/VerbooFeedback/useVerbooStartupFeedback.test.ts
  • bun run verify:privacy
  • bun run security:pr-scan -- --base origin/main
  • bun run test:isolated — all 315 files ran; two unrelated secure-storage lock tests failed under the parallel runner and passed immediately when rerun individually

Notes

  • provider/model path tested: active model identity is included in the validated startup request
  • screenshots attached: not included; this is inline terminal UI
  • dependency: requires the feedback API and migration being prepared in the backend repository
  • no release, package publication, or deployment was triggered

@ImPedro29
ImPedro29 merged commit f7b2482 into main Aug 16, 2026
12 checks passed
@ImPedro29
ImPedro29 deleted the feat/startup-feedback-campaigns branch August 16, 2026 02:33
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