github/penpot
1
0
Fork 0
mirror of https://github.com/penpot/penpot.git synced 2026-05-14 20:43:55 +00:00
Code Issues Packages Projects Releases Wiki Activity

✨ Catch serialisation issues in penpot.ui.sendMessage

Browse Source 
This prevents timeouts in the MCP server in cases where there is an
issue with the serialisation.

GitHub #9634
This commit is contained in:
Dominik Jain 2026-05-14 14:16:21 +02:00 committed by Alonso Torres
parent 94a5c6c4fd
commit 7c42a1f9ac
1 changed files with 23 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
mcp/packages/plugin/src/TaskHandler.ts
View File

@ -35,7 +35,29 @@ export class Task<TParams = any> {
};
// Send to main.ts which will forward to MCP server via WebSocket
penpot.ui.sendMessage(response);
try {
// TODO: Remove ts-ignore once Penpot types have been updated
// @ts-ignore
penpot.ui.sendMessage(response, true);
} catch (sendError) {
console.error("Failed to transfer task response for transmission to MCP server", sendError);
if (success) {
console.info("Original response was a success response, sending an error response instead.");
// If we were trying to send a success response, send an error response instead
let cause = sendError instanceof Error ? sendError.message : String(sendError);
this.sendError(
"Unexpected error sending task response: " +
cause +
" If you are an LLM and tried to serialize a regular object (shape, token, etc.), " +
"tell the user to report this error, providing information on the request you made."
);
} else {
console.info(
"Original response was an error response, abandoning attempt to send alternative response to avoid infinite error loop."
);
}
return;
}
console.log("Sent task response:", response);
this.isResponseSent = true;
}