chore: follow-up on #626 MCP-246

[nirinchev]

Proposed changes

I didn't get a chance to complete the review of #626 before it got merged, so this is a follow-up with some suggestions. Most are stylistic changes, while the only substantial one is a change to the field validation logic to include more information in the error response.

[Copilot]

Pull Request Overview

This is a follow-up PR addressing review comments from #626, focusing on code style improvements and enhanced error reporting for vector field validation. The main change improves the validation error messages by including actual vs. expected dimensions and quantization details.

Key Changes

• Enhanced vector field validation to return detailed error objects instead of simple pass/fail
• Extracted shared enum definitions for similarity and quantization types to reduce duplication
• Minor code style improvements (removing redundant async/await, unnecessary returns, and improving variable names)

Reviewed Changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
src/common/search/vectorSearchEmbeddingsManager.ts	Refactored validation logic to return detailed error objects; extracted shared enum definitions for reuse
src/tools/mongodb/create/insertMany.ts	Updated error message formatting to include actual dimensions/quantization details
src/tools/mongodb/create/createIndex.ts	Replaced inline enum definitions with shared enums; added blank line for spacing
src/common/config.ts	Updated type definition to use shared Similarity type
src/common/session.ts	Improved variable naming and removed redundant return statement
src/tools/mongodb/mongodbTool.ts	Removed redundant async/await wrapper
tests/integration/tools/mongodb/create/insertMany.test.ts	Updated test expectation to match new detailed error message format
tests/unit/common/search/vectorSearchEmbeddingsManager.test.ts	Added assertions for detailed error object structure