fix: PR #983 レビュー対応（play.ts パース順序バグ他）

[na-navi]

対象PR

heygen-com#983 (review)

やること

レビュー指摘への対応。詳細はPRレビュー本文を参照。

[na-navi]

PR heygen-com#983 のレビュー指摘に対応するコミットを push しました: 4affe2ba

対応内容

Bug

• play.ts のパース順序: parseRemoteDebuggingPort を server 起動前（既存の --user-data-dir バリデーションの直後）に移動。不正値（例 9222abc）で "Player running" / "Press Ctrl+C to stop" を表示してから失敗し、listening socket がリークする問題を解消。
• 検証ログ: bun packages/cli/dist/cli.js play test/zundamon --no-open --browser-path /usr/bin/chromium --user-data-dir /tmp/x --remote-debugging-port 9222abc → エラー後 netstat で :3003 / :3004 が無いことを確認。

Refactor

• validateRemoteDebuggingPortDeps(args) を openBrowser.ts に追加し、play.ts / preview.ts のコピペバリデーションを統合。

Test

• 重複していた "prepends --remote-debugging-port before the URL" を削除し、代わりに "omits --remote-debugging-port when userDataDir is missing (defense in depth)" を追加。buildBrowserArgs 側も userDataDir 不在時は CDP フラグを出さないよう変更（多層防御）。
• validateRemoteDebuggingPortDeps 用のテスト5本を追加（合計 23 tests in openBrowser.test.ts）。

Minor

• parseRemoteDebuggingPort(value: unknown) → (value: string | undefined) に narrow、null チェックと String() / Number.isInteger の dead code を削除。
• 重い JSDoc を削除し、ファイル全体のスタイルに合わせた。
• play.ts / preview.ts の examples に --remote-debugging-port のフルチェーン例を追加（--help 出力にも反映）。
• play.ts の --remote-debugging-port の description を preview.ts と統一（(requires --browser-path and --user-data-dir)）。

CI / 品質確認

• bun run --filter @hyperframes/cli test → 353 tests pass
• bun run --filter @hyperframes/cli typecheck → exit 0
• oxlint / oxfmt --check → 0 warnings / formatted

レビュー再確認をお願いします。