Skip to content

fix(build): name a Python that cannot start, and never reuse an old record - #6

Merged
MotimotiNotch merged 1 commit into
mainfrom
fix/missing-python
Oct 8, 2026
Merged

MotimotiNotch merged 1 commit into
mainfrom
fix/missing-python

Conversation

@MotimotiNotch

Copy link
Copy Markdown
Owner

Closes #5

What changes

  • A Python that cannot start is named. When the interpreter in "python" cannot be started (ENOENT, or the Windows Store stub that exits with 9009 / prints "Python was not found"), the build stops with:
    could not start Python to trace scripts/weekly_summary.py ("python": "..." in the config).
    Install Python 3, or set "python" in loopfinder/config.json to the interpreter (for example "py" or a full path).
    
    instead of trace did not finish: ... (exit null).
  • An old record is never used in place of this run's. Each trace's previous record is removed before the script runs. Before, if the script failed to start, a record left by an earlier build counted as this run's result, and the demo's declared loops were then reported as "declared but not found".

Not done: skipping .py traces and building from the rest. The declared loops that depend on those traces would be reported as "declared but not found" again, which looks like a broken flow rather than a missing interpreter.

Checked

The VS Code extension copies the CLI when it is packaged, so its users get this with the next extension release.

🤖 Generated with Claude Code

https://claude.ai/code/session_01CS2kGo81duzNHgecBAWHe4

…ecord

When the interpreter in "python" cannot be started (not found, or the Windows
Store stub that exits 9009), stop with a message that names the setting and how
to change it, instead of "trace did not finish ... (exit null)".

Remove a trace's previous record before running it, so a run that fails to
start can no longer pass an old record off as its own (which made declared
loops show up as "declared but not found").

Closes #5

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01CS2kGo81duzNHgecBAWHe4
@MotimotiNotch
MotimotiNotch merged commit bf84542 into main Oct 8, 2026
4 checks passed
@MotimotiNotch MotimotiNotch mentioned this pull request Oct 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant