@diottodev/runic 1.0.0 → 1.1.0

package/engineering/skills/agent-workflow-designer/SKILL.md

@@ -0,0 +1,83 @@
+---
+name: "agent-workflow-designer"
+description: "Design production-grade multi-agent workflows with clear pattern choice (sequential, parallel, hierarchical), handoff contracts, failure handling, and cost/context controls. Use when architecting a multi-step agent pipeline, choosing between single-agent vs multi-agent approaches, or refactoring an LLM workflow that suffers from context bloat or unreliable handoffs."
+---
+
+# Agent Workflow Designer
+
+**Tier:** POWERFUL  
+**Category:** Engineering  
+**Domain:** Multi-Agent Systems / AI Orchestration
+
+---
+
+## Overview
+
+Design production-grade multi-agent workflows with clear pattern choice, handoff contracts, failure handling, and cost/context controls.
+
+## Core Capabilities
+
+- Workflow pattern selection for multi-step agent systems
+- Skeleton config generation for fast workflow bootstrapping
+- Context and cost discipline across long-running flows
+- Error recovery and retry strategy scaffolding
+- Documentation pointers for operational pattern tradeoffs
+
+---
+
+## When to Use
+
+- A single prompt is insufficient for task complexity
+- You need specialist agents with explicit boundaries
+- You want deterministic workflow structure before implementation
+- You need validation loops for quality or safety gates
+
+---
+
+## Quick Start
+
+```bash
+# Generate a sequential workflow skeleton
+python3 scripts/workflow_scaffolder.py sequential --name content-pipeline
+
+# Generate an orchestrator workflow and save it
+python3 scripts/workflow_scaffolder.py orchestrator --name incident-triage --output workflows/incident-triage.json
+```
+
+---
+
+## Pattern Map
+
+- `sequential`: strict step-by-step dependency chain
+- `parallel`: fan-out/fan-in for independent subtasks
+- `router`: dispatch by intent/type with fallback
+- `orchestrator`: planner coordinates specialists with dependencies
+- `evaluator`: generator + quality gate loop
+
+Detailed templates: `references/workflow-patterns.md`
+
+---
+
+## Recommended Workflow
+
+1. Select pattern based on dependency shape and risk profile.
+2. Scaffold config via `scripts/workflow_scaffolder.py`.
+3. Define handoff contract fields for every edge.
+4. Add retry/timeouts and output validation gates.
+5. Dry-run with small context budgets before scaling.
+
+---
+
+## Common Pitfalls
+
+- Over-orchestrating tasks solvable by one well-structured prompt
+- Missing timeout/retry policies for external-model calls
+- Passing full upstream context instead of targeted artifacts
+- Ignoring per-step cost accumulation
+
+## Best Practices
+
+1. Start with the smallest pattern that can satisfy requirements.
+2. Keep handoff payloads explicit and bounded.
+3. Validate intermediate outputs before fan-in synthesis.
+4. Enforce budget and timeout limits in every step.

package/engineering/skills/agent-workflow-designer/references/workflow-patterns.md

@@ -0,0 +1,82 @@
+# Workflow Pattern Templates
+
+## Sequential
+
+Use when each step depends on prior output.
+
+```json
+{
+  "pattern": "sequential",
+  "steps": ["research", "draft", "review"]
+}
+```
+
+## Parallel
+
+Use when independent tasks can fan out and then fan in.
+
+```json
+{
+  "pattern": "parallel",
+  "fan_out": ["task_a", "task_b", "task_c"],
+  "fan_in": "synthesizer"
+}
+```
+
+## Router
+
+Use when tasks must be routed to specialized handlers by intent.
+
+```json
+{
+  "pattern": "router",
+  "router": "intent_router",
+  "routes": ["sales", "support", "engineering"],
+  "fallback": "generalist"
+}
+```
+
+## Orchestrator
+
+Use when dynamic planning and dependency management are required.
+
+```json
+{
+  "pattern": "orchestrator",
+  "orchestrator": "planner",
+  "specialists": ["researcher", "analyst", "coder"],
+  "dependency_mode": "dag"
+}
+```
+
+## Evaluator
+
+Use when output quality gates are mandatory before finalization.
+
+```json
+{
+  "pattern": "evaluator",
+  "generator": "content_agent",
+  "evaluator": "quality_agent",
+  "max_iterations": 3,
+  "pass_threshold": 0.8
+}
+```
+
+## Pattern Selection Heuristics
+
+- Choose `sequential` for strict linear workflows.
+- Choose `parallel` for throughput and latency reduction.
+- Choose `router` for intent- or type-based branching.
+- Choose `orchestrator` for complex adaptive workflows.
+- Choose `evaluator` when correctness/quality loops are required.
+
+## Handoff Minimum Contract
+
+- `workflow_id`
+- `step_id`
+- `task`
+- `constraints`
+- `upstream_artifacts`
+- `budget_tokens`
+- `timeout_seconds`

