fix: handle missing optional doctor binaries

[ubugeeei]

Summary

• Check whether each doctor binary exists before invoking --version.
• Keep missing required tools as errors.
• Report missing optional tools as not found (optional) instead of aborting on OSError.

Validation

• moon fmt
• moon build --target native src/cmd/actrun
• moon test --target native
• moon run src/cmd/actrun --target native -- doctor

[mizchi]

LGTM 主旨で、小さい指摘 2 つ。

nit: "not found" メッセージが 2 箇所に重複

新規の pre-check ブロック (L5405–) と既存の --version 失敗 fallback (L5440+ あたり) で、 optional の hint 文字列構築 + -- name: not found ... 出力が 行単位で同じ になってる。 小さい helper に纏めると差分が読みやすいかも:

fn doctor_print_not_found(check : DoctorCheck) -> Unit {
  if check.required {
    println("ERR " + check.name + ": not found (required)")
  } else {
    let hint = if check.env_override.length() > 0 {
      " (optional, override with " + check.env_override + ")"
    } else {
      " (optional)"
    }
    println("--  " + check.name + ": not found" + hint)
  }
}

その上で pre-check と post---version-fail の両方からこれを呼ぶ。 helper 化と本 PR を 1 commit で混ぜず separate にしたいなら follow-up でも全然 OK。

nit: "not found" の意味が pre/post で違う

PR 後の挙動:

シナリオ	表示	実際
bin 自体が PATH に無い	not found	bin missing (正しい)
bin はあるが --version が exit ≠ 0	not found	bin exists but flag rejected

後者は厳密には version probe failed (exit N) 的に書き分けた方が、 doctor を読んだ人がトラブルシュート時に迷わない (bin はあるのに not found って言われた!? を防げる)。 pre-existing なので blocker ではないけど、 今回触ったついでに分けても良い。

(informational) Windows portability

bin.contains("/") が absolute path 判定として使われてる。 C:\Tools\node.exe は \ なので fall through して which 呼び出しになる。 actrun が native Windows をターゲットにしてるなら where.exe も検討余地。 POSIX 専用なら無視で OK。

---

機能面は意図通り動くはずで、 これで node / docker / wasmtime 等が未インストールの環境でも doctor が abort せず動くようになるのは良い改善。

[ubugeeei]

レビューありがとうございます 🙏 2 点の nit に対応しました (2182eb3)。

• helper 化: doctor_optional_hint / doctor_print_not_found / doctor_print_version_probe_failed を抽出して、pre-check と post---version-fail の重複を解消。
• 意味の書き分け: post---version-fail 側は version probe failed (exit N) を表示するように変更。pre-check は従来通り not found で、bin の不在 vs --version 失敗 が一目で区別できるようにしました。

Windows portability の informational 指摘は今回はスコープ外として見送り、必要があれば別 PR で対応します。

Validation

• moon fmt (ok)
• moon build --target native src/cmd/actrun / moon check --deny-warn --target native は当方ローカルで moonc v0.8.3+cd28f524e の依存パッケージ (mizchi/crater, moonbitlang/quickcheck, moonbit-community/yaml) で ICE (Moonc.Stype.extract_tpath_exn) が出ており、PR 前の commit でも同じく再現するため、今回の変更とは無関係の環境/ツールチェイン由来のようです。CI 側で結果を確認させてください 🙇

[ubugeeei]

Claude にコメントさせたら自分が絶対に送らない文面でわろた