[PR #449] [MERGED] Default output environment variable #534

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

📋 Pull Request Information

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

Base: mainHead: cursor/default-output-environment-variable-66c3


📝 Commits (6)

  • bb470f2 feat: add ASC_DEFAULT_OUTPUT env var with DefaultOutputFormat()
  • 8ba338b refactor: use DefaultOutputFormat() in shared package command builders
  • 246cc97 refactor: use shared.DefaultOutputFormat() across all CLI commands
  • 8f69e04 docs: document ASC_DEFAULT_OUTPUT environment variable
  • 917cdb7 perf: cache DefaultOutputFormat() result with sync.Once
  • 696d6e8 test: cover ASC_DEFAULT_OUTPUT defaults to prevent regressions

📊 Changes

220 files changed (+1091 additions, -869 deletions)

View changed files

📝 AGENTS.md (+3 -0)
📝 README.md (+4 -0)
📝 internal/cli/accessibility/accessibility.go (+5 -5)
📝 internal/cli/actors/actors.go (+2 -2)
📝 internal/cli/agerating/age_rating.go (+2 -2)
📝 internal/cli/agreements/agreements.go (+1 -1)
📝 internal/cli/alternativedistribution/alternative_distribution_domains.go (+4 -4)
📝 internal/cli/alternativedistribution/alternative_distribution_keys.go (+5 -5)
📝 internal/cli/alternativedistribution/alternative_distribution_package_versions.go (+4 -4)
📝 internal/cli/alternativedistribution/alternative_distribution_packages.go (+5 -5)
📝 internal/cli/analytics/analytics_instances.go (+2 -2)
📝 internal/cli/analytics/analytics_reports.go (+2 -2)
📝 internal/cli/analytics/analytics_requests.go (+4 -4)
📝 internal/cli/analytics/analytics_segments.go (+1 -1)
📝 internal/cli/androidiosmapping/android_ios_mapping.go (+5 -5)
📝 internal/cli/app_events/app_events.go (+5 -5)
📝 internal/cli/app_events/localization_media.go (+4 -4)
📝 internal/cli/app_events/localizations.go (+5 -5)
📝 internal/cli/app_events/relationships.go (+1 -1)
📝 internal/cli/app_events/screenshots.go (+5 -5)

...and 80 more files

📄 Description

Add ASC_DEFAULT_OUTPUT environment variable to set a global default output format for CLI commands.

This allows users to configure their preferred output (e.g., table, json, markdown) once via an environment variable, reducing the need to specify --output on every command. A new shared.DefaultOutputFormat() function reads and validates the environment variable, falls back to json on invalid input with a single warning, and is now used across all relevant CLI commands.


Open in Cursor Open in Web


🔄 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/449 **Author:** [@rudrankriyam](https://github.com/rudrankriyam) **Created:** 2/8/2026 **Status:** ✅ Merged **Merged:** 2/8/2026 **Merged by:** [@rudrankriyam](https://github.com/rudrankriyam) **Base:** `main` ← **Head:** `cursor/default-output-environment-variable-66c3` --- ### 📝 Commits (6) - [`bb470f2`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/bb470f26a913f7b8578d92d347e7c71359af572b) feat: add ASC_DEFAULT_OUTPUT env var with DefaultOutputFormat() - [`8ba338b`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/8ba338b75b079bb381d521897cc7427699a904d8) refactor: use DefaultOutputFormat() in shared package command builders - [`246cc97`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/246cc97735e192e067d4d945fff14e692647af1d) refactor: use shared.DefaultOutputFormat() across all CLI commands - [`8f69e04`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/8f69e04184246160ed70b9026ac2e931c2564db5) docs: document ASC_DEFAULT_OUTPUT environment variable - [`917cdb7`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/917cdb7477faebd85cc95eee0ef2d7e7d68fe7f5) perf: cache DefaultOutputFormat() result with sync.Once - [`696d6e8`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/696d6e884842e3ff99af960d6d59f816b06998f6) test: cover ASC_DEFAULT_OUTPUT defaults to prevent regressions ### 📊 Changes **220 files changed** (+1091 additions, -869 deletions) <details> <summary>View changed files</summary> 📝 `AGENTS.md` (+3 -0) 📝 `README.md` (+4 -0) 📝 `internal/cli/accessibility/accessibility.go` (+5 -5) 📝 `internal/cli/actors/actors.go` (+2 -2) 📝 `internal/cli/agerating/age_rating.go` (+2 -2) 📝 `internal/cli/agreements/agreements.go` (+1 -1) 📝 `internal/cli/alternativedistribution/alternative_distribution_domains.go` (+4 -4) 📝 `internal/cli/alternativedistribution/alternative_distribution_keys.go` (+5 -5) 📝 `internal/cli/alternativedistribution/alternative_distribution_package_versions.go` (+4 -4) 📝 `internal/cli/alternativedistribution/alternative_distribution_packages.go` (+5 -5) 📝 `internal/cli/analytics/analytics_instances.go` (+2 -2) 📝 `internal/cli/analytics/analytics_reports.go` (+2 -2) 📝 `internal/cli/analytics/analytics_requests.go` (+4 -4) 📝 `internal/cli/analytics/analytics_segments.go` (+1 -1) 📝 `internal/cli/androidiosmapping/android_ios_mapping.go` (+5 -5) 📝 `internal/cli/app_events/app_events.go` (+5 -5) 📝 `internal/cli/app_events/localization_media.go` (+4 -4) 📝 `internal/cli/app_events/localizations.go` (+5 -5) 📝 `internal/cli/app_events/relationships.go` (+1 -1) 📝 `internal/cli/app_events/screenshots.go` (+5 -5) _...and 80 more files_ </details> ### 📄 Description Add `ASC_DEFAULT_OUTPUT` environment variable to set a global default output format for CLI commands. This allows users to configure their preferred output (e.g., `table`, `json`, `markdown`) once via an environment variable, reducing the need to specify `--output` on every command. A new `shared.DefaultOutputFormat()` function reads and validates the environment variable, falls back to `json` on invalid input with a single warning, and is now used across all relevant CLI commands. --- <p><a href="https://cursor.com/background-agent?bcId=bc-08c4f759-a77c-4e90-95a3-3d0c0b158cf5"><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-08c4f759-a77c-4e90-95a3-3d0c0b158cf5"><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></p> --- <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:24 +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#534
No description provided.