This repository was archived by the owner on Sep 19, 2026. It is now read-only.
Conversation
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
@hraness/local-custodyto exact registry version0.6.0and refreshbun.lockf24680adad7934ba885afb0e8143acb5e61dda2f461acc52aaa91afb32c02f9dCustody engine evaluation
Adopted:
src/daemon/local-transport.ts: capability publication is async, uses supported atomic publish with no commit hook, and preserves the existingPromise<void>behaviorsrc/daemon/claude-host-tool-transport.ts: the async post-bind mode-0600 socket assertion is covered by Rust and does not treat a missing path as an expected outcomeDeferred:
src/daemon/local-transport.ts: file/socket validation and capability stable-read stay on the TypeScript implementation because stale cleanup and client availability classification depend on a rawENOENT; the sidecar authoritatively returnsCustodyError(stat/open) insteadsrc/daemon/claude-host-tool-transport.ts: stale-socket validation stays TypeScript becauseENOENTmeans there is nothing to unlinksrc/claude/account.ts: the bounded metadata stable-read stays TypeScript becauseENOENTmaps to an absent optional document while other custody failures fail closedNo directory assertion was forced onto the sidecar. Unsupported platforms, including Windows, retain local-custody's automatic TypeScript path.
Verification
bun test src/daemon/local-transport.test.ts src/daemon/local-transport-authenticated-entry.test.ts src/daemon/claude-host-tool-transport.test.ts src/claude/account.test.ts src/install-preflight.test.ts scripts/release-package-policy.test.ts— 93 passbun run check— support/cost/install/effect/security/lint/typecheck all pass; test aggregates pass (2,212 scripts, 160 local-efficiency, 36 cloud-efficiency, 6,168 src, 407 Convex, 190 site, 668 app); site/app/CLI builds and package-policy tests passad92074fromorigin/codex/credits-service;bun ./scripts/check-package.tsfails identically on cleanorigin/mainbun ./scripts/check-package.tspasses for exact commit3540f8din an isolated single-branch checkout containing only the governed candidate refs, verifying the measured package inventory and installed package lifecycleA prior aggregate attempt hit the known load-sensitive one-second readiness timeout in
scripts/bounded-process.test.ts; the exact test then passed on the candidate and in 21 focused clean-main runs. The final aggregate test run was clean.