You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This PR only touches .github/workflows/ai-pr-review.yml, bumping the pinned commit SHA for the reusable aws/aws-durable-execution-ci/.github/workflows/ai-pr-review.yml workflow (from 8de63fa6... to d6b017da...) and adding an explicit run-claude: false input.
Review notes:
Both the old and new referenced SHAs are well-formed 40-character hex strings, and the new SHA matches what's quoted in the PR description, so there's no truncation/typo in the pin.
The workflow continues to pin the reusable workflow to an exact commit SHA (not a mutable tag), which is the correct supply-chain-security pattern for a pull_request_target workflow that runs with contents: write, id-token: write, pull-requests: write, and secrets: inherit.
The added run-claude: false line is indented consistently with the existing with: block (6 spaces, sibling of prompt-path), so the YAML structure is valid.
This is a config-only change with no core/testing/otel/examples package code touched, so none of the durable-execution correctness, replay-determinism, or public API concerns apply here.
No functional defects found in the diff. Residual risk is limited to items that can't be verified without network/CI access in this sandbox: (1) whether the new commit SHA truly corresponds to the v0.3.1 release of the external reusable workflow and (2) whether that workflow version actually accepts a run-claude input — both would only surface once the workflow is actually invoked by GitHub Actions. No test changes are included, which is appropriate since this is a CI configuration change rather than SDK code.
Reviewed commit cc004292574e18019de1fcca33e3916ee8afa582. Workflow run
No actionable findings. Residual risk: the newly pinned reusable workflow and its run-claude input contract were not exercised by this configuration-only change.
Reviewed commit cc004292574e18019de1fcca33e3916ee8afa582. Workflow run
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
d6b017da14385908951d23e26c790b28a4e5f9f0(v0.3.1).run-claude: falseso Claude AI review remains disabled.Testing
git diff --check.github/workflows/ai-pr-review.ymlwith PyYAML