fix(faces): strip trailing ; from module name in JaffaScript imports

[hyperpolymath]

Summary

Fixes a string-stripping bug in transform_import inside lib/js_face.ml that corrupted module names when the import ended with a semicolon (the normal JS style).

Root cause: import { println } from "io"; caused transform_import to set mod_part = "\"io\";". The code then sliced off the surrounding quote characters with String.sub mod_part 1 (String.length mod_part - 2), but because the last character was ; rather than ", the slice yielded "io\"" (trailing quote instead of nothing). The resulting use declaration was use io"::{println}; — a syntactically invalid module path.

Fix: In all three import branches (import { x } from "m";, import * from "m";, and import name from "m";), strip a trailing ; from mod_part / quoted before the quote-stripping slice.

Files changed

File	Change
lib/js_face.ml	Three-branch semicolon-strip fix in transform_import
examples/faces/hello-jaffa.affine	Added import { println } from "io"; to exercise the import path in the regression test
tests/faces/hello-jaffa.expected.txt	Refreshed snapshot — now includes use io::{println};

Test plan

• opam exec -- dune build lib/ bin/ — confirms OCaml compiles
• ./tools/run_face_transformer_tests.sh — snapshot regression: hello-jaffa.expected.txt should match preview-js output
• affinescript preview-js examples/faces/hello-jaffa.affine — visually confirm use io::{println}; appears correctly
• affinescript parse examples/faces/hello-jaffa.affine — confirm the lowered file parses without errors

Note: OCaml toolchain is not available in the authoring environment; changes were manually traced through the transformer logic. CI is the source of truth for build correctness.

---

Generated by Claude Code