Skip to content

Fix current Windows Codex package resolution - #10

Merged
awtprod merged 1 commit into
mainfrom
agent/fix-windows-codex-package-resolution
Aug 2, 2026
Merged

Fix current Windows Codex package resolution#10
awtprod merged 1 commit into
mainfrom
agent/fix-windows-codex-package-resolution

Conversation

@awtprod

@awtprod awtprod commented Aug 2, 2026

Copy link
Copy Markdown
Owner

Summary

  • resolve the current Windows Codex package layout at vendor/<triple>/bin/codex.exe
  • preserve compatibility with the older vendor/<triple>/codex/codex.exe layout
  • recognize absolute %APPDATA%\\npm\\*.cmd launchers as npm-managed installations so the in-app Update action remains available

Root cause

The native Command Center resolver only knew the older Windows Codex package layout. Current Codex releases moved the executable into the platform package's bin directory, so T3 Code found the package but rejected its valid native runtime. The provider maintenance classifier also treated an absolute Windows npm shim as a manual installation, disabling one-click updates.

Impact

Native Windows Command Center chats can use current Codex npm installations, and Windows npm-managed Codex providers can be updated from provider settings.

Validation

  • 138 focused tests across native runtime resolution, provider maintenance, update execution, adapter startup, and update UI logic
  • focused lint passed
  • server typecheck passed
  • formatting and git diff --check passed

@github-actions github-actions Bot added size:M vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. labels Aug 2, 2026
@awtprod
awtprod marked this pull request as ready for review August 2, 2026 01:52
@awtprod
awtprod merged commit 610db1f into main Aug 2, 2026
17 of 18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants