feat(hooks): extract scripts, add coverage hook, agent-command CI val…#1
Merged
codebysantosh-eng merged 6 commits intoApr 18, 2026
Merged
Conversation
…idation - Extract all 10 hook one-liners into readable hooks/scripts/*.js files - Add hooks/build.js to regenerate hooks.json from scripts (shell-escape pipeline) - Add coverage-threshold-warn hook (warns when jest/vitest coverage < 80%) - Update run-tests.sh to pipe stdin to scripts directly (25 tests, up from 21) - Add CI steps: hooks-in-sync check, every-hook-has-script check, agent-command mapping validation Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…n is current Ensures the generated hooks.json is always rebuilt from hooks/scripts/ at install time, so importing via Buildery or cloning and running install.sh never uses a stale committed artifact. Falls back gracefully with a warning if node is not available. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…commit time CI sync check guarantees hooks.json is always current before merge. No build step needed at install time; Buildery imports all files under hooks/ including scripts as-is. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Adds /onboard slash command that detects project stack, prints the toolkit summary and workflow overview, gives a personalised first-step recommendation, explains active hooks, and invites the first action. Updates install.sh summary to suggest /onboard after install. Updates CLAUDE.md command count 14 → 15. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- uninstall.sh now removes only Forge hooks by ID (mirrors install.sh deep-merge logic) instead of wiping the entire hooks key — user's custom hooks are no longer destroyed on uninstall - README.md: 14 → 15 commands, 10 → 11 hooks; add /onboard to commands table; add coverage-threshold-warn to hooks table - CLAUDE.md: 14 → 15 slash commands in structure section Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Add six new sections to skills/observability/SKILL.md: - Context propagation across service boundaries (W3C traceparent + baggage) - Sampling strategies: head-based, parent-based, tail-based with Collector config - Database span instrumentation (Prisma named spans + slow query events) - Hono observability middleware (logging + tracing + context extraction) - Health check endpoint patterns (Hono + Next.js App Router) - Local dev Docker Compose for Jaeger + Prometheus + Grafana Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
codebysantosh-eng
deleted the
feat/hook-maintainability-coverage-ci-validation
branch
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
1 participant
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.
…idation