Skip to content

Fix DbReader schema cache reuse for issue #1701 - #2629

Merged
Widthdom merged 2 commits into
mainfrom
codex/fix-issue1701
May 24, 2026
Merged

Fix DbReader schema cache reuse for issue #1701#2629
Widthdom merged 2 commits into
mainfrom
codex/fix-issue1701

Conversation

@Widthdom

@Widthdom Widthdom commented May 24, 2026

Copy link
Copy Markdown
Owner

Summary

  • Add a process-level DB-path keyed schema cache for DbReader schema discovery.
  • Reuse cached table columns, index lists, and table-existence checks across DbContext instances while invalidating on SQLite schema_version changes.
  • Document the schema cache contract and add a bilingual changelog fragment.

Fixes #1701

Documentation / Changelog

  • Updated DEVELOPER_GUIDE.md.
  • Added changelog.d/unreleased/1701.fixed.md.

Validation

  • dotnet build
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --filter "FullyQualifiedName~DbSchemaCacheTests"
  • dotnet run --project tools/CodeIndex.Changelog -- check
  • dotnet test
  • dotnet ./src/CodeIndex/bin/Debug/net8.0/cdidx.dll status --check --json

Review

  • Adversarial review: No blocking/actionable issues found.

Follow-up candidates

  • None.

@Widthdom
Widthdom merged commit 668da3a into main May 24, 2026
10 checks passed
@Widthdom
Widthdom deleted the codex/fix-issue1701 branch May 25, 2026 22:17
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.

DbReader runs PRAGMA table_info/index_list at every reader creation; no per-DB schema cache

1 participant