Conversation
- Throws a `ConfigurationError` if parsing the YAML fails - Add a couple of tests for it
Mergeback v4.30.9 refs/heads/releases/v4 into main
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Michael B. Gale <mbg@github.com>
Add experimental functionality for labelling PRs by their size
Bumps the npm-minor group with 5 updates: | Package | From | To | | --- | --- | --- | | [octokit](https://github.com/octokit/octokit.js) | `5.0.3` | `5.0.4` | | [@eslint/js](https://github.com/eslint/eslint/tree/HEAD/packages/js) | `9.37.0` | `9.38.0` | | [@typescript-eslint/eslint-plugin](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/eslint-plugin) | `8.46.0` | `8.46.1` | | [@typescript-eslint/parser](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/parser) | `8.46.0` | `8.46.1` | | [esbuild](https://github.com/evanw/esbuild) | `0.25.10` | `0.25.11` | Updates `octokit` from 5.0.3 to 5.0.4 - [Release notes](https://github.com/octokit/octokit.js/releases) - [Commits](octokit/octokit.js@v5.0.3...v5.0.4) Updates `@eslint/js` from 9.37.0 to 9.38.0 - [Release notes](https://github.com/eslint/eslint/releases) - [Commits](https://github.com/eslint/eslint/commits/v9.38.0/packages/js) Updates `@typescript-eslint/eslint-plugin` from 8.46.0 to 8.46.1 - [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases) - [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/CHANGELOG.md) - [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v8.46.1/packages/eslint-plugin) Updates `@typescript-eslint/parser` from 8.46.0 to 8.46.1 - [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases) - [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/parser/CHANGELOG.md) - [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v8.46.1/packages/parser) Updates `esbuild` from 0.25.10 to 0.25.11 - [Release notes](https://github.com/evanw/esbuild/releases) - [Changelog](https://github.com/evanw/esbuild/blob/main/CHANGELOG.md) - [Commits](evanw/esbuild@v0.25.10...v0.25.11) --- updated-dependencies: - dependency-name: octokit dependency-version: 5.0.4 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: npm-minor - dependency-name: "@eslint/js" dependency-version: 9.38.0 dependency-type: direct:development update-type: version-update:semver-minor dependency-group: npm-minor - dependency-name: "@typescript-eslint/eslint-plugin" dependency-version: 8.46.1 dependency-type: direct:development update-type: version-update:semver-patch dependency-group: npm-minor - dependency-name: "@typescript-eslint/parser" dependency-version: 8.46.1 dependency-type: direct:development update-type: version-update:semver-patch dependency-group: npm-minor - dependency-name: esbuild dependency-version: 0.25.11 dependency-type: direct:development update-type: version-update:semver-patch dependency-group: npm-minor ... Signed-off-by: dependabot[bot] <support@github.com>
Bumps [actions/setup-node](https://github.com/actions/setup-node) from 5 to 6. - [Release notes](https://github.com/actions/setup-node/releases) - [Commits](actions/setup-node@v5...v6) --- updated-dependencies: - dependency-name: actions/setup-node dependency-version: '6' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
…thub/workflows/actions/setup-node-6 Bump actions/setup-node from 5 to 6 in /.github/workflows
…r-5ed6ededba Bump the npm-minor group with 5 updates
Handle user errors for invalid `UserConfig`s and missing query files
Bump timeout for `analyze-action-env` test
Update wording in some log messages
…-limit Overlay: Lower size limit for overlay base databases
Perform SARIF post-processing independently of upload
Also add tests for it and `getRequiredEnvParam`
Bump minimum CodeQL Bundle version to 2.17.6
Add `getOptionalEnvVar` helper
Add changelog entry for post-processing change
Contributor
There was a problem hiding this comment.
Pull Request Overview
This is a release PR merging changes from main into releases/v4 for version 4.31.0. The PR includes multiple merged pull requests that implement improvements to SARIF file processing, error handling, dependency updates, and various code quality enhancements.
Key changes:
- Bumped minimum CodeQL bundle version from 2.16.6 to 2.17.6
- Refactored SARIF upload logic to always perform post-processing, even when uploads are disabled
- Added new
post-processed-sarif-pathinput to the analyze action for saving post-processed SARIF files
Reviewed Changes
Copilot reviewed 63 out of 66 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| package.json | Version bump to 4.31.0 and dependency updates |
| CHANGELOG.md | Release notes for version 4.31.0 |
| src/upload-sarif.ts | Renamed function and added post-processing support with upload kind parameter |
| src/upload-lib.ts | Refactored to separate post-processing from upload logic |
| src/analyze-action.ts | Updated to use new upload architecture with conditional upload |
| src/util.ts | Added asHTTPError helper and getOptionalEnvVar function |
| src/codeql.ts | Bumped minimum CodeQL version constant |
| src/workflow.ts, src/tar.ts, etc. | Applied optional chaining improvements |
| analyze/action.yml | Added new post-processed-sarif-path input |
| .github/workflows/*.yml | Updated setup-node action from v5 to v6 |
| lib/*.js | Generated JavaScript from TypeScript sources |
mbg
approved these changes
Oct 24, 2025
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Merging 1d36546 into
releases/v4.Conductor for this PR is @mbg.
Contains the following pull requests:
UserConfigs and missing query files #3203 (@mbg)uploadSarifrather thanuploadFilesinanalyzeaction #3206 (@mbg)AnalysisKind.CodeScanningis not enabled #3221 (@mbg)analyze-action-envtest #3228 (@mbg)getOptionalEnvVarhelper #3233 (@mbg)Please do the following:
releases/v4branch.Create a merge commitis selected rather thanSquash and mergeorRebase and merge.