Skip to content

fix: add a second pass of resolution with no extension - #161

Merged
43081j merged 1 commit into
mainfrom
jg/the-vladimir-fix
Jul 29, 2026
Merged

fix: add a second pass of resolution with no extension#161
43081j merged 1 commit into
mainfrom
jg/the-vladimir-fix

Conversation

@43081j

@43081j 43081j commented Jul 29, 2026

Copy link
Copy Markdown
Member

Fixes #160

Currently, if the command includes a . (like foo.js), we search for
foo.js, foo.js.EXE, foo.js.CMD, etc.

However, if the command is foo, we only search for foo.EXE,
foo.CMD, etc.

This means we're not picking up files literally called foo.

The fix here does a second pass with no extensions. This means we do the
existing searching, and if all else fails, we look for the name as-is.

Fixes #160

Currently, if the command includes a `.` (like `foo.js`), we search for
`foo.js`, `foo.js.EXE`, `foo.js.CMD`, etc.

However, if the command is `foo`, we only search for `foo.EXE`,
`foo.CMD`, etc.

This means we're not picking up files literally called `foo`.

The fix here does a second pass with no extensions. This means we do the
existing searching, and _if all else fails_, we look for the name as-is.
@43081j
43081j merged commit 7d37564 into main Jul 29, 2026
9 checks passed
@43081j
43081j deleted the jg/the-vladimir-fix branch July 29, 2026 09:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Windows: extensionless shebang scripts fail when passed by path

1 participant