test(webviews): keep base green on ui-rules growth from 7f67cf1a538 and eb596157723

[teamleaderleo]

Summary

Base red: ci-web's "ui rules > migrated page code has no raw roles, tab indexes, key handlers or portals" fails on feat-cmux-next with three new violations:

File	Violation	From
agent-session/acpmux/conversation/Markdown.tsx:522	role="checkbox"	7f67cf1 (accessible task checkboxes)
pages/markdown/editor.ts:481	setAttribute("role", "checkbox")	7f67cf1
agent-session/acpmux/changes/treeMotionDom.ts:275	addEventListener("keydown", ...)	eb59615 (changes-tree disclosure)

To get base green now without changing behavior:

• The two acpmux files go on ui-rules.pending.json with a count of 1 each. The note names both commits, so their owners clear the entries when they move the code onto src/ui.
• pages/markdown/editor.ts can't be pending: "the migrated pages stay off the pending list" forbids pages/markdown/. So the line gets the scanner's ui-allow: exemption with a reason. It is a read-only task mark drawn as a ProseMirror decoration, which src/ui can't render; the comment names 7f67cf1.

No runtime code changes; the only addition to source is the comment.

Verification

• Red: ci-web react-apps-check on run 37588214371 (base at the time) lists exactly these three violations.
• I ported the scanner's rules (webviews/test/ui-rules.ts) to Python and ran it on this head. It finds no fresh, grown or stale entries, so all three ratchet tests should pass. The bun suite runs in hosted ci-web.

Changelog

none

🤖 Generated with Claude Code

---

^{Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.}

---

Summary by cubic

Keeps the ui-rules ratchet green on feat-cmux-next after recent changes added raw widget roles and a keydown handler. The two acpmux files go on the pending list for their owners to clear; the pages/markdown/editor.ts violation gets a ui-allow exemption because pages/markdown/ is a migrated page that cannot be pending and the role is a read-only ProseMirror decoration.

^{Written for commit 9b54d1f. Summary will update on new commits.}

---

Note

Low Risk
Test-only ratchet updates and a comment exemption; no application logic changes.

Overview
Restores ci-web ui-rules on feat-cmux-next after new a11y-related violations from recent commits, without changing runtime behavior.

pages/markdown/editor.ts gets a scanner ui-allow: comment above the existing role="checkbox" on the ProseMirror task decoration (migrated markdown pages cannot use the pending list).

webviews/test/ui-rules.pending.json documents those two commits in its header note and adds pending entries (count 1 each) for conversation/Markdown.tsx (task checkbox role) and changes/treeMotionDom.ts (tree keydown), to be cleared when owners migrate to src/ui.

^{Reviewed by Cursor Bugbot for commit 9b54d1f. Bugbot is set up for automated code reviews on this repo. Configure here.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

• Configuration used: Repository: manaflow-ai/cmux/.coderabbit.yaml
• Review profile: ASSERTIVE
• Plan: Advanced
• Run ID: 850f87f7-6604-4fa8-9132-ddb065e46988

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

• Autopilot · 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

• X
• Mastodon
• Reddit
• LinkedIn

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

[github-actions]

Unclear: web / react-apps-check failed: no known signature; failed step: Test webviews.

CI failed on 9b54d1f329 (run 37594825763 attempt 1): 1 unknown.

Job	Verdict	Why
web / react-apps-check	unknown	no known signature; failed step: Test webviews

Not re-run automatically: web / react-apps-check is not a machine failure.

Written by scripts/ci/classify_failures.py (ci-failure-attribution.yml); signatures are its SIGNATURES table. A machine verdict is the runner's fault, not this PR's; yours means the failing file is one this PR changes, also red on main that main's latest full suite fails the same way, seen on other PRs that it failed on another pull request's run lately.

[github-actions]

The feat-cmux-next push run https://github.com/manaflow-ai/cmux/actions/runs/37597620459 at e39f213 failed: cmux-next checks (god files, concurrency, crash safety, l10n).
Those jobs last passed at 1aa7972. This pull request is one of 3 merged since: #18311, #18301, #18317.
If the failure is in your change, fix forward on feat-cmux-next. Pull requests run only the tiers their paths reach (docs/ci/cmux-next-tiers.md); label a batch PR full-ci to run them all before merging.