Skip to content

feat(skills): align unity skills with Anthropic guidance - #111

Merged
akiojin merged 1 commit into
developfrom
feature/update-skills
Mar 11, 2026
Merged

feat(skills): align unity skills with Anthropic guidance#111
akiojin merged 1 commit into
developfrom
feature/update-skills

Conversation

@akiojin

@akiojin akiojin commented Mar 10, 2026

Copy link
Copy Markdown
Owner

Summary

  • Rewrite the 14 unity-cli skill definitions to add explicit triggers, scoped workflows, and troubleshooting guidance so Claude can route Unity tasks more reliably.
  • Add per-skill reference playbooks under references/ so reusable safety and execution details stay out of the trigger-focused SKILL.md bodies.
  • Update the README and plugin metadata to describe the skills as workflow-oriented automation so the published docs match the revised skill behavior.

Changes

  • .claude-plugin/plugins/unity-cli/skills/*/SKILL.md: expanded frontmatter descriptions, added metadata.version, and standardized Use When, Do Not Use When, Recommended Flow, Examples, and Common Issues sections.
  • .claude-plugin/plugins/unity-cli/skills/*/references/*.md: added 14 reference playbooks covering runtime checks, editor operations, code search/edit safety, scene and prefab workflows, asset/input/test loops, UI automation, and gh skill sync behavior.
  • README.md, .claude-plugin/plugins/unity-cli/plugin.json, .claude-plugin/marketplace.json: updated public descriptions and the skill inventory to match the workflow-guide positioning.

Testing

  • ./scripts/skill-eval/static-skill-contract-check.sh --report /tmp/skill-static-report.json --json - exits with "passed": true and "errorsCount": 0.
  • python3 - <<"PY" ... PY - parsed every skill frontmatter as YAML and confirmed all referenced references/*.md files exist.
  • git diff --check - reports no whitespace or merge-marker issues.

Related Issues / Links

Checklist

  • Tests added/updated - N/A: no automated test files changed; validation is covered by the existing static skill checks.
  • Lint/format passed (cargo clippy, cargo fmt, svelte-check) - N/A: this PR only changes skill/docs metadata and does not touch Rust or Svelte sources.
  • Documentation updated (if user-facing change)
  • Migration/backfill plan included (if schema/data change) - N/A: no schema or data migration is involved.
  • CHANGELOG impact considered (breaking change flagged in commit)

Context

  • This PR aligns the unity-cli skill pack with the published Anthropic skills guidance by tightening trigger metadata, adopting progressive disclosure with per-skill references, and making each skill describe a concrete workflow instead of a flat command list.

@coderabbitai

coderabbitai Bot commented Mar 10, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 4e739325-fc03-4be0-ad9e-959eee5c143c

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • 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.

@akiojin
akiojin force-pushed the feature/update-skills branch from 0ec2d70 to c2902f3 Compare March 10, 2026 15:03
@akiojin

akiojin commented Mar 10, 2026

Copy link
Copy Markdown
Owner Author

Fixed items

  • Resolved the develop merge conflict by updating the branch to the latest base and reconciling the README changes so the skills update and managed binary maintenance docs both remain.
  • Rewrote the branch history to a single Conventional Commits message (docs(skills): align unity skills with Anthropic guidance) so Markdown & Commitlint passes.
  • Re-ran the local skill contract validation before pushing the refreshed branch.

Validation

@akiojin
akiojin merged commit 4bcd55c into develop Mar 11, 2026
17 checks passed
@akiojin
akiojin deleted the feature/update-skills branch March 11, 2026 00:04
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