Skip to content

XRAY-157127 - Keep each package manager resolving against its own Artifactory repo - #849

Draft
Jordanh1996 wants to merge 1 commit into
devfrom
XRAY-157127-isolate-polyglot-resolvers
Draft

XRAY-157127 - Keep each package manager resolving against its own Artifactory repo#849
Jordanh1996 wants to merge 1 commit into
devfrom
XRAY-157127-isolate-polyglot-resolvers

Conversation

@Jordanh1996

Copy link
Copy Markdown
Contributor

https://jfrog-int.atlassian.net/browse/XRAY-157127

Background

Polyglot CI audits with distinct Go/npm (and other) resolver configs intermittently resolved one technology against another's Artifactory repository, producing flaky 404s depending on which technology ran first in the same jf audit process.

Description

BuildInfoBomGenerator reused one shared params across technologies; SetResolutionRepoInParamsIfExists wrote the first tech's repo into that field, so later techs skipped their projects/<tech>.yaml. Each tech now resolves through resolveTechParams against a per-call copy, leaving the generator params clean while --deps-repo / IgnoreConfigFile still short-circuit the lookup.

Tests

Unit: TestResolveTechParamsIsolation, TestResolveTechParamsSkipsConfigLookup, TestBuildDependencyTreeDoesNotMutateGeneratorParams.
Local e2e: polyglot go+npm fixture on the local stack — before npm hit go-vir; after it hit npm-remote.

  • The pull request is targeting the dev branch.
  • The code has been validated to compile successfully by running go vet ./....
  • The code has been formatted properly using go fmt ./....
  • All static analysis checks passed.
  • All tests have passed. If this feature is not already covered by the tests, new tests have been added.
  • Updated the Contributing page / ReadMe page / CI Workflow files if needed.
  • All changes are detailed at the description. if not already covered at JFrog Documentation, new documentation have been added.

Made with Cursor

Co-authored-by: Cursor <cursoragent@cursor.com>
@Jordanh1996 Jordanh1996 added bug Something isn't working safe to test Approve running integration tests on a pull request labels Aug 13, 2026
@github-actions github-actions Bot removed the safe to test Approve running integration tests on a pull request label Aug 13, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant