Repository navigation
refactor: keep cache, reporting and SQLite history native - #1275
Merged
Merged
Conversation
Preserve existing cache snapshots and Python APIs while moving cache persistence and completed-test result consumers to native ownership. Amp-Thread-ID: https://ampcode.com/threads/T-01a18715-4e00-708e-a87d-6bd109748d66
kvlonge
enabled auto-merge (squash)
October 29, 2026 23:41
Construct reuse checks and compile records directly in Rust instead of round-tripping through Python containers. Reuse the cache artifact model while preserving Python-facing rows and launch-record strings. Validated with 65 focused unit and integration tests, cold/warm native launcher replay, and make check-ci. Existing launcher coverage caught and now guards the path serialization conversion. Amp-Thread-ID: https://ampcode.com/threads/T-01a18715-4e00-708e-a87d-6bd109748d66
Keep SQL-test plans, cached paths and managed paths native through preparation and publication. Preserve Python plan projection at the public API and platform path equality at target matching. Amp-Thread-ID: https://ampcode.com/threads/T-01a18715-4e00-708e-a87d-6bd109748d66
Keep cache payload objects, CTE arrays and sorted pairs as Rust JSON values. Convert record and adapter leaf values at the existing JSON boundary, preserving the version-5 digest format and scalar checks. Amp-Thread-ID: https://ampcode.com/threads/T-01a18715-4e00-708e-a87d-6bd109748d66
Convert executor audit results once, share projection metadata with build reporting, and remove the completed JSON parse/rewrite step. Preserve public formatting and warehouse history behavior. Amp-Thread-ID: https://ampcode.com/threads/T-01a18715-4e00-708e-a87d-6bd109748d66
Read scenario outcomes, comparisons, assertions and nested resource results at the executor edge. Retain typed data through summary, failure and JSON reporting while preserving public projections, sample values and runtime artifact publication. Amp-Thread-ID: https://ampcode.com/threads/T-01a18715-4e00-708e-a87d-6bd109748d66
Preserve reentrant callbacks and check Python signals during contended native waits. Retain connection hooks and transaction recovery, with a real SQLite cancellation regression. Amp-Thread-ID: https://ampcode.com/threads/T-01a18715-4e00-708e-a87d-6bd109748d66
Keep local history connections and query rows native while preserving the public storage API, connection overrides, transaction recovery and interruptible locking. Preserve literal filename semantics even when bundled SQLite enables URI interpretation globally. Cover persistence without Python connect and exact relative filenames through the public API. Amp-Thread-ID: T-01a18715-4e00-708e-a87d-6bd109748d66
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
Remove avoidable Python dependencies from caching, compile reuse and runtime bookkeeping while preserving supported Python execution and APIs.
Changes
Verification
make test: 12,185 passed after artifact-plan and identity migrations.make check-ci, bounded independent reviews and private hygiene scans passed.