siddharthd
6e179d3a0a
ci / lint-test (push) Successful in 53s
The review was written 2026-07-26 and last touched 2026-07-29, one day before the board replaced ACTIONS.md, so it was never migrated. Priorities 1-4 existed only in this file. They are now epic #154 with #155-#159 under it. Two claims in the doc were wrong. It said Priority 0 was guarded by analytics-sql.test.ts; that file covers the three SQL fragments and nothing else, so items 2, 3 and 6 -- pace/headline parity, the fees date range, and partial-month comparison -- have no test at all. Those are the three where a regression is silent, which is why the gap is worth a ticket rather than a note. And it predated two changes in the same metric-integrity family: the signed investments line (c70d2b1) and transfers hidden by default (f6c500b). The first matters to the doc directly -- the cashflow strip it describes now carries a signed invested figure, so a negative month is data rather than a bug.