Skip to content

Conversation

@ptkNktq
Copy link
Owner

@ptkNktq ptkNktq commented Jan 19, 2026

Summary by CodeRabbit

  • 改善

    • ライセンス表示を新しい表示コンポーネントに置き換え、見た目と操作感を向上しました。
    • 既存の静的ライセンスページを削除し、動的な表示に統合しました。
  • Tests

    • ライセンス表示のスクリーンショットテストを追加しました。
  • Chores

    • ビルド設定を更新し、ライブラリ管理プラグインと関連依存を導入しました。

✏️ Tip: You can customize this high-level summary in your review settings.

@ptkNktq ptkNktq self-assigned this Jan 19, 2026
@coderabbitai
Copy link

coderabbitai bot commented Jan 19, 2026

Walkthrough

about-libraries を導入し、WebView ベースのライセンス表示を LibrariesContainer ベースに置換。バージョンカタログ、モジュール依存、UI 実装、静的ライセンス資産、スクリーンショットテスト報告が併せて更新された。

Changes

Cohort / File(s) 変更内容
バージョンカタログ & プラグイン定義
AndroidApp/gradle/libs.versions.toml, AndroidApp/build.gradle.kts
libs.versions.toml に about-libraries バージョンとプラグイン/ライブラリエントリを追加。root build.gradle.kts で about-librariesapply false として宣言(com-jaredsburrows-license を削除)
モジュール依存・プラグイン
AndroidApp/app/build.gradle.kts, AndroidApp/ui/build.gradle.kts
alias(libs.plugins.about.libraries) をプラグインに追加(app モジュールで apply)、implementation(libs.about.libraries.core) を app に追加、ui に compose 用 libs.about.libraries.compose.* を追加
ライセンス表示 UI
AndroidApp/ui/src/main/kotlin/.../LicenseScreen.kt
WebView ベースの表示を削除し、produceLibraries() と AboutLibraries の LibrariesContainer を使用する LicenseContent を追加。カラー設定とカスタム仕切りを導入、snackbar/onBack を受け取る公開 composable を追加
静的ライセンス資産の削除
AndroidApp/ui/src/main/assets/open_source_licenses.html, AndroidApp/app/licenses.yml
長大な open_source_licenses.html を削除。app/licenses.yml から koin-androidgson のエントリを削除
スクリーンショットテスト追加・報告更新
AndroidApp/ui/src/screenshotTest/.../ContentScreenshotTest.kt, AndroidApp/ui/build/reports/screenshotTest/...
LicenseContent のプレビュー用スクリーンショットテストを追加。レポート HTML のテスト数・所要時間・Gradle バージョン表記を更新
IDE 設定変更
AndroidApp/.idea/kotlinc.xml
Kotlin JPS プラグインのバージョン設定を 2.2.21 から 2.2.0 に変更

Sequence Diagram(s)

sequenceDiagram
    participant Screen as LicenseScreen
    participant Producer as produceLibraries()
    participant AboutLib as LibrariesContainer
    participant UI as AppScaffold/Snackbar

    Screen->>Producer: 起動時にライブラリ情報生成要求
    Producer-->>Screen: Libs データを返す
    Screen->>AboutLib: LibrariesContainer(libraries=Libs)
    AboutLib->>UI: ユーザ操作(詳細表示 / ダイアログ)
    UI-->>AboutLib: ユーザ操作の結果(閉じる等)
    AboutLib-->>Screen: 戻る操作時に onBack コール
Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

  • Enhance/transition #295 — 同ファイル LicenseScreen.kt を修正し UI/ナビゲーション周りに変更がある(コードレベルで関連)。
  • Update/env #308.idea/kotlinc.xml の Kotlin JPS プラグインバージョンを編集しており衝突や差異の可能性あり。
  • ライブラリ更新 #251 — Kotlin JPS プラグイン設定の別差分で同一箇所に関係。

Poem

🐰 今日はライセンスを並べる日
古い HTML はそっと片付けて
LibrariesContainer で光を当てるよ
依存もバージョンも揃えてぴょん!
ありがとう、オープンソース 🌱

🚥 Pre-merge checks | ✅ 1 | ❌ 2
❌ Failed checks (1 warning, 1 inconclusive)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Title check ❓ Inconclusive プルリクエストのタイトル「Fix/license」は、変更内容の主要な目的を曖昧に表現しており、具体的な情報に欠ける。 タイトルを「Replace jaredsburrows license plugin with aboutlibraries」のようにより具体的で説明的なものに変更し、主要な変更内容を明確に反映させてください。
✅ Passed checks (1 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In `@AndroidApp/gradle/libs.versions.toml`:
- Line 36: Update the about-libraries dependency from the beta version to the
stable release by changing the toml entry about-libraries = "14.0.0-b01" to
about-libraries = "13.1.0" so the project uses the recommended stable 13.1.0
release instead of the beta.

com-google-code-gson = "2.13.2"
com-squareup-leakcanary = "2.14"
mockk = "1.14.7"
about-libraries = "14.0.0-b01"
Copy link

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟠 Major

🧩 Analysis chain

🌐 Web query:

aboutlibraries mikepenz latest stable version 14.0.0

💡 Result:

Short answer: No — 14.0.0 is not a stable release yet. The latest stable release of AboutLibraries is 13.1.0; 14.0.0 artifacts are published as pre-releases (14.0.0-a*/b*). [1][2][3]

Sources:

  • com.mikepenz artifact listing (shows aboutlibraries latest = 13.1.0). [1]
  • Maven / mvnrepository entry for aboutlibraries-core 13.1.0 (stable, Oct 13, 2025). [2]
  • Gradle Plugin Portal showing 14.0.0 pre-release versions (a01 / b01). [3]

References:
[1] (javadoc.io)
[2] (mvnrepository.com)
[3] (plugins.gradle.org)

Citations:


about-libraries を安定版にアップグレードしてください。

現在使用している 14.0.0-b01 はベータ版です。最新の安定版は 13.1.0 (2025年10月13日リリース)です。本番環境では安定版を使用してください。バージョンを 13.1.0 に更新することをお勧めします。

🤖 Prompt for AI Agents
In `@AndroidApp/gradle/libs.versions.toml` at line 36, Update the about-libraries
dependency from the beta version to the stable release by changing the toml
entry about-libraries = "14.0.0-b01" to about-libraries = "13.1.0" so the
project uses the recommended stable 13.1.0 release instead of the beta.

@ptkNktq ptkNktq merged commit aae6db4 into develop Jan 20, 2026
2 checks passed
@ptkNktq ptkNktq deleted the fix/license branch January 20, 2026 15:45
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