feat(loops): edit loop details inline

[MattPua]

Problem

Editing loop metadata, schedules, or model configuration requires navigating to the full edit flow.

Changes

• Edit titles, descriptions, and instructions inline
• Edit schedule and model configuration from loop details
• Preserve unsaved configuration across unrelated loop updates
• Save metadata on blur, with keyboard cancel support

How did you test this?

• UI typecheck and loop tests covered by CI
• Focused QA across logic, contracts/security, and performance/reliability

Automatic notifications

• Publish to changelog?
• Alert Sales and Marketing teams?

---

Created with PostHog Code

[trunk-io]

Merging to main in this repository is managed by Trunk.

• To merge this pull request, check the box to the left or comment /trunk merge below.

After your PR is submitted to the merge queue, this comment will be automatically updated with its status. If the PR fails, failure details will also be posted here

[github-actions]

React Doctor found 9 issues in 2 files · 9 warnings.

9 warnings

src/features/loops/components/InlineLoopScheduleEditor.tsx

• ⚠️ src/features/loops/components/InlineLoopScheduleEditor.tsx:12 Non-component export in component file only-export-components

src/features/loops/components/LoopDetailView.tsx

• ⚠️ src/features/loops/components/LoopDetailView.tsx:523 Chained array iterations js-combine-iterations
• ⚠️ src/features/loops/components/LoopDetailView.tsx:540 Many related useState calls prefer-useReducer
• ⚠️ src/features/loops/components/LoopDetailView.tsx:541 Prop derived into useState no-derived-useState
• ⚠️ src/features/loops/components/LoopDetailView.tsx:542 Prop derived into useState no-derived-useState
• ⚠️ src/features/loops/components/LoopDetailView.tsx:543 Prop derived into useState no-derived-useState
• ⚠️ src/features/loops/components/LoopDetailView.tsx:547 State only used in handlers rerender-state-only-in-handlers
• ⚠️ src/features/loops/components/LoopDetailView.tsx:585 Chained array iterations js-combine-iterations
• ⚠️ src/features/loops/components/LoopDetailView.tsx:679 Chained array iterations js-combine-iterations

_{Reviewed by React Doctor for commit 6e210c6.}

[greptile-apps]

Prompt To Fix All With AI

Fix the following 2 code review issues. Work through them one at a time, proposing concise fixes.

---

### Issue 1 of 2
packages/ui/src/features/loops/components/LoopDetailView.tsx:489-498
**Model edits create schedules**

When a loop has no schedule and the user changes only its model configuration, the synthetic enabled schedule draft is unconditionally appended to the update payload, causing the loop to begin running automatically on the default schedule.

### Issue 2 of 2
packages/ui/src/features/loops/components/LoopDetailView.tsx:416-421
**Loop updates leave stale drafts**

When the queried loop changes while this view remains mounted, these states retain the previous configuration; `hasChanges` then compares stale drafts with the latest loop and saving writes the stale model and schedule values back, overwriting newer configuration.

_{Reviews (1): Last reviewed commit: "fix: clarify loop agent selector label" | Re-trigger Greptile}

[charlesvien]

Does this work with GitHub triggers and the other options? Maybe we should just prompt that same wizard view and make it a shared dialog and just have an edit button.

[MattPua]

Does this work with GitHub triggers and the other options? Maybe we should just prompt that same wizard view and make it a shared dialog and just have an edit button.

do you mean like show the wizard view as a modal vs the inline?

i havent checked the other triggers yet, selfishly only looked at the stuff i needed