Remove map_run_resolution_error passthrough (DSK-126)

[daisuke8000]

Summary

• Remove map_run_resolution_error and use inline .map_err(|e| e.to_string()) at three resolution call sites.

Test plan

• cargo fmt --all --check
• cargo test -p rustern
• Local CodeRabbit review (0 findings)

Made with Cursor

Summary by CodeRabbit

• Documentation
  • 実行結果の解決に関する説明コメントが追加され、各処理の役割が分かりやすくなりました。
• Refactor
  • エラー変換の扱いを整理し、実行解決まわりの処理がより明確で保守しやすくなりました。

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: f81e92ae-bbbe-46c4-ba18-8e5f3c4061d8

📥 Commits

Reviewing files that changed from the base of the PR and between 657fa4d and aeaa7b9.

📒 Files selected for processing (1)

• src/run_defaults.rs

---

📝 Walkthrough

Walkthrough

src/run_defaults.rs において、RunResolutionError の文字列変換を共通関数 map_run_resolution_error の呼び出しから、各解決関数内の map_err(|e| e.to_string()) へ置き換えました。resolved_run、resolved_pod_query、resolved_namespaces にはドキュメントコメントが追加されています。rustern_core::discovery::run_resolution の import 記述も整形されました。公開APIの変更はありません。

Changes

分類	内容
リファクタ	エラー変換ロジックの共通関数呼び出しを各関数内インライン処理に変更
ドキュメント	3つの解決関数にコメント追加
整形	import文の行分割調整

重大度: 低。ロジック上の実質的な変更なし（同一のエラーメッセージ生成が呼び出し元で行われるのみ）。非同期処理・K8s通信・セキュリティへの影響なし。

Possibly related issues

• DSK-126: 本PRと同一の src/run_defaults.rs 変更（map_run_resolution_error を各呼び出し箇所での .map_err(|e| e.to_string()) に置換）を行っており、直接関連する。

🚥 Pre-merge checks | ❌ 2

❌ Failed checks (2 warnings)

Check name	Status	Explanation	Resolution
Title check	⚠️ Warning	重大: Conventional Commits形式の接頭辞がなく、変更内容もやや曖昧です。	refactor: remove map_run_resolution_error passthrough のようにprefixを付け、72文字以内で具体化してください。
Description check	⚠️ Warning	重大: テンプレート必須の「## 概要」「## 変更理由」がなく、英語のSummary/Test planのみです。	「## 概要」と「## 変更理由」を追加し、変更内容と背景を簡潔に埋めてください。

---

_{Comment @coderabbitai help to get the list of available commands.}