Update app.py

app.py

@@ -52,6 +52,9 @@ Primary goals for reconcile rate:
             table first and a *second* DA3 table whose rows start with ITM DEPOSIT (credits) plus duplicate
             debits. Reorder to Credits + ITM table + Electronic Debits + merged debits; drop dup keys from the
             second table against the first; merge remaining orphan rows into the debits table.
+- Fix 17i: After Fix 17b splits ITM prefix from debits, OCR often leaves *credit* rows (POS Return, DDA
+            Transfer IN, Apple Cash INST R / CUPERTINO) at the *end* of the debits table — move that contiguous
+            tail from Electronic Debits back to Electronic Credits (same statement layout as the PDF).
 - Fix 17e: Repeat EC/ED fused + 17d passes until stable (handles chained replacements). Centered section titles
             match both align=\"center\" and style=\"text-align:center\" on divs.
 - Post-pass: dedupe 3-col Date|Description|Amount tables when one is a duplicate fragment
@@ -2998,6 +3001,45 @@ def _ucb_html_table_body_looks_itm_credits_only(table_html: str) -> bool:
     return True
 
 
+def _is_ucb_electronic_credit_row_after_split(grid, r: int) -> bool:
+    """
+    Rows that belong under Electronic Credits but often land at the end of the debits table after OCR
+    (Fix 17i). Must not match outgoing debits (POS purchase, Acct Fund SENT, etc.).
+    """
+    if not grid or r < 1 or r >= len(grid):
+        return False
+    if _is_strong_ucb_electronic_debit_row(grid, r):
+        return False
+    t = _row_text_full_ucb(grid, r)
+    if "pos return" in t:
+        return True
+    if "dda transfer" in t and "in" in t:
+        return True
+    if "acct fund" in t and ("inst r" in t or "cupertino" in t):
+        return True
+    return False
+
+
+def _ucb_move_trailing_credit_rows_from_ed_to_ec(g_ec, g_ed):
+    """Fix 17i: contiguous tail of credit-like rows at bottom of g_ed -> append to g_ec."""
+    if not g_ec or not g_ed or len(g_ed) < 2 or not _is_da3_header(g_ec) or not _is_da3_header(g_ed):
+        return g_ec, g_ed
+    i = len(g_ed) - 1
+    while i >= 1 and _is_ucb_electronic_credit_row_after_split(g_ed, i):
+        i -= 1
+    tail_start = i + 1
+    if tail_start >= len(g_ed):
+        return g_ec, g_ed
+    credit_tail = [list(g_ed[r]) for r in range(tail_start, len(g_ed))]
+    g_ed_new = [list(g_ed[0])] + [list(g_ed[r]) for r in range(1, tail_start)]
+    g_ec_new = [list(g_ec[0])] + [list(r) for r in g_ec[1:]] + credit_tail
+    max_c = max(len(r) for r in g_ec_new + g_ed_new)
+    for row in g_ec_new + g_ed_new:
+        while len(row) < max_c:
+            row.append("")
+    return g_ec_new, g_ed_new
+
+
 def _ucb_split_grid_electronic_credits_debits_after_itm_prefix(grid):
     """
     Fix 17b: one fused DA3 table under Electronic Credits (+ optional empty EC/ED headings) where OCR
@@ -3036,18 +3078,16 @@ def _try_split_ucb_ec_ed_from_fused_table_html(
     pair = _ucb_split_grid_electronic_credits_debits_after_itm_prefix(grid)
     if pair:
         g_ec, g_ed = pair
+        g_ec, g_ed = _ucb_move_trailing_credit_rows_from_ed_to_ec(g_ec, g_ed)
         sect = '<div align="center">\n\n{title}\n\n</div>\n\n'
         parts = []
         if lead_before_ec and lead_before_ec.strip():
             parts.append(lead_before_ec.rstrip())
-        parts.extend(
-            [
-                sect.format(title="Electronic Credits"),
-                _grid_to_html(g_ec),
-                sect.format(title="Electronic Debits"),
-                _grid_to_html(g_ed),
-            ]
-        )
+        parts.append(sect.format(title="Electronic Credits"))
+        parts.append(_grid_to_html(g_ec))
+        if len(g_ed) > 1:
+            parts.append(sect.format(title="Electronic Debits"))
+            parts.append(_grid_to_html(g_ed))
         return "\n\n".join(parts)
 
     # Fix 17c: headings stacked as EC / ED / table but rows are all credits (e.g. ITM-only) —