Skip to content

chore: Stop CodeRabbit from rewriting pull request descriptions#1145

Merged
hatayama merged 2 commits into
v3-betafrom
feature/hatayama/coderabbit-summary-skills
May 17, 2026
Merged

chore: Stop CodeRabbit from rewriting pull request descriptions#1145
hatayama merged 2 commits into
v3-betafrom
feature/hatayama/coderabbit-summary-skills

Conversation

@hatayama
Copy link
Copy Markdown
Owner

@hatayama hatayama commented May 17, 2026

Summary

  • CodeRabbit no longer generates high-level PR summary text that rewrites pull request descriptions.
  • Local skill copies now carry updated tool metadata and current command guidance.

User Impact

  • PR authors can keep their PR descriptions intact while CodeRabbit still performs review comments.
  • Agents using the local skill files get current uloop compile examples and tool metadata.

Changes

  • Disable CodeRabbit high-level summary generation in .coderabbit.yaml.
  • Refresh generated .agents and .claude skill markdown copies with toolName metadata and updated compile/recovery wording.

Verification

  • ruby -e 'require "yaml"; YAML.load_file(".coderabbit.yaml"); puts "ok"'
  • git diff --check origin/v3-beta...HEAD

Configuration Changes

CodeRabbit Review Settings

Disabled high-level summary generation in .coderabbit.yaml by setting reviews.high_level_summary to false, preventing CodeRabbit from rewriting pull request descriptions while maintaining review comment functionality.

Skill Documentation Updates

Metadata Enrichment

Added explicit toolName metadata fields across all 12 skill files in both .agents/skills/ and .claude/skills/ directories:

  • uloop-clear-consoletoolName: clear-console
  • uloop-find-game-objectstoolName: find-game-objects
  • uloop-get-hierarchytoolName: get-hierarchy
  • uloop-get-logstoolName: get-logs
  • uloop-record-inputtoolName: record-input
  • uloop-replay-inputtoolName: replay-input
  • uloop-run-teststoolName: run-tests
  • uloop-screenshottoolName: screenshot
  • uloop-simulate-keyboardtoolName: simulate-keyboard
  • uloop-simulate-mouse-inputtoolName: simulate-mouse-input
  • uloop-simulate-mouse-uitoolName: simulate-mouse-ui

Command Contract Updates (uloop-compile)

Updated both .agents/skills/uloop-compile/SKILL.md and .claude/skills/uloop-compile/SKILL.md:

  • Replaced CLI flag from --wait-for-domain-reload to --no-wait-for-domain-reload in command usage signature and parameters table
  • Added example demonstrating compilation without waiting for domain reload completion
  • Updated troubleshooting section from "Stale lock files" to "Stale recovery state" with refreshed description text
  • uloop fix recovery command remains consistent across updated wording

Impact

  • PR authors' original descriptions are preserved during code review
  • Local skill files now provide accurate toolName metadata for agent routing
  • Updated compile command guidance reflects current CLI contract and failure-mode handling

Review Change Stack

hatayama added 2 commits May 17, 2026 09:34
Prevent CodeRabbit from generating PR summary text so it does not rewrite pull request descriptions.
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 17, 2026

📝 Walkthrough

Walkthrough

This PR adds toolName metadata fields to all 24 skill documentation files (12 agent, 12 Claude), updates the uloop compile CLI contract to use --no-wait-for-domain-reload instead of --wait-for-domain-reload with corresponding examples and troubleshooting updates, and disables CodeRabbit high-level summary output.

Changes

Skill Tooling and Documentation Updates

Layer / File(s) Summary
Agent skill toolName metadata alignment
.agents/skills/uloop-{clear-console,find-game-objects,get-hierarchy,get-logs,record-input,replay-input,run-tests,screenshot,simulate-keyboard,simulate-mouse-input,simulate-mouse-ui,compile}/SKILL.md
toolName field added to 12 agent skill documentation frontmatter, establishing tool routing identifiers that match command names for consistency across the agent skill ecosystem.
Claude skill toolName metadata alignment
.claude/skills/uloop-{clear-console,find-game-objects,get-hierarchy,get-logs,record-input,replay-input,run-tests,screenshot,simulate-keyboard,simulate-mouse-input,simulate-mouse-ui,compile}/SKILL.md
toolName field added to 12 Claude skill documentation frontmatter, mirroring agent skill changes to maintain parallel tool routing identifiers.
uloop-compile CLI contract revision
.agents/skills/uloop-compile/SKILL.md, .claude/skills/uloop-compile/SKILL.md
CLI flag changed from --wait-for-domain-reload to --no-wait-for-domain-reload in both agent and Claude versions. Usage and parameters tables updated; examples adjusted to show the new flag behavior. Troubleshooting section renamed from "Stale lock files" to "Stale recovery state" with updated description while retaining the uloop fix recovery command.
CodeRabbit configuration update
.coderabbit.yaml
reviews.high_level_summary setting toggled from true to false, disabling high-level summary output for reviews.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

  • hatayama/unity-cli-loop#1031: Earlier update to agents/skills/uloop-compile/SKILL.md addressing domain-reload waiting guidance in the compile command.
  • hatayama/unity-cli-loop#1028: Introduces toolName matching and internal skill/tool filtering logic that consumes the toolName metadata fields added in this PR.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
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 accurately describes the main change: disabling CodeRabbit's high-level summary generation to prevent overwriting PR descriptions, which aligns with the primary objective.

✏️ 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/coderabbit-summary-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.

Copy link
Copy Markdown

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

Choose a reason for hiding this comment

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

No issues found across 25 files

Re-trigger cubic

@hatayama hatayama changed the title fix: CodeRabbit no longer rewrites pull request descriptions chore: Stop CodeRabbit from rewriting pull request descriptions May 17, 2026
@hatayama hatayama merged commit 4e61787 into v3-beta May 17, 2026
7 checks passed
@hatayama hatayama deleted the feature/hatayama/coderabbit-summary-skills branch May 17, 2026 00:59
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