feat: Drone pane — Phase 1 node-graph canvas for automated agents#256
Open
Agent1-asaf wants to merge 4 commits intomainfrom
Open
feat: Drone pane — Phase 1 node-graph canvas for automated agents#256Agent1-asaf wants to merge 4 commits intomainfrom
Agent1-asaf wants to merge 4 commits intomainfrom
Conversation
There was a problem hiding this comment.
ReAgent Diagnostics
| Field | Value |
|---|---|
| ReAgent Version | 5.12.5 |
| Trigger | PR opened |
| Project Context | CLAUDE.md loaded |
| Model | claude-opus-4-6 |
| Effort | high |
| Ref Repos | Disabled |
| Merge Analysis | Clean |
| Review Time | 108.7s |
| Timestamp | 2026-03-30T20:39:41Z |
| Repository | agentmuxai/agentmux |
| PR | #256 |
Confirmed: updateTrigger at line 46 shallow-merges patch onto the existing trigger, so changing type from "cron" to "manual" would leave stale expr on the object.
Issues:
- drone-model.ts:14 -
viewIconreturns"drone"which is not a valid FontAwesome icon; should be"robot"to match widgets.json and blockutil.tsx (every other view model matches its widget icon) - drone-edit-panel.tsx:46 - Changing trigger type via dropdown shallow-merges onto the old trigger object, leaving stale type-specific fields (e.g.
exprpersists after switching from cron to manual); should reset to a clean trigger of the new type - package.json:9 - New feature (new view type + widget visible in launcher) version-bumped as patch (0.33.1 → 0.33.2); per semver policy new features should be a minor bump (0.34.0)
- drone-model.ts: viewIcon "drone" → "robot" (valid FA icon, matches widgets.json) - drone-edit-panel.tsx: reset trigger to clean object on type change instead of shallow-merging, preventing stale fields (e.g. `expr` persisting after cron → manual) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
There was a problem hiding this comment.
ReAgent Diagnostics
| Field | Value |
|---|---|
| ReAgent Version | 5.12.5 |
| Trigger | Re-review (new commits) |
| Project Context | CLAUDE.md loaded |
| Model | claude-opus-4-6 |
| Effort | high |
| Ref Repos | Disabled |
| Merge Analysis | Clean |
| Review Time | 154.6s |
| Timestamp | 2026-03-30T20:45:54Z |
| Repository | agentmuxai/agentmux |
| PR | #256 |
Issues:
- docs/analysis/cef-vs-tauri-value-prop.md:1 - Unrelated to drone pane feature; 308-line CEF-vs-Tauri strategic analysis belongs in a separate PR or with PR #253 (CEF integration)
- frontend/app/view/drone/drone-view.css:752 -
@keyframes drone-pulseanimation defined but never referenced in any drone component (dead code)
…of branch - drone-view.css: remove @Keyframes drone-pulse (defined but never referenced) - docs/analysis/cef-vs-tauri-value-prop.md: untrack from this branch (belongs with PR #253/CEF work) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
There was a problem hiding this comment.
ReAgent Diagnostics
| Field | Value |
|---|---|
| ReAgent Version | 5.12.5 |
| Trigger | Re-review (new commits) |
| Project Context | CLAUDE.md loaded |
| Model | claude-opus-4-6 |
| Effort | high |
| Ref Repos | Disabled |
| Merge Analysis | Clean |
| Review Time | 104.2s |
| Timestamp | 2026-03-30T20:51:12Z |
| Repository | agentmuxai/agentmux |
| PR | #256 |
LGTM
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds the Drone pane — a new view type (`drone`) for defining and monitoring automated agents triggered by cron schedules, events, or other drones completing.
Phase 1 is the full frontend scaffold: interactive node-graph canvas with pan/zoom, draggable nodes, an edit panel, and a run log drawer. No backend scheduler yet (Phase 2).
What's included
New files — `frontend/app/view/drone/`
Modified files
What works in Phase 1
What's next (Phase 2)
🤖 Generated with Claude Code
Co-Authored-By: Claude Sonnet 4.6 noreply@anthropic.com