Skip to content

fix: Simplify compile skill guidance#1206

Merged
hatayama merged 1 commit into
v3-betafrom
feature/hatayama/improve-compile-recovery-guidance
May 25, 2026
Merged

fix: Simplify compile skill guidance#1206
hatayama merged 1 commit into
v3-betafrom
feature/hatayama/improve-compile-recovery-guidance

Conversation

@hatayama

Copy link
Copy Markdown
Owner

Summary

  • Simplifies the compile skill guidance so agents see only the commands and output fields they need.
  • Clarifies that --force-recompile is a slower, broader validation path for cases such as warnings hidden by other asmdefs.

User Impact

  • Agents are no longer steered toward stale recovery or Unity restart steps from the compile skill.
  • The force-recompile option is described as a broader validation tool without making Domain Reload behavior the focus.

Changes

  • Trimmed the source compile skill.
  • Regenerated the Claude and Agents project skill copies.

Verification

  • git diff --check origin/v3-beta...HEAD
  • rg -n "Use for broader validation, including warnings hidden by other asmdefs; much slower than normal compile|Use to find warnings hidden|Force full recompilation|triggers Domain Reload|full Domain Reload compile|Troubleshooting|uloop launch|uloop fix" Packages/src/Editor/FirstPartyTools/Compile/Skill/SKILL.md .agents/skills/uloop-compile/SKILL.md .claude/skills/uloop-compile/SKILL.md
  • Unity compile not run because this change only edits Markdown skill files.

Remove recovery, restart, and redundant Domain Reload wording from the compile skill, and describe force recompile as the slower path for broader validation, including warnings hidden by other asmdefs.
@coderabbitai

coderabbitai Bot commented May 25, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

This PR updates the uloop-compile skill documentation across three locations (.agents/, .claude/, and Packages/) with identical changes: narrowing the usage description to "Use after C# edits," clarifying that --force-recompile performs broader validation and is slower, and removing an example in favor of a simpler no-wait variant.

Changes

uloop-compile Documentation Updates

Layer / File(s) Summary
Usage description and flag clarifications
.agents/skills/uloop-compile/SKILL.md, .claude/skills/uloop-compile/SKILL.md, Packages/src/Editor/FirstPartyTools/Compile/Skill/SKILL.md
Skill description narrowed to "Use after C# edits"; --force-recompile flag documentation updated to clarify broader validation behavior (including hidden warnings) and slower performance.
Examples refactoring and section formatting
.agents/skills/uloop-compile/SKILL.md, .claude/skills/uloop-compile/SKILL.md, Packages/src/Editor/FirstPartyTools/Compile/Skill/SKILL.md
Removed force-recompile-with-wait example; kept no-wait-for-Domain-Reload example; adjusted spacing around JSON output and Troubleshooting section transition.

Possibly Related PRs


🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description check ✅ Passed The description is well-related to the changeset, explaining the rationale for simplifying guidance, user impact, and specific changes made to documentation across three files.
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.
Title check ✅ Passed The title 'fix: Simplify compile skill guidance' accurately summarizes the main change: simplifying documentation guidance for the compile skill to focus on essential commands and clarify the --force-recompile option.

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

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/hatayama/improve-compile-recovery-guidance

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: Simplify compile skill guidance fix: Simplify compile skill guidance May 25, 2026

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 3 files

Re-trigger cubic

@hatayama
hatayama merged commit 71b080e into v3-beta May 25, 2026
5 checks passed
@hatayama
hatayama deleted the feature/hatayama/improve-compile-recovery-guidance branch May 25, 2026 14:41
@github-actions github-actions Bot mentioned this pull request May 25, 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