Skip to content

Remove Docker, run services natively#350

Draft
phoenixy1 wants to merge 1 commit intomasterfrom
ah-remove-docker
Draft

Remove Docker, run services natively#350
phoenixy1 wants to merge 1 commit intomasterfrom
ah-remove-docker

Conversation

@phoenixy1
Copy link
Copy Markdown
Collaborator

@phoenixy1 phoenixy1 commented Apr 18, 2026

Summary

  • Remove all Docker infrastructure (Dockerfiles, docker-compose, ngrok container, wait script)
  • Run Node.js server and Vite client directly on the host with npm run server / npm run client
  • PostgreSQL runs locally instead of in a container
  • Add root package.json with cross-platform npm scripts that work on macOS, Windows, and Linux
  • Add dotenv to server so it reads .env directly (previously injected by Docker Compose)
  • Update README with setup instructions for all three platforms

Test plan

  • cp .env.template .env and fill in Plaid keys
  • npm run install:all installs both client and server deps
  • npm run db:create initializes PostgreSQL tables
  • npm run server starts backend on port 5001
  • npm run client starts frontend on port 3001
  • App loads at http://localhost:3001 and can link a Plaid item
  • Verify on Windows and Linux in addition to macOS

🤖 Generated with Claude Code

Claude Session: 6c37be24-fabc-4e72-96dc-827ce4b7e635

Docker was the only way to run the app, requiring all contributors to have
Docker Desktop installed. This replaces the Docker orchestration with direct
Node.js + PostgreSQL, making the app simpler to set up and run on any OS.

- Remove Dockerfiles, docker-compose files, ngrok container, and wait script
- Add dotenv to server so it reads .env directly
- Default DB connection to localhost:5432 with postgres/password
- Point Vite proxy at localhost instead of Docker service hostname
- Add root package.json with cross-platform npm scripts
- Update .env.template with all required env vars and defaults
- Rewrite README with setup instructions for macOS, Windows, and Linux

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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