Skip to content

Chore/repo setup#1

Closed
TheRealDarkCoder wants to merge 4 commits into
developfrom
chore/repo-setup
Closed

Chore/repo setup#1
TheRealDarkCoder wants to merge 4 commits into
developfrom
chore/repo-setup

Conversation

@TheRealDarkCoder

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI review requested due to automatic review settings March 31, 2026 21:58

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Initial repository scaffolding for the Routis monorepo, setting up a Next.js frontend, FastAPI backend, local Docker Compose environment, and CI workflows to support ongoing development.

Changes:

  • Add baseline Next.js (apps/web) app with lint/build configuration and Docker image.
  • Add baseline FastAPI (apps/api) service with uv-managed dependencies, Docker image, and a /health endpoint.
  • Add Docker Compose + CI workflow + repo templates/docs to standardize local/CI setup.

Reviewed changes

Copilot reviewed 26 out of 36 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
README.md Adds top-level project description, stack, and dev commands.
package.json Introduces root-level script entry point(s).
docs/ARCHITECTURE.md Aligns architecture doc with current Python version and stack table.
docker-compose.yml Defines local dev stack (web, api, postgres, redis, backup stub).
apps/web/tsconfig.json Adds TypeScript configuration for the web app.
apps/web/public/window.svg Adds web static asset.
apps/web/public/vercel.svg Adds web static asset.
apps/web/public/next.svg Adds web static asset.
apps/web/public/globe.svg Adds web static asset.
apps/web/public/file.svg Adds web static asset.
apps/web/postcss.config.mjs Configures PostCSS with Tailwind plugin.
apps/web/package.json Defines web app dependencies and scripts.
apps/web/next.config.ts Adds Next.js config placeholder.
apps/web/eslint.config.mjs Adds ESLint flat config using Next presets/ignores.
apps/web/Dockerfile Adds multi-stage Docker build for Next.js app.
apps/web/CLAUDE.md Points to shared agent instructions file.
apps/web/app/page.tsx Adds a minimal “Coming Soon” landing page.
apps/web/app/layout.tsx Adds root layout, fonts, and metadata.
apps/web/app/globals.css Adds Tailwind import and base styles.
apps/web/app/favicon.ico Adds favicon asset.
apps/web/AGENTS.md Adds agent guidance for working with the Next.js version used.
apps/web/.gitignore Adds web-specific ignore rules.
apps/api/uv.lock Adds locked Python dependency set for the API service.
apps/api/tests/test_main.py Adds a placeholder test file.
apps/api/tests/init.py Marks tests directory as a package.
apps/api/README.md Adds API README placeholder file.
apps/api/pyproject.toml Defines API project metadata, dependencies, and Ruff config.
apps/api/main.py Adds FastAPI app with /health endpoint and uvicorn entrypoint.
apps/api/Dockerfile Adds Docker image for API using uv sync + uvicorn.
apps/api/.python-version Pins local Python version for the API.
.github/workflows/ci.yml Adds CI jobs for web (npm) and api (uv/ruff/pytest + Postgres).
.github/pull_request_template.md Adds PR template and checklist.
.github/ISSUE_TEMPLATE/feature.md Adds feature issue template.
.github/ISSUE_TEMPLATE/bug.md Adds bug issue template.
.env.example Adds documented environment variable template for local/prod.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread package.json
"name": "routis",
"private": true,
"scripts": {
"codegen": "cd apps/web && pnpm run codegen"

Copilot AI Mar 31, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The root codegen script uses pnpm, but the repo’s web app and CI are using npm (apps/web/package-lock.json, CI runs npm ci). Also, apps/web/package.json doesn't define a codegen script, so this command will fail. Consider either adding a codegen script in apps/web and invoking it via npm --prefix apps/web run codegen, or removing/renaming this root script until codegen is implemented.

Copilot uses AI. Check for mistakes.
Comment thread docker-compose.yml Outdated
@@ -0,0 +1,2 @@
def test_placeholder():
pass

Copilot AI Mar 31, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a placeholder test that will always pass and doesn’t validate any behavior. Since CI runs pytest, it would be better to add at least one minimal assertion (e.g., calling GET /health via TestClient and asserting the JSON/status) or remove the test until real coverage is added.

Suggested change
pass
assert True

Copilot uses AI. Check for mistakes.
Comment thread apps/api/pyproject.toml Outdated
Comment thread apps/web/app/layout.tsx Outdated
@CLAassistant

CLAassistant commented Mar 31, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@github-actions

Copy link
Copy Markdown


Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

@github-actions github-actions Bot locked and limited conversation to collaborators Mar 31, 2026
@TheRealDarkCoder TheRealDarkCoder deleted the chore/repo-setup branch March 31, 2026 22:12
@TheRealDarkCoder TheRealDarkCoder restored the chore/repo-setup branch March 31, 2026 22:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants