Mejora la Calidad del Código en .NET con GitHub Copilot y Prompts Personalizados

{
  "chat.promptFiles": true,
  "chat.promptFilesLocations": {
    ".github/prompts": true,
    ".vscode/prompts": true
  },
  "github.copilot.chat.codeGeneration.instructions": [
    { "file": ".vscode/rules/csharp/coding-guidelines.md" },
    { "file": ".vscode/rules/csharp/coding-style.md" }
  ],
  "github.copilot.chat.reviewSelection.enabled": true,
  "github.copilot.chat.reviewSelection.instructions": [
    { "file": ".vscode/rules/csharp/coding-guidelines.md" },
    { "file": ".vscode/rules/csharp/coding-style.md" }
  ],
  "github.copilot.chat.testGeneration.instructions": [
    { "file": ".vscode/rules/csharp/testing-xunit.md" }
  ],
  "github.copilot.chat.commitMessageGeneration.instructions": [
    { "file": ".vscode/rules/git-message.md" }
  ]
}

---
description: 🧼 Guía para escribir código en C# que sea limpio, mantenible y alineado con patrones de diseño reconocidos. Este documento proporciona principios claros y recomendaciones prácticas para desarrollar software de alta calidad y fácil evolución.
---

# 👥 Roles involucrados

- 🏗️ **Arquitecto de Software**  
  Diseña estructuras escalables y garantiza la alineación con principios de arquitectura.

- 💡 **Experto en C#**  
  Aplica buenas prácticas específicas del lenguaje y se mantiene actualizado con sus características más modernas.

- 🔍 **Especialista en Calidad de Código**  
  Promueve estándares de codificación, revisa la calidad y asegura legibilidad y sostenibilidad.

- 🌱 **Promotor de Software Sostenible**  
  Se enfoca en prácticas que reduzcan la deuda técnica, fomenten la evolución del sistema a largo plazo y consideren la eficiencia en el uso de recursos computacionales.

---

# 🧭 Principios generales

## 👁️ Legibilidad y claridad

- ✅ Priorizar código autoexplicativo mediante **nombres descriptivos** para clases, métodos y variables.
- 🧠 Reducir la necesidad de comentarios gracias a una estructura clara y lógica.

## ⚙️ Uso de características modernas de C#

- ⬆️ Utilizar la **versión más actual** del lenguaje disponible en el proyecto.
- 🧵 Incorporar características como `async/await`, **pattern matching** (switch avanzado) y **colecciones inmutables**, siempre que sean adecuadas.

## 🔗 Gestión de dependencias

- 🚫 Evitar dependencias innecesarias que dificulten el mantenimiento o aumenten la complejidad.
- 🔄 Aplicar principios como **Inversión de Dependencias** para desacoplar módulos y facilitar pruebas.

## 🧊 Inmutabilidad y simplicidad

- 🔒 Favorecer el uso de **objetos inmutables** para evitar errores por efectos secundarios.
- ✂️ Diseñar métodos y clases con **baja complejidad ciclomática** para facilitar su lectura y pruebas.

## 🧱 Patrones de diseño

- 🧩 Aplicar **patrones reconocidos** (Repositorio, Fábrica, Estrategia...) cuando sean relevantes.
- ⚖️ Evitar la sobreingeniería: los patrones deben ser **herramientas útiles, no reglas absolutas**.

---

✨ **Esta guía busca ayudarte a escribir código que no solo funcione, sino que sea un activo sostenible y comprensible para cualquier miembro del equipo.**

# 🎨 Guía de Estilo de Código C#

## 📦 Organización del código

- 📄 Una clase por archivo (salvo excepciones justificadas).
- 🧭 Orden recomendado de elementos:
  1. Campos estáticos
  2. Campos de instancia
  3. Constructores
  4. Propiedades
  5. Métodos públicos
  6. Métodos privados

## 🧹 Formato y espaciado

- 🔲 Indentación de 4 espacios.
- 📏 Líneas de máximo 120 caracteres.
- ➖ Línea en blanco entre métodos.
- 📐 Llaves en nueva línea:

```csharp
  if (condition)
  {
      DoSomething();
  }
```

## 🐪 Convenciones de nombres

- 🧱 Clases, interfaces, métodos públicos → PascalCase

- 🔧 Campos privados → _camelCase

- 💬 Parámetros y variables locales → camelCase

- 📐 Interfaces → prefijo I (e.g., IUserService)

## ⏱️ Métodos asíncronos

- ➕ Terminar con Async:

```csharp
public async Task<User> GetUserAsync(int id)
```
  
## 🧠 Uso de var
 - 👍 Solo cuando el tipo es evidente:

```csharp
  var order = new Order(); // Bien
  string name = "Carlos";  // Mejor explícito
```
##🔍 Usings
  
- 🧽 Eliminar usings no utilizados.
- 🧾 Agrupar por bloques:

```csharp
using System;
using System.Collections.Generic;

using Microsoft.Extensions.DependencyInjection;

using MyApp.Core.Repositories;
```

---
description: Checklist de calidad previa al push
---

# Definición:

Antes de hacer *push* o abrir un Pull Request, realiza una revisión automática de la rama actual siguiendo esta checklist. El objetivo es asegurar calidad, coherencia con las guías del equipo, y evitar errores frecuentes en revisiones de código.

## Reglas y Validaciones

✅ **Estructura del código**
- El código sigue las guías de estilo definidas en `.vscode/rules/coding-style.md`.
- No hay bloques de código comentado innecesario ni `Console.WriteLine` o `Debug.WriteLine` olvidados.
- No hay `TODO`, `FIXME` o anotaciones temporales sin resolver.

✅ **Nombrado y legibilidad**
- Los nombres de clases, métodos y variables son claros y expresivos.
- No hay abreviaciones crípticas ni nombres genéricos como `data`, `temp`, `obj`.

✅ **Pruebas**
- Se han actualizado o agregado pruebas unitarias si el cambio lo requiere.
- Todas las pruebas pasan localmente (xUnit/NUnit/MSTest según corresponda).

✅ **Mensajes de commit**
- Los mensajes siguen la convención definida en `.vscode/rules/git-message.md`.
- Cada commit tiene un propósito claro y es autocontenible.

✅ **Impacto**
- Se ha evaluado el impacto del cambio en otras capas del sistema (servicios, controladores, persistencia).
- Se ha actualizado la documentación si el cambio lo amerita (README, comentarios, Wiki).

## Estructura esperada del análisis

- **Resumen general del estado del checklist**
- **Items cumplidos**
- **Items no cumplidos o con observaciones**
- **Recomendaciones para corrección**
- **Conclusión general (¿Es seguro hacer push?)**

✋ Si alguno de los puntos no se cumple, sugiere una acción concreta para corregirlo.