fix: pre-populate registry URL in add-skill, add-mcp, and TUI wizard#290
Merged
peterj merged 2 commits intoagentregistry-dev:mainfrom Mar 7, 2026
Merged
Conversation
When --registry-url is omitted, default to the current registry URL (from ARCTL_API_BASE_URL or the running daemon) instead of requiring the user to type it every time. This matches the behavior already implemented in add-prompt. Also pre-populates the TUI wizard's registry URL input field so users can just press Enter to accept the default. Fixes agentregistry-dev#145 Co-Authored-By: Claude Opus 4.6 <[email protected]>
peterj
reviewed
Mar 6, 2026
peterj
reviewed
Mar 6, 2026
peterj
requested changes
Mar 6, 2026
Contributor
peterj
left a comment
peterj
left a comment
There was a problem hiding this comment.
added a couple of comments
Address review feedback: - Remove unnecessary if/else in chooseRegistryType - just set the value directly since empty string is fine - Restore //nolint:nestif directive that was removed by mistake Co-Authored-By: Claude Opus 4.6 <[email protected]>
|
You already have 3 pull requests open. Please consider working on getting the existing ones merged before opening new ones. Thanks! |
1 similar comment
|
You already have 3 pull requests open. Please consider working on getting the existing ones merged before opening new ones. Thanks! |
github-actions
bot
added
release-note
and removed
do-not-merge/release-note-invalid
labels
|
You already have 3 pull requests open. Please consider working on getting the existing ones merged before opening new ones. Thanks! |
peterj
approved these changes
Mar 6, 2026
github-merge-queue
bot
removed this pull request from the merge queue due to no response for status checks
christian-posta
pushed a commit
to christian-posta/agentregistry
that referenced
this pull request
Mar 9, 2026
…gentregistry-dev#290) # Description Fixes agentregistry-dev#145 When using `arctl agent add-mcp` or `arctl agent add-skill` with a registry MCP server, the user has to manually type the registry URL even though there's already a configured default. This pre-populates the registry URL field in both the TUI wizard and CLI flags. ## Key changes - TUI wizard's `chooseRegistryType()` pre-populates the registry URL input with `GetDefaultRegistryURL()` - CLI `add-mcp` falls back to default registry URL when `--registry-url` is not provided but `--registry-server-name` is - New `agentutils.GetDefaultRegistryURL()` helper reads from config # Change Type /kind fix # Changelog ```release-note Pre-populate registry URL in add-skill, add-mcp, and TUI wizard from configured default ``` # Additional Notes None. --------- Co-authored-by: Claude Opus 4.6 <[email protected]>
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
2 participants
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
Fixes #145
When using
arctl agent add-mcporarctl agent add-skillwith a registry MCP server, the user has to manually type the registry URL even though there's already a configured default. This pre-populates the registry URL field in both the TUI wizard and CLI flags.Key changes
chooseRegistryType()pre-populates the registry URL input withGetDefaultRegistryURL()add-mcpfalls back to default registry URL when--registry-urlis not provided but--registry-server-nameisagentutils.GetDefaultRegistryURL()helper reads from configChange Type
/kind fix
Changelog
Additional Notes
None.