Skip to content

chore(tools): register csharp write tools and docs - #42

Merged
akiojin merged 1 commit into
developfrom
feature/issue-40
Feb 26, 2026
Merged

chore(tools): register csharp write tools and docs#42
akiojin merged 1 commit into
developfrom
feature/issue-40

Conversation

@akiojin

@akiojin akiojin commented Feb 26, 2026

Copy link
Copy Markdown
Owner

Summary

  • Registered newly added C# LSP write tools in the CLI tool catalog so runtime lookup and manifest parity checks include them.
  • Expanded the unity-csharp-edit skill guide with concrete command examples so contributors can use the new write/edit flow correctly.

Changes

  • src/tool_catalog.rs: Added rename_symbol, replace_symbol_body, insert_before_symbol, insert_after_symbol, remove_symbol, validate_text_edits, and create_class to TOOL_NAMES and updated the parity-count test from 101 to 108.
  • .claude-plugin/plugins/unity-cli/skills/unity-csharp-edit/SKILL.md: Added LSP write-tool usage examples, apply mode guidance, and namePath navigation notes.

Testing

  • cargo test tool_catalog::tests::tool_catalog_keeps_manifest_parity_count -- --nocapture — parity test passes with updated tool count.

Related Issues / Links

Checklist

  • Tests added/updated
  • Lint/format passed (cargo clippy, cargo fmt, svelte-check) — cargo clippy/cargo fmt were not rerun for this docs+catalog commit.
  • Documentation updated (if user-facing change) — N/A: this PR itself is tool-catalog/skill documentation alignment.
  • Migration/backfill plan included (if schema/data change) — N/A: no schema/data changes.
  • CHANGELOG impact considered (breaking change flagged in commit)

@coderabbitai

coderabbitai Bot commented Feb 26, 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.

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/issue-40

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


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 merged commit 63683c6 into develop Feb 26, 2026
12 checks passed
@akiojin
akiojin deleted the feature/issue-40 branch February 26, 2026 01:12
akiojin added a commit that referenced this pull request Mar 3, 2026
chore(tools): register csharp write tools and docs
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