Réviseur Général

git commit -m "fix"
git commit -m "updates"
git commit -m "WIP"
git commit -m "asdfasdf"

Des messages vagues rendent l'historique inutile

git commit -m "fix(auth): handle expired token refresh"
git commit -m "feat(api): add pagination to users endpoint"
git commit -m "docs: update README with new env vars"
git commit -m "refactor(db): extract query builders"

Suivre les conventional commits pour un historique clair

src/
  UserComponent.tsx
  userUtils.ts
  USER_CONSTANTS.ts
  user-types.ts
  userApi.tsx      # Pourquoi .tsx pour une API?
  UserHelpers.ts

Nommage incohérent et structure plate

src/features/users/
  components/
    UserCard.tsx
    UserList.tsx
  hooks/
    useUser.ts
  api/
    users.api.ts
  types/
    user.types.ts
  constants/
    user.constants.ts

Suivre une organisation par fonctionnalité avec un nommage cohérent

// Dispersé dans tout le codebase
const API_URL = process.env.API_URL || 'http://localhost:3000'

// Dans un autre fichier
const apiBase = process.env.NEXT_PUBLIC_API_URL

// Dans encore un autre fichier
if (process.env.NODE_ENV === 'production') {
  baseUrl = 'https://api.example.com'
}

Variables d'environnement dispersées, gestion incohérente

// src/config/env.ts
export const config = {
  api: {
    baseUrl: requiredEnv('API_URL'),
    timeout: optionalEnv('API_TIMEOUT', 5000),
  },
  features: {
    analytics: boolEnv('ENABLE_ANALYTICS', false),
  },
} as const

// Utilisation partout
import { config } from '@/config/env'
fetch(config.api.baseUrl + '/users')

Configuration centralisée avec accès type-safe