Skip to content

Wearable APIが利用できないデバイスでのエラーを修正#5360

Merged
TinyKitten merged 1 commit intodevfrom
claude/fix-wearable-api-error-3RtI3
Feb 19, 2026
Merged

Wearable APIが利用できないデバイスでのエラーを修正#5360
TinyKitten merged 1 commit intodevfrom
claude/fix-wearable-api-error-3RtI3

Conversation

@TinyKitten
Copy link
Member

@TinyKitten TinyKitten commented Feb 19, 2026

Wearable APIが非対応のデバイスでpromise.rejectが発生し、
JS側で未処理のPromise拒否となっていた問題を修正。
API非対応時はresolveで正常終了するよう変更し、
JS側にもtry-catchを追加して防御的にエラーを処理。

https://claude.ai/code/session_01Lyo6X15Pbjj8ueR9uK8jtf

Summary by CodeRabbit

リリースノート

  • バグ修正
    • Wearableデバイス連携機能のエラーハンドリングを強化しました。API確認失敗や通信エラー時の対応を改善し、アプリケーションがより堅牢に動作するようになり、予期しないクラッシュが減少することで、より安定したユーザー体験を実現します。

Wearable APIが非対応のデバイスでpromise.rejectが発生し、
JS側で未処理のPromise拒否となっていた問題を修正。
API非対応時はresolveで正常終了するよう変更し、
JS側にもtry-catchを追加して防御的にエラーを処理。

https://claude.ai/code/session_01Lyo6X15Pbjj8ueR9uK8jtf
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 19, 2026

No actionable comments were generated in the recent review. 🎉


📝 Walkthrough

Walkthrough

WearableModule.ktおよびuseAndroidWearable.tsxのエラーハンドリングを改善しました。ネイティブウェアラブルス呼び出し失敗時、プロミス拒否からnull解決への動作変更と、TypeScript側にtry/catchラッパーを追加しました。

Changes

Cohort / File(s) Summary
Android Wearable Module Error Handling
android/app/src/main/java/me/tinykitten/trainlcd/WearableModule.kt
例外ハンドラの動作を変更。API可用性チェック失敗時に、エラーログ&プロミス拒否から警告ログ&null解決に変更。制御フローを非致命的パスへ移行。
TypeScript Hook Error Handling
src/hooks/useAndroidWearable.tsx
sendStationInfoToWatch呼び出しをtry/catchで包装。失敗時に警告ログを出力し、非同期関数の実行を継続。成功パスは変更なし。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • こまかいoptional chaining #4039: WearableModule.sendStationInfoToWatch呼び出しのガード処理をKotlin/TS両側で改善(オプショナルチェーンとより安全な例外ハンドリング)

Suggested labels

android

Poem

🐰 エラーログの優しき調べ
警告へと変わりて舞う
nullで返す、ウサギの知恵
止まらぬ流れ、グレースフルに 🌙

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

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.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed タイトルはPRの主要な変更内容を正確に反映しており、Wearable API非対応デバイスでのエラー修正という核心的な改善を明確に示しています。

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

✨ Finishing Touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch claude/fix-wearable-api-error-3RtI3

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

@TinyKitten TinyKitten merged commit 0ba235f into dev Feb 19, 2026
7 checks passed
@TinyKitten TinyKitten deleted the claude/fix-wearable-api-error-3RtI3 branch February 19, 2026 23:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants