Skip to content

fix: Make bundled skill guidance more concise#1178

Merged
hatayama merged 2 commits into
mainfrom
feature/hatayama/port-short-skill-descriptions
May 21, 2026
Merged

fix: Make bundled skill guidance more concise#1178
hatayama merged 2 commits into
mainfrom
feature/hatayama/port-short-skill-descriptions

Conversation

@hatayama
Copy link
Copy Markdown
Owner

@hatayama hatayama commented May 21, 2026

Summary

  • Shorten bundled uloop skill descriptions so agents see concise routing guidance.
  • Soften Input System prerequisites so skills describe when to use the tools instead of telling users to change Player Settings.

User Impact

  • Agents reading the source skill definitions get clearer, less prescriptive guidance when choosing uloop commands.
  • Input recording and simulation guidance now points agents toward projects that already use the New Input System, avoiding advice that could sound like a required project setting change.

Changes

  • Port concise v3-beta descriptions into the v2 source SKILL.md files.
  • Align ExecuteDynamicCode reference examples and AssetDatabase refresh guidance with v3-beta.
  • Keep generated .claude, .agents, and .codex copies out of this PR; they can be regenerated separately.

Verification

  • npm --prefix Packages/src/Cli~ run build
  • npm --prefix Packages/src/Cli~ run test:cli -- --runTestsByPath src/tests/skills-manager.test.ts src/tests/skills-command.test.ts
  • git diff --check
  • Checked that the final diff contains only source skill definition files and no generated skill copies.

Summary

This PR shortens and refines bundled uloop skill descriptions across source SKILL.md files to give agents more concise, actionable routing guidance and to soften prescriptive language around Input System prerequisites.

Key Changes

  • Shortened descriptions across multiple skills to be more concise and action-focused.
  • Ports concise v3-beta wording into v2 source SKILL.md files.
  • Softened Input System prerequisites: RecordInput, ReplayInput, SimulateKeyboard, and SimulateMouseInput now instruct agents to use those tools only when a project already uses the New Input System instead of recommending Player Settings changes.
  • ExecuteDynamicCode docs aligned with v3-beta:
    • Clarified AssetDatabase refresh guidance (asset-operations.md).
    • Simplified PlayMode automation examples in PowerShell and zsh by removing explicit default flags (--include-inactive false, --include-components true).

Files Modified (source SKILL.md only)

  • Packages/src/Cli~/src/skills/skill-definitions/cli-only/uloop-focus-window/Skill/SKILL.md
  • Packages/src/Cli~/src/skills/skill-definitions/cli-only/uloop-launch/Skill/SKILL.md
  • Packages/src/Cli~/src/skills/skill-definitions/cli-only/uloop-get-version/Skill/SKILL.md
  • Packages/src/Cli~/src/skills/skill-definitions/cli-only/uloop-get-project-info/Skill/SKILL.md
  • Packages/src/Editor/Api/McpTools/ClearConsole/Skill/SKILL.md
  • Packages/src/Editor/Api/McpTools/Compile/Skill/SKILL.md
  • Packages/src/Editor/Api/McpTools/ControlPlayMode/Skill/SKILL.md
  • Packages/src/Editor/Api/McpTools/ExecuteDynamicCode/Skill/SKILL.md
  • Packages/src/Editor/Api/McpTools/ExecuteDynamicCode/Skill/references/asset-operations.md
  • Packages/src/Editor/Api/McpTools/ExecuteDynamicCode/Skill/references/playmode-automation-powershell.md
  • Packages/src/Editor/Api/McpTools/ExecuteDynamicCode/Skill/references/playmode-automation-zsh.md
  • Packages/src/Editor/Api/McpTools/FindGameObjects/Skill/SKILL.md
  • Packages/src/Editor/Api/McpTools/GetHierarchy/Skill/SKILL.md
  • Packages/src/Editor/Api/McpTools/GetLogs/Skill/SKILL.md
  • Packages/src/Editor/Api/McpTools/RecordInput/Skill/SKILL.md
  • Packages/src/Editor/Api/McpTools/ReplayInput/Skill/SKILL.md
  • Packages/src/Editor/Api/McpTools/RunTests/Skill/SKILL.md
  • Packages/src/Editor/Api/McpTools/Screenshot/Skill/SKILL.md
  • Packages/src/Editor/Api/McpTools/SimulateKeyboard/Skill/SKILL.md
  • Packages/src/Editor/Api/McpTools/SimulateMouseInput/Skill/SKILL.md
  • Packages/src/Editor/Api/McpTools/SimulateMouseUi/Skill/SKILL.md

User Impact

  • Agents reading source skill definitions receive clearer, less prescriptive guidance for choosing uloop commands.
  • Input recording and simulation guidance now directs agents toward projects that already use the New Input System rather than implying required project setting changes.

Verification

  • Build and tests run: npm --prefix Packages/src/Cli~ run build; npm --prefix Packages/src/Cli~ run test:cli -- --runTestsByPath src/tests/skills-manager.test.ts src/tests/skills-command.test.ts
  • Performed git diff --check and confirmed final diff contains only source skill definition files (no generated .claude/.agents/.codex copies).

Review Change Stack

Port the concise v3-beta wording into the v2 source skill definitions so agent-facing descriptions stay focused on when each uloop command should be used.
Include the matching ExecuteDynamicCode references and input-system prerequisite wording while leaving generated skill copies unchanged.
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 21, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 88b0e1f9-3780-48fe-9503-05fad0eb3b25

📥 Commits

Reviewing files that changed from the base of the PR and between d3fde3e and 629805b.

📒 Files selected for processing (1)
  • Packages/src/Cli~/src/skills/skill-definitions/cli-only/uloop-launch/Skill/SKILL.md

📝 Walkthrough

