Skip to content

fix: refresh shared indexer after knowledge base updates#53

Merged
Helweg merged 2 commits intomainfrom
fix/knowledge-base-indexer-refresh
Apr 13, 2026
Merged

fix: refresh shared indexer after knowledge base updates#53
Helweg merged 2 commits intomainfrom
fix/knowledge-base-indexer-refresh

Conversation

@Helweg
Copy link
Copy Markdown
Owner

@Helweg Helweg commented Apr 12, 2026

Summary

Refresh the shared tool indexer immediately after knowledge base config changes so in-session indexing and search use the updated configuration without requiring another restart.

Changes

  • rebuild the shared Indexer from parsed config after add_knowledge_base updates the project config
  • rebuild the shared Indexer after remove_knowledge_base so the in-memory tool state stays in sync
  • add a regression test covering both add/remove refresh behavior in tests/tools-knowledge-bases.test.ts

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

N/A

@Helweg Helweg added bug Bug fix semver:patch Bug fix or patch test Test changes labels Apr 12, 2026
@Helweg Helweg merged commit d29373d into main Apr 13, 2026
3 checks passed
@Helweg Helweg deleted the fix/knowledge-base-indexer-refresh branch April 14, 2026 15:52
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.

1 participant