oh-my-customcode 0.32.0 → 0.33.1

package/templates/.claude/skills/java21-best-practices/SKILL.md

@@ -0,0 +1,305 @@
+---
+name: java21-best-practices
+description: Modern Java 21 patterns from Virtual Threads, Pattern Matching, Records, and Sealed Classes
+scope: core
+user-invocable: false
+---
+
+## Purpose
+
+Apply modern Java 21 patterns and best practices leveraging the latest language features for clean, performant, and maintainable Java code.
+
+## Core Principles
+
+```
+Embrace modern Java features over legacy patterns
+Prefer immutability and data-centric design
+Leverage Virtual Threads for scalable concurrency
+Use Pattern Matching for expressive type handling
+```
+
+## Rules
+
+### 1. Naming Conventions (Google Java Style)
+
+```yaml
+packages:
+  style: lowercase, no underscores
+  example: com.example.project
+
+classes:
+  style: UpperCamelCase
+  example: OrderProcessor, UserRecord
+
+methods_variables:
+  style: lowerCamelCase
+  example: processOrder(), itemCount
+
+constants:
+  style: UPPER_SNAKE_CASE
+  example: static final int MAX_RETRIES = 3
+
+records:
+  style: UpperCamelCase (same as class)
+  example: record Point(int x, int y) {}
+```
+
+### 2. Virtual Threads (JEP 444)
+
+```yaml
+principles:
+  - Use Virtual Threads for I/O-bound tasks
+  - Avoid pooling Virtual Threads (they are lightweight)
+  - Replace thread pools for blocking I/O with Virtual Thread executors
+
+patterns: |
+  // Preferred: Virtual Thread executor
+  try (var executor = Executors.newVirtualThreadPerTaskExecutor()) {
+      Future<String> result = executor.submit(() -> fetchFromDB());
+      System.out.println(result.get());
+  }
+
+  // Direct creation
+  Thread vThread = Thread.ofVirtual().start(() -> {
+      // blocking I/O is fine here
+      String data = callExternalApi();
+      process(data);
+  });
+
+  // With structured concurrency (JEP 453, preview)
+  try (var scope = new StructuredTaskScope.ShutdownOnFailure()) {
+      Future<String> user = scope.fork(() -> fetchUser(id));
+      Future<List<Order>> orders = scope.fork(() -> fetchOrders(id));
+      scope.join().throwIfFailed();
+      return new UserProfile(user.get(), orders.get());
+  }
+
+antipatterns:
+  - "Executors.newFixedThreadPool() for I/O tasks — use Virtual Threads instead"
+  - "Thread.sleep() on platform threads for rate limiting — use Virtual Threads"
+  - "synchronized blocks in Virtual Thread code — use ReentrantLock instead"
+```
+
+### 3. Pattern Matching
+
+```yaml
+instanceof_patterns: |
+  // Old style (avoid)
+  if (obj instanceof String) {
+      String s = (String) obj;
+      System.out.println(s.length());
+  }
+
+  // New style (prefer)
+  if (obj instanceof String s) {
+      System.out.println(s.length());
+  }
+
+  // With guard
+  if (obj instanceof String s && s.length() > 5) {
+      System.out.println("Long string: " + s);
+  }
+
+switch_patterns: |
+  // Pattern matching for switch (JEP 441)
+  String result = switch (obj) {
+      case Integer i -> "int " + i;
+      case String s  -> "str " + s;
+      case null      -> "null";
+      default        -> "other " + obj;
+  };
+
+  // With guards (when clause)
+  String label = switch (shape) {
+      case Circle c when c.radius() > 10 -> "large circle";
+      case Circle c                       -> "small circle";
+      case Rectangle r                    -> "rectangle";
+      default                             -> "unknown";
+  };
+```
+
+### 4. Records (JEP 395)
+
+```yaml
+principles:
+  - Use Records for immutable data carriers
+  - Prefer Records over POJOs with getters/setters for pure data
+  - Compact constructors for validation
+
+patterns: |
+  // Basic record
+  record Point(int x, int y) {}
+
+  // With validation (compact constructor)
+  record Range(int min, int max) {
+      Range {
+          if (min > max) throw new IllegalArgumentException(
+              "min %d > max %d".formatted(min, max));
+      }
+  }
+
+  // With custom methods
+  record Circle(double radius) {
+      double area() {
+          return Math.PI * radius * radius;
+      }
+  }
+
+  // Implementing interface
+  interface Shape { double area(); }
+  record Square(double side) implements Shape {
+      public double area() { return side * side; }
+  }
+
+antipatterns:
+  - "Mutable state in records — records are inherently immutable"
+  - "Using records for entities with behavior — prefer classes"
+```
+
+### 5. Record Patterns (JEP 440)
+
+```yaml
+patterns: |
+  // Deconstruct record in instanceof
+  if (obj instanceof Point(int x, int y)) {
+      System.out.println("x=" + x + ", y=" + y);
+  }
+
+  // In switch
+  String describe = switch (shape) {
+      case Circle(double r)        -> "circle r=" + r;
+      case Rectangle(double w, double h) -> "rect " + w + "x" + h;
+      default -> "unknown";
+  };
+
+  // Nested record patterns
+  record ColoredPoint(Point point, Color color) {}
+  if (obj instanceof ColoredPoint(Point(int x, int y), Color c)) {
+      System.out.println("colored point at " + x + "," + y);
+  }
+```
+
+### 6. Sealed Classes (JEP 409)
+
+```yaml
+principles:
+  - Use Sealed Classes for closed type hierarchies
+  - Combine with Pattern Matching switch for exhaustive handling
+  - Prefer sealed interfaces for behavior-focused hierarchies
+
+patterns: |
+  // Sealed interface with records
+  sealed interface Shape permits Circle, Rectangle, Triangle {}
+  record Circle(double radius) implements Shape {}
+  record Rectangle(double width, double height) implements Shape {}
+  record Triangle(double base, double height) implements Shape {}
+
+  // Exhaustive switch (no default needed)
+  double area = switch (shape) {
+      case Circle(double r)             -> Math.PI * r * r;
+      case Rectangle(double w, double h) -> w * h;
+      case Triangle(double b, double h)  -> 0.5 * b * h;
+  };
+```
+
+### 7. Sequenced Collections (JEP 431)
+
+```yaml
+principles:
+  - Use SequencedCollection for ordered access
+  - getFirst()/getLast() replace get(0) and get(size-1)
+
+patterns: |
+  // SequencedCollection methods
+  List<String> list = new ArrayList<>(List.of("a", "b", "c"));
+  String first = list.getFirst(); // "a"
+  String last  = list.getLast();  // "c"
+  list.addFirst("z");             // ["z", "a", "b", "c"]
+  list.addLast("end");            // ["z", "a", "b", "c", "end"]
+
+  // Reversed view
+  List<String> reversed = list.reversed();
+
+  // SequencedMap
+  SequencedMap<String, Integer> map = new LinkedHashMap<>();
+  map.put("one", 1);
+  map.put("two", 2);
+  Map.Entry<String, Integer> firstEntry = map.firstEntry(); // "one"=1
+```
+
+### 8. Text Blocks and String Features
+
+```yaml
+patterns: |
+  // Text blocks (since Java 15)
+  String json = """
+      {
+          "name": "Alice",
+          "age": 30
+      }
+      """;
+
+  // String.formatted() (since Java 15)
+  String msg = "Hello, %s! You are %d years old.".formatted(name, age);
+
+  // String templates (JEP 430, preview in Java 21)
+  // Prefer String.formatted() or MessageFormat for now
+```
+
+### 9. Error Handling
+
+```yaml
+principles:
+  - Prefer checked exceptions for recoverable conditions
+  - Use unchecked exceptions for programming errors
+  - Never swallow exceptions silently
+  - Use specific exception types
+
+patterns: |
+  // Specific exception type
+  public User findUser(long id) {
+      return userRepository.findById(id)
+          .orElseThrow(() -> new UserNotFoundException("User not found: " + id));
+  }
+
+  // Multi-catch
+  try {
+      process();
+  } catch (IOException | SQLException e) {
+      log.error("Data access failed", e);
+      throw new ServiceException("Processing failed", e);
+  }
+```
+
+### 10. Documentation
+
+```yaml
+format: |
+  /**
+   * Processes the given {@link Order} and returns a {@link Receipt}.
+   *
+   * @param order the order to process (must not be null)
+   * @return the generated receipt
+   * @throws OrderException if the order cannot be fulfilled
+   */
+  public Receipt processOrder(Order order) { }
+
+best_practices:
+  - Use @param and @return for public API
+  - Link related types with {@link}
+  - Document checked exceptions with @throws
+  - Keep Javadoc focused on "what", not "how"
+```
+
+## Application
+
+When writing or reviewing Java 21 code:
+
+1. **Use** Records for pure data classes over verbose POJOs
+2. **Use** Sealed Classes + Pattern Matching for type hierarchies
+3. **Use** Virtual Threads for I/O-bound concurrency
+4. **Use** `instanceof` pattern matching over explicit casts
+5. **Prefer** switch expressions over switch statements
+6. **Use** `getFirst()`/`getLast()` for sequenced collections
+7. **Write** exhaustive switch for sealed types (no default)
+8. **Document** public APIs with proper Javadoc

