悪夢の `shell_exec` に別れを告げよ：HSLで実現する型安全なプロセス管理の極意
PHPのレガシーコードを継承したエンジニアにとって、`exec()` や `shell_exec()` は「動くから」という理由で放置されがちな時限爆弾だ。文字列結合で生成されるコマンドラインは、常にシェルインジェクションの脆弱性と隣り合わせであり、実行結果のハンドリングもまた、泥臭い例外処理の積み重ねに過ぎない。
Hackの真髄は、こうした「ランタイムの不確実性」をコンパイル時に駆逐することにある。今回は、HSL（Hack Standard Library）の `HH\Lib\OS` モジュールを活用し、型安全かつ堅牢なプロセス管理アーキテクチャを構築する方法を伝授する。
—
なぜ PHP の `proc_open` は悪手なのか
PHPの `proc_open` は柔軟だが、以下の致命的な欠陥を抱えている。
1. 型なきストリーム: パイプの入出力が `resource` 型という「ブラックボックス」であり、クローズ忘れやバッファ溢れを静的に検知できない。
2. 非同期設計の欠如: 現代的な非同期I/Oと統合されておらず、ブロッキングが発生した瞬間にHHVMのイベントループを阻害する。
3. エスケープの脆弱性: `escapeshellarg()` を忘れた瞬間にゲームオーバーだ。
これに対し、HSLの `HH\Lib\OS` は、`Process` クラスを通じてプロセスを「オブジェクト」としてカプセル化し、ストリームを型安全なインターフェースで制御する。
—
実践：HSLによる型安全なプロセス実行パターン
以下は、外部コマンドを非同期に実行し、その結果を安全にハンドリングするプロダクションコードのテンプレートだ。
namespace App\System;
use HH\Lib\{OS, Vec, Str};
final class CommandRunner {
/
- 型安全なプロセス実行の設計パターン
/
public async function runAsync(string $command, vec
// 1. プロセスの定義（シェルを経由せず、引数を配列で渡すことでインジェクションを無効化）
$process = OS\Process::spawn($command, $args);
// 2. 非同期ストリームとして入出力を制御
$stdout = $process->getStdout();
$stderr = $process->getStderr();
// 3. プロセスの待機と結果の回収（型安全な例外処理）
$result = await $process->waitAsync();
if ($result->exitCode !== 0) {
// 終了コードに応じた厳密なエラーハンドリング
$error = await $stderr->readAllAsync();
throw new \RuntimeException(
Str\format(“Process failed with %d: %s”, $result->exitCode, $error)
);
}
return await $stdout->readAllAsync();
}
}
この設計の「魂」：なぜこれが最強なのか
1. 引数の完全分離: `OS\Process::spawn($command, $args)` はシェルを介さない。`$args` を個別の文字列としてOSに渡すため、スペースやクォートを含むファイル名であっても、シェルインジェクションは物理的に不可能になる。
2. Awaitable による非同期制御: `await $process->waitAsync()` を使うことで、プロセス実行中に他のI/Oを処理可能だ。HHVMのイベントループを一切ブロックさせない。
3. Read-All の最適化: `readAllAsync()` は、ストリームの終端までをメモリ消費効率良くバッファリングする。
—
パフォーマンスと保守性のためのチェックリスト
現場で運用する際、以下の3点を意識しなければ、どれほど美しいコードも破綻する。
1. ストリームのデッドロック回避
パイプのバッファサイズには限界がある。`stdout` をすべて読み込む前に `stderr` が溢れると、プロセスが停止（デッドロック）する。`HH\Lib\Async` モジュールを併用し、入出力を並行して消費する設計を徹底すること。
2. 環境変数の制御
`Process::spawn` には、必要な環境変数のみを最小限に制限する `env` オプションがある。デフォルトで親プロセスの環境を継承するPHP的なやり方は、セキュリティリスクを増大させる。常に `env` を明示的に渡す設計を推奨する。
3. タイムアウトの実装
外部プロセスが永久に終了しないケースを想定せよ。`await` には必ず `HH\Lib\Async\within_timeout` をラップすべきだ。
// 5秒以内に終わらなければ強制終了する
$output = await Async\within_timeout(
async { return await $runner->runAsync(‘git’, vec[‘log’]); },
5.0
);
—
結論：コードは「書く」のではなく「守る」もの
Hackにおけるリファクタリングとは、単なる書き換えではない。「実行時にしか判明しなかったエラー」を「コンパイル時に拒絶されるルール」へと昇華させる作業だ。
`exec` を捨て、`OS\Process` を導入することは、単なるモダン化ではない。それは、君のシステムを「落ちないもの」へと進化させるための、最もコスト対効果の高い投資である。
さあ、レガシーを捨て、型という最強の防具を纏え。コードが堅牢であれば、エンジニアの夜は穏やかになる。