Skip to content

Upgrade CodeQL CLI dependency to v2.25.1#344

Merged
data-douser merged 2 commits intomainfrom
codeql/upgrade-to-v2.25.1
Mar 30, 2026
Merged

Upgrade CodeQL CLI dependency to v2.25.1#344
data-douser merged 2 commits intomainfrom
codeql/upgrade-to-v2.25.1

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions github-actions bot commented Mar 28, 2026

This PR upgrades the CodeQL CLI version to v2.25.1.

Changes made:

  • Updated qlt.conf.json (CodeQLCLI, CodeQLStandardLibrary, CodeQLCLIBundle) to 2.25.1
  • Updated all version-bearing qlpack.yml files to 2.25.1
  • Upgraded CodeQL pack lock files
  • Compiled CAP CDS files
  • CodeQL unit tests passed ✅

To complete the release, merge this PR and then trigger the release workflow
via workflow_dispatch on release.yml with version v2.25.1.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Upgrades the repository’s CodeQL CLI and related CodeQL packs to v2.25.1, updating pack metadata and regenerating dependency lockfiles across the JavaScript frameworks and heuristic models.

Changes:

  • Bump CodeQL CLI and bundle references to 2.25.1 in qlt.conf.json.
  • Update CodeQL pack versions and pin external pack dependencies (e.g., codeql/javascript-all, codeql/javascript-queries) to resolved exact versions.
  • Regenerate codeql-pack.lock.yml files across packs to reflect the upgraded dependency graph.

Reviewed changes

Copilot reviewed 29 out of 29 changed files in this pull request and generated 10 comments.

Show a summary per file
File Description
scripts/upgrade-packs.sh Enhances pack-upgrade automation by pinning external dependency versions and excluding .codeql/ cache directories.
scripts/update-release-version.sh Improves internal dependency version rewriting to handle additional qlpack.yml formats.
qlt.conf.json Updates CodeQL CLI / standard library / bundle version references to 2.25.1.
javascript/heuristic-models/tests/qlpack.yml Bumps test pack version and pins codeql/javascript-all.
javascript/heuristic-models/tests/codeql-pack.lock.yml Updates resolved external dependency versions in lockfile.
javascript/heuristic-models/ext/qlpack.yml Bumps extension pack version and pins codeql/javascript-all in extensionTargets.
javascript/frameworks/xsjs/test/qlpack.yml Bumps test pack version and pins codeql/javascript-all.
javascript/frameworks/xsjs/test/codeql-pack.lock.yml Updates resolved external dependency versions in lockfile.
javascript/frameworks/xsjs/src/qlpack.yml Bumps queries pack version and pins codeql/javascript-all.
javascript/frameworks/xsjs/src/codeql-pack.lock.yml Updates resolved external dependency versions in lockfile.
javascript/frameworks/xsjs/lib/qlpack.yml Bumps library pack version and pins codeql/javascript-all.
javascript/frameworks/xsjs/lib/codeql-pack.lock.yml Updates resolved external dependency versions in lockfile.
javascript/frameworks/xsjs/ext/qlpack.yml Bumps models pack version and pins codeql/javascript-all in extensionTargets.
javascript/frameworks/ui5/test/qlpack.yml Bumps test pack version and pins codeql/javascript-all / codeql/javascript-queries.
javascript/frameworks/ui5/test/codeql-pack.lock.yml Updates resolved external dependency versions in lockfile.
javascript/frameworks/ui5/src/qlpack.yml Bumps queries pack version and pins codeql/javascript-all.
javascript/frameworks/ui5/src/codeql-pack.lock.yml Updates resolved external dependency versions in lockfile.
javascript/frameworks/ui5/lib/qlpack.yml Bumps library pack version and pins codeql/javascript-all.
javascript/frameworks/ui5/lib/codeql-pack.lock.yml Updates resolved external dependency versions in lockfile.
javascript/frameworks/ui5/ext/qlpack.yml Bumps models pack version and pins codeql/javascript-all in extensionTargets.
javascript/frameworks/ui5-webcomponents/test/qlpack.yml Bumps test pack version and pins codeql/javascript-all.
javascript/frameworks/ui5-webcomponents/test/codeql-pack.lock.yml Updates resolved external dependency versions in lockfile.
javascript/frameworks/cap/test/qlpack.yml Bumps test pack version and pins codeql/javascript-all.
javascript/frameworks/cap/test/codeql-pack.lock.yml Updates resolved external dependency versions in lockfile.
javascript/frameworks/cap/src/qlpack.yml Bumps queries pack version and pins codeql/javascript-all.
javascript/frameworks/cap/src/codeql-pack.lock.yml Updates resolved external dependency versions in lockfile.
javascript/frameworks/cap/lib/qlpack.yml Bumps library pack version and pins codeql/javascript-all.
javascript/frameworks/cap/lib/codeql-pack.lock.yml Updates resolved external dependency versions in lockfile.
javascript/frameworks/cap/ext/qlpack.yml Bumps models pack version and pins codeql/javascript-all in extensionTargets.
Comments suppressed due to low confidence (1)