package/templates/.claude/skills/jinja2-prompts/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: jinja2-prompts
 description: Parameterized prompt templates using Jinja2 patterns for reusable, dynamic agent prompts
+scope: core
 user-invocable: false
 ---
 

package/templates/.claude/skills/kafka-best-practices/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: kafka-best-practices
 description: Apache Kafka best practices for event streaming, topic design, and producer-consumer patterns
+scope: core
 user-invocable: false
 ---
 

package/templates/.claude/skills/kotlin-best-practices/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: kotlin-best-practices
 description: Idiomatic Kotlin patterns from JetBrains conventions
+scope: core
 user-invocable: false
 ---
 

package/templates/.claude/skills/lists/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: lists
 description: Show all available commands
+scope: core
 argument-hint: "[--category <category>] [--verbose]"
 ---
 

package/templates/.claude/skills/memory-management/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: memory-management
 description: Memory persistence operations using claude-mem
+scope: core
 user-invocable: false
 ---
 

package/templates/.claude/skills/memory-recall/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: memory-recall
 description: Search and recall memories from claude-mem
+scope: core
 argument-hint: "<query> [--recent] [--limit <n>]"
 ---
 

package/templates/.claude/skills/memory-save/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: memory-save
 description: Save current session context to claude-mem
+scope: core
 argument-hint: "[--tags <tags>] [--include-code]"
 disable-model-invocation: true
 ---

