Skip to content

fix(errors): keep meta's code, subcode and both sentences in graph error text - #1095

Merged
hoangle398 merged 1 commit into
mainfrom
feat/provider-error-logs
Sep 6, 2026
Merged

fix(errors): keep meta's code, subcode and both sentences in graph error text#1095
hoangle398 merged 1 commit into
mainfrom
feat/provider-error-logs

Conversation

@hoangle398

Copy link
Copy Markdown
Contributor

Every Meta integration threw away half of each Graph failure: code and error_subcode had nowhere to go (ErrorLog has no columns for them), and the developer-facing message was discarded whenever error_user_msg was present.

formatGraphError in @chatbotx.io/utils composes all four into the one text column the error-log table, the inbox failed-message tooltip and the app logger already read, and replaces the per-integration pickErrorMessage copies.

Meta already opens most Graph messages with its own (#100) , so that copy is stripped rather than printed twice. channelErrorMessage now skips an originError detail the composed message already contains, for the same reason.

@github-actions github-actions Bot added the bug Something isn't working as expected label Sep 6, 2026
…ror text

Every Meta integration threw away half of each Graph failure: `code` and
`error_subcode` had nowhere to go (`ErrorLog` has no columns for them), and the
developer-facing `message` was discarded whenever `error_user_msg` was present.

`formatGraphError` in `@chatbotx.io/utils` composes all four into the one `text`
column the error-log table, the inbox failed-message tooltip and the app logger
already read, and replaces the per-integration `pickErrorMessage` copies.

Meta already opens most Graph messages with its own `(#100) `, so that copy is
stripped rather than printed twice. `channelErrorMessage` now skips an
`originError` detail the composed message already contains, for the same reason.
@hoangle398
hoangle398 force-pushed the feat/provider-error-logs branch from da20177 to 47ed74b Compare September 6, 2026 12:41
@hoangle398
hoangle398 merged commit 9ae4314 into main Sep 6, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working as expected

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant