Conversation
| "idb": "^8.0.3", | ||
| "uuid": "^13.0.0" | ||
| }, | ||
| "exports": { |
There was a problem hiding this comment.
Im still not sure about having to define this on every 'class'/'service' of the project
But if its the only way, maybe it could be a good idea to add to the Readme.md some instructions so its clear how to implement a new functionality (like where we should add it -> package.json, tsdown.config.ts, src/index.ts, etc)
wdyt? @TamaraFinogina
There was a problem hiding this comment.
Defining exports is the only way it worked. Not sure if it's because I'm importing this library in commonjs projects and they need some help or not. I'll add read.me
There was a problem hiding this comment.
Maybe could use wildcard exports :)
"exports": {
".": {
"types": "./dist/index.d.ts",
"import": "./dist/index.mjs",
"require": "./dist/index.js"
},
"./*": {
"types": "./dist/*.d.ts",
"import": "./dist/*.mjs",
"require": "./dist/*.js"
}
}
|



Description
Make sure that we don't need to load the entire library if we just use a few functions
Related Issues
Relates to PB-5714
Checklist
Testing Process
Additional Notes
without any import shaking:

with this PR: