Repository navigation
fix: say only what the code does (Inspect commit label), credit what we build on, one copyright holder (LOW + Sean-approved licence) - #619
Conversation
…ght holder Code first: - check.ts: the run check labelled the commit fact "The commit exists on the remote", but no lookup is done. It now says "GitHub reports the commit this run ran on" (dist rebuilt; only dist/lib/check.js changes). The label is in the evidence record, so the pinned digest moves, on purpose; recomputed independently with Python. Then the copy: - README: "The portable agentic trust harness" becomes what runs today (Inspect) and where it is going (Carry, reputation that moves with you), with a link to hyperdag-protocol/docs/WHY.md. "ZK-verifiable receipt" and "portable reputation" go; the range proof's score is public. Inspect is listed first. Credits for x402 (Coinbase) and Plonky3 (Polygon Labs); the ERC-8004 badge points at the EIP; the hub blurb replaces "Built on". - README: the commercial-use term the LICENSE does not contain is removed (Sean, 2026-10-23). - LICENSE appendix and README: copyright Sean Goodwin (Sean, 2026-10-23). - package.json description: Inspect first; ships with the next release. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Strix Security Review✅ No open security findings. PR summary: This PR corrects the Inspect check's commit label to say only what the code verifies, moves the corresponding pinned record digest, and rewrites README, LICENSE and package metadata to narrow overstated claims, credit upstream projects, and name a single copyright holder. Review summaryThe diff contains no executable-logic change beyond one user-facing label string, so the review focused on whether the new wording matches what the code actually does and whether the moved test pin weakens any integrity check. The commit label now states that GitHub reports the commit the run ran on, which is exactly what the code produces ( Review 1 on this PR · Reviewed by Strix |
There was a problem hiding this comment.
The diff contains no executable-logic change beyond one user-facing label string, so the review focused on whether the new wording matches what the code actually does and whether the moved test pin weakens any integrity check. The commit label now states that GitHub reports the commit the run ran on, which is exactly what the code produces (head_sha, with no commit lookup), and the compiled artifact matches the source. The updated record digest was independently recomputed from the recorded GitHub fixture through the record builder and matches the new pin, while the companion assertion that recomputes the JCS SHA-256 at runtime still guards the record format. The documentation edits narrow previously overstated assurance claims (ZK-verifiable receipts, portable earned RepID, cross-provider quorum) and add an accurate description of the digest, which reduces rather than creates misleading security claims; the new statement that Inspect's only network call is to api.github.com is consistent with the code. The licence and package metadata edits have no runtime effect. No security-relevant issue was found in the changed code.
Reviewed by Strix
Configure security review settings
🧾 Receipt1 claim(s) checked against GitHub's own check runs on
What this receipt does not prove
Receipts by TrustShell. Read-only: it reads this PR and its check runs, and writes only this comment. No code leaves the repo. |
Part of the 2026-10-23 repo-alignment sprint (hub: DealAppSeo/hyperdag-protocol#61). Code first, then copy.
Weight.
Code
src/lib/check.ts:261(run check, commit fact)head_shafor the rundist/lib/check.jsgit diff --ignore-cr-at-eol)tests/action-evidence.test.tspinned digestcb114733…9b7c9646…json.dumps(sort_keys=True)+ sha256 on the same record gives the same valueCopy
checksectionBUILDERS.md#how-the-pieces-fit)tests/readme-where-this-sits.test.ts)Checks
tsc --noEmit: clean.action-evidence,readme-truth,readme-where-this-sits,readme-first-screenandinstall-docs.mainand pass on Linux CI:kernel-client,measure-home-samples,publish-trusted-publishing,receipts-workflow,scrub-battery,stranger-rail.Not in this PR
🤖 Generated with Claude Code