AIプログラマー

1.前提：小さく速く出す

• 1PRあたり 300行以内 を目安（レビュー品質が落ちにくい）
• 1機能＝1PR（リネームや整形だけのPRは分離）
• 変更の目的・背景・スクショ/ログ・動作確認手順を必ずPRに記載

2.レビューフロー（実運用）

1. ドラフトPR作成：方針が合っているかを早めに確認
2. 自己レビュー：差分を自分で1度読み、指摘を先に潰す
3. 自動チェック（CI）：lint/型/テスト/脆弱性スキャン
4. 人のレビュー：レビュワー2名 or 1名＋CIパス
5. 修正→再レビュー→マージ：Squash mergeで履歴を綺麗に

3.PRテンプレ（コピペ可）

### 目的
- 何の課題を解決するPRか（リンクやチケットID）

### 変更点
- 主要な変更の箇条書き

### 動作確認
- 再現手順/スクショ/動画/GIF
- 期待結果と実際の結果

### 影響範囲
- 既存機能への影響、非互換の有無、マイグレーション

### テスト
- 追加/修正したテスト一覧とカバレッジ
- 手動確認の観点

### リスクと対策
- フィーチャーフラグ/ロールバック手順/監視アラート

### 関連
- Issue/仕様書/デザイン

4.レビュー観点チェックリスト

設計

• 単一責任/依存関係が最小か
• public APIの破壊的変更がないか

可読性

• 命名・コメント・早期return・重複排除
• 複雑度（ネスト/分岐）を抑制しているか

正しさ

• 境界値/例外系/国際化/時刻/並行性
• I/O（DB・API・ファイル）の失敗時挙動

性能/コスト

• N+1・不要ループ・大きなオブジェクトコピー
• ネットワーク/ストレージ/メモリの無駄

セキュリティ

• 入力検証・エスケープ・秘密情報の扱い
• 依存パッケージの脆弱性

テスト

• テストピラミッドの適切な層に追加されているか
• 再現テストが先に書かれているか（バグ修正時）

5. レビュコメントの書き方（例）

• NG: 「ここダメ」
• OK: 「parseDate()はTZ依存の副作用がありそう。parseISO + utcToZonedTimeにするとE2Eの不安定が減ります（参考: 〇〇）。修正候補↓」
• 提案は最小diffでコード例を添える／必須・任意を明示（Must/Should/Nice）

1.戦略（テストピラミッド）

• ユニット（多い）：純粋関数・ドメインロジックを高速に網羅
• 統合（ほどほど）：DB/外部APIを含む実配線の確認
• E2E（少なめ）：ユーザー視点で主要シナリオを通す
• フロントは コンポーネントテスト を厚めに（Storybook＋再利用）

2.目標値（現実的ライン）

• ステートメントカバレッジ 80%（無理に100%を狙わない）
• 重要ユースケースの E2E 5〜10本 を安定化
• 変更の多い層・リスクの高い領域から先にテストを書く

3. 具体例：JavaScript/TypeScript

ユニット（Jest + ts-jest）

npm i -D jest @types/jest ts-jest
npx ts-jest config:init

// sum.ts
export const sum = (a:number,b:number)=>a+b;

// sum.test.ts
import { sum } from './sum';
test('adds', ()=>{ expect(sum(2,3)).toBe(5); });

統合（Supertest + Express）

// app.test.ts
import request from 'supertest';
import { app } from './app';
it('GET /users returns 200', async ()=>{
  const res = await request(app).get('/users');
  expect(res.status).toBe(200);
  expect(res.body).toEqual(expect.arrayContaining([expect.objectContaining({ id: expect.any(String) })]));
});

E2E（Playwright）

npm i -D @playwright/test
npx playwright test --ui

// e2e.spec.ts
import { test, expect } from '@playwright/test';
test('ログイン→購入フロー', async ({ page })=>{
  await page.goto('http://localhost:3000');
  await page.getByRole('button', { name: 'ログイン' }).click();
  await page.getByLabel('Email').fill('user@example.com');
  // ...
  await expect(page.getByText('購入完了')).toBeVisible();
});

Lint/型

npm i -D eslint prettier typescript ts-node @typescript-eslint/parser @typescript-eslint/eslint-plugin

• PRで eslint –max-warnings=0、tsc –noEmit を必須化

4.具体例：Python

ユニット（pytest）

