chore: convert icons to a compiled package and convert tests to vitest
#32649
+86
−47
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Additional details
Converts
@packages/icons
to use vitest and removes the ts-node entrypoint making ESM possibleSteps to test
How has the user experience changed?
PR Tasks
cypress-documentation
?type definitions
?Note
Convert @packages/icons to a compiled package with CJS/ESM builds, replace Mocha/Chai tests with Vitest, and update CI/docs accordingly.
ts-node
withtsx
; addbuild:cjs
/build:esm
viatsc
with newtsconfig.cjs.json
/tsconfig.esm.json
and sharedtsconfig.base.json
; settypes
and keepindex.js
/index.d.ts
entrypoints; update.gitignore
/.eslintignore
.index.ts
to__dirname/dist
.vitest.config.ts
); replacetest/icons_spec.ts
withtest/icons.spec.ts
.vitest
andtsx
; update scripts (build
,build-assets
,test-unit
,check-ts
).sanitize-verify-and-store-mocha-results
expectedResultCount
from10
to9
in unit tests.guides/esm-migration.md
checklist (markpackages/icons
complete; adjusthttps-proxy
entries).vitest
,tsx
) and cleanup mocha range.Written by Cursor Bugbot for commit 27aa98a. This will update automatically on new commits. Configure here.