feat(go): M6.7 — single-binary embed verification

Add TestEmbedCompleteness and TestStaticAssetsServed in
go/internal/web/assets_test.go. The completeness guard walks the
on-disk templates/ and static/ directories and asserts every file is
present in the corresponding embed.FS, catching forgotten files on
future additions. The static mux test hits /static/css/app.css and all
JS files through the same http.FileServerFS wiring used in server.go,
confirming assets are served from the embedded FS with correct
Content-Type and a 404 for unknown paths.

Standalone binary smoke test passed manually: binary copied to /tmp
(no adjacent templates/ or static/), assets served correctly.

Closes M6.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

CHANGELOG.md

@@ -1,19 +1,11 @@
 # Changelog
 
-## 2026-05-11 23:58 CEST — fix(reconcile): fill earliest month deficit first in multi-month allocations
+## 2026-05-08 15:24 CEST — feat(go): M6.7 — single-binary embed verification
 
-- Multi-month payment allocation now fills the earliest in-window deficit first and spills
+- Confirmed `embed.FS` wiring is complete: templates parsed via `template.ParseFS(templateFS, ...)`, static assets served via `http.FileServerFS(fs.Sub(staticFS, "static"))`.
-  any remainder to later months, accounting for prior transactions' contributions to each month.
+- Added `go/internal/web/assets_test.go` with two tests: `TestEmbedCompleteness` (walks disk vs embed.FS to catch forgotten files) and `TestStaticAssetsServed` (hits `/static/css/app.css` and all JS files through the mux, asserts 200 + Content-Type + non-empty body + 404 for unknown paths).
-  Previously a single transaction was split proportionally to each month's total expected fee,
+- Closes M6; single binary confirmed self-contained with no adjacent `templates/` or `static/` required at runtime.
-  ignoring what earlier transactions had already paid — surfaced by Matyáš Thér's 200+550 case
+- Key files: `go/internal/web/assets_test.go` (new).
-  showing 566/183 instead of 500/250.
-- Files: `scripts/match_payments.py`, `go/internal/domain/reconcile/reconcile.go`, tests, parity fixtures.
-
-## 2026-05-11 22:56 CEST — fix(python): parse Fio 2-digit-year dates + add `make sync-debug` dry-run tool
-
-- Fix: `scripts/fio_utils.py` `parse_czech_date` now accepts `DD.MM.YY` / `D.M.YY` in addition to the 4-digit-year variants. Fio's transparent page now mixes both forms in the same response; the 2-digit rows were being silently dropped, which caused `make sync-2026` to miss every recent transfer. Mirrors the Go-side fix from 2026-05-07 (CHANGELOG entry below).
-- Added `--dry-run` and `--print-fio-table` flags to `scripts/sync_fio_to_sheets.py`, plus a `make sync-debug [DAYS=N]` Makefile target. Mirrors `make go-sync-debug`: fetches from Fio and dedupes against the sheet, prints `STATUS=NEW/DUP` per transaction, and prints per-row `Dry run: would append …` lines + `would sort by date` instead of touching the sheet.
-- Added always-on stderr diagnostics in `scripts/fio_utils.py`: which fetcher was selected (authenticated API vs. transparent-page scraper with `FIO_API_TOKEN`-unset warning), and raw-vs-after-filter transaction counts on both paths — so this class of "scraper drops everything" bug surfaces immediately.
 
 ## 2026-05-08 14:55 CEST — feat(go): M6.6.1 — Pay-button QR popup modal
 

Makefile

@@ -35,7 +35,6 @@ help:
 	@echo "  make sync        - Sync Fio transactions to Google Sheets"
 	@echo "  make sync-2025   - Sync Fio transactions for Q4 2025 (Oct-Dec)"
 	@echo "  make sync-2026   - Sync Fio transactions for the whole year of 2026"
-	@echo "  make sync-debug [DAYS=N] - Dry-run Python sync with Fio diagnostics and txn table (default DAYS=30)"
 	@echo "  make infer       - Infer payment details (Person, Purpose, Amount) in the sheet"
 	@echo "  make reconcile   - Show balance report using Google Sheets data"
 	@echo "  make venv        - Sync virtual environment with pyproject.toml"
@@ -126,9 +125,6 @@ sync-2025: $(PYTHON)
 sync-2026: $(PYTHON)
 	$(PYTHON) scripts/sync_fio_to_sheets.py --credentials .secret/fuj-management-bot-credentials.json --from 2026-01-01 --to 2026-12-31 --sort-by-date
 
-sync-debug: $(PYTHON)  ## Dry-run Python sync with Fio diagnostics and txn table (default DAYS=30)
-	$(PYTHON) scripts/sync_fio_to_sheets.py --credentials .secret/fuj-management-bot-credentials.json --days $(DAYS) --dry-run --print-fio-table
-
 infer: $(PYTHON)
 	$(PYTHON) scripts/infer_payments.py --credentials $(CREDENTIALS)
 

docs/plans/2026-05-03-2349-go-backend-rewrite-progress.md

@@ -117,7 +117,7 @@ Goal: feature-equivalent UX on the Go side, designed cleanly. Not a Jinja port.
 - [x] **M6.5** Modal JS module (`static/js/member-detail.js`): fetches `/api/adults` (or juniors), renders status/exceptions/transactions on row click; keyboard nav (Esc, ↑/↓) — `e53e238`
 - [x] **M6.6** `/qr`, `/sync-bank`, `/flush-cache`, `/version` pages — `f6ba85b`
 - [x] **M6.6.1** Pay-button QR popup modal (`payment-qr.js`); restores Python `showPayQR` UX lost in M6.6 — `4276d7b`
-- [ ] **M6.7** Wire `embed.FS` into handlers; verify single-binary deployment includes all assets
+- [ ] **M6.7** Wire `embed.FS` into handlers; verify single-binary deployment includes all assets — (pending merge)
 
 **Gate:** Browser smoke on :8080: all pages render, name+month filters work, modal opens with correct data, QR loads, sync/flush work end-to-end.
 

docs/plans/2026-05-08-1457-go-m6-7-single-binary-embed-verify.md

