windows-free-tests on GitHub free windows-latest fails three cases that
depend on Unix tooling the runner doesn't have:
1. `setup-windows-fallback.test.ts` behavior matrix — IS_WINDOWS=0 cells
assert `ln -snf` produces a real symlink. On Windows-without-Developer-
Mode (which the free `windows-latest` runner is), `ln -snf` silently
creates a file copy. That's literally the bug `_link_or_copy` exists
to work around, so the assertion can never pass there. Skip the whole
describe block on win32. The static-invariant test (zero raw `ln`
outside the helper body) above the matrix still runs and pins the
shape the Windows install relies on.
2. `docs-config-keys.test.ts` round-trip — spawnSync(`bin/gstack-config`)
on Windows doesn't read the bash shebang and fails to exec. Skip on
win32; the deprecated-key denylist test in the same file still runs
and is the actual invariant defending the v1.27.0.0 rename at the doc
layer.
Use `describe.skipIf(process.platform === 'win32', ...)` and
`test.skipIf(process.platform === 'win32', ...)`. Tests still run on
macOS and Linux unchanged.
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Four new test files (29 cases total):
browse/test/server-sanitize-surrogates.test.ts:
- 11 unit cases for sanitizeLoneSurrogates (passthrough, valid pair,
lone high/low mid-string, trailing/leading lone, adjacent doubles,
pair-then-lone, lone-then-pair, empty)
- 2 bug-repro tests pinning the regression intent (UTF-8 round-trip,
JSON.parse round-trip with codepoint assertion)
- 4 wiring invariants asserting the architectural choke points stay
intact (handleCommandInternalImpl rename, central sanitization
line, sanitizeReplacer function exists, SSE producers stringify
with replacer)
Function extracted from server.ts via regex + eval'd in test scope
so no production-code export is needed.
test/setup-windows-fallback.test.ts:
- Static invariant (D7): zero raw `ln` calls outside the
_link_or_copy helper body and comments
- Helper-existence assertions
- 4-cell behavior matrix (file/dir × Windows/Unix) via awk-style
helper extraction + bash -c sourcing
- Windows-note printer registration check
Mirrors test/setup-conductor-worktree.test.ts patterns.
test/build-script-shell-compat.test.ts:
- Regex assertion that package.json scripts.* contain no bash brace
groups (Bun-Windows-hostile)
- Subshell-precedence check for `.version` redirects
Strips single-quoted strings before regexing so embedded JS code
inside echo '...' doesn't false-positive.
test/docs-config-keys.test.ts:
- DEPRECATED_KEYS denylist scanned across docs/**/*.md
- Round-trip test for `gstack-config get artifacts_sync_mode`
Defends the v1.27.0.0 rename from doc drift.
Updates to two existing tests:
- test/setup-conductor-worktree.test.ts: expect `_link_or_copy`
instead of `ln -snf` at the Conductor-worktree guard call site
- test/gen-skill-docs.test.ts: same swap at three assertion sites
(Codex section, Claude link_claude_skill_dirs body, Codex
link_codex_skill_dirs body)
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Garry Tan