Skip to content
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions .github/workflows/claude-review.yml
Original file line number Diff line number Diff line change
Expand Up @@ -404,9 +404,9 @@
&& vars.CLAUDE_REVIEW_DISABLED != 'true'
&& !contains(format(',{0},', inputs.skip-actors), format(',{0},', github.actor))
&& (!endsWith(github.actor, '[bot]')
|| github.actor == 'dependabot[bot]'

Check failure on line 407 in .github/workflows/claude-review.yml

View workflow job for this annotation

GitHub Actions / zizmor / zizmor

bot-conditions

claude-review.yml:407: spoofable bot actor check: actor context may be spoofable
|| (github.event_name != 'pull_request'
|| contains(format(',{0},', 'OWNER,MEMBER,COLLABORATOR'), format(',{0},', github.event.pull_request.author_association))))

Check warning on line 409 in .github/workflows/claude-review.yml

View workflow job for this annotation

GitHub Actions / zizmor / zizmor

obfuscation

claude-review.yml:409: obfuscated usage of GitHub Actions features: can be reduced to a constant
&& (github.event_name != 'pull_request'
|| github.event.pull_request.draft == false) }}
permissions:
Expand Down Expand Up @@ -541,7 +541,7 @@
# workflow_dispatch freshness works when the event has no pull_request.
- name: Check whether this head is still current
id: freshness
uses: melodic-software/ci-workflows/.github/actions/claude-lane-freshness@3c37af40fe1d90b16222a2887a27c87022fe111e # 3c37af4 2026-08-12

Check failure on line 544 in .github/workflows/claude-review.yml

View workflow job for this annotation

GitHub Actions / zizmor / zizmor

impostor-commit

claude-review.yml:544: commit with no history in referenced repository: uses a commit that doesn't belong to the specified org/repo
with:
pull-number: ${{ steps.resolve-pr.outputs.number }}
head-sha: ${{ steps.resolve-pr.outputs.head-sha }}
Expand Down Expand Up @@ -613,7 +613,7 @@
# persist-credentials:false makes that fetch fail and leaves an orphan
# tracking comment. The final strip step below scrubs the persisted token
# to restore zizmor `artipacked` compliance. Reversible when #1236 lands.
- name: Check out

Check warning on line 616 in .github/workflows/claude-review.yml

View workflow job for this annotation

GitHub Actions / zizmor / zizmor

artipacked

claude-review.yml:616: credential persistence through GitHub Actions artifacts: does not set persist-credentials: false
if: steps.freshness.outputs.superseded != 'true' && steps.review-count.outputs.capped != 'true'
uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7.0.1
with:
Expand All @@ -632,7 +632,7 @@
if: >-
inputs.standards-ref != '' && steps.freshness.outputs.superseded != 'true' &&
steps.review-count.outputs.capped != 'true'
uses: actions/create-github-app-token@bcd2ba49218906704ab6c1aa796996da409d3eb1 # v3.2.0

Check failure on line 635 in .github/workflows/claude-review.yml

View workflow job for this annotation

GitHub Actions / zizmor / zizmor

github-app

claude-review.yml:635: dangerous use of GitHub App tokens: app token inherits blanket installation permissions
with:
client-id: ${{ secrets.STANDARDS_REVIEW_APP_ID }}
private-key: ${{ secrets.STANDARDS_REVIEW_APP_PRIVATE_KEY }}
Expand Down Expand Up @@ -785,7 +785,7 @@
# retry below.
continue-on-error: true
timeout-minutes: 11
uses: anthropics/claude-code-action@56cf60fde42f7b19c3abfd5c9c48b69a1288461f # v1.0.222
uses: anthropics/claude-code-action@51db78a4b844e144f8d02425cb280435c04a3474 # v1.0.224
with:
claude_code_oauth_token: ${{ secrets.CLAUDE_CODE_OAUTH_TOKEN }}
# track_progress is pull_request-only upstream; forcing it on
Expand Down Expand Up @@ -1025,7 +1025,7 @@
# budget.
continue-on-error: true
timeout-minutes: 11
uses: anthropics/claude-code-action@56cf60fde42f7b19c3abfd5c9c48b69a1288461f # v1.0.222
uses: anthropics/claude-code-action@51db78a4b844e144f8d02425cb280435c04a3474 # v1.0.224
with:
claude_code_oauth_token: ${{ secrets.CLAUDE_CODE_OAUTH_TOKEN }}
# track_progress is pull_request-only upstream; forcing it on
Expand Down Expand Up @@ -1131,7 +1131,7 @@
# a PR, and a rejected API call must not turn the infra-failure path
# red (see the composite's description).
continue-on-error: true
uses: melodic-software/ci-workflows/.github/actions/claude-lane-marker-comment@3c37af40fe1d90b16222a2887a27c87022fe111e # 3c37af4 2026-08-12

Check failure on line 1134 in .github/workflows/claude-review.yml

View workflow job for this annotation

GitHub Actions / zizmor / zizmor

impostor-commit

claude-review.yml:1134: commit with no history in referenced repository: uses a commit that doesn't belong to the specified org/repo
with:
marker: "<!-- claude-review-infra-status -->"
mode: post-failure
Expand All @@ -1155,7 +1155,7 @@
# Tidying a stale comment is housekeeping, and a rejected API call
# during it must not turn a successful review red.
continue-on-error: true
uses: melodic-software/ci-workflows/.github/actions/claude-lane-marker-comment@3c37af40fe1d90b16222a2887a27c87022fe111e # 3c37af4 2026-08-12

Check failure on line 1158 in .github/workflows/claude-review.yml

