Skip to content

docs(testing): log the macOS retest of the in-app MCP recording on build run 38598557320 - #1225

Merged
EtienneLescot merged 1 commit into
mainfrom
docs/e2e-retest-38598557320-macos
Oct 14, 2026
Merged

EtienneLescot merged 1 commit into
mainfrom
docs/e2e-retest-38598557320-macos

Conversation

@EtienneLescot

@EtienneLescot EtienneLescot commented Oct 14, 2026 •

Copy link
Copy Markdown
Member

Results row for a scoped macOS retest of defect 1 from the 2026-10-14 macOS pass (#1224): in-app MCP startRecording timing out after 120 s and losing the take.

On main @ 2d98601c4 (build run 38598557320, includes c541fff), macOS 26.5 on an M1 Mac mini: startRecording returns in 4.9 s, stopRecording in 2.7 s with the project the editor opens on, the bar's settings come back, live edits undo with Cmd+Z, and export plus getFrames work. One check stays skipped: the refusal while the editor has unsaved changes could not be set up with a held drag.

Summary by CodeRabbit

  • Documentation
    • Added a macOS in-app recording test result covering faster recording startup, editor handoff, restored settings, and successful editing, export, and frame retrieval. The unsaved-changes refusal remains unverified, and previously noted defects remain open.

@coderabbitai

coderabbitai Bot commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 18ce24fb-50fc-4baa-a3c5-3fe9abe5b478
📥 Commits

Reviewing files that changed from the base of the PR and between 7e914af and 8411292.

📒 Files selected for processing: 1
  • technical-documentation/testing/manual-e2e-checklist.md

Included review availability: This review used your included allowance. Your plan provides up to 8 included reviews per hour; 5 remain after this review.

📝 Walkthrough

The manual E2E checklist now includes a macOS test-results entry for the in-app MCP recording fix. It records successful recording and follow-up checks, identifies an unverified unsaved-changes refusal, and notes that previously reported defects remain open.

Changes: macOS Recording Results

Layer / File(s) Summary
Record macOS MCP test results
technical-documentation/testing/manual-e2e-checklist.md
A new row records prompt recording startup, opening the returned project in the editor, restored bar settings, and successful editing, export, and frame retrieval checks. The unsaved-changes refusal remains unverified, and other previously reported defects remain open.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Other

Suggested reviewers: my-denia

Merge Risk: ⚪ Minimal · up to 84112

This change records scoped macOS test results and clearly notes the unverified check. No actionable merge blocker is identified.

Pre-merge checks: Passed 4 · Failed 1

Failed Failed checks: 1 warning

Check name Status Explanation Resolution
Description check Warning The description explains the retest and testing results, but it does not follow the required template. It omits the Summary, Related issue, Type of change, Release impact, Desktop impact, Screenshots … Add all required template sections. Select the applicable change type, release impact, and desktop impact. Replace "Fixes #" with an appropriate issue reference, such as "Refs #1224" if the issue is not fully resolved. Place the test enviro…
Passed Passed checks: 4 passed
Check name Status Explanation
Title check Passed The title clearly identifies the documentation change and the macOS retest of the in-app MCP recording behavior.
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Description check

Explanation

The description explains the retest and testing results, but it does not follow the required template. It omits the Summary, Related issue, Type of change, Release impact, Desktop impact, Screenshots / video, and Testing headings. The incomplete "Fixes #" entry also does not identify a related issue.

Resolution

Add all required template sections. Select the applicable change type, release impact, and desktop impact. Replace "Fixes #" with an appropriate issue reference, such as "Refs #1224" if the issue is not fully resolved. Place the test environment, steps, results, and skipped check under Testing.

  • Fix all pre-merge checks with AI
✨ 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.

@EtienneLescot
EtienneLescot merged commit 2754466 into main Oct 14, 2026
19 checks passed
@EtienneLescot
EtienneLescot deleted the docs/e2e-retest-38598557320-macos branch October 14, 2026 17:45
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