switch off microbundle #705
Conversation
🦋 Changeset detectedLatest commit: 83e6a88 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
All alerts resolved. Learn more about Socket for GitHub. This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored. |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
| entry: { 'index.umd': 'index.tsx' }, | ||
| format: 'iife', | ||
| // minify: true, | ||
| minify: false, |
There was a problem hiding this comment.
Bug: Minification Options Present Despite Disabled Minification
The UMD build includes terserOptions even though minify is explicitly set to false. This creates an inconsistent setup where minification is disabled, but its options are still present.
| "test": "jest --verbose", | ||
| "metrics": "DEBUG=\"${DEBUG:-1}\" node -e \"const fs=await import('fs');const {compiler}=await import('./dist/debug.module.js');console.log('='.repeat(50));compiler(fs.readFileSync('fixture.md','utf8'));\"", | ||
| "metrics": "DEBUG=\"${DEBUG:-1}\" node -e \"const fs=await import('fs');const {compiler}=await import('./dist/debug.mjs');console.log('='.repeat(50));compiler(fs.readFileSync('fixture.md','utf8'));\"", |
There was a problem hiding this comment.
Bug: Script Import Path Mismatch
The metrics script now imports from ./dist/debug.mjs. This path might not align with the actual output filename generated by the new tsup build system, which could cause the script to fail.
No description provided.