Skip to content

Comments

Fixes for v2.24.2 release prep#75

Merged
data-douser merged 3 commits intomainfrom
dd/release-prep/2
Feb 23, 2026
Merged

Fixes for v2.24.2 release prep#75
data-douser merged 3 commits intomainfrom
dd/release-prep/2

Conversation

@data-douser
Copy link
Collaborator

This pull request updates the CodeQL CLI version management process to consistently include the extensions/vscode/package.json file across documentation, scripts, and workflows. It also performs a minor dependency update for ESLint in several package.json files. The most important changes are grouped below:

Versioning Process Improvements:

  • Updated documentation (SKILL.md) to clarify that extensions/vscode/package.json is included in the CLI-aligned versioning strategy and to use generic version placeholders (e.g., X.Y.Z) for clarity. [1] [2] [3]
  • Modified the update workflow (update-codeql.yml) to explicitly mention and summarize changes to extensions/vscode/package.json alongside other version-bearing files. [1] [2]
  • Updated the update-release-version.sh script to detect and update the version in extensions/vscode/package.json in addition to other package.json files. [1] [2] [3]

Dependency Updates:

  • Bumped eslint from ^10.0.0 to ^10.0.1 in package.json, client/package.json, server/package.json, and extensions/vscode/package.json to ensure consistency and up-to-date linting across the codebase. [1] [2] [3] [4]

Version Bump:

  • Updated the version in extensions/vscode/package.json from 2.24.1 to 2.24.2 to reflect the new CLI version.

@data-douser data-douser self-assigned this Feb 23, 2026
Copilot AI review requested due to automatic review settings February 23, 2026 13:08
@data-douser data-douser added bug Something isn't working enhancement New feature or request labels Feb 23, 2026

Choose a reason for hiding this comment

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

@data-douser has requested to close this alert

Choose a reason for hiding this comment

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

@data-douser has requested to close this alert

@github-actions
Copy link
Contributor

Dependency Review

The following issues were found:
  • ✅ 0 vulnerable package(s)
  • ✅ 0 package(s) with incompatible licenses
  • ✅ 0 package(s) with invalid SPDX license definitions
  • ⚠️ 1 package(s) with unknown licenses.
See the Details below.

License Issues

package-lock.json

PackageVersionLicenseIssue Type
extensions/vscode2.24.2NullUnknown License

OpenSSF Scorecard