package/engineering/skills/agent-workflow-designer/scripts/workflow_scaffolder.py

@@ -0,0 +1,113 @@
+#!/usr/bin/env python3
+"""Generate workflow skeleton configs from common multi-agent patterns."""
+
+from __future__ import annotations
+
+import argparse
+import json
+from pathlib import Path
+from typing import Dict, List
+
+
+def sequential_template(name: str) -> Dict:
+    return {
+        "name": name,
+        "pattern": "sequential",
+        "steps": [
+            {"id": "research", "agent": "researcher", "next": "draft"},
+            {"id": "draft", "agent": "writer", "next": "review"},
+            {"id": "review", "agent": "reviewer", "next": None},
+        ],
+        "retry": {"max_attempts": 2, "backoff_seconds": 2},
+    }
+
+
+def parallel_template(name: str) -> Dict:
+    return {
+        "name": name,
+        "pattern": "parallel",
+        "fan_out": {
+            "tasks": ["research_a", "research_b", "research_c"],
+            "agent": "analyst",
+        },
+        "fan_in": {"agent": "synthesizer", "output": "combined_report"},
+        "timeouts": {"per_task_seconds": 180, "fan_in_seconds": 120},
+    }
+
+
+def router_template(name: str) -> Dict:
+    return {
+        "name": name,
+        "pattern": "router",
+        "router": {"agent": "router", "routes": ["sales", "support", "engineering"]},
+        "handlers": {
+            "sales": {"agent": "sales_specialist"},
+            "support": {"agent": "support_specialist"},
+            "engineering": {"agent": "engineering_specialist"},
+        },
+        "fallback": {"agent": "generalist"},
+    }
+
+
+def orchestrator_template(name: str) -> Dict:
+    return {
+        "name": name,
+        "pattern": "orchestrator",
+        "orchestrator": {"agent": "orchestrator", "planning": "dynamic"},
+        "specialists": ["researcher", "coder", "analyst", "writer"],
+        "execution": {
+            "dependency_mode": "dag",
+            "max_parallel": 3,
+            "completion_policy": "all_required",
+        },
+    }
+
+
+def evaluator_template(name: str) -> Dict:
+    return {
+        "name": name,
+        "pattern": "evaluator",
+        "generator": {"agent": "generator"},
+        "evaluator": {"agent": "evaluator", "criteria": ["accuracy", "format", "safety"]},
+        "loop": {
+            "max_iterations": 3,
+            "pass_threshold": 0.8,
+            "on_fail": "revise_and_retry",
+        },
+    }
+
+
+PATTERNS = {
+    "sequential": sequential_template,
+    "parallel": parallel_template,
+    "router": router_template,
+    "orchestrator": orchestrator_template,
+    "evaluator": evaluator_template,
+}
+
+
+def parse_args() -> argparse.Namespace:
+    parser = argparse.ArgumentParser(description="Generate a workflow skeleton config from a pattern.")
+    parser.add_argument("pattern", choices=sorted(PATTERNS.keys()), help="Workflow pattern")
+    parser.add_argument("--name", default="new-workflow", help="Workflow name")
+    parser.add_argument("--output", help="Optional output path for JSON config")
+    return parser.parse_args()
+
+
+def main() -> int:
+    args = parse_args()
+    config = PATTERNS[args.pattern](args.name)
+    payload = json.dumps(config, indent=2)
+
+    if args.output:
+        out = Path(args.output)
+        out.parent.mkdir(parents=True, exist_ok=True)
+        out.write_text(payload + "\n", encoding="utf-8")
+        print(f"Wrote workflow config to {out}")
+    else:
+        print(payload)
+    return 0
+
+
+if __name__ == "__main__":
+    raise SystemExit(main())

package/engineering/skills/api-design-reviewer/SKILL.md

