Skip to content

docs: clarify preInitialize fast path and Cardano limits#829

Draft
cursor[bot] wants to merge 1 commit into
onekeyfrom
cursor/engineering-documentation-updates-3455
Draft

docs: clarify preInitialize fast path and Cardano limits#829
cursor[bot] wants to merge 1 commit into
onekeyfrom
cursor/engineering-documentation-updates-3455

Conversation

@cursor

@cursor cursor Bot commented Jun 29, 2026

Copy link
Copy Markdown

Summary

  • update docs/architecture.md with a source-backed preInitialize fast-path guide
  • fix outdated package names in the same page to match current package.json names
  • document the current Cardano signing opt-out from usePreInitialize

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/cardano/CardanoSignTransaction.ts
  • packages/core/src/api/cardano/CardanoSignMessage.ts
  • packages/core/src/inject.ts
  • packages/core/src/types/api/preInitialize.ts
  • packages/core/src/types/params.ts

Knowledge gaps addressed

  • what HardwareSDK.preInitialize() actually does and returns
  • when usePreInitialize hits or misses, including log markers and miss reasons
  • how the 60s dedupe / TTL behavior works for repeated pre-warm calls
  • why Cardano signing should not rely on the pre-initialize fast path right now

Verification

  • docs-only change; verified against the source files listed above
Open in Web View Automation 

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

revan-zhang commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

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