Skip to content

refactor(data): rename ArtistRepositoryImpl.kt.kt to ArtistRepositoryImpl.kt - #44

Open
cuong-tran wants to merge 1 commit into
maxrave-dev:multiplatformfrom
cuong-tran:fix/artist-repository-file-name
Open

cuong-tran wants to merge 1 commit into
maxrave-dev:multiplatformfrom
cuong-tran:fix/artist-repository-file-name

Conversation

@cuong-tran

@cuong-tran cuong-tran commented Oct 19, 2026 •

Copy link
Copy Markdown
Contributor

data/src/commonMain/kotlin/com/maxrave/data/repository/ArtistRepositoryImpl.kt.kt carried a stray second .kt extension. Kotlin compiles it anyway, so nothing was broken, but tools that match files by name miss it. This renames it to ArtistRepositoryImpl.kt. The class inside is unchanged, and it was the only .kt.kt file in core.

Pure rename, no code change.

maxrave-dev/SimpMusic#2712 splits SimpMusic's CLAUDE.md into path-scoped topic files, and one of them (.claude/rules/data-storage.md) matches **/ArtistRepositoryImpl.kt. That pattern matches this file once this is merged and core is bumped there.

…Impl.kt

The file carried a stray second .kt extension. Kotlin compiled it anyway, but tools that match files by name, such as path-scoped agent rules, missed it. The class inside is unchanged, and this was the only .kt.kt file in core.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request. Check if the Files changed in this pull request are included in default exclusions.

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