Skip to content

chore: modernize package - #674

Open
weiland wants to merge 164 commits into
mainfrom
dev/chore-package-modernization
Open

chore: modernize package#674
weiland wants to merge 164 commits into
mainfrom
dev/chore-package-modernization

Conversation

@weiland

@weilandweiland commented Sep 6, 2025

Copy link
Copy Markdown
Member
  • analyze org code to see which functions are how often used
  • bump to node v22.18 and v24.6
  • migrate to ESM
  • migrate to TypeScript
  • add new ESLint config
  • fix all linting and formatting issues
  • migrate tests to node api (and away from mocha and chai)
    • bun works with the node apis
  • improve structure
  • create separate exports for each module
  • add deprecated notices
  • add documentation to the code directly
    • docs can be generated with JSdoc, JSR or will be displayed in an Editor/IDE
  • reduce dependencies
  • upgrade all other dependencies

next up:

  • test actions
  • add npm publish action with provenance
  • add jsr publish action
  • remove ./packages/ dir (but make sure the old structure still gets exported)
  • clean up markdown files (docs)
  • verify types

Now nodejs LTS v22 is required.
Further changes:
- use provenance for npm
- add more exports to reduce import size
- remove chai in favour of `node:assert` module
- this module is supported by other runtimes like bun and deno as well
run node tests:
$ node --test tests/*.mjs
run bun tests:
$ bun test tests/*.mjs
or to use the `test` command from `package.json`:
bun test
also add a bun test example.
- migrate ard
- start migrating date -> dates
but I don't think we need it. The fetch api should be sufficient nowadays.
@weilandweiland self-assigned this Sep 6, 2025
@weilandweiland added the don't merge Pull request not ready label Sep 6, 2025
@weilandweiland added the safe to test Only apply to PRs when test runs should be executed label Sep 6, 2025
@weiland
weilandforce-pushed the dev/chore-package-modernization branch from b1739c5 to c670bfaCompareSeptember 6, 2025 22:43
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

javascriptPull requests that update Javascript codesafe to testOnly apply to PRs when test runs should be executed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@weiland@frytg