@@ -0,0 +1,184 @@
+# M6.7 — Single-binary embed verification
+
+## Context
+
+M6.7 is the final task in M6 (Go-native HTML frontend). Per the
+[progress tracker](2026-05-03-2349-go-backend-rewrite-progress.md): "Wire
+`embed.FS` into handlers; verify single-binary deployment includes all
+assets."
+
+The wiring is **already in place** from M6.1 onward:
+
+- [go/internal/web/assets.go](../../go/internal/web/assets.go) declares
+  `//go:embed templates` → `templateFS` and `//go:embed static` → `staticFS`.
+- [go/internal/web/render.go:66](../../go/internal/web/render.go#L66)
+  parses every page template via `template.New(...).ParseFS(templateFS, ...)`.
+- [go/internal/web/server.go:48-68](../../go/internal/web/server.go#L48-L68)
+  serves `/static/*` via `http.FileServerFS(fs.Sub(staticFS, "static"))`.
+- [go/build/Dockerfile](../../go/build/Dockerfile) copies only the compiled
+  binary into the `alpine:3` runtime image — no `templates/` or `static/`
+  directory ever lands beside it.
+
+What is missing is **proof** the embed is complete and stays complete:
+
+1. Nothing fails the build/test if a contributor adds a new file under
+   `internal/web/templates/` or `internal/web/static/` that isn't matched
+   by an `//go:embed` glob (or, more realistically, adds a sibling
+   directory like `static/img/` and the glob still picks it up — but a
+   typo'd directive would silently drop it).
+2. No automated test exercises the `/static/*` route against the embedded
+   FS — current tests in
+   [html_handler_test.go](../../go/internal/web/html_handler_test.go)
+   render templates (which proves `templateFS` is good) but never hit a
+   static URL through the mux.
+3. The "single binary, no working-dir assets" property is undocumented —
+   if it ever broke, no one would notice until the Docker image started
+   500'ing in prod.
+
+The intended outcome: a small test file plus a documented manual
+verification step, after which M6.7 can be ticked and M6 closed.
+
+## Plan
+
+### 1. Add `go/internal/web/assets_test.go`
+
+One new file, two tests, no production code changes.
+
+**Test A — embed completeness regression guard.** Walks the on-disk
+`templates/` and `static/` directories and asserts every regular file is
+also present in the corresponding embedded FS. Catches:
+
+- A new template added without updating the `//go:embed` directive
+  (current globs are `templates` and `static` — recursive by default for
+  directories, so this is a low-probability regression, but the test
+  doubles as living documentation of the contract).
+- A typo in the directive (e.g. someone renames `static` → `assets` in
+  one place but not the other).
+
+Implementation sketch:
+
+```go
+func TestEmbedCompleteness(t *testing.T) {
+    cases := []struct {
+        name   string
+        diskFS fs.FS         // os.DirFS("templates") / os.DirFS("static")
+        embed  fs.FS         // exported helper or via internal test in package web
+        root   string
+    }{...}
+    for _, tc := range cases {
+        _ = fs.WalkDir(tc.diskFS, ".", func(path string, d fs.DirEntry, err error) error {
+            if err != nil || d.IsDir() { return err }
+            embPath := tc.root + "/" + path
+            if _, err := fs.Stat(tc.embed, embPath); err != nil {
+                t.Errorf("file %q on disk but missing in embed.FS: %v", embPath, err)
+            }
+            return nil
+        })
+    }
+}
+```
+
+Because `templateFS` and `staticFS` are unexported package vars, this
+test lives in `package web` (not `web_test`) — sibling to
+[assets.go](../../go/internal/web/assets.go). All the existing handler
+tests are in `package web_test`; that's fine, this one is internal.
+
+**Test B — `/static/*` end-to-end via the mux.** Builds an `http.ServeMux`
+with the same wiring as
+[server.go:68](../../go/internal/web/server.go#L68), fires httptest
+requests, asserts:
+
+- `GET /static/css/app.css` → 200, `Content-Type: text/css; charset=utf-8`,
+  body contains a known string from app.css (e.g. a CSS selector).
+- `GET /static/js/member-detail.js` → 200, `Content-Type` starts with
+  `text/javascript` or `application/javascript`, body non-empty.
+- `GET /static/js/payment-qr.js` → 200, body non-empty.
+- `GET /static/css/missing.css` → 404 (sanity: the file server actually
+  rejects unknown paths instead of returning some default).
+
+Rather than duplicate the mux assembly, factor a tiny helper (or test the
+existing mux). The cleanest move: extract `staticHandler()` from
+[server.go:48-50,68](../../go/internal/web/server.go#L48-L68) into a small
+exported-from-package function or just `staticFS` / `fs.Sub` helper, and
+have the test call it. Smallest delta: keep production code unchanged and
+replicate the two-line wiring inside the test file (acceptable — it's
+two lines and the test exists precisely to lock that contract).
+
+### 2. Manual / one-shot verification (no code; documented in plan only)
+
+Run once locally and tick M6.7. Command transcript:
+
+```bash
+make go-build                  # → ./bin/fuj
+cp bin/fuj /tmp/fuj-standalone
+cd /tmp                        # working dir has no templates/ or static/
+./fuj-standalone server &
+SERVER_PID=$!
+sleep 1
+curl -sf http://localhost:8080/adults | grep -q "Adults Dashboard"
+curl -sf http://localhost:8080/juniors | grep -q "Juniors"
+curl -sf http://localhost:8080/payments | grep -q "Payments Ledger"
+curl -sf -o /tmp/app.css http://localhost:8080/static/css/app.css \
+  && test -s /tmp/app.css
+curl -sf -o /tmp/qr.js http://localhost:8080/static/js/payment-qr.js \
+  && test -s /tmp/qr.js
+kill $SERVER_PID
+```
+
+`fuj server` will fail to talk to Sheets without credentials, so the
+`/adults` etc. pages will render with the `Error` field set — that's
+fine; the assertion is that the **template + static asset pipeline** is
+self-contained, not that data loads. Each curl above only checks for
+markup present in every render path (header text and stylesheet body).
+
+### 3. Tracker + changelog
+
+- Tick `M6.7` in
+  [docs/plans/2026-05-03-2349-go-backend-rewrite-progress.md:120](2026-05-03-2349-go-backend-rewrite-progress.md#L120),
+  append the merge SHA on the line.
+- Mark "Last updated" date and bump milestone status: M6 complete, next is
+  M7.
+- Append a `CHANGELOG.md` entry per CLAUDE.md convention (`date "+%Y-%m-%d %H:%M %Z"`).
+
+## Files touched
+
+| File | Change |
+| --- | --- |
+| [go/internal/web/assets_test.go](../../go/internal/web/assets_test.go) | **new** — two tests (embed completeness + `/static/*` mux) |
+| [docs/plans/2026-05-03-2349-go-backend-rewrite-progress.md](2026-05-03-2349-go-backend-rewrite-progress.md) | tick M6.7, bump "last updated" |
+| [CHANGELOG.md](../../CHANGELOG.md) | new top entry |
+
+No production source files change. (If extracting the static handler
+reads cleaner, a 4-line refactor in
+[server.go](../../go/internal/web/server.go) is acceptable but optional.)
+
+## Branch + MR
+
+Per project convention this is a feature, so:
+
+```bash
+git checkout -b feat/go-m6-7-embed-verify
+# … commits …
+git push -u origin feat/go-m6-7-embed-verify
+tea pr create --title "feat(go): M6.7 — single-binary embed verification" \
+  --description "<short body referencing M6.7>" --base main \
+  --head feat/go-m6-7-embed-verify
+```
+
+## Verification
+
+After implementation:
+
+1. `make go-test` → green (new `TestEmbedCompleteness` and `TestStaticAssetsServed` pass).
+2. `make go-lint` → clean.
+3. Run the manual transcript in §2 above — all curls succeed, no
+   "template not found" or 404 on static assets.
+4. `make go-build && docker build -f go/build/Dockerfile -t fuj-go:m6-7 go/` →
+   succeeds; `docker run --rm -p 8080:8080 fuj-go:m6-7` serves `/adults`
+   with stylesheet attached (visual smoke test in browser).
+
+## Out of scope
+
+- Re-architecting how templates are parsed or served.
+- Compressing / fingerprinting static assets (a separate concern).
+- Live integration test with real Sheets data — covered later in M7.

docs/plans/2026-05-11-2353-fill-first-multi-month-allocation.md

@@ -1,184 +0,0 @@
-# Fill-first multi-month payment allocation
-
-## Context
-
-Matyáš Thér paid in two transactions:
-
-| # | Amount | Purpose             |
-|---|--------|---------------------|
-| 1 | 200    | `2026-02`           |
-| 2 | 550    | `2026-02, 2026-03`  |
-
-Total 750 = his expected fee for the two months (likely 2026-02 = 500, 2026-03 = 250 — junior tier or exception-adjusted). The app currently shows 2026-02 = **566** paid, 2026-03 = **183** paid. The user wants:
-
-> First use the second payment for the rest of 2026-02 (no more, no less), then put the remainder toward 2026-03.
-
-Both Python ([scripts/match_payments.py](scripts/match_payments.py)) and Go ([go/internal/domain/reconcile/reconcile.go](go/internal/domain/reconcile/reconcile.go)) have the same bug: when a single transaction's `in_window_share` is less than the sum of in-window expected fees, both fall into a **proportional** branch that splits the new transaction across months in proportion to each month's *total* expected fee — never consulting what prior transactions already paid into earlier months.
-
-Trace for txn 2:
-
-- `in_window = [(2026-02, 500), (2026-03, 250)]`, `total_expected = 750`, `in_window_share = 550`.
-- `550 < 750` → proportional:
-  - 02 alloc = `550 × 500 / 750 = 366.67`
-  - 03 alloc = `550 − 366.67 = 183.33`
-- Combined with txn 1's 200 → 02 = 566.67, 03 = 183.33 → display `566 / 183`. ✓ matches reported numbers.
-
-The fix: replace the greedy + proportional branches with a single **fill-first** loop that iterates `in_window` in user-supplied order (already chronological by convention from [scripts/infer_payments.py:151](scripts/infer_payments.py#L151)) and allocates `min(remaining, max(0, expected − paid_so_far))` to each month, with any final surplus going to the credit bucket. This collapses three cases (greedy / proportional / pure overflow) into one and naturally consults the ledger's `paid` field which is already updated by prior transactions in the same reconcile pass.
-
-The even-split branch (`total_expected == 0`, prepayment before fees known) stays untouched — different semantic, folding it in would silently change behavior.
-
-## Changes
-
-### Python — [scripts/match_payments.py:471-498](scripts/match_payments.py#L471-L498)
-
-Replace both `total_expected > 0` branches (current lines 471–498) with a single unified loop. Keep lines 466–469 above and the even-split fallback below (lines 499–510) as-is.
-
-```python
-            if total_expected > 0:
-                # Fill-first: iterate in_window in matched_months order (chronological by
-                # convention), allocate min(remaining, deficit) to each month, where
-                # deficit accounts for prior transactions already credited to that month.
-                # Any surplus after all in-window deficits are covered → credit bucket.
-                remaining = in_window_share
-                for m, exp in in_window:
-                    paid_so_far = ledger[member_name][m]["paid"]
-                    deficit = max(0.0, float(exp) - paid_so_far)
-                    alloc = min(remaining, deficit)
-                    if alloc <= 0:
-                        continue
-                    ledger[member_name][m]["paid"] += alloc
-                    ledger[member_name][m]["transactions"].append({
-                        "amount": alloc,
-                        "date": tx["date"],
-                        "sender": tx["sender"],
-                        "message": tx["message"],
-                        "confidence": confidence,
-                    })
-                    remaining -= alloc
-                if remaining > 0:
-                    credits[member_name] = credits.get(member_name, 0) + int(remaining)
-            else:
-                # … existing even-split branch (lines 499–510) unchanged …
-```
-
-Note: skipping the `transactions.append` when `alloc <= 0` (e.g. month already fully paid by a prior txn) avoids zero-amount ghost rows in the per-month transaction list. This is a small UI-visible side effect; before committing, grep tests for assertions on `len(transactions)` per month to confirm nothing relies on the current "one row per (txn, month) regardless of alloc" behavior.
-
-### Go — [go/internal/domain/reconcile/reconcile.go:320-357](go/internal/domain/reconcile/reconcile.go#L320-L357)
-
-Same shape — replace both `totalExpected > 0` branches. Even-split branch (lines 358–372) stays.
-
-```go
-            if totalExpected > 0 {
-                // Fill-first; see Python reconcile() for rationale.
-                remaining := inWindowShare
-                for _, mw := range inWindow {
-                    md := ledger[memberName][mw.month]
-                    deficit := float64(mw.expected) - md.Paid
-                    if deficit < 0 {
-                        deficit = 0
-                    }
-                    alloc := remaining
-                    if deficit < alloc {
-                        alloc = deficit
-                    }
-                    if alloc <= 0 {
-                        continue
-                    }
-                    md.Paid += alloc
-                    md.Transactions = append(md.Transactions, TxEntry{
-                        Amount:     alloc,
-                        Date:       tx.Date,
-                        Sender:     tx.Sender,
-                        Message:    tx.Message,
-                        Confidence: string(m.Confidence),
-                    })
-                    ledger[memberName][mw.month] = md
-                    remaining -= alloc
-                }
-                if remaining > 0 {
-                    credits[memberName] += int(remaining)
-                }
-            } else {
-                // … existing even-split branch (lines 358–372) unchanged …
-            }
-```
-
-### Tests
-
-#### Python — [tests/test_reconcile_exceptions.py](tests/test_reconcile_exceptions.py)
-
-1. **Rewrite `test_proportional_underpayment`** ([line 96](tests/test_reconcile_exceptions.py#L96)) — its current assertions (`paid_02 < 750`, `paid_03 < 350`, `paid_04 < 750`, and 02/04 equal allocation) are incompatible with fill-first. Under fill-first with the same fixture (1250 across `02:750, 03:350, 04:750`):
-   - 02: `min(1250, 750) = 750` (full) → remaining 500
-   - 03: `min(500, 350) = 350` (full) → remaining 150
-   - 04: `min(150, 750) = 150` (partial) → remaining 0
-   
-   Replace assertions with these exact expected values, rename to `test_underpayment_fills_earliest_first`.
-
-2. **Add `test_fill_first_across_two_transactions`** — the Matyáš regression:
-
-   ```python
-   def test_fill_first_across_two_transactions(self):
-       """Prior txn fills 02 partially; later txn finishes 02 then spills to 03."""
-       members = [('Matyáš', 'A', {'2026-02': (500, 2), '2026-03': (250, 1)})]
-       sorted_months = ['2026-02', '2026-03']
-       tx1 = _tx('Matyáš', '2026-02', 200)
-       tx2 = _tx('Matyáš', '2026-02, 2026-03', 550)
-
-       result = reconcile(members, sorted_months, [tx1, tx2])
-       months = result['members']['Matyáš']['months']
-
-       self.assertAlmostEqual(months['2026-02']['paid'], 500, places=2)
-       self.assertAlmostEqual(months['2026-03']['paid'], 250, places=2)
-   ```
-
-3. `test_greedy_exact_match`, `test_greedy_overpayment_goes_to_credit`, `test_single_month_unchanged`, `test_two_members_multi_month` should pass unchanged — fill-first agrees with greedy when payment ≥ total expected.
-
-#### Go — [go/internal/domain/reconcile/reconcile_test.go](go/internal/domain/reconcile/reconcile_test.go)
-
-- Add `TestUnderpaymentFillsEarliestFirst` mirroring the rewritten Python test.
-- Add `TestFillFirstAcrossTwoTransactions` mirroring the Matyáš scenario.
-
-#### Parity — [go/tests/parity/reconcile/reconcile_parity_test.go](go/tests/parity/reconcile/reconcile_parity_test.go)
-
-Add a fixture for the Matyáš two-transaction case. Since both implementations change together and Python remains canonical, existing parity fixtures should continue to pass; verify after edits.
-
-### Changelog — [CHANGELOG.md](CHANGELOG.md)
-
-Append top entry (use `date "+%Y-%m-%d %H:%M %Z"` at commit time):
-
-```markdown
-## 2026-05-11 23:55 CET — fill-first multi-month payment allocation
-
-- Multi-month payment allocation now fills the earliest in-window deficit first
-  and spills the remainder to later months, accounting for prior transactions'
-  contributions. Previously a single transaction was split proportionally to
-  each month's total expected fee, ignoring earlier payments — surfaced by
-  Matyáš Thér's two-payment 200+550 case showing 566/183 instead of 500/250.
-- scripts/match_payments.py, go/internal/domain/reconcile/reconcile.go, tests.
-```
-
-## Critical files
-
-- [scripts/match_payments.py](scripts/match_payments.py) — Python reconcile (canonical)
-- [go/internal/domain/reconcile/reconcile.go](go/internal/domain/reconcile/reconcile.go) — Go reconcile (mirrors Python)
-- [tests/test_reconcile_exceptions.py](tests/test_reconcile_exceptions.py) — rewrite `test_proportional_underpayment` + add Matyáš test
-- [go/internal/domain/reconcile/reconcile_test.go](go/internal/domain/reconcile/reconcile_test.go) — add Go tests
-- [go/tests/parity/reconcile/reconcile_parity_test.go](go/tests/parity/reconcile/reconcile_parity_test.go) — add parity fixture
-- [CHANGELOG.md](CHANGELOG.md) — top entry
-
-## Verification
-
-1. `make test` — Python unit tests including the rewritten + new fill-first tests.
-2. `cd go && go test ./internal/domain/reconcile/... ./tests/parity/reconcile/...` — Go unit + parity tests.
-3. `make web` → load `/adults` or `/juniors` (whichever lists Matyáš Thér) → his 2026-02 row should be fully paid (no shortfall), 2026-03 fully paid, no leftover credit.
-4. Spot-check one other multi-month-purpose member to make sure fully-covered cases still look right.
-
-## Branch & MR (per CLAUDE.md)
-
-1. `git checkout -b fix/fill-first-multi-month-allocation`
-2. Apply edits + tests + CHANGELOG entry.
-3. `make test && (cd go && go test ./...)` — both green.
-4. Commit: `fix(reconcile): fill earliest month deficit first in multi-month allocations` with `Co-Authored-By` trailer.
-5. `git push -u origin fix/fill-first-multi-month-allocation`
-6. `tea pr create --title "fix(reconcile): fill earliest month deficit first" --description "<body>" --base main --head fix/fill-first-multi-month-allocation`
-7. Print MR URL. Do not merge from CLI.

go/internal/domain/reconcile/reconcile.go

@@ -115,11 +115,10 @@ type monthExpected struct {
 	expected int
 }
 
-// Reconcile matches transactions to members and months using two allocation phases:
+// Reconcile matches transactions to members and months using three allocation phases:
-//  1. Fill-first: iterate matched months in user-supplied order, allocating min(remaining,
+//  1. Greedy: payment ≥ total expected → fill each month exactly; overflow → credit.
-//     deficit) to each month where deficit = expected − already-paid. Surplus → credit.
+//  2. Proportional: payment < total → distribute by each month's share; last absorbs float remainder.
-//     Handles both the "greedy" (payment covers all) and "partial" cases in one pass.
+//  3. Even-split fallback: all expected fees are 0 (prepayment) → divide equally.
-//  2. Even-split fallback: all expected fees are 0 (prepayment) → divide equally.
 //
 // defaultYear seeds czech.ParseMonthReferences in the inference fallback.
 // Pass time.Now().Year() in production; pass a fixed year in tests.
@@ -318,26 +317,12 @@ func Reconcile(
 				totalExpected += mw.expected
 			}
 
-			if totalExpected > 0 {
+			if totalExpected > 0 && inWindowShare >= float64(totalExpected) {
-				// Fill-first: iterate inWindow in matched-months order (chronological by
+				// Greedy: payment covers all expected fees; overflow → credit
-				// convention), allocating min(remaining, deficit) to each month. Deficit
+				credits[memberName] += int(inWindowShare - float64(totalExpected))
-				// is net of what prior transactions already paid, so a second payment on
-				// the same months correctly fills only what remains due. Any surplus after
-				// all deficits are covered goes to the credit bucket.
-				remaining := inWindowShare
 				for _, mw := range inWindow {
+					alloc := float64(mw.expected)
 					md := ledger[memberName][mw.month]
-					deficit := float64(mw.expected) - md.Paid
-					if deficit < 0 {
-						deficit = 0
-					}
-					alloc := remaining
-					if deficit < alloc {
-						alloc = deficit
-					}
-					if alloc <= 0 {
-						continue
-					}
 					md.Paid += alloc
 					md.Transactions = append(md.Transactions, TxEntry{
 						Amount:     alloc,
@@ -347,10 +332,28 @@ func Reconcile(
 						Confidence: string(m.Confidence),
 					})
 					ledger[memberName][mw.month] = md
-					remaining -= alloc
 				}
-				if remaining > 0 {
+			} else if totalExpected > 0 {
-					credits[memberName] += int(remaining)
+				// Proportional: distribute by each month's share; last month absorbs float remainder
+				remaining := inWindowShare
+				for i, mw := range inWindow {
+					var alloc float64
+					if i == len(inWindow)-1 {
+						alloc = remaining
+					} else {
+						alloc = inWindowShare * float64(mw.expected) / float64(totalExpected)
+					}
+					remaining -= alloc
+					md := ledger[memberName][mw.month]
+					md.Paid += alloc
+					md.Transactions = append(md.Transactions, TxEntry{
+						Amount:     alloc,
+						Date:       tx.Date,
+						Sender:     tx.Sender,
+						Message:    tx.Message,
+						Confidence: string(m.Confidence),
+					})
+					ledger[memberName][mw.month] = md
 				}
 			} else {
 				// Even-split fallback: prepayment before attendance recorded

go/internal/domain/reconcile/reconcile_test.go

@@ -111,26 +111,36 @@ func TestReconcileGreedyOverpaymentGoesToCredit(t *testing.T) {
 	}
 }
 
-func TestReconcileUnderpaymentFillsEarliestFirst(t *testing.T) {
+func TestReconcileProportionalUnderpayment(t *testing.T) {
 	t.Parallel()
 	members := []Member{{
 		Name: "Alice", Tier: "A",
 		Fees: map[string]FeeData{"2026-02": {Expected: 750, Attendance: 3}, "2026-03": {Expected: 350, Attendance: 3}, "2026-04": {Expected: 750, Attendance: 3}},
 	}}
 	sortedMonths := []string{"2026-02", "2026-03", "2026-04"}
+	amount := 1250.0
 
-	result := Reconcile(members, sortedMonths, []Transaction{tx("Alice", "2026-02, 2026-03, 2026-04", 1250)}, nil, defaultYear)
+	result := Reconcile(members, sortedMonths, []Transaction{tx("Alice", "2026-02, 2026-03, 2026-04", amount)}, nil, defaultYear)
 
 	months := result.Members["Alice"].Months
-	// 02 filled first (750), then 03 (350), then remainder 150 to 04
+	paid02 := months["2026-02"].Paid
-	if math.Abs(months["2026-02"].Paid-750) > 0.01 {
+	paid03 := months["2026-03"].Paid
-		t.Errorf("02: want 750, got %f", months["2026-02"].Paid)
+	paid04 := months["2026-04"].Paid
+
+	if paid02 >= 750 {
+		t.Errorf("2026-02 should be underpaid, got %f", paid02)
 	}
-	if math.Abs(months["2026-03"].Paid-350) > 0.01 {
+	if paid03 >= 350 {
-		t.Errorf("03: want 350, got %f", months["2026-03"].Paid)
+		t.Errorf("2026-03 should be underpaid, got %f", paid03)
 	}
-	if math.Abs(months["2026-04"].Paid-150) > 0.01 {
+	if paid04 >= 750 {
-		t.Errorf("04: want 150, got %f", months["2026-04"].Paid)
+		t.Errorf("2026-04 should be underpaid, got %f", paid04)
+	}
+	if math.Abs(paid02+paid03+paid04-amount) > 0.01 {
+		t.Errorf("sum of paid want %f, got %f", amount, paid02+paid03+paid04)
+	}
+	if math.Abs(paid02-paid04) > 0.01 {
+		t.Errorf("02 and 04 have equal expected, want equal paid: %f vs %f", paid02, paid04)
 	}
 }
 
@@ -364,52 +374,3 @@ func TestReconcileNoTransactionsAllUnpaid(t *testing.T) {
 		t.Errorf("no txs: want empty unmatched, got %v", result.Unmatched)
 	}
 }
-
-// Payment < total expected → fill earliest months first, spill remainder to later.
-func TestUnderpaymentFillsEarliestFirst(t *testing.T) {
-	t.Parallel()
-	members := []Member{{Name: "Alice", Tier: "A", Fees: map[string]FeeData{
-		"2026-02": {Expected: 750, Attendance: 3},
-		"2026-03": {Expected: 350, Attendance: 3},
-		"2026-04": {Expected: 750, Attendance: 3},
-	}}}
-	txs := []Transaction{tx("Alice", "2026-02, 2026-03, 2026-04", 1250)}
-
-	result := Reconcile(members, []string{"2026-02", "2026-03", "2026-04"}, txs, nil, defaultYear)
-	months := result.Members["Alice"].Months
-
-	// 02 filled first (750), then 03 (350), then remainder 150 to 04
-	if math.Abs(months["2026-02"].Paid-750) > 0.01 {
-		t.Errorf("02: want 750, got %f", months["2026-02"].Paid)
-	}
-	if math.Abs(months["2026-03"].Paid-350) > 0.01 {
-		t.Errorf("03: want 350, got %f", months["2026-03"].Paid)
-	}
-	if math.Abs(months["2026-04"].Paid-150) > 0.01 {
-		t.Errorf("04: want 150, got %f", months["2026-04"].Paid)
-	}
-}
-
-// Prior txn fills 02 partially; later txn finishes 02 then spills to 03.
-func TestFillFirstAcrossTwoTransactions(t *testing.T) {
-	t.Parallel()
-	members := []Member{{Name: "Matyáš", Tier: "A", Fees: map[string]FeeData{
-		"2026-02": {Expected: 500, Attendance: 2},
-		"2026-03": {Expected: 250, Attendance: 1},
-	}}}
-	sortedMonths := []string{"2026-02", "2026-03"}
-	txs := []Transaction{
-		tx("Matyáš", "2026-02", 200),
-		tx("Matyáš", "2026-02, 2026-03", 550),
-	}
-
-	result := Reconcile(members, sortedMonths, txs, nil, defaultYear)
-	months := result.Members["Matyáš"].Months
-
-	if math.Abs(months["2026-02"].Paid-500) > 0.01 {
-		t.Errorf("02: want 500, got %f", months["2026-02"].Paid)
-	}
-	if math.Abs(months["2026-03"].Paid-250) > 0.01 {
-		t.Errorf("03: want 250, got %f", months["2026-03"].Paid)
-	}
-}

go/internal/web/assets_test.go

@@ -0,0 +1,93 @@
+package web
+
+import (
+	"io/fs"
+	"net/http"
+	"net/http/httptest"
+	"os"
+	"strings"
+	"testing"
+)
+
+// TestEmbedCompleteness guards against a new template or static file being
+// added to disk but missing from the embedded FS (e.g. a new directory that
+// the //go:embed glob does not match).
+func TestEmbedCompleteness(t *testing.T) {
+	cases := []struct {
+		name      string
+		diskDir   string
+		embedFS   fs.FS
+		embedRoot string
+	}{
+		{"templates", "templates", templateFS, "templates"},
+		{"static", "static", staticFS, "static"},
+	}
+
+	for _, tc := range cases {
+		t.Run(tc.name, func(t *testing.T) {
+			diskFS := os.DirFS(tc.diskDir)
+			_ = fs.WalkDir(diskFS, ".", func(path string, d fs.DirEntry, err error) error {
+				if err != nil || d.IsDir() {
+					return err
+				}
+				embPath := tc.embedRoot + "/" + path
+				if _, statErr := fs.Stat(tc.embedFS, embPath); statErr != nil {
+					t.Errorf("file %q exists on disk but is missing from embed.FS (%v)", embPath, statErr)
+				}
+				return nil
+			})
+		})
+	}
+}
+
+// TestStaticAssetsServed verifies that /static/* is served from the embedded
+// FS through the same mux wiring used in server.go, so a standalone binary
+// with no adjacent static/ directory still delivers assets.
+func TestStaticAssetsServed(t *testing.T) {
+	subFS, err := fs.Sub(staticFS, "static")
+	if err != nil {
+		t.Fatalf("fs.Sub static: %v", err)
+	}
+	mux := http.NewServeMux()
+	mux.Handle("/static/", http.StripPrefix("/static/", http.FileServerFS(subFS)))
+
+	cases := []struct {
+		path        string
+		wantCT      string
+		wantSnippet string
+	}{
+		{"/static/css/app.css", "text/css", "body {"},
+		{"/static/js/member-detail.js", "javascript", "Member-detail modal"},
+		{"/static/js/filters.js", "javascript", ""},
+		{"/static/js/payment-qr.js", "javascript", ""},
+	}
+
+	for _, tc := range cases {
+		t.Run(tc.path, func(t *testing.T) {
+			req := httptest.NewRequest(http.MethodGet, tc.path, nil)
+			w := httptest.NewRecorder()
+			mux.ServeHTTP(w, req)
+
+			if w.Code != http.StatusOK {
+				t.Fatalf("GET %s: status %d, want 200", tc.path, w.Code)
+			}
+			ct := w.Header().Get("Content-Type")
+			if !strings.Contains(ct, tc.wantCT) {
+				t.Errorf("GET %s: Content-Type %q, want it to contain %q", tc.path, ct, tc.wantCT)
+			}
+			if tc.wantSnippet != "" && !strings.Contains(w.Body.String(), tc.wantSnippet) {
+				t.Errorf("GET %s: body missing expected snippet %q", tc.path, tc.wantSnippet)
+			}
+		})
+	}
+
+	// Sanity: unknown path → 404 (file server doesn't fall through silently)
+	t.Run("missing-file", func(t *testing.T) {
+		req := httptest.NewRequest(http.MethodGet, "/static/css/nonexistent.css", nil)
+		w := httptest.NewRecorder()
+		mux.ServeHTTP(w, req)
+		if w.Code != http.StatusNotFound {
+			t.Errorf("unknown static path: status %d, want 404", w.Code)
+		}
+	})
+}

go/tests/fixtures/reconcile/03_proportional_remainder.json

@@ -1,7 +1,7 @@
 {
   "case": "03_proportional_remainder",
   "func": "scripts.match_payments.reconcile",
-  "captured_at": "2026-05-11",
+  "captured_at": "2026-05-06",
   "input": {
     "members": [
       {
@@ -54,10 +54,10 @@
             "original_expected": 750,
             "attendance_count": 3,
             "exception": null,
-            "paid": 750.0,
+            "paid": 324.3243243243243,
             "transactions": [
               {
-                "amount": 750.0,
+                "amount": 324.3243243243243,
                 "date": "2026-03-10",
                 "sender": "Member_d035d9f9",
                 "message": "",
@@ -70,10 +70,10 @@
             "original_expected": 750,
             "attendance_count": 2,
             "exception": null,
-            "paid": 50.0,
+            "paid": 324.3243243243243,
             "transactions": [
               {
-                "amount": 50.0,
+                "amount": 324.3243243243243,
                 "date": "2026-03-10",
                 "sender": "Member_d035d9f9",
                 "message": "",
@@ -86,17 +86,25 @@
             "original_expected": 350,
             "attendance_count": 2,
             "exception": null,
-            "paid": 0,
+            "paid": 151.35135135135135,
-            "transactions": []
+            "transactions": [
+              {
+                "amount": 151.35135135135135,
+                "date": "2026-03-10",
+                "sender": "Member_d035d9f9",
+                "message": "",
+                "confidence": "auto"
+              }
+            ]
           }
         },
         "other_transactions": [],
-        "total_balance": -1050
+        "total_balance": -1051
       }
     },
     "unmatched": [],
     "credits": {
-      "Member_d035d9f9": -1050
+      "Member_d035d9f9": -1051
     }
   }
 }

go/tests/fixtures/reconcile/09_multiperson_multimonth.json

@@ -1,7 +1,7 @@
 {
   "case": "09_multiperson_multimonth",
   "func": "scripts.match_payments.reconcile",
-  "captured_at": "2026-05-11",
+  "captured_at": "2026-05-06",
   "input": {
     "members": [
       {
@@ -63,10 +63,10 @@
             "original_expected": 750,
             "attendance_count": 3,
             "exception": null,
-            "paid": 750.0,
+            "paid": 500.0,
             "transactions": [
               {
-                "amount": 750.0,
+                "amount": 500.0,
                 "date": "2026-02-15",
                 "sender": "Member_d035d9f9",
                 "message": "",
@@ -79,10 +79,10 @@
             "original_expected": 750,
             "attendance_count": 2,
             "exception": null,
-            "paid": 250.0,
+            "paid": 500.0,
             "transactions": [
               {
-                "amount": 250.0,
+                "amount": 500.0,
                 "date": "2026-02-15",
                 "sender": "Member_d035d9f9",
                 "message": "",
@@ -102,10 +102,10 @@
             "original_expected": 750,
             "attendance_count": 2,
             "exception": null,
-            "paid": 750.0,
+            "paid": 681.8181818181819,
             "transactions": [
               {
-                "amount": 750.0,
+                "amount": 681.8181818181819,
                 "date": "2026-02-15",
                 "sender": "Member_d035d9f9",
                 "message": "",
@@ -118,10 +118,10 @@
             "original_expected": 350,
             "attendance_count": 2,
             "exception": null,
-            "paid": 250.0,
+            "paid": 318.18181818181813,
             "transactions": [
               {
-                "amount": 250.0,
+                "amount": 318.18181818181813,
                 "date": "2026-02-15",
                 "sender": "Member_d035d9f9",
                 "message": "",
@@ -131,13 +131,13 @@
           }
         },
         "other_transactions": [],
-        "total_balance": -100
+        "total_balance": -101
       }
     },
     "unmatched": [],
     "credits": {
       "Member_d035d9f9": -500,
-      "Member_f4a93e46": -100
+      "Member_f4a93e46": -101
     }
   }
 }

scripts/fio_utils.py

@@ -4,7 +4,6 @@
 import json
 import os
 import re
-import sys
 import urllib.request
 from datetime import datetime
 from html.parser import HTMLParser
@@ -90,11 +89,9 @@ def parse_czech_amount(s: str) -> float | None:
 
 
 def parse_czech_date(s: str) -> str | None:
-    """Parse a Czech date to 'YYYY-MM-DD'. Accepts 4-digit and 2-digit years
+    """Parse 'DD.MM.YYYY' to 'YYYY-MM-DD'."""
-    with dot or slash separators; Fio's transparent page mixes 'DD.MM.YYYY'
-    and 'DD.MM.YY' in the same response."""
     s = s.strip()
-    for fmt in ("%d.%m.%Y", "%d/%m/%Y", "%d.%m.%y", "%d/%m/%y"):
+    for fmt in ("%d.%m.%Y", "%d/%m/%Y"):
         try:
             return datetime.strptime(s, fmt).strftime("%Y-%m-%d")
         except ValueError:
@@ -149,7 +146,6 @@ def fetch_transactions_transparent(
             "bank_id": "", # HTML scraping doesn't give stable ID
         })
 
-    print(f"fio: transparent fetched {len(rows)} raw rows, {len(transactions)} transaction(s) after filtering", file=sys.stderr)
     return transactions
 
 
@@ -173,8 +169,7 @@ def fetch_transactions_api(
 
     transactions = []
     tx_list = data.get("accountStatement", {}).get("transactionList", {})
-    raw_list = tx_list.get("transaction") or []
+    for tx in (tx_list.get("transaction") or []):
-    for tx in raw_list:
         # Each field is {"value": ..., "name": ..., "id": ...} or null
         def val(col_id):
             col = tx.get(f"column{col_id}")
@@ -202,7 +197,6 @@ def fetch_transactions_api(
             "currency": str(val(14) or "CZK"), # column14 = Currency
         })
 
-    print(f"fio: api fetched {len(raw_list)} raw transaction(s), {len(transactions)} after filtering", file=sys.stderr)
     return transactions
 
 
@@ -210,14 +204,8 @@ def fetch_transactions(date_from: str, date_to: str) -> list[dict]:
     """Fetch transactions, using API if token available, else transparent page."""
     token = os.environ.get("FIO_API_TOKEN", "").strip()
     if token:
-        print(f"fio: using authenticated API, window {date_from}..{date_to}", file=sys.stderr)
         return fetch_transactions_api(token, date_from, date_to)
 
-    print(
-        f"fio: using transparent page (FIO_API_TOKEN unset — expect publishing lag), "
-        f"window {date_from}..{date_to}, account=2800359168",
-        file=sys.stderr,
-    )
     # Convert YYYY-MM-DD to DD.MM.YYYY for the transparent page URL
     from_dt = datetime.strptime(date_from, "%Y-%m-%d")
     to_dt = datetime.strptime(date_to, "%Y-%m-%d")

scripts/match_payments.py

@@ -468,19 +468,11 @@ def reconcile(
 
             total_expected = sum(e for _, e in in_window)
 
-            if total_expected > 0:
+            if total_expected > 0 and in_window_share >= total_expected:
-                # Fill-first: iterate in_window in matched_months order (chronological by
+                # Greedy phase: payment covers all in-window fees; overflow → credit.
-                # convention from infer_payments.py), allocating min(remaining, deficit) to
+                credits[member_name] = credits.get(member_name, 0) + int(in_window_share - total_expected)
-                # each month. Deficit is net of what prior transactions already paid, so a
-                # second payment on the same months correctly fills only what remains due.
-                # Any surplus after all deficits are covered goes to the credit bucket.
-                remaining = in_window_share
                 for m, exp in in_window:
-                    paid_so_far = ledger[member_name][m]["paid"]
+                    alloc = float(exp)
-                    deficit = max(0.0, float(exp) - paid_so_far)
-                    alloc = min(remaining, deficit)
-                    if alloc <= 0:
-                        continue
                     ledger[member_name][m]["paid"] += alloc
                     ledger[member_name][m]["transactions"].append({
                         "amount": alloc,
@@ -489,9 +481,21 @@ def reconcile(
                         "message": tx["message"],
                         "confidence": confidence,
                     })
+            elif total_expected > 0:
+                # Proportional phase: distribute in_window_share by each month's expected fee.
+                # Last month absorbs any float remainder so the sum equals in_window_share exactly.
+                remaining = in_window_share
+                for i, (m, exp) in enumerate(in_window):
+                    alloc = remaining if i == len(in_window) - 1 else in_window_share * exp / total_expected
                     remaining -= alloc
-                if remaining > 0:
+                    ledger[member_name][m]["paid"] += alloc
-                    credits[member_name] = credits.get(member_name, 0) + int(remaining)
+                    ledger[member_name][m]["transactions"].append({
+                        "amount": alloc,
+                        "date": tx["date"],
+                        "sender": tx["sender"],
+                        "message": tx["message"],
+                        "confidence": confidence,
+                    })
             else:
                 # Fallback: no expected fees (prepayment before attendance recorded); even split.
                 per_month = in_window_share / len(in_window)

scripts/sync_fio_to_sheets.py

@@ -77,35 +77,6 @@ def generate_sync_id(tx: dict) -> str:
     return hashlib.sha256(raw_str.encode("utf-8")).hexdigest()
 
 
-def _trunc(s: str, n: int = 40) -> str:
-    s = str(s)
-    return s if len(s) <= n else s[: n - 1] + "…"
-
-
-def _print_fio_table(transactions: list[dict], statuses: list[str]) -> None:
-    headers = ["DATE", "AMOUNT", "SENDER", "VS", "MESSAGE", "BANKID", "STATUS"]
-    rows = [
-        [
-            str(tx.get("date", "")),
-            f"{float(tx.get('amount', 0)):.2f}",
-            str(tx.get("sender", "")),
-            str(tx.get("vs", "")),
-            _trunc(str(tx.get("message", ""))),
-            str(tx.get("bank_id", "")),
-            status,
-        ]
-        for tx, status in zip(transactions, statuses)
-    ]
-    widths = [
-        max(len(headers[i]), max((len(r[i]) for r in rows), default=0))
-        for i in range(len(headers))
-    ]
-    sep = "  "
-    print(sep.join(h.ljust(w) for h, w in zip(headers, widths)))
-    for row in rows:
-        print(sep.join(cell.ljust(w) for cell, w in zip(row, widths)))
-
-
 def sort_sheet_by_date(service, spreadsheet_id):
     """Sort the sheet by the Date column (Column B)."""
     # Get the sheet ID (gid) of the first sheet
@@ -133,21 +104,12 @@ def sort_sheet_by_date(service, spreadsheet_id):
     print("Sheet sorted by date.")
 
 
-def sync_to_sheets(
+def sync_to_sheets(spreadsheet_id: str, credentials_path: str, days: int = None, date_from_str: str = None, date_to_str: str = None, sort_by_date: bool = False):
-    spreadsheet_id: str,
-    credentials_path: str,
-    days: int = None,
-    date_from_str: str = None,
-    date_to_str: str = None,
-    sort_by_date: bool = False,
-    dry_run: bool = False,
-    print_fio_table: bool = False,
-):
     print(f"Connecting to Google Sheets using {credentials_path}...")
     service = get_sheets_service(credentials_path)
     sheet = service.spreadsheets()
 
-    # 1. Read existing sync IDs from Column K
+    # 1. Fetch existing IDs from Column G (last column in A-G range)
     print(f"Reading existing sync IDs from sheet...")
     try:
         result = sheet.values().get(
@@ -155,22 +117,19 @@ def sync_to_sheets(
             range="A1:K" # Include header and all columns to check Sync ID
         ).execute()
         values = result.get("values", [])
-
+        
         # Check and insert labels if missing
         if not values or values[0] != COLUMN_LABELS:
-            if dry_run:
+            print("Inserting column labels...")
-                print("Dry run: would write header row")
+            sheet.values().update(
-            else:
+                spreadsheetId=spreadsheet_id,
-                print("Inserting column labels...")
+                range="A1",
-                sheet.values().update(
+                valueInputOption="USER_ENTERED",
-                    spreadsheetId=spreadsheet_id,
+                body={"values": [COLUMN_LABELS]}
-                    range="A1",
+            ).execute()
-                    valueInputOption="USER_ENTERED",
-                    body={"values": [COLUMN_LABELS]}
-                ).execute()
             existing_ids = set()
         else:
-            # Sync ID is the last column (index 10)
+            # Sync ID is now the last column (index 10)
             existing_ids = {row[10] for row in values[1:] if len(row) > 10}
     except Exception as e:
         print(f"Error reading sheet (maybe empty?): {e}")
@@ -191,12 +150,8 @@ def sync_to_sheets(
     transactions = fetch_transactions(df_str, dt_str)
     print(f"Found {len(transactions)} transactions.")
 
-    if dry_run:
+    # 3. Filter for new transactions
-        print(f"Dry run: window {df_str} to {dt_str}, fetched {len(transactions)} transaction(s) from Fio")
-
-    # 3. Determine NEW/DUP for each transaction
     new_rows = []
-    tx_statuses = []
     for tx in transactions:
         sync_id = generate_sync_id(tx)
         if sync_id not in existing_ids:
@@ -214,48 +169,24 @@ def sync_to_sheets(
                 tx.get("bank_id", ""),
                 sync_id,
             ])
-            tx_statuses.append("NEW")
-        else:
-            tx_statuses.append("DUP")
-
-    # 4. Print table (before early-return so all transactions are shown including DUPs)
-    if print_fio_table and transactions:
-        _print_fio_table(transactions, tx_statuses)
 
     if not new_rows:
-        if dry_run:
+        print("No new transactions to sync.")
-            print("Dry run: would sync 0 new transaction(s).")
-        else:
-            print("No new transactions to sync.")
         return
 
-    # 5. Append to sheet or print dry-run would-write lines
+    # 4. Append to sheet
-    if dry_run:
+    print(f"Appending {len(new_rows)} new transactions to the sheet...")
-        for tx, status in zip(transactions, tx_statuses):
+    body = {"values": new_rows}
-            if status == "NEW":
+    sheet.values().append(
-                print(
+        spreadsheetId=spreadsheet_id,
-                    f"Dry run: would append"
+        range="A2", # Appends to the end of the sheet
-                    f" date={tx.get('date', '')}"
+        valueInputOption="USER_ENTERED",
-                    f" amount={tx.get('amount', '')}"
+        body=body
-                    f" sender={tx.get('sender', '')}"
+    ).execute()
-                    f" vs={tx.get('vs', '')}"
+    print("Sync completed successfully.")
-                    f" message={tx.get('message', '')}"
+
-                )
+    if sort_by_date:
-        if sort_by_date:
+        sort_sheet_by_date(service, spreadsheet_id)
-            print("Dry run: would sort by date")
-        print(f"Dry run: would sync {len(new_rows)} new transaction(s).")
-    else:
-        print(f"Appending {len(new_rows)} new transactions to the sheet...")
-        body = {"values": new_rows}
-        sheet.values().append(
-            spreadsheetId=spreadsheet_id,
-            range="A2", # Appends to the end of the sheet
-            valueInputOption="USER_ENTERED",
-            body=body
-        ).execute()
-        print("Sync completed successfully.")
-        if sort_by_date:
-            sort_sheet_by_date(service, spreadsheet_id)
 
 
 def main():
@@ -266,20 +197,16 @@ def main():
     parser.add_argument("--from", dest="date_from", help="Start date YYYY-MM-DD")
     parser.add_argument("--to", dest="date_to", help="End date YYYY-MM-DD")
     parser.add_argument("--sort-by-date", action="store_true", help="Sort the sheet by date after sync")
-    parser.add_argument("--dry-run", action="store_true", help="Fetch and dedup without writing to the sheet")
-    parser.add_argument("--print-fio-table", action="store_true", help="Print aligned table of all fetched transactions with NEW/DUP status (use with --dry-run)")
     args = parser.parse_args()
 
     try:
         sync_to_sheets(
-            spreadsheet_id=args.sheet_id,
+            spreadsheet_id=args.sheet_id, 
             credentials_path=args.credentials,
             days=args.days,
             date_from_str=args.date_from,
             date_to_str=args.date_to,
-            sort_by_date=args.sort_by_date,
+            sort_by_date=args.sort_by_date
-            dry_run=args.dry_run,
-            print_fio_table=args.print_fio_table,
         )
     except Exception as e:
         print(f"Sync failed: {e}")

tests/test_reconcile_exceptions.py

@@ -93,8 +93,8 @@ class TestMultiMonthAllocation(unittest.TestCase):
         self.assertEqual(int(months['2026-02']['paid']), 750)
         self.assertEqual(result['credits'].get('Alice', 0), 500)
 
-    def test_underpayment_fills_earliest_first(self):
+    def test_proportional_underpayment(self):
-        """Payment < total expected → fill earliest months first, spill remainder to later."""
+        """Payment < total expected → proportional split; sum of paid == payment amount."""
         members = [('Alice', 'A', {'2026-02': (750, 3), '2026-03': (350, 3), '2026-04': (750, 3)})]
         sorted_months = ['2026-02', '2026-03', '2026-04']
         amount = 1250
@@ -103,28 +103,18 @@ class TestMultiMonthAllocation(unittest.TestCase):
         result = reconcile(members, sorted_months, [tx])
         months = result['members']['Alice']['months']
 
-        # 02 filled first (750), then 03 (350), then remainder 150 to 04
+        paid_02 = months['2026-02']['paid']
-        self.assertAlmostEqual(months['2026-02']['paid'], 750, places=2)
+        paid_03 = months['2026-03']['paid']
-        self.assertAlmostEqual(months['2026-03']['paid'], 350, places=2)
+        paid_04 = months['2026-04']['paid']
-        self.assertAlmostEqual(months['2026-04']['paid'], 150, places=2)
-        # No CZK lost
-        self.assertAlmostEqual(
-            months['2026-02']['paid'] + months['2026-03']['paid'] + months['2026-04']['paid'],
-            amount, places=2,
-        )
 
-    def test_fill_first_across_two_transactions(self):
+        # All months should be partial (underpaid)
-        """Prior txn fills 02 partially; later txn finishes 02 then spills to 03."""
+        self.assertLess(paid_02, 750)
-        members = [('Matyáš', 'A', {'2026-02': (500, 2), '2026-03': (250, 1)})]
+        self.assertLess(paid_03, 350)
-        sorted_months = ['2026-02', '2026-03']
+        self.assertLess(paid_04, 750)
-        tx1 = _tx('Matyáš', '2026-02', 200)
+        # Sum must equal the original payment (no CZK lost)
-        tx2 = _tx('Matyáš', '2026-02, 2026-03', 550)
+        self.assertAlmostEqual(paid_02 + paid_03 + paid_04, amount, places=2)
-
+        # 02 and 04 have equal expected → equal allocation
-        result = reconcile(members, sorted_months, [tx1, tx2])
+        self.assertAlmostEqual(paid_02, paid_04, places=2)
-        months = result['members']['Matyáš']['months']
-
-        self.assertAlmostEqual(months['2026-02']['paid'], 500, places=2)
-        self.assertAlmostEqual(months['2026-03']['paid'], 250, places=2)
 
     def test_single_month_unchanged(self):
         """Single-month payment: full amount goes to that month (regression guard)."""