emrg: add iscc compile gate to Windows test job

[argszero]

Summary

Fix the v0.2.30 Build Release Windows gate failure (iscc "Invalid number of parameters" at emrg.iss:183:47 on run 31661378619) and add a permanent CI gate so .iss compile errors surface on every PR instead of only at tag-push build time.

Root cause

The #727 stop-log block in packaging/make-installer.sh called LoadStringFromFile(LogFile) — but Inno Setup's Pascal Script signature is function LoadStringFromFile(const FileName: String; var S: AnsiString): Boolean (2-param out-param form, no 1-param string-return form). Test CI never compiles the .iss (only the tag-triggered Build Release runs iscc), so the error shipped.

Changes

1. .github/workflows/test.yml — new "Inno Setup script compile smoke test" step in the test-windows job: renders make-installer.sh's emrg.iss heredoc (self-syncing sed anchor, same variable expansion as the real build), stubs the gen-assets icon.ico + {app} referenced files, and compiles with the runner's preinstalled iscc. Every PR now catches .iss syntax/signature/type errors.
2. packaging/make-installer.sh — LoadStringFromFile(LogFile) → LoadStringFromFile(LogFile, LogText) (2-param out-param form).
3. packaging/make-installer.sh — LogText declared AnsiString (Inno 6 string = UnicodeString → passing it to var S: AnsiString is a compile-time Type mismatch, caught by the new gate).
4. tests/test_installer_stop.py — positive/negative assertions pinning the correct call form and the AnsiString variable type.

Verification

• test (ubuntu): actionlint + pytest 761 + GUI 229 — PASS
• test-windows: pytest + iscc compile smoke test — PASS (proves the .iss now compiles)

[argszero]

✅ LGTM — cycle (1/3)

Reviewed the full branch (6 commits, both channels' work converged). This fixes the v0.2.30 Build Release failure 31661378619 (windows iscc rejected the .iss [Code] section).

Root cause: Inno Setup's script API is LoadStringFromFile(const FileName: String; var S: AnsiString): Boolean (2-param out-arg; verified against issrc Shared.ScriptFunc.pas at is-6_7_1/is-6_7_3/is-7_1_0/main). #727's R125 code used the non-existent 1-param string-returning form → 'Invalid number of parameters' at compile; Test CI never compiled the .iss so it only surfaced at tag push (v0.2.7 lesson recurring).

Changes (all verified):

• make-installer.sh: LoadStringFromFile(LogFile, LogText) with LogText: AnsiString (var-param strict typing; string → 'Type mismatch', caught by 31662985004)
• make-installer.sh: heredoc comment de-backticked (backticks trigger command substitution inside unquoted <<EOF)
• test.yml windows job: iscc compile smoke gate — renders the .iss heredoc (stub payload + stub icon.ico + {app} files, since icon.ico is a gitignored gen-assets product) and runs the runner's iscc; actionlint-clean (SC2016/SC2097/SC2098/SC2318 all resolved)
• tests/test_installer_stop.py: positive asserts (2-param form + AnsiString) and negative assert (1-param form forbidden)

Verification: run 31663078864 SUCCESS — test (actionlint + 761 pytest + GUI) and test-windows (pytest + iscc compile) both green. MERGEABLE/CLEAN. This gate would have caught the v0.2.30 failure before tagging.

[argszero]

✅ LGTM — cycle. Verified: iscc compile gate passes on the 6-commit branch (LoadStringFromFile 2-param out-param form + LogText: AnsiString); CI run 31663078864 both test + test-windows SUCCESS (actionlint + doc-count guard + iscc gate). The gate reproduces the exact v0.2.30 Build Release failure mode and now compiles clean.

[argszero]

✅ LGTM — cycle (3/3)

Branch unchanged since 1/3 review (23de096); CI still green (test + test-windows, run 31663078864 SUCCESS). 3 consecutive approvals from different cycles — merging.