cleanup + document server envs

[Brendonovich]

Summary by CodeRabbit

• Removals

  • Removed the public waitlist API endpoint

• Chores

  • Simplified database configuration by removing alternate migration URL fallback
  • Streamlined environment setup by removing prompts for a migration URL
  • Switched internal storage endpoint configuration to use an internal S3 endpoint
  • Expanded and reorganized server environment configuration with additional documented options and defaults

• Documentation

  • Added an "Other Configuration" section linking to the server environment reference

[coderabbitai]

Warning

Rate limit exceeded

@Brendonovich has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 18 minutes and 2 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between ba8beea and ac7faf1.

📒 Files selected for processing (1)

• .github/workflows/ci.yml (2 hunks)

Walkthrough

Removes the waitlist POST API route, consolidates database URL usage to DATABASE_URL (dropping DATABASE_MIGRATION_URL), expands and documents the server environment schema with many new variables/groups, updates a comment in next.config, and removes related CLI prompts.

Changes

Cohort / File(s)	Summary
API Endpoint Removal apps/web/app/api/waitlist/route.ts	Deleted the POST route handler that validated emails and forwarded them to the Loops contacts API.
Database Configuration Simplification packages/database/drizzle.config.ts, scripts/env-cli.js	Removed fallback and prompt for DATABASE_MIGRATION_URL; configuration and CLI now rely solely on DATABASE_URL.
Server Environment Schema Expansion packages/env/server.ts	Reworked and expanded server environment schema: added groups (NextAuth, Database, S3, CloudFront, Google Auth, WorkOS, AI providers, Cap Cloud), many new vars, descriptions, defaults, and runtime mappings.
S3 Endpoint Adjustment apps/web/instrumentation.node.ts	Switched S3 client endpoint from CAP_AWS_ENDPOINT to S3_INTERNAL_ENDPOINT.
Documentation & Config Comment apps/web/content/docs/self-hosting.mdx, apps/web/next.config.mjs	Added "Other Configuration" docs section pointing to env/server.ts; updated an inline comment to reference NEXT_PUBLIC_DOCKER_BUILD (no behavior change).

Sequence Diagram(s)

(Skipped — changes remove an endpoint and adjust configuration/schema; no new runtime control flow to diagram.)

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~40 minutes

• Areas needing extra attention:
  • packages/env/server.ts — verify new variables, defaults, and runtimeEnv mappings for accuracy and unintended side effects.
  • packages/database/drizzle.config.ts & scripts/env-cli.js — ensure no remaining references to DATABASE_MIGRATION_URL and migration tooling still functions.
  • apps/web/app/api/waitlist/route.ts removal — confirm no frontend, docs, or external integrations expect this endpoint.

Possibly related PRs

• Setup IaC for staging environment #1059 — modifies the same environment/schema area and AWS-related env vars; likely overlaps with packages/env/server.ts.
• Button and api route for uploading logs #1221 — adds environment variables to the same server schema; related to the expanded environment schema here.

Poem

🐰 I hopped through envs and cleared the way,
One URL now guides the database bay.
A route gone quiet, configs grown wide,
New vars tucked in, with tidy pride.
🥕✨

Pre-merge checks and finishing touches

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main changes across the changeset: removing unused code (waitlist route, DATABASE_MIGRATION_URL) and documenting/extending server environment variables.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}