-
Notifications
You must be signed in to change notification settings - Fork 2
Description
Few bugs in the demos SDK according to claude
14:34
That was 100% a bug in the DEMOS SDK itself — nothing on your end.
Three specific bugs we found and had to patch manually:
Bug 1 — Missing .js extension in token module In build/types/token/index.js the import read from "./TokenUtils" — ESM requires explicit file extensions, so Node couldn't resolve it. Had to patch to from "./TokenUtils.js".
Bug 2 — Directory import in types index build/types/index.js was importing from "./blockchain/TransactionSubtypes" — pointing at a directory, not a file. ESM doesn't support directory imports. Had to patch to from "./blockchain/TransactionSubtypes/index.js".
Bug 3 — Same directory import for token Same file was also importing from "./token" instead of from "./token/index.js".
All three are in the build/types/ path of the SDK. The websdk path itself is fine — the bugs are in the types export which gets pulled in transitively.
Worth flagging to the KyneSys Labs team — these patches will get wiped if anyone runs npm install fresh without knowing to re-apply them. They should be a one-line fix in their build pipeline (just add .js extensions to those three imports before publishing to npm).