feat: setup project foundation with dependencies and build configuration - #3
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.
Replace deprecated $buildDir with layout.buildDirectory.dir() for Gradle 8+ compatibility
There was a problem hiding this comment.
Pull Request Overview
This PR establishes the foundational infrastructure for the project by updating dependencies to their latest stable versions and configuring proper build system support. The changes focus on modernizing the tech stack while ensuring future compatibility with Gradle 8+.
Key changes include:
- Dependency updates across Spring Boot, LangChain4j, SpringDoc, MinIO, and TestContainers
- QueryDSL integration with Gradle 8+ compatible configuration
- Comprehensive .gitignore setup for both core Java and mcp-adapter Node.js components
Reviewed Changes
Copilot reviewed 3 out of 5 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| core/build.gradle | Updated dependencies and added QueryDSL configuration with Gradle 8+ compatibility |
| core/.gitignore | Added comprehensive Java/Spring Boot project gitignore patterns |
| mcp-adapter/.gitignore | Added Node.js specific gitignore patterns for the adapter component |
| } | ||
|
|
||
| sourceSets { | ||
| main.java.srcDir querydslDir |
There was a problem hiding this comment.
[nitpick] Consider using a more descriptive variable name than 'querydslDir' to improve code readability, such as 'querydslGeneratedSourcesDir' or 'querydslOutputDirectory'.
| main.java.srcDir querydslDir | |
| def querydslGeneratedSourcesDir = layout.buildDirectory.dir("generated/querydsl").get().asFile | |
| querydsl { | |
| jpa = true | |
| querydslSourcesDir = querydslGeneratedSourcesDir | |
| } | |
| sourceSets { | |
| main.java.srcDir querydslGeneratedSourcesDir |
| extendsFrom annotationProcessor | ||
| } | ||
| querydsl.extendsFrom compileClasspath | ||
| } |
There was a problem hiding this comment.
[nitpick] The 'compileQuerydsl' task configuration references 'configurations.querydsl' which is defined later in the file at line 115. Consider moving the configurations block before this task configuration for better code organization.
| } | |
| configurations { | |
| compileOnly { | |
| extendsFrom annotationProcessor | |
| } | |
| querydsl.extendsFrom compileClasspath | |
| } | |
| compileQuerydsl { | |
| options.annotationProcessorPath = configurations.querydsl | |
| } |
Summary
Key Changes
Testing