feat: Restructure Hookdeck skills and add dynamic test discovery #35
Conversation
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
Refactor hookdeck-event-gateway into a high-level overview skill that sells the value proposition (queue replacement, guaranteed delivery, retries, rate limiting, observability) with quick start and doc links, and extract verification code/examples into a new hookdeck-event-gateway-webhooks skill for x-hookdeck-signature handling. - Rewrite hookdeck-event-gateway/SKILL.md as router skill with value prop, before/after code comparison, feature sections linked to docs - Create hookdeck-event-gateway-webhooks/ with verification code, framework handlers, headers reference, examples, and references - Move examples/ and relevant references/ from hookdeck-event-gateway to hookdeck-event-gateway-webhooks - Create outpost/ router skill for Hookdeck Outpost (outbound events) - Update README.md with Webhook Infrastructure Skills section - Update Related Skills in all 17 provider skills with consistent hookdeck-event-gateway description - Update providers.yaml, test-all-examples.sh, test-examples.yml CI, and TESTING.md to reflect new skill location Co-authored-by: Cursor <[email protected]>
Replace hardcoded provider list (5 skills) with filesystem discovery that automatically finds all skills with examples/ directories (19 skills). Accepts optional positional args to test specific skills. - Discover skills via ls skills/*/examples/ instead of hardcoded array - Accept optional args: ./scripts/test-examples.sh stripe-webhooks - Add --help with dynamic list of available skills and frameworks - Update providers.yaml to use default skillName convention - Update CI workflow to remove hookdeck-event-gateway special case - Update TESTING.md scenario 4 for hookdeck-event-gateway-webhooks - Update AGENTS.md references to new script name Co-authored-by: Cursor <[email protected]>
leggetter
changed the title
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.
Summary
hookdeck-event-gatewayinto a high-level router skill that sells the value proposition (queue replacement, guaranteed delivery, retries, rate limiting, observability) with doc links and before/after code exampleshookdeck-event-gateway-webhooksas a dedicated skill forx-hookdeck-signatureverification with framework handlers, examples, and references (moved from the old monolithic skill)outpostrouter skill for Hookdeck Outpost (outbound event delivery)test-all-examples.shwithtest-examples.shthat dynamically discovers skills via the filesystem (19 skills found vs 5 hardcoded), accepts optional positional args to test specific skillshookdeck-event-gatewayRelated Skills descriptionTest plan
hookdeck-event-gateway-webhookslocationtest-examples.sh --helpdiscovers all 19 skills with examplestest-examples.sh stripe-webhooks github-webhooksmulti-arg form workshookdeck-event-gateway-webhooks)Made with Cursor