[GH-ISSUE #319] Parity: Review submissions and submission items #93

Closed
opened 2026-02-26 21:33:19 +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/319

Overview

Expose remaining review submission endpoints: submission item CRUD and submission updates, plus relationship-only endpoints where useful. Keep JSON-first output and explicit flags.

Scope (OpenAPI resources)

  • reviewSubmissionItems GET/PATCH by ID
  • reviewSubmissions PATCH by ID
  • Relationship endpoints for submission items (IDs-only)

Proposed CLI

Top-level group:

  • asc review ...

New/expanded subcommands:

  • review items get --id ITEM_ID
  • review items update --id ITEM_ID --state READY_FOR_REVIEW|IN_REVIEW|...
  • review submissions update --id SUBMISSION_ID --canceled true|false
  • review submissions items-ids --id SUBMISSION_ID (optional IDs-only)

Examples:

  • asc review items get --id ITEM_ID
  • asc review items update --id ITEM_ID --state READY_FOR_REVIEW
  • asc review submissions update --id SUBMISSION_ID --canceled true

Flag patterns

Common:

  • --id, --output, --pretty, --confirm

Resource-specific:

  • --state, --canceled

Output

  • JSON minified by default
  • --pretty for JSON
  • --output table/markdown for list commands

Detailed TODO

  • Add client method for GET /v1/reviewSubmissionItems/{id}
  • Add client method for PATCH /v1/reviewSubmissionItems/{id}
  • Expose existing submission update method via CLI
  • (Optional) Add relationship IDs endpoint for submission items
  • Implement CLI commands for items get/update
  • Implement CLI command for submission update
  • Add cmdtests + HTTP client tests (mocked)

Acceptance criteria

  • asc review items get returns item details
  • asc review items update updates state
  • asc review submissions update updates submission attributes
  • JSON-first output preserved
  • Validation errors are clear and consistent

Tests

  • Flag validation tests
  • HTTP client tests for get/update
  • Output tests for json/table/markdown where applicable

Implementation notes

  • Add/update methods in internal/asc/client_reviews.go
  • Add CLI commands in internal/cli/reviews
  • Follow patterns in existing review submissions commands
Originally created by @rudrankriyam on GitHub (Jan 30, 2026). Original GitHub issue: https://github.com/rudrankriyam/App-Store-Connect-CLI/issues/319 # Overview Expose remaining review submission endpoints: submission item CRUD and submission updates, plus relationship-only endpoints where useful. Keep JSON-first output and explicit flags. # Scope (OpenAPI resources) - `reviewSubmissionItems` GET/PATCH by ID - `reviewSubmissions` PATCH by ID - Relationship endpoints for submission items (IDs-only) # Proposed CLI Top-level group: - `asc review ...` New/expanded subcommands: - `review items get --id ITEM_ID` - `review items update --id ITEM_ID --state READY_FOR_REVIEW|IN_REVIEW|...` - `review submissions update --id SUBMISSION_ID --canceled true|false` - `review submissions items-ids --id SUBMISSION_ID` (optional IDs-only) Examples: - `asc review items get --id ITEM_ID` - `asc review items update --id ITEM_ID --state READY_FOR_REVIEW` - `asc review submissions update --id SUBMISSION_ID --canceled true` # Flag patterns Common: - `--id`, `--output`, `--pretty`, `--confirm` Resource-specific: - `--state`, `--canceled` # Output - JSON minified by default - `--pretty` for JSON - `--output table/markdown` for list commands # Detailed TODO - [ ] Add client method for `GET /v1/reviewSubmissionItems/{id}` - [ ] Add client method for `PATCH /v1/reviewSubmissionItems/{id}` - [ ] Expose existing submission update method via CLI - [ ] (Optional) Add relationship IDs endpoint for submission items - [ ] Implement CLI commands for items get/update - [ ] Implement CLI command for submission update - [ ] Add cmdtests + HTTP client tests (mocked) # Acceptance criteria - [ ] `asc review items get` returns item details - [ ] `asc review items update` updates state - [ ] `asc review submissions update` updates submission attributes - [ ] JSON-first output preserved - [ ] Validation errors are clear and consistent # Tests - Flag validation tests - HTTP client tests for get/update - Output tests for json/table/markdown where applicable # Implementation notes - Add/update methods in `internal/asc/client_reviews.go` - Add CLI commands in `internal/cli/reviews` - Follow patterns in existing `review submissions` commands
kerem closed this issue 2026-02-26 21:33:19 +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:3822603172 --> @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 client support for review submission item GET/PATCH and submission item relationship IDs, plus new asc review commands for items-get, items-update --state, submissions-update --canceled, and submissions-items-ids, and updated help/examples accordingly.
  • I added mocked tests for the new client methods, CLI flag validation for the new commands, and table/markdown output coverage for review submissions/items.

Tests run (all green): make build, make lint, make test.

I pushed the changes to cursor/review-items-and-submissions-14ff. I can’t open a PR directly in this environment, but GitHub provided this link to open one: https://github.com/rudrankriyam/App-Store-Connect-CLI/pull/new/cursor/review-items-and-submissions-14ff.

View PR
Open in Cursor Open in Web

<!-- gh-comment-id:3822603531 --> @cursor[bot] commented on GitHub (Jan 30, 2026): Summary: - I added client support for review submission item GET/PATCH and submission item relationship IDs, plus new `asc review` commands for `items-get`, `items-update --state`, `submissions-update --canceled`, and `submissions-items-ids`, and updated help/examples accordingly. - I added mocked tests for the new client methods, CLI flag validation for the new commands, and table/markdown output coverage for review submissions/items. Tests run (all green): `make build`, `make lint`, `make test`. I pushed the changes to `cursor/review-items-and-submissions-14ff`. I can’t open a PR directly in this environment, but GitHub provided this link to open one: https://github.com/rudrankriyam/App-Store-Connect-CLI/pull/new/cursor/review-items-and-submissions-14ff. <a href="https://github.com/rudrankriyam/App-Store-Connect-CLI/pull/333"><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-22229d67-0639-4c24-87d7-474a54e2ec8d"><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-22229d67-0639-4c24-87d7-474a54e2ec8d"><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 #333. Included in release 0.20.2.

<!-- gh-comment-id:3825188211 --> @rudrankriyam commented on GitHub (Jan 30, 2026): Fixed by #333. 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#93
No description provided.