fix(ai): account modal — un-clip the chevron, highlight Manage account

[ndemianc]

Two small follow-ups on the merged chevron-hero modal (#51), from live feedback.

• Chevron no longer clipped at the top. Shrunk the mark 212 → 182px and absolutely positioned it 12px from the top of the hero, so the chevron's peak clears the modal's rounded top edge. The SVG is still taller than the hero, so only the lower soft circles bleed off — the chevron itself is never cut.
• Manage account is the primary CTA. It's now the accent-filled button on paid plans, so it reads as the tempting action. On free tier it stays quiet — the accent belongs to the Upgrade button there, and two filled buttons would compete (manageClass = upgradeBtn ? 'abtn' : 'abtn primary').

Motion/a11y unchanged (hero still aria-hidden, prefers-reduced-motion respected, HC defers to editor tokens).

Preview (updated): https://claude.ai/code/artifact/0c955831-6ce9-444f-95b2-c211b8706e88

Verified: chat.html scripts parse; webviewCss/creditFormat + full gate green. One file, +8/-3.

🤖 Generated with Claude Code

[Copilot]

Pull request overview

Adjusts the signed-in “classic” account modal in levelcode-ai based on live feedback, improving the hero’s chevron rendering and clarifying the primary call-to-action depending on plan state.

Changes:

• Unclips the chevron by resizing and absolutely positioning the portal SVG inside the hero.
• Makes “Manage account” the accent-filled CTA on paid plans while keeping “Upgrade” primary on free-tier gateway plans.

Comments suppressed due to low confidence (2)

extensions/levelcode-ai/media/chat.html:2823

• openAccount() currently focuses [data-act="manage"] before other actions. With Manage account no longer the primary CTA on free tier (when Upgrade is present), initial focus will still land on Manage rather than the highlighted/primary action, which is inconsistent with the intent and the nearby comment “Focus the primary action”.

      const manageClass = upgradeBtn ? 'abtn' : 'abtn primary';

extensions/levelcode-ai/media/chat.html:2823

• The new CTA-priority logic (Upgrade vs Manage account) isn’t covered by existing unit tests. Given the repo’s pattern of compiling/extracting chat.html logic in Node tests, add a small test that renderAccount() produces the expected button classes for (1) gateway+free (Upgrade primary, Manage quiet) and (2) paid/BYOK (Manage primary, no Upgrade).

      const manageClass = upgradeBtn ? 'abtn' : 'abtn primary';

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.