Repository navigation
chore: run the automatable #42 live checks with npm run live-check - #102
Merged
Merged
Conversation
This was referenced Oct 17, 2026
Merged
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.
Adds
npm run live-check(scripts/live-check.mjs), which runs the automatable checks from #42 through the real MCP server (dist/mcp.js) against the installed app, or a dev app named byDESKTOP_TOOLS_STATE. It prints pass, fail or skip for each check, with evidence, and then a summary grouped by PR.--only 82,88filters by PR,--listlists the checks, and--json <path>writes the results to a file.It starts with the 0.8 to 0.10 releases, plus #99 when the build has it: #77, #79, #81, #82, #83, #86, #88, #89, #90, #92, #96, #99, and one #70 wait check. That is 31 checks in all.
Safety:
desktop_appssystem.sessionbefore opening anything./tmp/desktop-tools-live-*folder:<th>table and a grid of inputs.desktop_openand acts only on the new windows.finallyblock, which also runs after Control-C, it:docs/live-checks.mddocuments the harness and lists the checks that stay manual: destructive ones, ones that change system settings, ones that need a second display, Space or password, and ones that need other apps.Verified
--list,--onlyand--only 80(no automated checks, exit 2) work.Refusing to run: the screen is locked or not on the console ({…"locked":true,"on_console":true}). Nothing was opened.npm testandnpm run checkpass.Pending: screen locked
None of the checks has been run on an unlocked session, so element matching (the labels and roles found in Safari and Finder) and the result fields used as evidence haven't been checked against live output yet. Expect a round of fixes after the first unlocked run.