fix: @review searches with vector terms

[dustinbyrne]

This pull request integrates the VectorTermsService into the ReviewCommand to enhance the performance of context lookups by using vector terms instead of full text. This change addresses the issue of slow searches when reviewing code in pull requests. The integration involves updates to the command constructor and modifications to how context is processed during review analysis.

Key Changes

• Integration of VectorTermsService: The ReviewCommand constructor now accepts a VectorTermsService, suggesting terms based on provided code diffs and user prompts.
• Modified Context Lookup: Use vector terms for querying context to reduce the processing time and improve performance, making the review process more efficient.
• Test Updates: Adjusted existing tests and added tests in review-command.spec.ts to verify the functionality involving vector terms, ensuring that the implementation remains stable and reliable.

Review Summary

Upon review:

• Correctness: Verified that the integration correctly uses vector terms, improving performance without sacrificing functionality.
• Code Quality: Suggested improvements include adding more documentation for constructor parameters and ensuring methods are self-explanatory.
• Performance: The change is likely to meet performance goals, but real-world scenario monitoring is advised to confirm the anticipated improvements.
• Security: Suggested a review of the VectorTermsService for potential security vulnerabilities given its handling of user input.
• Testing: Encouraged expanding test coverage to include more edge cases and failure modes.
• Compatibility: Emphasized verifying that existing functionalities remain unaffected by these changes.

Resolves #2122

[appland-release]

🎉 This PR is included in version @appland/navie-v1.36.2 🎉

The release is available on:

• @appland/navie-v1.36.2
• GitHub release
• @appland/navie-v1.36.2

Your semantic-release bot 📦🚀