Skip to content

harmonia-store-ref-scan: add README.md#1091

Merged
Mic92 merged 1 commit into
mainfrom
ref-scan-readme
Jun 23, 2026
Merged

harmonia-store-ref-scan: add README.md#1091
Mic92 merged 1 commit into
mainfrom
ref-scan-readme

Conversation

@Mic92

@Mic92 Mic92 commented Jun 23, 2026

Copy link
Copy Markdown
Member

Cargo.toml declares readme = "README.md" but the file did not exist,
which makes cargo package (and tooling that calls it, e.g. crane's
git-dep vendoring) fail with "readme README.md does not appear to
exist". Every other workspace member already ships its own README.

Cargo.toml declares readme = "README.md" but the file did not exist,
which makes `cargo package` (and tooling that calls it, e.g. crane's
git-dep vendoring) fail with "readme README.md does not appear to
exist".  Every other workspace member already ships its own README.
@Mic92 Mic92 enabled auto-merge June 23, 2026 12:51
@Mic92 Mic92 added this pull request to the merge queue Jun 23, 2026
Merged via the queue into main with commit 0760dbe Jun 23, 2026
3 checks passed
@Mic92 Mic92 deleted the ref-scan-readme branch June 23, 2026 12:52
@coderabbitai

coderabbitai Bot commented Jun 23, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: d352a8b9-4d11-4cec-996b-ba85bdd8b84d

📥 Commits

Reviewing files that changed from the base of the PR and between 773c2f5 and 1ec43ff.

📒 Files selected for processing (1)
  • harmonia-store-ref-scan/README.md

Walkthrough

A new README.md is added to the harmonia-store-ref-scan directory. It documents the crate as a streaming reference scanner for Nix store path outputs. The README describes the mechanism: arbitrary byte chunks (such as a NAR stream) are fed into the scanner after a build completes, and embedded store-path hash references are located using a Boyer-Moore-style window scan aligned with Nix's references.cc.

Possibly related PRs

  • nix-community/harmonia#1068: Introduces the harmonia-store-ref-scan crate itself — the implementation that this README now documents.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

Mic92 added a commit to Mic92/tribuchet that referenced this pull request Jun 23, 2026
rustPlatform.buildRustPackage rebuilds every crate in Cargo.lock on
any source change.  Splitting the build into crane's buildDepsOnly +
buildPackage keeps the ~300 dependency crates (incl. the harmonia git
deps and tonic-build) in their own derivation that only rebuilds when
Cargo.{toml,lock} change, taking incremental builds from ~3m to ~30s.

Bumps the harmonia rev past nix-community/harmonia#1091 so crane's
git-dep vendoring (which runs cargo package -l) no longer trips on
harmonia-store-ref-scan's missing README.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant