Skip to content

Conversation

@zackleman
Copy link

Summary

  • Fix Codex approvals failing with MCP error -32602 by returning a valid elicitation result (action: accept|decline|cancel).
  • Map Happy permission decisions (approved/denied/abort) to MCP actions so mobile approvals work reliably.

@ffflyZzz
Copy link

Hi @zleman1593 👋

 Thanks for the fix — this looks solid and addresses a real issue we’re seeing with Codex approvals failing due to MCP error -32602.

Returning explicit MCP elicitation actions (accept | decline | cancel) and mapping Happy permission decisions to MCP actions makes sense, and should significantly improve reliability, especially for mobile approvals.

From what I can see:
✅ Clear problem statement and scope
✅ Logic aligns with MCP expectations
✅ No conflicts with main, clean merge
✅ Single, focused commit

Since this unblocks Codex approval flows and there are no pending checks, it would be great to get this merged when you have a moment.
Thanks again for the contribution!

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