docs(stormtest): Windows/cross-platform pathing heads-up (stacked on #510)#512
Merged
Conversation
Reads/writes concurrency is platform-agnostic and works on Windows, but reload churn (SS_RELOAD=1, the default) currently does not. Against a plugin-managed server the harness wedges past CALL_TIMEOUT when the server is killed mid-reload under load (#513); the documented "run the server externally" mitigation also fails on Windows because serve-this-worktree is bash-only and a hand-started --reload server gets killed by the reload (#514). The editor itself survives reloads. Also documents the POSIX->Windows invocation differences (venv path, $TMPDIR -> %TEMP%). Resilience tracked in #509. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
dsarno
force-pushed
the
claude/outstanding-repo-issues-AHLFv
branch
from
28da13a to
fb6bad9
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
This was referenced Jun 2, 2026
Open
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds a Windows / cross-platform pathing heads-up for
stormtest— a docs callout indocs/STRESS_TESTING.mdplus a one-line pointer inAGENTS.md. No code changes.The harness logic is already platform-agnostic (in-editor
res://_stormtest/…scratch paths use Godot's virtual filesystem; the host-side report path goes throughtempfile.gettempdir()/os.path.join). What's POSIX-only is the invocation — so the note flags, for agents running it on Windows:.venv\Scripts\python.exe, not.venv/bin/pythonscript/serve-this-worktreeis bash → Git Bash / WSL, or start the--reloadserver by hand$TMPDIR→%TEMP%, with a forward-slash tip for an explicitSS_REPORTThe note advertises its own removal once the runner is made resilient, tracked in #509.
docs/STRESS_TESTING.mdand the AGENTS.md stress section are introduced by #510 (the stormtest harness), which isn't onmainyet. This branch is based ontest/stormtest-harness, so until #510 merges this PR's diff will also show #510's harness files; once #510 lands inmain, the diff here collapses to just the ~23-line note. Please merge #510 first, then this.Context: this is the rescued note from the now-closed integration branch #506 — kept off #510 itself to leave that PR clean, per request.
https://claude.ai/code/session_01Jq5X4ivngAf1N6r5UX2BVw
Generated by Claude Code