Walkthrough

This PR updates descriptions and prerequisites across 16 uloop skill definition files. Changes shorten and clarify frontmatter descriptions to emphasize practical use cases, simplify PlayMode automation command examples by removing redundant default flags, and consolidate New Input System prerequisites across input simulation tools into consistent, concise language.

Changes

Skill Descriptions and Prerequisites

Layer / File(s) Summary
Editor launch and window management
Packages/src/Cli~/src/skills/skill-definitions/cli-only/uloop-launch/Skill/SKILL.md, Packages/src/Cli~/src/skills/skill-definitions/cli-only/uloop-focus-window/Skill/SKILL.md
uloop-launch description updated to "Launch or restart Unity Editor" and uloop-focus-window shortened to emphasize bringing the Editor window to foreground.
Build and test workflow tools
Packages/src/Editor/Api/McpTools/ClearConsole/Skill/SKILL.md, Packages/src/Editor/Api/McpTools/Compile/Skill/SKILL.md, Packages/src/Editor/Api/McpTools/ControlPlayMode/Skill/SKILL.md, Packages/src/Editor/Api/McpTools/RunTests/Skill/SKILL.md
Descriptions shortened for uloop-clear-console (clearing logs before compile/tests), uloop-compile (compiling with error/warning reporting), uloop-control-play-mode (starting/stopping/pausing PlayMode), and uloop-run-tests (running EditMode/PlayMode tests).
Scene hierarchy and object inspection tools
Packages/src/Editor/Api/McpTools/FindGameObjects/Skill/SKILL.md, Packages/src/Editor/Api/McpTools/GetHierarchy/Skill/SKILL.md, Packages/src/Editor/Api/McpTools/GetLogs/Skill/SKILL.md, Packages/src/Editor/Api/McpTools/Screenshot/Skill/SKILL.md
uloop-find-game-objects and uloop-get-hierarchy descriptions clarified to emphasize selected object inspection and hierarchical tree structure. uloop-get-logs reframed to "read from running Editor" for investigation. uloop-screenshot shortened to focus on PNG capture for visual checks/debugging.
Dynamic code execution and asset management
Packages/src/Editor/Api/McpTools/ExecuteDynamicCode/Skill/SKILL.md, Packages/src/Editor/Api/McpTools/ExecuteDynamicCode/Skill/references/asset-operations.md
uloop-execute-dynamic-code description reframed to emphasize C# execution when other tools cannot inspect/edit (scene/prefab/SerializedObject/AssetDatabase/menu/PlayMode cases). Added clarification to asset-operations.md describing when to call AssetDatabase.Refresh() after terminal-based asset changes.
PlayMode automation command examples
Packages/src/Editor/Api/McpTools/ExecuteDynamicCode/Skill/references/playmode-automation-powershell.md, Packages/src/Editor/Api/McpTools/ExecuteDynamicCode/Skill/references/playmode-automation-zsh.md
Simplified find-game-objects and get-hierarchy CLI examples in both PowerShell and zsh docs by removing explicit default flag values (--include-inactive false and --include-components true).
Input recording and replay tools
Packages/src/Editor/Api/McpTools/RecordInput/Skill/SKILL.md, Packages/src/Editor/Api/McpTools/ReplayInput/Skill/SKILL.md
uloop-record-input and uloop-replay-input descriptions shortened to emphasize JSON recording for exact replay, E2E runs, and demos. Prerequisites unified: replaced "Active Input Handling" Player Settings requirement with simpler "use only when project already uses New Input System." uloop-replay-input gained a dedicated Prerequisites section listing PlayMode requirement and package/system dependency checks.
Keyboard and mouse input simulation tools
Packages/src/Editor/Api/McpTools/SimulateKeyboard/Skill/SKILL.md, Packages/src/Editor/Api/McpTools/SimulateMouseInput/Skill/SKILL.md, Packages/src/Editor/Api/McpTools/SimulateMouseUi/Skill/SKILL.md
uloop-simulate-keyboard reworded to explicitly target PlayMode via Unity Input System. uloop-simulate-mouse-input clarified to simulate Mouse.current and guide users toward New Input System projects. uloop-simulate-mouse-ui description updated. All three tools updated to remove fallback workarounds and state prerequisites as "use only when project already uses New Input System."

🎯 2 (Simple) | ⏱️ ~10 minutes


Possibly related PRs

  • hatayama/unity-cli-loop#1003: Documentation rewrites for uloop-find-game-objects, uloop-get-hierarchy, and uloop-execute-dynamic-code descriptions align with steering agents toward the correct tool for selected GameObject inspection.
  • hatayama/unity-cli-loop#1002: Both PRs update uloop-find-game-objects and uloop-get-hierarchy descriptions to emphasize selection-based inspection, overlapping in the same skill documentation files.
  • hatayama/unity-cli-loop#1151: Documentation-only updates that rewrite the same set of SKILL.md frontmatter descriptions and prerequisites across multiple tools (clear-console, compile, control-play-mode, execute-dynamic-code, find-game-objects, get-hierarchy, get-logs, launch, replay-input, run-tests, and input simulation tools).
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'fix: Make bundled skill guidance more concise' directly and accurately summarizes the main change: shortening skill descriptions across multiple bundled uloop tools for more concise agent guidance.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/hatayama/port-short-skill-descriptions

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@hatayama hatayama changed the title chore: Make bundled skill guidance more concise fix: Make bundled skill guidance more concise May 21, 2026
Avoid repeating open/restart wording while keeping the guidance focused on matching Unity Editor versions and clean restarts.
@hatayama hatayama merged commit dbcc38e into main May 21, 2026
4 checks passed
@hatayama hatayama deleted the feature/hatayama/port-short-skill-descriptions branch May 21, 2026 05:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant