Skip to content

fix(ci): authorize manual tarball dispatches - #2101

Closed
tejaskash wants to merge 2 commits into
refactorfrom
fix/manual-tarball-dispatch
Closed

fix(ci): authorize manual tarball dispatches#2101
tejaskash wants to merge 2 commits into
refactorfrom
fix/manual-tarball-dispatch

Conversation

@tejaskash

Copy link
Copy Markdown
Contributor

Summary

  • skip the pull-request author Secrets Manager gate for manual tarball dispatches
  • auto-authorize repository-writer dispatches, matching the existing manual review workflow behavior
  • retain the authorized-author gate for automatic pull request events

Context

PR #2100 merged before this correction landed. Its manual tarball run proved that the secrets-reader role does not trust workflow_dispatch OIDC identities.

Verification

  • go run github.com/rhysd/actionlint/cmd/actionlint@v1.7.7 .github/workflows/pr-automation.yml
  • bunx prettier --check .github/workflows/pr-automation.yml
  • git diff --check

@github-actions github-actions Bot added the size/xs PR size: XS label Aug 25, 2026
@agentcore-devx-automation agentcore-devx-automation Bot added claude-security-reviewing Claude Code /security-review in progress agentcore-harness-reviewing AgentCore Harness review in progress and removed agentcore-harness-reviewing AgentCore Harness review in progress labels Aug 25, 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 25, 2026
@codecov-commenter

codecov-commenter commented Aug 25, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.33%. Comparing base (ea92ba5) to head (e1e10ec).
⚠️ Report is 1 commits behind head on refactor.

Additional details and impacted files
@@            Coverage Diff            @@
##           refactor    #2101   +/-   ##
=========================================
  Coverage     97.33%   97.33%           
=========================================
  Files           417      417           
  Lines         25250    25250           
=========================================
  Hits          24578    24578           
  Misses          672      672           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions github-actions Bot added size/xs PR size: XS and removed size/xs PR size: XS labels Aug 25, 2026
@agentcore-devx-automation agentcore-devx-automation Bot added the claude-security-reviewing Claude Code /security-review in progress label Aug 25, 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 25, 2026
@tejaskash tejaskash closed this Aug 25, 2026
@aws aws deleted a comment from github-actions Bot Aug 25, 2026
@tejaskash
tejaskash deleted the fix/manual-tarball-dispatch branch August 25, 2026 19:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/xs PR size: XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants