Skip to content

Command error handling#547

Merged
shortishly merged 2 commits intotansu-io:mainfrom
shaneikennedy:command-error-handling
Jan 8, 2026
Merged

Command error handling#547
shortishly merged 2 commits intotansu-io:mainfrom
shaneikennedy:command-error-handling

Conversation

@shaneikennedy
Copy link
Copy Markdown
Contributor

Same logic from #545 but applied at the top level and specific client error handling for relevant commands

I originally picked up #346 which was specifically for topic command handling but I think the logic there applies to all commands:

  • for any non-None ErrorCode, just error! log it because ErrorCode implements Display
  • for any commands that export a Client Error variant that is just a re-export from tansu-sans-io, handle that speicfically. otherwise error! log an unknown error

Same logic from tansu-io#545 but applied
at the top level and specific client error handling for relevant commands
Probably a bit noisy since we're already logging these results in the
main tansu binary
@shortishly shortishly merged commit 64664e8 into tansu-io:main Jan 8, 2026
25 checks passed
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