Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Improve full tool result viewer#590

Merged
Open-Squilla merged 1 commit into
mainfrom
codex/improve-tool-result-viewer
Jul 10, 2026
Merged

Improve full tool result viewer#590
Open-Squilla merged 1 commit into
mainfrom
codex/improve-tool-result-viewer

Conversation

@Open-Squilla

@Open-Squilla Open-Squilla commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Summary

  • turn “view full” tool output into a file-aware code viewer with filename/path metadata, line numbers, syntax highlighting, wrapping controls, and one-click copy
  • propagate tool name/input/section context from the run trace into the viewer while preserving the existing JSON tree/raw modes
  • add complete locale coverage and rebuild the gateway’s checked-in WebUI bundle

Scope

Scope boundary: WebUI full tool-result rendering, run-trace context propagation, regression tests, six locale files, and the checked-in gateway bundle.

Non-goals: Backend tool-result formats, result retrieval/storage, or a general-purpose file editor.

Branch

Base branch: main

Target exception: N/A

Issue

Linked issue: None

If None, reason: Direct UX improvement requested and validated in the WebUI without a pre-existing issue.

Release Note

Release note: Improve the WebUI full tool-result viewer with file metadata, syntax highlighting, line numbers, wrapping, and copy controls.

Why

The previous drawer rendered every expanded result as a plain pre-wrapped text block. Large file reads and command outputs lost structure, long lines were difficult to inspect, and users had to manually select content to copy it.

Review notes

  • highlighted HTML is sanitized and the existing escaped JSON tree remains intact
  • code and tree scroll regions are keyboard focusable
  • failed read_file calls keep the file path but are displayed as error text instead of being mislabeled from the target extension
  • large payloads retain guardrails for highlighting, line numbers, and JSON tree rendering

Tests

Ruff: Not needed (frontend-only change).

Pytest: Not needed (frontend-only change).

Build: cd opensquilla-webui && npm run build passed, including typecheck, architecture/security guards, theme checks, i18n parity, and the production bundle.

Regression tests: added

Notes: cd opensquilla-webui && npm run test:unit passed — 95 files / 806 tests.

The default test path remains offline, deterministic, credential-free, and safe for forks.

Maintainer Live Check

Maintainer live check: no

Surface: gateway

Maintainer-only note: The implementation was visually exercised against the local WebUI; no credentialed live service is required.

Safety

Secrets, local-only artifacts, private prompts/transcripts, channel identifiers, AI session artifacts, non-public fixtures, and tests/_private/ contents are not committed.

Third-Party Origin

Third-party origin: none

Details if non-none: N/A. This change uses DOMPurify and highlight.js already present in the project dependencies; no upstream source or fixtures were copied.

Documentation Changes

  • No documentation changes are included.
  • Links, code fences, and examples remain unaffected.

@Open-Squilla Open-Squilla marked this pull request as ready for review July 10, 2026 18:59
@Open-Squilla Open-Squilla merged commit bdfbc32 into main Jul 10, 2026
16 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.

1 participant