Merged
Conversation
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
Add `agent watch` for streaming agent output as NDJSON, and bundle the amux Claude Code skill with helper scripts. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add `amux session list` for visibility into tmux sessions and `amux session prune` for cleaning up stale sessions. Prune targets orphaned workspace sessions and detached terminal tabs, with dry-run by default (--yes required), --older-than age filter, and safety guards for non-amux sessions and unknown creation times. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
workspace create now adds the project to the registry after saving metadata, so the TUI can discover CLI-created workspaces. Also adds a not_found test for workspace remove and a registry assertion test for workspace create. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Amp-Thread-ID: https://ampcode.com/threads/T-019c658d-879c-777b-8f8a-6818e87e8ed8 Co-authored-by: Amp <amp@ampcode.com>
- Add session/project to cliCommands dispatch map for TTY routing - Add WorkspaceStore.Update() for atomic load+modify+save with flock held; use it in removeTabFromStore to prevent lost-update races - Implement real LockFileEx/UnlockFileEx file locking on Windows - Add exponential backoff (20ms→1s cap) to send job queue polling - Fix nil slice JSON marshaling in agent list/stop responses - Fix shadowed job variable in agent send execute error path Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
andyrewlee
commented
Feb 16, 2026
Owner
Author
andyrewlee
left a comment
andyrewlee
left a comment
There was a problem hiding this comment.
Devin Review found 2 new potential issues.
View 30 additional findings in Devin Review.
- Add 'terminal' to cliCommands routing map - Use preserved jobID in not-found error path (cmd_agent_send_execute) - Replace fmt.Errorf with errors.New for static strings (perfsprint) Amp-Thread-ID: https://ampcode.com/threads/T-019c6887-c43a-75fe-bc86-c89c9ecb012d Co-authored-by: Amp <amp@ampcode.com>
andyrewlee
commented
Feb 16, 2026
Owner
Author
andyrewlee
left a comment
andyrewlee
left a comment
There was a problem hiding this comment.
Devin Review found 1 new potential issue.
View 34 additional findings in Devin Review.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
andyrewlee
commented
Feb 16, 2026
Owner
Author
andyrewlee
left a comment
andyrewlee
left a comment
There was a problem hiding this comment.
Devin Review found 1 new potential issue.
View 33 additional findings in Devin Review.
…top idempotency, update README Add comment explaining why sendJobStore.get() uses an exclusive lock (reconcileStale may write). Fix agent stop --all to not store idempotency key when blocking on missing --yes flag. Remove deprecated default_assistant from README config example. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The confirmation_required error represents a client-side precondition failure (missing --yes flag), not an actual operation outcome. Storing it in the idempotency store would block subsequent retries with --yes. Replace returnJSONErrorMaybeIdempotent with plain ReturnError calls in both cmd_agent_stop.go and cmd_workspace_remove.go so the idempotency key remains unconsumed and available for the actual retry. Amp-Thread-ID: https://ampcode.com/threads/T-019c6902-b634-7630-b1c3-247b7b2f2df9 Co-authored-by: Amp <amp@ampcode.com>
andyrewlee
changed the title
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
1 participant
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
Describe the change and intended behavior.
Quality Checklist
make devchecklocally.make lint-strict-newlocally for changed code.make harness-presets.go test ./internal/tmux ./internal/e2e.