javascript/frameworks/xsjs/test/qlpack.yml:10

  • This test pack is versioned 2.25.1 but its internal dependencies still reference 2.25.0 (xsjs-queries/xsjs-all/xsjs-models). Update these internal dependency versions to 2.25.1 (or an intended compatible range) to prevent tests from resolving older internal packs.
dependencies:
  codeql/javascript-all: "2.6.26"
  advanced-security/javascript-sap-xsjs-queries: "2.25.0"
  advanced-security/javascript-sap-xsjs-all: "2.25.0"
  advanced-security/javascript-sap-xsjs-models: "2.25.0"

Comment thread scripts/upgrade-packs.sh Outdated
Comment thread scripts/upgrade-packs.sh Outdated
Comment thread javascript/frameworks/ui5/src/qlpack.yml
Comment thread javascript/frameworks/xsjs/src/qlpack.yml
Comment thread scripts/upgrade-packs.sh Outdated
Comment thread javascript/frameworks/cap/src/qlpack.yml
Comment thread javascript/frameworks/cap/test/qlpack.yml
Comment thread javascript/frameworks/ui5/test/qlpack.yml
Comment thread javascript/frameworks/ui5-webcomponents/test/qlpack.yml
Comment thread javascript/heuristic-models/tests/qlpack.yml
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 30 out of 30 changed files in this pull request and generated 3 comments.

Comment thread scripts/upgrade-packs.sh Outdated
Comment thread scripts/install-packs.sh Outdated
Comment thread scripts/update-release-version.sh Outdated
@data-douser
Copy link
Copy Markdown
Collaborator

@copilot apply changes based on the comments in this thread

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 29, 2026

@copilot apply changes based on the comments in this thread

Applied all three changes from the review thread in commit 3d4c5f7:

  1. scripts/upgrade-packs.sh: Replaced -not -path "*/.codeql/*" with -path '*/.codeql' -prune -o so find no longer traverses into .codeql directories.
  2. scripts/install-packs.sh: Same -prune optimization applied.
  3. scripts/update-release-version.sh: Updated the header comment on line 27 from (e.g., ^X.Y.Z constraints) to (exact X.Y.Z versions) to accurately reflect what update_internal_deps actually handles.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 30 out of 30 changed files in this pull request and generated 3 comments.

Comment thread scripts/upgrade-packs.sh Outdated
Comment thread javascript/frameworks/ui5/test/qlpack.yml
Comment thread scripts/upgrade-packs.sh Outdated
@data-douser
Copy link
Copy Markdown
Collaborator

@copilot apply changes based on the comments in this thread

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 29, 2026

@copilot apply changes based on the comments in this thread

Applied all three changes from the review thread in commit 5c58461:

  1. scripts/upgrade-packs.sh:127-145: Added trap 'rm -rf "${tmp_dir}"' RETURN immediately after mktemp -d, and cleared it with trap - RETURN after the explicit cleanup, ensuring the temp dir is removed even if codeql pack upgrade fails.
  2. javascript/frameworks/ui5/test/qlpack.yml:7-8: Updated the stale comment from "allow any version >= 1.2.0" to reflect that the dependency is now pinned to a specific version for consistent and reproducible test results.
  3. scripts/upgrade-packs.sh:160-165: Added escaping of dep_name and dep_old_value for safe use in sed search patterns, preventing mismatches or unintended substitutions when values contain regex metacharacters (e.g., ., *, /).

@data-douser data-douser self-assigned this Mar 29, 2026
@data-douser data-douser added enhancement New feature or request dependencies Pull requests that update a dependency file labels Mar 29, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 30 out of 30 changed files in this pull request and generated 3 comments.

Comment thread scripts/upgrade-packs.sh Outdated
Comment thread scripts/upgrade-packs.sh Outdated
Comment thread scripts/upgrade-packs.sh Outdated
@data-douser data-douser added the bug Something isn't working label Mar 29, 2026
@github-actions github-actions bot force-pushed the codeql/upgrade-to-v2.25.1 branch from 64f2097 to 63d0849 Compare March 30, 2026 01:25
@data-douser data-douser merged commit 4c21292 into main Mar 30, 2026
17 checks passed
@data-douser data-douser deleted the codeql/upgrade-to-v2.25.1 branch March 30, 2026 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working dependencies Pull requests that update a dependency file enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants