docs(wallet): connect backup, portability and recovery guidance - #599
Conversation
|
ty-everett
left a comment
There was a problem hiding this comment.
Reviewed the complete documentation-only diff at 279bf08920ad913c68f092a8135adae14fcab1fd, including the follow-up ownership column, linked responsibilities and clarified custody example. No actionable findings; 23 Markdown files, no executable source, manifest, dependency or runtime changes. Local health, lint, formatting, build, typecheck, documentation validation/build and security audit pass.
The maintainer explicitly requested immediate administrative merge without waiting for hosted CI. Updated-head checks remain in progress and are not claimed successful. This is the author/maintainer review record, not an independent approval. The existing docs-site workflow runs on merge; npm/image publication is outside scope.



A wallet can recover its root key and still lose the records needed to use its outputs. This documentation-only change makes that requirement prominent in the Wallet Toolbox README and connects a complete recovery learning path across TS Stack.
Program and scope
mainat40dad06ddc96760470daf7fb20e856b44431c71e. Exact reviewed head:279bf08920ad913c68f092a8135adae14fcab1fd.Impact
Verification
Official Node 24.18.0 / pnpm 10.33.2:
pnpm health:check,pnpm lint,pnpm format:check,pnpm build,pnpm typecheck: pass. Health reports zero contract findings/control errors; the existing expired Sonar exception is a maintenance warning, unchanged by this PR.pnpm audit:security: no known vulnerabilities.pnpm --filter docs-site validate: 128 source pages pass frontmatter/link validation.pnpm docs:build: 133 HTML pages pass built-link validation.pnpm --filter @bsv/wallet-toolbox exec jest --runInBand --watchman=false --runTestsByPath test/storage/portable.test.ts: 11 existing tests pass, covering canonical export, restore, merge/remapping, encrypted round-trip and rejection cases.git diff --checkpasses; Markdown-only scope verified.Security and dependencies
Dependency evidence
Not applicable: no dependency, lockfile, export, type, wire, schema or runtime change. No performance or bundle-size claim is made.
Release and operations
Completion evidence
Hosted runs for the updated head (pending at the authorized merge): CI, CodeQL, Conformance, runtime contracts.