Skip to content

Conversation

@thedanchez
Copy link

@thedanchez thedanchez commented Oct 30, 2025

Summary

This PR migrates project to use bun as the primary tool for:

  • package management (replacing npm / pnpm / yarn)
  • script runner (replacing npm / pnpm / yarn)
  • test runner via bun:test (replacing mocha / chai / ts-node)

Additionally, this PR:

  • updates GitHub CI workflow to use bun (instead of yarn)
  • updates TypeScript to latest version
  • Removes unnecessary Makefile that was a proxy for running node_module binary scripts (move that responsibility to package.json where it belongs)

For the uninitiated, bun is an all-in-one JS toolchain. Moreover, it natively understands TypeScript which allows you to run any TS module without requiring transpilation (e.g. bun run module.ts). As a result of using bun, tests run an order of magnitude faster.


Note

Replaces Mocha/Chai with bun:test and updates all assertions to Bun’s expect API.

  • Testing:
    • Migrate test suite from mocha/chai to bun:test across all specs.
    • Replace Chai assertions with Bun/Jest-style expect APIs (e.g., toEqual, toBeGreaterThanOrEqual, not.toBeUndefined, toThrow).
    • Remove Mocha/Chai imports and adapt lifecycle hooks (beforeEach, describe, it) from bun:test.

Written by Cursor Bugbot for commit f71f16e. This will update automatically on new commits. Configure here.

@thedanchez thedanchez requested a review from a team as a code owner October 30, 2025 05:57
cursor[bot]

This comment was marked as outdated.

cursor[bot]

This comment was marked as outdated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant