Skip to content

feat: add MCP tool hint annotations to particle system tools#5

Merged
IvanMurzak merged 1 commit intomainfrom
claude/issue-3-20260227-0659
Feb 27, 2026
Merged

feat: add MCP tool hint annotations to particle system tools#5
IvanMurzak merged 1 commit intomainfrom
claude/issue-3-20260227-0659

Conversation

@IvanMurzak
Copy link
Copy Markdown
Owner

Adds MCP tool hint annotations to the two particle system tools as requested in issue #3.

particle-system-get: ReadOnlyHint=true, IdempotentHint=true
particle-system-modify: IdempotentHint=true

Closes #3

Generated with Claude Code

- particle-system-get: ReadOnlyHint=true, IdempotentHint=true
  (only reads/queries data, same arguments always yield same result)
- particle-system-modify: IdempotentHint=true
  (applying same values repeatedly has no additional effect beyond first call)

Co-authored-by: Ivan Murzak <IvanMurzak@users.noreply.github.com>
@IvanMurzak IvanMurzak self-assigned this Feb 27, 2026
Copilot AI review requested due to automatic review settings February 27, 2026 07:02
@IvanMurzak IvanMurzak added the enhancement New feature or request label Feb 27, 2026
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds MCP tool hint annotations to the two particle system tools in response to issue #3. The annotations indicate behavioral properties of the tools to the MCP SDK, enabling better tool categorization and usage.

Changes:

  • Added ReadOnlyHint=true and IdempotentHint=true to particle-system-get tool
  • Added IdempotentHint=true to particle-system-modify tool

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
Unity-Package/Assets/root/Editor/Scripts/Tools/ParticleSystem.Get.cs Added ReadOnlyHint=true and IdempotentHint=true annotations to indicate the tool only reads data and produces consistent results
Unity-Package/Assets/root/Editor/Scripts/Tools/ParticleSystem.Modify.cs Added IdempotentHint=true annotation to indicate multiple calls with same arguments produce the same final state

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@IvanMurzak IvanMurzak merged commit 535e5c4 into main Feb 27, 2026
36 checks passed
@IvanMurzak IvanMurzak deleted the claude/issue-3-20260227-0659 branch February 27, 2026 07:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

MCP Tool annotations adding

2 participants