Skip to content

docs(testing): log the macOS e2e pass on main build run 38581408372 - #1224

Merged
EtienneLescot merged 1 commit into
mainfrom
docs/e2e-main-38581408372-macos-linear
Oct 14, 2026
Merged

EtienneLescot merged 1 commit into
mainfrom
docs/e2e-main-38581408372-macos-linear

Conversation

@EtienneLescot

@EtienneLescot EtienneLescot commented Oct 14, 2026 •

Copy link
Copy Markdown
Member

Supersedes #1212 with the same content as one commit on top of main: #1212 carries two conflict-resolution merges, so GitHub cannot rebase it, and rebase is the only merge method this repository allows. Its CI was green and CodeRabbit's one finding is addressed.

Results row for the computer-use e2e pass on macOS, run against the CI artifact of main @ 0bed36852 (build run 38581408372, notarized DMG, About 2.0.0 (dmg)), on macOS 26.5, Mac mini M1, one 4K display at 2x.

Scope: MCP server section on both servers (stdio and in-app HTTP, SDK client), #1194 on macOS, the #1180 editor tools, MP4/GIF export over MCP (#1197), Openscreen captions, and a scoped app slice. Every take used the legacy selector or MCP; Apple's picker slice was skipped at the operator's request.

Defects found:

  1. P1: in-app startRecording returns "capture had not started" after 120 s while the bar records, and on macOS the take is then lost (no file, no editor). Same root cause as on Windows, fixed on main by c541fff after this build; needs a macOS re-run.
  2. P2: while an Openscreen mcp stdio session runs, opening OpenScreen from Finder or open starts no GUI; LaunchServices activates the mcp process, which turns Foreground with its own Dock icon after its first take. open -n works around it.
  3. P3: getFrames cannot read a GIF ("Chromium could not decode the video (media error 4)"), with path or after a GIF export, on both servers.
  4. P3: an MCP window take with system audio is silent when the sound comes from another app (SCContentFilter(desktopIndependentWindow:) keeps only that app's audio); display takes are fine.

Checklist text: step 4 now names universalAccessAuthWarn as the owner of the macOS permission prompts, and the CI-artifact section records that a launch from Terminal files them under Terminal, with the open --env launch that avoids it.

Summary by CodeRabbit

  • Documentation
    • Recorded observed macOS 26 behavior for Screen Recording and Accessibility prompts, Automation consent, notification alerts, and save panels.
    • Updated packaged-app launch guidance for macOS 13–15, including when to use open -n while an MCP stdio server is running.
    • Added macOS test results covering MCP and recording checks, observed server behavior, defects, and skipped checks.

Partial pass on macOS 26.5 (M1): MCP section on both servers, #1194, #1180 tools, CLI captions and a scoped app slice under the legacy picker. Also records which process owns the macOS permission prompts, what a UserNotificationCenter grant gives, and that a launch from Terminal files the prompts under Terminal. Same content as #1212, in one commit so it can be rebased.
@EtienneLescot
EtienneLescot merged commit a1fddfb into main Oct 14, 2026
17 of 18 checks passed
@EtienneLescot
EtienneLescot deleted the docs/e2e-main-38581408372-macos-linear branch October 14, 2026 16:31
@coderabbitai

coderabbitai Bot commented Oct 14, 2026

Copy link
Copy Markdown
Contributor

Review in Change Stack →

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 419ea594-f168-493a-93c0-c624cec8f128
📥 Commits

Reviewing files that changed from the base of the PR and between 2d98601 and 7a64dca.

📒 Files selected for processing: 1
  • technical-documentation/testing/manual-e2e-checklist.md
 __________________________________________________________
< This abstraction leaks more than a sieve in a rainstorm. >
 ----------------------------------------------------------
  \
   \   (\__/)
       (•ㅅ•)
       /   づ
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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