Skip to content

ci: add Slack PR review notification caller - #1947

Merged
aidandaly24 merged 2 commits into
mainfrom
feat/slack-pr-review-notification
Aug 7, 2026
Merged

ci: add Slack PR review notification caller#1947
aidandaly24 merged 2 commits into
mainfrom
feat/slack-pr-review-notification

Conversation

@jariy17

@jariy17 jariy17 commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Adds the caller workflow that pings PR reviewers in Slack (#agentcore-sdk-cli-prs) via the shared reusable reusable-slack-pr-review-notification (pinned @69340df).

Behavior

event pings
review requested the requested reviewer
review submitted the PR author (approved / changes / commented)
new revision (synchronize) prior reviewers + commenters + pending requested reviewers

Mentions render as real Slack pills (Person-type vars, IDs resolved from the SLACK_PR_REVIEW_USER_MAP secret). Verified end-to-end against the live Slack workflow.

Depends on

Pings reviewers in Slack via the shared reusable workflow (pinned @69340df):
- review requested -> ping the reviewer
- review submitted -> ping the author
- new revision      -> ping prior reviewers/commenters
@jariy17
jariy17 requested a review from a team August 7, 2026 17:32
@github-actions github-actions Bot added size/s PR size: S agentcore-harness-reviewing AgentCore Harness review in progress labels Aug 7, 2026
@agentcore-devx-automation agentcore-devx-automation Bot added the claude-security-reviewing Claude Code /security-review in progress label Aug 7, 2026
@agentcore-devx-automation

Copy link
Copy Markdown
Contributor

Claude Security Review: no high-confidence findings. (run)

@agentcore-devx-automation agentcore-devx-automation Bot removed the claude-security-reviewing Claude Code /security-review in progress label Aug 7, 2026
@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Package Tarball

aws-agentcore-0.26.0.tgz

How to install

gh release download pr-1947-tarball --repo aws/agentcore-cli --pattern "*.tgz" --dir /tmp/pr-tarball
npm install -g /tmp/pr-tarball/aws-agentcore-0.26.0.tgz

@github-actions github-actions Bot added size/s PR size: S and removed size/s PR size: S labels Aug 7, 2026
@agentcore-devx-automation agentcore-devx-automation Bot added the claude-security-reviewing Claude Code /security-review in progress label Aug 7, 2026
@agentcore-devx-automation

Copy link
Copy Markdown
Contributor

Claude Security Review: no high-confidence findings. (run)

@agentcore-devx-automation agentcore-devx-automation Bot removed the claude-security-reviewing Claude Code /security-review in progress label Aug 7, 2026
@github-actions github-actions Bot removed the agentcore-harness-reviewing AgentCore Harness review in progress label Aug 7, 2026
@aidandaly24
aidandaly24 merged commit 6fb2b5d into main Aug 7, 2026
28 of 30 checks passed
@aidandaly24
aidandaly24 deleted the feat/slack-pr-review-notification branch August 7, 2026 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/s PR size: S

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants