Repository navigation
feat(desktop): simplify Diagnostics to a status line and Copy report - #7413
Merged
Merged
Conversation
Run diagnostics opened a section in Settings > Advanced that listed about twenty rows of internal state, waited for a Refresh click before checking anything, and copied only the Cloud tools state. Show one plain status line, the app and engine versions, and a Copy report button. The check runs once when a signed-in person opens the page. Copy report copies the same sanitized report as the command palette's "Copy diagnostics", with the result of the check included. The previous rows and the health JSON stay available under a closed "Technical details". The section is renamed Diagnostics in settings search and in the hints that point to it. Developer mode still opens the Debug page. Signed-off-by: Jalil <reachjalil@gmail.com>
Contributor
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
Contributor
OpenWork evidence
Run details · Open the preview to inspect evidence and sandbox options. Started by @reachjalil. Author evidence: these tests run the PR's own code. Not a CI verification or merge gate. |
Settings can open before the workspace has a server connection. The automatic check ran once at that moment, got no result, and never ran again, so the section stayed on "Not checked" until someone pressed Check again. Wait for the connection before the one automatic check, say so while waiting, and keep Check again disabled until it can run. The proof now accepts any finished result of the check. Signed-off-by: Jalil <reachjalil@gmail.com>
This branch had an error being deployed
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 is this about?
Settings → Advanced → Diagnostics in the desktop app, the screen "Run diagnostics" opens. It now shows one status line, the app version and a Copy report button.
What problem does it solve?
Someone who hits a problem wants to know what is wrong and send us something we can read. The section answered with about twenty rows of internal state, and its copy button left out the app version and the logs.
What was the situation before?
"Agent access diagnostics" listed rows such as Desired revision and Plugin canaries. Nothing was checked until you pressed Refresh, and "Copy sanitized diagnostic" copied only the Cloud tools state.
How is this implemented?
advanced-diagnostics-section.tsxreplaces the old section: status, versions, Check again, Copy report, and the old rows inside a closed "Technical details".diagnostics-summary.tsturns the health result into one plain line, reusingcloudMcpDisplaySummary.settings-route.tsxcopies the command palette's "Copy diagnostics" report (buildDiagnosticsBundleJson), with the check result included.What else changed?
evals/worlds/account-diagnostics.tsgrants clipboard access so the proof can read what was copied.Release note
Settings → Advanced → Diagnostics now tells you in one line whether your connected tools work, with the app version and a Copy report button for sharing with support.
Evidence
evals/proofs/diagnostics-copy-report.proof.test.ts— before: about twenty rows of internal state behind a Refresh button; after: one status line, the versions and Copy report, with the rows closed under Technical details.Design: P1, P2, P3, S1, S3, C6