Skip to content

test(cache): cover large and unreadable entries#556

Closed
wan9chi wants to merge 1 commit into
agent/cache-row-recoveryfrom
agent/cache-regression-coverage
Closed

test(cache): cover large and unreadable entries#556
wan9chi wants to merge 1 commit into
agent/cache-row-recoveryfrom
agent/cache-regression-coverage

Conversation

@wan9chi

@wan9chi wan9chi commented Jul 21, 2026

Copy link
Copy Markdown
Member

Motivation

The preallocation failure appears only after a large inferred-input cardinality. Unreadable-row recovery also depends on exact deletion and concurrency behavior that is easy to regress.

Changes

  • Exercise a real SQLite cache round-trip above wincode's former 4 MiB limit.
  • Cover malformed cache-entry and task-fingerprint values.
  • Verify cleanup cannot delete a concurrently replaced value of another SQLite type.
  • Cover recovery from an unreadable entry followed by a dangling fingerprint mapping.

wan9chi commented Jul 21, 2026

Copy link
Copy Markdown
Member Author

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

Co-authored-by: GPT-5.6 Codex <codex@openai.com>
@wan9chi
wan9chi force-pushed the agent/cache-regression-coverage branch from f9d2ab5 to a5e544a Compare July 21, 2026 03:56
@wan9chi
wan9chi force-pushed the agent/cache-row-recovery branch from a159706 to ffed460 Compare July 21, 2026 03:56
@wan9chi wan9chi closed this Jul 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant