Skip to content

fix(security): bump vulnerable packages#1466

Merged
PMerlet merged 9 commits intomainfrom
fix/high-vulnerabilities
Feb 12, 2026
Merged

fix(security): bump vulnerable packages#1466
PMerlet merged 9 commits intomainfrom
fix/high-vulnerabilities

Conversation

@PMerlet
Copy link
Contributor

@PMerlet PMerlet commented Feb 12, 2026

Definition of Done

General

  • Write an explicit title for the Pull Request, following Conventional Commits specification
  • Test manually the implemented changes
  • Validate the code quality (indentation, syntax, style, simplicity, readability)

Security

  • Consider the security impact of the changes made

@qltysh
Copy link

qltysh bot commented Feb 12, 2026

Qlty

Coverage Impact

⬆️ Merging this pull request will increase total coverage on main by 0.37%.

Modified Files with Diff Coverage (1)

RatingFile% DiffUncovered Line #s
Coverage rating: B Coverage rating: A
packages/mcp-server/src/server.ts100.0%
Total100.0%
🚦 See full report on Qlty Cloud »

🛟 Help
  • Diff Coverage: Coverage for added or modified lines of code (excludes deleted files). Learn more.

  • Total Coverage: Coverage for the whole repository, calculated as the sum of all File Coverage. Learn more.

  • File Coverage: Covered Lines divided by Covered Lines plus Missed Lines. (Excludes non-executable lines including blank lines and comments.)

    • Indirect Changes: Changes to File Coverage for files that were not modified in this PR. Learn more.

@qltysh
Copy link

qltysh bot commented Feb 12, 2026

1 new issue

Tool Category Rule Count
qlty Structure Function with many parameters (count = 5): registerToolWithLogging 1

PMerlet and others added 2 commits February 12, 2026 15:28
- Replace local ToolResult type with SDK's CallToolResult
- Type extra parameter as RequestHandlerExtra instead of any/unknown
- Use Parameters<McpServer['registerTool']> to bridge zod duplication
- Remove as-any casts in ai-proxy examples and tests
- Fix missing beforeAll closing bracket in integration test

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@Scra3 Scra3 force-pushed the fix/high-vulnerabilities branch from 5182b91 to f83c1ac Compare February 12, 2026 14:30
- Wrap server.connect/transport.handleRequest in try/finally for cleanup
- Await transport.close() and server.close() to prevent unhandled rejections
- Clean up immediately when response is already ended (non-streaming)
- Catch and log cleanup errors in both paths to avoid replacing original errors
- Demote per-request "Registered N tools" log from Info to Debug
- Remove remaining as-any in ai-proxy example

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@Scra3 Scra3 force-pushed the fix/high-vulnerabilities branch from f83c1ac to 5164b15 Compare February 12, 2026 14:44
PMerlet and others added 2 commits February 12, 2026 17:37
- Add proper assertion in "should call next() for non-MCP routes" test
- Fix misplaced eslint-disable comment for @typescript-eslint/no-explicit-any
- Remove 5 unnecessary no-param-reassign disable comments (rule allows props)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@PMerlet PMerlet merged commit 15a9279 into main Feb 12, 2026
28 checks passed
@PMerlet PMerlet deleted the fix/high-vulnerabilities branch February 12, 2026 18:36
forest-bot added a commit that referenced this pull request Feb 12, 2026
forest-bot added a commit that referenced this pull request Feb 12, 2026
## @forestadmin/forestadmin-client [1.37.11](https://github.com/ForestAdmin/agent-nodejs/compare/@forestadmin/forestadmin-client@1.37.10...@forestadmin/forestadmin-client@1.37.11) (2026-02-12)

### Bug Fixes

* **security:** bump vulnerable packages ([#1466](#1466)) ([15a9279](15a9279))

### Dependencies

* **@forestadmin/ai-proxy:** upgraded to 1.4.2
forest-bot added a commit that referenced this pull request Feb 12, 2026
## @forestadmin/mcp-server [1.8.2](https://github.com/ForestAdmin/agent-nodejs/compare/@forestadmin/mcp-server@1.8.1...@forestadmin/mcp-server@1.8.2) (2026-02-12)

### Bug Fixes

* **security:** bump vulnerable packages ([#1466](#1466)) ([15a9279](15a9279))

### Dependencies

* **@forestadmin/agent-client:** upgraded to 1.4.7
* **@forestadmin/forestadmin-client:** upgraded to 1.37.11
forest-bot added a commit that referenced this pull request Feb 12, 2026
## @forestadmin/agent [1.72.8](https://github.com/ForestAdmin/agent-nodejs/compare/@forestadmin/agent@1.72.7...@forestadmin/agent@1.72.8) (2026-02-12)

### Bug Fixes

* **security:** bump vulnerable packages ([#1466](#1466)) ([15a9279](15a9279))

### Dependencies

* **@forestadmin/ai-proxy:** upgraded to 1.4.2
* **@forestadmin/forestadmin-client:** upgraded to 1.37.11
* **@forestadmin/mcp-server:** upgraded to 1.8.2
forest-bot added a commit that referenced this pull request Feb 12, 2026
## @forestadmin/forest-cloud [1.12.88](https://github.com/ForestAdmin/agent-nodejs/compare/@forestadmin/forest-cloud@1.12.87...@forestadmin/forest-cloud@1.12.88) (2026-02-12)

### Bug Fixes

* **security:** bump vulnerable packages ([#1466](#1466)) ([15a9279](15a9279))

### Dependencies

* **@forestadmin/agent:** upgraded to 1.72.8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants