Skip to content

CHANGE @W-22462041@ Update CLI dependencies to released core packages#2056

Merged
nikhil-mittal-165 merged 1 commit into
devfrom
nikhil.mittal/cli-dependency-upgrade-20260624
Jun 25, 2026
Merged

CHANGE @W-22462041@ Update CLI dependencies to released core packages#2056
nikhil-mittal-165 merged 1 commit into
devfrom
nikhil.mittal/cli-dependency-upgrade-20260624

Conversation

@nikhil-mittal-165

Copy link
Copy Markdown
Contributor

Summary

Updates CLI dependencies to use the newly released core packages from the monthly release.

Core Package Updates

  • code-analyzer-core: 0.48.0 → 0.49.0
  • code-analyzer-engine-api: 0.38.0 → 0.39.0
  • code-analyzer-pmd-engine: 0.41.0 → 0.43.0
  • code-analyzer-eslint-engine: 0.43.0 → 0.44.0
  • code-analyzer-flow-engine: 0.37.0 → 0.38.0
  • code-analyzer-regex-engine: 0.36.0 → 0.37.0
  • code-analyzer-retirejs-engine: 0.35.0 → 0.36.0
  • code-analyzer-sfge-engine: 0.21.0 → 0.22.0

Other Dependency Updates (minor/patch)

  • @oclif/core: 4.11.3 → 4.11.6
  • @oclif/table: 0.5.8 → 0.5.9
  • @oclif/plugin-help: 6.2.49 → 6.2.50
  • @salesforce/sf-plugins-core: 12.2.6 → 12.2.24
  • @salesforce/cli-plugins-testkit: 5.3.53 → 5.3.61
  • @vitest/coverage-istanbul: 4.1.7 → 4.1.9
  • vitest: 4.1.7 → 4.1.9
  • ansis: 4.3.0 → 4.3.1
  • js-yaml: 4.1.1 → 4.2.0
  • oclif: 4.23.7 → 4.23.16
  • typescript-eslint: 8.59.4 → 8.61.1

Test Fix

  • Added getEngineInsights method to test stubs to match updated RunResults interface

Skipped (Major Version Updates)

  • @salesforce/core (requires TypeScript config changes)
  • @eslint/js, eslint (v9 → v10)
  • @types/node (keep v20)
  • typescript (v5 → v6)
  • vite (v7 → v8)

Test Results

  • ✅ Build successful
  • ✅ All 380 tests passing
  • ✅ Lint passing
  • ✅ Coverage: 97.27% statements, 97.45% lines

Update all @salesforce/code-analyzer-* package dependencies to their newly released versions:
- code-analyzer-core: 0.48.0 → 0.49.0
- code-analyzer-engine-api: 0.38.0 → 0.39.0
- code-analyzer-pmd-engine: 0.41.0 → 0.43.0
- code-analyzer-eslint-engine: 0.43.0 → 0.44.0
- code-analyzer-flow-engine: 0.37.0 → 0.38.0
- code-analyzer-regex-engine: 0.36.0 → 0.37.0
- code-analyzer-retirejs-engine: 0.35.0 → 0.36.0
- code-analyzer-sfge-engine: 0.21.0 → 0.22.0

Update additional CLI dependencies (minor/patch):
- @oclif/core: 4.11.3 → 4.11.6
- @oclif/table: 0.5.8 → 0.5.9
- @oclif/plugin-help: 6.2.49 → 6.2.50
- @salesforce/sf-plugins-core: 12.2.6 → 12.2.24
- @salesforce/cli-plugins-testkit: 5.3.53 → 5.3.61
- @vitest/coverage-istanbul: 4.1.7 → 4.1.9
- vitest: 4.1.7 → 4.1.9
- ansis: 4.3.0 → 4.3.1
- js-yaml: 4.1.1 → 4.2.0
- oclif: 4.23.7 → 4.23.16
- typescript-eslint: 8.59.4 → 8.61.1

Fix test stub to implement new getEngineInsights method from RunResults interface.

Skipped major version updates:
- @salesforce/core (requires TypeScript config changes)
- @eslint/js, eslint (v9 → v10 major)
- @types/node (keep v20)
- typescript (v5 → v6 major)
- vite (v7 → v8 major)

All tests passing.

@aruntyagiTutu aruntyagiTutu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

APPROVED

Reviewed against all 13 Code Analyzer team standards - all areas PASS.

Why This Is Exemplary:

  • ✅ Complete PR description with all required sections
  • ✅ Core package updates documented: 0.48.0 → 0.49.0
  • ✅ Other dependency updates listed with versions
  • ✅ Test fix explained (getEngineInsights stub)
  • ✅ Skipped major versions documented with reasons
  • ✅ Test results: 380 passing, 97%+ coverage
  • ✅ Maintained frozen dependencies: @types/node at v20
  • ✅ No internal Nexus URLs - clean package-lock.json

This PR demonstrates perfect adherence to all Code Analyzer team standards and should be used as a template for future dependency update PRs.

13-Point Review: All applicable areas PASS ✅

@nikhil-mittal-165 nikhil-mittal-165 merged commit 024f620 into dev Jun 25, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants