theus-cli 0.2.83 → 0.2.84

package/vendor/theus-arsenal/skills/patrones-rust/SKILL.md

@@ -1,504 +0,0 @@
----
-name: patrones-rust
-description: Patrones idiomáticos de Rust — ownership, manejo de errores, traits, concurrencia y buenas
-  prácticas para construir aplicaciones seguras y de alto rendimiento. Úsala al escribir, revisar o
-  refactorizar código Rust, o al diseñar la estructura de un crate.
----
-
-# Patrones de desarrollo en Rust
-
-Patrones idiomáticos de Rust y buenas prácticas para construir aplicaciones seguras, performantes y mantenibles.
-
-## Cuándo usarla
-
-- Al escribir código Rust nuevo.
-- Al revisar código Rust.
-- Al refactorizar código Rust existente.
-- Al diseñar la estructura de crates y la organización de módulos.
-
-## Cómo funciona
-
-Esta skill impone convenciones idiomáticas de Rust en seis áreas clave: ownership y borrowing para prevenir data races en tiempo de compilación, propagación de errores con `Result`/`?` usando `thiserror` en bibliotecas y `anyhow` en aplicaciones, enums con matching exhaustivo para hacer irrepresentables los estados ilegales, traits y genéricos como abstracción de coste cero, concurrencia segura vía `Arc<Mutex<T>>`, canales y async/await, y superficies `pub` mínimas organizadas por dominio.
-
-## Principios fundamentales
-
-### 1. Ownership y borrowing
-
-El sistema de ownership de Rust previene data races y errores de memoria en tiempo de compilación.
-
-```rust
-// Bien: pasa referencias cuando no necesitas ownership
-fn process(data: &[u8]) -> usize {
-    data.len()
-}
-
-// Bien: toma ownership solo cuando necesitas almacenar o consumir
-fn store(data: Vec<u8>) -> Record {
-    Record { payload: data }
-}
-
-// Mal: clonar innecesariamente para esquivar el borrow checker
-fn process_bad(data: &Vec<u8>) -> usize {
-    let cloned = data.clone(); // Desperdicio — basta con prestar
-    cloned.len()
-}
-```
-
-### Usa `Cow` para ownership flexible
-
-```rust
-use std::borrow::Cow;
-
-fn normalize(input: &str) -> Cow<'_, str> {
-    if input.contains(' ') {
-        Cow::Owned(input.replace(' ', "_"))
-    } else {
-        Cow::Borrowed(input) // Coste cero cuando no hace falta mutar
-    }
-}
-```
-
-## Manejo de errores
-
-### Usa `Result` y `?` — nunca `unwrap()` en producción
-
-```rust
-// Bien: propaga errores con contexto
-use anyhow::{Context, Result};
-
-fn load_config(path: &str) -> Result<Config> {
-    let content = std::fs::read_to_string(path)
-        .with_context(|| format!("no se pudo leer la configuración de {path}"))?;
-    let config: Config = toml::from_str(&content)
-        .with_context(|| format!("no se pudo parsear la configuración de {path}"))?;
-    Ok(config)
-}
-
-// Mal: entra en pánico ante un error
-fn load_config_bad(path: &str) -> Config {
-    let content = std::fs::read_to_string(path).unwrap(); // ¡Panic!
-    toml::from_str(&content).unwrap()
-}
-```
-
-### Errores de biblioteca con `thiserror`, errores de aplicación con `anyhow`
-
-```rust
-// Código de biblioteca: errores estructurados y tipados
-use thiserror::Error;
-
-#[derive(Debug, Error)]
-pub enum StorageError {
-    #[error("registro no encontrado: {id}")]
-    NotFound { id: String },
-    #[error("fallo de conexión")]
-    Connection(#[from] std::io::Error),
-    #[error("datos inválidos: {0}")]
-    InvalidData(String),
-}
-
-// Código de aplicación: manejo de errores flexible
-use anyhow::{bail, Result};
-
-fn run() -> Result<()> {
-    let config = load_config("app.toml")?;
-    if config.workers == 0 {
-        bail!("el número de workers debe ser > 0");
-    }
-    Ok(())
-}
-```
-
-### Combinadores de `Option` en lugar de matching anidado
-
-```rust
-// Bien: cadena de combinadores
-fn find_user_email(users: &[User], id: u64) -> Option<String> {
-    users.iter()
-        .find(|u| u.id == id)
-        .map(|u| u.email.clone())
-}
-
-// Mal: matching profundamente anidado
-fn find_user_email_bad(users: &[User], id: u64) -> Option<String> {
-    match users.iter().find(|u| u.id == id) {
-        Some(user) => match &user.email {
-            email => Some(email.clone()),
-        },
-        None => None,
-    }
-}
-```
-
-## Enums y pattern matching
-
-### Modela los estados como enums
-
-```rust
-// Bien: los estados imposibles son irrepresentables
-enum ConnectionState {
-    Disconnected,
-    Connecting { attempt: u32 },
-    Connected { session_id: String },
-    Failed { reason: String, retries: u32 },
-}
-
-fn handle(state: &ConnectionState) {
-    match state {
-        ConnectionState::Disconnected => connect(),
-        ConnectionState::Connecting { attempt } if *attempt > 3 => abort(),
-        ConnectionState::Connecting { .. } => wait(),
-        ConnectionState::Connected { session_id } => use_session(session_id),
-        ConnectionState::Failed { retries, .. } if *retries < 5 => retry(),
-        ConnectionState::Failed { reason, .. } => log_failure(reason),
-    }
-}
-```
-
-### Matching exhaustivo — sin comodín en lógica de negocio
-
-```rust
-// Bien: maneja cada variante explícitamente
-match command {
-    Command::Start => start_service(),
-    Command::Stop => stop_service(),
-    Command::Restart => restart_service(),
-    // Añadir una variante nueva obliga a manejarla aquí
-}
-
-// Mal: el comodín oculta variantes nuevas
-match command {
-    Command::Start => start_service(),
-    _ => {} // Ignora en silencio Stop, Restart y variantes futuras
-}
-```
-
-## Traits y genéricos
-
-### Acepta genéricos, devuelve tipos concretos
-
-```rust
-// Bien: entrada genérica, salida concreta
-fn read_all(reader: &mut impl Read) -> std::io::Result<Vec<u8>> {
-    let mut buf = Vec::new();
-    reader.read_to_end(&mut buf)?;
-    Ok(buf)
-}
-
-// Bien: trait bounds para múltiples restricciones
-fn process<T: Display + Send + 'static>(item: T) -> String {
-    format!("procesado: {item}")
-}
-```
-
-### Trait objects para dispatch dinámico
-
-```rust
-// Úsalos con colecciones heterogéneas o sistemas de plugins
-trait Handler: Send + Sync {
-    fn handle(&self, request: &Request) -> Response;
-}
-
-struct Router {
-    handlers: Vec<Box<dyn Handler>>,
-}
-
-// Usa genéricos cuando necesites rendimiento (monomorfización)
-fn fast_process<H: Handler>(handler: &H, request: &Request) -> Response {
-    handler.handle(request)
-}
-```
-
-### Patrón newtype para seguridad de tipos
-
-```rust
-// Bien: tipos distintos evitan confundir argumentos
-struct UserId(u64);
-struct OrderId(u64);
-
-fn get_order(user: UserId, order: OrderId) -> Result<Order> {
-    // Imposible intercambiar por accidente los IDs de usuario y pedido
-    todo!()
-}
-
-// Mal: fácil intercambiar los argumentos
-fn get_order_bad(user_id: u64, order_id: u64) -> Result<Order> {
-    todo!()
-}
-```
-
-## Structs y modelado de datos
-
-### Patrón builder para construcciones complejas
-
-```rust
-struct ServerConfig {
-    host: String,
-    port: u16,
-    max_connections: usize,
-}
-
-impl ServerConfig {
-    fn builder(host: impl Into<String>, port: u16) -> ServerConfigBuilder {
-        ServerConfigBuilder { host: host.into(), port, max_connections: 100 }
-    }
-}
-
-struct ServerConfigBuilder { host: String, port: u16, max_connections: usize }
-
-impl ServerConfigBuilder {
-    fn max_connections(mut self, n: usize) -> Self { self.max_connections = n; self }
-    fn build(self) -> ServerConfig {
-        ServerConfig { host: self.host, port: self.port, max_connections: self.max_connections }
-    }
-}
-
-// Uso: ServerConfig::builder("localhost", 8080).max_connections(200).build()
-```
-
-## Iteradores y closures
-
-### Prefiere cadenas de iteradores a bucles manuales
-
-```rust
-// Bien: declarativo, perezoso, componible
-let active_emails: Vec<String> = users.iter()
-    .filter(|u| u.is_active)
-    .map(|u| u.email.clone())
-    .collect();
-
-// Mal: acumulación imperativa
-let mut active_emails = Vec::new();
-for user in &users {
-    if user.is_active {
-        active_emails.push(user.email.clone());
-    }
-}
-```
-
-### Usa `collect()` con anotación de tipo
-
-```rust
-// Colecta hacia distintos tipos
-let names: Vec<_> = items.iter().map(|i| &i.name).collect();
-let lookup: HashMap<_, _> = items.iter().map(|i| (i.id, i)).collect();
-let combined: String = parts.iter().copied().collect();
-
-// Colectar Results — cortocircuita en el primer error
-let parsed: Result<Vec<i32>, _> = strings.iter().map(|s| s.parse()).collect();
-```
-
-## Concurrencia
-
-### `Arc<Mutex<T>>` para estado mutable compartido
-
-```rust
-use std::sync::{Arc, Mutex};
-
-let counter = Arc::new(Mutex::new(0));
-let handles: Vec<_> = (0..10).map(|_| {
-    let counter = Arc::clone(&counter);
-    std::thread::spawn(move || {
-        let mut num = counter.lock().expect("mutex envenenado");
-        *num += 1;
-    })
-}).collect();
-
-for handle in handles {
-    handle.join().expect("un hilo worker entró en pánico");
-}
-```
-
-### Canales para paso de mensajes
-
-```rust
-use std::sync::mpsc;
-
-let (tx, rx) = mpsc::sync_channel(16); // Canal acotado con backpressure
-
-for i in 0..5 {
-    let tx = tx.clone();
-    std::thread::spawn(move || {
-        tx.send(format!("mensaje {i}")).expect("receptor desconectado");
-    });
-}
-drop(tx); // Cierra el emisor para que el iterador de rx termine
-
-for msg in rx {
-    println!("{msg}");
-}
-```
-
-### Async con Tokio
-
-```rust
-use tokio::time::Duration;
-
-async fn fetch_with_timeout(url: &str) -> Result<String> {
-    let response = tokio::time::timeout(
-        Duration::from_secs(5),
-        reqwest::get(url),
-    )
-    .await
-    .context("la petición expiró")?
-    .context("la petición falló")?;
-
-    response.text().await.context("no se pudo leer el cuerpo")
-}
-
-// Lanza tareas concurrentes
-async fn fetch_all(urls: Vec<String>) -> Vec<Result<String>> {
-    let handles: Vec<_> = urls.into_iter()
-        .map(|url| tokio::spawn(async move {
-            fetch_with_timeout(&url).await
-        }))
-        .collect();
-
-    let mut results = Vec::with_capacity(handles.len());
-    for handle in handles {
-        results.push(handle.await.unwrap_or_else(|e| panic!("la tarea lanzada entró en pánico: {e}")));
-    }
-    results
-}
-```
-
-## Código unsafe
-
-### Cuándo es aceptable
-
-```rust
-// Aceptable: frontera FFI con invariantes documentadas (Rust 2024+)
-/// # Safety
-/// `ptr` debe ser un puntero válido y alineado a un `Widget` inicializado.
-unsafe fn widget_from_raw<'a>(ptr: *const Widget) -> &'a Widget {
-    // SAFETY: quien llama garantiza que ptr es válido y está alineado
-    unsafe { &*ptr }
-}
-
-// Aceptable: ruta crítica de rendimiento con prueba de corrección
-// SAFETY: index siempre es < len por la cota del bucle
-unsafe { slice.get_unchecked(index) }
-```
-
-### Cuándo NO es aceptable
-
-```rust
-// Mal: usar unsafe para esquivar el borrow checker
-// Mal: usar unsafe por comodidad
-// Mal: usar unsafe sin comentario de Safety
-// Mal: transmutar entre tipos no relacionados
-```
-
-## Sistema de módulos y estructura del crate
-
-### Organiza por dominio, no por tipo
-
-```text
-my_app/
-├── src/
-│   ├── main.rs
-│   ├── lib.rs
-│   ├── auth/          # Módulo de dominio
-│   │   ├── mod.rs
-│   │   ├── token.rs
-│   │   └── middleware.rs
-│   ├── orders/        # Módulo de dominio
-│   │   ├── mod.rs
-│   │   ├── model.rs
-│   │   └── service.rs
-│   └── db/            # Infraestructura
-│       ├── mod.rs
-│       └── pool.rs
-├── tests/             # Tests de integración
-├── benches/           # Benchmarks
-└── Cargo.toml
-```
-
-### Visibilidad — expón lo mínimo
-
-```rust
-// Bien: pub(crate) para compartir internamente
-pub(crate) fn validate_input(input: &str) -> bool {
-    !input.is_empty()
-}
-
-// Bien: re-exporta la API pública desde lib.rs
-pub mod auth;
-pub use auth::AuthMiddleware;
-
-// Mal: hacer todo pub
-pub fn internal_helper() {} // Debería ser pub(crate) o privado
-```
-
-## Herramientas
-
-### Comandos esenciales
-
-```bash
-# Compilar y verificar
-cargo build
-cargo check              # Verificación de tipos rápida sin codegen
-cargo clippy             # Lints y sugerencias
-cargo fmt                # Formatear el código
-
-# Testing
-cargo test
-cargo test -- --nocapture    # Muestra la salida de println
-cargo test --lib             # Solo tests unitarios
-cargo test --test integration # Solo tests de integración
-
-# Dependencias
-cargo audit              # Auditoría de seguridad
-cargo tree               # Árbol de dependencias
-cargo update             # Actualizar dependencias
-
-# Rendimiento
-cargo bench              # Ejecutar benchmarks
-```
-
-## Referencia rápida: idiomas de Rust
-
-| Idioma | Descripción |
-|-------|-------------|
-| Presta, no clones | Pasa `&T` en lugar de clonar salvo que necesites ownership |
-| Estados ilegales irrepresentables | Usa enums que solo modelen estados válidos |
-| `?` sobre `unwrap()` | Propaga errores; nunca entres en pánico en código de biblioteca/producción |
-| Parsea, no valides | Convierte datos sin estructura en structs tipados en la frontera |
-| Newtype para seguridad de tipos | Envuelve primitivos en newtypes para evitar intercambios de argumentos |
-| Iteradores sobre bucles | Las cadenas declarativas son más claras y a menudo más rápidas |
-| `#[must_use]` en Results | Obliga a los llamantes a manejar los valores de retorno |
-| `Cow` para ownership flexible | Evita asignaciones cuando basta con prestar |
-| Matching exhaustivo | Sin comodín `_` en enums críticos de negocio |
-| Superficie `pub` mínima | Usa `pub(crate)` para APIs internas |
-
-## Antipatrones a evitar
-
-```rust
-// Mal: .unwrap() en código de producción
-let value = map.get("key").unwrap();
-
-// Mal: .clone() para satisfacer al borrow checker sin entender por qué
-let data = expensive_data.clone();
-process(&original, &data);
-
-// Mal: usar String cuando basta &str
-fn greet(name: String) { /* debería ser &str */ }
-
-// Mal: Box<dyn Error> en bibliotecas (usa thiserror en su lugar)
-fn parse(input: &str) -> Result<Data, Box<dyn std::error::Error>> { todo!() }
-
-// Mal: ignorar advertencias must_use
-let _ = validate(input); // Descarta un Result en silencio
-
-// Mal: bloquear en contexto async
-async fn bad_async() {
-    std::thread::sleep(Duration::from_secs(1)); // ¡Bloquea el executor!
-    // Usa: tokio::time::sleep(Duration::from_secs(1)).await;
-}
-```
-
-**Recuerda**: si compila, probablemente es correcto — pero solo si evitas `unwrap()`, minimizas `unsafe` y dejas que el sistema de tipos trabaje por ti.
-
-## Skills relacionadas
-
-- `testing-rust`