Skip to content

chore: remove dead electric-proxy stub and stale streams docs#1574

Merged
saddlepaddle merged 1 commit into
mainfrom
cleanup/electric-managed
Feb 18, 2026
Merged

chore: remove dead electric-proxy stub and stale streams docs#1574
saddlepaddle merged 1 commit into
mainfrom
cleanup/electric-managed

Conversation

@saddlepaddle
Copy link
Copy Markdown
Collaborator

@saddlepaddle saddlepaddle commented Feb 18, 2026

Summary

  • Remove apps/electric-proxy/ — empty stub package that never had source code
  • Remove 4 outdated docs that reference the old self-hosted apps/streams server (now replaced by Electric Cloud managed service)
  • Clean up .env.example — remove STREAMS_PORT, STREAMS_INTERNAL_*, STREAMS_DATA_DIR vars for the defunct self-hosted server
  • Update AGENTS.md structure list to remove apps/streams and apps/electric-proxy

Test plan

  • bun run lint:fix — pass
  • bun run typecheck — pass
  • bun test — pass (1230 pass, 0 fail)
  • bunx sherif --fix — pass

Summary by CodeRabbit

  • Chores

    • Removed Streams service and Electric Proxy components from project configuration, package manifests, and environment variables.
    • Cleaned up related service declarations and configuration entries.
  • Documentation

    • Removed comprehensive documentation files covering architectural planning, production deployment strategies, streaming performance and reliability recommendations, and AI chat system implementation guidance.

- Remove `apps/electric-proxy/` (empty stub, never had code)
- Remove outdated docs: ai-chat-plan, productionize-chat,
  realign-streams-architecture, streaming-performance-reliability
- Remove old self-hosted streams server env vars from .env.example
- Update AGENTS.md to remove apps/streams and apps/electric-proxy
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Feb 18, 2026

No actionable comments were generated in the recent review. 🎉


📝 Walkthrough

Walkthrough

This PR removes the Streams service (apps/streams), electric-proxy package, Streams-related environment variables from .env.example, associated documentation files, and updates AGENTS.md to reflect these removals. Total: 12 environment variables removed, 2 AGENTS.md entries deleted, 1 package.json file deleted, and 4 documentation files deleted (2,263 lines total).

Changes

Cohort / File(s) Summary
Environment Configuration
.env.example
Removed five Streams-related environment variables: NEXT_PUBLIC_STREAMS_URL, STREAMS_PORT, STREAMS_INTERNAL_PORT, STREAMS_INTERNAL_URL, and STREAMS_DATA_DIR from the Freestyle section.
Project Manifest
AGENTS.md, apps/electric-proxy/package.json
Removed entries for apps/streams (Streams service) and apps/electric-proxy (Electric proxy service) from AGENTS.md; deleted the entire electric-proxy package.json manifest file.
Documentation
docs/ai-chat-plan.md, docs/productionize-chat.md, docs/realign-streams-architecture.md, docs/streaming-performance-reliability-recommendations.md
Deleted four comprehensive documentation files totaling 2,263 lines, including architecture plans for AI chat, production rollout guides, Streams architecture alignment strategy, and performance/reliability recommendations.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Poem

🐰 Farewell, dear streams and proxy true,
Your docs and configs bid adieu,
We've tidied up the garden well,
With cleaner paths to weave and tell.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main change: removing dead code and stale documentation related to electric-proxy and streams.
Description check ✅ Passed The description provides a clear summary of all changes, includes testing results, but is missing the standard template sections like Related Issues and Type of Change.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ 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 cleanup/electric-managed

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.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 18, 2026

🧹 Preview Cleanup Complete

The following preview resources have been cleaned up:

  • ✅ Neon database branch
  • ✅ Electric Fly.io app

Thank you for your contribution! 🎉

@saddlepaddle saddlepaddle merged commit 0047cae into main Feb 18, 2026
14 checks passed
@Kitenite Kitenite deleted the cleanup/electric-managed branch March 15, 2026 16:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant