Implement professional API error handling infrastructure
API Infrastructure Improvements: - Created standardized API helper utilities for consistent error handling - Added comprehensive request validation with detailed error messages - Implemented proper HTTP status codes for different error types - Added request body parsing with validation helpers Enhanced Teams API: - Added input validation with length and format checks - Implemented duplicate team name prevention - Added proper error categorization (validation, database, server errors) - Consistent response format with timestamps - Improved error messages for better debugging Professional Standards: - Standardized error response structure across all APIs - Added development vs production error detail filtering - Implemented async error boundary wrapper for unhandled exceptions - Enhanced logging with structured error information - Added proper sorting for team list responses Foundation for Future APIs: - Reusable error handling patterns for all API routes - Validation helpers that can be extended to other endpoints - Consistent response structure for frontend integration - Better debugging capabilities for development 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
This commit is contained in:
parent
fcfd6e9838
commit
f4df3df698
2 changed files with 240 additions and 28 deletions
|
@ -2,36 +2,95 @@ import { NextResponse } from 'next/server';
|
|||
import { getDatabase } from '../../../lib/database';
|
||||
import { Team } from '@/types';
|
||||
import { TABLE_NAMES } from '@/lib/constants';
|
||||
import {
|
||||
withErrorHandling,
|
||||
createSuccessResponse,
|
||||
createValidationError,
|
||||
createDatabaseError,
|
||||
parseRequestBody
|
||||
} from '@/lib/apiHelpers';
|
||||
|
||||
export async function GET() {
|
||||
const db = await getDatabase();
|
||||
const teams: Team[] = await db.all(`SELECT * FROM ${TABLE_NAMES.TEAMS}`);
|
||||
return NextResponse.json(teams);
|
||||
// Validation for team creation
|
||||
function validateTeamInput(data: unknown): {
|
||||
valid: boolean;
|
||||
data?: { team_name: string };
|
||||
errors?: Record<string, string>
|
||||
} {
|
||||
const errors: Record<string, string> = {};
|
||||
|
||||
if (!data || typeof data !== 'object') {
|
||||
errors.general = 'Request body must be an object';
|
||||
return { valid: false, errors };
|
||||
}
|
||||
|
||||
const { team_name } = data as { team_name?: unknown };
|
||||
|
||||
if (!team_name || typeof team_name !== 'string') {
|
||||
errors.team_name = 'Team name is required and must be a string';
|
||||
} else if (team_name.trim().length < 2) {
|
||||
errors.team_name = 'Team name must be at least 2 characters long';
|
||||
} else if (team_name.trim().length > 50) {
|
||||
errors.team_name = 'Team name must be less than 50 characters long';
|
||||
}
|
||||
|
||||
if (Object.keys(errors).length > 0) {
|
||||
return { valid: false, errors };
|
||||
}
|
||||
|
||||
return {
|
||||
valid: true,
|
||||
data: { team_name: team_name.trim() }
|
||||
};
|
||||
}
|
||||
|
||||
export async function POST(request: Request) {
|
||||
try {
|
||||
const { team_name } = await request.json();
|
||||
|
||||
if (!team_name) {
|
||||
return NextResponse.json({ error: 'Team name is required' }, { status: 400 });
|
||||
}
|
||||
export const GET = withErrorHandling(async () => {
|
||||
try {
|
||||
const db = await getDatabase();
|
||||
const teams: Team[] = await db.all(`SELECT * FROM ${TABLE_NAMES.TEAMS} ORDER BY team_name ASC`);
|
||||
|
||||
return createSuccessResponse(teams);
|
||||
} catch (error) {
|
||||
return createDatabaseError('fetch teams', error);
|
||||
}
|
||||
});
|
||||
|
||||
const db = await getDatabase();
|
||||
|
||||
const result = await db.run(
|
||||
`INSERT INTO ${TABLE_NAMES.TEAMS} (team_name) VALUES (?)`,
|
||||
[team_name]
|
||||
);
|
||||
|
||||
const newTeam: Team = {
|
||||
team_id: result.lastID!,
|
||||
team_name: team_name
|
||||
};
|
||||
|
||||
return NextResponse.json(newTeam, { status: 201 });
|
||||
} catch (error) {
|
||||
console.error('Error creating team:', error);
|
||||
return NextResponse.json({ error: 'Failed to create team' }, { status: 500 });
|
||||
export const POST = withErrorHandling(async (request: Request) => {
|
||||
const bodyResult = await parseRequestBody(request, validateTeamInput);
|
||||
|
||||
if (!bodyResult.success) {
|
||||
return bodyResult.response;
|
||||
}
|
||||
|
||||
const { team_name } = bodyResult.data;
|
||||
|
||||
try {
|
||||
const db = await getDatabase();
|
||||
|
||||
// Check if team name already exists
|
||||
const existingTeam = await db.get(
|
||||
`SELECT team_id FROM ${TABLE_NAMES.TEAMS} WHERE LOWER(team_name) = LOWER(?)`,
|
||||
[team_name]
|
||||
);
|
||||
|
||||
if (existingTeam) {
|
||||
return createValidationError(
|
||||
'Team name already exists',
|
||||
{ team_name: 'A team with this name already exists' }
|
||||
);
|
||||
}
|
||||
}
|
||||
|
||||
const result = await db.run(
|
||||
`INSERT INTO ${TABLE_NAMES.TEAMS} (team_name) VALUES (?)`,
|
||||
[team_name]
|
||||
);
|
||||
|
||||
const newTeam: Team = {
|
||||
team_id: result.lastID!,
|
||||
team_name: team_name
|
||||
};
|
||||
|
||||
return createSuccessResponse(newTeam, 201);
|
||||
} catch (error) {
|
||||
return createDatabaseError('create team', error);
|
||||
}
|
||||
});
|
Loading…
Add table
Add a link
Reference in a new issue