CodeCritic AI logo

CodeCritic AI

Premium AI code review workspace

AC
Back to Dashboard
Review Details

TypeScript / Full Review

TypeScript
Full Review
Apr 6, 2026, 7:12 PM

Code Quality Score

75/100

Summary

The code works but lacks error handling and type safety, impacting maintainability and robustness.

Issues Found

Detailed findings

3 issues

Lack of Error Handling

high

Line 2-3

The code does not handle potential errors from the fetch call or the JSON parsing, which could lead to unhandled promise rejections.

Suggested fix: Wrap the fetch call and JSON parsing in a try-catch block to gracefully handle errors.

Use of 'any' Type

medium

Line 6

The 'any' type in the map function removes type safety, making the code more prone to errors and harder to maintain.

Suggested fix: Define a proper interface for 'project' and use it instead of 'any'.

String Concatenation

medium

Line 2

String concatenation for the fetch URL can lead to mistakes and is harder to read.

Suggested fix: Use template literals for constructing the fetch URL.

Improved Code

Suggested rewrite

export async function getUserProfile(userId: string) {
  try {
    const response = await fetch(`/api/users/${userId}`);
    const user = await response.json();

    if (user.isAdmin) {
      console.log("admin user");
    }

    return {
      name: user.name,
      projects: user.projects.map((project: Project) => project.title),
    };
  } catch (error) {
    console.error('Failed to fetch user profile:', error);
    throw error;
  }
}

interface Project {
  title: string;
}