package/templates/.claude/skills/model-escalation/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: model-escalation
 description: Advisory model escalation based on task outcome tracking
+scope: core
 ---
 
 # Model Escalation Skill

package/templates/.claude/skills/monitoring-setup/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: monitoring-setup
 description: Enable/disable OpenTelemetry console monitoring for Claude Code usage tracking
+scope: package
 argument-hint: "[enable|disable|status]"
 ---
 

package/templates/.claude/skills/multi-model-verification/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: multi-model-verification
 description: Parallel code verification using multiple models with severity classification
+scope: core
 version: 1.0.0
 user-invocable: false
 ---

package/templates/.claude/skills/npm-audit/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: npm-audit
 description: Audit npm dependencies for security and updates
+scope: package
 argument-hint: "[--fix] [--production]"
 ---
 

package/templates/.claude/skills/npm-publish/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: npm-publish
 description: Publish package to npm registry with pre-checks
+scope: package
 argument-hint: "[--tag <tag>] [--dry-run]"
 disable-model-invocation: true
 ---

package/templates/.claude/skills/npm-version/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: npm-version
 description: Manage semantic versions for npm packages
+scope: package
 argument-hint: "<major|minor|patch> [--no-tag] [--no-commit]"
 disable-model-invocation: true
 ---

package/templates/.claude/skills/optimize-analyze/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: optimize-analyze
 description: Analyze bundle size and performance metrics
+scope: core
 argument-hint: "[target] [--verbose]"
 ---
 

package/templates/.claude/skills/optimize-bundle/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: optimize-bundle
 description: Apply bundle size optimizations
+scope: core
 argument-hint: "[--dry-run] [--safe]"
 disable-model-invocation: true
 ---

package/templates/.claude/skills/optimize-report/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: optimize-report
 description: Generate comprehensive optimization report
+scope: core
 argument-hint: "[--baseline <file>] [--format <format>]"
 ---
 

package/templates/.claude/skills/pipeline-architecture-patterns/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: pipeline-architecture-patterns
 description: Data pipeline architecture patterns for ETL/ELT design, orchestration, and data quality frameworks
+scope: core
 user-invocable: false
 ---
 

package/templates/.claude/skills/pipeline-guards/SKILL.md

@@ -1,6 +1,8 @@
 ---
 name: pipeline-guards
 description: Safety constraints and quality gates for pipeline and workflow execution
+scope: core
+context: fork
 ---
 
 # Pipeline Guards Skill

package/templates/.claude/skills/postgres-best-practices/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: postgres-best-practices
 description: PostgreSQL best practices for database design, query optimization, and performance tuning
+scope: core
 user-invocable: false
 ---
 

package/templates/.claude/skills/pr-auto-improve/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: pr-auto-improve
 description: Opt-in post-PR analysis and improvement suggestions for code quality enhancement
