Skip to content

Conversation

@luxiaolei
Copy link
Contributor

Summary

  • emit a ready event once the Codex queue drains so clients leave the busy state
  • mirror the Claude push notification to keep mobile/web in sync
  • factor the ready emitter into a helper with unit coverage

Fixes slopus/happy#140

Testing

  • corepack yarn test

@luxiaolei luxiaolei force-pushed the fix/codex-ready-event branch 3 times, most recently from 7beb1d4 to ec3165c Compare September 29, 2025 02:33
@luxiaolei luxiaolei force-pushed the fix/codex-ready-event branch from ec3165c to e76d260 Compare September 29, 2025 02:39
@luxiaolei
Copy link
Contributor Author

Update: addressed Codex MCP conversation ID handling so codex-reply no longer errors when conversation_id is omitted. Retested end-to-end via DEBUG=1 happy codex (mobile + CLI) including multiple prompts and tool executions; sessions now stay responsive after repeated tasks.

@zhhugh
Copy link

zhhugh commented Oct 1, 2025

hi, the bug of "task finished after the first requeset in codex" is fixed by these new changes, isn't it ?

@xiang-lee
Copy link

Any updates for this PR to be merged?

@jcr7467
Copy link

jcr7467 commented Oct 7, 2025

@bra1nDump , any chance we could get this merged? Can't use codex on the Happy app, can only send 1 message per session before it hangs and have to make a new one

@bra1nDump bra1nDump merged commit e1f6bff into slopus:main Oct 9, 2025
@bra1nDump
Copy link
Contributor

Slipped my radar, thanks for the ping!

Merged and will release the new beta cli version shortly to test

@bra1nDump
Copy link
Contributor

Thank you
@luxiaolei for the fix!
@zhhugh @xiang-lee @jcr7467 for pings

Published to npm version 0.11.1

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.

Bug about codex

5 participants