Skip to content

fix(desktop): ラベル統一 + スケジュール編集干渉 + IME Enter + ステータスPopover#269

Merged
MocA-Love merged 4 commits into
mainfrom
fix/268-model-effort-labels
Apr 17, 2026
Merged

fix(desktop): ラベル統一 + スケジュール編集干渉 + IME Enter + ステータスPopover#269
MocA-Love merged 4 commits into
mainfrom
fix/268-model-effort-labels

Conversation

@MocA-Love
Copy link
Copy Markdown
Owner

@MocA-Love MocA-Love commented Apr 17, 2026

概要

変更点

  1. ラベル統一 (”CLAUDEモデル"、"思考EFFORT"になっているが、"Model", "Effort"でいいとおもう。 #268): ClaudeRuntimePicker, TodoManager, ScheduleListRow の表示を Model / Effort に
  2. スケジュール一覧の編集メニュー干渉: ScheduleListRow で 3 点メニュー → 編集を押すとダイアログが一瞬で閉じる / 動作が不安定な問題を修正
    • DropdownMenumodal={false} + DropdownMenuItemonSelecte.preventDefault() + setTimeout(onEdit, 0) で Dialog オープンを次 tick に遅延
  3. IME 変換確定 Enter の誤送信: TodoManager.tsx の送信入力欄とリネーム入力で、e.nativeEvent.isComposing を見て IME 変換中は Enter を無視
  4. ヘッダーサービスステータスの UX 改善: Claude / Codex アイコンを Tooltip (hover) から Popover (click) に置換。ポップ内の status.claude.com を開く 等のリンクをクリックで外部ブラウザオープン

確認

  • bun run lint:fix
  • cd apps/desktop && bun run typecheck

Closes #268

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 17, 2026

Warning

Rate limit exceeded

@MocA-Love has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 53 minutes and 9 seconds before requesting another review.

Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 53 minutes and 9 seconds.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 48733515-2ada-45e8-b6ef-cd3e25038aab

📥 Commits

Reviewing files that changed from the base of the PR and between acd5d1a and 2e450c1.

📒 Files selected for processing (3)
  • apps/desktop/src/renderer/features/todo-agent/TodoManager/SchedulesSection/components/ScheduleListRow/ScheduleListRow.tsx
  • apps/desktop/src/renderer/features/todo-agent/TodoManager/TodoManager.tsx
  • apps/desktop/src/renderer/routes/_authenticated/_dashboard/components/TopBar/components/ServiceStatusIndicators/ServiceStatusIndicators.tsx
📝 Walkthrough

概要

ClaudeRuntimePickerおよびTodoManagerコンポーネント内の日本語ラベルテキストを英語に変更しました。「Claude モデル」を「Model」に、「思考 effort」を「Effort」に統一する国際化対応です。コンポーネントロジックや機能は変更されていません。

変更内容

コホート / ファイル 要約
UIラベルの英語化
apps/desktop/src/renderer/features/todo-agent/ClaudeRuntimePicker/ClaudeRuntimePicker.tsx, apps/desktop/src/renderer/features/todo-agent/TodoManager/TodoManager.tsx
ピッカーおよびセッション詳細表示のラベルテキストを日本語から英語に変更。「Claude モデル」→「Model」、「思考 effort」→「Effort」。レンダリング値や機能は変更なし。

推定コードレビュー工数

🎯 1 (Trivial) | ⏱️ ~3 minutes

関連する可能性のあるPR

兎からの応援詩

🐰 ラベルが変わりて清し
Model、Effortと響けば
世界の言葉で踊り出す
シンプルなテキスト、大きな一歩
ぴょん、と進む国際化の道

🚥 Pre-merge checks | ✅ 2 | ❌ 3

❌ Failed checks (2 warnings, 1 inconclusive)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 50.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Description check ⚠️ Warning PR の説明は日本語で構成されており、Template で要求される English での記述がなく、また Type of Change などの必須セクションが填却されていません。 PR 説明を英語で記載し、Description Template の必須項目(Type of Change, Related Issues など)をすべて填却してください。
Title check ❓ Inconclusive PR タイトルは日本語で複数の修正内容を列挙しており、実際の変更(ラベル統一)の一部を含みますが、他の記載内容との整合性が不明確です。 タイトルを英語に統一し、主要な変更内容を明確に反映させることをお勧めします。例:「fix(desktop): Unify TODOAgent labels to Model/Effort"
✅ Passed checks (2 passed)
Check name Status Explanation
Linked Issues check ✅ Passed Issue #268は「ラベルを 'Model' と 'Effort' に統一する」という要件を定義しており、PRの変更内容(ClaudeRuntimePickerとTodoManagerの2つのコンポーネントでラベルを日本語から英語に変更)がこの要件を完全に満たしている。
Out of Scope Changes check ✅ Passed すべての変更は明確にスコープ内である。ClaudeRuntimePickerとTodoManagerのラベルテキストのみが変更されており、ロジック、プロップス、状態管理、イベント処理は変更されていない。

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/268-model-effort-labels

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 and usage tips.

@MocA-Love
Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector
Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Keep them coming!

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@MocA-Love
Copy link
Copy Markdown
Owner Author

@coderabbitai review

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 17, 2026

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

- ScheduleListRow の編集/削除メニューが Dialog と干渉して一瞬で閉じる問題を修正 (setTimeout で open を遅延、modal=false)
- TodoManager の Enter 送信で IME 変換確定時の誤送信を防止 (isComposing チェック)
- ヘッダーの Claude/Codex ステータスをホバー起動 Tooltip からクリック起動 Popover に変更し、中のリンクからサイトを開く形に
@MocA-Love MocA-Love changed the title fix(desktop): TODOエージェントのラベルを "Model" / "Effort" に fix(desktop): ラベル統一 + スケジュール編集干渉 + IME Enter + ステータスPopover Apr 17, 2026
@MocA-Love
Copy link
Copy Markdown
Owner Author

@codex review

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 6e0b44240e

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Codex レビュー指摘対応。onSelect 内の e.preventDefault() を外すことで
Radix の既定クローズ挙動を残しつつ、setTimeout で Dialog を次 tick に
遅延して Outside-click 競合も回避。
@MocA-Love MocA-Love merged commit a92a8ee into main Apr 17, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

”CLAUDEモデル"、"思考EFFORT"になっているが、"Model", "Effort"でいいとおもう。

1 participant