NativeLink Rust Change Workflow
Use this skill for Rust implementation work in nativelink/.
Read The Local Contract First
Before editing, inspect the trait, caller, and tests around the change. NativeLink has several shared contracts where small behavior changes can affect remote execution correctness.
Crate map:
nativelink-store/: storage backends, CAS/AC, compression, Redis/S3/GCS/filesystem/memory stores.nativelink-scheduler/: scheduling, worker matching, action queues, awaited action state.nativelink-worker/: worker execution, input materialization, process execution, output upload.nativelink-service/: REAPI, bytestream, BEP, worker API, health checks.nativelink-config/: JSON5 config structs and validation.nativelink-util/: digest, filesystem, proto, retry, metrics, TLS, and async helpers.nativelink-proto/: protobuf-generated API surfaces.
Implementation Rules
- Prefer existing traits and helper types over new abstractions.
- Preserve error semantics. Map user/config/input mistakes to explicit error codes and reserve internal errors for unexpected failures.
- Avoid blocking filesystem, process, or network work inside async code unless the surrounding code already does so intentionally.
- Keep metrics labels bounded and consistent with existing metric names.
- For storage and scheduler changes, think through cancellation, retries, duplicate work, and missing CAS entries.
- For worker changes, preserve stdout/stderr, exit status, timeouts, and output upload behavior.
- For service changes, preserve protocol compatibility and avoid changing response shape without tests.
Tests
Add or update the closest existing tests first. Prefer focused tests that prove the contract:
bazel test //nativelink-store/...
bazel test //nativelink-scheduler/...
bazel test //nativelink-worker/...
bazel test //nativelink-service/...
For narrow Cargo-side checks:
cargo test --all --profile=smol
For server-wide confidence:
bazel build //:nativelink
bazel test //...
Final Report
Summarize:
- The behavior changed.
- Files touched.
- Focused tests run.
- Broader tests skipped or still needed.
1---2name: nativelink-rust-change3description: Use when implementing or reviewing Rust changes in NativeLink crates, especially storage, scheduler, worker, service, config, and shared utility behavior.4---56# NativeLink Rust Change Workflow78Use this skill for Rust implementation work in `nativelink/`.910## Read The Local Contract First1112Before editing, inspect the trait, caller, and tests around the change. NativeLink has several shared contracts where small behavior changes can affect remote execution correctness.1314Crate map:1516- `nativelink-store/`: storage backends, CAS/AC, compression, Redis/S3/GCS/filesystem/memory stores.17- `nativelink-scheduler/`: scheduling, worker matching, action queues, awaited action state.18- `nativelink-worker/`: worker execution, input materialization, process execution, output upload.19- `nativelink-service/`: REAPI, bytestream, BEP, worker API, health checks.20- `nativelink-config/`: JSON5 config structs and validation.21- `nativelink-util/`: digest, filesystem, proto, retry, metrics, TLS, and async helpers.22- `nativelink-proto/`: protobuf-generated API surfaces.2324## Implementation Rules2526- Prefer existing traits and helper types over new abstractions.27- Preserve error semantics. Map user/config/input mistakes to explicit error codes and reserve internal errors for unexpected failures.28- Avoid blocking filesystem, process, or network work inside async code unless the surrounding code already does so intentionally.29- Keep metrics labels bounded and consistent with existing metric names.30- For storage and scheduler changes, think through cancellation, retries, duplicate work, and missing CAS entries.31- For worker changes, preserve stdout/stderr, exit status, timeouts, and output upload behavior.32- For service changes, preserve protocol compatibility and avoid changing response shape without tests.3334## Tests3536Add or update the closest existing tests first. Prefer focused tests that prove the contract:3738```bash39bazel test //nativelink-store/...40bazel test //nativelink-scheduler/...41bazel test //nativelink-worker/...42bazel test //nativelink-service/...43```4445For narrow Cargo-side checks:4647```bash48cargo test --all --profile=smol49```5051For server-wide confidence:5253```bash54bazel build //:nativelink55bazel test //...56```5758## Final Report5960Summarize:6162- The behavior changed.63- Files touched.64- Focused tests run.65- Broader tests skipped or still needed.
tracemachina