Skip to content

feat(web): continue blocked issues from comments - #298

Merged
version-1 merged 3 commits into
mainfrom
feat/continue-from-blocker-comment
Jul 31, 2026
Merged

feat(web): continue blocked issues from comments#298
version-1 merged 3 commits into
mainfrom
feat/continue-from-blocker-comment

Conversation

@version-1

Copy link
Copy Markdown
Owner

blocker コメントから Change Request を作成して Issue を継続

Summary

blocked Issue の最新 blocker コメントから、Change Request を作成して Issue を ready に戻せる操作を追加します。既存 Reject の処理とモーダルを共通化し、導線ごとに表示文言を切り替えます。

主な変更点:

  • 全コメントカード右上に3点リーダーを追加
  • blocked Issue の最新 blocker コメントだけに「コメントをつけて継続」を表示
  • Change Request 作成と ready 遷移を Reject と共通化
  • 日本語・英語の専用文言と回帰テストを追加

やったこと

  • コメントカードへ既存 ContextMenu / IconProxy を使った操作メニューを追加
  • コメントが複数ページにまたがる blocked Issue では全ページ取得後に最新 blocker を確定
  • 後続ページ取得失敗時の自動リトライループを防止し、手動 Load more による再試行を維持
  • RejectIssueDialog を ChangeRequestDialog へ一般化し、Reject / Continue の variant を追加
  • Change Request 本文は空欄で開始し、author は reviewer 固定
  • Storybook の Continue variant とページング境界を含むテストを追加

動作確認

  • npm run typecheck
  • npm test(21 files / 111 tests)
  • git diff --check origin/main...HEAD

レビュー & 動作確認 チェックリスト

  • 操作対象 - blocked Issue の最新 blocker コメントだけにメニュー項目が表示されること
  • 非対象コメント - 3点リーダーを押してもメニューが開かないこと
  • Change Request - モーダルが空欄で開き、送信後に Issue が ready へ移ること
  • ページング - 最新 blocker が後続ページにある場合も古い blocker が操作可能にならないこと

推奨テスト計画:

  1. blocked Issue の Comments タブを開き、各コメントの3点リーダーを確認する
  2. 最新 blocker の「コメントをつけて継続」から本文を入力して送信する
  3. Change Request が作成され、Issue が ready へ移ることを確認する

その他気になることや相談ごと

共通 ContextMenu / Dialog には、フォーカス移動・フォーカストラップ・Escape 操作などの横断的なアクセシビリティ改善余地があります。既存 Reject を含む共通課題のため、この PR のスコープ外としています。

Add comment-card action menus and expose the continue flow only on the latest blocker comment of a blocked issue. Generalize the reject dialog so both entry points create reviewer change requests and move the issue to ready with route-specific copy. Add Storybook states and issue-detail regression coverage.\n\nVerified with npm run typecheck and npm test (110 tests).
Automatically load remaining comment pages for blocked issues and keep blocker actions inactive until the final page is available. Cover the paginated latest-blocker selection in the issue detail regression test.\n\nVerified with npm run typecheck and the issue detail test suite.
Pause automatic blocked-comment pagination when a request error is present so a failed later page does not trigger an immediate request loop. Add regression coverage for the two-request failure boundary.\n\nVerified with npm run typecheck and 16 issue detail tests.
@version-1
version-1 merged commit 39a20f1 into main Jul 31, 2026
5 checks passed
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.

1 participant