[Aaryan430]

Removes the accessibilityExpert tool. Nobody uses it, so it goes now rather than after a deprecation release.

What changes

• Tool: accessibilityExpert is no longer registered. Its handler in src/tools/accessibility.ts is removed.
• Knowledge-base client: src/tools/accessiblity-utils/accessibility-rag.ts is deleted. It called accessibility.browserstack.com/api/tcg-proxy/search, and nothing else used it.
• Tests:
  • the tool's response test is removed
  • the manual prompt-injection case for the knowledge-base client is removed
  • the tool-registration test now checks for the remaining four tools and that accessibilityExpert is gone
• Docs: its row is removed from docs/TOOLS.md.

The remaining accessibility tools are unchanged: startAccessibilityScan, createAccessibilityAuthConfig, getAccessibilityAuthConfig and fetchAccessibilityIssues.

This PR first marked the tool deprecated (0be9a14), then removed it outright (e8b8ac9). The net diff against main is the removal only.

Merging with the open PRs

• fix(accessibility): read shared report links, wait out large exports, read one page and compare runs #509 and fix(appautomate): ask native runs for devices as App Automate lists them #510 merge cleanly.
• feat(accessibility): read App Accessibility builds, and set up App Automate runs for them #504 and the PRs stacked on it (feat(accessibility): App Accessibility checks on native builds, and App Automate links #505, feat(accessibility): list App Accessibility builds and reports, and read manual reports #506, feat(accessibility): route a11y checks, app build summary + compareWith, exact paging, reliable app build links #511) still contain main's copy of the tool. After this merges, they need a small resolve in docs/TOOLS.md and tests/tools/accessibility.test.ts, where both sides edited the tool list.
• accessibility.ts merges cleanly, and the tool stays removed.

Verification

• The registration test failed before the removal, while accessibilityExpert was still registered, and passes after.
• No test is left referencing the tool or its client.
• npm run build passes on Node 22: lint, prettier, 1349 tests, tsc.
• Against main's 1352 tests, the 3 that go are the two removed tests and the per-tool description-length check for accessibilityExpert.

🤖 Generated with Claude Code

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. 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

• Source mode: Auto
• Repository: Unavailable
• Branch: Unavailable
• File: No configuration file from this repository applied
• Commit: Unavailable
• Layers: Central YAML (base), Organization UI (inherited), Workspace UI (inherited); Enforced overrides: Organization
• Review profile: ASSERTIVE
• Plan: Enterprise
• Run ID: fcd9a650-be75-4bb0-8eae-2db36735085a

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

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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