https://daitai.dev/tags/oss/Recent content in OSS on 大体動かないHugojaSat, 09 May 2026 23:59:17 +0900https://daitai.dev/posts/rtk-pr-1800/Sat, 09 May 2026 23:59:17 +0900https://daitai.dev/posts/rtk-pr-1800/<blockquote> <p><strong>TL;DR</strong> — RTK に立てた issue が1週間放置されたので自分でプルリクを出した。<code>run_in_terminal</code> の未認識バグを修正し、重複関数を統合。ローカル確認では mise と linuxbrew の PATH 競合に引っかかったが解決し、PR <a href="https://github.com/rtk-ai/rtk/pull/1800">#1800</a> を提出した。</p> </blockquote> <p><a href="https://daitai.dev/posts/rtk-copilot-chat/">前の記事</a>で「引き続き更新を待つかぁ」と書いたやつの続き。</p> <h2 id="1週間待ったけど動きがなかった">1週間待ったけど動きがなかった</h2> <p><a href="https://github.com/rtk-ai/rtk/issues/1425">Issue #1425</a> を立てたあと、1週間ほど本体の動向を見ていた。メンテナさんも忙しいだろうし、そんなもんかと思っていたけど、なかなか動きがない…</p> <p>うん。待つの飽きた。</p> <p>「どうせなら自分でプルリクを出してしまえ」</p> <h2 id="リファクタしてからプルリクを作る">リファクタしてからプルリクを作る</h2> <p>ローカルには <code>fix(hooks): VS Code Copilot Chat...</code> というコミットがすでにあった。ただそのまま出すのは品質的に不安だったので、upstream/develop ベースでブランチを切り直して cherry-pick し、ついでに既存のコードベースのお作法を調査してリファクタも入れた。</p> <p>変更の内容は3つ：</p> <ol> <li> <p><strong><code>detect_format()</code> に <code>&quot;run_in_terminal&quot;</code> を追加</strong> VS Code Copilot Chat が渡す <code>tool_name</code> の値で、これが認識されないとフックが PassThrough になっていた。根本的なバグ修正。</p> </li> <li> <p><strong><code>handle_vscode()</code> と <code>handle_copilot_cli()</code> を <code>handle_copilot_rewrite()</code> に統合</strong> ほぼ同じロジックが2関数に分散していたので1つにまとめた。</p> </li> <li> <p><strong>VS Code 向けのレスポンスを <code>allow/ask + updatedInput</code> から <code>deny + suggestion</code> に変更</strong> 前の記事で書いた「承認ループ」の回避。Copilot Chat は <code>updatedInput</code> を承認後に無視するので、Copilot CLI と同じ <code>deny</code> + reason 方式に統一した（詳細は <a href="https://github.com/rtk-ai/rtk/issues/1425">#1425</a>）。</p>