Skip to content

docs(testing): log the Windows re-run of the merged fixes on build run 38370716184 - #1172

Merged
EtienneLescot merged 1 commit into
mainfrom
docs/e2e-v2.1.0-38370716184-windows
Oct 13, 2026
Merged

EtienneLescot merged 1 commit into
mainfrom
docs/e2e-v2.1.0-38370716184-windows

Conversation

@EtienneLescot

@EtienneLescot EtienneLescot commented Oct 13, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Results row for the Windows computer-use re-run of the fixes merged after the v2.1.0 pass (#1132), on build run 38370716184 (release/v2.1.0 fast-forwarded to main @ 912b44800). Verdict Pass for the scoped points, one new minor finding.

Fix Result
#1165 screen vs audio drift (#945) 10:57 take under load: screen, audio and webcam end within 2 ms; HUD-timer flips hold the wall clock to one frame throughout
#1163 export frame grid 4886 frames for 4886.8 predicted at 60 fps
#1154 stdio edit to an open project open: true, edit refused, file untouched
#1145 CLI session data installed mcp locks its own session folder; GUI-side symptom not re-testable while older dev mcp servers hold the profile
#1150 / #1153 MCP wording descriptions and not-found error correct (stack trace still appended)
#1152 chat time first message shows a local time
#1156 Enter in Open project opens the highlighted match
#1161 HUD position bar returns where it was dropped once a popover closes
#1144 identical regions drag stops flush, length kept

New, minor: adding a region with its key while the playhead is inside an existing region of the same kind creates an overlapping region (creation counterpart of #1133).

Type of change

  • Documentation

Desktop impact

  • Windows

Testing

Manual pass through computer-use on the installed CI artifact, measurements with ffprobe/ffmpeg on the files. Docs-only change.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation
    • Added Windows 11 test results covering media-duration synchronization during repeated exports and checks for export frame counts, editor-open project protection, isolated MCP storage, updated messages, chat timestamps, project search, HUD positioning, and speed-region dragging.
    • Recorded an issue where adding a same-kind region with a shortcut can create an overlap when the playhead is already inside a region.

@coderabbitai

coderabbitai Bot commented Oct 13, 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: b3a50a00-a052-4678-bf52-f2d4e0fb0ea8
📥 Commits

Reviewing files that changed from the base of the PR and between 912b448 and fc1f59b.

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

📝 Walkthrough

The manual E2E checklist adds a Windows 11 results entry dated 2026-10-13. It records recording synchronization, export and MCP checks, verified UI and region-drag fixes, and a newly observed same-kind region overlap.

Changes: Windows E2E results

Layer / File(s) Summary
Record Windows test results
technical-documentation/testing/manual-e2e-checklist.md
Adds Windows 11 test observations, including synchronized recording durations, export and MCP checks, UI behavior, region dragging, and a same-kind region overlap when adding a region by shortcut.

Priority: ⬇️ Low

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

Change: Other

Suggested reviewers: my-denia

Merge Risk: ⚪ Minimal · up to fc1f5

This change only adds manual test results; it reports the overlapping-region behavior as an issue rather than a passing check.

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 re-run recorded in the pull request.
Description check Passed The description explains the documentation change, records the test results and minor finding, identifies the change as documentation-only, and specifies Windows impact and testing. It omits the templ…
Linked Issues check Passed No active directly linked issue remains. Issue #1144 is closed and supplies historical context only. Therefore, this pull request has no linked-issue coding requirements. The changed file documents th…
Out of Scope Changes check Passed The whole-PR summary reports one documentation-only change in technical-documentation/testing/manual-e2e-checklist.md. The change records the stated Windows validation results and the new overlappin…
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…
✨ 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 d4cb326 into main Oct 13, 2026
19 checks passed
@EtienneLescot
EtienneLescot deleted the docs/e2e-v2.1.0-38370716184-windows branch October 13, 2026 11:52
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