Repository navigation
fix(skills): match Hookdeck value_prefix and API key field names - #269
Merged
Merged
Conversation
Hookdeck's API-key sources for Redo, Oyster, OneSpan, Coterie, UKG Ready, YouMail and OpenClaw now take the bare secret under a type-specific field and accept the header with or without the scheme prefix. Update setup notes, field names and claims about the Hookdeck source to match. The example receivers still compare the whole header value. Also rename the generic API key fields from Header Name/Header Value to API Key Header/API Key wherever a skill describes the Hookdeck source. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01D7eqaQdKAUzQXRbGtym1PF
Generator notes for redo, oyster, onespan, ukg-ready, youmail and openclaw now describe the Hookdeck field and optional scheme prefix, so a regenerated skill doesn't bring back the old setup notes. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01D7eqaQdKAUzQXRbGtym1PF
Generic API key provider notes now name core's field labels, and the Coterie note describes the Token field and optional 'token ' prefix, so regenerating skills keeps the corrected wording. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01D7eqaQdKAUzQXRbGtym1PF
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
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.
Requested by Gareth · project thread
Before: several skills said Hookdeck's source compares the whole
Authorizationvalue and told readers to pasteBearer …(orBasic …/token …) into the source. This affected Redo, Oyster, OneSpan, Coterie, UKG Ready, YouMail and OpenClaw, and some quoted setup notes core no longer shows. Other skills named the generic API key fields "Header Name" / "Header Value".After: these skills name each Hookdeck field as core labels it (Subscription Secret, Shared Secret, Callback Key, Token, Bearer Token, Webhook Token; API Key Header / API Key for generic API key types). They say to paste the bare secret, which the source accepts with or without the scheme prefix, and they quote the current setup notes. The example receivers' verification logic is unchanged. They still compare the whole header, which is a valid choice for the user's own code. Only comments that claimed Hookdeck does the same were removed.
How: core staging
sourceTypeSchemas.ts(labels, configuration_alert) andintegrations/index.ts(value_prefix), withgeneric/apiKey.tsfor the prefix handling. Theproviders/*.yamlgenerator notes for those types were updated too, so regenerating a skill doesn't bring the old wording back../scripts/validate-provider.shpasses for all changed skills.Generated by Claude Code
Update (f97051e): the generator notes in
providers/*.yamlfor 24 generic API key providers now say API Key Header / API Key. The Coterie note now describes the Token field and the optionaltokenprefix.Generated by Claude Code