Skip to content

Commit 33b13b6

Browse files
committed
fix: add platform field to all ErrorData objects
1 parent f0baea1 commit 33b13b6

File tree

2 files changed

+17
-4
lines changed

2 files changed

+17
-4
lines changed

src/modules/tools.ts

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,10 @@ export function setupMCPCatTools(server: MCPServerLike): void {
6464
writeToLog(
6565
`Warning: Original list tools handler failed, this suggests an error MCPCat did not cause - ${error}`,
6666
);
67-
event.error = { message: getMCPCompatibleErrorMessage(error) };
67+
event.error = {
68+
message: getMCPCompatibleErrorMessage(error),
69+
platform: "javascript",
70+
};
6871
event.isError = true;
6972
event.duration =
7073
(event.timestamp &&
@@ -85,7 +88,10 @@ export function setupMCPCatTools(server: MCPServerLike): void {
8588
writeToLog(
8689
"Warning: No tools found in the original list. This is likely due to the tools not being registered before MCPCat.track().",
8790
);
88-
event.error = { message: "No tools were sent to MCP client." };
91+
event.error = {
92+
message: "No tools were sent to MCP client.",
93+
platform: "javascript",
94+
};
8995
event.isError = true;
9096
event.duration =
9197
(event.timestamp &&

src/modules/tracing.ts

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,10 @@ export function setupListToolsTracing(
7474
writeToLog(
7575
`Warning: Original list tools handler failed, this suggests an error MCPCat did not cause - ${error}`,
7676
);
77-
event.error = { message: getMCPCompatibleErrorMessage(error) };
77+
event.error = {
78+
message: getMCPCompatibleErrorMessage(error),
79+
platform: "javascript",
80+
};
7881
event.isError = true;
7982
event.duration =
8083
(event.timestamp &&
@@ -95,7 +98,10 @@ export function setupListToolsTracing(
9598
writeToLog(
9699
"Warning: No tools found in the original list. This is likely due to the tools not being registered before MCPCat.track().",
97100
);
98-
event.error = { message: "No tools were sent to MCP client." };
101+
event.error = {
102+
message: "No tools were sent to MCP client.",
103+
platform: "javascript",
104+
};
99105
event.isError = true;
100106
event.duration =
101107
(event.timestamp &&
@@ -258,6 +264,7 @@ export function setupToolCallTracing(server: MCPServerLike): void {
258264
event.isError = true;
259265
event.error = {
260266
message: `Tool call handler not found for ${request.params?.name || "unknown"}`,
267+
platform: "javascript",
261268
};
262269
event.duration =
263270
(event.timestamp &&

0 commit comments

Comments
 (0)