Skip to content

feat(hub)!: mount @corbits/webhooks; delete webhook-triggers and collections (CL-8389) - #848

Merged
TheGreatAxios merged 2 commits into
mainfrom
cl-8389-corbits-webhooks
Sep 18, 2026
Merged

TheGreatAxios merged 2 commits into
mainfrom
cl-8389-corbits-webhooks

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Contributor

Summary

  • Pin apps/hub to github:corbitsdev/webhooks#570bd52688920c0ce018a8fbcd046199fa4125dd (v0.1.2) and mount installWebhooks once, alongside the hub's other Corbits libraries: a webhook is now a tenant credential with metadata.webhook, verified by POST /api/hooks, firing the workflow's onTrigger mail as the run principal.
  • A thin, in-repo adapter narrows HookMailRouter's unknown-typed payloads back to sidecarRouter's own types at the mount boundary — no changes to the library or to vendored Interchange code.
  • Delete packages/webhook-triggers (own table, ingress, and management routes) and packages/collections (webhook-triggers' only consumer), the migrateHub step for webhook-triggers, and apps/web's webhook-secret-panel/webhook-triggers-api — a webhook is created through the existing credentials section.
  • Clean workspaces, tsconfig.build.json, and docs (ARCHITECTURE.md) of webhook-triggers/collections references.

Test plan

  • bun run check (typecheck, lint, fmt, test) green
  • bun run build in apps/web succeeds

@linear-code

linear-code Bot commented Sep 18, 2026

Copy link
Copy Markdown

CL-8389

…ections (CL-8389)

Pin apps/hub to github:corbitsdev/webhooks#570bd526 (v0.1.2) and mount it
once alongside the hub's other Corbits libraries; a webhook is now a
tenant credential with metadata.webhook, verified by POST /api/hooks,
firing the workflow's onTrigger mail as the run principal. A thin,
in-repo adapter narrows HookMailRouter's unknown-typed payloads back to
sidecarRouter's own types at the boundary -- no library or vendored
Interchange code touched.

Delete packages/webhook-triggers (own table, ingress, and management
routes) and packages/collections (webhook-triggers' only consumer),
their migrateHub step, and apps/web's webhook-secret-panel and
webhook-triggers-api -- a webhook is created through the existing
credentials section.
@TheGreatAxios
TheGreatAxios force-pushed the cl-8389-corbits-webhooks branch from 2bd8f40 to 99bd2a2 Compare September 18, 2026 01:48
@TheGreatAxios
TheGreatAxios merged commit c913a24 into main Sep 18, 2026
1 check passed
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.

1 participant