feat: add Sprites integration with exec API improvements #287
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
Implement PR #103 feedback to use the Sprites exec API directly instead of tmux-based execution. This supersedes PR #103 with the improvements suggested in the review comments.
Key Improvements from PR #103 Review
Native exec sessions instead of tmux (PR comment #1)
cmd.Wait()blocks until completion (no 2-second polling)apt-get install tmuxfrom setupDirect stdout streaming (PR comment #3)
Filesystem API for setup (PR comment #4)
sprite.Filesystem().WriteFile()andMkdirAll()Port notifications (PR comment #6)
PortNotificationMessagefor dev serversCrash recovery foundation (PR comment #2)
Network policy documentation (PR comment #5)
New Files
cmd/task/sprite.go- CLI commands for sprite managementinternal/sprites/sprites.go- Shared token/client logicinternal/executor/executor_sprite.go- SpriteRunner with exec APIdocs/sprites-design.md- Architecture documentationdocs/sprites-discussion.md- Design discussionUsage
Test plan
task spritesubcommandsRelated
Supersedes #103 - Implements the exec API improvements suggested in the PR review comments.
🤖 Generated with Claude Code