Skip to content

XRAY-145828 - Uv curation audit fix - #925

Merged
gauriy-tech merged 2 commits into
jfrog:devfrom
Phavya-jfrog:bugfix/XRAY-145828-uv
Oct 22, 2026
Merged

gauriy-tech merged 2 commits into
jfrog:devfrom
Phavya-jfrog:bugfix/XRAY-145828-uv

Conversation

@Phavya-jfrog

@Phavya-jfrog Phavya-jfrog commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor
  • 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.

Problem

PR #880 removed ~/.config/uv/uv.toml as a uv signal, which caused a regression in jf curation-audit. For uv, uv.toml is the native config file — if it's there, the project is uv.

Fix

Added it back, but only for jf curation-audit (jf audit/scan unchanged).

Before:

  • Before: jf ca fails with "no pip config found"
Screenshot 2026-10-21 at 10 34 52 AM

After:

  • After: correctly detected as uv, reports blocked packages
Screenshot 2026-10-21 at 10 35 11 AM

@coderabbitai

coderabbitai Bot commented Oct 21, 2026 •

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Enterprise
  • Run ID: 0b39d895-b168-438c-89c2-3ad5613eff6b

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

@Phavya-jfrog Phavya-jfrog added bug Something isn't working safe to test Approve running integration tests on a pull request labels Oct 21, 2026
@github-actions github-actions Bot removed the safe to test Approve running integration tests on a pull request label Oct 21, 2026
@Phavya-jfrog
Phavya-jfrog force-pushed the bugfix/XRAY-145828-uv branch from 50e2f4d to 93ac9cb Compare October 21, 2026 05:36
@Phavya-jfrog Phavya-jfrog added the safe to test Approve running integration tests on a pull request label Oct 21, 2026
@github-actions github-actions Bot removed the safe to test Approve running integration tests on a pull request label Oct 21, 2026
@Phavya-jfrog
Phavya-jfrog force-pushed the bugfix/XRAY-145828-uv branch from 93ac9cb to f7a0276 Compare October 21, 2026 05:49
@Phavya-jfrog Phavya-jfrog added the safe to test Approve running integration tests on a pull request label Oct 21, 2026
@github-actions github-actions Bot removed the safe to test Approve running integration tests on a pull request label Oct 21, 2026
@Phavya-jfrog
Phavya-jfrog force-pushed the bugfix/XRAY-145828-uv branch from f7a0276 to 19fa70c Compare October 21, 2026 06:39
@Phavya-jfrog Phavya-jfrog added the safe to test Approve running integration tests on a pull request label Oct 21, 2026
@github-actions github-actions Bot removed the safe to test Approve running integration tests on a pull request label Oct 21, 2026
Comment thread sca/bom/buildinfo/technologies/common.go Outdated
@Phavya-jfrog
Phavya-jfrog force-pushed the bugfix/XRAY-145828-uv branch from 19fa70c to 9e6a867 Compare October 21, 2026 08:57
@Phavya-jfrog Phavya-jfrog added the safe to test Approve running integration tests on a pull request label Oct 21, 2026
@github-actions github-actions Bot removed the safe to test Approve running integration tests on a pull request label Oct 21, 2026
@Phavya-jfrog Phavya-jfrog added the safe to test Approve running integration tests on a pull request label Oct 21, 2026
@github-actions github-actions Bot removed the safe to test Approve running integration tests on a pull request label Oct 21, 2026
@Phavya-jfrog Phavya-jfrog added the safe to test Approve running integration tests on a pull request label Oct 21, 2026
@github-actions github-actions Bot removed the safe to test Approve running integration tests on a pull request label Oct 21, 2026
@Phavya-jfrog Phavya-jfrog added the safe to test Approve running integration tests on a pull request label Oct 22, 2026
@github-actions github-actions Bot removed the safe to test Approve running integration tests on a pull request label Oct 22, 2026
@github-actions

Copy link
Copy Markdown
Contributor

👍 Frogbot scanned this pull request and did not find any new security issues.


@Phavya-jfrog Phavya-jfrog added the safe to test Approve running integration tests on a pull request label Oct 22, 2026
@github-actions github-actions Bot removed the safe to test Approve running integration tests on a pull request label Oct 22, 2026
@gauriy-tech
gauriy-tech merged commit be59ed7 into jfrog:dev Oct 22, 2026
163 of 179 checks passed
@Phavya-jfrog
Phavya-jfrog deleted the bugfix/XRAY-145828-uv branch October 22, 2026 05:38

This branch was successfully deployed

1 active deployment
frogbot — 6d1cc3d6 Deployed Oct 22, 2026 by Phavya-jfrog via scan-pull-request #4721
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.

2 participants