Skip to content

Commit cfd5f47

Browse files
committed
ci: pin sfw uv sync to the locked dependency set on Dependabot review
`sfw uv sync` is the intended way to route uv through Socket Firewall (per Socket's own uv-wrapper guidance), so the python-sfw-smoke job was already exercising the firewall -- uv's integration is just quieter than npm/pip (no "N packages fetched" footer), which made it look like a no-op. Add `--locked` so the check verifies the exact uv.lock set and fails on lockfile drift instead of silently re-resolving to newer versions than the PR locked. This makes the firewall inspect precisely what would be installed and aligns with the deterministic-verification guidance for uv-based repos. Signed-off-by: lelia <2418071+lelia@users.noreply.github.com>
1 parent 5bde67f commit cfd5f47

1 file changed

Lines changed: 7 additions & 1 deletion

File tree

.github/workflows/dependabot-review.yml

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -109,7 +109,13 @@ jobs:
109109
run: python -m pip install --upgrade pip uv
110110

111111
- name: Sync project through Socket Firewall
112-
run: sfw uv sync --extra test --extra dev
112+
# `sfw uv sync` is the intended way to route uv through Socket Firewall
113+
# (per Socket's own uv wrapper guidance). --locked verifies the exact
114+
# uv.lock set and fails on lockfile drift rather than silently
115+
# re-resolving, so the firewall inspects precisely what would install.
116+
# Note: uv's sfw integration is quieter than npm/pip -- it does not
117+
# print the "N packages fetched" footer, but interception is active.
118+
run: sfw uv sync --locked --extra test --extra dev
113119

114120
- name: Import smoke test
115121
run: |

0 commit comments

Comments
 (0)