peezy-tech/codex-toys
2
0
Fork
You've already forked codex-toys
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Migrate codex-flows to Node pnpm and VitePlus #2

Closed
matamune wants to merge 3 commits from node-pnpm-viteplus into main
pull from: node-pnpm-viteplus
merge into: peezy-tech:main
Conversation 1 Commits 3 Files changed 154 +10275 -2982
matamune commented 2026-05-20 21:26:16 +00:00
Owner
Copy link

Summary

  • migrates the monorepo from Bun to Node 24 LTS + pnpm + VitePlus
  • replaces runner = "bun" / defineBunFlow with the Node flow runner and helpers, with no compatibility path
  • replaces Bun runtime APIs with Node APIs, including node:sqlite, Node HTTP, Node child processes, and smol-toml
  • updates CI, docs, bundled flows, gateway packaging, and package-on-demand commands to vp / pnpm
  • aligns package versions to 0.132.1 and documents the Codex-line versioning policy in RELEASE.md
  • intentionally exports CodexEventEmitter from @peezy.tech/codex-flows and removes broad source catch-all aliases

Verification

  • pnpm install --frozen-lockfile
  • pnpm run check:types
  • pnpm run test
  • pnpm run build
  • pnpm run release:check
  • git diff --check
  • targeted Bun/vpx/bunx cruft search
  • package-boundary scan for relative imports leaving workspace package roots

Notes

This is intentionally a breaking migration. The branch removes Bun lock/config/type dependencies and does not preserve runner = "bun" or defineBunFlow aliases.

This replaces #1, whose Forgejo PR metadata became stale with a missing-fork-information warning even though the Git branch remained healthy.

## Summary - migrates the monorepo from Bun to Node 24 LTS + pnpm + VitePlus - replaces `runner = "bun"` / `defineBunFlow` with the Node flow runner and helpers, with no compatibility path - replaces Bun runtime APIs with Node APIs, including `node:sqlite`, Node HTTP, Node child processes, and `smol-toml` - updates CI, docs, bundled flows, gateway packaging, and package-on-demand commands to `vp` / `pnpm` - aligns package versions to `0.132.1` and documents the Codex-line versioning policy in `RELEASE.md` - intentionally exports `CodexEventEmitter` from `@peezy.tech/codex-flows` and removes broad source catch-all aliases ## Verification - `pnpm install --frozen-lockfile` - `pnpm run check:types` - `pnpm run test` - `pnpm run build` - `pnpm run release:check` - `git diff --check` - targeted Bun/vpx/bunx cruft search - package-boundary scan for relative imports leaving workspace package roots ## Notes This is intentionally a breaking migration. The branch removes Bun lock/config/type dependencies and does not preserve `runner = "bun"` or `defineBunFlow` aliases. This replaces #1, whose Forgejo PR metadata became stale with a missing-fork-information warning even though the Git branch remained healthy.
matamune commented 2026-05-20 22:34:36 +00:00
Author
Owner
Copy link

Closing in favor of #3. Force-pushing #2 exposed stale Forgejo PR metadata again; #3 is opened from a fresh branch at the final head and should not be force-pushed.

Closing in favor of #3. Force-pushing #2 exposed stale Forgejo PR metadata again; #3 is opened from a fresh branch at the final head and should not be force-pushed.
matamune closed this pull request 2026-05-20 22:34:36 +00:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
ci:reviewed-workflow
Maintainer reviewed workflow changes for PR policy.

  
ci:run-full
Maintainer-approved one-shot full PR CI run.

No labels
ci:reviewed-workflow
ci:run-full
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
peezy-tech/codex-toys!2
No description provided.