Skip to content

enhance(calls): 終了した通話の開始・終了日時と合計時間を表示 - #69

Merged
mattyatea merged 2 commits into
developfrom
codex/calls-ended-summary
Oct 5, 2026
Merged

mattyatea merged 2 commits into
developfrom
codex/calls-ended-summary

Conversation

@mattyatea

@mattyatea mattyatea commented Oct 5, 2026 •

Copy link
Copy Markdown

What

終了したCallsのルーム画面と共有リンクカードに、開始日時・終了日時・ルーム全体の通話時間を表示します。終了時にルーム画面を残し、後から同じリンクで開き直せます。

終了イベントを受けたら既存の calls/rooms/show から確定した時刻を取得します。古い応答で状態が戻らないようにし、日時の欠落・不正値は「不明」と表示します。24時間を超える通話にも対応します。

Why

元の要望に対応し、通話が何時から何時まで続いたかを終了後に確認できるようにします。

関連: #17。既存の #50 の個人の参加時間タイマーと #48 のルーム情報更新を参考にしています。参加中の個人タイマーは変更していません。

Additional info (optional)

DB・API・閲覧権限の変更はありません。履歴一覧や個人の参加時間の累積は今回の対象外です。

検証:

  • PASS: 最終コミット 6cb44f4 のGitHub CIは実行された25チェックすべて成功。frontend unit/E2E、全lint/型検査、本番ビルド、locale、SPDX、frontend diagnostics、Dockleを含みます。Storybookはdraft条件でスキップされました。
  • PASS: frontend unit 全395件(41ファイル)。初回から終了済みのルーム、通話中から終了への遷移、ホスト終了と通常退出、閉じて再表示、時刻欠落・不正・逆転・日跨ぎ・24時間超、既存の個人タイマーを含みます。
  • PASS: frontend本番ビルド(pnpm exec node --import tsx ./build.ts)。
  • PASS: node scripts/check-shipping.mjs --base 2bb19b9(変更ファイルlint、SPDX、locale safety)、git diff --check。
  • PASS: frontend typecheck。実装前から存在した calls-session.test.ts の beforeunload 型比較エラー2件は、テストのspy対象をDOMの EventTarget と明示する最小修正で解消しました。テストの検証内容と本番コードは変更していません。
  • BLOCKED: ブラウザーでの目視確認とスクリーンショット。クラウド環境でChromiumのIPC作成とlocalhostプレビューへのアクセスが制限されているため未実施です。表示例のStorybook storyは追加しています。
  • SKIPPED: 実際の音声通話を伴う複数端末E2E。

CHANGELOGはリポジトリ方針に従い未編集です。候補: Enhance: 終了したCallsに開始・終了日時とルーム全体の通話時間を表示

Checklist

  • Read the contribution guide
  • Test working in a local environment
  • (If needed) Add story of storybook
  • (If needed) Update CHANGELOG.md
  • (If possible) Add tests

@coderabbitai

coderabbitai Bot commented Oct 5, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 55 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Repository: Type4ny-Project/Misskey/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: b5030a23-c656-4197-a584-3efce76a4644
📥 Commits

Reviewing files that changed from the base of the PR and between 2bb19b9 and 6cb44f4.

📒 Files selected for processing (15)
  • locales/ja-JP.yml
  • packages/frontend/src/components/MkCallsRoomCard.stories.impl.ts
  • packages/frontend/src/components/MkCallsRoomCard.vue
  • packages/frontend/src/components/MkCallsRoomSummary.stories.impl.ts
  • packages/frontend/src/components/MkCallsRoomSummary.vue
  • packages/frontend/src/components/MkCallsRoomWindow.stories.impl.ts
  • packages/frontend/src/components/MkCallsRoomWindow.vue
  • packages/frontend/src/composables/use-calls-room.ts
  • packages/frontend/test/unit/calls-join-confirmation.test.ts
  • packages/frontend/test/unit/calls-room-summary.test.ts
  • packages/frontend/test/unit/calls-session.test.ts
  • packages/frontend/test/unit/calls-url-card.test.ts
  • packages/frontend/test/unit/calls-window.test.ts
  • packages/frontend/test/unit/use-calls-room.test.ts
  • packages/i18n/src/autogen/locale.ts
  • Autopilot · 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.

❤️ Share

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

@github-actions

github-actions Bot commented Oct 5, 2026 •

Copy link
Copy Markdown

🖥 Frontend Diagnostics Report

(No significant changes)

View details

Requests by resource type
Type Requests Encoded bytes
Base Head Δ Base Head Δ
Document 2 2 0 34 KB 34 KB 0 B
Script 151 151 0 2.4 MB 2.4 MB $\color{orange}{\text{+3.3 KB}}$
Stylesheet 63 63 0 319 KB 320 KB $\color{orange}{\text{+934 B}}$
Fetch 28 28 0 47 KB 47 KB 0 B
Image 9 9 0 238 KB 238 KB 0 B
Font 2 2 0 129 KB 129 KB 0 B
Other 9 9 0 421 KB 421 KB $\color{orange}{\text{+21 B}}$
V8 heap snapshot statistics
Metric @ Base @ Head Δ MAD
$\color{gray}{\rule{8pt}{8pt}}$ Total 12 MB
± 13 KB
12 MB
± 15 KB
$\text{-9.4 KB}$
$\text{-0.1\%}$
20 KB
$\color{orange}{\rule{8pt}{8pt}}$ Code 3 MB 3 MB $\text{+3 KB}$ 2.3 KB
$\color{red}{\rule{8pt}{8pt}}$ Strings 1.3 MB 1.3 MB $\text{+200 B}$ 364 B
$\color{cyan}{\rule{8pt}{8pt}}$ JS arrays 121 KB 121 KB $\text{+64 B}$ 0 B
$\color{green}{\rule{8pt}{8pt}}$ Typed arrays 0 B 0 B 0 B 0 B
$\color{yellow}{\rule{8pt}{8pt}}$ System objects 0 B 0 B 0 B 0 B
$\color{violet}{\rule{8pt}{8pt}}$ Other JS objs 2.1 MB 2.1 MB $\text{+372 B}$ 649 B
$\color{pink}{\rule{8pt}{8pt}}$ Other non-JS objs 5.4 MB 5.4 MB $\text{-2.4 KB}$ 22 KB

