Skip to content

Conversation

@jmjava
Copy link
Contributor

@jmjava jmjava commented Jan 17, 2026

Summary

This PR consolidates startup improvements and removes obsolete documentation.

Changes

Dockerfile:

  • Add COPY codegen-gradle and chmod +x gradlew to fix multi-stage build

compose.yaml:

  • Make Neo4j ports configurable via NEO4J_HTTP_PORT, NEO4J_BOLT_PORT, NEO4J_HTTPS_PORT

README.md:

  • Add Docker Build Details section explaining multi-stage build
  • Add Neo4j port environment variables to documentation
  • Fix all docker compose commands to include --profile java where needed
  • Ensure docker compose down also uses --profile java

Deleted:

  • CURSOR-PR.md - content was migrated to README.md

Testing

All documented commands verified working:

  • docker compose --profile java up --build -d
  • docker compose up neo4j -d (Neo4j only)
  • COMPOSE_PROFILES= docker compose up -d
  • GUIDE_PORT=1338 docker compose --profile java up --build -d
  • ✅ SSE endpoint responds with text/event-stream
  • docker compose --profile java down --remove-orphans

Closes #24
Incorporates changes from PR #25

@jmjava
Copy link
Contributor Author

jmjava commented Jan 17, 2026

tests pass -- something with report generation?

@jmjava jmjava force-pushed the issue-24 branch 4 times, most recently from 83b6cc1 to ca39347 Compare January 17, 2026 18:52
@jmjava
Copy link
Contributor Author

jmjava commented Jan 18, 2026

@jasperblues i can't resolve the conflict but it looks like simple format of values in the README - either one will work. Maybe one with quotes follows the pattern for other values though. According to GitHub Copilot Chat Assistant the changes in the other files are already in main.

I could also just start over it might be cleaner LMK

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.

Remove CURSOR-PR.md documentation file

1 participant