Skip to content

[Infra] Pin workflow actions to commit SHAs and audit movable refs - #66

Open
DongyuZhao wants to merge 2 commits into
mainfrom
pin-actions-to-commit-shas
Open

[Infra] Pin workflow actions to commit SHAs and audit movable refs#66
DongyuZhao wants to merge 2 commits into
mainfrom
pin-actions-to-commit-shas

Conversation

@DongyuZhao

Copy link
Copy Markdown
Contributor

Summary

Verification

audit:ci green on the pinned tree; negative test confirmed the new check fails with a pointed message when a single ref is reverted to @v7.

🤖 Generated with Claude Code

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: c7ab965bf3

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread scripts/audit-ci-policy.sh Outdated
Comment thread scripts/audit-ci-policy.sh Outdated
Review: a repository-local composite action's nested uses: refs execute
in the same jobs but were not scanned, so the first .github/actions
manifest added could reintroduce a movable ref. Include that directory
in the audit when it exists.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Jul 28, 2026

Copy link
Copy Markdown

PR performance and binary-size diff

Benchmark execution is required; hosted-runner timing, memory, and size deltas are informational only.

Baseline: exact PR base 384e8a744487 (successful main CI).

Performance

Runtime Workload Median Perf diff Memory Memory diff
c representative_large 6.294 ms ⚠️ +20.6% 8,592 KiB ✅ -0.7%
es deep_nesting 0.304 ms ⚠️ +4.9% 131,588 KiB ✅ -12.1%
es large_document 32.612 ms ✅ -2.1% 131,588 KiB ✅ -12.1%
kotlin deep_nesting 0.279 ms ⚠️ +14.2% 97,548 KiB ✅ -5.7%
kotlin large_document 23.289 ms ✅ -29.7% 95,056 KiB ✅ -6.8%
swift deep_nesting 0.074 ms ✅ -7.9% 26,080 KiB ✅ +0.2%
swift large_document 9.898 ms ✅ -55.2% 25,904 KiB ✅ -0.3%

Binary size

Artifact Bytes KiB Size diff
c-shared-library 501,888 490.1 ✅ 0 B (0.0%)
es-wasm 288,573 281.8 ✅ 0 B (0.0%)
kotlin-jvm-jar 312,972 305.6 ✅ 0 B (0.0%)

Head CI run

Size deltas are deterministic for identical build inputs. Performance deltas compare separate hosted runners and should be treated as directional evidence, not a pass/fail threshold.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant