[PR #276] [MERGED] Auth: add validate command #400

New issue

Closed

opened 2026-02-26 22:30:46 +03:00 by kerem · 0 comments

kerem commented

2026-02-26 22:30:46 +03:00

Owner

📋 Pull Request Information

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

Base: main ← Head: feat/auth-validate

📝 Commits (10+)

abd3618 feat: add auth validate command
3668012 feat: improve auth diagnostics and storage clarity
3341705 feat: validate creds via auth status
7d521d3 chore: tweak auth status validation wording
2d6e333 Merge branch 'main' into feat/auth-validate
73c613a Fix strict auth env parsing
65e514a chore: include profile name in validation output
493154e Merge pull request #277 from rudrankriyam/cursor/strict-auth-value-handling-1018
87e5873 chore: treat forbidden as limited validation
c407be9 Merge branch 'feat/auth-validate' of https://github.com/rudrankriyam/App-Store-Connect-CLI into feat/auth-validate

📊 Changes

13 files changed (+1541 additions, -55 deletions)

View changed files

📝 Agents.md (+1 -0)
📝 cmd/auth.go (+308 -5)
📝 cmd/commands_test.go (+359 -0)
➕ cmd/errors.go (+32 -0)
📝 cmd/root.go (+1 -0)
📝 cmd/shared.go (+124 -21)
📝 cmd/shared_test.go (+81 -8)
📝 docs/GO_STANDARDS.md (+2 -0)
📝 internal/asc/client_http.go (+9 -3)
➕ internal/auth/doctor.go (+453 -0)
➕ internal/auth/doctor_test.go (+109 -0)
📝 internal/auth/keychain.go (+57 -17)
📝 main.go (+5 -1)

📄 Description

Summary

add auth status validation with network checks for all stored credentials
add auth doctor diagnostics, temp key cleanup, login-time JWT validation, and strict mixed-source warnings
improve auth status storage reporting with tests and config/source metadata

Test plan

go test ./...

_{🔄 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/276 **Author:** [@rudrankriyam](https://github.com/rudrankriyam) **Created:** 1/28/2026 **Status:** ✅ Merged **Merged:** 1/28/2026 **Merged by:** [@rudrankriyam](https://github.com/rudrankriyam) **Base:** `main` ← **Head:** `feat/auth-validate` --- ### 📝 Commits (10+) - [`abd3618`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/abd36180f3b7295c0480418472ef5bf5adf91310) feat: add auth validate command - [`3668012`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/36680127ace233387939f3b7c4d89e3bfc46ba1b) feat: improve auth diagnostics and storage clarity - [`3341705`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/334170553ba5905737a5a54cdb73fda2d50fcf66) feat: validate creds via auth status - [`7d521d3`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/7d521d34a63dc721e5c0aee5b17e09748034a0a5) chore: tweak auth status validation wording - [`2d6e333`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/2d6e333e4f113528212ae7a44f6520d49f3dcaf5) Merge branch 'main' into feat/auth-validate - [`73c613a`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/73c613a186ecfbccd8f64101fd0a6f2cd1a73c07) Fix strict auth env parsing - [`65e514a`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/65e514a249c3dd33f0137dcc5932dac1daa041c0) chore: include profile name in validation output - [`493154e`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/493154ed7ed055c003cea86f26afbc395e598259) Merge pull request #277 from rudrankriyam/cursor/strict-auth-value-handling-1018 - [`87e5873`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/87e58739a181736d7ded0fb9f5d9e361d4f59362) chore: treat forbidden as limited validation - [`c407be9`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/c407be9286b52252d6df11f6cefba3223d831f14) Merge branch 'feat/auth-validate' of https://github.com/rudrankriyam/App-Store-Connect-CLI into feat/auth-validate ### 📊 Changes **13 files changed** (+1541 additions, -55 deletions) <details> <summary>View changed files</summary> 📝 `Agents.md` (+1 -0) 📝 `cmd/auth.go` (+308 -5) 📝 `cmd/commands_test.go` (+359 -0) ➕ `cmd/errors.go` (+32 -0) 📝 `cmd/root.go` (+1 -0) 📝 `cmd/shared.go` (+124 -21) 📝 `cmd/shared_test.go` (+81 -8) 📝 `docs/GO_STANDARDS.md` (+2 -0) 📝 `internal/asc/client_http.go` (+9 -3) ➕ `internal/auth/doctor.go` (+453 -0) ➕ `internal/auth/doctor_test.go` (+109 -0) 📝 `internal/auth/keychain.go` (+57 -17) 📝 `main.go` (+5 -1) </details> ### 📄 Description ## Summary - add auth status validation with network checks for all stored credentials - add auth doctor diagnostics, temp key cleanup, login-time JWT validation, and strict mixed-source warnings - improve auth status storage reporting with tests and config/source metadata ## Test plan - [x] go test ./... --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>