Skip to content

Improve request log error details UX#296

Merged
Soju06 merged 1 commit intoSoju06:mainfrom
L1st3r:fix/improve_log_error_details
Apr 3, 2026
Merged

Improve request log error details UX#296
Soju06 merged 1 commit intoSoju06:mainfrom
L1st3r:fix/improve_log_error_details

Conversation

@L1st3r
Copy link
Copy Markdown
Contributor

@L1st3r L1st3r commented Apr 1, 2026

This PR makes the dashboard request logs usable for debugging failures.

The request logs table now exposes a clear View Details action for error rows, replaces the old hard-to-discover truncated error behavior, and opens a request details dialog with
full metadata plus the complete error text. The dialog layout was iterated to reduce wasted space, keep long request IDs readable, and place copy actions where users expect them.

Changes

  • add a visible request-details entry point for error rows in the recent requests table
  • show full request metadata in the dialog, including request ID, status, model, transport, time, and error code
  • show full untruncated error text in the dialog
  • add compact copy actions for request ID and full error text
  • tighten dialog copy and layout to reduce clutter

Validation

  • npm test in frontend/
  • focused Vitest and ESLint checks during iteration for the request logs UI

Notes

The full frontend suite passes: 59 files, 292 tests. Existing non-failing warnings remain in unrelated dialog/canvas test paths.

@L1st3r L1st3r requested a review from Soju06 as a code owner April 1, 2026 22:48
@L1st3r L1st3r force-pushed the fix/improve_log_error_details branch from 9b2ed46 to 72297e0 Compare April 2, 2026 03:01
  - add visible request detail access from error rows
  - show full request metadata and untruncated error text in dialog
  - add compact copy actions for request id and full error
  - refine dialog layout to reduce wasted space and improve scanability
  - update dev-local compose for hot-reloading frontend/backend work
@L1st3r L1st3r force-pushed the fix/improve_log_error_details branch from 72297e0 to c07cb9a Compare April 3, 2026 02:26
@Soju06
Copy link
Copy Markdown
Owner

Soju06 commented Apr 3, 2026

@codex review

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: c07cb9a33d

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@Soju06 Soju06 merged commit b8b98ae into Soju06:main Apr 3, 2026
15 checks passed
@Soju06
Copy link
Copy Markdown
Owner

Soju06 commented Apr 3, 2026

@all-contributors please add @L1st3r for code, test

@allcontributors
Copy link
Copy Markdown
Contributor

@Soju06

I've put up a pull request to add @L1st3r! 🎉

@Soju06
Copy link
Copy Markdown
Owner

Soju06 commented Apr 3, 2026

Thank you for your 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