Fix local transcript cache mtime writes#241
Conversation
- Include fileMtime when writing transcript cache entries - Add real cache-store regression coverage for local file transcripts
|
Codex review: needs real behavior proof before merge. Reviewed June 8, 2026, 1:02 AM ET / 05:02 UTC. Summary Reproducibility: unclear. The review failed before ClawSweeper could establish a reproduction path. Review metrics: none identified. Merge readiness Overall follows the weaker of proof and patch quality, so missing proof can cap an otherwise strong patch. Risk before merge
Maintainer options:
Next step before merge
Review detailsBest possible solution: Retry the Codex review after fixing the execution failure. Do we have a high-confidence way to reproduce the issue? Unclear. The review failed before ClawSweeper could establish a reproduction path. Is this the best way to solve the issue? Unclear. Retry the review first so ClawSweeper can evaluate the actual issue and fix direction. AGENTS.md: unclear because the file could not be read completely. Codex review notes: model gpt-5.5, reasoning high; reviewed against 769d01f0898f. Label changesLabel changes:
Label justifications:
Evidence reviewedWhat I checked:
Likely related people:
What the crustacean ranks mean
Shiny media proof means a screenshot, video, or linked artifact directly shows the changed behavior. Runtime, network, CSP, and security claims still need visible diagnostics. How this review workflow works
|
Fixes #240
Summary
fileMtimewhen writing transcript cache entriesTests
pnpm test tests/cache.store.test.ts tests/transcript.cache.filemtime.test.ts tests/cli.asset.audio-file-transcript-cache.test.tspnpm exec oxfmt --check src/cache.ts tests/cache.store.test.tspnpm typecheck