Repository navigation
Complete Windows npm shim launcher fix and regressions - #3
Merged
Merged
Conversation
The backend contract resolves CLI paths with Python's shutil.which. Under native Windows Python that walks PATHEXT and returns the `foo.CMD` sibling spelled with backslashes. The value is then consumed by bash, which can neither `command -v` that path nor exec-test it: npm writes the .cmd shim without the executable bit, and the launcher bash can use is the extensionless name on its own PATH. Every npm-installed backend was therefore rejected with a false "backend CLI not found", which also masked the real backend error. Fall back to the extensionless basename when the resolved value looks like a path and is unusable. A bare name that resolves to nothing still fails where it did before. Fixes #1 Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Resolve version conflicts in .claude-plugin/*.json by taking upstream 2.0.0-beta.1; the 1.3.5 bump from this branch is superseded. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Re-run smoke tests after a flaky ubuntu failure caused by interleaved writes to the fake backend argv log. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Preserve av-frox's PR #2 fix and integrate current main. Emit LF-only shell assignments from native Python, add portable platform regressions for CRLF and same-directory shim selection, and bump plugin manifests to 2.0.0-beta.3. Local verification: offline suite 993 passed, 0 failed; steer suite 401 passed; platform suite 22 tests with 4 native-Windows-only skips; Bash syntax and diff checks passed. Native Windows and GitHub CI remain unverified because Actions jobs cannot start due to the account billing lock.
rodion-m
marked this pull request as ready for review
October 5, 2026 07:13
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
2 participants
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.
Fixes #1. Builds on #2 by av-frox, preserving the original commits and author history.
Changes
Validation
Merge blocker
The original PR's Actions run was retried on October 5 and still could not start any jobs: GitHub reports an account billing lock. Keep this PR in draft until CI can run successfully, including Windows.
This upstream branch exists because the connected integration cannot write to the contributor's fork (403). The original PR remains open and unchanged.