chore: normalize Hookdeck CLI invocation across existing skills + top-level README#54
Merged
Merged
Conversation
… top-level README Applies the convention set in PR #40 to every pre-existing skill and the top-level Quick Start. Two related changes per file: 1. Replace `hookdeck listen <port> --path /webhooks/<source>` (or `npx hookdeck-cli listen <port> --path /webhooks/<source>` without a source arg) with `npx hookdeck-cli listen <port> <source> --path /webhooks/<source>`. The `[source]` positional is required syntactically — omitting it makes the command fall into an interactive prompt instead of running. 2. Drop the `npm i -g hookdeck-cli` / `brew install hookdeck/hookdeck/hookdeck` prereq lines (and their orphan "# Install Hookdeck CLI" / "# or:" comments). The new `npx` form doesn't need a global install, and webhook-skills is provider-neutral — skills shouldn't push readers to globally install a third-party CLI. Three edge cases handled by hand: - `skills/openclaw-webhooks/references/setup.md` — kept the OpenClaw-specific path `/hooks/agent` and port 18789; just added `openclaw` as the source. - `skills/replicate-webhooks/references/setup.md` — collapsed the multi-line invocation with deprecated `--source` flag into the new positional form. - `skills/hookdeck-event-gateway*` — path `/webhooks` has no provider suffix; used `gateway` as the source name. Excludes `AGENTS.md` (the normative spec lives in PR #40) and the 12 new provider skills (each fixed on its own feat PR: #41–#52). https://claude.ai/code/session_01NNTgQRJss1V7gyzzJ9rjnB
…r SDK 15+ Stripe Python SDK 15.x added a check for `event.object == "v2.core.event"` inside `stripe.Webhook.construct_event()` to route v2 events. The test fixture payloads here build events with only `id`, `type`, and `data` fields, so when `requirements.txt` (`stripe>=7.0.0`) resolves to v15+ the SDK raises `AttributeError: object` on every `construct_event` call. Adds a top-level `"object": "event"` field to the three event fixtures so the SDK can read it (and route to the v1 path). Pre-existing flake — not caused by the docs cleanup in this PR — but resolved here so CI is green end-to-end. Verified locally: all 5 tests pass against `stripe==15.1.0`. https://claude.ai/code/session_01NNTgQRJss1V7gyzzJ9rjnB
This was referenced May 11, 2026
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
2 participants
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
Sweeps every pre-existing skill (and the top-level
README.md) onto the new Hookdeck CLI convention set in PR #40.Per file, two changes:
Replace
hookdeck listen <port> --path /webhooks/<source>(or the in-betweennpx hookdeck-cli listen <port> --path /webhooks/<source>without a source arg) with:npx hookdeck-cli listen <port> <source> --path /webhooks/<source>The
[source]positional is required syntactically — omitting it makes the command fall into an interactive prompt instead of running. Passing it explicitly gives a copy-paste-runnable command.Drop the
npm i -g hookdeck-cli/brew install hookdeck/hookdeck/hookdeckprereq lines (and their orphan# Install Hookdeck CLI/# or:comments). The newnpxform doesn't need a global install, and webhook-skills is provider-neutral — skills shouldn't push readers to globally install a third-party CLI.Scope
93 files touched across 20 existing skill directories plus the top-level
README.md. Net+99 / −340lines.Edge cases handled by hand
skills/openclaw-webhooks/references/setup.md— kept the OpenClaw-specific path/hooks/agentand port18789; just addedopenclawas the source.skills/replicate-webhooks/references/setup.md— collapsed the multi-line invocation with the deprecated--sourceflag into the new positional form.skills/hookdeck-event-gateway*— path/webhookshas no provider suffix; usedgatewayas the source name in those examples.Excluded
AGENTS.md— the normative spec for this convention lives in PR docs: IS_SANDBOX=1 generator note + normative Hookdeck CLI invocation #40 and shouldn't be duplicated.Dependency
Best merged after PR #40 lands so the spec in
AGENTS.md+scripts/skill-generator/prompts/generate-skill.mdmatches the sweep here. There's no file overlap with PR #40, so the merge order is flexible — but readers will be less confused if PR #40 is merged first.Test plan
npx hookdeck-cli listen <port> <source> --path /webhooks/<source>everywhere, no orphan install stepshookdeck-event-gateway*examples still make sense with the genericgatewaysource nameopenclaw-webhookssetup is intact (custom path/port preserved)replicate-webhooks/references/setup.mdreads correctly after the multi-line collapsehttps://claude.ai/code/session_01NNTgQRJss1V7gyzzJ9rjnB
Generated by Claude Code