Skip to content

fix: Clarify pause point skill guidance#1345

Merged
hatayama merged 3 commits into
v3-betafrom
feature/update-skills
Jun 15, 2026
Merged

fix: Clarify pause point skill guidance#1345
hatayama merged 3 commits into
v3-betafrom
feature/update-skills

Conversation

@hatayama

@hatayama hatayama commented Jun 15, 2026

Copy link
Copy Markdown
Owner

Summary

  • Clarifies when agents should use the pause-point skill for frozen-frame inspection.
  • Softens E2E guidance so pause points are recommended when transition-frame evidence adds confidence.

User Impact

  • Agents get a clearer, shorter description that frames pause points as an IDE-breakpoint alternative.
  • E2E guidance no longer reads as a mandatory pause for every pass when durable state, logs, or screenshots are sufficient.

Changes

  • Updated the source uloop-wait-for-pause-point skill description and E2E guidance.
  • Refreshed the generated Agents and Claude skill copies.

Verification

  • uv run --with pyyaml python <skill-creator>/scripts/quick_validate.py Packages/src/Editor/CliOnlyTools~/PausePoint/Skill
  • git diff --check
  • cli/dist/darwin-arm64/uloop --project-path "$(git rev-parse --show-toplevel)" skills install --claude --agents

Review in cubic

hatayama added 3 commits June 15, 2026 18:29
Shorten the trigger description so it states the pause-point workflow and use cases without repeating detailed evidence guidance.
Reduce the strength of the E2E pause-point recommendation so the skill suggests using a pause when transition-frame evidence adds confidence.
Apply the updated generated Agents skill description back to the source skill and refresh the Claude copy so installed skills stay consistent.
@coderabbitai

coderabbitai Bot commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: c5ccd562-5694-4177-be13-55aab059a462

📥 Commits

Reviewing files that changed from the base of the PR and between 7dd609d and d0c2f11.

📒 Files selected for processing (3)
  • .agents/skills/uloop-wait-for-pause-point/SKILL.md
  • .claude/skills/uloop-wait-for-pause-point/SKILL.md
  • Packages/src/Editor/CliOnlyTools~/PausePoint/Skill/SKILL.md

📝 Walkthrough

Walkthrough

Three parallel copies of the uloop-wait-for-pause-point SKILL.md file receive identical two-line edits: the top-level description field is reworded from "standard paused-frame proof" framing to "alternative to IDE breakpoints" language, and the E2E "When To Use" bullet is changed from a requirement to pause representative transitions to a conditional "consider" recommendation.

Changes

Pause-point skill documentation wording update

Layer / File(s) Summary
Description and E2E guidance rewording
.agents/skills/uloop-wait-for-pause-point/SKILL.md, .claude/skills/uloop-wait-for-pause-point/SKILL.md, Packages/src/Editor/CliOnlyTools~/PausePoint/Skill/SKILL.md
Line 3 description reworded to frame the skill as an IDE-breakpoint alternative that temporarily pauses Unity and inspects a frozen frame. Line 45 E2E bullet changed from "pause at least one representative transition per E2E pass" to "consider a pause point during E2E passes when transition-frame evidence would add confidence." Applied identically to all three file copies.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • hatayama/unity-cli-loop#1302: Modifies the same pause-point skill documentation in Packages/src/Editor/CliOnlyTools~/PausePoint/Skill/SKILL.md, reworking the "wait for debug-break/pause point" guidance and "When To Use" wording.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/update-skills

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: Clarify pause point skill guidance fix: Clarify pause point skill guidance Jun 15, 2026
@hatayama hatayama merged commit d93c28e into v3-beta Jun 15, 2026
4 of 5 checks passed
@hatayama hatayama deleted the feature/update-skills branch June 15, 2026 09:44
@github-actions github-actions Bot mentioned this pull request Jun 15, 2026
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