Merged
Conversation
This commit updates the diff-range data extension to use the new checkPresence field being introduced in CodeQL CLI 2.21.0, so that diff-informed analysis no longer fails when a query pack does not have the restrictAlertsTo extensible predicate.
Mergeback v3.28.14 refs/heads/releases/v3 into main
Raise the file limit for debug artifacts by producing zip64 files where necessary
Set checkPresence in diff-range data extension
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR merges the latest changes from main into the releases/v3 branch, updating configuration values, replacing the zip creation library, and adding a new flag to the analyze extension. Key changes include:
- Updating the minimum version for the DiffInformedQueries feature flag and removing the now-unused toolsFeature property.
- Replacing AdmZip with archiver for creating debug artifact bundles.
- Adding a new checkPresence flag to the analyze extension.
Reviewed Changes
Copilot reviewed 43 out of 44 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| src/feature-flags.ts | Updated minimum version value and removed the toolsFeature property. |
| src/debug-artifacts.ts | Switched from AdmZip to archiver for zip compression and added error events. |
| src/analyze.ts | Added a new checkPresence flag in the extension configuration. |
| lib/feature-flags.js | Mirror update to feature flags similar to src/feature-flags.ts. |
| lib/debug-artifacts.js | Mirror switch to archiver for debug artifact bundling. |
| lib/analyze.js | Mirror update to include the checkPresence flag. |
| CHANGELOG.md | Added release notes for version 3.28.15. |
Files not reviewed (1)
- package.json: Language not supported
Comments suppressed due to low confidence (4)
src/feature-flags.ts:114
- The removal of the 'toolsFeature' property may lead to unexpected behavior if downstream logic still relies on it. Verify that its removal is intentional and that any dependent code is updated accordingly.
toolsFeature: ToolsFeature.DatabaseInterpretResultsSupportsSarifRunProperty,
lib/feature-flags.js:98
- The removal of the 'toolsFeature' property here could affect behavior if any dependent usage exists. Confirm the removal is deliberate and adjust dependent logic if needed.
toolsFeature: tools_features_1.ToolsFeature.DatabaseInterpretResultsSupportsSarifRunProperty,
src/debug-artifacts.ts:347
- Consider adding or updating tests to verify the new error and warning handling behavior introduced with archiver.
const output = fs.createWriteStream(databaseBundlePath);
lib/debug-artifacts.js:253
- Consider adding or updating tests to verify that the new archiver implementation properly handles errors and warnings, ensuring consistency with previous behavior.
const output = fs.createWriteStream(databaseBundlePath);
aeisenberg
approved these changes
Apr 7, 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 a35ae8c into
releases/v3.Conductor for this PR is @aeisenberg.
Contains the following pull requests:
Please do the following:
releases/v3branch.Create a merge commitis selected rather thanSquash and mergeorRebase and merge.