docs(simulation): capture Fix My Agent and correct four claims - #816
Open
khushalsonawat wants to merge 2 commits into
Open
docs(simulation): capture Fix My Agent and correct four claims#816khushalsonawat wants to merge 2 commits into
khushalsonawat wants to merge 2 commits into
Conversation
The page had no images at all. Its tracker row was filed as a video, so it never entered the screenshot queue. Adds one capture that carries the whole page in a single frame: the panel open beside the tabs, the run Summary, the suggestion count, Optimize My Agent, the Agent Level and Branch Level toggle, and two branch-level issues with their priority, recommendation, branch category and affected calls. Both execution identifiers are masked. Corrects what the capture disproved: - there is no domain level. The toggle is Agent Level and Branch Level, and a branch-level entry names its path on a Branch Category line - the counts are not two but nested: Suggestions heads the group, and the per-tab count says how many of that total the current tab shows. The two tabs sum to the heading - Infra based suggestions is about the agent's runtime, latency, timeouts and looping, not about how it is set up. Its entries carry the same priority, recommendation and calls as the prompt-based ones, and the group opens with a Human Comparison Summary - the collapse chevron sits at the panel's left edge. Without that, clicking Fix My Agent reads as doing nothing Also names the see more truncation on a recommendation, and the grid header switching from All Chats to (n) Chats selected when an issue is selected, which was verified against a suggestion reporting six affected calls. The empty-state string this page quotes remains unverified: the run carries a cached analysis, so that state was not reachable. Signed-off-by: khushalsonawat <khushalsonawat@gmail.com>
Reopening the panel shows the stored analysis rather than starting a new one, stamped Last updated at beside the suggestion count. Two executions on the same run carried analyses from different days, which is what makes the empty state a first-time-only thing rather than something you see whenever you open the panel. This is also why the empty-state string the page quotes stayed unverified in the previous commit: every execution reachable here has already been analysed. Signed-off-by: khushalsonawat <khushalsonawat@gmail.com>
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.
What
The Fix My Agent page had no images at all. Its tracker row was filed as a video, so it never entered the screenshot sweep and the gap went unnoticed until now.
Adds one capture that carries the whole page in a single frame: the panel open beside the tabs, the run Summary, the suggestion count, Optimize My Agent, the Agent Level / Branch Level toggle, and two branch-level issues with their priority chip, recommendation, branch category and affected calls. One callout, on the toggle. Both execution identifiers masked.
Driving the panel to get that shot turned up four wrong claims and one missing behaviour.
Why
One image is the right number here. Every state this page describes lives in the same panel, so a second shot would have been the same screenshot with a different row highlighted, which is the pattern the reader-analyst passes on #814 called out as training people to stop looking at images.
What cases does this cover
How
Captured over CDP at 3024×1890 on a completed 100-chat run, annotated per
annotation-conventions.md. The oval was placed from anempty_boxesscan rather than by eye, and sits on canvas measuring flat and dark.Claims were checked rather than assumed:
Masking: both execution identifiers, in the breadcrumb and the heading.
Known gaps
Recording
n/a — static page.
Checklist