Download representative heap snapshot: base / head

📦 Bundle Stats

Chunk size diff (16 updated, 0 added, 0 removed)
Chunk Base Head Δ Δ (%)
(total) 6.1 MB 6.1 MB $\color{orange}{\text{+1.6 KB}}$ $\text{+0\%}$
src/components/MkCallsRoomWindow.vue ja-JP/5DD2B7tRW-DFUKMmdj.js → ja-JP/4lNOX5vHU-CGj2J6AZ.js
24 KB 24 KB $\text{+408 B}$ $\color{orange}{\text{+1.7\%}}$
src/pages/user/home.vue ja-JP/5DD2B7tRW-UJxs5ito.js → ja-JP/4lNOX5vHU-BiqmZuFv.js
15 KB 15 KB $\text{-69 B}$ $\color{green}{\text{-0.5\%}}$
src/pages/instance-info.vue ja-JP/5DD2B7tRW-Xwx6DO51.js → ja-JP/4lNOX5vHU-CCN2iG3X.js
10 KB 10 KB $\text{-36 B}$ $\color{green}{\text{-0.3\%}}$
src/pages/note.vue ja-JP/5DD2B7tRW-HLnFW8IF.js → ja-JP/4lNOX5vHU-BMhLPKaQ.js
5.2 KB 5.2 KB $\text{-36 B}$ $\color{green}{\text{-0.7\%}}$
src/pages/admin/users.vue ja-JP/5DD2B7tRW-CHmMmiH_.js → ja-JP/4lNOX5vHU-CJQ7QntG.js
4.5 KB 4.5 KB $\text{-36 B}$ $\color{green}{\text{-0.8\%}}$
src/pages/user/index.vue ja-JP/5DD2B7tRW-Dk9vFZjq.js → ja-JP/4lNOX5vHU-CG1Rnh24.js
8.9 KB 8.9 KB $\text{-34 B}$ $\color{green}{\text{-0.4\%}}$
src/pages/about.vue ja-JP/5DD2B7tRW-CMrFUdlQ.js → ja-JP/4lNOX5vHU-M_JuCBz2.js
4.4 KB 4.3 KB $\text{-33 B}$ $\color{green}{\text{-0.8\%}}$
src/pages/admin/abuses.vue ja-JP/5DD2B7tRW-BdkYrl_8.js → ja-JP/4lNOX5vHU-CAzXAKO1.js
7.5 KB 7.4 KB $\text{-32 B}$ $\color{green}{\text{-0.4\%}}$
src/pages/reversi/game.vue ja-JP/5DD2B7tRW-BGdFvbVi.js → ja-JP/4lNOX5vHU-DoPDXdMc.js
33 KB 33 KB $\text{+7 B}$ $\text{+0\%}$
src/_boot_.ts ja-JP/5DD2B7tRW-CU2o-p-K.js → ja-JP/4lNOX5vHU-DTDeGilK.js
71 KB 71 KB $\text{-6 B}$ $\text{-0\%}$
(other generated chunks) 2.3 MB 2.3 MB $\color{orange}{\text{+1.4 KB}}$ $\text{+0.1\%}$
(other) 283 KB 283 KB $\text{+10 B}$ $\text{+0\%}$
Startup chunk size (1 updated, 0 added, 0 removed)
Chunk Base Head Δ Δ (%)
(total) 935 KB 937 KB $\color{orange}{\text{+1.6 KB}}$ $\color{orange}{\text{+0.2\%}}$
src/_boot_.ts ja-JP/5DD2B7tRW-CU2o-p-K.js → ja-JP/4lNOX5vHU-DTDeGilK.js
71 KB 71 KB $\text{-6 B}$ $\text{-0\%}$
(other generated chunks) 861 KB 863 KB $\color{orange}{\text{+1.6 KB}}$ $\color{orange}{\text{+0.2\%}}$
(other) 2.9 KB 2.9 KB 0 B 0%

Startup chunks are the Vite entry for src/_boot_.ts and its static imports.

Bundles Modules Entries Imports Size
Static Dynamic Rendered Gzip Brotli
Base 506 3,004 22 11,213 364 11 MB 3 MB 2.6 MB
Head 505 3,007 22 11,221 364 11 MB 3 MB 2.6 MB
Δ $\color{green}{\text{-1}}$ $\color{orange}{\text{+3}}$ 0 $\color{orange}{\text{+8}}$ 0 $\color{orange}{\text{+5.2 KB}}$ $\color{orange}{\text{+1.7 KB}}$ $\color{orange}{\text{+1.5 KB}}$
Δ (%) $\color{green}{\text{-0.2\%}}$ $\text{+0.1\%}$ 0% $\text{+0.1\%}$ 0% $\text{+0\%}$ $\text{+0.1\%}$ $\text{+0.1\%}$

Open treemap HTML

@mattyatea
mattyatea marked this pull request as ready for review October 5, 2026 13:53
Copilot AI balanced review requested due to automatic review settings October 5, 2026 13:53
@mattyatea
mattyatea merged commit 0bb1aea into develop Oct 5, 2026
27 checks passed

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

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