fix: Remove fork context from one-shot uloop tool skills#1909
Merged
hatayama merged 1 commit intoJul 21, 2026
Merged
Conversation
Skills marked with "context: fork" run as forked subagents that do not receive the caller's task, so invoking simulate-keyboard, simulate-mouse-input, simulate-mouse-ui, or execute-dynamic-code through a Skill() call stalled asking what to do instead of executing anything (observed in the Round-6 field verification sessions). These skills wrap single CLI commands and gain nothing from subagent isolation, so drop the fork context and let them load inline with the caller's context. Generated copies under .claude/skills and .agents/skills are regenerated with "uloop skills install --claude --agents".
Contributor
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (12)
💤 Files with no reviewable changes (12)
📝 WalkthroughWalkthroughRemoved ChangesSkill frontmatter cleanup
Estimated code review effort: 1 (Trivial) | ~2 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ 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 |
hatayama
merged commit Jul 21, 2026
64fddf1
into
feature/round6-pause-point-improvements
2 checks passed
2 tasks
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
Part of Round-6 pause-point usability improvements (PR-3 of 5, targeting the
feature/round6-pause-point-improvementsintegration branch).Removes the
context: forkfrontmatter from the four uloop tool skills that wrap single CLI commands:uloop-simulate-keyboarduloop-simulate-mouse-inputuloop-simulate-mouse-uiuloop-execute-dynamic-codeWhy
context: forkmakes a skill run as a forked subagent that does not receive the caller's task. In the Round-6 field verification sessions (3D voxel sandbox project), invoking these skills through aSkill()call spawned a subagent that only asked "what should I do?" and executed nothing, forcing the agent to fall back to rawuloopCLI calls via Bash. These skills are thin wrappers around one-shot commands and gain nothing from subagent isolation, so they should load inline with the caller's context.Changes
context: forkline from the four sourceSKILL.mdfiles underPackages/src/Editor/FirstPartyTools/<Tool>/Skill/..claude/skills/and.agents/skills/copies withuloop skills install --claude --agents.Verification
git grep -n "context: fork" -- Packages .claude .agentsreturns only a historical CHANGELOG entry; no skill file retains the fork context.fork/subagentreferences).