Repository navigation
improvement(billing): show the Chat plan card above the composer instead of redirecting on send - #9557
Conversation
…ead of redirecting on send
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
[Medium impact] The PR appears safe to merge; no new actionable issues were found. SummaryShows the paid-plan card above the composer instead of redirecting users when they send.
Reviews (3) · Last reviewed commit: "fix(billing): resume held queued sends o..." · Reviewed by Greptile |
There was a problem hiding this comment.
All reported issues were addressed across 15 files
Reply with feedback, questions, or to request a fix.
Turn on auto-fix | Re-trigger cubic
|
@cubic-dev-ai review this PR |
@waleedlatif1 I have started the AI code review. It will take a few minutes to complete. |
There was a problem hiding this comment.
All reported issues were addressed across 15 files
Reply with feedback, questions, or to request a fix.
Turn on auto-fix | Re-trigger cubic
…Send now side-effect free
|
@cubic-dev-ai review this PR |
@waleedlatif1 I have started the AI code review. It will take a few minutes to complete. |
Summary
useChatPlanGate(navigate/toast callback) becomesuseChatPlanRequired, a boolean derived from the host context'sfeatures.chat;useChat.sendMessagestill holds programmatic and queued sendsCHAT_PLAN_REQUIRED_UPGRADEmoves to the client-safelib/billing/chat-plan.tsso the composer can render it;usage-upgrade.tskeeps a type-only importPlanUsageLimitDisplayis exported from the special-tags barrel, like the other displays reused outside messagesType of Change
Testing
tools/billing.test.ts,lifecycle/run.test.ts, andjobssuites pass locally (270 tests)Checklist
test-auditauthoring gate)