| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
dist-verify.ts scanned only declaration files while calling them cjsFiles, kept obsolete commented-out checks, and exempted any file containing 'as default' from the forbidden require() check, so a forbidden import of an ESM-only UnoCSS package could pass silently. Scan the real CJS outputs (dist/**/*.cjs and dist/**/*.d.cts), drop the obsolete commented implementation and the default-export exemption, and cover the checker with tests that run it on fixture trees.
✅ Deploy Preview for unocss ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Sorry, something went wrong.
Runtime .cjs may require() ESM-only packages on Node >= 22.12 (require of ESM), so scanning runtime output produced false positives for the nuxt and webpack integrations. Declarations have no such escape hatch because no CJS types ship.
|
commit: c841c0b |
Sorry, something went wrong.
process.argv[1] is a Windows path while import.meta.url is a file URL with a /C:/ prefix, so the direct-execution guard never matched and the checker exited silently on Windows.
| Back | FazBrowse Home | New Git URL |
Problem
The postbuild dist checker (scripts/dist-verify.ts) could report success while a forbidden import was present. Its forbidden-require() rule was skipped for any file containing as default, so a declaration like:
passed validation. The script also carried obsolete commented-out checks, misleading cjsFiles naming, and had no test coverage.
Changes
Validation