Skip to content

fix(desktop): Agent Manager 設定ダイアログでマネージャが閉じる問題を修正 (#217)#221

Merged
MocA-Love merged 1 commit into
mainfrom
fix/217-agent-manager-settings-dialog
Apr 16, 2026
Merged

fix(desktop): Agent Manager 設定ダイアログでマネージャが閉じる問題を修正 (#217)#221
MocA-Love merged 1 commit into
mainfrom
fix/217-agent-manager-settings-dialog

Conversation

@MocA-Love
Copy link
Copy Markdown
Owner

概要

Issue #217 の修正。Agent Manager の「設定 / プリセット」ボタンで開く設定ダイアログが、
開くたびに Agent Manager 自体を閉じてしまう 問題を修正した。

原因

TodoManager.tsx では PresetsDialog を次のように DialogContent の外側・Dialog
兄弟として配置していた:

```tsx

... // ← DialogContent の外 \`\`\`

この配置だと Radix UI の DismissableLayer 側で 兄弟関係のネストを「ネスト」として
認識できない
。結果、設定ダイアログ内のクリックが親 DialogContent から見て
「外側クリック」と判定され、onOpenChange(false) が発火して Agent Manager が
閉じてしまっていた。

スケジュールタブの「新規」ボタンで開く ScheduleEditorDialog
SchedulesSection の中(= DialogContent の内側)で描画されているため、
Radix が正しく入れ子と認識して親は閉じない。これが挙動差の原因だった。

修正

PresetsDialogDialogContent の内側へ移動し、ScheduleEditorDialog
同じ配置パターンに揃えた。

テスト手順

  • Agent Manager を開く
  • 左下の「設定 / プリセット」をクリック
  • 設定ダイアログが Agent Manager の に重ねて開き、Agent Manager は
    閉じないことを確認
  • 設定ダイアログを閉じると Agent Manager だけが残ることを確認
  • スケジュールタブ → 「新規」ボタンの従来挙動に変化がないことを確認

Closes #217

PresetsDialog を DialogContent の外側(Dialog の兄弟)に置いていたため、
Radix の DismissableLayer が兄弟同士のネストを認識できず、設定ダイアログ
内のクリックが「外側クリック」と判定されて Agent Manager が閉じていた。
正しく動作している ScheduleEditorDialog と同じく DialogContent 内部で
描画するよう修正した。

Closes #217
@chatgpt-codex-connector
Copy link
Copy Markdown

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Repo admins can enable using credits for code reviews in their settings.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 16, 2026

Warning

Rate limit exceeded

@MocA-Love has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 2 minutes and 8 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 2 minutes and 8 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: e8c41895-ca7e-4272-a0e2-e9c4564ea0c8

📥 Commits

Reviewing files that changed from the base of the PR and between 8c082df and 16a68ca.

📒 Files selected for processing (1)
  • apps/desktop/src/renderer/features/todo-agent/TodoManager/TodoManager.tsx
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/217-agent-manager-settings-dialog

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 MocA-Love self-assigned this Apr 16, 2026
MocA-Love added a commit that referenced this pull request Apr 16, 2026
Conflict resolution: keep composer Dialog inside the parent DialogContent
(per #221 fix for Issue #217) instead of as a sibling, to prevent the
outer Manager Dialog from closing.
@MocA-Love MocA-Love merged commit bd09979 into main Apr 16, 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.

[bug] Agent Managerの設定ダイアログについて

1 participant