Repository navigation
feat(accessibility): give a report's score, and send report questions to BrowserStack AI - #568
Merged
Merged
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the
|
… to BrowserStack AI - fetchAccessibilityIssues (first page) and startAccessibilityScan now give the report's accessibility score as the dashboard shows it: the share of points each severity lost, the passed/failed checks per severity, and how the score is worked out (a check is one rule on one page; critical 10, serious 7, moderate 3, minor 1; a failed check counts double). - Read from the report's own summary for Website Scanner runs, Automated Tests builds and Workflow Analyzer reports. A scanner run read by its build id is found in the scan list first. App builds and reports have no score here. - The same results tell the agent to call askBrowserStackAI (product a11y) for anything about the report beyond listing its issues, such as a plan or what to fix first, instead of working out a plan itself. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Aaryan430
force-pushed
the
feat/a11y-report-score
branch
from
November 10, 2026 08:26
ad12d0d to
05352e0
Compare
…es only For a Website Scanner link, the score finds the scan's ids by walking the scan list, up to 25 pages at ~1.2 s a page. The list has only each scan's latest run, so a link to an earlier run is never found: the walk ran in full on every first page, after the export's wait of up to 45 s, and could push the issue list past the client's 60 s timeout. The score now searches the newest 2 pages, which still find a scan that was just run. Reading issues and comparing runs keep the full 25-page walk. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
SavioBS629
approved these changes
Nov 11, 2026
Aaryan430
added a commit
that referenced
this pull request
Nov 11, 2026
Resolves the conflicts with #566 and #568: - accessibilityExpert answers start with the deprecation note, then main's "question was shortened" note when the question was cut, then the answer. - docs/TOOLS.md keeps main's re-run wording for startAccessibilityScan and marks accessibilityExpert deprecated. - accessibilityExpertQuery tests now expect the deprecation note first. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
scoreData). The MCP already reads that summary, but kept only the run settings and the changes since the last run.askBrowserStackAI, the agent made up its own plan from one page of issues. BrowserStack AI answers from the whole report and the product's own knowledge, so report questions should go there.What changed
The score is given with the issues
fetchAccessibilityIssuesand on a finishedstartAccessibilityScan(a new scan or a re-run), a line gives:GET …/scan_runs/{id},…/builds/{id},…/reports/{id}) for Website Scanner runs, Automated Tests builds and Workflow Analyzer reports.Report questions go to BrowserStack AI
askBrowserStackAIwithproduct: a11y, the user's question and the report, for anything beyond listing its issues (a plan, what to fix first, why it scored as it did), and not to work out the plan itself.Verified
tsc, eslint and prettier (src) pass; vitest passes (151 files, 2269 tests), on top of fix(accessibility): land #541 and #553 on main #566.startAccessibilityScan.claude -psessions against a real scan scoring 52, two runs of each question:askBrowserStackAIwith the question and the reportaskBrowserStackAIcall currently fails with a CDN503 first byte timeoutafter about 60 s, on this branch and on the released 2.6.1 alike, while the dashboard's Ask AI answers the same question for the same account. That is being raised separately. Until it is fixed, the agent tells the user BrowserStack AI failed and falls back to its own plan from the issues and score.🤖 Generated with Claude Code