Skip to content

Conversation

@DogukanUrker
Copy link
Owner

@DogukanUrker DogukanUrker commented Feb 5, 2026

Summary

  • Add Dockerfile with single-stage build (uv + Python 3.14 on Alpine, non-root user, optimized layers)
  • Add .dockerignore with whitelist approach — only runtime files enter the build context
  • Add make docker, make docker-build, make docker-run targets
  • Commit uv.lock for reproducible builds (removed from .gitignore)
  • Fix Python 3.14 escape sequence warning in terminal_ascii.py
  • Update README.md and AGENTS.md with Docker and .env configuration docs

Test plan

  • make docker builds the image and starts the container
  • App responds at http://localhost:1283
  • Login with admin / admin works
  • Logs stream in real time (not buffered)
  • make docker-build followed by make docker-run works independently

Summary by CodeRabbit

  • New Features

    • Added Docker containerization support for simplified deployment with automated environment configuration and dependency management.
  • Documentation

    • Added Docker setup and usage guide to README with quick-start instructions.
    • Expanded configuration documentation with environment variable details.
    • Documented new Docker-related Make targets.

- add Dockerfile (single-stage, uv + python 3.14 alpine, non-root user)
- add .dockerignore with whitelist approach for minimal build context
- add docker make targets (docker, docker-build, docker-run)
- commit uv.lock for reproducible builds (removed from .gitignore)
- fix python 3.14 escape sequence warning in terminal_ascii.py
- update README.md and AGENTS.md with docker and .env docs
@coderabbitai
Copy link

coderabbitai bot commented Feb 5, 2026

📝 Walkthrough

Walkthrough

This PR introduces Docker containerization support by adding a Dockerfile and .dockerignore file for building optimized container images, establishes Make targets for building and running containers, removes uv.lock from version control, and updates documentation with Docker setup and configuration guidance. Minor refactoring updates terminal ASCII formatting.

Changes

Cohort / File(s) Summary
Docker Infrastructure
.dockerignore, Dockerfile, Makefile
Introduces Dockerfile with Python 3.14 Alpine base, non-root user setup, and uv-based dependency installation. Adds .dockerignore to optimize build context. Adds three new Make targets (docker, docker-build, docker-run) for containerized workflows.
Configuration & Version Control
.gitignore
Removes uv.lock from ignore list to allow tracking of lock file in version control.
Documentation
AGENTS.md, README.md
Expands developer guide with Docker section describing Dockerfile behavior, environment variable configuration, and port binding. Adds Docker usage instructions to README including quick-start command and step-by-step docker-build/run guidance.
Application Code
app/utils/terminal_ascii.py
Refactors ASCII art string construction to use concatenation with color escape sequences instead of a single literal block.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 A Docker burrow, snug and clean,
Alpine base with walls unseen,
Non-root and safe, my app shall dwell,
In containers built with care—all's well! 🐳

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description check ✅ Passed The description is comprehensive and well-structured, covering all major changes (Dockerfile, .dockerignore, Make targets, uv.lock commit, escape sequence fix, documentation updates) with a test plan, though it deviates from the template structure.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
Title check ✅ Passed The title 'feat: add docker support' clearly and concisely summarizes the primary change—adding Docker containerization support to the project, which is the main focus of all file modifications.

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

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/docker-support

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.

@DogukanUrker DogukanUrker changed the title add docker support feat: add docker support Feb 5, 2026
@DogukanUrker DogukanUrker self-assigned this Feb 5, 2026
@DogukanUrker DogukanUrker merged commit fe759cb into main Feb 5, 2026
3 checks passed
@DogukanUrker DogukanUrker deleted the feat/docker-support branch February 5, 2026 21:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant