[owayo]

概要

age の設定の通知 (⚠ --age ignored: project's minimumReleaseAge ...) と、mise の minimum_release_age_excludes の警告が、マニフェストの数だけ繰り返し表示されていました。進捗バーの表示中に、バーを避けずに出力されていました。同じ通知と警告は実行全体で 1 回だけ、進捗バーと混ざらないように出します。

原因

orchestrator.rs の build_filter_for が通知と警告を出しており、check の段階はこの関数をマニフェストごとに呼びます。進捗バーはその前に始まっているので、出力がバーの描画と混ざっていました。AGENTS.md は「バー表示中の警告は Progress::suspend で包む」としています。

変更内容

• build_filter_for は通知を出さずに、UpdateFilter と一緒に返す。age の解決 (orchestrator/age.rs) の emit_notice は通知の文字列を返す notices に、mise の excludes の警告も文字列を返す mise_age_excludes_warning にした。
• check の段階は、まだ出していない通知だけを (unseen_notices)、Progress::suspend で進捗バーを避けて出す。同じ通知は実行全体で 1 回だけ出し、ディレクトリごとに違う通知 (別の設定ファイルの値など) はそれぞれ出す。
• AGENTS.md に書いた。

検証

• 回帰テスト: orchestrator::tests::filter_notices_are_returned_once_instead_of_printed_per_manifest (build_filter_for が age の通知を出さずに返し、同じ通知は 2 回目から出さない。新しい関数を使うので、修正前のコードではコンパイルできない)
• 手元での修正前後の比較: 3 パッケージの pnpm workspace (ルートに minimumReleaseAge: 14400) に depup --age 2w --dry-run を実行し、--age ignored の通知が修正前は 4 回 (ルートと各パッケージ)、修正後は 1 回出ることを確かめた
• cargo fmt --all -- --check: pass
• cargo clippy --locked --all-targets -- -D warnings: pass
• cargo test --locked (Windows): pass

Closes #108

🤖 Generated with Claude Code

[coderabbitai]

Important

• Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration

• Source mode: Auto
• Repository: Unavailable
• Branch: Unavailable
• File: No configuration file from this repository applied
• Commit: Unavailable
• Layers: defaults
• Review profile: CHILL
• Plan: Advanced
• Run ID: 6828b869-88a5-4e96-9e38-785388ca6a69

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

• X
• Mastodon
• Reddit
• LinkedIn