Skip to content

migrate from yarn to pnpm#1206

Merged
pirosiki197 merged 6 commits intomainfrom
switch-to-pnpm
Apr 13, 2026
Merged

migrate from yarn to pnpm#1206
pirosiki197 merged 6 commits intomainfrom
switch-to-pnpm

Conversation

@pirosiki197
Copy link
Copy Markdown
Contributor

@pirosiki197 pirosiki197 commented Apr 13, 2026

なぜやるか & やったこと

yarnであるうれしみが特にないので、速くてストレージにやさしいpnpmに移行した

やらなかったこと

資料

Summary by CodeRabbit

リリースノート

  • Chores
    • パッケージマネージャーを Yarn から pnpm に移行しました
    • CI、ローカル開発コマンド、Docker ビルド/起動コマンドを pnpm に合わせて更新しました
    • パッケージ管理メタ情報を pnpm に固定し、依存版の一部を固定化しました
    • pnpm ワークスペース設定を追加しました
  • Style
    • UnoCSS のプリセットを切り替えました
  • Chores
    • Docker/Git/Dockerignore の不要な Yarn 関連除外設定を整理しました

@github-actions
Copy link
Copy Markdown
Contributor

Preview (prod backend + PR dashboard) → https://1206.ns-preview.trapti.tech/

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 13, 2026

Warning

Rate limit exceeded

@pirosiki197 has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 22 minutes and 10 seconds before requesting another review.

Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 22 minutes and 10 seconds.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 0d990af7-8248-43d0-bb4a-b3de563f42f8

📥 Commits

Reviewing files that changed from the base of the PR and between e1631e0 and 1c36f5f.

📒 Files selected for processing (1)
  • dashboard/Dockerfile
📝 Walkthrough

Walkthrough

CI、Docker、ワークスペース、ツール設定および関連設定で Yarn から pnpm へ移行し、Yarn固有のignore/設定を削除、UnoCSSプリセットを差替えました(アプリケーションのロジック自体は変更なし)。

Changes

Cohort / File(s) Summary
CI 設定
.github/workflows/dashboard-ci.yaml
全ジョブで Yarn → pnpm に切替。Nodeセットアップアクションと依存インストールが置換され、yarn ... コマンドを pnpm ... に変更。
Docker & dockerignore
dashboard/Dockerfile, dashboard/.dockerignore
ベースイメージを node:25-slimnode:24-slim、Yarn手順を削除して pnpm 用の環境変数/キャッシュマウントと pnpm install --frozen-lockfile を追加。.dockerignore から Yarn/PNP 関連の除外パターンを削除。
パッケージ & ツール設定
dashboard/package.json, mise.toml, dashboard/pnpm-workspace.yaml
packageManageryarn@4.13.0pnpm@10.33.0fix スクリプトを pnpm 呼び出しに変更。virtua^0.49.00.49.0 に固定。mise.toml で Yarn エントリを pnpm に置換。新規 dashboard/pnpm-workspace.yaml 追加(minimumReleaseAge: 10080)。
リポジトリ設定 / Ignore / Yarn 設定
dashboard/.gitignore, dashboard/.yarnrc.yml
.gitignore から .pnp.* / .yarn/*yarn-error.log 等を削除。.yarnrc.ymlnodeLinker: node-modules を削除。
フロントエンド設定
dashboard/uno.config.ts
UnoCSS のプリセットを presetUno() から presetWind3() に置換(インポートと presets 配列を更新)。

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed タイトル「migrate from yarn to pnpm」はPRの主要な変更内容を明確に要約しており、Yarnからpnpmへの移行というプロジェクト全体の目的と合致している。
Description check ✅ Passed PRの説明では「なぜやるか」と「やったこと」が記入されており、テンプレートの必須セクションが概ね完成している。「やらなかったこと」と「資料」は空欄だが、非必須項目と判断できる。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch switch-to-pnpm

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Comment thread .github/workflows/dashboard-ci.yaml Outdated
Copy link
Copy Markdown

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@dashboard/Dockerfile`:
- Line 8: The COPY instruction in the Dockerfile currently lists multiple source
files without an explicit destination directory—update the COPY line in
dashboard/Dockerfile (the COPY instruction that references package.json and
pnpm-lock.yaml) to include an explicit destination ending with a slash, e.g.
change "COPY package.json pnpm-lock.yaml ." to "COPY package.json pnpm-lock.yaml
./", so the destination is clearly a directory and satisfies Hadolint DL3021.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 86546ca8-c4b3-457a-8109-23d8697e5504

📥 Commits

Reviewing files that changed from the base of the PR and between 7e64ac3 and 4e9475c.

⛔ Files ignored due to path filters (3)
  • dashboard/.yarn/.gitkeep is excluded by !**/.yarn/**
  • dashboard/pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
  • dashboard/yarn.lock is excluded by !**/yarn.lock, !**/*.lock
📒 Files selected for processing (8)
  • .github/workflows/dashboard-ci.yaml
  • dashboard/.dockerignore
  • dashboard/.gitignore
  • dashboard/.yarnrc.yml
  • dashboard/Dockerfile
  • dashboard/package.json
  • dashboard/pnpm-workspace.yaml
  • mise.toml
💤 Files with no reviewable changes (3)
  • dashboard/.gitignore
  • dashboard/.yarnrc.yml
  • dashboard/.dockerignore

Comment thread dashboard/Dockerfile Outdated
Copy link
Copy Markdown

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@dashboard/uno.config.ts`:
- Around line 1-2: Update the dependency version of `@unocss/preset-mini` in
dashboard/package.json to match unocss (change `@unocss/preset-mini` to ^66.6.7)
to resolve the version mismatch; leave the imports in uno.config.ts
(defineConfig, presetIcons, presetWind3, transformerVariantGroup, parseColor)
and their usage as-is since parseColor is a stable public API.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 1c8d024a-9717-4b14-bbc3-084bdaf37b72

📥 Commits

Reviewing files that changed from the base of the PR and between 528f8ca and 7aeae6a.

⛔ Files ignored due to path filters (1)
  • dashboard/pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (3)
  • .github/workflows/dashboard-ci.yaml
  • dashboard/package.json
  • dashboard/uno.config.ts
✅ Files skipped from review due to trivial changes (1)
  • .github/workflows/dashboard-ci.yaml
🚧 Files skipped from review as they are similar to previous changes (1)
  • dashboard/package.json

Comment thread dashboard/uno.config.ts
@pirosiki197 pirosiki197 enabled auto-merge (squash) April 13, 2026 11:14
@pirosiki197 pirosiki197 merged commit 2be5a81 into main Apr 13, 2026
20 checks passed
@pirosiki197 pirosiki197 deleted the switch-to-pnpm branch April 13, 2026 11:17
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