Общий Ревьюер

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

Расплывчатые сообщения делают историю бесполезной

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"

Следуйте conventional commits для четкой истории

src/
  UserComponent.tsx
  userUtils.ts
  USER_CONSTANTS.ts
  user-types.ts
  userApi.tsx      # Почему .tsx для API?
  UserHelpers.ts

Непоследовательное именование и плоская структура

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

Следуйте организации по функциям с согласованным именованием

// Разбросано по всей кодовой базе
const API_URL = process.env.API_URL || 'http://localhost:3000'

// В другом файле
const apiBase = process.env.NEXT_PUBLIC_API_URL

// Еще в другом файле
if (process.env.NODE_ENV === 'production') {
  baseUrl = 'https://api.example.com'
}

Переменные окружения разбросаны, управление непоследовательно

// 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

// Использование везде
import { config } from '@/config/env'
fetch(config.api.baseUrl + '/users')

Централизованная конфигурация с type-safe доступом