fix(plugins): pin Codex hooks to merged runtime#1140
Merged
Conversation
Signed-off-by: phernandez <paul@basicmachines.co>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
PR #1138 merged the Codex checkpoint runtime as commit
c28159d2, but the Codex hook launchers still referenced the intermediate feature commit1cb7a541. Aligning the hook launchers with the durable merged revision keeps plugin execution and operator CLI usage on the same source.What Changed
c28159d2077158c4f596fb62f351e6e9012b95a5.uv run --scriptexecution model.Implementation Details
The PEP 723 dependency declarations continue to use an immutable commit SHA rather than the moving
mainref. The plugin therefore remains reproducible and does not depend on a separately installedbasic-memoryexecutable being present onPATH.Testing
Automated
git diff --checkuv run pytest -q --no-cov plugins/codex/hooks tests/test_codex_plugin_package.py— 21 passedjust package-check— passed the Claude Code, Codex, shared skills, Hermes, and OpenClaw package gatesManual
Not required for this dependency-metadata-only repin; the hook package tests exercise the launchers and the full package gate validates all bundled agent packages.
Risks / Follow-ups
Risk is low because runtime logic is unchanged. Existing installed marketplace caches will continue using the previous immutable revision until the updated plugin is released and reinstalled.