Feature/01 setup gitignore and dependencies#2
Conversation
- Add root-level .gitignore for Docker volumes and system files - Add core/.gitignore for Spring Boot Java backend - Add mcp-adapter/.gitignore for Node.js MCP adapter - Prevent tracking of build artifacts, logs, and environment files
- Spring Boot: 3.3.4 → 3.3.11 - LangChain4j: 0.34.0 → 0.36.2 - SpringDoc OpenAPI: 2.5.0 → 2.7.0 - MinIO: 8.5.7 → 8.5.12 - TestContainers BOM: 1.19.3 → 1.20.4 - Make gradlew executable for consistent build environment Verified compatibility through successful Docker build process.
There was a problem hiding this comment.
Pull Request Overview
This PR focuses on modernizing the project's development setup by updating core dependencies to their latest stable versions and establishing proper Git repository hygiene. The changes aim to improve security, performance, and maintainability while ensuring a clean development environment.
- Updated Spring Boot from 3.3.4 to 3.3.11 and other core dependencies to latest stable versions
- Added comprehensive .gitignore files for both Java (core) and Node.js (mcp-adapter) components
- Integrated new frameworks including LangChain4j for RAG functionality and QueryDSL for type-safe queries
Reviewed Changes
Copilot reviewed 3 out of 5 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| mcp-adapter/.gitignore | Added Node.js-specific ignore patterns for dependencies, logs, and environment files |
| core/build.gradle | Updated Spring Boot and dependencies, added LangChain4j, QueryDSL, and Flyway integrations |
| core/.gitignore | Added comprehensive Java/Gradle ignore patterns including IDE files and generated code |
| } | ||
|
|
||
| compileQuerydsl { | ||
| options.annotationProcessorPath = configurations.querydsl |
There was a problem hiding this comment.
The 'querydsl' configuration is referenced but not properly defined until line 115. This will cause a build failure. Move the querydsl configuration block before the compileQuerydsl task configuration.
| } | ||
|
|
||
| // QueryDSL Configuration | ||
| def querydslDir = "$buildDir/generated/querydsl" |
There was a problem hiding this comment.
Using 'buildDir' is deprecated in newer Gradle versions. Consider using 'layout.buildDirectory.dir("generated/querydsl").get().asFile' for Gradle 7+ compatibility.
| def querydslDir = "$buildDir/generated/querydsl" | |
| def querydslDir = layout.buildDirectory.dir("generated/querydsl").get().asFile |
Replace deprecated $buildDir with layout.buildDirectory.dir() for Gradle 8+ compatibility
Summary
Key Implementation Notes
Dependencies Updated