Skip to content

Fix JSON encoding of Figma archive reports - #1655

Merged
chubes4 merged 1 commit into
trunkfrom
fix/1654-figma-report-json
Sep 10, 2026
Merged

Fix JSON encoding of Figma archive reports#1655
chubes4 merged 1 commit into
trunkfrom
fix/1654-figma-report-json

Conversation

@chubes4

@chubes4 chubes4 commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Summary

  • omit raw archive asset bytes from report metadata while preserving generated export bytes
  • omit invalid UTF-8 decoded archive metadata rather than applying lossy substitution
  • cover a synthetic binary archive transform and worker-style JSON encoding

Testing

  • composer test (from figma-transformer)
  • php -l figma-transformer/src/FigmaTransformer.php
  • php -l figma-transformer/tests/contract/KiwiParserContract.php

Fixes #1654

AI assistance: GPT-6 Astra was orchestrated with OpenCode to investigate, implement, and test this repair; human direction and review guide the work.

@chubes4
chubes4 merged commit 85c325b into trunk Sep 10, 2026
4 checks passed
@chubes4
chubes4 deleted the fix/1654-figma-report-json branch September 10, 2026 17:31
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.

Make Figma transform reports JSON-safe

1 participant