# CHANGES_REQUIRED ## Findings - **P2 — Neutral-transfer KPI is now always displayed as CHF 0.00.** The backend hard-codes the existing `neutral_transfers_chf` contract field to `"0.00"` and places the actual volume in the new `neutral_transfer_volume_chf` field (`src/jarvis_finance/services/household_import.py:2129-2131`). The frontend contract does not expose the new field (`frontend/src/api/household.ts:10-15`), and the visible “Neutrale Transfers” card still reads `neutral_transfers_chf` (`frontend/src/pages/HouseholdOverviewPage.vue:60-65`). A month with CHF 91.00 transfer volume therefore displays CHF 0.00 despite saying transfers are shown separately. - **P2 — Imported expenses can render with two minus signs.** Transaction amounts retain their stored sign, while `signedAmount()` adds another type-derived sign without taking the absolute value (`frontend/src/pages/HouseholdTransactionsPage.vue:114-122`). For the normal imported expense value `-42.00`, the actual formatter path produces `− CHF -42.00`. The candidate also extends this path to unconverted `amount_original`, so foreign-currency imported expenses have the same incorrect presentation. Existing frontend fixtures use positive expense amounts and do not cover the signed import contract. ## Prior findings verification All three findings from the `71183ca` review are otherwise closed: - Mutating each cursor-relevant family produced **409** for the old review cursor: open candidates, confirmed candidate history, confirmed transactions, categories, merchants, aliases, review rules, recurring payments, source mappings, budget accounts, and canonical accounts. - `needs_decision` is requested server-side; focused pagination and stale-review tests passed, including replacement of appended stale rows with fresh page one. - Frontend category-type filtering and `can_confirm` behavior passed. Synthetic server probes rejected wrong category type, special-case normal confirmation, non-CHF rows, and incompatible mappings with **409**. ## Verification - Focused backend review test: **1 passed** - Focused frontend review-contract tests: **2 passed** - Focused financial-semantics tests: **4 passed** - Exact HEAD before and after: `8f6eee0a2520b663c8fe92bca1ebd5c7870ef615` - Base is an ancestor; `git diff --check` passed. - Worktree remained clean. - No repository files created or modified.