Repository navigation
docs: drop the unshipped six-interface kernel and its two default packages (MAJOR: scope, Sean approved in chat; Sean merges) - #62
Conversation
…kages (Sean approved) The README, BUILDERS.md and CONTRIBUTING.md described a six-interface kernel (@hyperdag/protocol plus six default packages) that was designed in a May sprint and never shipped: the source sat on feat/modular-kernel-interfaces-2026-05-04, nothing was published to npm, and the "Known broken / not live" table listed things that did not exist. The two packages on main (packages/defaults/*) imported @hyperdag/interfaces, which is not in the repo, so they did not build from a clean clone; no other repository imports them. - README: says what is here (BUILDERS.md and the ERC-8004 team's contracts) and that nothing installs from it; drops the kernel section, the kernel API sample, the Known-broken table and the HAL parity row; the trust-model and threat-model tables name where each piece runs today (repid-engine, HyperDAG-core). - Receipts: one agent per row with the address that holds each token, re-read with ownerOf on Base Sepolia today (all 12 answer; 1 own address, 3 custodian, 8 deployer, as the 2026-10-05 check said). The old table put two agents per row. - BUILDERS.md, CONTRIBUTING.md, CLAUDE.md, the About proposal: no kernel. - Removed packages/defaults (hallucination-hal-local, identity-erc8004-viem), their dependabot entries, and check-map's read of the removed constants file; the registry addresses stay in BUILDERS.md and packages/contracts/README.md, and the existing map check still fails if BUILDERS stops saying we do not operate them. The May source is kept at the tag archive/modular-kernel-interfaces-2026-05-04. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Strix Security Review✅ No open security findings. PR summary: This PR removes the never-shipped six-interface trust kernel from the documentation and deletes the two unbuildable default packages under `packages/defaults/`, rewriting the README, BUILDERS.md and related docs so the repository is described as a builder's map plus the ERC-8004 registry contracts, with `@hyperdag/trustshell` as the only installable product. It also updates the README on-chain receipts table to one row per agent with the holding address. Review summaryReviewed all 33 changed paths: the README/BUILDERS/CONTRIBUTING/CLAUDE/GITHUB_ABOUT documentation rewrites, the two comment-only edits in No security-relevant code path is introduced by this change, and no security control is lost on a reachable path. The deleted packages were never workspace members (root Ruled out (3)
Review 1 on this PR · Reviewed by Strix |
Weight: MAJOR (r31: product scope). Sean approved this change in chat on 2026-10-24: remove the unshipped kernel from the README and BUILDERS, remove or fix the two packages that don't build, and tag the May branch so nothing is lost. Sean merges.
Why
The README described three things, one of which did not exist: a "six-interface kernel" (
@hyperdag/protocolplus six default packages). It was designed in a May sprint and never shipped:npm viewreturns 404 for@hyperdag/protocoland every default package.feat/modular-kernel-interfaces-2026-05-04(71 files), never merged.packages/protocolandpackages/interfaceswere never in git; only build output on one machine.The two packages that were on
main(packages/defaults/hallucination-hal-local,identity-erc8004-viem) were the kernel's defaults. Both declare@hyperdag/interfacesas a dependency, which does not exist; the HAL package's own test file says its build fails on a clean clone. No other repository imports either one (searched the default branches of trustshell, repid-engine, HyperDAG-core, trustrepid and trinity-symphony-shared; other repos not searched). So they are removed, not patched.What changes
README.mdREADME.mdReceiptsownerOfon Base Sepolia on 2026-10-24: all 12 answer; 1 at the agent's own address, 3 at one address, 8 at another, matching the 2026-10-05 statement's custodian/deployer counts.BUILDERS.md,CONTRIBUTING.md,CLAUDE.md,docs/living/GITHUB_ABOUT.mdCLAUDE.mdrecords the removal and the archive tag so agents stop describing it.packages/defaults/.github/dependabot.ymlscripts/check-map.mjspackages/contracts/README.md, and the existing map check already fails if BUILDERS stops saying the registries are the ERC-8004 team's and that we do not operate them..github/workflows/ci.ymlNothing is lost: tag
archive/modular-kernel-interfaces-2026-05-04→3fadb864, the branch tip ("Phase 8 — Smoke test passing, sprint complete"). The branch itself is untouched.Checks run locally
npm cifrom the lockfile: exit 0 (the lockfile never listedpackages/defaults/*; workspaces arepackages/*, one level).npm run check:builders: VERIFIED (1.8.0 = npm).check:mapagainst the four siblings' default branches (exported the way CI's sparse checkout lays them out): VERIFIED. Control: with "we do not operate them" removed from BUILDERS.md it FAILS, then passes again with it restored.check:sdk-claims: VERIFIED.Not in this PR
@hyperdag/trustshell-mcpon npm is 1.0.0 from 2026-07-08 and older than thetrustshell-mcpbin inside@hyperdag/trustshell. The README already says to use the bin. Deprecating the old package is an npm action for Sean.docs/living/GITHUB_ABOUT.md; changing it is a repo setting.packages/circuits/README.mdsays "In production, this directory would contain actual circuit files." That is a separate question.🤖 Generated with Claude Code