You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Fully document the code using TypeScript compatible JSDoc comments (I don't want a TypeScript rewrite for the same reasons Svelte switched from TypeScript to JSDoc, mainly, there are too many incompatible TypeScript versions in the wild [like Deno's vs official tsc]).
Remove all direct usage of MEGA's API outside an API translation class, as MEGA's API is full of abbreviations for methods and parameters that hurt code maintainability (idea from this comment).
For the next major release:
Remove the CJS build
Remove callbacks, change code to always use promises and async functions
Replace rsa.mjs with something better (it's use of a global state is not desirable and I guess Node.js already supports RSA natively)
Rename Files and Folders into Nodes to avoid collision with globalThis.File.
For a future major release:
Change the API from being based on classes to be programmatic in order to improve tree-shaking and allow for more custom workflows (based on those ideas).
Include platform specific code for Node.js, Deno (maybe it is pointless to support it since it's almost 100% Node.js compatible) and browsers to handle common cases of the library like:
Reading files from a system path (or from a File or Blob object in browsers)
Writing files from a system path (or using a[download] in browsers)
reacted with thumbs up emoji reacted with thumbs down emoji reacted with laugh emoji reacted with hooray emoji reacted with confused emoji reacted with heart emoji reacted with rocket emoji reacted with eyes emoji
-
For a minor release:
_
prefixed method implementations elsewhere (like functions in module scope instead of in the class scope), like this oneFor the next major release:
globalThis.File
.For a future major release:
File
orBlob
object in browsers)a[download]
in browsers)Beta Was this translation helpful? Give feedback.
All reactions