Skip to content

Move package updaters to jfrog-cli-security#1350

Open
orto17 wants to merge 6 commits into
jfrog:v3_erfrom
orto17:move-package-updaters
Open

Move package updaters to jfrog-cli-security#1350
orto17 wants to merge 6 commits into
jfrog:v3_erfrom
orto17:move-package-updaters

Conversation

@orto17

@orto17 orto17 commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

Removes the packageupdaters/ directory from Frogbot — all implementations have been relocated to jfrog-cli-security/utils/remediation/packageupdaters/. Updates scanrepository to import and use the shared package directly. Removes stale testdata for all technologies (both unsupported ones like Yarn/Gradle/Nuget/Conan and the supported ones now covered by cli-security tests).

  • All tests passed. If this feature is not already covered by the tests, I added new tests.
  • This pull request is on the dev branch.
  • I used gofmt for formatting the code before submitting the pull request.
  • Update documentation about new features / new supported technologies

Depends on:

Removes the packageupdaters/ directory from Frogbot — all implementations have been
relocated to jfrog-cli-security/utils/remediation/packageupdaters/. Updates scanrepository
to import and use the shared package directly. Removes stale testdata for all technologies
(both unsupported ones like Yarn/Gradle/Nuget/Conan and the supported ones now covered by
cli-security tests).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- handleUpdatePackageErrors now checks both utils.ErrUnsupportedFix (legacy)
  and securitypkgupdaters.ErrUnsupportedFix (new shared type) so indirect-
  dependency skips are treated as expected non-errors rather than propagating
  as real failures
- Build FixDetails once in updatePackageToFixedVersion and reuse it for both
  the factory lookup and the UpdateDependency call

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@orto17 orto17 added the safe to test Approve running integration tests on a pull request label Jun 14, 2026
@github-actions github-actions Bot removed the safe to test Approve running integration tests on a pull request label Jun 14, 2026
@orto17 orto17 added the safe to test Approve running integration tests on a pull request label Jun 15, 2026
@github-actions github-actions Bot removed the safe to test Approve running integration tests on a pull request label Jun 15, 2026
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.

1 participant