Skip to content

fix(ui): preserve structured API error messages - #1027

Closed
wunitb wants to merge 1 commit into
Agent-Field:mainfrom
wunitb:fix/structured-api-errors
Closed

fix(ui): preserve structured API error messages#1027
wunitb wants to merge 1 commit into
Agent-Field:mainfrom
wunitb:fix/structured-api-errors

Conversation

@wunitb

@wunitb wunitb commented Aug 31, 2026

Copy link
Copy Markdown

Summary

  • extract user-facing messages from nested API error payloads instead of coercing objects to [object Object]
  • apply the shared parser to session and control-plane UI requests
  • keep unknown structured payloads fail-closed behind existing status-based fallback messages

Verification

  • npm test -- --run — 132 files, 689 tests passed
  • npm run build
  • npx eslint src/services/errorMessage.ts src/services/sessionsApi.ts src/services/api.ts src/test/services/errorMessage.test.ts src/test/services/sessionsApi.test.ts
  • git diff --check

@wunitb
wunitb requested review from a team and AbirAbbas as code owners August 31, 2026 17:19
@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@wunitb
wunitb force-pushed the fix/structured-api-errors branch from c7507d1 to b9837a5 Compare August 31, 2026 17:35
@wunitb

wunitb commented Aug 31, 2026

Copy link
Copy Markdown
Author

Closing: this change was opened against the wrong upstream repository after I misattributed the downstream m5bp UI to the requested project scope.

@wunitb wunitb closed this Aug 31, 2026
@wunitb
wunitb deleted the fix/structured-api-errors branch August 31, 2026 17:38
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