Scorecard details
PackageVersionScoreDetails
npm/eslint ^10.0.1 UnknownUnknown
npm/eslint ^10.0.1 UnknownUnknown
npm/minimatch 3.1.3 🟢 5
Details
CheckScoreReason
Code-Review⚠️ 1Found 4/28 approved changesets -- score normalized to 1
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Binary-Artifacts🟢 10no binaries found in the repo
Maintained🟢 910 commit(s) and 1 issue activity found in the last 90 days -- score normalized to 9
Token-Permissions⚠️ 0detected GitHub workflow tokens with excessive permissions
Packaging⚠️ -1packaging workflow not detected
Pinned-Dependencies⚠️ 0dependency not pinned by hash detected -- score normalized to 0
CII-Best-Practices⚠️ 0no effort to earn an OpenSSF best practices badge detected
Fuzzing⚠️ 0project is not fuzzed
License🟢 10license file detected
Signed-Releases⚠️ -1no releases found
Security-Policy🟢 10security policy file detected
Branch-Protection⚠️ -1internal error: error during branchesHandler.setup: internal error: some github tokens can't read classic branch protection rules: https://github.com/ossf/scorecard-action/blob/main/docs/authentication/fine-grained-auth-token.md
SAST⚠️ 0SAST tool is not run on all commits -- score normalized to 0
npm/@eslint/config-array 0.23.2 UnknownUnknown
npm/@eslint/object-schema 3.0.2 UnknownUnknown
npm/ajv 6.14.0 🟢 5.2
Details
CheckScoreReason
Code-Review🟢 9Found 27/30 approved changesets -- score normalized to 9
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Maintained⚠️ 22 commit(s) and 1 issue activity found in the last 90 days -- score normalized to 2
Security-Policy🟢 10security policy file detected
Token-Permissions⚠️ 0detected GitHub workflow tokens with excessive permissions
Packaging⚠️ -1packaging workflow not detected
Binary-Artifacts🟢 10no binaries found in the repo
Pinned-Dependencies⚠️ 0dependency not pinned by hash detected -- score normalized to 0
CII-Best-Practices⚠️ 2badge detected: InProgress
License🟢 10license file detected
Fuzzing⚠️ 0project is not fuzzed
Signed-Releases⚠️ -1no releases found
Branch-Protection⚠️ -1internal error: error during branchesHandler.setup: internal error: some github tokens can't read classic branch protection rules: https://github.com/ossf/scorecard-action/blob/main/docs/authentication/fine-grained-auth-token.md
SAST⚠️ 0SAST tool is not run on all commits -- score normalized to 0
npm/eslint 10.0.1 🟢 6.3
Details
CheckScoreReason
Maintained🟢 1030 commit(s) and 18 issue activity found in the last 90 days -- score normalized to 10
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Code-Review🟢 6Found 18/27 approved changesets -- score normalized to 6
Packaging⚠️ -1packaging workflow not detected
CII-Best-Practices⚠️ 0no effort to earn an OpenSSF best practices badge detected
Token-Permissions⚠️ 0detected GitHub workflow tokens with excessive permissions
License🟢 10license file detected
Binary-Artifacts🟢 10no binaries found in the repo
Pinned-Dependencies⚠️ 1dependency not pinned by hash detected -- score normalized to 1
Branch-Protection🟢 5branch protection is not maximal on development and all release branches
Signed-Releases⚠️ -1no releases found
Security-Policy🟢 10security policy file detected
Fuzzing⚠️ 0project is not fuzzed
SAST🟢 9SAST tool detected but not run on all commits
npm/eslint-scope 9.1.1 UnknownUnknown
npm/eslint-visitor-keys 5.0.1 UnknownUnknown
npm/espree 11.1.1 UnknownUnknown
npm/extensions/vscode 2.24.2 UnknownUnknown
npm/hono 4.12.2 UnknownUnknown
npm/eslint ^10.0.1 UnknownUnknown

Scanned Files

  • client/package.json
  • extensions/vscode/package.json
  • package-lock.json
  • server/package.json

Copy link
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

This PR enhances the CodeQL CLI version management infrastructure to consistently track and update the extensions/vscode/package.json file alongside other version-bearing files in the repository. It also performs a minor ESLint dependency update from ^10.0.0 to ^10.0.1 and bumps all versions from 2.24.1 to 2.24.2 to align with CodeQL CLI version 2.24.2.

Changes:

  • Enhanced version management to include extensions/vscode/package.json in the CLI-aligned versioning strategy across scripts, workflows, and documentation
  • Updated ESLint from ^10.0.0 to ^10.0.1 across all package.json files
  • Bumped version from 2.24.1 to 2.24.2 in all version-bearing files

Reviewed changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated no comments.

Show a summary per file
File Description
server/scripts/update-release-version.sh Added extensions/vscode/package.json to both collect_versions() and update_versions() functions for consistent version tracking
.github/workflows/update-codeql.yml Updated PR description template and summary table to explicitly mention extensions/vscode/package.json updates
.github/skills/upgrade-codeql-cli-and-packs/SKILL.md Updated documentation to clarify that extensions/vscode/package.json is included in versioning and replaced specific version examples with generic placeholders (X.Y.Z)
package.json Updated ESLint to ^10.0.1 and version to 2.24.2
client/package.json Updated ESLint to ^10.0.1 and version to 2.24.2
server/package.json Updated ESLint to ^10.0.1 and version to 2.24.2
extensions/vscode/package.json Updated ESLint to ^10.0.1 and version from 2.24.1 to 2.24.2
package-lock.json Regenerated with ESLint 10.0.1 and transitive dependency updates, all package versions set to 2.24.2

@data-douser data-douser marked this pull request as ready for review February 23, 2026 13:12
@data-douser data-douser requested review from a team and enyil as code owners February 23, 2026 13:12
@data-douser data-douser merged commit cb4e5a4 into main Feb 23, 2026
20 checks passed
@data-douser data-douser deleted the dd/release-prep/2 branch February 23, 2026 13:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant