Skip to content
This repository was archived by the owner on May 19, 2026. It is now read-only.

Phase 1 scope: PHASE1.md, remove Docker/Redis/vmix/Alembic/companion - #17

Merged
kubo6472 merged 2 commits into
mainfrom
cursor/phase1-scope-consolidation-125f
May 14, 2026
Merged

Phase 1 scope: PHASE1.md, remove Docker/Redis/vmix/Alembic/companion#17
kubo6472 merged 2 commits into
mainfrom
cursor/phase1-scope-consolidation-125f

Conversation

@kubo6472

Copy link
Copy Markdown
Member

Purpose

Single consolidation PR aligned with Phase 1 as defined in the revised Cursor handoff. Replaces the scattered direction of closed GHCR/nginx PRs with one explicit scope document and a repo that matches it.

Source of truth

  • docs/PHASE1.md — full Phase 1 brief (out-of-scope list, stack, models/beats, API surface, UX, seed, local run).
  • AGENTS.md — short agent/runbook; defers to docs/PHASE1.md for product decisions.

Removed (explicitly out of scope for Phase 1)

  • Docker / docker-compose, Dockerfile.nginx, nginx/, backend Dockerfile, entrypoint.sh, .dockerignore
  • Alembic and migrations — schema remains create_all in database.py
  • Redis, WebSockets, vMix bridge, rundown engine, deps.py, routers/vmix.py, routers/rundown_ops.py, ws/
  • companion-module

Dependencies

  • backend/pyproject.toml: dropped redis, psycopg, alembic; uv.lock regenerated.

CI

  • CI workflow still named CI (Dependabot auto-merge). Job runs uv sync + pytest only; Docker Compose build removed.

Seed

  • When ADMIN_PASSWORD is unset in non-production, admin password defaults to duopus2025 (with email admin@example.com) per Phase 1 seed spec; production still requires ADMIN_PASSWORD.

Not in this PR

  • Full UI/data-model parity with every line of PHASE1.md (e.g. User username vs FastAPI-Users email) is documented as an implementation note in the brief; follow-up PRs can close remaining gaps.
Open in Web Open in Cursor 

…lembic, companion

- Add docs/PHASE1.md with revised Cursor handoff (scope, stack, API, UX, seed).
- Rewrite AGENTS.md to point at PHASE1.md and document uv + SQLite + no Docker.
- Remove docker-compose, nginx image, backend Dockerfile, Alembic, companion-module,
  and unused backend vmix/rundown_engine/ws/deps code paths.
- Trim pyproject.toml (drop redis, psycopg, alembic); refresh uv.lock.
- CI: backend tests only via uv; drop Docker Compose build.
- Seed: stable dev admin password duopus2025 when ADMIN_PASSWORD unset.
- README links to PHASE1 and AGENTS.

Co-authored-by: Jakub Doboš <kubo6472@users.noreply.github.com>
@coderabbitai

coderabbitai Bot commented May 14, 2026

Copy link
Copy Markdown

Warning

Rate limit exceeded

@kubo6472 has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 19 minutes and 47 seconds before requesting another review.

You’ve run out of usage credits. Purchase more in the billing tab.

⌛ 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.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 25d169b3-ab3f-4ad0-9ea7-902e0f2b23ef

📥 Commits

Reviewing files that changed from the base of the PR and between fa746c1 and 92ce9f1.

⛔ Files ignored due to path filters (1)
  • backend/uv.lock is excluded by !**/*.lock
📒 Files selected for processing (28)
  • .env.example
  • .github/workflows/ci.yml
  • AGENTS.md
  • Dockerfile.nginx
  • README.md
  • backend/.dockerignore
  • backend/Dockerfile
  • backend/alembic.ini
  • backend/alembic/env.py
  • backend/alembic/script.py.mako
  • backend/alembic/versions/20240511_000001_initial.py
  • backend/alembic/versions/20240512_000002_story_rundown_position_unique.py
  • backend/deps.py
  • backend/entrypoint.sh
  • backend/pyproject.toml
  • backend/routers/rundown_ops.py
  • backend/routers/vmix.py
  • backend/seed.py
  • backend/services/rundown_engine.py
  • backend/services/vmix_bridge.py
  • backend/ws/__init__.py
  • backend/ws/hub.py
  • companion-module/companion/manifest.json
  • companion-module/package.json
  • companion-module/src/index.js
  • docker-compose.yml
  • docs/PHASE1.md
  • nginx/nginx.conf
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch cursor/phase1-scope-consolidation-125f

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

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

@kubo6472
kubo6472 marked this pull request as ready for review May 14, 2026 07:56
@kubo6472
kubo6472 merged commit 437d3d8 into main May 14, 2026
3 checks passed
@kubo6472
kubo6472 deleted the cursor/phase1-scope-consolidation-125f branch May 14, 2026 07:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants