# PR: GitHub連携サイドバーの遷移/実行分離 と ADR-0007 完全移行方針への更新

[yusuke0610]

概要

2 つの独立した変更を含む。

1. GitHub連携サイドバー項目の責務分離（UI 改善） — サイドバーの「GitHub連携」を、クリックで連携 API をトリガーする button から、画面遷移に徹する NavLink へ変更。連携の実行はサブパネル内の「連携実行」ボタンに分離した。
2. ADR-0007 の方針更新 — OpenAPI → TypeScript 型生成を「導入検討（折衷案あり）」から「完全移行（手書き DTO 全廃）」方針へ書き換え。

1. GitHub連携サイドバーの責務分離

背景・課題

従来はサイドバーの「GitHub連携」項目クリックがそのまま連携 API（/api/github-link/run）をトリガーしていた。単に画面を見たいだけの遷移でも毎回連携リクエストが飛ぶため、不要なリクエストの温床になっていた。

変更内容

• frontend/src/components/AuthenticatedLayout.tsx
  • 「GitHub連携」項目を button（triggerGitHubLink 呼び出し）→ NavLink（/github_link へ遷移のみ）に変更。
  • 連携実行は、▼ で開くサブパネル内の新規「連携実行」ボタンに移設。
  • active 判定を useLocation の手動比較から NavLink の isActive に置き換え（useLocation import 削除）。

テスト

• frontend/e2e/github-link.spec.ts
  • 「サイドバークリックは画面遷移のみで連携 API を呼ばない」ことを検証するテストを追加（runCalled === false を assert）。
  • 「サブパネルの『連携実行』ボタンで連携が走りポーリング表示になる」ことを検証するテストへ更新。
• frontend/e2e/navigation.spec.ts
  • GitHub連携 をボタン→リンクとして期待する形に修正。

サイドバー（ナビゲーション/レイアウト）コンポーネントの変更のため E2E を実行。

2. ADR-0007 の方針更新

OpenAPI → TypeScript 型生成について、当初の「Phase 1 パイロットの結果を見て採用判断」「FE 独自名は alias で温存する折衷案」から、手書き DTO 型を全廃し生成物へ完全移行する方針へ更新した。リリース前で呼び出し側 rename コストが低いことが判断根拠。

主な追記:

• 命名統一マップ（FE 独自名 → BE 名）の追加。CareerResumeResponse → ResumeResponse 等、呼び出し側ごと BE 名へ rename する。
• Phase 0 の前提作業を明確化：response_model 未設定エンドポイント（実測 40 中 14）の棚卸し・schema 化・付与を完全移行の前提として内包。
• トレードオフに 入出力兼用 schema の -Input/-Output 分裂・optional/required の解釈ズレ・新規 schema 命名規約の論点を追加。
• status は Proposed 据え置き（Phase 0 + Phase 1 が make ci green かつ CI ドリフト検知が機能した時点で Accepted に昇格）。

ドキュメント（ADR）のみの変更。実装は本 ADR が定義する後続 PR で行う。

確認事項

• make ci
• npm run test:e2e（サイドバー/ナビゲーション変更のため必須）

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: e85b8a05-5400-4662-9e25-624c62acb49b

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch refactor/backend/deadcode

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}