@rubix0270/arboris 1.0.1 → 1.0.3

package/prisma/skills/rules-distill/scripts/scan-rules.sh

@@ -1,58 +0,0 @@
-#!/usr/bin/env bash
-# scan-rules.sh — enumerate rule files and extract H2 heading index
-# Usage: scan-rules.sh [RULES_DIR]
-# Output: JSON to stdout
-#
-# Environment:
-#   RULES_DISTILL_DIR  Override ~/.claude/rules (for testing only)
-
-set -euo pipefail
-
-RULES_DIR="${RULES_DISTILL_DIR:-${1:-$HOME/.claude/rules}}"
-
-if [[ ! -d "$RULES_DIR" ]]; then
-  jq -n --arg path "$RULES_DIR" '{"error":"rules directory not found","path":$path}' >&2
-  exit 1
-fi
-
-# Collect all .md files (excluding _archived/)
-files=()
-while IFS= read -r f; do
-  files+=("$f")
-done < <(find "$RULES_DIR" -name '*.md' -not -path '*/_archived/*' -print | sort)
-
-total=${#files[@]}
-
-tmpdir=$(mktemp -d)
-_rules_cleanup() { rm -rf "$tmpdir"; }
-trap _rules_cleanup EXIT
-
-for i in "${!files[@]}"; do
-  file="${files[$i]}"
-  rel_path="${file#"$HOME"/}"
-  rel_path="~/$rel_path"
-
-  # Extract H2 headings (## Title) into a JSON array via jq
-  headings_json=$({ grep -E '^## ' "$file" 2>/dev/null || true; } | sed 's/^## //' | jq -R . | jq -s '.')
-
-  # Get line count
-  line_count=$(wc -l < "$file" | tr -d ' ')
-
-  jq -n \
-    --arg path "$rel_path" \
-    --arg file "$(basename "$file")" \
-    --argjson lines "$line_count" \
-    --argjson headings "$headings_json" \
-    '{path:$path,file:$file,lines:$lines,headings:$headings}' \
-    > "$tmpdir/$i.json"
-done
-
-if [[ ${#files[@]} -eq 0 ]]; then
-  jq -n --arg dir "$RULES_DIR" '{rules_dir:$dir,total:0,rules:[]}'
-else
-  jq -n \
-    --arg dir "$RULES_DIR" \
-    --argjson total "$total" \
-    --argjson rules "$(jq -s '.' "$tmpdir"/*.json)" \
-    '{rules_dir:$dir,total:$total,rules:$rules}'
-fi

package/prisma/skills/rules-distill/scripts/scan-skills.sh

@@ -1,129 +0,0 @@
-#!/usr/bin/env bash
-# scan-skills.sh — enumerate skill files, extract frontmatter and UTC mtime
-# Usage: scan-skills.sh [CWD_SKILLS_DIR]
-# Output: JSON to stdout
-#
-# When CWD_SKILLS_DIR is omitted, defaults to $PWD/.claude/skills so the
-# script always picks up project-level skills without relying on the caller.
-#
-# Environment:
-#   RULES_DISTILL_GLOBAL_DIR   Override ~/.claude/skills (for testing only;
-#                              do not set in production — intended for bats tests)
-#   RULES_DISTILL_PROJECT_DIR  Override project dir detection (for testing only)
-
-set -euo pipefail
-
-GLOBAL_DIR="${RULES_DISTILL_GLOBAL_DIR:-$HOME/.claude/skills}"
-CWD_SKILLS_DIR="${RULES_DISTILL_PROJECT_DIR:-${1:-$PWD/.claude/skills}}"
-# Validate CWD_SKILLS_DIR looks like a .claude/skills path (defense-in-depth).
-# Only warn when the path exists — a nonexistent path poses no traversal risk.
-if [[ -n "$CWD_SKILLS_DIR" && -d "$CWD_SKILLS_DIR" && "$CWD_SKILLS_DIR" != */.claude/skills* ]]; then
-  echo "Warning: CWD_SKILLS_DIR does not look like a .claude/skills path: $CWD_SKILLS_DIR" >&2
-fi
-
-# Extract a frontmatter field (handles both quoted and unquoted single-line values).
-# Does NOT support multi-line YAML blocks (| or >) or nested YAML keys.
-extract_field() {
-  local file="$1" field="$2"
-  awk -v f="$field" '
-    BEGIN { fm=0 }
-    /^---$/ { fm++; next }
-    fm==1 {
-      n = length(f) + 2
-      if (substr($0, 1, n) == f ": ") {
-        val = substr($0, n+1)
-        gsub(/^"/, "", val)
-        gsub(/"$/, "", val)
-        print val
-        exit
-      }
-    }
-    fm>=2 { exit }
-  ' "$file"
-}
-
-# Get file mtime in UTC ISO8601 (portable: GNU and BSD)
-get_mtime() {
-  local file="$1"
-  local secs
-  secs=$(stat -c %Y "$file" 2>/dev/null || stat -f %m "$file" 2>/dev/null) || return 1
-  date -u -d "@$secs" +%Y-%m-%dT%H:%M:%SZ 2>/dev/null ||
-  date -u -r "$secs" +%Y-%m-%dT%H:%M:%SZ
-}
-
-# Scan a directory and produce a JSON array of skill objects
-scan_dir_to_json() {
-  local dir="$1"
-
-  local tmpdir
-  tmpdir=$(mktemp -d)
-  local _scan_tmpdir="$tmpdir"
-  _scan_cleanup() { rm -rf "$_scan_tmpdir"; }
-  trap _scan_cleanup RETURN
-
-  local i=0
-  while IFS= read -r file; do
-    local name desc mtime dp
-    name=$(extract_field "$file" "name")
-    desc=$(extract_field "$file" "description")
-    mtime=$(get_mtime "$file")
-    dp="${file/#$HOME/~}"
-
-    jq -n \
-      --arg path "$dp" \
-      --arg name "$name" \
-      --arg description "$desc" \
-      --arg mtime "$mtime" \
-      '{path:$path,name:$name,description:$description,mtime:$mtime}' \
-      > "$tmpdir/$i.json"
-    i=$((i+1))
-  done < <(find "$dir" -name "SKILL.md" -type f 2>/dev/null | sort)
-
-  if [[ $i -eq 0 ]]; then
-    echo "[]"
-  else
-    jq -s '.' "$tmpdir"/*.json
-  fi
-}
-
-# --- Main ---
-
-global_found="false"
-global_count=0
-global_skills="[]"
-
-if [[ -d "$GLOBAL_DIR" ]]; then
-  global_found="true"
-  global_skills=$(scan_dir_to_json "$GLOBAL_DIR")
-  global_count=$(echo "$global_skills" | jq 'length')
-fi
-
-project_found="false"
-project_path=""
-project_count=0
-project_skills="[]"
-
-if [[ -n "$CWD_SKILLS_DIR" && -d "$CWD_SKILLS_DIR" ]]; then
-  project_found="true"
-  project_path="$CWD_SKILLS_DIR"
-  project_skills=$(scan_dir_to_json "$CWD_SKILLS_DIR")
-  project_count=$(echo "$project_skills" | jq 'length')
-fi
-
-# Merge global + project skills into one array
-all_skills=$(jq -s 'add' <(echo "$global_skills") <(echo "$project_skills"))
-
-jq -n \
-  --arg global_found "$global_found" \
-  --argjson global_count "$global_count" \
-  --arg project_found "$project_found" \
-  --arg project_path "$project_path" \
-  --argjson project_count "$project_count" \
-  --argjson skills "$all_skills" \
-  '{
-    scan_summary: {
-      global: { found: ($global_found == "true"), count: $global_count },
-      project: { found: ($project_found == "true"), path: $project_path, count: $project_count }
-    },
-    skills: $skills
-  }'

package/prisma/skills/rust-patterns/SKILL.md

@@ -1,500 +0,0 @@
----
-name: rust-patterns
-description: Idiomatic Rust patterns, ownership, error handling, traits, concurrency, and best practices for building safe, performant applications.
-metadata:
-  origin: ECC
----
-
-# Rust Development Patterns
-
-Idiomatic Rust patterns and best practices for building safe, performant, and maintainable applications.
-
-## When to Use
-
-- Writing new Rust code
-- Reviewing Rust code
-- Refactoring existing Rust code
-- Designing crate structure and module layout
-
-## How It Works
-
-This skill enforces idiomatic Rust conventions across six key areas: ownership and borrowing to prevent data races at compile time, `Result`/`?` error propagation with `thiserror` for libraries and `anyhow` for applications, enums and exhaustive pattern matching to make illegal states unrepresentable, traits and generics for zero-cost abstraction, safe concurrency via `Arc<Mutex<T>>`, channels, and async/await, and minimal `pub` surfaces organized by domain.
-
-## Core Principles
-
-### 1. Ownership and Borrowing
-
-Rust's ownership system prevents data races and memory bugs at compile time.
-
-```rust
-// Good: Pass references when you don't need ownership
-fn process(data: &[u8]) -> usize {
-    data.len()
-}
-
-// Good: Take ownership only when you need to store or consume
-fn store(data: Vec<u8>) -> Record {
-    Record { payload: data }
-}
-
-// Bad: Cloning unnecessarily to avoid borrow checker
-fn process_bad(data: &Vec<u8>) -> usize {
-    let cloned = data.clone(); // Wasteful — just borrow
-    cloned.len()
-}
-```
-
-### Use `Cow` for Flexible Ownership
-
-```rust
-use std::borrow::Cow;
-
-fn normalize(input: &str) -> Cow<'_, str> {
-    if input.contains(' ') {
-        Cow::Owned(input.replace(' ', "_"))
-    } else {
-        Cow::Borrowed(input) // Zero-cost when no mutation needed
-    }
-}
-```
-
-## Error Handling
-
-### Use `Result` and `?` — Never `unwrap()` in Production
-
-```rust
-// Good: Propagate errors with context
-use anyhow::{Context, Result};
-
-fn load_config(path: &str) -> Result<Config> {
-    let content = std::fs::read_to_string(path)
-        .with_context(|| format!("failed to read config from {path}"))?;
-    let config: Config = toml::from_str(&content)
-        .with_context(|| format!("failed to parse config from {path}"))?;
-    Ok(config)
-}
-
-// Bad: Panics on error
-fn load_config_bad(path: &str) -> Config {
-    let content = std::fs::read_to_string(path).unwrap(); // Panics!
-    toml::from_str(&content).unwrap()
-}
-```
-
-### Library Errors with `thiserror`, Application Errors with `anyhow`
-
-```rust
-// Library code: structured, typed errors
-use thiserror::Error;
-
-#[derive(Debug, Error)]
-pub enum StorageError {
-    #[error("record not found: {id}")]
-    NotFound { id: String },
-    #[error("connection failed")]
-    Connection(#[from] std::io::Error),
-    #[error("invalid data: {0}")]
-    InvalidData(String),
-}
-
-// Application code: flexible error handling
-use anyhow::{bail, Result};
-
-fn run() -> Result<()> {
-    let config = load_config("app.toml")?;
-    if config.workers == 0 {
-        bail!("worker count must be > 0");
-    }
-    Ok(())
-}
-```
-
-### `Option` Combinators Over Nested Matching
-
-```rust
-// Good: Combinator chain
-fn find_user_email(users: &[User], id: u64) -> Option<String> {
-    users.iter()
-        .find(|u| u.id == id)
-        .map(|u| u.email.clone())
-}
-
-// Bad: Deeply nested matching
-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 and Pattern Matching
-
-### Model States as Enums
-
-```rust
-// Good: Impossible states are unrepresentable
-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),
-    }
-}
-```
-
-### Exhaustive Matching — No Catch-All for Business Logic
-
-```rust
-// Good: Handle every variant explicitly
-match command {
-    Command::Start => start_service(),
-    Command::Stop => stop_service(),
-    Command::Restart => restart_service(),
-    // Adding a new variant forces handling here
-}
-
-// Bad: Wildcard hides new variants
-match command {
-    Command::Start => start_service(),
-    _ => {} // Silently ignores Stop, Restart, and future variants
-}
-```
-
-## Traits and Generics
-
-### Accept Generics, Return Concrete Types
-
-```rust
-// Good: Generic input, concrete output
-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)
-}
-
-// Good: Trait bounds for multiple constraints
-fn process<T: Display + Send + 'static>(item: T) -> String {
-    format!("processed: {item}")
-}
-```
-
-### Trait Objects for Dynamic Dispatch
-
-```rust
-// Use when you need heterogeneous collections or plugin systems
-trait Handler: Send + Sync {
-    fn handle(&self, request: &Request) -> Response;
-}
-
-struct Router {
-    handlers: Vec<Box<dyn Handler>>,
-}
-
-// Use generics when you need performance (monomorphization)
-fn fast_process<H: Handler>(handler: &H, request: &Request) -> Response {
-    handler.handle(request)
-}
-```
-
-### Newtype Pattern for Type Safety
-
-```rust
-// Good: Distinct types prevent mixing up arguments
-struct UserId(u64);
-struct OrderId(u64);
-
-fn get_order(user: UserId, order: OrderId) -> Result<Order> {
-    // Can't accidentally swap user and order IDs
-    todo!()
-}
-
-// Bad: Easy to swap arguments
-fn get_order_bad(user_id: u64, order_id: u64) -> Result<Order> {
-    todo!()
-}
-```
-
-## Structs and Data Modeling
-
-### Builder Pattern for Complex Construction
-
-```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 }
-    }
-}
-
-// Usage: ServerConfig::builder("localhost", 8080).max_connections(200).build()
-```
-
-## Iterators and Closures
-
-### Prefer Iterator Chains Over Manual Loops
-
-```rust
-// Good: Declarative, lazy, composable
-let active_emails: Vec<String> = users.iter()
-    .filter(|u| u.is_active)
-    .map(|u| u.email.clone())
-    .collect();
-
-// Bad: Imperative accumulation
-let mut active_emails = Vec::new();
-for user in &users {
-    if user.is_active {
-        active_emails.push(user.email.clone());
-    }
-}
-```
-
-### Use `collect()` with Type Annotation
-
-```rust
-// Collect into different types
-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();
-
-// Collect Results — short-circuits on first error
-let parsed: Result<Vec<i32>, _> = strings.iter().map(|s| s.parse()).collect();
-```
-
-## Concurrency
-
-### `Arc<Mutex<T>>` for Shared Mutable State
-
-```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 poisoned");
-        *num += 1;
-    })
-}).collect();
-
-for handle in handles {
-    handle.join().expect("worker thread panicked");
-}
-```
-
-### Channels for Message Passing
-
-```rust
-use std::sync::mpsc;
-
-let (tx, rx) = mpsc::sync_channel(16); // Bounded channel with backpressure
-
-for i in 0..5 {
-    let tx = tx.clone();
-    std::thread::spawn(move || {
-        tx.send(format!("message {i}")).expect("receiver disconnected");
-    });
-}
-drop(tx); // Close sender so rx iterator terminates
-
-for msg in rx {
-    println!("{msg}");
-}
-```
-
-### Async with 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("request timed out")?
-    .context("request failed")?;
-
-    response.text().await.context("failed to read body")
-}
-
-// Spawn concurrent tasks
-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!("spawned task panicked: {e}")));
-    }
-    results
-}
-```
-
-## Unsafe Code
-
-### When Unsafe Is Acceptable
-
-```rust
-// Acceptable: FFI boundary with documented invariants (Rust 2024+)
-/// # Safety
-/// `ptr` must be a valid, aligned pointer to an initialized `Widget`.
-unsafe fn widget_from_raw<'a>(ptr: *const Widget) -> &'a Widget {
-    // SAFETY: caller guarantees ptr is valid and aligned
-    unsafe { &*ptr }
-}
-
-// Acceptable: Performance-critical path with proof of correctness
-// SAFETY: index is always < len due to the loop bound
-unsafe { slice.get_unchecked(index) }
-```
-
-### When Unsafe Is NOT Acceptable
-
-```rust
-// Bad: Using unsafe to bypass borrow checker
-// Bad: Using unsafe for convenience
-// Bad: Using unsafe without a Safety comment
-// Bad: Transmuting between unrelated types
-```
-
-## Module System and Crate Structure
-
-### Organize by Domain, Not by Type
-
-```text
-my_app/
-├── src/
-│   ├── main.rs
-│   ├── lib.rs
-│   ├── auth/          # Domain module
-│   │   ├── mod.rs
-│   │   ├── token.rs
-│   │   └── middleware.rs
-│   ├── orders/        # Domain module
-│   │   ├── mod.rs
-│   │   ├── model.rs
-│   │   └── service.rs
-│   └── db/            # Infrastructure
-│       ├── mod.rs
-│       └── pool.rs
-├── tests/             # Integration tests
-├── benches/           # Benchmarks
-└── Cargo.toml
-```
-
-### Visibility — Expose Minimally
-
-```rust
-// Good: pub(crate) for internal sharing
-pub(crate) fn validate_input(input: &str) -> bool {
-    !input.is_empty()
-}
-
-// Good: Re-export public API from lib.rs
-pub mod auth;
-pub use auth::AuthMiddleware;
-
-// Bad: Making everything pub
-pub fn internal_helper() {} // Should be pub(crate) or private
-```
-
-## Tooling Integration
-
-### Essential Commands
-
-```bash
-# Build and check
-cargo build
-cargo check              # Fast type checking without codegen
-cargo clippy             # Lints and suggestions
-cargo fmt                # Format code
-
-# Testing
-cargo test
-cargo test -- --nocapture    # Show println output
-cargo test --lib             # Unit tests only
-cargo test --test integration # Integration tests only
-
-# Dependencies
-cargo audit              # Security audit
-cargo tree               # Dependency tree
-cargo update             # Update dependencies
-
-# Performance
-cargo bench              # Run benchmarks
-```
-
-## Quick Reference: Rust Idioms
-
-| Idiom | Description |
-|-------|-------------|
-| Borrow, don't clone | Pass `&T` instead of cloning unless ownership is needed |
-| Make illegal states unrepresentable | Use enums to model valid states only |
-| `?` over `unwrap()` | Propagate errors, never panic in library/production code |
-| Parse, don't validate | Convert unstructured data to typed structs at the boundary |
-| Newtype for type safety | Wrap primitives in newtypes to prevent argument swaps |
-| Prefer iterators over loops | Declarative chains are clearer and often faster |
-| `#[must_use]` on Results | Ensure callers handle return values |
-| `Cow` for flexible ownership | Avoid allocations when borrowing suffices |
-| Exhaustive matching | No wildcard `_` for business-critical enums |
-| Minimal `pub` surface | Use `pub(crate)` for internal APIs |
-
-## Anti-Patterns to Avoid
-
-```rust
-// Bad: .unwrap() in production code
-let value = map.get("key").unwrap();
-
-// Bad: .clone() to satisfy borrow checker without understanding why
-let data = expensive_data.clone();
-process(&original, &data);
-
-// Bad: Using String when &str suffices
-fn greet(name: String) { /* should be &str */ }
-
-// Bad: Box<dyn Error> in libraries (use thiserror instead)
-fn parse(input: &str) -> Result<Data, Box<dyn std::error::Error>> { todo!() }
-
-// Bad: Ignoring must_use warnings
-let _ = validate(input); // Silently discarding a Result
-
-// Bad: Blocking in async context
-async fn bad_async() {
-    std::thread::sleep(Duration::from_secs(1)); // Blocks the executor!
-    // Use: tokio::time::sleep(Duration::from_secs(1)).await;
-}
-```
-
-**Remember**: If it compiles, it's probably correct — but only if you avoid `unwrap()`, minimize `unsafe`, and let the type system work for you.