Repository navigation
fix(research): make cited command output readable - #7287
Merged
Merged
Conversation
Signed-off-by: LoopX Agent <337587101+loopx-agent@users.noreply.github.com>
loopx-agent
requested review from
huangruiteng,
maxliux5 and
songoow
as code owners
October 24, 2026 20:32
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Goal And Delivered Outcome
Opening cited command evidence currently shows an escaped JSONL event, making the printed result hard to inspect. The shared Research citation reader now presents the exact output and exit code first, with command arguments and the complete original record available in place.
This completes a bounded part of progressive research reading: ordinary report or deep explanation → citation → readable evidence → return to the same citation. Target:
mainat8a718ca5f17b8036a57c6d4e88a02185eaf4581d.Author Declaration
docs/development/design.md(Research Workspace, Explanations On Demand and Populated Page Review) anddocs/development/frontend-delivery.md(Task-first delivery), at8a718ca5f17b8036a57c6d4e88a02185eaf4581d.ResearchCitationPanelingoal-research-analysis.tsxCitationContentand existing source stylesScope And Continuation
The presenter recognizes
event_msg/item_completed/CommandExecutionrecords. Unknown, mixed, malformed or numerically unsafe records retain the complete literal fallback. Source eligibility, hashes, line ranges and analysis execution remain with the existing backend owner.Complete within this scope. Broader analysis quality and release acceptance remain separate. This is a local renderer in the existing component, with no new API or generic event-decoding module; later formats can retain the fallback until explicitly supported.
Validation
b398a068f640d4e13d93fdad9c134c7093f271f7; the packaged tests and captures used these same committed component/CSS/fixture contents, verified against source after commit.research-native-reading: actual registration, qualified source and HTTP owner; no intercepted citation responsesresearch-historyscenario, including existing read failure/recovery pathsThe new presentation changes no analysis or experiment results. Installed App adoption, reader task-time benefit and scientific interpretation quality are unqualified. No merge or auto-merge is requested.
Frontend / Visual Evidence
8a718ca5f17b8036a57c6d4e88a02185eaf4581d, after sourceb398a068f640d4e13d93fdad9c134c7093f271f7. No public hosted preview is available.Type of Change
LoopX Area
Technical Direction
Operator surface; the existing Research evidence-reading contract. Shared-authority RFC fixture impact: N/A; no authority-store, routing, compatibility or control-plane decision change.
Boundary Checklist