From 594013c5ce989d17b0def4886a29a10f40d21e71 Mon Sep 17 00:00:00 2001 From: iamtoruk Date: Mon, 20 Apr 2026 18:08:38 -0700 Subject: [PATCH] docs: add cached-vs-uncached verification rule for parser PRs --- CLAUDE.md | 1 + 1 file changed, 1 insertion(+) diff --git a/CLAUDE.md b/CLAUDE.md index 1661a28..329d28c 100644 --- a/CLAUDE.md +++ b/CLAUDE.md @@ -5,6 +5,7 @@ - Run `npx tsx src/cli.ts report` and `npx tsx src/cli.ts today` to verify changes before any commit - For dashboard changes: run the interactive TUI and visually confirm rendering - For new features: test the happy path AND edge cases (empty data, missing config, pipe mode) +- For any PR touching parser, source cache, or data aggregation: run cached vs uncached comparison before merge. Costs must match within $0.10. Use: `parseAllSessions(range, 'all', { noCache: false })` vs `parseAllSessions(range, 'all', { noCache: true })` and compare totals. A mismatch means the cache is serving stale or incorrect data ## Code Quality - Clean, minimal code. No dead code, no commented-out blocks, no TODO placeholders