Skip to content

feat: Implement common modules - #5

Merged
qowlgur121 merged 5 commits into
developfrom
feature/common-modules
Aug 10, 2025
Merged

feat: Implement common modules#5
qowlgur121 merged 5 commits into
developfrom
feature/common-modules

Conversation

@kAYI0019

@kAYI0019 kAYI0019 commented Aug 9, 2025

Copy link
Copy Markdown
Member

Summary

  • Implement standardized API response structures (CommonResponse, PageResponse)
  • Add core configuration classes (JPA Auditing, Async, Web)
  • Establish error handling system (ErrorCode, BusinessException, GlobalExceptionHandler)
  • Set up Gradle wrapper files for consistent development environment

Key Changes

API Response Structure

  • CommonResponse: Standard API response with success/error handling
  • PageResponse: Pagination response with Spring Data integration

Configuration & Error Handling

  • Config Classes: JpaAuditingConfig, AsyncConfig, WebConfig
  • Error System: Systematic error codes and global exception handler

Testing

  • All unit tests passing for response structures

- Add gradle-wrapper.jar for Gradle 8.5 execution
- Add gradlew.bat for Windows environment support
- Add CommonResponse: standard API response format
- Add PageResponse: pagination response DTO
- Add unit tests for CommonResponse and PageResponse
- add JpaAuditingConfig for automatic entity timestamping
- add AsyncConfig with dedicated thread pools for ingestion and general tasks
- add WebConfig to enable CORS for local development environment
- add IngestionStatus to define the document processing lifecycle
- add ErrorCode to standardize API error responses
- add BusinessException to represent custom business errors
- add GlobalExceptionHandler to catch all exceptions and return a standard CommonResponse
@kAYI0019 kAYI0019 self-assigned this Aug 9, 2025
@kAYI0019
kAYI0019 changed the base branch from main to develop August 9, 2025 21:51

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@qowlgur121
qowlgur121 merged commit e634faa into develop Aug 10, 2025
qowlgur121 added a commit that referenced this pull request Aug 10, 2025
- Remove redundant 'core' package nesting (com.opencontext.core.* → com.opencontext.*)
- Eliminate duplicate IngestionStatus enum, use team member's implementation
- Maintain full entity relationship functionality and test coverage
- Update all imports to reflect new package structure

This resolves package structure inconsistencies and enables seamless
integration with common modules established in PR #5.
@Yoo-SH Yoo-SH mentioned this pull request Aug 21, 2025
53 tasks
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