Skip to content

feat(desktop): TODO composer に PTY / Remote Control トグルを追加#320

Merged
MocA-Love merged 1 commit intomainfrom
feat/todo-composer-pty-toggle
Apr 18, 2026
Merged

feat(desktop): TODO composer に PTY / Remote Control トグルを追加#320
MocA-Love merged 1 commit intomainfrom
feat/todo-composer-pty-toggle

Conversation

@MocA-Love
Copy link
Copy Markdown
Owner

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

概要

PR #278 で TodoModal (ダイアログ版) に追加した「PTY モードで実行」「Remote Control を有効化」の2つのチェックボックスが、TodoManager 内のインライン composer (「新しい TODO」カード) には入っていなかった。通常の TODO 作成フロー (Agent Manager 画面) からは composer を使うため、その経路では PTY / Remote Control を有効化できない状態だった。

変更内容

  • TodoManager.tsx の composer セクションに ptyEnabled / remoteControlEnabled の useState を追加
  • PTY を OFF に戻したとき RC が残っていたら自動でクリアする useEffect
  • createMut.mutateAsync の呼び出しと dependency array に両フィールドを追加
  • worktree チェックの直下に TodoModal と同じスタイル (emerald / indigo) のチェックボックス label を配置
  • RC チェックは PTY が OFF のとき disabled

確認済み

  • typecheck 通過 (既存の codemirror バージョン不整合警告は本 PR 無関係)
  • lint pass

Test plan

  • Agent Manager から「新しい TODO」を開き、PTY モードと Remote Control のトグルが表示されること
  • PTY OFF のとき RC は disabled になること
  • PTY ON → RC ON で作成したセッションが PTY エンジンで起動すること
  • PTY OFF で作成したセッションは従来通り -p 経路で起動すること

Summary by CodeRabbit

新機能

  • PTYモード実行: タスクをPTYモードで実行するオプションが追加されました。デフォルトは無効です。
  • リモートコントロール機能: PTYモード有効時にのみ利用可能なリモートコントロール機能が追加されました。PTYモードが無効になると自動的に無効化されます。

TodoManager 内のインライン composer (「新しい TODO」カード) にも
TodoModal と同じ「PTY モードで実行」「Remote Control を有効化」の
2 つのチェックボックスを追加。これまで TodoModal 経由でしか
toggle を出せていなかったため、通常の作成フローからは PTY /
Remote Control を有効化できない状態だった。

- useState で ptyEnabled / remoteControlEnabled を追加
- PTY を OFF にしたとき RC が残っている場合に自動クリアする
  useEffect を追加
- createMut.mutateAsync と dependency 配列に両フィールドを追加
- TodoModal と同じスタイル (emerald / indigo 系) の label を
  worktree チェックの直下に配置
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 18, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 916a7574-54cc-4123-b734-ba36f311e50f

📥 Commits

Reviewing files that changed from the base of the PR and between d4a277e and e69bfe4.

📒 Files selected for processing (1)
  • apps/desktop/src/renderer/features/todo-agent/TodoManager/TodoManager.tsx

📝 Walkthrough

Walkthrough

TodoManager.tsx に新しい composer state として ptyEnabledremoteControlEnabled の 2 つのトグルが追加されました。ptyEnabled が false の場合に自動的に remoteControlEnabled をクリアする useEffect ロジックとともに、対応するチェックボックス UI 控制も実装されています。

Changes

Cohort / File(s) Summary
TodoManager コンポーザー状態拡張
apps/desktop/src/renderer/features/todo-agent/TodoManager/TodoManager.tsx
ptyEnabled および remoteControlEnabled の新規状態トグルを追加。ptyEnabled が false の場合に remoteControlEnabled を自動的にクリアする useEffect ロジックを実装。これらの値を electronTrpc.todoAgent.create ペイロードに含め、対応するチェックボックス UI 控制(「PTY モードで実行」と「Remote Control を有効化」)を追加。Remote Control チェックボックスは ptyEnabled が無効な場合は無効化される。

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Poem

🐰 PTY モード、リモート制御と、
新しきスイッチを実装す。
依存関係も無碍に処理し、
トグルの連鎖、うさぎ流儀。
コードよ輝け、バグなく🌙

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/todo-composer-pty-toggle

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 merged commit e6df4e6 into main Apr 18, 2026
5 of 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.

1 participant