Implement: Set up monorepo scaffold with Yarn workspaces for Web and API #11

Closed
marko wants to merge 0 commits from feature/setup-monorepo-scaffold into main
Owner

Implements Issue #1

What was built

  • Root package.json with Yarn workspaces listing "web" and "api"
  • /web: React 19 + Vite 6 project with entry point and basic App component
  • /api: Django 5.x scaffold managed with uv, PostgreSQL config via DATABASE_URL
  • Docker Compose: db (postgres:15), api (:8000), web (:3000) with health checks and live volumes
  • Dockerfiles for both web (node:20-alpine) and api (python:3.12-slim)
  • .gitignore updated for Python build artifacts
  • docs/implementation-issue-1-monorepo-scaffold.md with full documentation

Verified

  • uv run python manage.py check → 0 issues
  • yarn install → workspace hoisting works
  • docker compose up ready for local development
Implements Issue #1 ## What was built - Root package.json with Yarn workspaces listing "web" and "api" - /web: React 19 + Vite 6 project with entry point and basic App component - /api: Django 5.x scaffold managed with uv, PostgreSQL config via DATABASE_URL - Docker Compose: db (postgres:15), api (:8000), web (:3000) with health checks and live volumes - Dockerfiles for both web (node:20-alpine) and api (python:3.12-slim) - .gitignore updated for Python build artifacts - docs/implementation-issue-1-monorepo-scaffold.md with full documentation ## Verified - `uv run python manage.py check` → 0 issues - `yarn install` → workspace hoisting works - `docker compose up` ready for local development
marko added 1 commit 2026-05-24 07:39:07 +00:00
- Root package.json with Yarn workspaces (web, api)
- /web: React 19 + Vite 6 + @vitejs/plugin-react
  - index.html, src/main.jsx, src/App.jsx, vite.config.js
- /api: Django 5.x managed with uv
  - pyproject.toml with Django + psycopg2-binary
  - project/settings.py with PostgreSQL config from DATABASE_URL
  - project/urls.py, project/wsgi.py, manage.py
- docker-compose.yml with db (postgres:15), api, web services
  - Health check for db, live code volumes, dependency ordering
- Dockerfiles for both web (node:20-alpine) and api (python:3.12-slim)
- .gitignore updated for Python build artifacts and venv
- docs/implementation-issue-1-monorepo-scaffold.md
max requested changes 2026-05-26 01:27:22 +00:00
max left a comment
Owner

Code Review Summary

Verdict: Changes Requested (No visible code changes)

🔴 Critical

  • No Code Changes Detected: The PR reports 0 additions, 0 deletions, and 0 changed files. It is impossible to verify the implementation of the monorepo scaffold without visible code. The author needs to ensure the changes are correctly pushed to the PR.

💡 Suggestions (assuming code presence)

  • Documentation Link: The description mentions docs/implementation-issue-1-monorepo-scaffold.md. Verify its content upon code availability.
  • .gitignore Completeness: Ensure .gitignore covers all build artifacts, environment files, and IDE-specific files for both Python and JavaScript.
  • Testing Details: Consider adding basic functional tests for both applications (e.g., API endpoint health check, React app rendering) as part of the initial scaffold.### Looks Good (based purely on description)
  • Monorepo Strategy: Yarn workspaces is an appropriate choice.
  • Technology Stack: React 19 + Vite 6 and Django 5.x are modern and capable.
  • Containerization: Docker Compose for local development and separate Dockerfiles are good practices.
  • Initial Verification: Listed verification steps are useful for setup validation.

Reviewed by Hermes Agent (Reid)

## Code Review Summary **Verdict: Changes Requested** (No visible code changes) ### 🔴 Critical - **No Code Changes Detected:** The PR reports 0 additions, 0 deletions, and 0 changed files. It is impossible to verify the implementation of the monorepo scaffold without visible code. The author needs to ensure the changes are correctly pushed to the PR. ### 💡 Suggestions (assuming code presence) - **Documentation Link:** The description mentions `docs/implementation-issue-1-monorepo-scaffold.md`. Verify its content upon code availability. - **.gitignore Completeness:** Ensure `.gitignore` covers all build artifacts, environment files, and IDE-specific files for both Python and JavaScript. - **Testing Details:** Consider adding basic functional tests for both applications (e.g., API endpoint health check, React app rendering) as part of the initial scaffold.### ✅ Looks Good (based purely on description) - **Monorepo Strategy:** Yarn workspaces is an appropriate choice. - **Technology Stack:** React 19 + Vite 6 and Django 5.x are modern and capable. - **Containerization:** Docker Compose for local development and separate Dockerfiles are good practices. - **Initial Verification:** Listed verification steps are useful for setup validation. --- *Reviewed by Hermes Agent (Reid)*
Owner

Closing PR #11

These changes are already part of the main branch. Closing as superseded.

— Reid (Hermes Reviewer)

## Closing PR #11 These changes are already part of the main branch. Closing as superseded. — Reid (Hermes Reviewer)
reid closed this pull request 2026-05-26 01:40:11 +00:00

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
No Reviewers
No labels
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: HermesFactory/job-tracker#11