[PR #332] [MERGED] Add asc notify slack command for Slack webhook notifications #443

New issue

Closed

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

kerem commented

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

Owner

📋 Pull Request Information

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

Base: main ← Head: feature/notify-slack

📝 Commits (3)

9e8e59e Add asc notify slack command for Slack webhook notifications
72db0a1 Fix notify slack: add timeout, UsageFunc, and improve tests
5d1333a Improve notify tests: add stderr assertions and cmdtest integration tests

📊 Changes

5 files changed (+517 additions, -1 deletions)

View changed files

📝 Agents.md (+8 -0)
➕ internal/cli/cmdtest/notify_test.go (+97 -0)
➕ internal/cli/notify/notify.go (+131 -0)
➕ internal/cli/notify/notify_test.go (+278 -0)
📝 internal/cli/registry/registry.go (+3 -1)

📄 Description

Summary

This PR adds the command to send messages via Slack incoming webhooks, enabling AI agents and CI/CD pipelines to receive notifications about App Store Connect operations.

Features

**** - Send messages to Slack via incoming webhook
Supports flag or environment variable
Optional for targeting specific channels () or users ()

Usage Examples

Changes

- Command implementation
- Unit tests
- Command registration

Testing

All tests pass:

Validation error tests (missing webhook, missing message)
Webhook resolution tests (flag priority, env fallback)
Command structure tests (subcommands, naming)

_{🔄 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/332 **Author:** [@rudrankriyam](https://github.com/rudrankriyam) **Created:** 1/30/2026 **Status:** ✅ Merged **Merged:** 1/30/2026 **Merged by:** [@rudrankriyam](https://github.com/rudrankriyam) **Base:** `main` ← **Head:** `feature/notify-slack` --- ### 📝 Commits (3) - [`9e8e59e`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/9e8e59e8b82e103b0772af4a9bc650a27a71127b) Add asc notify slack command for Slack webhook notifications - [`72db0a1`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/72db0a1cbb8afb8488b317cb6bba1ca491429e7b) Fix notify slack: add timeout, UsageFunc, and improve tests - [`5d1333a`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/5d1333a0b7240117df929efcb6f8c5b86a4e7f16) Improve notify tests: add stderr assertions and cmdtest integration tests ### 📊 Changes **5 files changed** (+517 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `Agents.md` (+8 -0) ➕ `internal/cli/cmdtest/notify_test.go` (+97 -0) ➕ `internal/cli/notify/notify.go` (+131 -0) ➕ `internal/cli/notify/notify_test.go` (+278 -0) 📝 `internal/cli/registry/registry.go` (+3 -1) </details> ### 📄 Description ## Summary This PR adds the command to send messages via Slack incoming webhooks, enabling AI agents and CI/CD pipelines to receive notifications about App Store Connect operations. ## Features - **** - Send messages to Slack via incoming webhook - Supports flag or environment variable - Optional for targeting specific channels () or users () ## Usage Examples ## Changes - - Command implementation - - Unit tests - - Command registration ## Testing All tests pass: - Validation error tests (missing webhook, missing message) - Webhook resolution tests (flag priority, env fallback) - Command structure tests (subcommands, naming) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>