Repository navigation
ci: require workflow checks and production dependency audits - #1128
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 5f519627bd
ℹ️ 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".
soyeon-oai
left a comment
There was a problem hiding this comment.
Reviewed workflow selection and aggregate gating, pinned lint installation, production audit propagation, and release dependencies. No actionable findings. Inspected successful hosted CI/lint evidence; no independent test execution.
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
The workflow gates, blocking audits, configuration, documentation, and focused tests are consistent and complete.
Review effort: Balanced
Findings: None
What changed in this PR
Adds required workflow linting and makes production dependency audits blocking in CI and release validation.
Changes:
- Adds actionlint, ShellCheck, and zizmor workflow checks.
- Integrates workflow checks into required Unix CI gates.
- Enforces production dependency audit failures and documents policies.
| File | Description |
|---|---|
.github/actionlint.yaml |
Configures release-queue lint exceptions. |
.github/zizmor.yml |
Documents privileged-trigger exceptions. |
.github/workflows/workflow-quality.yml |
Defines pinned workflow linting checks. |
.github/workflows/node-ci.yml |
Selects and gates workflow checks and audits. |
.github/workflows/node-release.yml |
Makes release audits blocking. |
.github/workflows/native-unix.yml |
Clarifies empty PATH assignments. |
.github/workflows/container-release.yml |
Disables Bun caching during release verification. |
CONTRIBUTING.md |
Documents workflow and audit checks. |
RELEASING.md |
Documents blocking production audits. |
sdk/typescript/tests-ts/workflow-quality.test.ts |
Tests workflow-check selection and gating. |
sdk/typescript/tests-ts/skeleton.test.ts |
Updates CI structure and audit assertions. |
sdk/typescript/tests-ts/release-automation.test.ts |
Selects coverage gates by step name. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Summary
Require workflow linting and production dependency audits to pass before CI succeeds or npm release validation proceeds.
Changes
Testing
Risk and rollout
Required check names stay unchanged. High or critical production advisories and audit-service failures now block package builds and npm release validation. Workflow and audit failures must be fixed before publication. There are no public CLI or runtime changes.
Public disclosure review