Skip to content

docs: remove stale not-ported lines from COMPATIBILITY.md - #657

Merged
kblok merged 2 commits into
mainfrom
docs/compatibility-stale-lines
Oct 25, 2026
Merged

kblok merged 2 commits into
mainfrom
docs/compatibility-stale-lines

Conversation

@kblok

@kblok kblok commented Oct 25, 2026 •

Copy link
Copy Markdown
Member

Doc-only. Each line below said the port lacked something it now has. No real-model run (docs only).

Report: no tests added or changed (docs only). No .NET-only symbols.


Generated by Claude Code

Summary by CodeRabbit

  • Documentation
    • Updated compatibility notes for screenshot status-bar handling, Android test ID prefixes, supported mobile providers, and status-bar normalization behavior in TestMu mobile.
    • Clarified that agent turns appear filled on trace pages.
    • Removed outdated notes about route-pattern differences, closed-shadow-root screenshot limitations, and the web engine’s screencast option.

The port has since closed #512 (closed shadow root masking), #134 (record id in route patterns), #445 (agent turns, done by #481) and the Android test id prefix match, and WebEngineOptions has Screencast. MobileEngine normalizes the status bar for its masked screenshot, and the Kernel provider is ported on IBrowserProvider.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Qas4amyVdMjkreo3fcH5tu
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Qas4amyVdMjkreo3fcH5tu
@coderabbitai

coderabbitai Bot commented Oct 25, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Caution

Review failed

The pull request is closed.

Walkthrough

COMPATIBILITY.md updates documented mobile screenshot and test-id behavior, changes the listed MobileEngine providers, removes several parity differences, and updates the trace-page note about agent turns.

Changes: Compatibility documentation

Layer / File(s) Summary
Mobile behavior notes
COMPATIBILITY.md
The screenshot notes describe normalizeStatusBar behavior for MobileEngine and TestMu. The document describes Android test-id prefix matching as ported and lists EAS and Limrun as MobileEngine providers.
Compatibility and trace status
COMPATIBILITY.md
The document removes the route-pattern, closed-shadow-root screenshot, and web screencast differences. It marks agent turns as filled since #445.

Estimated code review effort: 2 (Simple) | ~8 minutes

Change: Other

Suggested reviewers: cursoragent

Merge Risk: Low · up to 2abb0

Users may rely on route patterns that fail to intercept requests with different IDs or query strings; restore the limitation before merging.

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

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

@kblok
kblok merged commit c8885e9 into main Oct 25, 2026
2 of 3 checks passed
@kblok
kblok deleted the docs/compatibility-stale-lines branch October 25, 2026 11:20
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.

2 participants