feat: 1セッションから複数PRが出たときの表記を分ける（#12 +2） - #95
Merged
Merged
Conversation
セッションが自分で別ブランチを切って `gh pr create` すると 1 セッションに複数の PR が 紐づくが、一覧はセッションブランチの PR しか出せず 2 本目以降はどこにも出なかった。 - `core/pr-detect.ts`（純粋・新規）: `gh pr create` の tool_use 判定、結果からの PR URL 抽出、参照の追加・除去（変化が無ければ同じ配列参照を返す = state.json を無駄に 書き直さない）、表示ヘルパ（primaryPr / otherPrs / prCount / hasMultiplePrs / allPrs） - `core/sdk-parse.ts`: tool_use id を控えて tool_result と突き合わせる。ログ全体から URL を 拾うと `gh pr list` の出力や他人の PR まで数えるため、作成コマンドの結果だけを走査する - `SessionState.extraPrs` を追加して永続化（壊れた要素は 1 件ずつ落とす）。reducer は ブランチの PR と重複したら畳む - 一覧: `#12 +2`（代表 + 件数）。PR 列は複数 PR の行があるときだけ 10 → 14 桁に広げ、 描画とクリック当たり判定で同じ幅を使う - 詳細: 複数 PR のときだけ `PR 3 件: ✓ #12 · #13 · #14` の 1 行。PrCell / prStatusBadge は `ui/pr-cell.tsx` に共通化 - 代表はセッションブランチの PR（グリフとクリック先を一致させる）。自分で作った PR は codiva が追跡・操作しないので番号のみ。gh の追加呼び出しはゼロ
takecchi
marked this pull request as ready for review
August 6, 2026 14:28
- 行が 2 行に折り返すと rowLineAtPoint（1 セッション = 1 行が前提）が壊れ、以降の行の
クリックが全部ズレる。広い PR 列を使う間はブランチ列の閾値を 4 桁ぶん厳しくし、
PR セル・経過時間を truncate-end に、PR セルの Box を flexShrink={0}（描いた幅と
当たり判定の幅を必ず一致させる）。80 桁での 1 行維持は回帰テストで固定
- `gh pr create … || gh pr list …` のような読み取り系との混在コマンドを検知対象から外す
（一覧に出た PR を全部このセッションの PR として数えてしまう）
- `gh pr create` が「既に PR がある」と既存 URL を返すケースで、ブランチの PR が extraPrs に
二重に積まれるのを検知側で弾く（reducer の畳み込みは pr が変わったときしか走らない）
- addPrRefs が上限到達後も新配列を返していて state.json の再保存が走るのを止める
- 並列 `gh pr create` の保留 id 上限を 4 → 8。順不同・サブエージェント経由のテストを追加
- toPrRef で http(s) 以外の URL を弾く（openUrl に渡るため多層防御）
- 詳細ビューの PR 行を入れ子 Text にして狭い端末でも見出し・番号が切れないように
- 未使用の prCount を削除、正規表現の /g 共有インスタンスをやめる、doc の型記載を修正
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.
背景
セッションが自分で別ブランチを切って
gh pr createすると、1 セッションに複数の PR が紐づきます。しかし一覧の PR セルはセッションブランチ（codiva/<slug>）の PR しか出せず、2 本目以降はどこにも表示されませんでした。変更内容
検知（
core/pr-detect.ts新規・純粋）gh pr createを実行した tool_use の結果に出る PR URL を拾う（sdk-parseが tool_use id を控えて tool_result と突き合わせる）。gh pr listの出力や、gh pr view/ WebFetch で覗いただけの他人の PR まで「このセッションの PR」になってしまう。create_pull_request系ツールもツール名で拾う。state.jsonを無駄に書き直さない / 再描画を増やさない）。状態・永続化
SessionState.extraPrs（識別のみ・永続）とprCreateToolIds（結果待ちの対応付け・transient）を追加。state.jsonに保存・復元（壊れた要素は 1 件ずつ落とす）。reducer はブランチの PR と重複したら畳む。表示
✓ #12 +2（代表 + 残り件数）。PR 列は複数 PR の行があるときだけ 10 → 14 桁に広げるPR 3 件: ✓ #12 · #13 · #14（複数 PR のときだけ 1 行。1 本ならログの縦幅を 1 行も譲らない）prStatus（グリフ）を持つ唯一の PR で、クリック /pで開く先でもあるため、見えている番号と開く先とグリフの意味を一致させた。ghの追加呼び出しはゼロ（API クォータを増やさない）。core/list-hit.tsのprCellWidth）を使う。PrCell/prStatusBadgeはui/pr-cell.tsxに共通化（一覧と詳細で同じグリフ・同じ色）。ドキュメント
README.md（利用者向けの表記説明）/docs/ARCHITECTURE.md（状態フィールド・設計判断）/docs/TASKS.md（Phase 27）/CLAUDE.md（コードの地図）。テスト
src/core/pr-detect.spec.ts（新規・テーブルドリブン）: コマンド判定 / URL 抽出（GHE ホスト・サブパス・重複・issues リンク）/ 参照同一性 / 上限 / 表示ヘルパ。src/core/sdk-parse.spec.ts: 実フィクスチャと同じメッセージ形で tool_use → tool_result の対応、gh pr listを数えないこと、ブランチ PR と判明したら畳むこと。src/core/persistence.spec.ts: 往復と壊れた JSON の扱い。src/core/list-hit.spec.ts: 列幅の切替。tests/app.test.tsx: 一覧に✓ #42 +2が出て#43は出ないこと、詳細に全件が並ぶこと、1 本のときは詳細に PR 行を出さないこと。動作確認
npm run lint/npm run typecheck/npm test（2,232 件）/npm run build#12 +2になり、PR セルのクリックで代表の PR が開く / 詳細ビューに全件が並ぶ / 再起動しても+nが残る。🤖 Generated with Claude Code