CL-5785: Kill /approvals page; approvals in notifications band

[TheGreatAxios]

Summary

• Delete the /approvals route and page — approvals were never a destination
• Render pending needs-you approvals as actionable ApprovalCards in the contextual panel notifications band (with pending count badge)
• Keep packages/approvals backend unchanged

Partial vs full acceptance

• Done: no Approvals nav/route; actionable cards in notifications; band-level pending badge
• Still partial: channel-inline approval cards (where the action originated) and per-channel rail badges — follow-up if not already covered by HITL run surfaces

Test plan

• bun test test/routes.test.tsx test/contextual-panel.test.tsx in apps/web (16 pass)
• CI checks + walking-skeleton

Linear: https://linear.app/abklabs/issue/CL-5785/approvals-kill-the-page-notifications-inline-channel-cards

[TheGreatAxios]

Multi-agent review (orchestrator)

Verdict: APPROVE (cannot self-approve via API)

What it does

Removes /approvals as a destination page and mounts NotificationsBand globally on the contextual panel with needs-you ApprovalCards (approve once / reject + optional message). README and route tests updated; page file gone.

Checked

• Diff vs origin/main: routes, panel contributions, notifications-band, contextual-panel, tests
• Local web tests: routes / contextual-panel / pages — 27 pass
• CI: checks + walking-skeleton green
• No leftover ApprovalsPage / /approvals route; API mutations still hub-scoped

Non-blocking

• Channel-inline approval cards / rail badges noted as partial in the commit body — intentional follow-on, not a merge blocker for the page kill.

Merging.