Skip to content

Conversation

@jmjava
Copy link
Contributor

@jmjava jmjava commented Dec 19, 2025

Files changed:

  • .dockerignore: Include source files for multi-stage Docker build
  • .gitignore: Ignore *.pid files
  • CURSOR-PR.md: Detailed PR documentation (see this file for full change descriptions)
  • Dockerfile: Multi-stage build to compile from source
  • README.md: Cursor integration docs, Docker startup improvements, testing prerequisites
  • compose.yaml: Parameterized port, optional services via profiles
  • images/cursor-mcp-installed-servers.svg: Documentation screenshot
  • src/main/kotlin/.../SecurityConfig.kt: Bypass Spring Security for /sse and /mcp endpoints
  • src/test/kotlin/.../Neo4jTestContainer.kt: Use local Neo4j for tests
  • src/test/kotlin/.../McpSecurityTest.kt: Regression tests for MCP endpoint accessibility

See CURSOR-PR.md for detailed file-by-file analysis and rationale.

Files changed:
- .dockerignore: Include source files for multi-stage Docker build
- .gitignore: Ignore *.pid files
- CURSOR-PR.md: Detailed PR documentation (see this file for full change descriptions)
- Dockerfile: Multi-stage build to compile from source
- README.md: Cursor integration docs, Docker startup improvements, testing prerequisites
- compose.yaml: Parameterized port, optional services via profiles
- images/cursor-mcp-installed-servers.svg: Documentation screenshot
- src/main/kotlin/.../SecurityConfig.kt: Bypass Spring Security for /sse and /mcp endpoints
- src/test/kotlin/.../McpSecurityTest.kt: Regression tests for MCP endpoint accessibility

See CURSOR-PR.md for detailed file-by-file analysis and rationale.
- USE_LOCAL_NEO4J is now an environment variable (not a code constant)
- Developers can run: USE_LOCAL_NEO4J=true ./mvnw test
- No source code edits required for local testing
Documents Option 2 for fixing Docker API version issues locally,
though Option 1 (USE_LOCAL_NEO4J=true) is more reliable.
@jmjava
Copy link
Contributor Author

jmjava commented Dec 19, 2025

Sorry for the issues here:

I am trying to document changes i needed in CURSOR-PR.md - test containers always seem to be issue :) FWIW i am working on using telepresence PR based testing with Tekton against "live" clusters to get around issues similar to this in another project. https://telepresence.io/

@johnsonr johnsonr merged commit 8faeab2 into embabel:main Dec 19, 2025
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.

2 participants