feat: Review page — batch confirm/reject categorizations by confidence

[Copilot]

Implements the missing Review.svelte component for the financial-advisor plugin. The route was already registered in index.ts but the component file did not exist.

What's in the page

• Unconfirmed-only queue — filters to userConfirmed === false category inferences, sorted by confidence ascending (lowest certainty first)
• Statistics bar — live counters for total / pending / auto-confirmed / user-confirmed / user-corrected
• Filter by rule strategy — scopes the queue to a single inference rule (fa-recurring-amount, fa-vendor-clustering, fa-refund-detection, fa-tax-variance)
• Batch confirm — per-row checkboxes + select-all; "Confirm N Selected" button writes userConfirmed: true to each inference via infCollection.put() with optimistic local state update
• Single reject — opens a dialog pre-populated with the suggested category; correction saved as sourceId: 'user', confidence: 1.0, userConfirmed: true
• Decision chain expansion — ▼/▲ toggle per row fetches ctx.inference.getDecisionChain(inf.id) and renders the numbered step list alongside reasoning/rule/confidence metadata

Key shape of the reject write

const updated: TransactionInference = {
  ...inf,
  value: rejectCategory,
  confidence: 1.0,
  reasoning: 'Manually set by user.',
  sourceId: 'user',
  userConfirmed: true,
  updatedAt: now,
};
await infCollection.put(inf.id, updated);

Styling follows the same CSS custom-property tokens and component patterns as Transactions.svelte.

[Copilot]

Pull request overview

Adds the missing Review.svelte page for the financial-advisor plugin, enabling users to review and batch-confirm or correct low-confidence category inferences already produced/stored in fa-transaction-inferences.

Changes:

• Introduces an unconfirmed-only review queue sorted by confidence (lowest first), with rule-strategy filtering.
• Adds batch confirm flow (multi-select + select-all) that writes userConfirmed: true back to the inferences collection.
• Adds per-row reject/correct dialog that overwrites the inference with a user-sourced, confidence-1.0 correction and supports decision-chain expansion.

[Copilot]

batchConfirm() builds toConfirm from allInferences.filter(inf => selectedIds.has(inf.id)). Because selectedIds can include items not currently visible under the active filterStrategy, clicking “Confirm N Selected” can end up confirming hidden rows (and the “N” shown is selectedIds.size, not the visible selection count). Consider either clearing selectedIds when filterStrategy changes, or deriving a selectedVisibleIds set from unconfirmedRows and using that both for the button count and for toConfirm.

Suggested change

	const toConfirm = allInferences.filter(inf => selectedIds.has(inf.id));
	const toConfirm = unconfirmedRows
	.filter(row => selectedIds.has(row.inf.id))
	.map(row => row.inf);

[Copilot]

generateInferenceId is imported but never used in this component. Removing unused imports keeps the file easier to maintain and avoids confusion about intended behavior.

Suggested change

generateInferenceId,

[Copilot]

inferenceMap is defined but not referenced anywhere in this page. If it’s leftover from copying Transactions.svelte, it should be removed; if it’s intended to be used, wire it into the row building logic to avoid dead derived state.

Suggested change

	// ── Derived: inference map keyed by transactionId ────────────────────────
	const inferenceMap = $derived(
	new Map<string, TransactionInference>(
	allInferences
	.filter(inf => inf.field === 'category')
	.map(inf => [inf.transactionId, inf]),
	),
	);

[kayodebristol]

Auto-approved: CI green + code review complete.