Skip to content

Commit 4f73044

Browse files
committed
ci: declare least-privilege permissions on the deepnsm-wired workflows
Addresses CodeRabbit review on #479: style.yml and rust-test.yml had no explicit `permissions:` block, so they inherited the repo-default token scope. These jobs only checkout, build, lint, and test — declare `contents: read` so GITHUB_TOKEN is least-privilege. Codecov upload uses its own token secret and is non-fatal (fail_ci_if_error: false). Also re-triggers CI: the prior run's `test (stable)` failed on a transient rust-lld SIGBUS (signal 7) while linking lance-graph's datafusion test binaries — intermittent linker memory-pressure flake, unrelated to deepnsm (the same code linked fine in test-with-coverage; deepnsm fmt+clippy gates already passed). https://claude.ai/code/session_014A4JuRCqKP2yNENrQ9Ha7H
1 parent a36600b commit 4f73044

2 files changed

Lines changed: 9 additions & 0 deletions

File tree

.github/workflows/rust-test.yml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,11 @@ env:
2020
RUST_BACKTRACE: "1"
2121
CARGO_INCREMENTAL: "0"
2222

23+
# Least-privilege: these jobs only read the repo (checkout + build + test).
24+
# Codecov upload uses its own token secret and is non-fatal (fail_ci_if_error: false).
25+
permissions:
26+
contents: read
27+
2328
jobs:
2429
test:
2530
runs-on: ubuntu-24.04

.github/workflows/style.yml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,10 @@ env:
1818
CARGO_TERM_COLOR: always
1919
RUSTFLAGS: "-C debuginfo=1 -C target-cpu=x86-64-v3"
2020

21+
# Least-privilege: these jobs only read the repo (checkout + build + lint).
22+
permissions:
23+
contents: read
24+
2125
jobs:
2226
# Clippy runs FIRST and is mandatory — logical soundness before syntax.
2327
# Discipline:

0 commit comments

Comments
 (0)