deepseek-code-quality

DeepSeek Code Quality Skill — Universal

Si deepseek-reasoning te enseña cómo pensar, esta skill te enseña cómo escribir. Los principios son universales. Los ejemplos usan TypeScript pero aplican a cualquier lenguaje tipado.

---

1. Tipos — Escribir código que se defiende solo

1.1 Nunca evadir el sistema de tipos

any, casts forzados, y type assertions son una rendición. Siempre existe un tipo correcto.

// ❌ WRONG — evadiendo el sistema de tipos
const result = (api.user as any).listItems?.();

// ✅ CORRECT — usando el tipo real
const result = api.user.listItems();

Si el compilador se queja, el problema es el tipo — no el cast. Leé el error, encontrá el tipo correcto.

Alternativas a any:

• unknown — cuando el tipo es genuinamente desconocido; manejalo con type guards
• Generics — cuando el tipo varía pero es predecible
• Tipos de unión explícitos — cuando puede ser una de varias cosas conocidas

1.2 Tipos de retorno explícitos en funciones públicas

Las funciones que forman parte de la API de un módulo deben declarar su tipo de retorno:

// ❌ WRONG — el lector no sabe qué esperar
async function getUserById(id: string) {
  // ...
}

// ✅ CORRECT — contrato claro
async function getUserById(id: string): Promise<User | null> {
  // ...
}

1.3 Guards para valores potencialmente nulos

Antes de acceder a una propiedad, verificar que el objeto existe:

// ❌ WRONG — crash si rows[0] no existe
const user = rows[0];
return user.name;

// ✅ CORRECT
const user = rows[0];
if (!user) return null;
return user.name;

// ✅ CORRECT — alternativa concisa
return rows[0]?.name ?? null;

1.4 Estados explícitos en lugar de booleans ambiguos

// ❌ WRONG — ¿isActive: false significa suspendido? ¿eliminado? ¿bloqueado?
interface Resource {
  isActive: boolean;
}

// ✅ CORRECT — el estado es inequívoco
interface Resource {
  status: "active" | "suspended" | "deleted" | "pending";
}