From 7dd1414353afec8f949479fda1b487f3fed21816 Mon Sep 17 00:00:00 2001 From: Hisenberg Date: Wed, 16 Sep 2026 15:35:17 +0200 Subject: [PATCH] perf: Probe python inter. sequentially, stop at the first hit Step 0 ran all three probes in parallel and waited for all of them before picking an interpreter. It now tries python3, py -3 and python in that order and stops at the first one that prints a current: line. In the common case that is one probe instead of three. The README now lists the same order. Co-Authored-By: Claude Fable 5.1 --- README.md | 6 +++--- skills/render-spec/SKILL.md | 10 +++++----- 2 files changed, 8 insertions(+), 8 deletions(-) diff --git a/README.md b/README.md index 035d2e9..6501aad 100644 --- a/README.md +++ b/README.md @@ -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 diff --git a/skills/render-spec/SKILL.md b/skills/render-spec/SKILL.md index c9a4d86..dd24c9d 100644 --- a/skills/render-spec/SKILL.md +++ b/skills/render-spec/SKILL.md @@ -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. -`` is required Python 3.8 or newer. Resolve it now and reuse it everywhere in this skill where `` 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 "/scripts/check_version.py" check` -* `py -3 "/scripts/check_version.py" check` -* `python "/scripts/check_version.py" check` +`` is required Python 3.8 or newer. Resolve it now and reuse it everywhere in this skill where `` 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 ``. Do not run the remaining candidates once one has succeeded, and do not run them in parallel: +1. `python3 "/scripts/check_version.py" check` +2. `py -3 "/scripts/check_version.py" check` +3. `python "/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 ``. 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: ". * If the output shows `status: update-available`, warn with a message: "A newer pyro release is available (this is )".