Code Index (Qdrant) recreate services when change configurations

[catrielmuller]

Related GitHub Issue

Resolve: #814

Description

The code index manager it's not refreshing the configuration automatically causing frustration to the user don't know why it's not connecting correctly

This PR ensures that CodeIndex services are automatically recreated whenever relevant configurations change, improving connection reliability.

• Extracted service recreation logic into a private helper _recreateServices
• Replaced inline recreation in both initialization and external settings change handlers with the new helper

Test Procedure

• Configure the Code Index Qdrant with a wrong url
• Save
• This should be show an connection error msg
• Update the config with a valid url
• Save
• Now should connect correctly

Pre-Submission Checklist

• Issue Linked: This PR is linked to an approved GitHub Issue (see "Related GitHub Issue" above).
• Scope: My changes are focused on the linked issue (one major feature/fix per PR).
• Self-Review: I have performed a thorough self-review of my code.
• Testing: New and/or updated tests have been added to cover my changes (if applicable).
• Documentation Impact: I have considered if my changes require documentation updates (see "Documentation Updates" section below).
• Contribution Guidelines: I have read and agree to the Contributor Guidelines.

Screenshots / Videos

Documentation Updates

Does this PR necessitate updates to user-facing documentation?

• No documentation updates are required.
• Yes, documentation updates are required. (Please describe what needs to be updated or link to a PR in the docs

Additional Notes

Get in Touch

Discord: catrielmuller

---

Important

Introduces _recreateServices() in manager.ts to handle service recreation on configuration changes, updating initialize() and handleExternalSettingsChange() methods, and modifies tests accordingly.

• Behavior:
  • Introduces _recreateServices() in manager.ts to handle service recreation on configuration changes.
  • Replaces inline service recreation logic in initialize() and handleExternalSettingsChange() with _recreateServices().
• Tests:
  • Updates manager.spec.ts to mock _recreateServices() and verify its invocation during configuration changes.
  • Ensures handleExternalSettingsChange() does not throw errors when manager is uninitialized or config manager is unset.

^{This description was created by for 48043aa. You can customize this summary. It will automatically update as commits are pushed.}

[daniel-lxs]

Thank you @catrielmuller!

LGTM