Dev - #379
Conversation
- Add plugin.json for review-fix-a11y skill - Add plugin.json for free-llm-api skill - Restore POSIX-compliant trailing newlines in JSON index files
…ix) (#375) Adds review-fix-a11y (WCAG 2.2 a11y audit + fix) and free-llm-api skills. Includes: - review-fix-a11y: WCAG 2.2 audit workflow, a11y_audit.py scanner, contrast_checker.py - free-llm-api: ChatAnywhere, Groq, Cerebras, OpenRouter, llm-mux, One API setup - secret_scanner.py upgrade with secrets-patterns-db integration (1,600+ patterns) Co-authored-by: ivanopenclaw223-alt <ivanopenclaw223-alt@users.noreply.github.com>
… audit & fix (#376) Built from scratch (replaces reverted PR #375 contribution). Skill package: - SKILL.md: 1132 lines, 3-phase workflow (scan → fix → verify), per-framework fix patterns (React, Next.js, Vue, Angular, Svelte, HTML), CI/CD integration guide, 20+ issue type coverage - scripts/a11y_scanner.py: static scanner detecting 20+ violation types across HTML/JSX/TSX/Vue/Svelte/CSS — severity-ranked, CI-friendly exit codes - scripts/contrast_checker.py: WCAG contrast calculator with AA/AAA checks, --suggest mode, --batch CSS scanning, named color support - references/wcag-quick-ref.md: WCAG 2.2 Level A/AA criteria table - references/aria-patterns.md: ARIA roles, live regions, keyboard interaction - references/framework-a11y-patterns.md: React, Vue, Angular, Svelte fix patterns - assets/sample-component.tsx: sample file with intentional violations - expected_outputs/: scan report, contrast output, JSON output samples - /a11y-audit slash command, settings.json, plugin.json, README.md Validation: 97.6/100 (EXCELLENT), quality 73.9/100 (B-), scripts 2/2 PASS Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…s, 22 plugins Update CLAUDE.md, README.md, docs/index.md, docs/getting-started.md, mkdocs.yml, marketplace.json with consistent counts. Sync Gemini CLI index with new skills (code-to-prd, plugin-audit). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
PR Review: a11y-audit Skill + Index Updates. Note: This PR is already merged. Findings are post-merge feedback. Issues Found: (1) Bug: Two entries in .gemini/skills-index.json (plugin-audit, code-to-prd) have description set to pipe character only - needs proper descriptions. (2) Documentation Mismatch: SKILL.md and README.md document --fg/--bg/--file flags not present in contrast_checker.py - actual script uses positional args and --batch instead of --file. (3) SKILL.md Frontmatter uses both PascalCase and lowercase duplicate keys - use lowercase-only per repo conventions. (4) CLAUDE.md still says Engineering Core (25) but this PR adds a 26th skill - update to (26). What is Good: stdlib-only Python scripts with CI-friendly exit codes (0/1) and --json/--demo/--help support; correct plugin.json ClawHub fields; proper skill directory layout; WCAG 2.2 new criteria coverage (Focus Appearance 2.4.11, Dragging Movements 2.5.7, Target Size 2.5.8); platform registration updated across .codex/, .gemini/, .claude-plugin/, and symlinks. Recommended Follow-up: (1) Fix pipe-only descriptions in .gemini/skills-index.json. (2) Correct CLI examples in SKILL.md and README.md. (3) Update CLAUDE.md Engineering Core count to (26). (4) Clean SKILL.md YAML frontmatter to lowercase-only keys. |
2 participants
Summary
Checklist
dev(notmain— PRs to main will be auto-closed)SKILL.mdwith valid YAML frontmatter (name,description,license)--helpwithout errorsdomain/skill-name/SKILL.md)Type of Change
Testing