Coding Standards

// PASS: BIEN: Nombres descriptivosconst marketSearchQuery = 'election'const isUserAuthenticated = trueconst totalRevenue = 1000
// FAIL: MAL: Nombres poco clarosconst q = 'election'const flag = trueconst x = 1000

// PASS: BIEN: Patrón verbo-sustantivoasync function fetchMarketData(marketId: string) { }function calculateSimilarity(a: number[], b: number[]) { }function isValidEmail(email: string): boolean { }
// FAIL: MAL: Poco claro o solo sustantivoasync function market(id: string) { }function similarity(a, b) { }function email(e) { }

// PASS: SIEMPRE usar el operador spreadconst updatedUser = {  ...user,  name: 'New Name'}
const updatedArray = [...items, newItem]
// FAIL: NUNCA mutar directamenteuser.name = 'New Name'  // MALitems.push(newItem)     // MAL

// PASS: BIEN: Manejo de errores comprensivoasync function fetchData(url: string) {  try {    const response = await fetch(url)
    if (!response.ok) {      throw new Error(`HTTP ${response.status}: ${response.statusText}`)    }
    return await response.json()  } catch (error) {    console.error('Fetch failed:', error)    throw new Error('Failed to fetch data')  }}
// FAIL: MAL: Sin manejo de erroresasync function fetchData(url) {  const response = await fetch(url)  return response.json()}

// PASS: BIEN: Ejecución paralela cuando sea posibleconst [users, markets, stats] = await Promise.all([  fetchUsers(),  fetchMarkets(),  fetchStats()])
// FAIL: MAL: Secuencial cuando no es necesarioconst users = await fetchUsers()const markets = await fetchMarkets()const stats = await fetchStats()

// PASS: BIEN: Tipos apropiadosinterface Market {  id: string  name: string  status: 'active' | 'resolved' | 'closed'  created_at: Date}
function getMarket(id: string): Promise<Market> {  // Implementación}
// FAIL: MAL: Usar 'any'function getMarket(id: any): Promise<any> {  // Implementación}

// PASS: BIEN: Componente funcional con tiposinterface ButtonProps {  children: React.ReactNode  onClick: () => void  disabled?: boolean  variant?: 'primary' | 'secondary'}
export function Button({  children,  onClick,  disabled = false,  variant = 'primary'}: ButtonProps) {  return (    <button      onClick={onClick}      disabled={disabled}      className={`btn btn-${variant}`}    >      {children}    </button>  )}
// FAIL: MAL: Sin tipos, estructura poco claraexport function Button(props) {  return <button onClick={props.onClick}>{props.children}</button>}

// PASS: BIEN: Custom hook reutilizableexport function useDebounce<T>(value: T, delay: number): T {  const [debouncedValue, setDebouncedValue] = useState<T>(value)
  useEffect(() => {    const handler = setTimeout(() => {      setDebouncedValue(value)    }, delay)
    return () => clearTimeout(handler)  }, [value, delay])
  return debouncedValue}
// Usoconst debouncedQuery = useDebounce(searchQuery, 500)

// PASS: BIEN: Actualizaciones de estado correctasconst [count, setCount] = useState(0)
// Actualización funcional para estado basado en el estado previosetCount(prev => prev + 1)
// FAIL: MAL: Referencia de estado directasetCount(count + 1)  // Puede estar obsoleta en escenarios async

// PASS: BIEN: Renderizado condicional claro{isLoading && <Spinner />}{error && <ErrorMessage error={error} />}{data && <DataDisplay data={data} />}
// FAIL: MAL: Infierno de ternarios{isLoading ? <Spinner /> : error ? <ErrorMessage error={error} /> : data ? <DataDisplay data={data} /> : null}

// PASS: BIEN: Estructura de respuesta consistenteinterface ApiResponse<T> {  success: boolean  data?: T  error?: string  meta?: {    total: number    page: number    limit: number  }}
// Respuesta exitosareturn NextResponse.json({  success: true,  data: markets,  meta: { total: 100, page: 1, limit: 10 }})
// Respuesta de errorreturn NextResponse.json({  success: false,  error: 'Invalid request'}, { status: 400 })

