Skip to content

Harden identity protocol bootstrap#104

Merged
LiranCohen merged 1 commit into
mainfrom
fix/strict-protocol-bootstrap
May 27, 2026
Merged

Harden identity protocol bootstrap#104
LiranCohen merged 1 commit into
mainfrom
fix/strict-protocol-bootstrap

Conversation

@LiranCohen

Copy link
Copy Markdown
Contributor

Summary

  • retry dependency-ordered protocol bootstrap per DWN endpoint for freshly published did:dht DIDs
  • fail identity creation/import before profile writes if required remote protocol install cannot complete
  • clean up local identity state on bootstrap failure and guard against unpublished did:dht creation

Validation

  • bun run test -- src/enbox/tests/protocols.test.ts src/enbox/mutations/tests/identity-mutations.test.ts
  • bunx tsc -b

Context

This follows up merged PR #103. The deployed wallet can still hit a just-published did:dht propagation window where Fly DWN cannot resolve the new DID to verify the SocialGraph ProtocolsConfigure JWS. Previously the wallet logged that failure, continued to Profile, then wrote profile records locally, leaving the remote tenant without required protocols and producing identity shells without profile data on other wallets.

@github-actions

Copy link
Copy Markdown

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 40.1% (🎯 25%) 815 / 2032
🔵 Statements 38.76% (🎯 25%) 882 / 2275
🔵 Functions 41.32% (🎯 30%) 243 / 588
🔵 Branches 36.35% (🎯 29%) 537 / 1477
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
src/enbox/protocols.ts 95.12% 85.71% 100% 94.87% 56, 86-88
src/enbox/mutations/identity-mutations.ts 52.94% 28.84% 55.55% 54.54% 41, 162-166, 171-175, 189, 211-286, 298, 311-312, 330, 347-350
Generated in workflow #274 for commit 030a60c by the Vitest Coverage Report Action

@LiranCohen LiranCohen merged commit c6ac7c3 into main May 27, 2026
4 checks passed
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