Skip to content

fix: PR本文のGitHub補完を有効にする - #266

Merged
kyu08 merged 5 commits into
mainfrom
fix/pr-body-completion
Oct 27, 2026
Merged

kyu08 merged 5 commits into
mainfrom
fix/pr-body-completion

Conversation

@kyu08

@kyu08 kyu08 commented Oct 27, 2026 •

Copy link
Copy Markdown
Collaborator

概要

PR作成と編集の本文で、@user、#issue/PR、_commitの補完を使えるようにする。
レビューセッションを開始していなくても候補を取得する。

Closes #253

変更内容

  • PR本文を補完対象として登録し、blink.cmpとnvim-cmpの両方から利用する。既存のb:fude_commentはコメント画面の終了判定にも使うため、PR本文には付けない。
  • コミット候補は、編集時には対象PRから、作成時には選択したbaseからHEADまでのローカル履歴から取得する。これにより、レビュー中の別PRに依存せず、作成前の未pushコミットも候補に含める。
  • PR本文ごとに作業ディレクトリとキャッシュを保持する。フォーカス移動や:cdで取得先が変わることを防ぎ、画面終了後の応答は反映しない。コマンドの起動失敗も通常のエラー経路で処理し、待機状態を解除して再試行できるようにする。

テスト計画

  • make allでlint、整形、全テスト、状態依存、純粋性、ドキュメント整合性を確認した。
  • 作成と編集の両画面で、両補完アダプターの3種類の呼び出しを検証した。タイトル欄では無効で、PR本文がコメント画面の終了処理に巻き込まれないことも確認した。
  • 一時gitリポジトリを使い、未pushコミット、remoteにだけ存在するbase、cwdとHEADの変更、空の履歴、取得失敗後の再試行を検証した。gitとghの起動失敗もテストした。
  • GitHub呼び出しへのcwdの引き渡しを一時的に外すと、追加した3件のテストが失敗することを確認し、実装を元に戻した。

備考

補完プラグインのポップアップを操作する手動確認は未実施。
Copilotレビューは待たず、PR作成後にもセルフレビューを行い、コマンド起動失敗時の処理を修正した。PRはdraftのままにする。

Copilot AI balanced review requested due to automatic review settings October 27, 2026 13:52

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

Copilot AI balanced review requested due to automatic review settings October 27, 2026 14:04

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

@kyu08
kyu08 marked this pull request as ready for review October 27, 2026 14:07
@kyu08
kyu08 merged commit 88d55d1 into main Oct 27, 2026
9 of 10 checks passed
@kyu08
kyu08 deleted the fix/pr-body-completion branch October 27, 2026 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PR Bodyの作成/編集画面でuser/commit hash/issueの補完が効かない

2 participants