Repository navigation
Conversation
…ort's counts first - A page of a web report now holds 30,000 characters of issues (about 8k tokens, under Claude Code's 10k-token warning and 50,000-character limit) instead of 10,000. - Each rule's name, description, WCAG criteria, best-practice mark and usual fix are said once per page under rules; an issue keeps its own fix or description only where it differs, and needs_review stays on the issue. The rule text counts against the page the first time the page names the rule, a severity's issues are listed rule by rule (most common rule first), HTML snippets over 400 characters are cut (saying how much the CSV has), and the issues are compact JSON. - The first page (and a scan's answer) starts with the whole report's counts: by severity, needing review, best practices, by rule (top 20) and by page (top 10). The CSV line says it holds every issue. Ported from feat/a11y-issues-per-call (99ff7dd), onto #525's per-issue tags. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…into feat/a11y-more-issues-per-call # Conflicts: # src/tools/accessibility.ts # tests/tools/accessibilityIssueRules.test.ts # tests/tools/accessibilityScanOutput.test.ts
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the
|
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
fetchAccessibilityIssueshanded out 10–17 web issues per call, so a mid-size report took a dozen calls and a large one about 30. There were three reasons:Also, nothing gave the whole report at a glance. To answer "how bad is it?", an agent had to page through everything.
What
A. Bigger pages
B. Compact issues
rules.needs_reviewstays on the issue.needs_review.C. The whole report first
#525's tags (
needs_review, best practice, WCAG criteria) and its dashboard link are kept. Best practice and WCAG criteria now sit on the rule. The tag note keeps #525's correction: only an issue with noneeds_reviewand no best-practice rule is a confirmed WCAG failure. App Accessibility answers are unchanged.Verified
npm run buildpasses: lint, prettier, 1,615 tests, tsc.🤖 Generated with Claude Code