claudecode-omc 5.11.0 → 5.12.0

package/bundled/upstream/ecc/skills/quarkus-tdd/SKILL.md

@@ -1,7 +1,8 @@
 ---
 name: quarkus-tdd
 description: Test-driven development for Quarkus 3.x LTS using JUnit 5, Mockito, REST Assured, Camel testing, and JaCoCo. Use when adding features, fixing bugs, or refactoring event-driven services.
-origin: ECC
+metadata:
+  origin: ECC
 ---
 
 # Quarkus TDD Workflow
@@ -34,19 +35,19 @@ Follow this structured approach for comprehensive, readable tests:
 @ExtendWith(MockitoExtension.class)
 @DisplayName("OrderService Unit Tests")
 class OrderServiceTest {
-  
+
   @Mock
   private OrderRepository orderRepository;
-  
+
   @Mock
   private EventService eventService;
-  
+
   @Mock
   private FulfillmentPublisher fulfillmentPublisher;
-  
+
   @InjectMocks
   private OrderService orderService;
-  
+
   private CreateOrderCommand validCommand;
 
   @BeforeEach
@@ -60,16 +61,16 @@ class OrderServiceTest {
   @Nested
   @DisplayName("Tests for createOrder")
   class CreateOrder {
-    
+
     @Test
     @DisplayName("Should persist order and publish fulfillment event")
     void givenValidCommand_whenCreateOrder_thenPersistsAndPublishes() {
       // ARRANGE
       doNothing().when(orderRepository).persist(any(Order.class));
-      
+
       // ACT
       OrderReceipt receipt = orderService.createOrder(validCommand);
-      
+
       // ASSERT
       assertThat(receipt).isNotNull();
       assertThat(receipt.customerId()).isEqualTo("customer-123");
@@ -83,7 +84,7 @@ class OrderServiceTest {
     void givenMissingCustomerId_whenCreateOrder_thenThrowsBadRequest() {
       // ARRANGE
       CreateOrderCommand invalid = new CreateOrderCommand("", validCommand.lines());
-      
+
       // ACT & ASSERT
       WebApplicationException exception = assertThrows(
           WebApplicationException.class,
@@ -101,13 +102,13 @@ class OrderServiceTest {
       // ARRANGE
       doThrow(new PersistenceException("database unavailable"))
           .when(orderRepository).persist(any(Order.class));
-      
+
       // ACT & ASSERT
       PersistenceException exception = assertThrows(
           PersistenceException.class,
           () -> orderService.createOrder(validCommand)
       );
-      
+
       assertThat(exception.getMessage()).contains("database unavailable");
       verify(eventService).createErrorEvent(
           eq(validCommand),
@@ -125,7 +126,7 @@ class OrderServiceTest {
           NullPointerException.class,
           () -> orderService.createOrder(null)
       );
-      
+
       verify(orderRepository, never()).persist(any(Order.class));
     }
   }
@@ -184,7 +185,7 @@ class BusinessRulesRouteTest {
       // ARRANGE
       MockEndpoint mockRabbitMQ = camelContext.getEndpoint("mock:rabbitmq", MockEndpoint.class);
       mockRabbitMQ.expectedMessageCount(1);
-      
+
       // Replace real endpoint with mock for testing
       camelContext.getRouteController().stopRoute("business-rules-publisher");
       AdviceWith.adviceWith(camelContext, "business-rules-publisher", advice -> {
@@ -192,13 +193,13 @@ class BusinessRulesRouteTest {
         advice.weaveByToString(".*spring-rabbitmq.*").replace().to("mock:rabbitmq");
       });
       camelContext.getRouteController().startRoute("business-rules-publisher");
-      
+
       // ACT
       producerTemplate.sendBody("direct:business-rules-publisher", testPayload);
-      
+
       // ASSERT — body is a JSON String after .marshal().json(JsonLibrary.Jackson)
       mockRabbitMQ.assertIsSatisfied(5000);
-      
+
       assertThat(mockRabbitMQ.getExchanges()).hasSize(1);
       String body = mockRabbitMQ.getExchanges().get(0).getIn().getBody(String.class);
       assertThat(body).contains("\"documentId\":1");
@@ -211,19 +212,19 @@ class BusinessRulesRouteTest {
       MockEndpoint mockMarshal = new MockEndpoint("mock:marshal");
       camelContext.addEndpoint("mock:marshal", mockMarshal);
       mockMarshal.expectedMessageCount(1);
-      
+
       camelContext.getRouteController().stopRoute("business-rules-publisher");
       AdviceWith.adviceWith(camelContext, "business-rules-publisher", advice -> {
         advice.weaveAddLast().to("mock:marshal");
       });
       camelContext.getRouteController().startRoute("business-rules-publisher");
-      
+
       // ACT
       producerTemplate.sendBody("direct:business-rules-publisher", testPayload);
-      
+
       // ASSERT
       mockMarshal.assertIsSatisfied(5000);
-      
+
       String body = mockMarshal.getExchanges().get(0).getIn().getBody(String.class);
       assertThat(body).contains("\"documentId\":1");
       assertThat(body).contains("\"flowProfile\":\"BASIC\"");
@@ -240,17 +241,17 @@ class BusinessRulesRouteTest {
       // ARRANGE
       MockEndpoint mockInvoice = camelContext.getEndpoint("mock:invoice", MockEndpoint.class);
       mockInvoice.expectedMessageCount(1);
-      
+
       camelContext.getRouteController().stopRoute("document-processing");
       AdviceWith.adviceWith(camelContext, "document-processing", advice -> {
         advice.weaveByToString(".*direct:process-invoice.*").replace().to("mock:invoice");
       });
       camelContext.getRouteController().startRoute("document-processing");
-      
+
       // ACT
-      producerTemplate.sendBodyAndHeader("direct:process-document", 
+      producerTemplate.sendBodyAndHeader("direct:process-document",
           testPayload, "documentType", "INVOICE");
-      
+
       // ASSERT
       mockInvoice.assertIsSatisfied(5000);
     }
@@ -261,23 +262,23 @@ class BusinessRulesRouteTest {
       // ARRANGE
       MockEndpoint mockError = camelContext.getEndpoint("mock:error", MockEndpoint.class);
       mockError.expectedMessageCount(1);
-      
+
       camelContext.getRouteController().stopRoute("document-processing");
       AdviceWith.adviceWith(camelContext, "document-processing", advice -> {
         advice.weaveByToString(".*direct:validation-error-handler.*")
             .replace().to("mock:error");
       });
       camelContext.getRouteController().startRoute("document-processing");
-      
+
       // Mock validator bean to throw exception
       when(documentValidator.validate(any())).thenThrow(new ValidationException("Invalid document"));
-      
+
       // ACT
       producerTemplate.sendBody("direct:process-document", testPayload);
-      
+
       // ASSERT
       mockError.assertIsSatisfied(5000);
-      
+
       Exception exception = mockError.getExchanges().get(0).getException();
       assertThat(exception).isInstanceOf(ValidationException.class);
       assertThat(exception.getMessage()).contains("Invalid document");
@@ -295,13 +296,13 @@ class EventServiceTest {
 
   @Mock
   private EventRepository eventRepository;
-  
+
   @Mock
   private ObjectMapper objectMapper;
-  
+
   @InjectMocks
   private EventService eventService;
-  
+
   private BusinessRulesPayload testPayload;
 
   @BeforeEach
@@ -314,19 +315,19 @@ class EventServiceTest {
   @Nested
   @DisplayName("Tests for createSuccessEvent")
   class CreateSuccessEvent {
-    
+
     @Test
     @DisplayName("Should create success event with correct attributes")
     void givenValidPayload_whenCreateSuccessEvent_thenEventPersisted() throws Exception {
       // ARRANGE
       when(objectMapper.writeValueAsString(testPayload)).thenReturn("{\"documentId\":1}");
-      
+
       // ACT
-      assertDoesNotThrow(() -> 
+      assertDoesNotThrow(() ->
           eventService.createSuccessEvent(testPayload, "DOCUMENT_PROCESSED"));
-      
+
       // ASSERT
-      verify(eventRepository).persist(argThat(event -> 
+      verify(eventRepository).persist(argThat(event ->
           event.getType().equals("DOCUMENT_PROCESSED") &&
           event.getStatus() == EventStatus.SUCCESS &&
           event.getPayload().equals("{\"documentId\":1}") &&
@@ -339,13 +340,13 @@ class EventServiceTest {
     void givenNullPayload_whenCreateSuccessEvent_thenThrowsException() {
       // ARRANGE
       Object nullPayload = null;
-      
+
       // ACT & ASSERT
       NullPointerException exception = assertThrows(
           NullPointerException.class,
           () -> eventService.createSuccessEvent(nullPayload, "EVENT_TYPE")
       );
-      
+
       assertThat(exception.getMessage()).isEqualTo("Payload cannot be null");
       verify(eventRepository, never()).persist(any());
     }
@@ -354,20 +355,20 @@ class EventServiceTest {
   @Nested
   @DisplayName("Tests for createErrorEvent")
   class CreateErrorEvent {
-    
+
     @Test
     @DisplayName("Should create error event with error message")
     void givenError_whenCreateErrorEvent_thenEventPersistedWithMessage() throws Exception {
       // ARRANGE
       String errorMessage = "Processing failed";
       when(objectMapper.writeValueAsString(testPayload)).thenReturn("{\"documentId\":1}");
-      
+
       // ACT
-      assertDoesNotThrow(() -> 
+      assertDoesNotThrow(() ->
           eventService.createErrorEvent(testPayload, "PROCESSING_ERROR", errorMessage));
-      
+
       // ASSERT
-      verify(eventRepository).persist(argThat(event -> 
+      verify(eventRepository).persist(argThat(event ->
           event.getType().equals("PROCESSING_ERROR") &&
           event.getStatus() == EventStatus.ERROR &&
           event.getErrorMessage().equals(errorMessage) &&
@@ -384,7 +385,7 @@ class EventServiceTest {
           IllegalArgumentException.class,
           () -> eventService.createErrorEvent(testPayload, "ERROR", blankMessage)
       );
-      
+
       assertThat(exception.getMessage()).contains("Error message cannot be blank");
     }
   }
@@ -400,13 +401,13 @@ class FileStorageServiceTest {
 
   @Mock
   private S3Client s3Client;
-  
+
   @Mock
   private ExecutorService executorService;
-  
+
   @InjectMocks
   private FileStorageService fileStorageService;
-  
+
   private InputStream testInputStream;
   private LogContext testLogContext;
 
@@ -421,7 +422,7 @@ class FileStorageServiceTest {
   @Nested
   @DisplayName("Tests for uploadOriginalFile")
   class UploadOriginalFile {
-    
+
     @Test
     @DisplayName("Should successfully upload file and return document info")
     void givenValidFile_whenUpload_thenReturnsDocumentInfo() throws Exception {
@@ -430,23 +431,23 @@ class FileStorageServiceTest {
         ((Runnable) invocation.getArgument(0)).run();
         return null;
       }).when(executorService).execute(any(Runnable.class));
-      
+
       when(s3Client.putObject(any(PutObjectRequest.class), any(RequestBody.class)))
           .thenReturn(PutObjectResponse.builder().build());
-      
+
       // ACT
-      CompletableFuture<StoredDocumentInfo> future = 
-          fileStorageService.uploadOriginalFile(testInputStream, 1024L, 
+      CompletableFuture<StoredDocumentInfo> future =
+          fileStorageService.uploadOriginalFile(testInputStream, 1024L,
               testLogContext, InvoiceFormat.UBL);
-      
+
       StoredDocumentInfo result = future.join();
-      
+
       // ASSERT
       assertThat(result).isNotNull();
       assertThat(result.getPath()).isNotBlank();
       assertThat(result.getSize()).isEqualTo(1024L);
       assertThat(result.getUploadedAt()).isNotNull();
-      
+
       verify(s3Client).putObject(any(PutObjectRequest.class), any(RequestBody.class));
     }
 
@@ -458,15 +459,15 @@ class FileStorageServiceTest {
         ((Runnable) invocation.getArgument(0)).run();
         return null;
       }).when(executorService).execute(any(Runnable.class));
-      
+
       when(s3Client.putObject(any(PutObjectRequest.class), any(RequestBody.class)))
           .thenThrow(new StorageException("S3 unavailable"));
-      
+
       // ACT
-      CompletableFuture<StoredDocumentInfo> future = 
-          fileStorageService.uploadOriginalFile(testInputStream, 1024L, 
+      CompletableFuture<StoredDocumentInfo> future =
+          fileStorageService.uploadOriginalFile(testInputStream, 1024L,
               testLogContext, InvoiceFormat.UBL);
-      
+
       // ASSERT
       assertThatThrownBy(() -> future.join())
           .isInstanceOf(CompletionException.class)
@@ -479,17 +480,17 @@ class FileStorageServiceTest {
     void givenLogContext_whenUpload_thenContextPropagated() throws Exception {
       // ARRANGE
       AtomicReference<LogContext> capturedContext = new AtomicReference<>();
-      
+
       doAnswer(invocation -> {
         capturedContext.set(CustomLog.getCurrentContext());
         ((Runnable) invocation.getArgument(0)).run();
         return null;
       }).when(executorService).execute(any(Runnable.class));
-      
+
       // ACT
-      fileStorageService.uploadOriginalFile(testInputStream, 1024L, 
+      fileStorageService.uploadOriginalFile(testInputStream, 1024L,
           testLogContext, InvoiceFormat.UBL).join();
-      
+
       // ASSERT
       assertThat(capturedContext.get()).isNotNull();
       assertThat(capturedContext.get().get("traceId")).isEqualTo("trace-123");
@@ -641,7 +642,7 @@ class DocumentIntegrationTest {
         <goal>prepare-agent</goal>
       </goals>
     </execution>
-    
+
     <!-- Generate coverage report -->
     <execution>
       <id>report</id>
@@ -650,7 +651,7 @@ class DocumentIntegrationTest {
         <goal>report</goal>
       </goals>
     </execution>
-    
+
     <!-- Enforce coverage thresholds -->
     <execution>
       <id>check</id>
@@ -705,14 +706,14 @@ mvn jacoco:check
         <artifactId>quarkus-junit5-mockito</artifactId>
         <scope>test</scope>
     </dependency>
-    
+
     <!-- Mockito -->
     <dependency>
         <groupId>org.mockito</groupId>
         <artifactId>mockito-core</artifactId>
         <scope>test</scope>
     </dependency>
-    
+
     <!-- AssertJ (preferred over JUnit assertions) -->
     <dependency>
         <groupId>org.assertj</groupId>
@@ -720,14 +721,14 @@ mvn jacoco:check
         <version>3.24.2</version>
         <scope>test</scope>
     </dependency>
-    
+
     <!-- REST Assured -->
     <dependency>
         <groupId>io.rest-assured</groupId>
         <artifactId>rest-assured</artifactId>
         <scope>test</scope>
     </dependency>
-    
+
     <!-- Camel Testing -->
     <dependency>
         <groupId>org.apache.camel.quarkus</groupId>

package/bundled/upstream/ecc/skills/quarkus-verification/SKILL.md

@@ -1,7 +1,8 @@
 ---
 name: quarkus-verification
 description: "Verification loop for Quarkus projects: build, static analysis, tests with coverage, security scans, native compilation, and diff review before release or PR."
-origin: ECC
+metadata:
+  origin: ECC
 ---
 
 # Quarkus Verification Loop
@@ -437,28 +438,28 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v3
-      
+
       - name: Set up JDK 21
         uses: actions/setup-java@v3
         with:
           java-version: '21'
           distribution: 'temurin'
-      
+
       - name: Cache Maven packages
         uses: actions/cache@v3
         with:
           path: ~/.m2
           key: ${{ runner.os }}-m2-${{ hashFiles('**/pom.xml') }}
-      
+
       - name: Build
         run: mvn clean verify -DskipTests
-      
+
       - name: Test with Coverage
         run: mvn test jacoco:report jacoco:check
-      
+
       - name: Security Scan
         run: mvn org.owasp:dependency-check-maven:check
-      
+
       - name: Upload Coverage
         uses: codecov/codecov-action@v3
         with:

package/bundled/upstream/ecc/skills/ralphinho-rfc-pipeline/SKILL.md

@@ -1,7 +1,8 @@
 ---
 name: ralphinho-rfc-pipeline
 description: RFC-driven multi-agent DAG execution pattern with quality gates, merge queues, and work unit orchestration.
-origin: ECC
+metadata:
+  origin: ECC
 ---
 
 # Ralphinho RFC Pipeline

package/bundled/upstream/ecc/skills/react-patterns/SKILL.md

@@ -1,7 +1,8 @@
 ---
 name: react-patterns
 description: React 18/19 patterns including hooks discipline, server/client component boundaries, Suspense + error boundaries, form actions, data fetching, state management decision trees, and accessibility-first composition. Use when writing or reviewing React components.
-origin: ECC
+metadata:
+  origin: ECC
 ---
 
 # React Patterns

package/bundled/upstream/ecc/skills/react-performance/SKILL.md

@@ -1,7 +1,8 @@
 ---
 name: react-performance
 description: React and Next.js performance optimization patterns adapted from Vercel Engineering's React Best Practices (https://github.com/vercel-labs/agent-skills). Organizes 70+ rules across 8 priority categories — waterfalls, bundle size, server-side, client fetching, re-render, rendering, JS micro-perf, advanced. Use when writing, reviewing, or refactoring React/Next.js code for performance.
-origin: ECC
+metadata:
+  origin: ECC
 ---
 
 # React Performance

package/bundled/upstream/ecc/skills/react-testing/SKILL.md

@@ -1,7 +1,8 @@
 ---
 name: react-testing
 description: React component testing with React Testing Library, Vitest/Jest, MSW for network mocking, accessibility assertions with axe, and the decision boundary between component tests and Playwright/Cypress end-to-end runs. Use when writing or fixing tests for React components, hooks, or pages.
-origin: ECC
+metadata:
+  origin: ECC
 ---
 
 # React Testing

package/bundled/upstream/ecc/skills/recsys-pipeline-architect/SKILL.md

@@ -1,7 +1,8 @@
 ---
 name: recsys-pipeline-architect
 description: Design composable recommendation, ranking, and feed pipelines using the six-stage Source→Hydrator→Filter→Scorer→Selector→SideEffect framework popularized by xAI's open-sourced For You algorithm. Use this skill whenever the user is building any system that picks "the top K items for a (user, context)" — social feeds, content CMSs, RAG rerankers, task prioritizers, notification triage, search reranking, ad ranking.
-origin: community
+metadata:
+  origin: community
 ---
 
 # recsys-pipeline-architect

package/bundled/upstream/ecc/skills/recursive-decision-ledger/SKILL.md

@@ -1,7 +1,8 @@
 ---
 name: recursive-decision-ledger
 description: Use when the user asks for repeated rollouts, marked decision processes, high-dimensional search, stochastic optimization, local-optima exploration, ensemble comparison, or recursive reasoning with a visible evidence trail.
-origin: ECC
+metadata:
+  origin: ECC
 tools: Read, Write, Edit, Bash, Grep, Glob
 ---
 

package/bundled/upstream/ecc/skills/redis-patterns/SKILL.md

@@ -1,7 +1,8 @@
 ---
 name: redis-patterns
 description: Redis data structure patterns, caching strategies, distributed locks, rate limiting, pub/sub, and connection management for production applications.
-origin: ECC
+metadata:
+  origin: ECC
 ---
 
 # Redis Patterns

package/bundled/upstream/ecc/skills/regex-vs-llm-structured-text/SKILL.md

@@ -1,7 +1,8 @@
 ---
 name: regex-vs-llm-structured-text
 description: Decision framework for choosing between regex and LLM when parsing structured text — start with regex, add LLM only for low-confidence edge cases.
-origin: ECC
+metadata:
+  origin: ECC
 ---
 
 # Regex vs LLM for Structured Text Parsing