通用审查员

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      # 为什么 API 用 .tsx？
  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')

集中配置，类型安全访问