fix(coding-agent): bootstrap IPython kernel venv with the Windows interpreter path

[MarcusNeufeldt]

Summary

Fixes IPython kernel bootstrap on Windows. The kernel bootstrapper hardcoded the POSIX venv interpreter path (venv/bin/python) in both bootstrapVenv and the resolution path in ensureKernelPythonUncached. On Windows a uv-created venv places the interpreter at venv/Scripts/python.exe, so uv pip install --python <venv>/bin/python ... failed with exit code 2 and the IPython tool could never start.

Change

• Added a small kernelVenvPython(venv) helper that selects Scripts\python.exe on win32 and bin\python elsewhere (mirroring the existing process.platform === "win32" pattern already used in this file).
• Used it in both places that need the venv interpreter path.
• Added a win32-layout test to kernel-bootstrap.test.ts (the fake uv now supports a Windows-style venv shape).
• Updated docs/rlm-runtime.md and a changelog entry.

The POSIX path is unchanged, so existing Linux/CI behavior is preserved (the helper returns bin/python when process.platform is not win32; existing tests assert that path and stay green).

Testing

• npm run check pre-commit hook: biome + tsgo --noEmit pass.
• Manually verified the Windows path on a real Windows host: a patched install rebuilt the managed venv with the Scripts\python.exe layout, installed ipykernel + prime-agent-runtime + default packages, and a launched IPython kernel executed print(6*7) correctly.

Note

Fix IPython kernel venv bootstrap to use Scripts/python.exe on Windows

The venv interpreter path was hardcoded to the POSIX bin/python layout, causing bootstrap failures on Windows. A new kernelVenvPython helper in bootstrap.ts resolves to venv/Scripts/python.exe on win32 and venv/bin/python otherwise. This path is now used consistently across pip installs, skill sync, and the ensureKernelPython readiness check.

^{Macroscope summarized 93eb5a2.}

---

Note

Low Risk
Scoped to kernel venv path resolution on Windows; POSIX paths are unchanged and covered by existing tests plus a new win32-layout test.

Overview
Fixes IPython kernel bootstrap on Windows by resolving the managed venv interpreter through a new kernelVenvPython() helper instead of hardcoding POSIX bin/python.

Previously, bootstrapVenv and ensureKernelPython always pointed uv pip install --python at venv/bin/python, which does not exist on Windows (Scripts\python.exe). Bootstrap could fail before the kernel could start.

The helper picks Scripts\python.exe when process.platform === "win32" and bin/python otherwise; POSIX behavior is unchanged. Docs and changelog note the dual layout; tests cover the Windows path via a fake uv that creates a Scripts venv.

^{Reviewed by Cursor Bugbot for commit 93eb5a2. Bugbot is set up for automated code reviews on this repo. Configure here.}

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 93eb5a2b7f

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[chatgpt-codex-connector]

Keep the Unreleased section as a flat bullet list

Remove the new ### Fixed heading so this entry sits directly beneath ## [Unreleased]; repository changelogs explicitly prohibit Added/Changed/Fixed/Removed subsections in the unreleased section.

AGENTS.md reference: AGENTS.md:L109-L111

Useful? React with 👍 / 👎.

[sethkarten]

Thank you for the report and proposed work. This root cause is now covered by maintainer-owned stacked PR #1163, authored independently from upstream/main.

We did not inspect or reuse this PR's diff, branch, commits, implementation code, or tests; its public description/comments were used only as a bug report. To keep one review surface, this PR is superseded by #1163 and is being closed.

The complete review stack is #1158–#1165. It is being left unmerged for human review after CI and review-bot findings are cleared.