docs(a11y): add accTitle/accDescr to all Mermaid diagrams — Wave 3B (#513)#526
Conversation
|
Warning Review limit reached
More reviews will be available in 7 minutes and 54 seconds. Learn how PR review limits work. Your organization has run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Repository YAML (base), Organization UI (inherited) Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (4)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Code Review
This pull request adds accTitle and accDescr accessibility attributes to 15 Mermaid diagrams across 8 README files to achieve 100% WCAG 2.2 AA compliance, and introduces a comprehensive audit report, findings CSV, and repair report under .github/reports/mermaid-audit/. Feedback on these changes highlights a structural mismatch in .github/projects/README.md where the diagram and metadata describe the reports directory instead of the projects directory. Additionally, several relative links to instruction files in the newly added audit and repair reports are incorrect and need to be adjusted to resolve properly.
| accTitle: "Reports Directory Structure" | ||
| accDescr { | ||
| Tree diagram showing the .github/reports directory branching into sixteen | ||
| subdirectories: agents, analysis, audits, coverage, frontmatter, implementation, | ||
| issue metrics, labeling, linting, meta, metrics, migration, optimisation, | ||
| tech-debt, and validation. | ||
| } |
There was a problem hiding this comment.
The accessibility title and description added here refer to the "Reports Directory Structure" and .github/reports directory. However, this file is .github/projects/README.md.
There appears to be a copy-paste or structural mismatch in this file, as the entire section and Mermaid diagram describe the reports directory structure instead of the projects directory structure. Please update the diagram and metadata to correctly reflect the projects directory structure (e.g., active/, completed/, planning/, ADR/).
| - [Mermaid Instructions](../../../instructions/mermaid.instructions.md) | ||
| - [Markdown Instructions](.././../instructions/markdown.instructions.md) | ||
| - [Accessibility Instructions](../../../instructions/a11y.instructions.md) |
There was a problem hiding this comment.
The relative paths to the instruction files are incorrect or inconsistent.
Since this report is located at .github/reports/mermaid-audit/audit-report-2026-05-28.md (3 levels deep), going up 2 levels (../../) brings you to .github/, where the canonical instructions/ directory resides.
../../../instructions/mermaid.instructions.mdgoes up 3 levels to the repository root, pointing to a non-existent or non-canonical root-level path..././../instructions/markdown.instructions.mdcontains a redundant.segment.
Correcting these to use ../../instructions/ ensures the links resolve correctly to .github/instructions/.
| - [Mermaid Instructions](../../../instructions/mermaid.instructions.md) | |
| - [Markdown Instructions](.././../instructions/markdown.instructions.md) | |
| - [Accessibility Instructions](../../../instructions/a11y.instructions.md) | |
| - [Mermaid Instructions](../../instructions/mermaid.instructions.md) | |
| - [Markdown Instructions](../../instructions/markdown.instructions.md) | |
| - [Accessibility Instructions](../../instructions/a11y.instructions.md) |
|
|
||
| ## References | ||
|
|
||
| - [Mermaid Instructions](../../../instructions/mermaid.instructions.md) |
There was a problem hiding this comment.
The relative path to the Mermaid instructions file is incorrect.
Since this report is located at .github/reports/mermaid-audit/repair-report-2026-05-28.md (3 levels deep), going up 2 levels (../../) brings you to .github/, where the canonical instructions/ directory resides.
../../../instructions/mermaid.instructions.md goes up 3 levels to the repository root, pointing to a non-existent or non-canonical root-level path. Correcting it to ../../instructions/mermaid.instructions.md ensures the link resolves correctly.
| - [Mermaid Instructions](../../../instructions/mermaid.instructions.md) | |
| - [Mermaid Instructions](../../instructions/mermaid.instructions.md) |
948ed44 to
1236d4a
Compare
🔍 Reviewer Summary for PR #526CI Status: ✅ Recommendations
|
🔍 Reviewer Summary for PR #526CI Status: ✅ Recommendations
|
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
🔍 Reviewer Summary for PR #526CI Status: ✅ Recommendations
|
4966ab1 to
b319fc8
Compare
🔍 Reviewer Summary for PR #526CI Status: ✅ Recommendations
|
1 similar comment
🔍 Reviewer Summary for PR #526CI Status: ✅ Recommendations
|
) Adds missing WCAG 2.2 AA accessibility attributes to all 15 Mermaid diagrams across 8 README files, bringing compliance from 21% to 100%. Publishes Wave 3A/3B audit report, findings CSV, and repair log to .github/reports/mermaid-audit/. No diagram content was changed. https://claude.ai/code/session_01Ukh7BPcSJQcr3zwA6QqqCV
Fixes broken absolute GitHub blob URLs in .github/README.md: migrates coding-standards path from .github/instructions/ to root instructions/, consolidates four deleted block-theme/ links to plugin-structure.instructions.md, and corrects PULL_REQUEST_TEMPLATE.md case to lowercase. Fixes stale relative links in Development Standards section. Removes two auto-generated duplicate footer blocks with stale lsx-demo-theme links. Fixes broken relative link in scripts/README.md (TEST_COVERAGE_SUMMARY.md → README.md). Adds .lycheeignore to exclude Twitter/X and LinkedIn from lychee checks (both block automated HTTP clients). https://claude.ai/code/session_01Ukh7BPcSJQcr3zwA6QqqCV
d5d93c5 to
b00391a
Compare
🔍 Reviewer Summary for PR #526CI Status: ✅ Recommendations
|
b00391a to
b9efba7
Compare
🔍 Reviewer Summary for PR #526CI Status: ✅ Recommendations
|
cf34529 to
7cde6f1
Compare
🔍 Reviewer Summary for PR #526CI Status: ✅ Recommendations
|
1 similar comment
🔍 Reviewer Summary for PR #526CI Status: ✅ Recommendations
|
Summary
accTitleandaccDescrWCAG 2.2 AA accessibility attributes to all 15 Mermaid diagrams across 8 README files that were missing themREADME.mdwas already compliant; 8 other files repaired).github/reports/mermaid-audit/Files changed
.github/README.mdprofile/README.mdscripts/README.mdtests/README.mdscripts/validation/README.md.github/ISSUE_TEMPLATE/README.md.github/projects/README.md.vscode/README.mdDeliverables
.github/reports/mermaid-audit/audit-report-2026-05-28.md— full audit with per-file inventory and compliance summary.github/reports/mermaid-audit/findings.csv— machine-readable findings inventory for Wave 3C tooling.github/reports/mermaid-audit/repair-report-2026-05-28.md— line-by-line repair logAcceptance criteria
accTitleandaccDescrattributes.github/reports/mermaid-audit/Dependencies
Test plan
grep -r 'accTitle' . --include="README.md" | grep -v node_modulesreturns entries for all 8 repaired filesaccTitlewas removed fromREADME.md(root)https://claude.ai/code/session_01Ukh7BPcSJQcr3zwA6QqqCV
Generated by Claude Code