Repository navigation
Conversation
Contributor
|
No actionable comments were generated in the recent review.
 |
| Layer / File(s) | Summary |
|---|---|
エラー情報の収集と整形 packages/frontend/src/utility/error-info.ts, packages/frontend/src/utility/misskey-api.ts, packages/frontend/test/unit/api-error-info.test.ts, packages/frontend/test/unit/error-info.test.ts |
API失敗時にエンドポイントを含めて拒否するよう変更しました。formatErrorInfoは環境情報、コンテキスト、エラー詳細をJSON文字列に整形します。関連する出力内容をテストします。 |
診断情報のダイアログ表示とコピー packages/frontend/src/os.ts, packages/frontend/src/components/MkDialog.vue, packages/frontend/src/components/MkDialog.stories.impl.ts, packages/frontend/test/unit/dialog-error-info.test.ts |
alertとpromiseDialogからエラー情報をダイアログへ渡します。診断情報があるエラーダイアログにコピー操作を追加し、表示と操作をテストします。 |
通話操作のエラー伝達 packages/frontend/src/components/MkCallsCreateDialog.vue, packages/frontend/src/components/MkCallsRoomWindow.vue, packages/frontend/src/utility/calls-window.ts, packages/frontend/test/unit/calls-join-confirmation.test.ts |
通話ルームの作成や各種操作が失敗した場合に、捕捉したエラーをアラートへ渡します。関連するアラートの期待値を更新します。 |
Priority:
Estimated code review effort: 3 (Moderate) | ~20 minutes
Change: Feature
Sequence Diagram(s)
sequenceDiagram
participant misskeyApi
participant apiWithDialog
participant alert
participant MkDialog
misskeyApi->>apiWithDialog: エラーとendpointを含むreject
apiWithDialog->>alert: errorとendpointを渡す
alert->>MkDialog: 整形したerrorInfoを渡す
Suggested labels: packages/frontend
Merge Risk: 
3a895
-
This change adds a copy action for error diagnostics to the error dialog and passes captured errors from the call features into it. No concrete merge-blocking risk remains. Real-browser clipboard behavior was not tested, but that is normal verification uncertainty and not a defect.
Pre-merge checks: 
-
Check name Status Explanation Resolution Docstring Coverage Docstring coverage is 33.33% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 6 functions across 9 files. (3 skipped: 3… Write docstrings for the functions missing them to satisfy the coverage threshold. -
Check name Status Explanation Title check PRタイトルは「エラーダイアログから診断情報をコピーできるように」で、変更の主目的（共通エラーダイアログへの診断情報コピー機能追加）と一致しています。『enhance(client)』の形式も妥当で、変更内容を具体的に示しています。 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. Description check PRの説明はテンプレートの必須項目である「What」「Why」「Additional info」「Checklist」を含み、変更内容、目的、テスト結果、既知の制約、未実施項目を具体的に説明しています。
Full details: Docstring Coverage
-
Explanation
Docstring coverage is 33.33% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 6 functions across 9 files. (3 skipped: 3 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.
Contributor
🖥 Frontend Diagnostics Report(No significant changes) Requests by resource type
V8 heap snapshot statistics
📦 Bundle StatsChunk size diff (2 updated, 0 added, 0 removed)
Startup chunk size (0 updated, 0 added, 0 removed)
Startup chunks are the Vite entry for
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
共通の
os.alert({ type: 'error' })に「エラー情報をコピー」を追加します。コピーしてもダイアログは閉じません。errorを渡した場合はエラー名・メッセージ・スタック・APIのコード／ID／エンドポイント・サーバーエラー識別情報を含めます。日時・アプリバージョン・ブラウザ情報・表示文言も記録します。通話の参加・作成などのエラーにも接続し、apiWithDialogの内部エラー専用コピー処理を共通化しました。Why
「問題が発生しました」だけでは利用者から調査に必要な情報を受け取れず、開発者ツールの操作を依頼するのも難しいため。
Additional info (optional)
エラーオブジェクトは診断用フィールドのみ抽出します。コンソール全体・リクエスト本文・認証トークンなどの任意プロパティは収集しません。エラー本体を渡さない既存の呼び出し元では、表示文言と実行環境のみコピーします。
node scripts/check-shipping.mjs --base origin/develop（変更ソースのlint、SPDX、locale safety）、追加テスト3ファイルのESLint、git diff --check。vue-tsc --noEmitは既存の生成型・localeなどのエラーで失敗。同じ依存環境で変更前のorigin/developも検査し、パスと行番号を除いた診断内容が一致しました。calls-join-confirmation.test.tsには今回触れていない2箇所のprefer-arrow-callback違反があります。ErrorWithCopyのStorybook storyも追加しています。Enhance: エラーダイアログから診断情報をコピーできるようにモバイル幅の表示例です。PRの
MkDialog/MkModal/MkButton/MkToastと既定のMi Lightテーマを使用したコンポーネントプレビューで、サンプルのエラーを渡しています。利用者の障害を再現した画像ではありません。エラー表示時：
コピー後（ダイアログを維持し、完了通知を表示）：
Checklist