chore(agentctl): remove ApprovalPolicy field that never governed approval (#3209)

Co-authored-by: nova28 <17953305+nova28@users.noreply.github.com>
Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>
Co-authored-by: Carlos Florencio <carlosmflorencio@gmail.com>
This commit is contained in:
Henry
2026-08-31 21:05:02 +00:00
committed by GitHub
parent 93428921bb
commit 6271fc37ed
3 changed files with 0 additions and 16 deletions
@@ -259,11 +259,6 @@ type Config struct {
// AutoApprove automatically approves permission requests
AutoApprove bool
// ApprovalPolicy controls when the agent requests approval.
// Valid values: "untrusted" (always), "on-failure", "on-request", "never".
// Defaults to "on-request" if empty.
ApprovalPolicy string
// McpServers is a list of MCP servers to configure for the agent
McpServers []McpServerConfig
@@ -323,7 +318,6 @@ func (c *Config) ToSharedConfig() *shared.Config {
return &shared.Config{
WorkDir: c.WorkDir,
AutoApprove: c.AutoApprove,
ApprovalPolicy: c.ApprovalPolicy,
McpServers: mcpServers,
AgentID: c.AgentID,
AgentName: c.AgentName,
@@ -14,15 +14,6 @@ type Config struct {
// AutoApprove automatically approves permission requests
AutoApprove bool
// ApprovalPolicy controls when the agent requests approval.
// Valid values: "untrusted" (always), "on-failure", "on-request", "never".
// Defaults to "on-request" if empty.
ApprovalPolicy string
// PermissionPolicy controls the permission mode: "autonomous", "supervised", "plan".
// Used to determine hook registration and --permission-mode flag.
PermissionPolicy string
// PermissionTimeout is the maximum time to wait for a permission response.
// After timeout, the request is auto-denied with interrupt. Defaults to DefaultPermissionTimeout.
PermissionTimeout time.Duration
@@ -1340,7 +1340,6 @@ func (m *Manager) buildAdapterConfig() error {
m.adapterCfg = &adapter.Config{
WorkDir: m.cfg.WorkDir,
AutoApprove: m.cfg.AutoApprovePermissions,
ApprovalPolicy: m.cfg.ApprovalPolicy,
McpServers: mcpServers,
AgentID: m.cfg.AgentType, // From registry (e.g., "auggie", "amp", "claude-code")
AssumeMcpSse: m.cfg.AssumeMcpSse,