refactor(telemetry): simplest, deterministic, true

[blafourcade]

🎯 What & why

Simplifies the R1 telemetry code under one rule: the simplest design that stays deterministic and true. A Claude Code call with no requestId (0 of 265,472 usage lines measured, Claude Code 2.1.232 to 2.1.302; 0 of 512,820 before) is no longer stored. It is counted in coverage.left_out.requestless, so the code that kept such calls folded across days is deleted. The local-path guard becomes line-based. The process rule is written once.

🛠️ How it works

• One predicate, isRequestless (claude-transcript-usage.ts): a Claude Code key naming its own session, as this version (msg:session) or cli-v5.5.0 (msg:session:time) wrote it.
  • Ingest: the call is left out, hashed by tool, message and session (ingest-usage-use-case.ts).
  • Migration: one pass over the whole ledger moves what an earlier version stored into left-out.txt before its rewrite (stored-usage-migration.ts).
  • Every read: parseStoredUsage does not accept such a line. One an older CLI writes after the migration is kept as found and counted in run.skipped_ledger_lines, like any line this version does not accept. It is not a figure, so left_out never depends on the days a report reads.
• Deleted: isKeyedBySession, the full-ledger read in storeCalls, legacy-usage-key.ts (re-keying), the window's second bound, the older-CLI limit rows, the window property's exemption, the tests that existed only for them.
• Guard (check-no-local-paths.js): no array pass, no 64 KB cap, no offset-to-line search, no sort. Multi-line structures, and arrays of ids, are not parsed. The key-then-UUID form stayed: "any UUID on the key's line" flagged 46 tracked lines.
• Rule merged into "Before the code" (coding-assertions.md).

Lines: +383 / −412, net −29; without the task folder, +279 / −412, net −133. Bundle: 716,561 → 717,130 bytes (+569).

🧪 How to verify

• cd cli && npx vitest run tests/contexts/telemetry/application/report/report-window.property.integration.test.ts: a bounded report's figures (totals, rows, coverage) equal the full ledger's filtered, requestless and timestamped lines in the generator, no exemption.
• cd cli && npx vitest run tests/contexts/telemetry/application/legacy/stored-usage-migration.property.integration.test.ts: a migrated store equals one fresh ingest byte for byte, left-out.txt included.
• node scripts/check-no-local-paths.js --all: 0.
• Evidence, red runs and mutation: evidence.md.

⚠️ Heads-up

• coverage.left_out gains requestless, an addition allowed in envelope v2.
• A request-id line an older CLI appends after the upgrade out of time order is read right, but not rewritten into canonical order. That was already the case before this PR, and it is unchanged.
• Mutation survivor kept: usage-ledger-adapter.ts:90, the re-id unsettled mark, reachable only by an older CLI writing earlier ids after the upgrade.

🔗 Linked issue

None. Follows the R1 telemetry audit: simplest, deterministic, true.

✅ I certify

• I DO CERTIFY I READ EACH LINE OF THE PULL REQUEST BECAUSE I AM A SOFTWARE ENGINEER, NOT A AI PUPPY.

🤖 Generated with Claude Code

https://claude.ai/code/session_01VWNxk63AGKkqE8HRqHLjGp