build(nix): stop shipping 727MB of rust-docs in every CI job's closure

[daniel-noland]

Stacked on #1746 — review/merge that first.

fromRustupToolchain applies components as extensions on top of a profile,
and profile defaults to rustup's default, which already carries rust-docs.
So rust-docs was arriving twice, and deleting it from the component list alone
would have removed nothing.

Naming profile = "minimal" is what actually drops it. Verified by set-diffing
the joined component paths of the toolchain derivation across three variants:

variant	rust-docs in the joined paths
today ("rust-docs" listed, no profile)	yes, twice
component deleted, profile still unset	yes, still there
this PR (deleted + profile = "minimal")	no

A full diff of the deduplicated path sets between the last two shows exactly
one line: rust-docs. Nothing else about the toolchain moves.
passthru.availableComponents.rust-src — which default.nix uses in three
places — still resolves to the same store path.

Why bother

727MB of prebuilt HTML, in the closure of all 8 dev-shell jobs. The lab runners
are ephemeral (directory /nix does not exist; creating it every job), so it is
pulled over the network each time, not cached on disk. Measured against the
current dev shell: 5.96GB closure, 325 paths, ~22s to realize.

Nothing in the tree reads it — grep for rust-docs, share/doc/rust, and
rustup doc finds only this comment.

Cost

Every rust-built store path in the shell changes hash, so the first CI run after
this merges repopulates that half of Cachix.

🤖 Generated with Claude Code

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: c29b1cab-5089-49e0-a753-49cc40a14179

📥 Commits

Reviewing files that changed from the base of the PR and between 2a10caf and bfb86f8.

📒 Files selected for processing (1)

• nix/overlays/llvm.nix

Included review availability: 4 reviews are currently available. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour.

---

📝 Walkthrough

Walkthrough

Changes

Rust toolchain configuration

Layer / File(s)	Summary
Configure the minimal Rust toolchain nix/overlays/llvm.nix	The rustup toolchain now uses profile = "minimal" and no longer includes rust-docs. The channel and remaining components are unchanged.

Suggested reviewers: sergeymatov, fredi-raspall, mvachhar, qmonnet

Merge Risk: ⚪ Minimal · up to bfb86

The PR removes unused Rust documentation from CI toolchain closures, reducing download and storage cost without changing the remaining toolchain contents; no actionable merge-blocking risk remains beyond normal checks and review.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly summarizes the main change: removing the 727 MB rust-docs payload from CI job closures through the Nix Rust toolchain configuration.
Description check	✅ Passed	The description directly explains the profile change, component removal, validation results, closure-size reduction, cache impact, and dependency on PR #1746.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

---

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

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!