Skip to content

refactor: remove @supabase/server/wrappers module#21

Merged
tomaspozo merged 1 commit intomainfrom
tomas/remove-wrappers-module
Mar 31, 2026
Merged

refactor: remove @supabase/server/wrappers module#21
tomaspozo merged 1 commit intomainfrom
tomas/remove-wrappers-module

Conversation

@tomaspozo
Copy link
Copy Markdown
Member

@tomaspozo tomaspozo commented Mar 31, 2026

Summary

  • Remove the @supabase/server/wrappers entry point and verifyWebhookSignature utility
  • Webhook signature verification varies too much across providers (Stripe uses timestamps, GitHub uses different headers, etc.) to justify a generic helper — each case should use the provider's own SDK
  • The module is currently unused and removing it now keeps the export surface clean for when we add more wrappers with a clearer scope later

Changes

  • Delete src/wrappers/ (source, barrel export, tests)
  • Remove ./wrappers from package.json exports and tsdown.config.ts entry
  • Remove references from SKILL.md and docs/security.md

Webhook signature verification varies too much across providers
to justify a generic helper. Each case should be handled with
the provider's own SDK (e.g., Stripe, GitHub).
Copilot AI review requested due to automatic review settings March 31, 2026 21:22
@tomaspozo tomaspozo changed the title Remove @supabase/server/wrappers module chore!: remove @supabase/server/wrappers module Mar 31, 2026
@tomaspozo tomaspozo requested a review from kallebysantos March 31, 2026 21:23
@tomaspozo tomaspozo changed the title chore!: remove @supabase/server/wrappers module refactor: remove @supabase/server/wrappers module Mar 31, 2026
@tomaspozo tomaspozo enabled auto-merge (squash) March 31, 2026 21:23
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Removes the unused @supabase/server/wrappers public entry point (and its generic webhook signature helper) to keep the package’s export surface focused on core primitives and adapters.

Changes:

  • Removed src/wrappers/ (implementation, barrel export, and tests).
  • Dropped the ./wrappers export from package.json and removed the wrappers build entry from tsdown.config.ts.
  • Updated docs (SKILL.md, docs/security.md) to remove references to the wrappers entry point and verifyWebhookSignature.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.

Show a summary per file
File Description
tsdown.config.ts Removes wrappers from the build entry list.
src/wrappers/webhook.ts Deletes verifyWebhookSignature implementation.
src/wrappers/webhook.test.ts Removes tests covering verifyWebhookSignature.
src/wrappers/index.ts Removes the wrappers barrel export.
package.json Removes ./wrappers from exports.
SKILL.md Removes wrappers entry point and webhook example referencing it.
docs/security.md Removes webhook signature verification section and references to wrappers implementation.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@tomaspozo tomaspozo merged commit 7c67416 into main Mar 31, 2026
6 of 7 checks passed
@tomaspozo tomaspozo deleted the tomas/remove-wrappers-module branch March 31, 2026 21:31
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.

3 participants