Skip to content

docs: document pre-initialize flow#807

Draft
cursor[bot] wants to merge 1 commit into
onekeyfrom
cursor/technical-documentation-updates-4a6b
Draft

docs: document pre-initialize flow#807
cursor[bot] wants to merge 1 commit into
onekeyfrom
cursor/technical-documentation-updates-4a6b

Conversation

@cursor
Copy link
Copy Markdown

@cursor cursor Bot commented May 25, 2026

Summary

  • update docs/architecture.md with a source-backed section for the new preInitialize() warm-up flow
  • fix outdated package references on the same page (@onekeyfe/hd-core, @onekeyfe/hd-transport-web-device)

Docs updated

  • docs/architecture.md

Codepaths covered

  • packages/core/src/api/device/PreInitialize.ts
  • packages/core/src/core/index.ts
  • packages/core/src/device/Device.ts
  • packages/core/src/api/BaseMethod.ts
  • packages/core/src/types/params.ts
  • packages/core/src/inject.ts

Knowledge gaps addressed

  • when to call preInitialize() versus when to set usePreInitialize
  • hit/miss conditions for skipping Initialize
  • TTL, context matching, and disconnect invalidation behavior
  • concurrency, deduplication, and the relevant [PRE-INIT] troubleshooting logs
  • clarified that warm-up is a best-effort optimization and does not bypass the normal safety checks
Open in Web View Automation 

Co-authored-by: Leon <lixiao.dev@gmail.com>
@revan-zhang
Copy link
Copy Markdown
Contributor

revan-zhang commented May 25, 2026

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

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.

2 participants