import { z } from 'zod'
// PASS: BIEN: Validación con esquemaconst CreateMarketSchema = z.object({  name: z.string().min(1).max(200),  description: z.string().min(1).max(2000),  endDate: z.string().datetime(),  categories: z.array(z.string()).min(1)})
export async function POST(request: Request) {  const body = await request.json()
  try {    const validated = CreateMarketSchema.parse(body)    // Proceder con datos validados  } catch (error) {    if (error instanceof z.ZodError) {      return NextResponse.json({        success: false,        error: 'Validation failed',        details: error.errors      }, { status: 400 })    }  }}

// PASS: BIEN: Explicar el POR QUÉ, no el QUÉ// Usar backoff exponencial para evitar sobrecargar la API durante interrupcionesconst delay = Math.min(1000 * Math.pow(2, retryCount), 30000)
// Usando mutación deliberadamente aquí por rendimiento con arrays grandesitems.push(newItem)
// FAIL: MAL: Declarar lo obvio// Incrementar contador en 1count++
// Establecer nombre al nombre del usuarioname = user.name

/** * Busca markets usando similitud semántica. * * @param query - Consulta de búsqueda en lenguaje natural * @param limit - Número máximo de resultados (por defecto: 10) * @returns Array de markets ordenados por puntuación de similitud * @throws {Error} Si la API de OpenAI falla o Redis no está disponible * * @example * ```typescript * const results = await searchMarkets('election', 5) * console.log(results[0].name) // "Trump vs Biden" * ``` */export async function searchMarkets(  query: string,  limit: number = 10): Promise<Market[]> {  // Implementación}

import { useMemo, useCallback } from 'react'
// PASS: BIEN: Memoizar cómputos costososconst sortedMarkets = useMemo(() => {  return markets.sort((a, b) => b.volume - a.volume)}, [markets])
// PASS: BIEN: Memoizar callbacksconst handleSearch = useCallback((query: string) => {  setSearchQuery(query)}, [])

import { lazy, Suspense } from 'react'
// PASS: BIEN: Cargar componentes pesados de forma diferidaconst HeavyChart = lazy(() => import('./HeavyChart'))
export function Dashboard() {  return (    <Suspense fallback={<Spinner />}>      <HeavyChart />    </Suspense>  )}

// PASS: BIEN: Seleccionar solo las columnas necesariasconst { data } = await supabase  .from('markets')  .select('id, name, status')  .limit(10)
// FAIL: MAL: Seleccionar todoconst { data } = await supabase  .from('markets')  .select('*')

test('calculates similarity correctly', () => {  // Arrange (Preparar)  const vector1 = [1, 0, 0]  const vector2 = [0, 1, 0]
  // Act (Actuar)  const similarity = calculateCosineSimilarity(vector1, vector2)
  // Assert (Verificar)  expect(similarity).toBe(0)})

// PASS: BIEN: Nombres de prueba descriptivostest('returns empty array when no markets match query', () => { })test('throws error when OpenAI API key is missing', () => { })test('falls back to substring search when Redis unavailable', () => { })
// FAIL: MAL: Nombres de prueba vagostest('works', () => { })test('test search', () => { })

// FAIL: MAL: Función > 50 líneasfunction processMarketData() {  // 100 líneas de código}
// PASS: BIEN: Dividir en funciones más pequeñasfunction processMarketData() {  const validated = validateData()  const transformed = transformData(validated)  return saveData(transformed)}

// FAIL: MAL: 5+ niveles de anidamientoif (user) {  if (user.isAdmin) {    if (market) {      if (market.isActive) {        if (hasPermission) {          // Hacer algo        }      }    }  }}
// PASS: BIEN: Retornos tempranosif (!user) returnif (!user.isAdmin) returnif (!market) returnif (!market.isActive) returnif (!hasPermission) return
// Hacer algo

// FAIL: MAL: Números sin explicaciónif (retryCount > 3) { }setTimeout(callback, 500)
// PASS: BIEN: Constantes con nombreconst MAX_RETRIES = 3const DEBOUNCE_DELAY_MS = 500
if (retryCount > MAX_RETRIES) { }setTimeout(callback, DEBOUNCE_DELAY_MS)