Skip to content

fix: add missing OneKey module export parity across package targets#85

Merged
earrietadev merged 1 commit intoCreit-Tech:mainfrom
marcelosalloum:fix/missing-onekey-exports
Mar 4, 2026
Merged

fix: add missing OneKey module export parity across package targets#85
earrietadev merged 1 commit intoCreit-Tech:mainfrom
marcelosalloum:fix/missing-onekey-exports

Conversation

@marcelosalloum
Copy link
Contributor

  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

Bug fix + docs update.

  • What is the current behavior? (You can also link to an open issue here)

OneKey support exists in source (onekey.module.ts) and is included in defaultModules, but it was missing from some public/export surfaces. Specifically, src/deno.json and npm build entry points were not fully aligned, and wallet documentation lists were out of date.

These entries were missed from PR #80 .

  • What is the new behavior (if this is a feature change)?

OneKey is now consistently included in package export/entrypoint definitions and docs:

  • Added ./modules/onekey export mapping in src/deno.json

  • Added ./modules/onekey entry point in src/build_npm.ts

  • Added OneKey to supported wallets docs and README compatibility list

  • Other information:

No API behavior changes beyond exposing the already-implemented OneKey module through all documented/public paths.

@earrietadev earrietadev merged commit 355726e into Creit-Tech:main Mar 4, 2026
@marcelosalloum marcelosalloum deleted the fix/missing-onekey-exports branch March 4, 2026 17:14
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