Conversation
- Remove Yarn-specific files (.yarnrc.yml, .yarn/, .pnp.cjs, .pnp.loader.mjs, yarn.lock) - Add bun.lock for dependency locking - Update package.json: - Change all scripts from 'yarn' to 'bun' - Update packageManager field to 'bun@1.3.1' - Convert 'resolutions' to 'overrides' (Bun/npm format) - Remove Yarn-specific 'dependenciesMeta' - Update documentation: - README.md: Update installation command - AGENTS.md/CLAUDE.md: Update all development commands - CONTRIBUTING.md: Update setup and build instructions - Individual package READMEs: Update installation commands - Update .gitignore to remove Yarn-specific entries - Add workflows-bun-migration.patch for CI/CD workflow updates (to be applied separately with workflow permissions) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
|
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ❌ Deployment failed View logs |
stackflow-docs | 1bc78eb | Oct 26 2025, 06:51 PM |
📝 WalkthroughSummary by CodeRabbit
WalkthroughRemoved Yarn PnP/zero-install artifacts, deleted the PnP loader, updated .gitignore to ignore Changes
Sequence Diagram(s)sequenceDiagram
participant Dev as Developer push
participant GH as GitHub Actions
participant Checkout as actions/checkout
participant SetupOld as setup-node (old)
participant SetupNew as oven-sh/setup-bun (new)
participant InstallOld as yarn install --immutable
participant InstallNew as bun install --frozen-lockfile
participant BuildOld as yarn build / yarn run
participant BuildNew as bun run build / bun run
Dev->>GH: push
GH->>Checkout: checkout repo
alt Before (Yarn/Node)
GH->>SetupOld: setup-node (node-version)
GH->>InstallOld: yarn install --immutable
GH->>BuildOld: yarn build / yarn test / yarn changeset
else After (Bun)
GH->>SetupNew: setup-bun (bun-version)
GH->>InstallNew: bun install --frozen-lockfile
GH->>BuildNew: bun run build / bun run test / bun run changeset
end
Note right of GH: Cache keys and paths updated to Bun-specific locations
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes
Pre-merge checks and finishing touches❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Disabled knowledge base sources:
📒 Files selected for processing (1)
💤 Files with no reviewable changes (1)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Deploying stackflow-demo with
|
| Latest commit: |
1bc78eb
|
| Status: | 🚫 Build failed. |
- Update all package.json scripts: yarn → bun run - Update scripts/publish-preview-packages.sh: yarn dlx → bun x - Update biome.json: remove .yarn and .pnp exclusions - Update .vscode/settings.json: remove Yarn SDK references - Update documentation: npm2yarn → npm2bun in all MDX files - Clean up Yarn-specific configuration across all workspace packages This completes the migration from Yarn to Bun across the entire monorepo. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (2)
.github/workflows/release.yml (2)
20-23: Useregistriesinput instead of deprecatedregistry-url.The
registry-urlparameter is deprecated in favor of theregistriesinput. Update this to use the newerregistriesformat for better maintainability and future compatibility.Apply this diff to use the
registriesinput:- - uses: oven-sh/setup-bun@v2 - with: - bun-version: 1.3.1 - registry-url: https://registry.npmjs.org + - uses: oven-sh/setup-bun@v2 + with: + bun-version: 1.3.1 + registries: | + https://registry.npmjs.org/
55-63: Consider usingBUN_AUTH_TOKENinstead ofNODE_AUTH_TOKENfor Bun.The Bun-specific environment variable for authentication is
BUN_AUTH_TOKEN, whileNODE_AUTH_TOKENis Node-specific. For clarity and consistency with Bun tooling, update the environment variable name.Apply this diff to use
BUN_AUTH_TOKEN:uses: changesets/action@v1 with: publish: bun run changeset publish version: bun run changeset:version env: - NODE_AUTH_TOKEN: ${{ secrets.NPM_PUBLISH_TOKEN }} + BUN_AUTH_TOKEN: ${{ secrets.NPM_PUBLISH_TOKEN }} GITHUB_TOKEN: ${{ secrets.DAANGNBOT_PAT }}
📜 Review details
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Disabled knowledge base sources:
- Jira integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (3)
.github/workflows/build.yml(2 hunks).github/workflows/integration.yml(5 hunks).github/workflows/release.yml(3 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: Workers Builds: stackflow-docs
🔇 Additional comments (4)
.github/workflows/build.yml (1)
22-29: Migration to Bun looks good.Bun 1.3.1 is a valid release, and the setup and installation steps are properly configured. The
bun install --frozen-lockfileflag is the correct Bun syntax for deterministic installs..github/workflows/integration.yml (3)
18-25: Migration to Bun in test job is well-structured.Bun 1.3.1 is a valid release, and the setup and installation match the patterns in other jobs.
50-66: Bun cache configuration properly updated for type-check job.The new Bun-specific cache paths at
~/.bun/install/cache/and thebun.lockhash key are appropriate for Bun's package manager, replacing Yarn's PnP cache strategy.
100-120: Changeset job migration looks good.The Bun setup and cache configuration mirror the type-check job correctly. Ensure
bun.lockexists in the repository root as indicated in the PR objectives.
Replace all 'node ./esbuild.config.js' with 'bun ./esbuild.config.js' across all workspace packages. This completes the migration to use Bun as the primary runtime instead of Node.js. Changed files: - All package.json build:js scripts now use Bun runtime - esbuild configs are compatible with Bun (CommonJS/ESM support) This improves build performance and consistency across the monorepo. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
2 participants
🤖 Generated with Claude Code