chore(frontend): switch to bun (remove pnpm/node refs)#7
Merged
Conversation
Bun is the canonical frontend package manager (bun.lock, bunfig.toml, bun test). The leftover packageManager: pnpm@10.30.3 field caused corepack to enforce pnpm, conflicting with bun-first workflow. fn-17-switch-frontend-to-bun-remove-pnpmnode.1
No vitest package is installed and no source file imports from vitest. bun test is the canonical test runner (configured via bunfig.toml preload + @happy-dom/global-registrator). Verified: bun run typecheck ✓ | bun run build ✓ | bun test ✓ (10 pass) | bun run dev ✓ fn-17-switch-frontend-to-bun-remove-pnpmnode.2
There was a problem hiding this comment.
Pull request overview
Completes the frontend’s migration to Bun by removing remaining pnpm/corepack enforcement and deleting unused Vitest configuration, aligning the repo with a bun-first workflow.
Changes:
- Removed the
packageManager: pnpm@10.30.3field fromfrontend/package.jsonto avoid corepack enforcing pnpm. - Deleted
frontend/vitest.config.ts, which is no longer needed given the current Bun-based test setup.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| frontend/package.json | Removes pnpm packageManager pin; keeps scripts aligned with Bun (bun test). |
| frontend/vitest.config.ts | Deletes vestigial Vitest config file now that tests run via Bun. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
z23cc
added a commit
that referenced
this pull request
Apr 9, 2026
P0 fixes (state loss — root cause of 5 issues): - get_flow_dir() now walks up directory tree (FLOW_STATE_DIR env → walk-up → CWD) Fixes: #1 state loss, #3 state not persistent, #5 worker parallel fail, #9 .flow symlink issues. Same pattern as git finding .git. - flowctl recover --epic <id> [--dry-run]: rebuilds task completion status from git log. Fixes #11 no recovery after state loss. P1 fixes (guard + review): - Guard graceful fallback: missing tools → "skipped" (not "failed"). Only actual failures block pipeline. Fixes #8. - Review-backend availability check: if rp-cli/codex not in PATH, auto-fallback to "none" with warning. Fixes #7. P2 fixes (UX): - Slug max length 40→20 chars. "Django+React platform with account management" → "fn-3-django-react-plat" not 40-char monster. Fixes #2 #12. - Brainstorm auto-skip: trivial tasks (≤10 words, contains "fix"/"typo"/etc) skip brainstorm entirely. Fixes #6. - --interactive flag: pause at key decisions. Fixes #10. 370 tests pass. Zero new dependencies. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary
Completes the frontend migration to bun. Bun was already partially set up (
bun.lock,bunfig.toml,bun test), but a stalepackageManager: pnpm@10.30.3field inpackage.jsonwas causing corepack to enforce pnpm, breaking bun-first workflows. Also removed vestigialvitest.config.ts.Changes
packageManager: pnpm@10.30.3fromfrontend/package.jsonfrontend/vitest.config.ts(vitest not installed, no source imports)bun installverifies lockfile integrity (136 packages, 305ms, no warnings)Verification
All four frontend scripts pass:
bun run typecheckbun run buildbun testbun run devTest plan
Epic: fn-17-switch-frontend-to-bun-remove-pnpmnode