Skip to content

Migrate to Aube and Node - #45

Merged
joeldrapper merged 1 commit into
mainfrom
migrate-aube-node
May 22, 2026
Merged

Migrate to Aube and Node#45
joeldrapper merged 1 commit into
mainfrom
migrate-aube-node

Conversation

@joeldrapper

Copy link
Copy Markdown
Collaborator

Switch from Bun to Aube and Node.

Copilot AI review requested due to automatic review settings May 22, 2026 10:50

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Migrates the project’s tooling from Bun to Aube + Node, updating local tasks, CI, and build/benchmark execution accordingly.

Changes:

  • Switch package management and task runner usage to Aube (new aube-lock.yaml, updated mise.toml, updated CI workflow).
  • Replace Bun-based build pipeline with an esbuild-based build script and Node stdlib APIs.
  • Adjust TypeScript config and benchmark imports for the new module/runtime setup.

Reviewed changes

Copilot reviewed 8 out of 10 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
tsconfig.json Sets rootDir to src to align TS emit paths with the project structure.
package.json Updates scripts to Node/Aube assumptions; adds packageManager, Node types, and esbuild.
mise.toml Switches tool management/tasks from Bun to Aube + Node.
bun.lock Removes Bun lockfile.
build.ts Replaces Bun build with esbuild and Node stdlib utilities; runs tsgo for d.ts emit.
benchmark/run.ts Updates imports to explicit .ts paths for Node ESM resolution.
aube-lock.yaml Adds Aube lockfile for reproducible installs.
AGENTS.md Updates contributor guidance from Bun to Aube + Node.
.node-version Removes pinned Node version file.
.github/workflows/tests.yml Switches CI to mise + Aube, updates Playwright caching/install commands, runs build/tests via Aube.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread package.json
Comment on lines +20 to +23
"build": "node build.ts",
"bench": "node benchmark/run.ts",
"bench:thorough": "node benchmark/run.ts --thorough",
"bench:decision": "node benchmark/run.ts --repeats 3 --thorough",
Comment thread mise.toml
Comment thread build.ts
target: "es2022",
})

// Generate TypeScript declarations (skip lib check to avoid node type errors)

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 0b508b9f43

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread build.ts
@joeldrapper
joeldrapper merged commit ef0e777 into main May 22, 2026
6 checks passed
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.

2 participants