ci: grant id-token: write for Slack notify workflow - #642
Conversation
b2a1834 to
6f71e10
Compare
Codex AI reviewNo actionable findings. Residual risk is limited to runtime validation of the external reusable workflow and OIDC role configuration. Reviewed commit |
Claude AI reviewThis PR makes a minimal, CI-only change to
No Java/SDK source, tests, checkpoint-replay, serialization, or lifecycle code is affected, so the repository's unit/integration/replay test requirements do not apply here. Findings: None. Residual risk: Functional success still depends on the Reviewed commit |
Fixes aws/aws-durable-execution-ci#30.
The reusable
notify.ymlin aws-durable-execution-ci changed itssummarizejob at ac55d9f to assumeBEDROCK_ROLE_ARNvia OIDC,requiring
id-token: write. Consumernotify.ymlfiles still grantonly
contents: read+models: read, and GitHub caps a reusableworkflow's token at the caller's grant — aborting the run at startup
and dropping all Slack notifications.
Changes (identical across all four consumer repos):
id-token: writepermissionBEDROCK_ROLE_ARN(enables AI-generated summarieswhen the org secret is configured) alongside the existing Slack webhooks