fix(ci): fix streams preview deploy Dockerfile path doubling

[Kitenite]

Summary

• Replace superfly/fly-pr-review-apps action with direct flyctl commands for streams preview deployment — the action resolved the Dockerfile path relative to the config file directory, doubling it (apps/streams/apps/streams/Dockerfile)
• Fix fly.toml dockerfile path to be relative to its own directory
• Add missing workspace deps (durable-session, ui, shared) to Dockerfile for the new session-based auth
• Drop --frozen-lockfile since Docker builds use a workspace subset

Test plan

• Streams preview deployment succeeds on Fly.io

[coderabbitai]

Note

Reviews paused

It looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the reviews.auto_review.auto_pause_after_reviewed_commits setting.

Use the following commands to manage reviews:

• @coderabbitai resume to resume automatic reviews.
• @coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:

• ▶️ Resume reviews
• 🔍 Trigger review

📝 Walkthrough

Walkthrough

Replaces the preview workflow's single action with explicit setup and flyctl CLI steps (app create, secrets staging, deploy), updates production deploy to include repo-root context (flyctl deploy .), and adjusts Fly config and Dockerfile to use repo-root build context and include durable-session in build/runtime.

Changes

Cohort / File(s)	Summary
Preview workflow /.github/workflows/deploy-preview.yml	Replace fly-pr-review-apps action with setup-flyctl@master, then run `flyctl apps create ...
Production workflow /.github/workflows/deploy-production.yml	Change deploy invocation from flyctl deploy --config apps/streams/fly.toml --remote-only to flyctl deploy . --config apps/streams/fly.toml --remote-only (adds explicit repo-root build context).
Fly config apps/streams/fly.toml	Update dockerfile path from apps/streams/Dockerfile to Dockerfile; add comments clarifying Dockerfile is relative to fly.toml and flyctl deploy . sets repo-root build context for workspace COPY usage.
Dockerfile / build & runtime packages apps/streams/Dockerfile	Expand build context copies and production installs to include workspace packages (agent, auth, durable-session, email, local-db, mcp, scripts, shared, trpc, ui) alongside db and streams; copy durable-session into builder and runtime images and adjust copy semantics/comments.

Sequence Diagram(s)

sequenceDiagram
    participant Runner as "GitHub Actions Runner"
    participant Flyctl as "flyctl (CLI)"
    participant Flyio as "Fly.io API"

    Runner->>Runner: checkout repo
    Runner->>Flyctl: setup-flyctl action installs flyctl
    Runner->>Flyctl: flyctl apps create <app> || true
    Flyctl->>Flyio: Create app request (tolerant)
    Runner->>Flyctl: flyctl secrets set ... --stage <stage>
    Flyctl->>Flyio: Stage secrets
    Runner->>Flyctl: flyctl deploy . --config apps/streams/fly.toml --app <app> --region <rgn> --remote-only
    Flyctl->>Flyio: Deploy request (with repo-root context)
    Flyio-->>Flyctl: Deployment status
    Flyctl-->>Runner: Exit code / result

Loading

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~45 minutes

Possibly related PRs

• feat(streams): add bearer token auth and CI/CD deployment pipelines #1343 — Overlaps with Fly.io deployment workflow changes and preview deployment behavior.
• feat: add streams server (Hono + durable-streams) #1203 — Related edits to apps/streams/fly.toml and apps/streams/Dockerfile affecting build context and copied packages.

Poem

🐇 I hopped into the CI with a tiny drum,
Replaced one click with commands that hum,
Secrets snugged, context set to root,
Durable sessions tucked in the boot,
The app sprang up — a carrot-cloud strum 🥕

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately reflects the main fix: replacing the GitHub Action with direct flyctl commands to resolve the Dockerfile path doubling issue in the streams preview deployment.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description check	✅ Passed	The pull request description covers the main changes, related context, and includes a test plan checklist, meeting the template requirements.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch kitenite/fix-streams-preview-deploy

---

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

[github-actions]

🚀 Preview Deployment

🔗 Preview Links

Service	Status	Link
Database (Neon)	✅	View Branch
Electric (Fly.io)	✅	View App
Streams (Fly.io)	✅	View App
API (Vercel)	✅	Open Preview
Web (Vercel)	✅	Open Preview
Marketing (Vercel)	✅	Open Preview
Admin (Vercel)	✅	Open Preview
Docs (Vercel)	✅	Open Preview

---

Preview updates automatically with new commits