@@ -0,0 +1,421 @@
+---
+name: "api-design-reviewer"
+description: "Comprehensive REST API design review with automated linting, breaking-change detection, and design scorecards. Catches inconsistent conventions, missing versioning, and design smells before APIs ship. Use when reviewing a PR that adds or changes API endpoints, auditing an existing API for v2 migration, or establishing API standards for a team."
+---
+
+# API Design Reviewer
+
+**Tier:** POWERFUL  
+**Category:** Engineering / Architecture  
+**Maintainer:** Claude Skills Team  
+
+## Overview
+
+The API Design Reviewer skill provides comprehensive analysis and review of API designs, focusing on REST conventions, best practices, and industry standards. This skill helps engineering teams build consistent, maintainable, and well-designed APIs through automated linting, breaking change detection, and design scorecards.
+
+## Core Capabilities
+
+### 1. API Linting and Convention Analysis
+- **Resource Naming Conventions**: Enforces kebab-case for resources, camelCase for fields
+- **HTTP Method Usage**: Validates proper use of GET, POST, PUT, PATCH, DELETE
+- **URL Structure**: Analyzes endpoint patterns for consistency and RESTful design
+- **Status Code Compliance**: Ensures appropriate HTTP status codes are used
+- **Error Response Formats**: Validates consistent error response structures
+- **Documentation Coverage**: Checks for missing descriptions and documentation gaps
+
+### 2. Breaking Change Detection
+- **Endpoint Removal**: Detects removed or deprecated endpoints
+- **Response Shape Changes**: Identifies modifications to response structures
+- **Field Removal**: Tracks removed or renamed fields in API responses
+- **Type Changes**: Catches field type modifications that could break clients
+- **Required Field Additions**: Flags new required fields that could break existing integrations
+- **Status Code Changes**: Detects changes to expected status codes
+
+### 3. API Design Scoring and Assessment
+- **Consistency Analysis** (30%): Evaluates naming conventions, response patterns, and structural consistency
+- **Documentation Quality** (20%): Assesses completeness and clarity of API documentation
+- **Security Implementation** (20%): Reviews authentication, authorization, and security headers
+- **Usability Design** (15%): Analyzes ease of use, discoverability, and developer experience
+- **Performance Patterns** (15%): Evaluates caching, pagination, and efficiency patterns
+
+## REST Design Principles
+
+### Resource Naming Conventions
+```
+✅ Good Examples:
+- /api/v1/users
+- /api/v1/user-profiles
+- /api/v1/orders/123/line-items
+
+❌ Bad Examples:
+- /api/v1/getUsers
+- /api/v1/user_profiles
+- /api/v1/orders/123/lineItems
+```
+
+### HTTP Method Usage
+- **GET**: Retrieve resources (safe, idempotent)
+- **POST**: Create new resources (not idempotent)
+- **PUT**: Replace entire resources (idempotent)
+- **PATCH**: Partial resource updates (not necessarily idempotent)
+- **DELETE**: Remove resources (idempotent)
+
+### URL Structure Best Practices
+```
+Collection Resources: /api/v1/users
+Individual Resources: /api/v1/users/123
+Nested Resources: /api/v1/users/123/orders
+Actions: /api/v1/users/123/activate (POST)
+Filtering: /api/v1/users?status=active&role=admin
+```
+
+## Versioning Strategies
+
+### 1. URL Versioning (Recommended)
+```
+/api/v1/users
+/api/v2/users
+```
+**Pros**: Clear, explicit, easy to route  
+**Cons**: URL proliferation, caching complexity
+
+### 2. Header Versioning
+```
+GET /api/users
+Accept: application/vnd.api+json;version=1
+```
+**Pros**: Clean URLs, content negotiation  
+**Cons**: Less visible, harder to test manually
+
+### 3. Media Type Versioning
+```
+GET /api/users
+Accept: application/vnd.myapi.v1+json
+```
+**Pros**: RESTful, supports multiple representations  
+**Cons**: Complex, harder to implement
+
+### 4. Query Parameter Versioning
+```
+/api/users?version=1
+```
+**Pros**: Simple to implement  
+**Cons**: Not RESTful, can be ignored
+
+## Pagination Patterns
+
+### Offset-Based Pagination
+```json
+{
+  "data": [...],
+  "pagination": {
+    "offset": 20,
+    "limit": 10,
+    "total": 150,
+    "hasMore": true
+  }
+}
+```
+
+### Cursor-Based Pagination
+```json
+{
+  "data": [...],
+  "pagination": {
+    "nextCursor": "eyJpZCI6MTIzfQ==",
+    "hasMore": true
+  }
+}
+```
+
+### Page-Based Pagination
+```json
+{
+  "data": [...],
+  "pagination": {
+    "page": 3,
+    "pageSize": 10,
+    "totalPages": 15,
+    "totalItems": 150
+  }
+}
+```
+
+## Error Response Formats
+
+### Standard Error Structure
+```json
+{
+  "error": {
+    "code": "VALIDATION_ERROR",
+    "message": "The request contains invalid parameters",
+    "details": [
+      {
+        "field": "email",
+        "code": "INVALID_FORMAT",
+        "message": "Email address is not valid"
+      }
+    ],
+    "requestId": "req-123456",
+    "timestamp": "2024-02-16T13:00:00Z"
+  }
+}
+```
+
+### HTTP Status Code Usage
+- **400 Bad Request**: Invalid request syntax or parameters
+- **401 Unauthorized**: Authentication required
+- **403 Forbidden**: Access denied (authenticated but not authorized)
+- **404 Not Found**: Resource not found
+- **409 Conflict**: Resource conflict (duplicate, version mismatch)
+- **422 Unprocessable Entity**: Valid syntax but semantic errors
+- **429 Too Many Requests**: Rate limit exceeded
+- **500 Internal Server Error**: Unexpected server error
+
+## Authentication and Authorization Patterns
+
+### Bearer Token Authentication
+```
+Authorization: Bearer <token>
+```
+
+### API Key Authentication
+```
+X-API-Key: <api-key>
+Authorization: Api-Key <api-key>
+```
+
+### OAuth 2.0 Flow
+```
+Authorization: Bearer <oauth-access-token>
+```
+
+### Role-Based Access Control (RBAC)
+```json
+{
+  "user": {
+    "id": "123",
+    "roles": ["admin", "editor"],
+    "permissions": ["read:users", "write:orders"]
+  }
+}
+```
+
+## Rate Limiting Implementation
+
+### Headers
+```
+X-RateLimit-Limit: 1000
+X-RateLimit-Remaining: 999
+X-RateLimit-Reset: 1640995200
+```
+
+### Response on Limit Exceeded
+```json
+{
+  "error": {
+    "code": "RATE_LIMIT_EXCEEDED",
+    "message": "Too many requests",
+    "retryAfter": 3600
+  }
+}
+```
+
+## HATEOAS (Hypermedia as the Engine of Application State)
+
+### Example Implementation
+```json
+{
+  "id": "123",
+  "name": "John Doe",
+  "email": "john@example.com",
+  "_links": {
+    "self": { "href": "/api/v1/users/123" },
+    "orders": { "href": "/api/v1/users/123/orders" },
+    "profile": { "href": "/api/v1/users/123/profile" },
+    "deactivate": { 
+      "href": "/api/v1/users/123/deactivate",
+      "method": "POST"
+    }
+  }
+}
+```
+
+## Idempotency
+
+### Idempotent Methods
+- **GET**: Always safe and idempotent
+- **PUT**: Should be idempotent (replace entire resource)
+- **DELETE**: Should be idempotent (same result)
+- **PATCH**: May or may not be idempotent
+
+### Idempotency Keys
+```
+POST /api/v1/payments
+Idempotency-Key: 123e4567-e89b-12d3-a456-426614174000
+```
+
+## Backward Compatibility Guidelines
+
+### Safe Changes (Non-Breaking)
+- Adding optional fields to requests
+- Adding fields to responses
+- Adding new endpoints
+- Making required fields optional
+- Adding new enum values (with graceful handling)
+
+### Breaking Changes (Require Version Bump)
+- Removing fields from responses
+- Making optional fields required
+- Changing field types
+- Removing endpoints
+- Changing URL structures
+- Modifying error response formats
+
+## OpenAPI/Swagger Validation
+
+### Required Components
+- **API Information**: Title, description, version
+- **Server Information**: Base URLs and descriptions
+- **Path Definitions**: All endpoints with methods
+- **Parameter Definitions**: Query, path, header parameters
+- **Request/Response Schemas**: Complete data models
+- **Security Definitions**: Authentication schemes
+- **Error Responses**: Standard error formats
+
+### Best Practices
+- Use consistent naming conventions
+- Provide detailed descriptions for all components
+- Include examples for complex objects
+- Define reusable components and schemas
+- Validate against OpenAPI specification
+
+## Performance Considerations
+
+### Caching Strategies
+```
+Cache-Control: public, max-age=3600
+ETag: "123456789"
+Last-Modified: Wed, 21 Oct 2015 07:28:00 GMT
+```
+
+### Efficient Data Transfer
+- Use appropriate HTTP methods
+- Implement field selection (`?fields=id,name,email`)
+- Support compression (gzip)
+- Implement efficient pagination
+- Use ETags for conditional requests
+
+### Resource Optimization
+- Avoid N+1 queries
+- Implement batch operations
+- Use async processing for heavy operations
+- Support partial updates (PATCH)
+
+## Security Best Practices
+
+### Input Validation
+- Validate all input parameters
+- Sanitize user data
+- Use parameterized queries
+- Implement request size limits
+
+### Authentication Security
+- Use HTTPS everywhere
+- Implement secure token storage
+- Support token expiration and refresh
+- Use strong authentication mechanisms
+
+### Authorization Controls
+- Implement principle of least privilege
+- Use resource-based permissions
+- Support fine-grained access control
+- Audit access patterns
+
+## Tools and Scripts
+
+### api_linter.py
+Analyzes API specifications for compliance with REST conventions and best practices.
+
+**Features:**
+- OpenAPI/Swagger spec validation
+- Naming convention checks
+- HTTP method usage validation
+- Error format consistency
+- Documentation completeness analysis
+
+### breaking_change_detector.py
+Compares API specification versions to identify breaking changes.
+
+**Features:**
+- Endpoint comparison
+- Schema change detection
+- Field removal/modification tracking
+- Migration guide generation
+- Impact severity assessment
+
+### api_scorecard.py
+Provides comprehensive scoring of API design quality.
+
+**Features:**
+- Multi-dimensional scoring
+- Detailed improvement recommendations
+- Letter grade assessment (A-F)
+- Benchmark comparisons
+- Progress tracking
+
+## Integration Examples
+
+### CI/CD Integration
+```yaml
+- name: "api-linting"
+  run: python scripts/api_linter.py openapi.json
+
+- name: "breaking-change-detection"
+  run: python scripts/breaking_change_detector.py openapi-v1.json openapi-v2.json
+
+- name: "api-scorecard"
+  run: python scripts/api_scorecard.py openapi.json
+```
+
+### Pre-commit Hooks
+```bash
+#!/bin/bash
+python engineering/api-design-reviewer/scripts/api_linter.py api/openapi.json
+if [ $? -ne 0 ]; then
+  echo "API linting failed. Please fix the issues before committing."
+  exit 1
+fi
+```
+
+## Best Practices Summary
+
+1. **Consistency First**: Maintain consistent naming, response formats, and patterns
+2. **Documentation**: Provide comprehensive, up-to-date API documentation
+3. **Versioning**: Plan for evolution with clear versioning strategies
+4. **Error Handling**: Implement consistent, informative error responses
+5. **Security**: Build security into every layer of the API
+6. **Performance**: Design for scale and efficiency from the start
+7. **Backward Compatibility**: Minimize breaking changes and provide migration paths
+8. **Testing**: Implement comprehensive testing including contract testing
+9. **Monitoring**: Add observability for API usage and performance
+10. **Developer Experience**: Prioritize ease of use and clear documentation
+
+## Common Anti-Patterns to Avoid
+
+1. **Verb-based URLs**: Use nouns for resources, not actions
+2. **Inconsistent Response Formats**: Maintain standard response structures
+3. **Over-nesting**: Avoid deeply nested resource hierarchies
+4. **Ignoring HTTP Status Codes**: Use appropriate status codes for different scenarios
+5. **Poor Error Messages**: Provide actionable, specific error information
+6. **Missing Pagination**: Always paginate list endpoints
+7. **No Versioning Strategy**: Plan for API evolution from day one
+8. **Exposing Internal Structure**: Design APIs for external consumption, not internal convenience
+9. **Missing Rate Limiting**: Protect your API from abuse and overload
+10. **Inadequate Testing**: Test all aspects including error cases and edge conditions
+
+## Conclusion
+
+The API Design Reviewer skill provides a comprehensive framework for building, reviewing, and maintaining high-quality REST APIs. By following these guidelines and using the provided tools, development teams can create APIs that are consistent, well-documented, secure, and maintainable.
+
+Regular use of the linting, breaking change detection, and scoring tools ensures continuous improvement and helps maintain API quality throughout the development lifecycle.