Fix Codex detection across npm and package-manager installs #4

Merged

mathieu merged 1 commit from fix/issue-2-detect-codex-installation into main

2026-08-29 03:29:42 +00:00

hephaistos commented

2026-08-29 03:19:01 +00:00

Owner

Fixes #2

Summary

move openai-codex from a hard pacman dependency to an optional dependency
keep Codex availability and minimum-version enforcement at runtime through the configured executable on PATH
add regression coverage for npm-style arbitrary PATH installs and Arch package metadata
bump the packaged recipe to pkgrel=2 and update .SRCINFO, README, and the Codex dependency note

Verification

PYTHONPATH=src python3 -W error -m unittest discover -s tests -v — 24 passed
bash -n aur/PKGBUILD packaging/PKGBUILD.template — passed by the implementation worker
git diff --check — passed by the implementation worker
namcap /work/aur/PKGBUILD in archlinux:base — clean in the implementation worker

Identity correction

The code commit was re-authored and re-signed as Hephaistos after detecting that the first worker run inherited Hermes repository credentials. The original Hermes-authored PR was closed and replaced by this one.

Risks

The package can now be installed without Codex, by design, because pacman cannot account for npm-installed executables. Audit and doctor paths remain fail-closed when codex is missing, too old, unauthenticated, or unusable.

Review guidance

Confirm that openai-codex appears only in optdepends, .SRCINFO matches the PKGBUILD metadata, and runtime executable/version checks remain unchanged in config.py and cli.py.

Fixes #2 ## Summary - move `openai-codex` from a hard pacman dependency to an optional dependency - keep Codex availability and minimum-version enforcement at runtime through the configured executable on `PATH` - add regression coverage for npm-style arbitrary PATH installs and Arch package metadata - bump the packaged recipe to `pkgrel=2` and update `.SRCINFO`, README, and the Codex dependency note ## Verification - `PYTHONPATH=src python3 -W error -m unittest discover -s tests -v` — 24 passed - `bash -n aur/PKGBUILD packaging/PKGBUILD.template` — passed by the implementation worker - `git diff --check` — passed by the implementation worker - `namcap /work/aur/PKGBUILD` in `archlinux:base` — clean in the implementation worker ## Identity correction The code commit was re-authored and re-signed as Hephaistos after detecting that the first worker run inherited Hermes repository credentials. The original Hermes-authored PR was closed and replaced by this one. ## Risks The package can now be installed without Codex, by design, because pacman cannot account for npm-installed executables. Audit and doctor paths remain fail-closed when `codex` is missing, too old, unauthenticated, or unusable. ## Review guidance Confirm that `openai-codex` appears only in `optdepends`, `.SRCINFO` matches the PKGBUILD metadata, and runtime executable/version checks remain unchanged in `config.py` and `cli.py`.

hephaistos added 1 commit

2026-08-29 03:19:01 +00:00

fix: detect Codex independently of package manager 779a02349b

hephaistos referenced this pull request

2026-08-29 03:19:04 +00:00

L'installation échoue en ne reconnaissant pas la vesion de codex installée #2