// ❌ 問題のコード（52行の重複実装）
func businessHandler(ctx context.Context, request events.APIGatewayProxyRequest, 
    log *logger.Logger, metricsClient *middleware.MetricsClient) (events.APIGatewayProxyResponse, error) {
    
    // Authorizationヘッダーを取り出す
    authHeader := request.Headers["Authorization"]
    if authHeader == "" {
        authHeader = request.Headers["authorization"] // 小文字も試す
    }

    // ...ログ出力処理...

    if authHeader == "" {
        // ヘッダーがない場合のエラー処理
        return utils.CreateErrorResponse(401, "Unauthorized", "Authorization header missing"), nil
    }

    // "Bearer <token>" の形式をパースする
    parts := strings.Split(authHeader, " ")
    if len(parts) != 2 || parts[0] != "Bearer" {
        // フォーマットが違う場合のエラー処理
        return utils.CreateErrorResponse(401, "Unauthorized", "Invalid authorization header format"), nil
    }

    accessToken := parts[1]
    // ... 続く処理
}

// ✅ 既存の優れた実装 in pkg/utils/auth.go
// ExtractTokenFromRequest は、リクエストのヘッダーからJWTトークンを抽出します。
// 大文字・小文字を区別せず、様々な形式のヘッダーに対応します。
func ExtractTokenFromRequest(request events.APIGatewayProxyRequest) (string, error) {
    // ...（中略）...
}

// ExtractTokenFromHeader は、"Bearer <token>"という文字列からトークン部分だけを抽出します。
func ExtractTokenFromHeader(authHeader string) (string, error) {
    // ...（中略）...
    // strings.Fields() を使っているので、複数スペースにも対応！
    parts := strings.Fields(authHeader) 
    // strings.ToLower() で小文字に変換しているので、"bearer"でもOK！
    if strings.ToLower(parts[0]) != "bearer" {
        return "", ErrInvalidAuthFormat
    }
    // ...（中略）...
}

// cmd/get-user/authorization_test.go
func TestAuthorizationExtraction_Consistency(t *testing.T) {
    testCases := []struct {
        name          string
        headers       map[string]string
        // ...
    }{
        // ...
        {
            name: "小文字のbearer（現在の実装では失敗するはず）",
            headers: map[string]string{"Authorization": "bearer my-token"},
            // ...
        },
        {
            name: "複数スペース（現在の実装では失敗するはず）",
            headers: map[string]string{"Authorization": "Bearer   my-token"},
            // ...
        },
    }

    // ... テスト実行ロジック ...
    // 期待：現在の実装(currentToken)ではエラーになり、共通関数(utilsToken)では成功する
}

// ✅ 改善後のコード（たったこれだけ！）
func businessHandler(ctx context.Context, request events.APIGatewayProxyRequest, 
    log *logger.Logger, metricsClient *middleware.MetricsClient) (events.APIGatewayProxyResponse, error) {
    
    // ...（前略）...

    // 改善ポイント：自前の50行以上の処理を、共通関数の呼び出し1行に置き換える！
    accessToken, err := utils.ExtractTokenFromRequest(request)
    if err != nil {
        // エラーハンドリングも、共通関数が返す詳細なエラーを使うことで、より丁寧になる
        // ...（エラー処理）...
        return response, nil
    }

    // ...（後略）...
}

// ✅ 最終的に短縮されたメインロジック
func businessHandler(...) (events.APIGatewayProxyResponse, error) {
    // ...（前略）...

    // 認証処理は、たったの3行に！
    accessToken, err := utils.ExtractTokenFromRequest(request)
    if err != nil {
        // エラー処理は別の関数に任せることで、メインの流れがスッキリ
        return handleAuthenticationError(err, ...), nil
    }

    // ...（後略）...
}

// ✅ エラーハンドリングを専門に行う関数
// こうすることで、他の場所でも同じエラー処理を使い回せるようになる
func handleAuthenticationError(...) events.APIGatewayProxyResponse {
    // ...（エラーの種類に応じて、適切なレスポンスを作成する処理）...
}