Repository navigation
feat(tools): add create_community and create_domain tools - #160
Closed
agliznetsov wants to merge 1 commit into
Closed
agliznetsov wants to merge 1 commit into
agliznetsov wants to merge 1 commit into
Conversation
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 write tools,
create_communityandcreate_domain(DEV-217560).create_community:name, optionalparentCommunity(UUID or name), optionaldescription,confirm.create_domain:name,community(UUID or name),domainType(UUID or name), optionaldescription,confirm.docs/TOOL_CONTRIBUTION_STANDARDS.mdsection 5:confirm=false(default) returns a preview of every field to be written and sends no write;confirm=truecreates.resolve.Community/resolve.DomainType. An unknown or ambiguous name returns avalidation_errorlisting the valid options.AddCommunityRequest/AddDomainRequest.POST /rest/2.0/communitiesandPOST /rest/2.0/domainsthroughexecuteCollibraRequest, so Collibra's error message reaches the caller.create_asset.Tested against a local Collibra instance: preview writes nothing, confirm creates, nested communities, name and UUID resolution, invalid domain type, and duplicate community name.
Impact Analysis
Additive change: two new tools in the default tool surface, one new shared resolver file, and two new client files. Existing tools are unchanged. Both tools need Collibra permission to create communities or domains; without it the write returns an
errorstatus.Checklist
🤖 Generated with Claude Code