Skip to content

chore: test simulation cache with native sqlite#321407

Closed
deepak1556 wants to merge 5 commits into
mainfrom
robo/test_cache_with_native_sqlite
Closed

chore: test simulation cache with native sqlite#321407
deepak1556 wants to merge 5 commits into
mainfrom
robo/test_cache_with_native_sqlite

Conversation

@deepak1556

Copy link
Copy Markdown
Collaborator

No description provided.

@deepak1556 deepak1556 added this to the 1.125.0 milestone Jun 15, 2026
@deepak1556 deepak1556 self-assigned this Jun 15, 2026
Copilot AI review requested due to automatic review settings June 15, 2026 13:51

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the Copilot simulation test cache implementation to stop relying on native sqlite3 via keyv/@keyv/sqlite, and instead use Node’s built-in node:sqlite APIs. This reduces native dependency surface area for the test harness while keeping existing committed cache DBs usable.

Changes:

  • Replace keyv/@keyv/sqlite usage in the simulation cache with a minimal node:sqlite-backed key/value store.
  • Remove keyv and @keyv/sqlite from extensions/copilot dependencies and re-resolve the lockfile.
  • Update cache-layer enumeration logic to work with the new store.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 2 comments.

File Description
extensions/copilot/test/base/cache.ts Introduces a minimal node:sqlite store and swaps the simulation cache over from keyv.
extensions/copilot/package.json Drops keyv and @keyv/sqlite dependencies (and related overrides).
extensions/copilot/package-lock.json Removes the transitive native sqlite dependency tree and updates lock metadata.
Files not reviewed (1)
  • extensions/copilot/package-lock.json: Generated file

Comment thread extensions/copilot/test/base/cache.ts
Comment thread extensions/copilot/test/base/cache.ts
@deepak1556 deepak1556 closed this Jun 15, 2026
@deepak1556 deepak1556 deleted the robo/test_cache_with_native_sqlite branch June 15, 2026 16:28
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.

2 participants