Skip to content

feat: expose webhook helpers via ./webhooks subpath export (Closes #410) - #457

Merged
David-patrick-chuks-02 merged 8 commits into
Lilly-Protocol:mainfrom
Zhiyilang074811:fix-webhooks-410
Sep 6, 2026
Merged

feat: expose webhook helpers via ./webhooks subpath export (Closes #410)#457
David-patrick-chuks-02 merged 8 commits into
Lilly-Protocol:mainfrom
Zhiyilang074811:fix-webhooks-410

Conversation

@Zhiyilang074811

Copy link
Copy Markdown
Contributor

Summary

Publishes the existing webhook signature helpers through a ./webhooks\ subpath, making \�erifyWebhookSignature, \�erifyWebhookJSON, \parseWebhookHeader, and \�erifyWebhookWithReplay\ available to consumers.

Changes

  • tsup.config.ts: Added \src/webhooks.ts\ to the Node build entry points
  • package.json: Added ./webhooks\ export entry with import/CJS/type conditions
  • src/index.ts: Added root re-exports for all four webhook helpers
  • test/dist-smoke.mjs: Added runtime checks for all four exports in both ESM and CJS
  • tests/api-report-snapshot.test.ts: Added \webhooks.d.ts\ to the subpath .d.ts existence check
  • tests/exports-subpath-resolves.test.ts: Added ./webhooks\ to the expected subpaths assertion

Acceptance Criteria

  • \import { verifyWebhookSignature } from '@lily-protocol/sdk/webhooks'\ resolves against the built package (verified by exports-subpath-resolves test)
  • The four helpers appear in the dist smoke test and will ship as \webhooks.d.ts\ in the package
  • Root re-exports ensure they're also accessible from the main entry point

Closes #410

@Zhiyilang074811

Copy link
Copy Markdown
Contributor Author

Hi! Friendly ping — open for a couple of days, all CI checks passing. Ready for review whenever convenient. Thanks! 🙏

@Zhiyilang074811

Copy link
Copy Markdown
Contributor Author

Hi! Friendly ping — bounty PR (#410) open for ~2 days, all checks passing. Ready for review. Thanks! 🙏

@David-patrick-chuks-02
David-patrick-chuks-02 merged commit a9727d6 into Lilly-Protocol:main Sep 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bounty: $75] Expose the webhook signature helpers via a public ./webhooks export subpath

3 participants