From 5e6f45408e8efef37f2130af5852d61ee32fa0a1 Mon Sep 17 00:00:00 2001
From: PJ <ipriyanshujain@gmail.com>
Date: Sun, 31 May 2026 21:15:21 +0530
Subject: [PATCH] fix(folio): totalBalance carrier tracks only Home, not Ledger

Home cardSum is a multi-account total; Ledger's LedgerBalance is a single
account on a different scale. Blending them in the carrier produced bogus
cross-screen deltas (prev from Ledger, curr from Home), triggering false
positives in submitMovesBalanceByTypedAmount. Restrict the carrier to
Home AccountCard totals via the computeHomeTotalBalance helper.
---
 examples/folio/sanderling/spec.ts | 29 ++++++++++++++---------------
 1 file changed, 14 insertions(+), 15 deletions(-)

diff --git a/examples/folio/sanderling/spec.ts b/examples/folio/sanderling/spec.ts
index 24a0e42..ee21cd3 100644
--- a/examples/folio/sanderling/spec.ts
+++ b/examples/folio/sanderling/spec.ts
@@ -10,7 +10,11 @@ import {
   whenRoute,
 } from "@sanderling/spec";
 import { defaultActions } from "@sanderling/spec/defaults";
-import { parseTypedAmount, submitChangesBalanceByTypedAmount } from "./predicates";
+import {
+  computeHomeTotalBalance,
+  parseTypedAmount,
+  submitChangesBalanceByTypedAmount,
+} from "./predicates";
 
 interface Account {
   name: string;
@@ -43,22 +47,17 @@ const accounts = extract<Account[]>("accounts", s =>
     balance: parseDollarCents(card.find({ testTag: "AccountBalance" })?.text),
   })));
 
-// Total balance: sum of AccountCard balances visible on Home plus the
-// LedgerBalance shown on Ledger. Both screens read the same DB, so summing
-// across whichever screen is visible gives the same authoritative number.
-// Screens that show neither signal (e.g. AddTransactionScreen) carry forward
-// the last-seen sum so `previous` and `current` stay on the same scale.
-let lastSeenTotalBalance = 0;
+// Total balance: sum of AccountCard balances visible on Home. The carrier
+// deliberately tracks only the Home multi-account total. Ledger's
+// LedgerBalance is a single-account number on a different scale and would
+// corrupt cross-screen comparisons if mixed in. Off-Home steps carry forward
+// the last-seen Home sum so `previous` and `current` stay on the same scale.
+let lastHomeTotal = 0;
 const totalBalance = extract("totalBalance", s => {
   const cards = s.ax.findAll([{ testTag: "HomeScreen" }, { testTag: "AccountCard" }]);
-  const cardSum = cards.reduce(
-    (sum, c) => sum + parseDollarCents(c.find({ testTag: "AccountBalance" })?.text),
-    0,
-  );
-  const ledgerBalText = s.ax.find({ testTag: "LedgerBalance" })?.text;
-  if (cards.length === 0 && !ledgerBalText) return lastSeenTotalBalance;
-  lastSeenTotalBalance = cardSum + parseDollarCents(ledgerBalText);
-  return lastSeenTotalBalance;
+  const cardBalanceTexts = cards.map(c => c.find({ testTag: "AccountBalance" })?.text);
+  lastHomeTotal = computeHomeTotalBalance({ cardBalanceTexts, previousCarrier: lastHomeTotal });
+  return lastHomeTotal;
 });
 
 const lastAction = extract("lastAction", s => s.lastAction);