Skip to content

fix(happy-app): include server error details in RPC failure messages#848

Open
EricSeastrand wants to merge 1 commit intoslopus:mainfrom
EricSeastrand:fix/rpc-error-details
Open

fix(happy-app): include server error details in RPC failure messages#848
EricSeastrand wants to merge 1 commit intoslopus:mainfrom
EricSeastrand:fix/rpc-error-details

Conversation

@EricSeastrand
Copy link
Contributor

Summary

  • Surfaces server error details in RPC failure messages instead of swallowing them
  • One-line change in apiSocket.ts — appends the server's error message to the thrown error

Test plan

  • Verify RPC errors now include the server's error message in the client-side error
  • Verify normal (non-error) RPC calls are unaffected

🤖 Generated with Claude Code
via Happy

Instead of the opaque "RPC call failed", surface the actual error reason
from the server (e.g. "RPC method not available", timeout, etc.).

Generated with [Claude Code](https://claude.ai/code)
via [Happy](https://happy.engineering)

Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Happy <yesreply@happy.engineering>
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