Consolidate webhooks repo into virtualcoffee.io

[JoeKarow]

Linked Issue

No tracking issue; this lands the consolidation discussed alongside webhooks PR Virtual-Coffee/webhooks#28.

Description

This PR folds the standalone Virtual-Coffee/webhooks Netlify site into virtualcoffee.io so we maintain one repo, one deploy, and one set of secrets.

What's moving in

• Three Slack/Zoom webhook endpoints, ported from webhooks/typescript-migration:
  • /zoom-meeting-webhook-handler → netlify/functions/zoom-meeting-webhook-handler/{index,airtable,slack}.ts
  • /slack-events and /slack-interactivity → netlify/functions/slack/{index,messages}.ts (a single function backs both routes)
• Three scheduled functions with inline cron config (export const config: Config = { schedule: '…' }):
  • event-reminders-daily (0 12 * * *)
  • event-reminders-hourly (50 * * * *)
  • event-reminders-weekly (0 12 * * 1)
• Shared utilities and types under netlify/functions/_shared/{env,slack,verify,types/*}.ts (the _shared/ prefix prevents Netlify from treating those folders as deployable functions).
• A scripts/build-rooms.ts prebuild step that pulls co-working room records from Airtable into data/rooms.json. Wired into prebuild and pretypecheck. Writes an empty [] (with a warning) if AIRTABLE_COWORKING_BASE is not set, so contributors without webhook secrets can still build/typecheck locally.

What changed in-repo

• Migrated existing CommonJS functions netlify/functions/join-{coffee,slack}.js to TS Functions v2 with default-exported (req: Request) => Response. Behavior preserved; missing env vars now throw via a shared requireEnv helper instead of silently 302-ing to Location: undefined.
• netlify.toml: added [functions] included_files = ["data/*.json"] and three new redirects (/zoom-meeting-webhook-handler, /slack-events, /slack-interactivity). Existing /join-coffee and /join-slack redirects are unchanged — they target function names, so the JS→TS migration is transparent.
• package.json: added @netlify/functions, @slack/bolt, slackify-html, plus @types/slackify-html/dotenv dev deps. Wired pnpm build-rooms into prebuild and pretypecheck.
• netlify/env.d.ts: typed process.env for all webhook variables (CMS_*, SLACK_*, ZOOM_*, AIRTABLE_COWORKING_BASE, plus the existing ZOOM_TUESDAYS/THURSDAYS/SLACK_JOIN_LINK used by the migrated JS functions).
• .env.example and README.md: documented the new env vars and a "Webhooks" section summarizing each endpoint and scheduled function.

Out of scope / follow-ups

• External Slack and Zoom webhook URLs still point at the old webhooks host and will continue working through whatever DNS/redirect the maintainers set up at cutover. Updating Slack app + Zoom marketplace settings to hit virtualcoffee.io/... directly is a follow-up, not a blocker.
• Once integrations are migrated and stable, archive Virtual-Coffee/webhooks and stop the old Netlify site.

Methodology

Two repos for one project's webhooks meant duplicate dependencies, duplicate Netlify config, duplicate env-var management, and two places to reason about the Slack/Zoom contract surface. The webhooks repo's typescript-migration branch had finished modernizing it (TS, Functions v2, ESM, pnpm) into the same stack virtualcoffee.io already uses, so consolidation was a clean port rather than a rewrite.

Concrete choices worth flagging:

• Working-tree port, not a merge. I copied the post-migration source state of webhooks/ rather than preserving its git history, because the v1→v2 refactor on that branch contains files (functions/slack-{events,interactivity}/) that no longer exist and would need to be reverted post-merge. A clean port keeps this PR's diff readable.
• _shared/ directory naming. Netlify treats every direct subdirectory of netlify/functions/ as a deployable function. Prefixing with _shared/ is the documented opt-out and keeps shared code co-located with the functions that import it.
• requireEnv for join-coffee/join-slack env vars. Strictly stricter than the legacy JS — missing config now produces a 500 instead of a 302 to Location: undefined. Prod has these set; this only changes behavior in misconfigured environments, where loud failure is what we want.
• build-rooms.ts writes [] on missing creds. Lets pnpm install && pnpm build succeed for contributors without webhook secrets. The Zoom handler then matches no rooms, which is the correct behavior in that environment anyway.
• Cron declared inline on each function. Using export const config: Config = { schedule: '…' } per Netlify Functions v2 — no separate cron config needed. The stale /event-reminders → event-reminders-background redirect from the old netlify.toml was dropped (the function it referenced no longer exists; scheduled functions don't need an HTTP route).

Before merge: copy the webhook env vars from the old webhooks Netlify site to the virtualcoffee.io site (CMS_URL, CMS_TOKEN, SLACK_BOT_TOKEN, SLACK_SIGNING_SECRET, SLACK_ANNOUNCEMENTS_CHANNEL, SLACK_EVENT_ADMIN_CHANNEL, ZOOM_WEBHOOK_SECRET_TOKEN, ZOOM_WEBHOOK_AUTH, AIRTABLE_COWORKING_BASE).

Code of Conduct

By submitting this pull request, you agree to follow our Code of Conduct

[netlify]

👷 Deploy Preview for virtual-coffee-io processing.

Name	Link
🔨 Latest commit	f95fc2f
🔍 Latest deploy log	https://app.netlify.com/projects/virtual-coffee-io/deploys/6a0c76fee52e9d000806508e

[BekahHW]

@JoeKarow I just bumped a couple of packages to removed the next.js vulnerabilities. You'll need to update the package.json.

[meg-gutshall]

LGTM! I would merge it, but I saw the note about the env variables that need to be added in Netlify. I'll hold off to make sure all our ducks are in a row first.

[JoeKarow]

LGTM! I would merge it, but I saw the note about the env variables that need to be added in Netlify. I'll hold off to make sure all our ducks are in a row first.

env vars have been added to Netlify.