Use params.format in exportNodeAsImage instead of hardcoded PNG

[bimawa]

Previously the format was hardcoded to "PNG" regardless of what was
passed in params. Now uses params.format with "PNG" as fallback,
enabling SVG, JPG, and PDF exports.

[bimawa]

Thanks for taking the time to review! I'd really appreciate it if you could leave a link to your solution so I can learn the best way to approach this. Once I've had a chance to check it out, should we go ahead and close my pull request?

…

On 17 Mar 2026 at 05:08:30, David Bowman ***@***.***> wrote: *dabowman* left a comment (grab/cursor-talk-to-figma-mcp#146) <#146 (comment)> Thanks for catching this! The bug is real — params.format is indeed ignored because of the hardcoded const format = "PNG". However, this PR targets src/cursor_mcp_plugin/code.js, which doesn't exist in this fork — the plugin was moved to src/figma_plugin/ and is now built from source modules (src/figma_plugin/src/commands/document.js). The fix would need to go in that source file, then bun run build:plugin regenerates the bundled code.js. The fix itself is correct in spirit. One style note: since params is already guaranteed non-null via params || {} at the top of the function, the cleaner pattern (consistent with how scale is handled) would be: const { nodeId, scale = 1, format = "PNG" } = params || {}; That way format is a proper destructured default rather than an inline || expression, and .toUpperCase() can be dropped since the MCP layer validates via z.enum(["PNG", "JPG", "SVG", "PDF"]) and always sends uppercase values. I'll port this fix to the correct source location. Thanks for surfacing it! — Reply to this email directly, view it on GitHub <#146 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAKUJEPV2YGYQYAGFCPCAZD4RCJP5AVCNFSM6AAAAACWD4T7YGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHM2DANZRGQZTIMZYGE> . You are receiving this because you authored the thread.Message ID: ***@***.***>