[PR #79] [MERGED] Asc financial reports support #268

New issue

Closed

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

kerem commented

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

Owner

📋 Pull Request Information

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

Base: main ← Head: cursor/asc-financial-reports-support-18e5

📝 Commits (10+)

b86a2d9 Use default keychain and migrate legacy entries
8645cc3 Add finance reports support
37092e5 Align finance report JSON fields
117f1f1 Remove outdated PLAN.md
65e201f Update README with analytics and build upload notes
7ea2e64 Update docs after PLAN removal
61bb321 Add finance region listing and validation.
6ba7c94 Merge pull request #81 from rudrankriyam/cursor/finance-report-json-fields-c62e
b5b4373 Merge pull request #69 from rudrankriyam/fix/keychain-default
5e9657f Add finance reports support

📊 Changes

19 files changed (+955 additions, -53 deletions)

View changed files

📝 Agents.md (+10 -0)
📝 CONTRIBUTING.md (+1 -1)
➖ PLAN.md (+0 -47)
📝 README.md (+35 -1)
📝 cmd/analytics_test.go (+3 -0)
📝 cmd/commands.go (+1 -0)
➕ cmd/finance.go (+178 -0)
➕ cmd/finance_helpers.go (+44 -0)
➕ cmd/finance_helpers_test.go (+56 -0)
➕ cmd/finance_test.go (+113 -0)
📝 internal/asc/client.go (+8 -0)
➕ internal/asc/finance.go (+54 -0)
➕ internal/asc/finance_output.go (+80 -0)
➕ internal/asc/finance_regions.go (+72 -0)
➕ internal/asc/finance_regions_test.go (+26 -0)
➕ internal/asc/finance_test.go (+92 -0)
📝 internal/asc/output_test.go (+92 -0)
📝 internal/auth/keychain.go (+85 -4)
📝 internal/auth/keychain_test.go (+5 -0)

📄 Description

Add asc finance reports CLI command to download App Store Connect financial reports.

This implements the functionality to retrieve monthly finance reports (FINANCIAL and FINANCE_DETAIL types) from the App Store Connect API, providing options for specifying vendor, region, date, output file, and decompression.

_{🔄 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/79 **Author:** [@rudrankriyam](https://github.com/rudrankriyam) **Created:** 1/24/2026 **Status:** ✅ Merged **Merged:** 1/24/2026 **Merged by:** [@rudrankriyam](https://github.com/rudrankriyam) **Base:** `main` ← **Head:** `cursor/asc-financial-reports-support-18e5` --- ### 📝 Commits (10+) - [`b86a2d9`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/b86a2d9bf520b71b0cda20812f478e520063bb0c) Use default keychain and migrate legacy entries - [`8645cc3`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/8645cc3a49345a75beed3ce638b1c0d6ac963e72) Add finance reports support - [`37092e5`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/37092e582d58047b41191dccff630547bd3290fa) Align finance report JSON fields - [`117f1f1`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/117f1f116ed9c4882a907d2945d0294ff27e40d5) Remove outdated PLAN.md - [`65e201f`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/65e201f01ed4a405531bc659b83a99ec0742b14c) Update README with analytics and build upload notes - [`7ea2e64`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/7ea2e6417aa5fc5ebc84afb33ad0c82ca4a84fe1) Update docs after PLAN removal - [`61bb321`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/61bb32190662b0e8dd1931a205c8cae887ef6b36) Add finance region listing and validation. - [`6ba7c94`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/6ba7c94337995f8f36c14769aa247c3cf582497e) Merge pull request #81 from rudrankriyam/cursor/finance-report-json-fields-c62e - [`b5b4373`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/b5b4373309acdd812017b12ec6f7a7775bc36794) Merge pull request #69 from rudrankriyam/fix/keychain-default - [`5e9657f`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/5e9657fc8cb37408375d24c639de17b39ab58b69) Add finance reports support ### 📊 Changes **19 files changed** (+955 additions, -53 deletions) <details> <summary>View changed files</summary> 📝 `Agents.md` (+10 -0) 📝 `CONTRIBUTING.md` (+1 -1) ➖ `PLAN.md` (+0 -47) 📝 `README.md` (+35 -1) 📝 `cmd/analytics_test.go` (+3 -0) 📝 `cmd/commands.go` (+1 -0) ➕ `cmd/finance.go` (+178 -0) ➕ `cmd/finance_helpers.go` (+44 -0) ➕ `cmd/finance_helpers_test.go` (+56 -0) ➕ `cmd/finance_test.go` (+113 -0) 📝 `internal/asc/client.go` (+8 -0) ➕ `internal/asc/finance.go` (+54 -0) ➕ `internal/asc/finance_output.go` (+80 -0) ➕ `internal/asc/finance_regions.go` (+72 -0) ➕ `internal/asc/finance_regions_test.go` (+26 -0) ➕ `internal/asc/finance_test.go` (+92 -0) 📝 `internal/asc/output_test.go` (+92 -0) 📝 `internal/auth/keychain.go` (+85 -4) 📝 `internal/auth/keychain_test.go` (+5 -0) </details> ### 📄 Description Add `asc finance reports` CLI command to download App Store Connect financial reports. This implements the functionality to retrieve monthly finance reports (FINANCIAL and FINANCE_DETAIL types) from the App Store Connect API, providing options for specifying vendor, region, date, output file, and decompression. --- <a href="https://cursor.com/background-agent?bcId=bc-fcff6d6e-2f9f-4d21-b83e-ff877ae93aba"><picture><source media="(prefers-color-scheme: dark)" srcset="https://cursor.com/open-in-cursor-dark.svg"><source media="(prefers-color-scheme: light)" srcset="https://cursor.com/open-in-cursor-light.svg"><img alt="Open in Cursor" src="https://cursor.com/open-in-cursor.svg"></picture></a> <a href="https://cursor.com/agents?id=bc-fcff6d6e-2f9f-4d21-b83e-ff877ae93aba"><picture><source media="(prefers-color-scheme: dark)" srcset="https://cursor.com/open-in-web-dark.svg"><source media="(prefers-color-scheme: light)" srcset="https://cursor.com/open-in-web-light.svg"><img alt="Open in Web" src="https://cursor.com/open-in-web.svg"></picture></a> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>