8000 feat(rcache): expose RemoteCache::sha256 accessor by bryan-minimal · Pull Request #284 · gominimal/minimal · GitHub
[go: up one dir, main page]

Skip to content

feat(rcache): expose RemoteCache::sha256 accessor - #284

Merged
bryan-minimal merged 1 commit into
mainfrom
rcache-sha256-accessor
Jun 2, 2026
Merged

bryan-minimal merged 1 commit into
mainfrom
rcache-sha256-accessor

Conversation

@bryan-minimal
Copy link
Copy Markdown
Member

Adds RemoteCache::sha256(spec_hash) -> Option<[u8; 32]> — a thin passthrough to the content digest the index already stores (RemoteIndex::sha256), mirroring the existing exists. Lets a caller read a cached artifact's sha256 without materializing it.

Why: needed for our SLSA L3 provenance work — recording each resolved dependency's content digest (sha256) so the dependency closure is externally verifiable, not just identifiable by spec hash.

Unit test added (present → Some(hash), absent → None). cargo test -p rcache green.

Thin passthrough to the content digest the index already stores
(`RemoteIndex::sha256`), mirroring `exists`. Lets a caller read a cached
artifact's sha256 without materializing it — needed to record dependency
content digests in our SLSA provenance.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@coderabbitai
coderabbitai Bot commented Jun 2, 2026
Copy link
Copy Markdown
Contributor

Warning

Review limit reached

@bryan-minimal, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 44 minutes and 23 seconds. Learn how PR review limits work.

Your organization has run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: bb893e9c-6769-4821-bb41-a9636d8d3bbb

📥 Commits

Reviewing files that changed from the base of the PR and between 48eda35 and 95690a4.

📒 Files selected for processing (1)
  • crates/rcache/src/remote.rs

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

@bryan-minimal
bryan-minimal merged commit abdc894 into main Jun 2, 2026
15 of 16 checks passed
@bryan-minimal
bryan-minimal deleted the rcache-sha256-accessor branch June 2, 2026 14:38
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.

2 participants

0