Move bot integrations to vc-bots Cloudflare Worker - #1518
Merged
Conversation
Removes the Slack/Zoom webhook handlers, scheduled event reminders, and the build-rooms prebuild step brought in by #1501, along with their config, env vars, docs, and now-unused dependencies. The join-coffee and join-slack redirect functions stay.
Proxies /bots/* to the worker that replaced the removed webhook functions, stripping the /bots prefix via :splat.
👷 Deploy Preview for virtual-coffee-io processing.
|
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
1 participant
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.
Linked Issue
No linked issue — follow-up to #1501.
Description
This PR moves the Slack/Zoom bot integrations off Netlify Functions in preparation for the new
vc-botsCloudflare Worker.event-reminders-daily/-hourly/-weekly), along with their shared modules innetlify/functions/_shared/.scripts/build-rooms.ts, the generateddata/rooms.jsonplumbing (data/dir +included_filesinnetlify.toml), and thebuild-rooms/pretypecheckpackage scripts.@slack/bolt,slackify-html,@netlify/functions,@types/slackify-html,dotenv.netlify.toml, webhook env vars innetlify/env.d.tsand.env.example, and the README "Webhooks" section./bots/*rewrite that proxies to the new worker (https://vc-bots.lucky-art-2f02.workers.dev/:splat, prefix stripped), so the bot gets stable URLs on the virtualcoffee.io domain.The site's own redirect functions —
join-coffeeandjoin-slack— are untouched, along with their_shared/env.tshelper and env vars.Deploy notes:
/bots/...(or the worker directly) around deploy time.SLACK_BOT_TOKEN,SLACK_SIGNING_SECRET,SLACK_ANNOUNCEMENTS_CHANNEL,SLACK_EVENT_ADMIN_CHANNEL,ZOOM_WEBHOOK_SECRET_TOKEN,ZOOM_WEBHOOK_AUTH,AIRTABLE_COWORKING_BASE) can be removed in the Netlify UI.Verified locally:
pnpm typecheck,pnpm lint, andpnpm buildall pass, and vianetlify devthe/bots/*rewrite proxies to the worker (/bots/health→ worker/health→ok) while existing routes still serve.Methodology
Consolidating the webhooks repo into this site (#1501) was a stepping stone; the bot functionality is now moving to a dedicated Cloudflare Worker (
vc-bots) where it can evolve independently of the website deploys. This PR clears the Netlify side and wires up the/bots/*proxy so external integrations can use stable site-domain URLs instead of the rawworkers.devhost. The rewrite usesforce = truebecause the Next.js runtime's catch-all handler can otherwise shadow proxy rewrites.Code of Conduct