Rethrow if there's an MCP error

[heyitsaamir]

I added logging if there was an MCP error, but I also need to rethrow it. Added that and a test.

[Copilot]

Pull Request Overview

This PR ensures that errors from the MCP tool handler are not only logged but also propagated, and it adds a corresponding unit test.

• Added throw e after logging the error in the catch block.
• Introduced a test to verify that errors are logged and rethrown by the handler.

Reviewed Changes

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

File	Description
external/mcpclient/src/mcp-client-plugin.ts	Rethrow error in catch block after logging
external/mcpclient/src/mcp-client-plugin.spec.ts	Added test to assert logging and propagation of errors

Comments suppressed due to low confidence (2)

external/mcpclient/src/mcp-client-plugin.ts:126

• [nitpick] Use catch (e: unknown) instead of untyped catch (e) to improve type safety and enforce explicit error handling.

} catch (e) {

external/mcpclient/src/mcp-client-plugin.spec.ts:384

• The test verifies logging and rethrow but doesn’t assert that client.close() is still called in the finally block on error. Consider mocking the client and adding an assertion to ensure resources are always cleaned up.

await expect(functions[0].handler({})).rejects.toThrow('Tool failed');