Repository navigation
Conversation
Core::db() を crate の中に閉じた (#1098 §7) とき、Android 限定の mod android が生の DB ハンドルを取っていてタグ時の Release (Android) だけが落ちた。デスクトップではコンパイルされず手元の check を抜けた。 Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
API surface diff外部アプリ向け API 面 ( Full diffdiff --git a/src-tauri/openapi.json b/src-tauri/openapi.json
index db2070d..ae7bcbb 100644
--- a/src-tauri/openapi.json
+++ b/src-tauri/openapi.json
@@ -6,7 +6,7 @@
"license": {
"name": "MIT"
},
- "version": "1.98.1"
+ "version": "1.98.2"
},
"paths": {
"/api": {
|
|
No actionable comments were generated in the recent review.
 |
| Layer / File(s) | Summary |
|---|---|
Android pushのアカウント読み込み src-tauri/src/commands/push.rs |
android::envがblockingクロージャ内でload_accounts()を実行します。読み込みエラーは引き続き伝播します。 |
バージョン値の更新 crates/notemaid/Cargo.toml, src-tauri/Cargo.toml, package.json, src-tauri/tauri.conf.json |
各ファイルのバージョンを1.98.2に更新しました。 |
Priority:
Estimated code review effort: 2 (Simple) | ~10 minutes
Change: Bug fix
Possibly related issues
- meta: アーキテクチャ監査 2026-09 — 設計は健全だが適用率 1/3 で強制ゼロ。再設計ではなく適用完了と lint 化 #1098: 非同期処理からの同期DBアクセスを共通経路へ移す課題に対し、pushアカウントのDB読み込みを
Core::blocking経由に変更しています。 - Android版の動作確認 #870: Android検証の課題に関連します。Android pushコードのリリースビルド上の問題を修正しますが、実機検証は追加していません。
- feat: Android版をGoogle Playで配布 #868: Androidリリースビルドの阻害要因を修正します。ただし、Google Play向けAAB生成やCI設定は変更していません。
Possibly related PRs
- notedeck-dev/notedeck#1280: 同じAndroid pushコマンド経路を追加したPRです。この変更は、その経路のアカウント読み込みを修正します。
- notedeck-dev/notedeck#1281: 同じAndroid pushコマンド経路とpush登録処理を変更したPRです。この変更は、その経路のアカウント読み込みを修正します。
Merge Risk: 
1d149
-
The Android push fix and release version updates are consistent, with no actionable merge-blocking risk identified.
Pre-merge checks: 
-
Check name Status Explanation Resolution Docstring Coverage Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 1 files. (4 skipped: 4 … Write docstrings for the functions missing them to satisfy the coverage threshold. -
Check name Status Explanation Title check タイトルは v1.98.2 のリリースであることを示し、バージョン更新と Android ビルド修正という主な変更に関連しています。 Description check 変更内容と修正理由は明確です。テンプレートの見出し、テスト結果、設計原則の判定は未記載ですが、リリース内容を理解するための主要情報は含まれています。 Linked Issues check Check skipped because no linked issues were found for this pull request. Out of Scope Changes check Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage
-
Explanation
Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 1 files. (4 skipped: 4 unsupported.)
Resolution
Write docstrings for the functions missing them to satisfy the coverage threshold.
-
Finishing Touches: 💡 1
-
📝 Generate docstrings: 💡
-
- Commit to this branch
- Create a new PR
🧪 Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
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.
Comment @coderabbitai help to get the list of available commands.
Deploying notedeck with
|
| Latest commit: |
1d14973
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://998c71f9.notedeck-d3a.pages.dev |
| Branch Preview URL: | https://develop.notedeck-d3a.pages.dev |
Code reviewNo issues found. Checked for bugs and CLAUDE.md compliance. |
変更一覧
v1.98.1 で Release (Android) だけが落ちた (Android 限定コードが notecore の private になった
Core::db()を呼んでいた) ための修正リリース。本体の v1.98.1 は公開済み。🤖 Generated with Claude Code