Conversation
…d self-hosted deployment
- .env.example: Add trailing newline, use dynamic DATABASE_URL template, rename auth vars to match env.ts schema (BETTERBASE_JWT_SECRET, BETTERBASE_PUBLIC_URL, CORS_ORIGINS) - docker-compose.yml: Use env var interpolation for MinIO credentials, add minio-init service for automatic bucket creation - docker-compose.self-hosted.yml: Pin MinIO images to specific releases, rename auth env vars, add dashboard build step documentation - apps/dashboard/Dockerfile: Build specifically for dashboard with correct WORKDIR and COPY paths, add non-root nginx user, healthcheck endpoint, and /health location in nginx config - packages/server/Dockerfile: Pin wget to Alpine 3.22 version 1.25.0-r1 - docs/docker-setup.md: Clarify MinIO credentials for local-dev vs self-hosted environments - docs/getting-started/installation.md: Add text language specifier to project tree code block for MD040 compliance
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (5)
WalkthroughUpdated environment variables and examples (.env.example), added contribution docs, reworked README, added a Bun+nginx dashboard Dockerfile, and restructured Docker Compose files into separate local, self-hosted, and production manifests with new services (postgres, minio, minio-init, inngest, betterbase-server, dashboard) and adjusted ports/env names. Changes
Sequence Diagram(s)sequenceDiagram
rect rgba(0,128,255,0.5)
participant Dev as Developer
end
rect rgba(0,200,0,0.5)
participant Compose as docker-compose
end
rect rgba(255,165,0,0.5)
participant MinIO as MinIO
participant MinInit as minio-init
participant Postgres as Postgres
participant Inngest as Inngest
participant Server as betterbase-server
participant Dashboard as Dashboard (nginx)
end
Dev->>Compose: docker compose up (selected manifest)
Compose->>Postgres: start + wait healthcheck (pg_isready)
Compose->>MinIO: start + wait /minio/health/ready
Compose->>MinInit: run after MinIO healthy (mc alias & mb local/betterbase)
MinInit->>MinIO: create bucket (betterbase)
Compose->>Inngest: start (persist /data)
Compose->>Server: build/start when Postgres+MinIO+minio-init healthy
Server->>Postgres: connect via DATABASE_URL (POSTGRES_*)
Server->>MinIO: connect via STORAGE_ENDPOINT and keys
Dev->>Dashboard: build/serve (apps/dashboard Dockerfile) or Compose serves `dist`
Dashboard->>Server: API requests (VITE_API_URL / BETTERBASE_PUBLIC_URL)
Estimated code review effort🎯 4 (Complex) | ⏱️ ~45 minutes Possibly related PRs
Suggested labels
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
Summary by CodeRabbit
Documentation
Deployment & Infrastructure
Configuration