Skip to content

Fix Codex command resolution on Windows - #2

Merged
calculatetech merged 1 commit into
mainfrom
task/cpk-060-windows-codex-resolution
Sep 26, 2026
Merged

calculatetech merged 1 commit into
mainfrom
task/cpk-060-windows-codex-resolution

Conversation

@calculatetech

Copy link
Copy Markdown
Owner

Summary

  • resolve the Windows npm codex.CMD shim before native plugin and Doctor commands
  • isolate Git fixtures from host core.autocrlf settings
  • publish the fix as toolkit version 0.24.1

Validation

  • pwsh -File .\run-tests.ps1 — 111 tests passed, 1 skipped
  • pwsh -File .\install.ps1 — passed
  • pwsh -File .\doctor.ps1 — Result: ready
  • local native Codex review — clean

Copy link
Copy Markdown
Owner Author

@codex review

Reviewers must not execute tests or delegate test execution. This includes suites, focused tests, and reproduction scripts.

Review target: e10ddd34fc30200e74202fd53a08755242336e29.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 26, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-09-26T03:55:28.690418Z e10ddd3 Manual request
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Keep it up!

Reviewed commit: e10ddd34fc

ℹ️ 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".

@calculatetech
calculatetech marked this pull request as ready for review September 26, 2026 03:55
@calculatetech
calculatetech merged commit 00f3af7 into main Sep 26, 2026
1 check passed
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.

1 participant