🔧 Compose ImageLoader に乗り換え

[tatsutakein]

Issue

• close [FR]: Coil から Compose ImageLoader に置き換える #65 🦕

概要

Compose ImageLoader に乗り換えます。

レビュー観点

特になし

レビューレベル

• Lv0: まったく見ないで Approve する
• Lv1: ぱっとみて違和感がないかチェックして Approve する
• Lv2: 仕様レベルまで理解して、仕様通りに動くかある程度検証して Approve する
• Lv3: 実際に環境で動作確認したうえで Approve する

レビュー優先度

• すぐに見てもらいたい ( hotfix など ) 🚀
• 今日中に見てもらいたい 🚗
• 今日〜明日中で見てもらいたい 🚶
• 数日以内で見てもらいたい 🐢

参考リンク

• https://github.com/qdsfdhvh/compose-imageloader

スクリーンショット

Before	After

Summary by CodeRabbit

• 新機能

  • アプリに画像ローダーの依存関係を追加しました。

• 改善点

  • Androidプラットフォーム固有の依存関係を削除し、Androidに関連する機能や挙動を最適化しました。

• バグ修正

  • プロフィール画像を表示する機能を更新し、Jetpack ComposeのImageコンポーネントを使用するように変更しました。

[coderabbitai]

Walkthrough

この変更は、Kotlinプロジェクトにおける依存関係の更新と、ProfileImage関数の実装の変更を含んでいます。特に、ProfileImage関数はexpectキーワードを取り除き、Jetpack ComposeのImageコンポーネントを使用するように更新されました。これは、マルチプラットフォーム対応していないCoilからCompose ImageLoaderへの置き換えを示しています。

Changes

ファイル	変更概要
core/designsystem/build.gradle.kts	libs.imageLoaderへの依存関係の追加
core/ui/build.gradle.kts	androidMainブロックからの依存関係の削除
.../core/ui/ProfileImage.kt	expectキーワードの削除とImageコンポーネントへの更新

Assessment against linked issues

Objective	Addressed	Explanation
#65: CoilからCompose ImageLoaderに置き換える	✅

Poem

🐰✨
コードの変更、新たな流れ、
画像読み込み、軽やかに変われ。
Composeの光、プロジェクトに映え、
バグのない朝を、うさぎは願え。

---

Tips

Chat with CodeRabbit Bot (@coderabbitai)

• If you reply to a review comment from CodeRabbit, the bot will automatically respond.
• To engage with CodeRabbit bot directly around the specific lines of code in the PR, mention @coderabbitai in your review comment
• Note: Review comments are made on code diffs or files, not on the PR overview.
• Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.

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 a review. This is useful when automatic reviews are disabled for the repository.
• @coderabbitai resolve resolve all the CodeRabbit review comments.
• @coderabbitai help to get help.

Note: For conversation with the bot, please use the review comments on code diffs or files.

CodeRabbit Configration File (.coderabbit.yaml)

• You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
• The JSON schema for the configuration file is available here.
• 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/coderabbit-overrides.v2.json

[coderabbitai]

Review Status

Actionable comments generated: 1

Configuration used: CodeRabbit UI

Commits

Files that changed from the base of the PR and between a82bf48 and 6f094ed.

Files ignored due to filter (1)

• gradle/libs.versions.toml

Files selected for processing (3)

• core/designsystem/build.gradle.kts (1 hunks)
• core/ui/build.gradle.kts (1 hunks)
• core/ui/src/commonMain/kotlin/club/nito/core/ui/ProfileImage.kt (1 hunks)

Files skipped from review due to trivial changes (1)

• core/designsystem/build.gradle.kts

Additional comments: 1

core/ui/build.gradle.kts (1)

• 31-35: プルリクエストの概要にはlibs.imageLoaderがcommonMainに追加されたとありますが、このhunkにはその変更が見られません。libs.imageLoaderの依存関係がcommonMainのdependenciesブロックに追加されていることを確認してください。

[coderabbitai]

コードレビューを行いましたが、ProfileImage 関数に関しては、特に問題は見つかりませんでした。ただし、画像の読み込みが失敗した場合のエラーハンドリングやプレースホルダー/フォールバック画像の設定が見受けられません。これは改善の余地があるかもしれません。また、contentDescription が profile.displayName に設定されていますが、これはアクセシビリティの観点から画像を説明するのに十分ではない可能性があります。最後に、新しい実装が期待通りに機能するかどうかのテストが行われているかどうかを確認することをお勧めします。