モジュールを外から見た振る舞いだけAIレビューさせる smoke-review

$ git clone https://github.com/mizchi/ts-starter-with-smoke-review myproj
$ cd myproj
$ claude
> /smoke-review

export * from "./core";
export * from "./utils";
export * from "./internal-methods";

function calc() {
  
}

it("calc() は 42 を返す", () => {
  const expected = 1;
  const output = calc();
  expect(output).toBe(expected);
});

import { dts } from "rolldown-plugin-dts";
export default {
  input: "./src/index.ts",
  plugins: [dts()],
  output: [{ dir: "dist", format: "es" }],
};

type Rect = {
  x: number;
  y: number;
  width: number;
  height: number;
};
declare function calculateAreaDifference(a: Rect, b: Rect): number;
type SimilarityResult = {
  firstIndex: number;
  secondIndex: number;
  similarity: number;
};
declare function calculatePairwiseSimilarities(
  rects: Rect[]
): SimilarityResult[];

export {
  Rect,
  SimilarityResult,
  calculateAreaDifference,
  calculatePairwiseSimilarities,
};

.claude/scripts/get-vitest-specs.bash

#!/bin/bash


set -e


cd "$(dirname "$0")/../.."
PROJECT_ROOT=$(pwd)


TEMP_FILE=$(mktemp)
trap "rm -f $TEMP_FILE" EXIT


if [ -d "tests" ]; then
    
    npx vitest run tests --reporter=json > "$TEMP_FILE" 2>/dev/null || true
else
    
    npx vitest run --reporter=json > "$TEMP_FILE" 2>/dev/null || true
fi


if command -v jq &> /dev/null; then
    jq -r --arg root "$PROJECT_ROOT" '
      .testResults |
      group_by(.name) |
      .[] |
      .[0] as $suite |
      "\n\($suite.name | sub($root + "https://zenn.dev/"; ""))" +
      ($suite.assertionResults |
        group_by(.ancestorTitles[0] // "") |
        map(
          if .[0].ancestorTitles[0] then
            "\n  \(.[0].ancestorTitles[0])" +
            (map("\n    - \(.title) [\(.status)]") | join(""))
          else
            map("\n  - \(.title) [\(.status)]") | join("")
          end
        ) | join("")
      )
    ' "$TEMP_FILE" 2>/dev/null
else
    echo "Error: jq is required"
    exit 1
fi

tests/integration.test.ts
  Rectangle Library Integration
    - should calculate area difference for UI layout comparison [passed]
    - should handle game object collision detection preparation [passed]
    - should work with responsive design breakpoints [passed]
    - should handle floating point rectangles [passed]
    - should handle very large rectangles without overflow [passed]
    - should work with array of rectangles [passed]
    - should support functional programming patterns [passed]

---
name: smoke-reviewer
description: 公開型定義とインテグレーション命名と一覧を評価し、ユーザー視点でモジュールを評価します
color: yellow
---

このライブラリのユーザー視点でレビューします。提供されている API に対してどのようなテストケースが準備されているかを確認し、必要なテストケースの修正/追加/削除を提案します。

## 入力

1. バンドルされた公開 API(`dist/index.d.ts`)
2. インテグレーションテストの一覧： `bash .claude/scripts/get-vitest-specs.bash`

**重要**: 実装コードを参照せずに、型定義の公開 API と、インテグレーションテストのスペックのみでインターフェースを評価してください。

/smoke-review
...

● 3つのサブエージェントを並行して実行し、ライブラリをレビュー
  します。

● smoke-public-api-reviewer(Review public API)
  ⎿  Done (2 tool uses · 17.8k tokens · 37.5s)

● smoke-examples-reviewer(Review examples)
  ⎿  Done (12 tool uses · 25.8k tokens · 2m 3.1s)

● smoke-integration-reviewer(Review integration tests)
  ⎿  Done (6 tool uses · 22.4k tokens · 55.5s)

● スモークレビュー結果サマリー

  1. 公開APIレビュー 🔴 要改善

  - APIリネームは完了したが、基本的な矩形操作機能が不足
  - calculateAreaDifference の名前は改善されたが、まだ不明確
  - 矩形計算ライブラリとして期待される基本機能（面積計算、交
  差判定など）がない

  2. サンプルコードレビュー 🟢 良好

  - calculatePairwiseSimilarities のサンプルが追加されて改善
  - 実用的なユースケースが豊富
  - ただし、類似度の計算アルゴリズムの説明が不足

  3. 統合テストレビュー 🟢 良好

  - 実用的なシナリオが適切にテストされている
  - エッジケースの基本的なカバレッジがある
  - calculatePairwiseSimilarities のより詳細なテストが必要

  総合評価: 部分的改善 🟡

  前回のレビューから以下の改善が確認されました：
  - ✅ API名の改善（calculateSurface →
  calculateAreaDifference）
  - ✅ 類似度計算のサンプルコード追加
  - ✅ SimilarityResult のプロパティ名改善（rect1/rect2 →
  firstIndex/secondIndex）

  ただし、まだ以下の課題が残っています：
  - ❌ 基本的な矩形操作関数の不足
  - ❌ 類似度計算アルゴリズムの説明不足
  - ❌ JSDocコメントによる関数の動作説明がない

  次のステップの提案:
  1. 基本的な矩形操作関数（area, intersects, contains
  など）を追加
  2. 各関数にJSDocコメントを追加して動作を明確化
  3. 類似度の計算方法についてドキュメントを追加