Skip to content

chore: switch packageImportMethod to auto and add audit ignorePrune - #4016

Merged
KATO-Hiro merged 1 commit into
stagingfrom
#4015
Sep 5, 2026
Merged

KATO-Hiro merged 1 commit into
stagingfrom
#4015

Conversation

@KATO-Hiro

@KATO-Hiro KATO-Hiro commented Sep 5, 2026 •

Copy link
Copy Markdown
Collaborator

pnpm v12 defaults to auto; clone-or-copy is no longer needed. Enable audit ignorePrune to auto-remove resolved GHSA entries.

Claude-Session: https://claude.ai/code/session_01LpBu3jvGWLNEeyWVL2PQZT

close #4015

Summary by CodeRabbit

  • 改善
    • 依存関係のインストール方式を自動選択に変更し、環境に応じた処理に対応しました。
    • 監査レポートから、該当しなくなったセキュリティ警告を自動的に削除するよう改善しました。

pnpm v12 defaults to auto; clone-or-copy is no longer needed.
Enable audit ignorePrune to auto-remove resolved GHSA entries.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01LpBu3jvGWLNEeyWVL2PQZT
@coderabbitai

coderabbitai Bot commented Sep 5, 2026 •

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Team

Run ID: 1ffee606-928c-452d-8892-e7ae513d8473

📥 Commits

Reviewing files that changed from the base of the PR and between fdeee83 and 14edd48.

📒 Files selected for processing (1)
  • pnpm-workspace.yaml

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

pnpm-workspace.yamlのパッケージインポート方式をautoに変更しました。監査レポートでは、現在存在しないGHSAエントリを自動削除する設定を追加しました。

Changes

pnpm 12設定

Layer / File(s) Summary
パッケージ管理設定の更新
pnpm-workspace.yaml
パッケージのインポート方式をautoに変更しました。audit.ignorePruneを追加し、存在しないGHSAエントリを自動削除します。

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to 14edd

This updates pnpm import behavior and removes stale ignored audit advisories automatically. The configuration change is ready to merge with no identified current-head risk.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed タイトルは、packageImportMethod の変更と audit.ignorePrune の追加を明確に示しています。主な変更内容と一致します。
Linked Issues check ✅ Passed 変更内容は、Issue #4015 の pnpm 12 の新機能利用という目的に一致します。packageImportMethod: auto と audit.ignorePrune を設定しています。
Out of Scope Changes check ✅ Passed 確認できる変更は、pnpm 設定の更新だけです。Issue #4015 の目的から外れる変更はありません。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch #4015

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.

@KATO-Hiro KATO-Hiro left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

LGTM

@KATO-Hiro
KATO-Hiro merged commit 68402fc into staging Sep 5, 2026
3 checks passed
@KATO-Hiro
KATO-Hiro deleted the #4015 branch September 5, 2026 22:48

This branch was successfully deployed

1 active deployment
preview — 14edd484 Deployed Sep 5, 2026 by KATO-Hiro via preview #8626
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.

chore(deps) use new features in pnpm 12

1 participant