View workflow job for this annotation

GitHub Actions / zizmor / zizmor

impostor-commit

claude-review.yml:1158: commit with no history in referenced repository: uses a commit that doesn't belong to the specified org/repo
with:
marker: "<!-- claude-review-infra-status -->"
mode: clear-on-success
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/claude-security-review.yml
Original file line number Diff line number Diff line change
Expand Up @@ -927,9 +927,9 @@
&& needs.changes.outputs.relevant != 'false'
&& !contains(format(',{0},', inputs.skip-actors), format(',{0},', github.actor))
&& (!endsWith(github.actor, '[bot]')
|| github.actor == 'dependabot[bot]'

Check failure on line 930 in .github/workflows/claude-security-review.yml

View workflow job for this annotation

GitHub Actions / zizmor / zizmor

bot-conditions

claude-security-review.yml:930: spoofable bot actor check: actor context may be spoofable
|| (github.event_name != 'pull_request'
|| contains(format(',{0},', 'OWNER,MEMBER,COLLABORATOR'), format(',{0},', github.event.pull_request.author_association))))

Check warning on line 932 in .github/workflows/claude-security-review.yml

View workflow job for this annotation

GitHub Actions / zizmor / zizmor

obfuscation

claude-security-review.yml:932: obfuscated usage of GitHub Actions features: can be reduced to a constant
&& (github.event_name != 'pull_request'
|| github.event.pull_request.head.repo.full_name == github.repository) }}
permissions:
Expand Down Expand Up @@ -1041,7 +1041,7 @@
# workflow_dispatch freshness works when the event has no pull_request.
- name: Check whether this head is still current
id: freshness
uses: melodic-software/ci-workflows/.github/actions/claude-lane-freshness@3c37af40fe1d90b16222a2887a27c87022fe111e # 3c37af4 2026-08-12

Check failure on line 1044 in .github/workflows/claude-security-review.yml

View workflow job for this annotation

GitHub Actions / zizmor / zizmor

impostor-commit

claude-security-review.yml:1044: commit with no history in referenced repository: uses a commit that doesn't belong to the specified org/repo
with:
pull-number: ${{ steps.resolve-pr.outputs.number }}
head-sha: ${{ steps.resolve-pr.outputs.head-sha }}
Expand All @@ -1053,7 +1053,7 @@
# tracking comment. The final strip step below scrubs the persisted token
# to restore zizmor `artipacked` compliance. Reversible when #1236 lands.
# Explicit head SHA so workflow_dispatch checks out the PR under review.
- name: Check out

Check warning on line 1056 in .github/workflows/claude-security-review.yml

View workflow job for this annotation

GitHub Actions / zizmor / zizmor

artipacked

claude-security-review.yml:1056: credential persistence through GitHub Actions artifacts: does not set persist-credentials: false
if: steps.freshness.outputs.superseded != 'true'
uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7.0.1
with:
Expand Down Expand Up @@ -1140,7 +1140,7 @@
# leaves room for the retry (see the retry formula there).
continue-on-error: true
timeout-minutes: 18
uses: anthropics/claude-code-action@56cf60fde42f7b19c3abfd5c9c48b69a1288461f # v1.0.222
uses: anthropics/claude-code-action@51db78a4b844e144f8d02425cb280435c04a3474 # v1.0.224
with:
claude_code_oauth_token: ${{ secrets.CLAUDE_CODE_OAUTH_TOKEN }}
# track_progress is pull_request-only upstream; forcing it on
Expand Down Expand Up @@ -1409,7 +1409,7 @@
# and the same step timeout keeps the retry inside the job budget.
continue-on-error: true
timeout-minutes: 18
uses: anthropics/claude-code-action@56cf60fde42f7b19c3abfd5c9c48b69a1288461f # v1.0.222
uses: anthropics/claude-code-action@51db78a4b844e144f8d02425cb280435c04a3474 # v1.0.224
with:
claude_code_oauth_token: ${{ secrets.CLAUDE_CODE_OAUTH_TOKEN }}
# track_progress is pull_request-only upstream; forcing it on
Expand Down Expand Up @@ -1568,7 +1568,7 @@
# comment is the PR-visible half of an alarm whose check is GREEN. It
# has to carry the weight the conclusion no longer does.
continue-on-error: true
uses: melodic-software/ci-workflows/.github/actions/claude-lane-marker-comment@3c37af40fe1d90b16222a2887a27c87022fe111e # 3c37af4 2026-08-12

Check failure on line 1571 in .github/workflows/claude-security-review.yml

View workflow job for this annotation

GitHub Actions / zizmor / zizmor

impostor-commit

claude-security-review.yml:1571: commit with no history in referenced repository: uses a commit that doesn't belong to the specified org/repo
with:
marker: "<!-- claude-security-review-infra-status -->"
mode: post-failure
Expand Down Expand Up @@ -1597,7 +1597,7 @@
# Tidying a stale comment is housekeeping, and a rejected API call
# during it must not turn a successful review red.
continue-on-error: true
uses: melodic-software/ci-workflows/.github/actions/claude-lane-marker-comment@3c37af40fe1d90b16222a2887a27c87022fe111e # 3c37af4 2026-08-12

Check failure on line 1600 in .github/workflows/claude-security-review.yml

View workflow job for this annotation

GitHub Actions / zizmor / zizmor

impostor-commit

claude-security-review.yml:1600: commit with no history in referenced repository: uses a commit that doesn't belong to the specified org/repo
with:
marker: "<!-- claude-security-review-infra-status -->"
mode: clear-on-success
Expand Down
Loading