[PR #178] [MERGED] feat: add auth profile switching #340

New issue

Closed

opened 2026-02-26 21:34:39 +03:00 by kerem · 0 comments

kerem commented

2026-02-26 21:34:39 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/rudrankriyam/App-Store-Connect-CLI/pull/178
Author: @rudrankriyam
Created: 1/26/2026
Status: ✅ Merged
Merged: 1/26/2026
Merged by: @rudrankriyam

Base: main ← Head: feat/auth-profiles

📝 Commits (10+)

6cf7101 feat: add auth profiles and switching
19f5b69 test: add auth profile tests
f7e8cb4 Fix auth name trimming checks
7eefa2e Merge pull request #179 from rudrankriyam/cursor/credential-name-whitespace-c4fa
e15f7e4 fix: honor config credentials with bypass
165ea45 Fix credential profile and list errors
94dccfb Merge pull request #180 from rudrankriyam/cursor/credential-resolution-errors-e3fe
477ddab fix: tighten auth logout handling
dfb2ab8 test: add logout edge case coverage
7313ed6 test: cover auth profile edge cases

📊 Changes

10 files changed (+1212 additions, -100 deletions)

View changed files

📝 README.md (+13 -0)
📝 cmd/auth.go (+74 -3)
📝 cmd/commands_test.go (+80 -0)
📝 cmd/root.go (+1 -0)
📝 cmd/shared.go (+24 -7)
➕ cmd/shared_test.go (+78 -0)
📝 internal/auth/integration_test.go (+60 -0)
📝 internal/auth/keychain.go (+380 -86)
📝 internal/auth/keychain_test.go (+489 -0)
📝 internal/config/config.go (+13 -4)

📄 Description

Summary

add auth profile selection via --profile/ASC_PROFILE and config multi-credentials
add asc auth switch and named logout handling
document multi-profile usage in README

Test plan

Not run (tests will follow)

_{🔄 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/178 **Author:** [@rudrankriyam](https://github.com/rudrankriyam) **Created:** 1/26/2026 **Status:** ✅ Merged **Merged:** 1/26/2026 **Merged by:** [@rudrankriyam](https://github.com/rudrankriyam) **Base:** `main` ← **Head:** `feat/auth-profiles` --- ### 📝 Commits (10+) - [`6cf7101`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/6cf710117adc10c93d17ff8995be29964372a1ca) feat: add auth profiles and switching - [`19f5b69`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/19f5b6915d9855195c9e7cbc79728dbf5942d809) test: add auth profile tests - [`f7e8cb4`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/f7e8cb43614ec751a3b8293d2d5e5835afd74c99) Fix auth name trimming checks - [`7eefa2e`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/7eefa2e4e7db27b84871b48c852c6b4245429010) Merge pull request #179 from rudrankriyam/cursor/credential-name-whitespace-c4fa - [`e15f7e4`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/e15f7e49acc5e5957268cbedb02089918157c3c4) fix: honor config credentials with bypass - [`165ea45`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/165ea45bb9d88388e351f1d1b3fbd92e3c6f8026) Fix credential profile and list errors - [`94dccfb`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/94dccfbae1f712da7feb937b487f699910a39b7e) Merge pull request #180 from rudrankriyam/cursor/credential-resolution-errors-e3fe - [`477ddab`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/477ddabb5af1c18cb8f6d07179ab7df7208b46f5) fix: tighten auth logout handling - [`dfb2ab8`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/dfb2ab89f4c08369018c3b7c678fe19ad82f86a1) test: add logout edge case coverage - [`7313ed6`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/7313ed64021fa1b38520d9d046f5b8e43b5bebe8) test: cover auth profile edge cases ### 📊 Changes **10 files changed** (+1212 additions, -100 deletions) <details> <summary>View changed files</summary> 📝 `README.md` (+13 -0) 📝 `cmd/auth.go` (+74 -3) 📝 `cmd/commands_test.go` (+80 -0) 📝 `cmd/root.go` (+1 -0) 📝 `cmd/shared.go` (+24 -7) ➕ `cmd/shared_test.go` (+78 -0) 📝 `internal/auth/integration_test.go` (+60 -0) 📝 `internal/auth/keychain.go` (+380 -86) 📝 `internal/auth/keychain_test.go` (+489 -0) 📝 `internal/config/config.go` (+13 -4) </details> ### 📄 Description ## Summary - add auth profile selection via `--profile`/`ASC_PROFILE` and config multi-credentials - add `asc auth switch` and named logout handling - document multi-profile usage in README ## Test plan - [ ] Not run (tests will follow) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>