Skip to content

fix: reset shared index state on force rebuild - #62

Merged
Helweg merged 15 commits into
mainfrom
fix/issue-61-clear-index-embeddings
Apr 21, 2026
Merged

fix: reset shared index state on force rebuild#62
Helweg merged 15 commits into
mainfrom
fix/issue-61-clear-index-embeddings

Conversation

@Helweg

@Helweg Helweg commented Apr 20, 2026

Copy link
Copy Markdown
Owner

Summary

Make force=true behave like a true shared index reset so provider/model dimension changes cannot reuse stale SQLite embeddings during the next rebuild.

Changes

  • add a native database reset path that clears shared embeddings, chunks, branch catalogs, symbols, and call edges
  • update Indexer.clearIndex() to wipe persisted shared index state and failed-batch state before rebuilding
  • add regression coverage for an in-process embedding dimension switch during a force rebuild

Testing

How were these changes tested?

  • Unit tests added/updated
  • Manual testing performed
  • Build passes (npm run build)
  • Typecheck passes (npm run typecheck)
  • Tests pass (npm run test:run)
  • Lint passes (npm run lint)

Release Labels

  • Added at least one release category label (feature, bug, performance, documentation, dependencies, refactor, test, chore, or skip-changelog)
  • Added at most one semver label (semver:major, semver:minor, semver:patch) when needed

Related Issues

Fixes #61

@Helweg Helweg added bug Bug fix test Test changes semver:patch Bug fix or patch labels Apr 20, 2026
@Helweg
Helweg merged commit ec9477c into main Apr 21, 2026
3 checks passed
@Helweg
Helweg deleted the fix/issue-61-clear-index-embeddings branch April 21, 2026 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Bug fix semver:patch Bug fix or patch test Test changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] clearIndex not clear embedding database

1 participant