fix: update dependencies, add tests, and improve validation #58
+3,880
−472
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- update @modelcontextprotocol/sdk 1.0.1 → 1.20.2 - update axios 1.6.2 → 1.13.1 - update dotenv 16.3.1 → 16.6.1 - update shx 0.3.4 → 0.4.0 - update typescript 5.6.2 → 5.9.3 - add vitest 4.0.5 for testing resolves npm warnings for deprecated [email protected] and [email protected] fixes 5 moderate severity vulnerabilities in vitest dependency chain
- validate choices array exists and is non-empty - validate message content is string type - add message structure validation in tool handlers - export functions for testing prevents null pointer crashes on malformed API responses follows MCP security best practices for zero-trust validation
unit tests: - formatSearchResults edge cases - performChatCompletion success and error paths - performSearch parameter handling - API response validation - timeout and error handling - concurrent requests - special characters and long content integration tests: - all four tool types (ask, research, reason, search) - real-world scenarios - multi-turn conversations - performance and reliability tests run against actual implementation via imports vitest configured with proper env vars and exclusions
27Bslash6
force-pushed
the
fix/deps-security-tests
branch
from
08f5599 to
c9e9d59
Compare
- Add coverage configuration to vitest for improved test reporting - Introduce a new GitHub Actions workflow for automated testing on push and pull requests - Update .gitignore to exclude coverage directory - Add @vitest/coverage-v8 as a development dependency for coverage analysis - Introduce a new test command for coverage reporting in package.json
kesku
approved these changes
Nov 4, 2025
|
Awesome work! Thank you very much, will include in next release |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changes
Dependency Updates
Resolves:
[email protected]and[email protected]Bug Fixes
Testing
Code Quality
Test Results
Build: ✓ Clean
Security: ✓ 0 vulnerabilities
MCP: ✓ All tools functional
Breaking Changes
None. All changes are backward compatible.