feat(desktop): Start前のTODOのdescription/goalを編集可能に#197
Conversation
AgentManagerで一度作成した自律TODOの 「やって欲しいこと」 と 「ゴール」 を Start 前に修正できず、作り直しになっていた。 - todoAgent.updateFields mutation を追加。status が queued/ failed/aborted/escalated のときのみ description/goal の更新を 許可し、変更後は goal.md も書き直す(次回 Start 時に必ず 新しい内容が読まれるように) - SessionDetail に行内編集 UI を追加。canStart && !isRunning の セッションでのみ DetailBlock に 「編集」 ボタンが出る Closes #186
|
Warning Rate limit exceeded
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 41 minutes and 6 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the 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 configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 9079294c25
ℹ️ 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".
| const [editingField, setEditingField] = useState< | ||
| "description" | "goal" | null | ||
| >(null); | ||
| const [editDraft, setEditDraft] = useState(""); |
There was a problem hiding this comment.
Reset edit state when switching selected session
SessionDetail now stores editingField/editDraft in component state, but those values are never cleared when session.id changes. This component is reused across selections (the file already documents that behavior for streamEvents), so if a user starts editing session A and then clicks session B, the draft from A stays visible and pressing save will update B with stale text. Add a useEffect keyed by session.id (or key the component by id) to clear edit mode and draft on selection changes.
Useful? React with 👍 / 👎.
SessionDetailが再利用されるため、editingField/editDraftが 前のセッションに残ったまま別セッションを選ぶと誤保存の 可能性があった。session.id変更時のresetブロックで一緒に クリアする。 Refs: Codex review on PR #197
概要
Issue #186 の対応。AgentManager で作成したばかりの自律 TODO について、Start 前は 「やって欲しいこと」 と 「ゴール」 を編集できるようにしたい、という要望。これまでは一度作ってしまうと作り直すしか手段がなかった。
実装
Backend
Frontend
Test plan
Closes #186