Skip to content

sync test: fixture instead of the publisher's ~/Data (unblocks 2.5.1 publish) - #12

Merged
plur9 merged 1 commit into
mainfrom
fix/sync-test-fixture
Sep 26, 2026
Merged

plur9 merged 1 commit into
mainfrom
fix/sync-test-fixture

Conversation

@plur9

@plur9 plur9 commented Sep 26, 2026

Copy link
Copy Markdown
Contributor

The prepublish gate failed on statusAll timing out at 5s. The test ran git status over every real space in the publisher's ~/Data. It couldn't be pointed anywhere else, because sync.ts fixed DATA_DIR when the module loaded. The data dir is now resolved per call, the same way upgrade.ts already does it, and the test uses a two-repo fixture. It now asserts that both repos are found; before, an empty array passed. 131/131 pass, three runs in a row.

🤖 Generated with Claude Code

https://claude.ai/code/session_01GSzwHUALnUZdoW7v21dDtr

statusAll ran git status over every live space and timed out at 5s often
enough to block the 2.5.1 publish. sync.ts froze DATA_DIR at import, so the
test could not be pointed anywhere else; it is now resolved per call.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01GSzwHUALnUZdoW7v21dDtr
@plur9
plur9 merged commit b8aa0b2 into main Sep 26, 2026
2 checks passed
@plur9
plur9 deleted the fix/sync-test-fixture branch September 26, 2026 17:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant