Claude Codeで例外処理を改善する方法

このプロジェクトの例外処理を改善してください。
目的は以下です。

・想定されるエラーを明確にする
・不要なtry-catchを減らす
・ログに原因調査しやすい情報を残す
・ユーザーに内部エラーの詳細を表示しない
・既存の動作を壊さない
・必要に応じてテストを追加する

このリポジトリ内の例外処理を調査してください。
try-catch、throw、Promiseのcatch、エラーログ出力、独自例外クラスの使われ方を確認し、問題点を一覧化してください。
まだコードは変更しないでください。

try {
  await saveUser(user);
} catch (e) {
}

空のcatchブロックや、エラーを握りつぶしている処理を探してください。
必要なログ出力、再throw、ユーザー向けエラーメッセージへの変換を提案し、最小限の変更で修正してください。

try {
  await saveUser(user);
} catch (error) {
  logger.error("ユーザー保存に失敗しました", {
    userId: user.id,
    error,
  });

  throw new Error("ユーザー情報の保存に失敗しました。");
}

この処理で発生しうるエラーを分類してください。
バリデーションエラー、認証エラー、権限エラー、外部APIエラー、DBエラー、想定外エラーに分けて、適切な例外処理に修正してください。

class AppError extends Error {
  constructor(
    message: string,
    public statusCode: number,
    public code: string
  ) {
    super(message);
  }
}

class ValidationError extends AppError {
  constructor(message: string) {
    super(message, 400, "VALIDATION_ERROR");
  }
}

非同期処理の例外処理漏れを確認してください。
await漏れ、Promiseのcatch漏れ、Expressのasync handlerで捕捉されないエラーがないか確認し、必要な修正をしてください。

const asyncHandler = (fn) => {
  return (req, res, next) => {
    Promise.resolve(fn(req, res, next)).catch(next);
  };
};

app.get("/users/:id", asyncHandler(async (req, res) => {
  const user = await getUser(req.params.id);
  res.json(user);
}));

API全体のエラーハンドリングを共通化してください。
各ルートで重複しているtry-catchを減らし、共通エラーハンドラーでレスポンス形式を統一してください。
既存のAPIレスポンス仕様は壊さないでください。

app.use((err, req, res, next) => {
  logger.error("APIエラーが発生しました", {
    path: req.path,
    method: req.method,
    error: err,
  });

  const statusCode = err.statusCode || 500;

  res.status(statusCode).json({
    success: false,
    error: {
      code: err.code || "INTERNAL_SERVER_ERROR",
      message: statusCode === 500
        ? "サーバーエラーが発生しました。"
        : err.message,
    },
  });
});

エラーログの内容を見直してください。
原因調査に必要な情報は残しつつ、パスワード、トークン、個人情報などの機密情報を出力しないように修正してください。

res.status(500).json({
  error: error.message
});

res.status(500).json({
  error: {
    code: "INTERNAL_SERVER_ERROR",
    message: "処理中にエラーが発生しました。時間をおいて再度お試しください。"
  }
});

ユーザーに返しているエラーメッセージを確認してください。
内部エラーの詳細、SQL、スタックトレース、ファイルパス、外部APIの詳細レスポンスを返している箇所があれば、安全なメッセージに変更してください。

今回変更した例外処理に対してテストを追加してください。
正常系、バリデーションエラー、外部API失敗、DBエラー、想定外エラーのケースを確認してください。

it("ユーザー保存に失敗した場合はエラーレスポンスを返す", async () => {
  saveUserMock.mockRejectedValue(new Error("DB connection failed"));

  const response = await request(app)
    .post("/users")
    .send({ name: "test" });

  expect(response.status).toBe(500);
  expect(response.body.error.code).toBe("INTERNAL_SERVER_ERROR");
  expect(response.body.error.message).toBe("サーバーエラーが発生しました。");
});

このプロジェクトの例外処理を調査してください。
問題のある箇所をファイル名、行付近、問題内容、改善案の形式で一覧化してください。
まだ修正はしないでください。

先ほど見つけた問題のうち、影響範囲が小さいものから順に修正してください。
既存仕様を変えず、エラー処理、ログ、レスポンス形式を改善してください。
変更後にテストも追加してください。

今回の変更をレビューしてください。
例外の握りつぶし、過剰なtry-catch、ログの機密情報、ユーザー向けメッセージ、テスト不足がないか確認してください。

例外処理改善の変更差分を確認してください。
本番環境で問題になりそうな点、破壊的変更、レスポンス形式の変更、ログ出力のリスクがないか確認してください。