-
Notifications
You must be signed in to change notification settings - Fork 16
Closed
Labels
automation/auto-implementIssue auto-implement: auto-implementIssue auto-implement: auto-implement
Description
Summary
hookdeck connection pause and hookdeck connection unpause (and the equivalent hookdeck gateway connection … commands) currently take a positional argument documented as connection ID only. Users who work from hookdeck connection list often have the name handy; connection get already accepts name or ID via resolveConnectionID in pkg/cmd/connection_get.go, but pause/unpause pass the argument straight to the API.
Expected behavior
- Accept the same name-or-ID resolution as
connection get: try ID (includingconn_/web_prefix path), then list/filter by name; surface the existing ambiguous-name error when multiple connections share a name.
Suggested implementation
- Reuse
resolveConnectionIDbeforePauseConnection/UnpauseConnectioninpkg/cmd/connection_pause.goandpkg/cmd/connection_unpause.go. - Update
Use,Long, andcli.argumentsannotations to say name or ID (mirrorconnection_get.go). - MCP:
hookdeck_connectionspause/unpause currently require anidfield; extend tool handling (andtools.go/tool_help.godescriptions) so agents can pass a connection name as well—same resolution as the CLI.
References
resolveConnectionID:pkg/cmd/connection_get.go(approx. lines 180–211)- Pause:
pkg/cmd/connection_pause.go - Unpause:
pkg/cmd/connection_unpause.go - MCP:
pkg/gateway/mcp/tool_connections.go(connectionsPause/connectionsUnpause)
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
automation/auto-implementIssue auto-implement: auto-implementIssue auto-implement: auto-implement