+scope: core
 ---
 
 # PR Auto-Improvement Skill

package/templates/.claude/skills/python-best-practices/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: python-best-practices
 description: Pythonic patterns from PEP 8 and PEP 20
+scope: core
 user-invocable: false
 ---
 

package/templates/.claude/skills/qa-lead-routing/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: qa-lead-routing
 description: Coordinates QA workflow across planning, writing, and execution agents. Use when user requests testing, quality assurance, or test documentation.
+scope: core
 user-invocable: false
 context: fork
 ---

package/templates/.claude/skills/react-best-practices/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: react-best-practices
 description: React/Next.js performance optimization with 40+ rules in 8 categories
+scope: core
 user-invocable: false
 ---
 

package/templates/.claude/skills/redis-best-practices/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: redis-best-practices
 description: Redis best practices for caching, data structures, and in-memory data architecture
+scope: core
 user-invocable: false
 ---
 

package/templates/.claude/skills/research/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: research
 description: 10-team parallel deep analysis with cross-verification for any topic, repository, or technology. Use when user invokes /research or asks for comprehensive research.
+scope: core
 user-invocable: true
 ---
 

package/templates/.claude/skills/result-aggregation/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: result-aggregation
 description: Aggregate parallel agent results into concise output
+scope: core
 user-invocable: false
 ---
 

package/templates/.claude/skills/rust-best-practices/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: rust-best-practices
 description: Idiomatic Rust patterns from official guidelines
+scope: core
 user-invocable: false
 ---
 

package/templates/.claude/skills/sauron-watch/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: sauron-watch
 description: Full R017 verification (5+3 rounds) before commit
+scope: harness
 disable-model-invocation: true
 ---
 

package/templates/.claude/skills/secretary-routing/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: secretary-routing
 description: Routes agent management tasks to the correct manager agent. Use when user requests agent creation, updates, audits, git operations, or verification.
+scope: core
 user-invocable: false
 context: fork
 ---

package/templates/.claude/skills/skills-sh-search/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: skills-sh-search
 description: Search and install skills from skills.sh marketplace when internal skills are insufficient
+scope: core
 argument-hint: "<query> [--install] [--global]"
 ---
 

package/templates/.claude/skills/snowflake-best-practices/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: snowflake-best-practices
 description: Snowflake best practices for cloud data warehouse design, query optimization, and cost management
+scope: core
 user-invocable: false
 ---
 

package/templates/.claude/skills/spark-best-practices/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: spark-best-practices
 description: Apache Spark best practices for PySpark and Scala distributed data processing
+scope: core
 user-invocable: false
 ---
 

package/templates/.claude/skills/springboot-best-practices/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: springboot-best-practices
 description: Spring Boot patterns for enterprise Java applications
+scope: core
 user-invocable: false
 ---
 

package/templates/.claude/skills/status/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: status
 description: Show system status and health checks
+scope: core
 argument-hint: "[--verbose] [--health]"
 ---
 

package/templates/.claude/skills/structured-dev-cycle/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: structured-dev-cycle
 description: 6-stage structured development cycle with stage-based tool restrictions
+scope: core
 version: 1.0.0
 user-invocable: true
 ---

package/templates/.claude/skills/stuck-recovery/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: stuck-recovery
 description: Detect stuck loops and advise recovery strategies
+scope: core
 ---
 
 # Stuck Recovery Skill

package/templates/.claude/skills/supabase-postgres-best-practices/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: supabase-postgres-best-practices
 description: PostgreSQL performance optimization guidelines from Supabase. Apply when writing SQL, designing schemas, configuring RLS, or optimizing database performance.
+scope: core
 user-invocable: false
 ---
 
@@ -96,4 +97,4 @@ user-invocable: false
 - PostgreSQL Official Docs: https://www.postgresql.org/docs/
 - Supabase Agent Skills: https://github.com/supabase/agent-skills
 
-For detailed rule files with specific examples, see guides/supabase-postgres/.
+For detailed rule files with specific examples, see templates/guides/supabase-postgres/.

package/templates/.claude/skills/task-decomposition/SKILL.md

@@ -1,6 +1,8 @@
 ---
 name: task-decomposition
 description: Auto-decompose large tasks into DAG-compatible parallel subtasks
+scope: core
+context: fork
 ---
 
 # Task Decomposition Skill

package/templates/.claude/skills/typescript-best-practices/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: typescript-best-practices
 description: Type-safe TypeScript patterns from industry standards
+scope: core
 user-invocable: false
 ---