Skip to content

SCANNPM-134 Fix build#416

Merged
zglicz merged 1 commit intomasterfrom
fix-build
Feb 9, 2026
Merged

SCANNPM-134 Fix build#416
zglicz merged 1 commit intomasterfrom
fix-build

Conversation

@vdiez
Copy link
Contributor

@vdiez vdiez commented Feb 9, 2026

Part of

@vdiez vdiez requested a review from zglicz February 9, 2026 09:27
@hashicorp-vault-sonar-prod hashicorp-vault-sonar-prod bot changed the title Fix build SCANNPM-134 Fix build Feb 9, 2026
@hashicorp-vault-sonar-prod
Copy link

hashicorp-vault-sonar-prod bot commented Feb 9, 2026

SCANNPM-134

@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 9, 2026

SonarQube reviewer guide

Review in SonarQube

Summary: Updates dependency versions and npm registry configuration, primarily downgrading ESLint from 10.0.0 to 9.39.2 and switching package resolution to JFrog Artifactory.

Review Focus: The ESLint downgrade from 10.0.0 to 9.39.2 is the most significant change and warrants verification—confirm this is intentional and not a regression. Additionally, note the systematic registry URL changes from npmjs.org to JFrog Artifactory across all packages, which affects dependency resolution. Several peer dependency markers were removed (Babel, acorn, ESLint, Prettier, TypeScript, nyc, jiti), potentially altering how these dependencies are managed.

Start review at: package.json. This is the source of truth for the ESLint downgrade and should clarify the intent behind reverting from the major version 10.0.0 to 9.39.2, ensuring this wasn't accidental.

💬 Please send your feedback

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues
0 Dependency risks

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@zglicz zglicz merged commit 69aafc5 into master Feb 9, 2026
9 checks passed
@zglicz zglicz deleted the fix-build branch February 9, 2026 09:57
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