Skip to content

Agent friendliness - #685

Merged
fmell merged 11 commits into
mainfrom
agent_friendliness
May 4, 2026
Merged

Agent friendliness#685
fmell merged 11 commits into
mainfrom
agent_friendliness

Conversation

@fmell

@fmell fmell commented Apr 29, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@fmell
fmell force-pushed the agent_friendliness branch from b72ac62 to c92d925 Compare April 29, 2026 14:24
@fmell
fmell marked this pull request as ready for review April 30, 2026 09:44

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

This PR improves local/Codespaces developer experience by adjusting dev server/networking defaults, updating container and devcontainer setup, and refreshing documentation to match the new workflows and ports.

Changes:

  • Expose Vite dev server on all interfaces and wire Docker Compose to use .env files plus correct backend port 8001.
  • Update backend local container to Python 3.12 + install OpenJDK 21; adjust production Dockerfile Java installation and JAVA_HOME.
  • Improve developer docs and devcontainer bootstrap (copy .env templates, install backend pg extras, add VS Code debug extension).

Reviewed changes

Copilot reviewed 13 out of 14 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
frontend/vite.config.ts Bind dev server to all interfaces for container/Codespaces friendliness.
frontend/package.json Move/bump knip to devDependencies.
frontend/package-lock.json Lockfile updates reflecting knip bump and dependency graph changes.
frontend/Dockerfile.local Remove ad-hoc npm install --save @types/... step.
frontend/Dockerfile Remove ad-hoc npm install --save @types/... step.
frontend/.env.example Replace placeholder secrets with commented guidance + empty values.
docker-compose.yml Add env files, backend dependency, and move backend port mapping to 8001.
backend/src/acidwatch_api/app.py Allow CORS from Codespaces forwarded URLs via origin regex.
backend/Dockerfile.local Upgrade to Python 3.12-slim, install JDK 21, run on port 8001.
backend/Dockerfile Use headless JDK 21 and update JAVA_HOME.
README.md Add Docker Compose run instructions and update backend/frontend run commands/ports.
.devcontainer/install-dependencies.sh Copy env templates, install backend with pg extra, set Codespaces URLs.
.devcontainer/devcontainer.json Add ms-python.debugpy extension customization.
.devcontainer/DockerFile Update Node setup script to Node 22.x.
Files not reviewed (1)
  • frontend/package-lock.json: Language not supported

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

Comment thread backend/src/acidwatch_api/app.py
Comment thread backend/Dockerfile.local Outdated
Comment thread docker-compose.yml
Comment thread .devcontainer/install-dependencies.sh
@fmell
fmell force-pushed the agent_friendliness branch 2 times, most recently from ca98bb1 to c655f7e Compare April 30, 2026 12:41
@fmell
fmell force-pushed the agent_friendliness branch from c655f7e to f1f1b1c Compare April 30, 2026 12:55
@fmell
fmell merged commit 60c1690 into main May 4, 2026
7 checks passed
@fmell
fmell deleted the agent_friendliness branch May 4, 2026 07:32
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.

3 participants