feat: PR 作成フロー（push→gh pr create）＋ツール診断 doctor#33
Merged
Conversation
#14: セッションバーに PR ボタンを追加。PR が無ければ作成シート （ベースブランチ/タイトル/本文/Draft 既定ON）→ git push -u origin HEAD → gh pr create → 作成した PR をブラウザで開く。PR があればブラウザで開く。 gh の stderr をエラーに載せ、未認証・重複PR などの理由を UI に表示。 #15: ToolDoctor が起動時に git/gh/claude の存在とバージョンを検査 （GUI の乏しい PATH を考慮し Homebrew/mise shims/~/.claude/local も探索）。 設定 > 一般に「ツール診断」セクション（状態・バージョン・再検査）。 claude 不在は起動ボタン、gh 不在は PR ボタンを無効化し理由を help に表示。 Closes #14 Closes #15
This was referenced Jul 2, 2026
sasagar
added a commit
that referenced
this pull request
Jul 2, 2026
コードレビュー(#33)で見つかった「doctor のゲートが実行経路と食い違う」系の 不具合をまとめて対処する。 - ToolLocator にツール解決を集約（固定候補＋PATH＋ログインシェル）。doctor(gh) の判定と GitHubEngine.locateGH が同じ解決を使い、PR ボタンの有効状態と実際の gh 起動可否が一致する（mise shims / Nix パスの非対称を解消）。 - Claude 起動ボタンの doctor 依存の無効化を撤去。起動は端末（ログインシェル）で claude を解決するため、検出漏れで中核機能を塞がない。未検出時は help で注意喚起。 - ProcessRunner（両パイプ並行 drain＋timeout）を共有化し、ToolDoctor.runVersion の 未 drain・無 timeout による恒久ハング（checking が固まる）を解消。 - gh pr create の stdout から URL 行を抽出（助言行が混ざっても PR を開ける）。 - PR 作成失敗時、空 stderr で空エラーラベルにならないよう localizedDescription に フォールバック。 - createPullRequest はセッション未検出時に "" を返さず throw（黙って dismiss しない）。 - GitHubEngine.parsePRURL / parseChecks に単体テストを追加。
sasagar
added a commit
that referenced
this pull request
Jul 2, 2026
#14: セッションバーに PR ボタンを追加。PR が無ければ作成シート （ベースブランチ/タイトル/本文/Draft 既定ON）→ git push -u origin HEAD → gh pr create → 作成した PR をブラウザで開く。PR があればブラウザで開く。 gh の stderr をエラーに載せ、未認証・重複PR などの理由を UI に表示。 #15: ToolDoctor が起動時に git/gh/claude の存在とバージョンを検査 （GUI の乏しい PATH を考慮し Homebrew/mise shims/~/.claude/local も探索）。 設定 > 一般に「ツール診断」セクション（状態・バージョン・再検査）。 claude 不在は起動ボタン、gh 不在は PR ボタンを無効化し理由を help に表示。 Closes #14 Closes #15
sasagar
added a commit
that referenced
this pull request
Jul 2, 2026
コードレビュー(#33)で見つかった「doctor のゲートが実行経路と食い違う」系の 不具合をまとめて対処する。 - ToolLocator にツール解決を集約（固定候補＋PATH＋ログインシェル）。doctor(gh) の判定と GitHubEngine.locateGH が同じ解決を使い、PR ボタンの有効状態と実際の gh 起動可否が一致する（mise shims / Nix パスの非対称を解消）。 - Claude 起動ボタンの doctor 依存の無効化を撤去。起動は端末（ログインシェル）で claude を解決するため、検出漏れで中核機能を塞がない。未検出時は help で注意喚起。 - ProcessRunner（両パイプ並行 drain＋timeout）を共有化し、ToolDoctor.runVersion の 未 drain・無 timeout による恒久ハング（checking が固まる）を解消。 - gh pr create の stdout から URL 行を抽出（助言行が混ざっても PR を開ける）。 - PR 作成失敗時、空 stderr で空エラーラベルにならないよう localizedDescription に フォールバック。 - createPullRequest はセッション未検出時に "" を返さず throw（黙って dismiss しない）。 - GitHubEngine.parsePRURL / parseChecks に単体テストを追加。
sasagar
added a commit
that referenced
this pull request
Jul 2, 2026
#14: セッションバーに PR ボタンを追加。PR が無ければ作成シート （ベースブランチ/タイトル/本文/Draft 既定ON）→ git push -u origin HEAD → gh pr create → 作成した PR をブラウザで開く。PR があればブラウザで開く。 gh の stderr をエラーに載せ、未認証・重複PR などの理由を UI に表示。 #15: ToolDoctor が起動時に git/gh/claude の存在とバージョンを検査 （GUI の乏しい PATH を考慮し Homebrew/mise shims/~/.claude/local も探索）。 設定 > 一般に「ツール診断」セクション（状態・バージョン・再検査）。 claude 不在は起動ボタン、gh 不在は PR ボタンを無効化し理由を help に表示。 Closes #14 Closes #15
sasagar
added a commit
that referenced
this pull request
Jul 2, 2026
コードレビュー(#33)で見つかった「doctor のゲートが実行経路と食い違う」系の 不具合をまとめて対処する。 - ToolLocator にツール解決を集約（固定候補＋PATH＋ログインシェル）。doctor(gh) の判定と GitHubEngine.locateGH が同じ解決を使い、PR ボタンの有効状態と実際の gh 起動可否が一致する（mise shims / Nix パスの非対称を解消）。 - Claude 起動ボタンの doctor 依存の無効化を撤去。起動は端末（ログインシェル）で claude を解決するため、検出漏れで中核機能を塞がない。未検出時は help で注意喚起。 - ProcessRunner（両パイプ並行 drain＋timeout）を共有化し、ToolDoctor.runVersion の 未 drain・無 timeout による恒久ハング（checking が固まる）を解消。 - gh pr create の stdout から URL 行を抽出（助言行が混ざっても PR を開ける）。 - PR 作成失敗時、空 stderr で空エラーラベルにならないよう localizedDescription に フォールバック。 - createPullRequest はセッション未検出時に "" を返さず throw（黙って dismiss しない）。 - GitHubEngine.parsePRURL / parseChecks に単体テストを追加。
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #14
Closes #15
概要
セッションから PR を作成できるようにし（#14）、外部ツールの存在検査と機能ゲートを追加（#15）。
#14: PR 作成
git push -u origin HEAD→gh pr create→ 作成 PR をブラウザで開く → バッジ再取得。ghの stderr をエラー表示（未認証・重複 PR などの理由が見える）。#15: doctor
ToolDoctor: 起動時に git / gh / claude の存在＋バージョンを検査。GUI の乏しい PATH を考慮し Homebrew・mise shims・~/.claude/local等も探索。確認
swift test全緑・xcodebuild成功・起動してクラッシュ無し。