feat(cli): add fullsend agent subcommand (ADR 0058 Phase 2) - #3
Closed
ggallen wants to merge 2 commits into
Closed
Conversation
Implements Phase 1 of ADR 0058 (agent registration). Adds AgentEntry type with custom YAML unmarshaler supporting both string shorthand and object form, validation for URL integrity hashes, allowlist membership, path traversal, and name uniqueness. Seeds default AllowedRemoteResources in NewPerRepoConfig. Legacy role/name/slug agent entries are now rejected with a clear error instead of silently ignored. Signed-off-by: Greg Allen <gallen@redhat.com> Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Signed-off-by: Greg Allen <gallen@redhat.com>
…ions Add `fullsend agent` CLI with add/list/update/remove subcommands for managing agent entries in org and per-repo config (ADR 0058 Phase 2). - `agent add <url-or-path>` pins URLs to a commit SHA with integrity hash and auto-updates allowed_remote_resources - `agent list` displays registered agents with their sources - `agent update <name> [sha]` re-pins a URL agent to a new commit - `agent remove <name>` removes an agent and cleans up unused allowlist prefixes Also adds GetBranchRef to the forge.Client interface for resolving branch HEAD SHAs, used by add and update for commit pinning. Signed-off-by: Claude Opus 4.6 <noreply@anthropic.com> Signed-off-by: Greg Allen <gallen@redhat.com>
ggallen
force-pushed
the
agent-registration-config-schema
branch
from
June 30, 2026 03:28
00fca1e to
4c09de0
Compare
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
1 participant
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.
Summary
fullsend agentCLI subcommand withadd,list,update,removefor managing agent registrations in config (ADR 0058 Phase 2)agent addpins URLs to a commit SHA with integrity hash and auto-updatesallowed_remote_resourcesagent updatere-pins a URL agent to a new commit SHA (explicit or default branch HEAD)agent removecleans up unused allowlist prefixes when the last agent using a prefix is removedGetBranchReftoforge.Clientinterface for resolving branch HEAD SHAsTest plan
internal/clitests passinternal/forgetests passgo build ./...succeedsmake go-testpasses (all packages)make go-vetclean--nameflag, duplicate rejection (case-insensitive), path traversal rejection, list empty/populated/hash-stripped, update re-pin with default branch, update with explicit SHA, update rejects local path, update rejects invalid SHA, remove with allowlist cleanup, remove preserves allowlist when other agents share prefix, per-repo config support, helper function unit tests🤖 Generated with Claude Code