refactor: core:common -> domain:common

[jinukeu]

📮 관련 이슈

• Resolved: [Refactor] core:common -> domain:common #138

Summary by CodeRabbit

• Chores
  • 프로젝트의 의존성 구조를 projects.core.common에서 projects.domain.common으로 업데이트하였습니다.
  • settings.gradle.kts 파일에서 ":core:common"을 제거하고 ":domain:common"을 추가하여 모듈 구성을 변경하였습니다.

[coderabbitai]

Walkthrough

이번 업데이트는 projects.core.common을 projects.domain.common으로 변경하는 리팩토링 작업을 포함합니다. 여러 build.gradle.kts 파일에서 종속성 선언이 변경되었으며, 패키지 구조와 모듈 포함 설정도 수정되었습니다. 이러한 수정은 코드베이스의 모듈 구조를 재정비하여 유지보수를 용이하게 하고 일관성을 확보하려는 것입니다.

Changes

파일 경로	변경 요약
domain/common/src/main/java/com/suwiki/domain/common/RunCatchingIgnoreCancelled.kt	패키지 선언을 com.suwiki.core.common에서 com.suwiki.domain.common으로 변경
domain/lectureevaluation/editor/build.gradle.kts domain/lectureevaluation/my/build.gradle.kts domain/lectureevaluation/viewerreporter/build.gradle.kts domain/login/build.gradle.kts domain/notice/build.gradle.kts domain/openmajor/build.gradle.kts domain/signup/build.gradle.kts domain/timetable/build.gradle.kts domain/user/build.gradle.kts	implementation(projects.core.common)을 implementation(projects.domain.common)으로 변경
settings.gradle.kts	":core:common"의 포함을 제거하고 ":domain:common"의 포함을 추가

Assessment against linked issues

