Skip to content

Clear CMUX Codex pills on Stop hooks#34

Merged
ictechgy merged 1 commit into
mainfrom
fix/codex-cmux-stop-clears-status
Jul 5, 2026
Merged

Clear CMUX Codex pills on Stop hooks#34
ictechgy merged 1 commit into
mainfrom
fix/codex-cmux-stop-clears-status

Conversation

@ictechgy

@ictechgy ictechgy commented Jul 5, 2026

Copy link
Copy Markdown
Owner

Summary

  • Clear observed understatus.codex.* CMUX sidebar keys when Codex sends a Stop hook.
  • Preserve the existing non-Stop hook path that writes CPU/memory/model/git event snapshots.
  • Document that CMUX pills are hook-event snapshots, not live polling, and that hook binary paths must be rebuilt/reinstalled after binary changes.

Verification

  • cargo fmt --all -- --check
  • cargo test --test oneline cmux --locked -- --nocapture
  • cargo test --all-targets --locked
  • cargo clippy --all-targets --locked -- -D warnings
  • cargo build --release --locked
  • git diff --check -- README.md src/main.rs tests/oneline.rs
  • Live CMUX non-Stop/Stop verification: non-Stop updated keys; Stop returned {} and cleared understatus.codex.* keys.

Release note

  • No version bump or public release publishing is included in this follow-up PR.

Stop hooks now clear observed understatus Codex sidebar keys instead of writing a final cpu/mem snapshot. The hook path still returns neutral JSON and non-Stop hooks keep applying status pills.

Constraint: keep CMUX hook fail-soft, target-scoped, and dependency-free.

Confidence: high

Scope-risk: narrow

Directive: Preserve event-snapshot semantics; do not add polling without a separate lifecycle design.

Tested: cargo fmt --all -- --check; cargo test --test oneline cmux --locked -- --nocapture; cargo test --all-targets --locked; cargo clippy --all-targets --locked -- -D warnings; cargo build --release --locked; live CMUX non-Stop/Stop verification

Not-tested: public release publishing for this follow-up
@ictechgy ictechgy merged commit 66e4348 into main Jul 5, 2026
1 check passed
@ictechgy ictechgy deleted the fix/codex-cmux-stop-clears-status branch July 5, 2026 13:48
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