Skip to content

docs(testing): log the Windows MCP pass on main build run 38538195250 - #1191

Merged
EtienneLescot merged 1 commit into
mainfrom
docs/e2e-main-38538195250-windows
Oct 14, 2026
Merged

EtienneLescot merged 1 commit into
mainfrom
docs/e2e-main-38538195250-windows

Conversation

@EtienneLescot

@EtienneLescot EtienneLescot commented Oct 14, 2026 •

Copy link
Copy Markdown
Member

Summary

Results row for the computer-use pass of the MCP server section on build run 38538195250 (main @ b668b2c8d, with #1174 and #1180 merged). Verdict Partial: MCP section only, 2 defects.

Works

Area Result
Recording startRecording with webcam and system cursor; auto-zoom grouping
Transcription and captions transcribe (40 words in about 15 s); French translation by the client; captions on in the bold style
Project look setEditorSettings: frame, wallpaper, 16:9, camera cut-out
Annotations 72 px text; mosaic blur
Clips and media insertClip, duplicateClip, crop, importMedia, removeMedia guard, createProject, setProjectInfo
MP4 export 30 fps, frame count exact; frames show captions, title and blur
Open project live edits, one Ctrl+Z each; restoreCheckpoint is one undo step
App quit and relaunch one client session survives it (file mode with a checkpoint, then live again)
CLI captions writes the v2 model

Defects

Repro Evidence Severity
1 exportVideo to a .gif path, with or without the app "--quality is only supported for MP4 exports"; media-tools.ts always sends quality: "good" P1: GIF export over MCP never works
2 Open the source picker or the language prompt on the bar, then close it HUD style 0x280028 → 0x200000; the bar sinks behind other windows P2: Ctrl+Shift+O brings it back, still not topmost

Not run

  • The in-app HTTP server: copying its token is refused to the agent.
  • The microphone: no working input on this machine.
  • Every section other than MCP.

Type of change

  • Documentation

Release impact

  • No release note needed

Desktop impact

  • Windows

Testing

Manual pass through computer-use on the installed CI artifact, with MCP calls from an SDK client. Measurements with ffprobe/ffmpeg on the files. Docs-only change.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation
    • Added a Windows end-to-end test log covering recording, media import, transcription, captions, editing, export, checkpoints, and CLI captions.
    • Documented two issues found during testing: GIF exports fail, and the recording bar may lose its always-on-top behavior after certain prompts close.

@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: f3658f57-81a5-4877-94b5-65917ceb17e5
📥 Commits

Reviewing files that changed from the base of the PR and between b668b2c and 8997e7d.

📒 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; 7 remain after this review.

📝 Walkthrough

The checklist adds a Windows MCP server test-results entry. It records tested workflows, two defects, other observations, and untested areas.

Changes: Windows MCP test results

Layer / File(s) Summary
Record Windows MCP test results
technical-documentation/testing/manual-e2e-checklist.md
The results log records tested MCP workflows, a GIF export failure, loss of always-on-top state, other observations, and untested checks.

Priority: ⬇️ Low

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

Change: Other

Suggested reviewers: my-denia

Merge Risk: ⚪ Minimal · up to 8997e

This documentation-only change records a scoped Windows MCP test pass and its findings without changing product behavior. No actionable merge-blocking risk is established.

Pre-merge checks: Passed 5
Passed Passed checks: 5 passed
Check name Status Explanation
Title check Passed The title clearly identifies the documentation change and the Windows MCP pass recorded for the main build.
Description check Passed The description is complete and matches the template. It explains the documented results, defects, scope, platform, testing method, and release impact. The blank related-issue field is non-critical fo…
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.
✨ 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

Warning

Some tools did not complete. Review the errors below.

🔧 LanguageTool

LanguageTool checks are incomplete after three consecutive availability failures. Remaining chunks and files were skipped; findings from completed checks are retained.

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 5bc5591 into main Oct 14, 2026
19 checks passed
@EtienneLescot
EtienneLescot deleted the docs/e2e-main-38538195250-windows branch October 14, 2026 12:31
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