Repository navigation
docs(readme): fix broken bundle badge, add docs/CI/types badges, group hook index - #8
Conversation
…p hook index - Replace the chronically rate-limited bundlephobia badge (renders "rate limited by upstream service" on npm) with always-green npm/types + GitHub Actions CI-status badges; add a docs badge linking hookli.vercel.app. - Group the Available-hooks index by category (State / Effects / DOM / Data) with per-category counts, so npm/GitHub visitors can skim 65 hooks. - gen-readme.mjs emits the grouped index between HOOKS:START/END markers and fails loudly on a hook with an unknown category. Mirrors to the repo root. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AuT9BTjvgatPxvkXSPSo1i
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
ECC bundle files are already tracked in this repository. Skipping generation of another bundle PR. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Team Run ID: 📒 Files selected for processing (3)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe README files replace the Bundlephobia badge and flat hook list. They now use documentation and CI badges, generated markers, and categorized hook sections. The generator renders these sections, counts hooks, and validates categories and markers. ChangesGenerated Hook Index
Estimated code review effort: 2 (Simple) | ~10 minutes Merge Risk: ⚪ Minimal · up to This change updates documentation badges and groups the generated hook index without changing runtime behavior; no actionable merge-blocking risk remains after normal checks and review. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Description checkExplanation The description explains the purpose, lists the key badge and hook-index changes, documents the npm publishing limitation, and reports successful validation. It does not use the template's exact "## Changes" and "## Checklist" headings, but it provides the required information overall. Full details: Docstring CoverageExplanation No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1 files. (2 skipped: 2 unsupported.) ✨ Finishing Touches📝 Generate docstrings
🧪 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 |
What
Enhances the npm/GitHub landing (README) — the page shown on npmjs.com and the repo home.
Badges
ci.yml) andtypes: TypeScript(npm/types).hookli.vercel.app/docs.Hook index
gen-readme.mjsgenerates the grouped index betweenHOOKS:START/HOOKS:ENDmarkers (mirrors category order/labels fromapps/docs/lib/hooks-registry.ts), and fails loudly if a hook has an unknown category. Still manifest-driven — zero drift.Notes
bash ralph/check.sh— 65 test files, 196 tests, typecheck + tsup build pass.🤖 Generated with Claude Code
Summary by CodeRabbit