Repository navigation
feat: add edit_domain and edit_community tools - #161
Closed
agliznetsov wants to merge 2 commits into
Closed
agliznetsov wants to merge 2 commits into
agliznetsov wants to merge 2 commits into
Conversation
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description of your changes
Adds two MCP tools,
edit_domainandedit_community, following theedit_assetpattern: a single tool per resource type taking a typed list of operations. Each operation is validated and applied independently, and the result issuccess,partial_successorerror.edit_domain:set_name,set_description,set_community,set_responsibility,remove_responsibility.edit_community:set_name,set_description,set_parent,set_responsibility,remove_responsibility.set_parentrequires exactly one ofparentormakeTopLevel: true.set_parentandset_communitycheck the PATCH response rather than the HTTP status, so a silent no-op is reported as an error (see DEV-217539, where clearing a community's parent returns 200 without changing anything)./rest/2.0/responsibilitiesendpoints with resource typeDomain/Community. The shared logic is in a newpkg/tools/responsibilitypackage.Jira: DEV-217538
The tools are registered ungated next to
edit_asset, which differs from §3.1 ofdocs/TOOL_CONTRIBUTION_STANDARDS.md(new write tools behind an experimental flag). Reviewers should decide whether to gate them together with the create tools from DEV-217560.Impact Analysis
pkg/tools/register.goand a README update.golangci-lintpass.Checklist
🤖 Generated with Claude Code