Merge fix/10-stitch-count-off-by-one -> main #36

Closed

Mark wants to merge 4 commits from fix/10-stitch-count-off-by-one into main

Mark commented

2026-09-12 18:02:42 +00:00

Owner

Auto-opened batch merger.

Mark added 4 commits

2026-09-12 18:02:43 +00:00

Fix X-Stitch-Count off-by-one (issue #10 )

CI / test (pull_request) Failing after 5s

Details

CI / docker-build (pull_request) Has been skipped

Details

3d11f258b2

Derive X-Stitch-Count from a round-trip read of the encoded bytes
rather than the request payload. pyembroidery's writers append a
trailing END (and PES injects JUMP/TRIM around color changes), so
len(pattern.stitches) before write does not match the on-disk record
count. Use EmbPattern.read_embroidery on the BytesIO buffer to get the
authoritative count after the writer has run.

Tests now assert header == len(pyembroidery.read(path).stitches) and a
new regression test test_export_header_matches_pes_byte_count locks
in the fix.

chore(gitignore): ignore _shakedown_output/ E2E test artifacts 1fd9a5446b

docs(plan): add plan for issue #16 SHAKEDOWN 6 post-fix ingress verify 8c6142c836

Mark/easystitch-service#16

The verify gate halts at slice 1: the deployed hostname
easystitch-service.tail018ac4.ts.net does not resolve from this runner
(NXDOMAIN), and the easystitch-service app is not registered in
nodehub's /api/apps. Local-only checks (ruff + pytest tests/unit/)
still pass. _shakedown_output/ingress_report.md captures the verdict
and evidence; it lives under _shakedown_output/ which is gitignored.

docs(plan): add diagnostic plan + report for issue #17 launcher stdout capture

CI / test (pull_request) Failing after 1s

Details

CI / docker-build (pull_request) Has been skipped

Details

c0e52ed43c

Issue #17 is a diagnostic: verify that the kilo-gsd-runner launcher's
terminal-tool channel round-trips stdout to the agent's tool-result surface.
No source code in easystitch-service is modified.

Six probes (trivial, multi-line, 4 KB size floor, stderr separability,
non-zero exit, JSON parseable) were executed and transcribed into
_shakedown_output/launcher_stdout_diagnostic.md (gitignored).

Findings: 5 PASS, 1 FAIL (informational) — stderr and stdout are merged in
the tool result with no stream annotation. No prior shakedown depends on
stderr/stdout distinguishability, so this does not invalidate any earlier
verify gate.