Objective	Addressed	Explanation
프로젝트 구조에서 core:common -> domain:common으로 수정 (#138)	✅

Poem

코드 속에서 춤추는 토끼는 말했네,
"패키지를 옮겨 더 깔끔하게!"
core는 이제 없고 domain이 왔다네,
종속성도 바꾸니 참 좋구나.
새로운 구조, 더욱 단단히,
유지보수도 이젠 문제없으리! 🐇

---

Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?

Share

• X
• Mastodon
• Reddit
• LinkedIn

Tips

Chat

There are 3 ways to chat with CodeRabbit:

• Review comments: Directly reply to a review comment made by CodeRabbit. Example:
  • I pushed a fix in commit <commit_id>.
  • Generate unit testing code for this file.
  • Open a follow-up GitHub issue for this discussion.
• Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
  • @coderabbitai generate unit testing code for this file.
  • @coderabbitai modularize this function.
• PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
  • @coderabbitai generate interesting stats about this repository and render them as a table.
  • @coderabbitai show all the console.log statements in this repository.
  • @coderabbitai read src/utils.ts and generate unit testing code.
  • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
  • @coderabbitai help me debug CodeRabbit configuration file.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (invoked as PR comments)

• @coderabbitai pause to pause the reviews on a PR.
• @coderabbitai resume to resume the paused reviews.
• @coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
• @coderabbitai full review to do a full review from scratch and review all the files again.
• @coderabbitai summary to regenerate the summary of the PR.
• @coderabbitai resolve resolve all the CodeRabbit review comments.
• @coderabbitai configuration to show the current CodeRabbit configuration for the repository.
• @coderabbitai help to get help.

Additionally, you can add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.

CodeRabbit Configration File (.coderabbit.yaml)

• You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
• Please see the configuration documentation for more information.
• If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

• Visit our Documentation for detailed information on how to use CodeRabbit.
• Join our Discord Community to get help, request features, and share feedback.
• Follow us on X/Twitter for updates and announcements.

[coderabbitai]

Actionable comments posted: 0

Review details

Configuration used: CodeRabbit UI
Review profile: CHILL

Commits

Files that changed from the base of the PR and between 46aff95 and 0972afd.

Files selected for processing (11)

• domain/common/src/main/java/com/suwiki/domain/common/RunCatchingIgnoreCancelled.kt (1 hunks)
• domain/lectureevaluation/editor/build.gradle.kts (1 hunks)
• domain/lectureevaluation/my/build.gradle.kts (1 hunks)
• domain/lectureevaluation/viewerreporter/build.gradle.kts (1 hunks)
• domain/login/build.gradle.kts (1 hunks)
• domain/notice/build.gradle.kts (1 hunks)
• domain/openmajor/build.gradle.kts (1 hunks)
• domain/signup/build.gradle.kts (1 hunks)
• domain/timetable/build.gradle.kts (1 hunks)
• domain/user/build.gradle.kts (1 hunks)
• settings.gradle.kts (2 hunks)

Files skipped from review due to trivial changes (1)

• domain/common/src/main/java/com/suwiki/domain/common/RunCatchingIgnoreCancelled.kt

Additional comments not posted (10)

domain/openmajor/build.gradle.kts (1)

8-8: 이 변경 사항은 PR의 목표에 부합합니다. 모듈 이름이 core:common에서 domain:common으로 변경되었습니다.

domain/signup/build.gradle.kts (1)

8-8: 이 변경 사항은 PR의 목표에 부합합니다. 모듈 이름이 core:common에서 domain:common으로 변경되었습니다.

domain/lectureevaluation/editor/build.gradle.kts (1)

9-9: 이 변경 사항은 PR의 목표에 부합합니다. 모듈 이름이 core:common에서 domain:common으로 변경되었습니다. 그러나 projects.core.model의 사용은 확인이 필요합니다.

Verification successful

---

projects.core.model 의존성은 여러 모듈에서 일관되게 사용되고 있으며, 이는 의도된 것으로 보입니다. 따라서 이 변경 사항은 문제가 없습니다.

• remote/common/build.gradle.kts
• local/user/build.gradle.kts
• local/common/build.gradle.kts
• local/openmajor/build.gradle.kts
• local/login/build.gradle.kts
• local/timetable/build.gradle.kts
• domain/user/build.gradle.kts
• domain/timetable/build.gradle.kts
• domain/notice/build.gradle.kts
• domain/login/build.gradle.kts
• domain/lectureevaluation/viewerreporter/build.gradle.kts
• domain/lectureevaluation/my/build.gradle.kts
• domain/lectureevaluation/editor/build.gradle.kts
• core/ui/build.gradle.kts
• core/security/build.gradle.kts
• core/designsystem/build.gradle.kts
• core/android/build.gradle.kts

Scripts executed

The following scripts were executed for the analysis:

---

Script:

#!/bin/bash
# Description: Verify the usage of `projects.core.model` in other modules.

# Test: Search for the dependency usage. Expect: Consistent usage or planned refactoring.
rg --type kotlin $'projects.core.model'

Length of output: 1264

domain/lectureevaluation/my/build.gradle.kts (1)

9-9: 이 변경 사항은 PR의 목표에 부합합니다. 모듈 이름이 core:common에서 domain:common으로 변경되었습니다. 그러나 projects.core.model의 사용은 확인이 필요합니다.

domain/lectureevaluation/viewerreporter/build.gradle.kts (1)

9-9: 이 변경사항은 프로젝트의 모듈 구조를 개선하고, 관심사의 분리를 명확히 하는 데 도움이 됩니다. 모든 참조가 새로운 모듈 이름을 반영하도록 확인하세요.

domain/login/build.gradle.kts (1)

9-9: 이 변경은 모듈 이름을 core:common에서 domain:common으로 올바르게 업데이트합니다. 모듈 간의 종속성이 올바르게 유지되고 있는지 확인하세요.

domain/notice/build.gradle.kts (1)

9-9: 코드 구조와 조직을 개선하기 위한 모듈 이름 변경이 반영되었습니다. 프로젝트 전체에서 일관성을 유지하는지 확인해야 합니다.

domain/timetable/build.gradle.kts (1)

9-9: 모듈 이름이 domain:common으로 변경되어 코드의 구조가 개선되었습니다. 이 변경이 다른 모듈과의 종속성에 영향을 미치지 않도록 주의하세요.

domain/user/build.gradle.kts (1)

9-9: 코드 변경 사항이 목표와 일치합니다. domain:common 모듈이 제대로 설정되었는지 확인하십시오.

Verification successful

---

코드 변경 사항이 목표와 일치하며 domain:common 모듈이 제대로 설정되고 여러 파일에서 참조되고 있음을 확인했습니다.

• domain/signup/build.gradle.kts
• domain/user/build.gradle.kts
• domain/timetable/build.gradle.kts
• domain/openmajor/build.gradle.kts
• domain/notice/build.gradle.kts
• domain/login/build.gradle.kts
• domain/lectureevaluation/my/build.gradle.kts
• domain/lectureevaluation/viewerreporter/build.gradle.kts
• domain/lectureevaluation/editor/build.gradle.kts

Scripts executed

The following scripts were executed for the analysis:

---

Script:

#!/bin/bash
# Description: Verify that the new `domain:common` module is properly set up.

# Test: Search for the `build.gradle.kts` file in the `domain:common` directory. Expect: It should exist and contain necessary configurations.
fd 'build.gradle.kts' --exec cat {}

Length of output: 25590

---

---

Script:

#!/bin/bash
# Description: Search for references to `domain:common` across the repository to ensure it is correctly referenced.

rg 'projects.domain.common'

Length of output: 736

settings.gradle.kts (1)

77-77: 새로운 모듈 :domain:common의 포함이 PR 목표와 일치합니다. 이전 :core:common 모듈에 대한 모든 참조가 제거되었는지 확인하십시오.

Verification successful

---

새로운 모듈 :domain:common의 포함이 PR 목표와 일치합니다. 이전 :core:common 모듈에 대한 모든 참조가 제거되었습니다.

• settings.gradle.kts 파일에서 :domain:common 모듈 포함이 추가됨.
• 코드베이스에서 :core:common에 대한 참조가 모두 제거됨.

Scripts executed

The following scripts were executed for the analysis:

---

Script:

#!/bin/bash
# Description: Verify that all references to the old `:core:common` module are removed.

# Test: Search for any remaining references to `:core:common`. Expect: No occurrences should be found.
rg ':core:common'

Length of output: 17

---

---

Script:

#!/bin/bash
# Description: Verify that all references to the old `:core:common` module are removed.

# Test: Search for any remaining references to `:core:common` in all files. Expect: No occurrences should be found.
rg ':core:common' --hidden --glob '!.git/**'

Length of output: 44