theus-cli-linux-arm64 0.2.84

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

@@ -0,0 +1,504 @@
+---
+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`