[Gherkin]

Fixes #160.

exportJson.il wrote net names into the JSON as they are. A net name with " or \ made the file invalid JSON.

• New helper jsonEscape escapes \ first, then ". It also doubles %, because writeJsonFile passes each line to fprintf as the format string. The component table already doubles % for the same reason.
• Used at every place a net name is written: tracks, arcs, vias, zones, pads and the net list.
• Separate commit: axlUIPrompt returns nil for rev and company when run without a GUI. Both now fall back to "", so addMetadata does not fail.

Testing

• No test runs the SKILL script, so this change is not tested here. It needs an export from Allegro.
• npm run test:api passes.
• Playwright: 174 passed, 1 failed. The failure was the inner layer checkbox timeout (Inner layer checkbox test times out in a local full run with Firefox #72); it passes when run alone.

🤖 Generated with Claude Code

https://claude.ai/code/session_01UCbPYsA5LYNX8HZaGGZCpK

[github-actions]

PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-10-31 10:44 UTC

[Gherkin]

/prr round 1

Fixing:

1. allegro-skills/exportJson.il:2933 — each line is passed to fprintf as the format string. Any field without %% doubling (text, refdes, footprint, pin, rev, company) breaks the write. Fix: fprintf( outPort "%s\n" line ), then drop the % doubling in jsonEscape and the component table.
2. allegro-skills/exportJson.il:25 — rev, company and design name are quoted but not escaped. Fix: jsonEscape.
3. allegro-skills/exportJson.il:347 — board text escapes " only, not \. Fix: jsonEscape.
4. allegro-skills/exportJson.il:1624 — pin number escapes " only. Fix: jsonEscape.
5. allegro-skills/exportJson.il:1700 — component table: value lacks \ escape; footprint, refdes, property key unescaped (:1737, :1760, :1762). Fix: jsonEscape.

FYI, not fixing:

6. allegro-skills/exportJson.il:68 — pcreCompile three times per call, per segment/via/pad. Not measured; the issue reports a ~16k pad board exports fine.
7. allegro-skills/exportJson.il:70 — control characters (tab etc.) are not escaped. Unlikely in Allegro names; left out.
8. allegro-skills/exportJson.il:14 — rev/company prompt repeats once per variant. Older behaviour, not from this PR.
9. allegro-skills/exportJson.il:2941 — axlUIConfirm in the error paths still needs a GUI. Older code. The axlUIPrompt nil return was seen in a real no-GUI run (per Allegro export writes invalid JSON when a net name contains " or \ #160), so the new comment stands.

None of this is runnable here; no test runs the SKILL script.

[Gherkin]

/prr round 2 (56ded7c..531be4d)

Fixing:

1. allegro-skills/exportJson.il:1916 — footprint ref is written raw; the component table escapes the same refdes. Fix: jsonEscape.
2. allegro-skills/exportJson.il:2789 — inner layer names go into tracks/zones keys unescaped. Fix: jsonEscape.

FYI, not fixing:

3. allegro-skills/exportJson.il:1691 — jsonEscape on a numeric VALUE property would error in pcreReplace. The old code called pcreReplace on it too, so not new. Converting numbers means picking a format.
4. allegro-skills/exportJson.il:1727 — property key comes from sprintf "%L", which may add SKILL escapes. Older code; this commit makes it valid JSON where it was invalid. Cannot check %L output without Allegro.
5. allegro-skills/exportJson.il:67 — control characters not escaped. Same as round 1 item 7.
6. allegro-skills/exportJson.il:69 — two pcreCompile per call. Same as round 1 item 6.
7. allegro-skills/exportJson.il:1747 — one jsonString helper that quotes and escapes would stop missed sites. Larger refactor; not for this PR.

None of this is runnable here; no test runs the SKILL script.