Handle card comment load rejection

[Chris0Jeky]

Summary

• attach a rejection handler to CardModal comment loads on open and card switch
• keep store-owned error/toast handling and cached comments intact
• continue provenance/editing setup when comments fail and cover stale A-to-B rejection ordering

The prepared CardModal locator resolves to useCardModal.ts on the current base; no CardModal template change was needed.

Verification

• baseline after adding detectors: 1 failed test with 2 unhandled rejections
• final focused CardModal Vitest: 33/33 passed
• npm run typecheck: passed
• npm run build: passed; existing ineffective dynamic-import warning remains
• scoped ESLint: passed
• git diff --check: passed

The parent issue remains open for its other independent inspector follow-ups.

Refs #2129

[chatgpt-codex-connector]

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

[Chris0Jeky]

Fresh-context review record (coordinator, 2026-09-06; read-only Opus 5 reviewer over the diff against the merge base; route: the standing D-12 rule for Codex-session PRs — review, fix MEDIUM+ in-PR, merge, disposition comment).

VERDICT: SHIP (round 1). Fixed in-PR (round 2, coordinator commit bfc121478; CardModal.spec.ts and useCardModal.spec.ts 97/97): MEDIUM-1 — the new empty catch removed the only error-reporting sink for a comment-load failure (the rejection used to reach the window/Vue error handlers, which log and report to Sentry; handleApiError sets state and a toast but never logs), so a session where comments 500 on every card would show transient toasts and nothing in console or Sentry. The catch now calls logError, as every other catch in the file does.

Tracked for #2129, not fixed: LOW (pre-existing) — a rejected load renders "No comments yet." in CardModalComments.vue, the honest-empty lie class; LOW — the new "stale rejection" test's assertions also pass on base (it discriminates only through vitest's unhandled-rejection reporting); LOW — a now-false ordering comment in useCardModal.spec.ts:253-255; LOW — one tautological get(...).exists() assertion.