Skip to content

perf: Python interpreter detection #45

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
hisenb3rg merged 1 commit into from
Sep 16, 2026
Merged

perf: Python interpreter detection #45

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -84,9 +84,9 @@ Update it by pulling this repo and copying the folder again.
## Requirements

Rendering needs **Python 3.8 or newer** on the machine — nothing else, no
third-party packages. The skill finds the interpreter itself, trying `py -3`,
`python3`, and `python` in that order, and stops with a clear message if none of
them works.
third-party packages. The skill finds the interpreter itself, trying `python3`,
`py -3`, and `python` in that order, stopping at the first that works, and aborts
with a clear message if none of them does.

## Tests

Expand Down
10 changes: 5 additions & 5 deletions skills/render-spec/SKILL.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,12 +22,12 @@ Do every step yourself. Never hand any step to a subagent, Agent/Task tool or wo
* On Claude Code it is `${CLAUDE_SKILL_DIR}`; on other agents derive it from the location this skill was loaded from.
* Never reach a file inside this skill by a relative path.

`<python>` is required Python 3.8 or newer. Resolve it now and reuse it everywhere in this skill where `<python>` is used. Run all three of these as independent tool calls in one single turn, in parallel - do not run them one after another:
* `python3 "<skill_folder>/scripts/check_version.py" check`
* `py -3 "<skill_folder>/scripts/check_version.py" check`
* `python "<skill_folder>/scripts/check_version.py" check`
`<python>` is required Python 3.8 or newer. Resolve it now and reuse it everywhere in this skill where `<python>` is used. Try these candidates one at a time, in this order, and stop at the first one whose output contains a `current:` line - its interpreter is `<python>`. Do not run the remaining candidates once one has succeeded, and do not run them in parallel:
1. `python3 "<skill_folder>/scripts/check_version.py" check`
2. `py -3 "<skill_folder>/scripts/check_version.py" check`
3. `python "<skill_folder>/scripts/check_version.py" check`

Among the commands whose output contains a `current:` line, keep the first one in the order listed above: its interpreter is `<python>`. Report the Python tool you settled on. If no command prints a `current:` line, Python is a missing dependency: abort the rendering and report it.
Report the Python tool you settled on. If none of the three prints a `current:` line, Python is a missing dependency: abort the rendering and report it.

From that same command's output report with a message "Running pyro: <current>".
* If the output shows `status: update-available`, warn with a message: "A newer pyro release <latest> is available (this is <current>)".
Expand Down