pip install pytest pytest-cov
pytest -q --cov=src --cov-report=term-missing

# src/tax.py
def calc(price: int, rate: float) -> int:
    if price < 0: raise ValueError("price")
    return int(round(price * (1+rate)))

# tests/test_tax.py
import pytest
from src.tax import calc
def test_calc_basic(): assert calc(100, 0.1)==110
def test_calc_negative(): 
    with pytest.raises(ValueError): calc(-1, 0.1)

統合（FastAPI + httpx + Test DB）

• Test DBを Dockerで分離、マイグレーションを毎回適用
• 外部APIは responses / respx でモック

E2E（Playwright/Pytest）：RESTを立ててシナリオ通し

5.モック/スタブ/フィクスチャ

• ユニット：外部I/Oはモックし、純粋にロジックを検証
• 統合：主要な外部だけ実体 or TestContainer（Docker）で
• フィクスチャ：最小データをfixtures/に置き、生成関数で重複排除

6.回帰防止の黄金パターン

• バグを見つけたら 再現テストを先に追加 → 修正 → 緑でマージ
• スナップショットは乱用しない（脆くなる）
• 日付・乱数・IDは シード固定 で安定化

---

自動化（CI）の実例

# .github/workflows/ci.yml
name: CI
on:
  pull_request:
    branches: [ main ]
jobs:
  build-test:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v4
      - uses: actions/setup-node@v4
        with: { node-version: '20' }
      - run: npm ci
      - run: npm run lint
      - run: npm run typecheck
      - run: npm run test -- --coverage
      - name: Upload coverage
        uses: codecov/codecov-action@v4
  security:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v4
      - run: npm audit --audit-level=high || true
      - run: npx depcheck || true

NPMスクリプト例

{
  "scripts": {
    "lint": "eslint . --max-warnings=0",
    "typecheck": "tsc --noEmit",
    "test": "jest --runInBand"
  }
}

Python用ジョブ（追加）

  py-tests:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v4
      - uses: actions/setup-python@v5
        with: { python-version: '3.11' }
      - run: pip install -r requirements.txt
      - run: pytest -q --cov=src --cov-report=xml

---

さらに品質を上げる仕掛け

pre-commit（ローカルでの自動整形・静的検査）

• Node: husky + lint-staged
• Python: pre-commit（black/isort/flake8）
• コミット前に数秒で「整形→簡易チェック」まで回して、PRに“汚れ”を持ち込まない

セキュリティ＆依存管理

• Dependabotで依存を自動更新
• Node: npm audit、Python: pip-audit
• Python: bandit、JS: eslint-plugin-security をCIに

変更検知テスト（速い）

• 影響のあるテストだけ実行：Jestの --findRelatedTests、pytestの -k など

観測性

• 重要E2Eには ログ/メトリクス/トレース の検証を含める
• 本番でエラー検知 → 再現テスト 化のサイクルを短く

---

AIを使ったレビュー/テスト支援の実例

レビュー用プロンプト（ChatGPT/Claudeなど）

役割: シニアエンジニアのコードレビュア
入力: 変更差分(抜粋)・要件・仕様・テストコード
出力: Must/Should/Nice の指摘。根拠・修正例・安全性/性能/可読性の観点を必ず含める。
制約: 変更の意図を尊重し、最小差分の修正提案を返す。冗長なリファクタは避ける。

テスト生成プロンプト

役割: テストエンジニア
入力: 関数/モジュールの仕様と境界条件
出力: ユニットテストのケース表(正常/異常/境界)とJest/pytestの実装テンプレ
制約: モック範囲を明示。データは最小化。ID/時刻は固定化。

---

フォルダ構成（例）

/app
  /src
    /components
    /lib
    /services
  /tests
    /unit
    /integration
    /e2e
  /fixtures
  jest.config.ts / pytest.ini

---

まずはこの順で着手（今日から運用）

1. PRテンプレを追加
2. ESLint/Prettier/型チェックをスクリプト化
3. Jest or pytest を最小1本で通す（成功体験）
4. GitHub Actions を貼ってPRで自動実行
5. PlaywrightでE2Eを1本だけ通す（ログイン等の幸せルート）
6. Dependabot + audit をON
7. バグ報告→ 再現テスト→修正 を「習慣化

• 脱字の確認
• リンクの確認
• 画面表示の確認
• パソコン表示
• 携帯表示