Skip to content

use codex native installer - #467

Merged
hbrodin merged 1 commit into
mainfrom
brad/codex-native-installer
Sep 14, 2026
Merged

hbrodin merged 1 commit into
mainfrom
brad/codex-native-installer

Conversation

@BradSwain

Copy link
Copy Markdown
Contributor

The native installer script should automatically handle installing any utilities codex wants to bundle. The native installer also allows codex upgrade to work from within a vm.

@hbrodin hbrodin left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Using the native installer makes sense for keeping the bundled tools and self-update behavior aligned with upstream.

One nonblocking question: does upstream provide a supported way to clean up older completed releases? The installer currently retains them, whereas coop's previous installer pruned inactive releases. If there isn't an upstream cleanup mechanism, I'm happy to track this as a follow-up rather than couple coop to the native package layout here.

Review coverage: correctness, design, conventions, security, API usage, tests, docs, and comments. No blocking findings. All nine host probe tests passed, four deliberately broken variants were caught, and shell syntax checks passed. CI is green; Lima and Firecracker VM integration were not run in this review and are absent from the PR checks.

@BradSwain

Copy link
Copy Markdown
Contributor Author

does upstream provide a supported way to clean up older completed releases?

Doesn't look like the installer supports cleaning up older releases.

I can add a follow-on PR to have coop handle cleanup. I believe the installer is part of the open source codex repo, so we could potentially open a PR updating the installer itself as well.

@hbrodin
hbrodin force-pushed the brad/codex-native-installer branch from 33ebdeb to a9b6269 Compare September 14, 2026 08:02
@hbrodin
hbrodin merged commit df5909c into main Sep 14, 2026
10 checks passed
@hbrodin
hbrodin deleted the brad/codex-native-installer branch September 14, 2026 08:24
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.

2 participants