Skip to content

fix: Trim redundant sections in small tool skills#1977

Merged
hatayama merged 1 commit into
feature/skill-prose-compressionfrom
feature/skill-prose-small-tools
Jul 24, 2026
Merged

fix: Trim redundant sections in small tool skills#1977
hatayama merged 1 commit into
feature/skill-prose-compressionfrom
feature/skill-prose-small-tools

Conversation

@hatayama

@hatayama hatayama commented Jul 24, 2026

Copy link
Copy Markdown
Owner

Summary

Small fact-preserving trims across five skills, same editorial rule as #1975/#1976 (each fact stated once, restatements removed):

  • find-game-objects: the "Zero-hit Exact-mode hint" section restated the Search Modes table's Exact-mode trap — its own text acknowledged it ("the same trap described above"). Deleted; the table row keeps the hint fact.
  • control-play-mode: the Resume bullet loses its adoption rationale, the Status bullet loses its closing restatement; every behavioral fact (no side effects, BlockedByCompileErrors from last compile result, BlockedByUnsavedChanges semantics) remains.
  • simulate-mouse-ui: empty-space click/drag semantics were stated in both Coordinate System and the Output note; kept only in the Output note (the more complete one, with HitGameObjectName = null). Two of three bypass-raycast examples removed; the remaining Click and drop-target Drag examples cover both flag usages.
  • get-version: removed the stale "This is a sample custom tool" note — hello-world is the sample tool, and grep -ri sample docs/ has no get-version reference.
  • get-project-info: removed the Examples section that repeated the Usage command verbatim.

Test plan

  • dist/darwin-arm64/uloop skills install --claude --agents; the three distributed skills verified byte-identical between source and .claude/ / .agents/ copies (get-version / get-project-info are internal: true and have no generated copies, unchanged behavior)

Review in cubic

Remove prose that restates facts already documented in the same file:

- find-game-objects: delete the "Zero-hit Exact-mode hint" section,
  which restated the Exact-mode trap already described in the Search
  Modes table (and said so itself)
- control-play-mode: drop the Resume alias adoption rationale and the
  closing restatement in the Status bullet; all behavioral facts stay
- simulate-mouse-ui: keep empty-space click/drag semantics only in the
  Output note (which also documents HitGameObjectName = null) instead
  of stating them twice, and drop two bypass-raycast examples covered
  by the remaining Click and drop-target Drag examples
- get-version: delete the stale "sample custom tool" note (hello-world
  is the sample tool; no docs reference get-version as a sample)
- get-project-info: delete the Examples section that only repeated the
  Usage command
@coderabbitai

coderabbitai Bot commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

@hatayama, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 57 minutes

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 5e67d703-fd47-4dac-8af9-1f806695442c

📥 Commits

Reviewing files that changed from the base of the PR and between 6394326 and 0cca13f.

📒 Files selected for processing (11)
  • .agents/skills/uloop-control-play-mode/SKILL.md
  • .agents/skills/uloop-find-game-objects/SKILL.md
  • .agents/skills/uloop-simulate-mouse-ui/SKILL.md
  • .claude/skills/uloop-control-play-mode/SKILL.md
  • .claude/skills/uloop-find-game-objects/SKILL.md
  • .claude/skills/uloop-simulate-mouse-ui/SKILL.md
  • Packages/src/Editor/CliOnlyTools~/GetProjectInfo/Skill/SKILL.md
  • Packages/src/Editor/CliOnlyTools~/GetVersion/Skill/SKILL.md
  • Packages/src/Editor/FirstPartyTools/ControlPlayMode/Skill/SKILL.md
  • Packages/src/Editor/FirstPartyTools/FindGameObjects/Skill/SKILL.md
  • Packages/src/Editor/FirstPartyTools/SimulateMouseUi/Skill/SKILL.md
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/skill-prose-small-tools

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.

@hatayama
hatayama merged commit 90a174c into feature/skill-prose-compression Jul 24, 2026
2 checks passed
@hatayama
hatayama deleted the feature/skill-prose-small-tools branch July 24, 2026 09:31
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