[PR #539] [MERGED] Improve root help UX and media asset wording #603

Closed
opened 2026-02-26 22:31:43 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/rudrankriyam/App-Store-Connect-CLI/pull/539
Author: @rudrankriyam
Created: 2/14/2026
Status: Merged
Merged: 2/14/2026
Merged by: @rudrankriyam

Base: mainHead: feat/root-help-grouping


📝 Commits (2)

  • dd61a03 feat(cli): group root help and clarify media wording
  • ee1126e Handle -- separator in positional arg detection (#540)

📊 Changes

9 files changed (+302 additions, -10 deletions)

View changed files

📝 ASC.md (+1 -1)
📝 cmd/root.go (+3 -3)
cmd/root_usage.go (+203 -0)
📝 cmd/run.go (+29 -0)
📝 cmd/run_additional_test.go (+52 -0)
📝 internal/cli/assets/assets.go (+2 -2)
📝 internal/cli/assets/assets_previews.go (+2 -2)
📝 internal/cli/assets/assets_test.go (+9 -1)
📝 internal/cli/docs/templates/ASC.md (+1 -1)

📄 Description

Summary

  • group root asc --help commands into gh-style sections with command: alignment for faster scanning
  • make bare asc print root help and exit successfully instead of returning a usage error
  • clarify asset/previews wording to explicitly call out app preview videos in command help and command catalog docs

Test plan

  • make format
  • make lint
  • ASC_BYPASS_KEYCHAIN=1 make test
  • go run . --help (verify grouped root help and updated assets wording)
  • go build -o /tmp/asc . && /tmp/asc (verify bare root command prints help and exits 0)

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/rudrankriyam/App-Store-Connect-CLI/pull/539 **Author:** [@rudrankriyam](https://github.com/rudrankriyam) **Created:** 2/14/2026 **Status:** ✅ Merged **Merged:** 2/14/2026 **Merged by:** [@rudrankriyam](https://github.com/rudrankriyam) **Base:** `main` ← **Head:** `feat/root-help-grouping` --- ### 📝 Commits (2) - [`dd61a03`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/dd61a03e3661c4c448fb3bb107e628eba0c8df86) feat(cli): group root help and clarify media wording - [`ee1126e`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/ee1126eaacdd446c532d2c9c9df3c5cfd9a5ae93) Handle -- separator in positional arg detection (#540) ### 📊 Changes **9 files changed** (+302 additions, -10 deletions) <details> <summary>View changed files</summary> 📝 `ASC.md` (+1 -1) 📝 `cmd/root.go` (+3 -3) ➕ `cmd/root_usage.go` (+203 -0) 📝 `cmd/run.go` (+29 -0) 📝 `cmd/run_additional_test.go` (+52 -0) 📝 `internal/cli/assets/assets.go` (+2 -2) 📝 `internal/cli/assets/assets_previews.go` (+2 -2) 📝 `internal/cli/assets/assets_test.go` (+9 -1) 📝 `internal/cli/docs/templates/ASC.md` (+1 -1) </details> ### 📄 Description ## Summary - group root `asc --help` commands into gh-style sections with `command:` alignment for faster scanning - make bare `asc` print root help and exit successfully instead of returning a usage error - clarify asset/previews wording to explicitly call out app preview videos in command help and command catalog docs ## Test plan - [x] `make format` - [x] `make lint` - [x] `ASC_BYPASS_KEYCHAIN=1 make test` - [x] `go run . --help` (verify grouped root help and updated `assets` wording) - [x] `go build -o /tmp/asc . && /tmp/asc` (verify bare root command prints help and exits `0`) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-26 22:31:43 +03:00
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#603
No description provided.