[GH-ISSUE #323] Parity: Webhooks relationship endpoint (deliveries) #94

Closed
opened 2026-02-26 21:33:20 +03:00 by kerem · 3 comments
Owner

Originally created by @rudrankriyam on GitHub (Jan 30, 2026).
Original GitHub issue: https://github.com/rudrankriyam/App-Store-Connect-CLI/issues/323

Overview

Expose the remaining webhooks relationship endpoint for deliveries (IDs-only). This is low priority but completes OpenAPI parity for webhooks.

Scope (OpenAPI resources)

  • GET /v1/webhooks/{id}/relationships/deliveries

Proposed CLI

Top-level group:

  • asc webhooks ...

New/expanded subcommands:

  • webhooks deliveries relationships --webhook-id ID (IDs-only)

Examples:

  • asc webhooks deliveries relationships --webhook-id WEBHOOK_ID

Flag patterns

Common:

  • --webhook-id, --output, --pretty

Output

  • JSON minified by default
  • --pretty for JSON

Detailed TODO

  • Add client method for deliveries relationship endpoint
  • Add CLI command under webhooks deliveries for relationships
  • Add cmdtests + HTTP client tests (mocked)

Acceptance criteria

  • Deliveries relationship endpoint accessible via CLI
  • JSON-first output preserved

Tests

  • Flag validation tests
  • HTTP client tests for relationship endpoint

Implementation notes

  • Add client method in internal/asc/client_webhooks.go
  • Add CLI command in internal/cli/webhooks
Originally created by @rudrankriyam on GitHub (Jan 30, 2026). Original GitHub issue: https://github.com/rudrankriyam/App-Store-Connect-CLI/issues/323 # Overview Expose the remaining webhooks relationship endpoint for deliveries (IDs-only). This is low priority but completes OpenAPI parity for webhooks. # Scope (OpenAPI resources) - `GET /v1/webhooks/{id}/relationships/deliveries` # Proposed CLI Top-level group: - `asc webhooks ...` New/expanded subcommands: - `webhooks deliveries relationships --webhook-id ID` (IDs-only) Examples: - `asc webhooks deliveries relationships --webhook-id WEBHOOK_ID` # Flag patterns Common: - `--webhook-id`, `--output`, `--pretty` # Output - JSON minified by default - `--pretty` for JSON # Detailed TODO - [ ] Add client method for deliveries relationship endpoint - [ ] Add CLI command under `webhooks deliveries` for relationships - [ ] Add cmdtests + HTTP client tests (mocked) # Acceptance criteria - [ ] Deliveries relationship endpoint accessible via CLI - [ ] JSON-first output preserved # Tests - Flag validation tests - HTTP client tests for relationship endpoint # Implementation notes - Add client method in `internal/asc/client_webhooks.go` - Add CLI command in `internal/cli/webhooks`
kerem closed this issue 2026-02-26 21:33:21 +03:00
Author
Owner

@rudrankriyam commented on GitHub (Jan 30, 2026):

@cursor Please implement per issue description. No live credentials; use mocked tests. Run make build/lint/test and open a PR when green. Keep JSON-first output and explicit flags; no interactive prompts.

<!-- gh-comment-id:3822603156 --> @rudrankriyam commented on GitHub (Jan 30, 2026): @cursor Please implement per issue description. No live credentials; use mocked tests. Run make build/lint/test and open a PR when green. Keep JSON-first output and explicit flags; no interactive prompts.
Author
Owner

@cursor[bot] commented on GitHub (Jan 30, 2026):

Summary:

  • I added the webhook deliveries relationship client method + linkages response alias and wired a new asc webhooks deliveries relationships --webhook-id ... subcommand (JSON-first output, explicit flags, no prompts), plus updated the webhooks help examples.
  • I added mocked HTTP client coverage for the relationships endpoint and a CLI validation test for missing --webhook-id.

Tests run:

  • make build
  • make lint
  • make test
  • (recorded) go test ./internal/asc -run TestGetWebhookDeliveriesRelationships
  • (recorded) go test ./internal/cli/cmdtest -run TestWebhooksValidationErrors

Walkthrough artifact:

  • Video: /opt/cursor/artifacts/webhook_delivery_relationship_tests/recording_demo.mp4 (shows both focused tests passing).
    I pushed the branch; PR creation is handled by the environment automation here, so I didn’t open one manually.

View PR
Open in Cursor Open in Web

<!-- gh-comment-id:3822603523 --> @cursor[bot] commented on GitHub (Jan 30, 2026): Summary: - I added the webhook deliveries relationship client method + linkages response alias and wired a new `asc webhooks deliveries relationships --webhook-id ...` subcommand (JSON-first output, explicit flags, no prompts), plus updated the webhooks help examples. - I added mocked HTTP client coverage for the relationships endpoint and a CLI validation test for missing `--webhook-id`. Tests run: - `make build` - `make lint` - `make test` - (recorded) `go test ./internal/asc -run TestGetWebhookDeliveriesRelationships` - (recorded) `go test ./internal/cli/cmdtest -run TestWebhooksValidationErrors` Walkthrough artifact: - Video: `/opt/cursor/artifacts/webhook_delivery_relationship_tests/recording_demo.mp4` (shows both focused tests passing). I pushed the branch; PR creation is handled by the environment automation here, so I didn’t open one manually. <a href="https://github.com/rudrankriyam/App-Store-Connect-CLI/pull/331"><picture><source media="(prefers-color-scheme: dark)" srcset="https://cursor.com/assets/images/view-pr-dark.png"><source media="(prefers-color-scheme: light)" srcset="https://cursor.com/assets/images/view-pr-light.png"><img alt="View PR" width="86" height="28" src="https://cursor.com/assets/images/view-pr-light.png"></picture></a> <a href="https://cursor.com/background-agent?bcId=bc-12ed379f-e66b-4a60-b176-515cc0229c2b"><picture><source media="(prefers-color-scheme: dark)" srcset="https://cursor.com/assets/images/open-in-cursor-dark.png"><source media="(prefers-color-scheme: light)" srcset="https://cursor.com/assets/images/open-in-cursor-light.png"><img alt="Open in Cursor" width="131" height="28" src="https://cursor.com/assets/images/open-in-cursor-dark.png"></picture></a>&nbsp;<a href="https://cursor.com/agents?id=bc-12ed379f-e66b-4a60-b176-515cc0229c2b"><picture><source media="(prefers-color-scheme: dark)" srcset="https://cursor.com/assets/images/open-in-web-dark.png"><source media="(prefers-color-scheme: light)" srcset="https://cursor.com/assets/images/open-in-web-light.png"><img alt="Open in Web" width="114" height="28" src="https://cursor.com/assets/images/open-in-web-dark.png"></picture></a>
Author
Owner

@rudrankriyam commented on GitHub (Jan 30, 2026):

Fixed by #331 (deliveries relationships) and validation fixes in #334. Included in release 0.20.2.

<!-- gh-comment-id:3825187748 --> @rudrankriyam commented on GitHub (Jan 30, 2026): Fixed by #331 (deliveries relationships) and validation fixes in #334. Included in release 0.20.2.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/App-Store-Connect-CLI#94
No description provided.