feat(mixins): add Prometheus metrics and Redis cache backends (OMN-1188)

CLAUDE.md

@@ -74,6 +74,13 @@ When spawning polymorphic agents or AI assistants:
 - **NEVER** run `git commit` or `git push` in background mode - always foreground
 - **NEVER** run agents in background mode (`run_in_background: true`) - run parallel polymorphic agents in the foreground instead by dispatching multiple agents in a single message
 
+### Git Commit Rules
+
+⚠️ **CRITICAL - ABSOLUTELY FORBIDDEN**:
+- **NEVER use `--no-verify`** when committing. Pre-commit hooks exist for a reason.
+- **NEVER use `--no-gpg-sign`** unless explicitly requested by the user.
+- **NEVER skip hooks** - if hooks fail, fix the issues instead of bypassing them.
+
 ---
 
 ## Architecture Fundamentals

docs/architecture/MIXINS_TO_HANDLERS_REFACTOR.md

@@ -250,7 +250,7 @@ class ProtocolPipelineHook(Protocol):
     """Hook that wraps pipeline execution."""
 
     @property
-    def hook_id(self) -> str:
+    def hook_name(self) -> str:
         """Unique identifier for this hook instance."""
         ...
 
@@ -316,11 +316,11 @@ execution_profile:
   phases:
     preflight:
       hooks:
-        - hook_id: "auth_check"
+        - hook_name: "auth_check"
           priority: 100
     execute:
       hooks:
-        - hook_id: "retry"
+        - hook_name: "retry"
           config:
             max_attempts: 3
             backoff_ms: 100
@@ -640,7 +640,7 @@ class ModelOrderingSummary(BaseModel):
 
 class ModelPhaseHookOrder(BaseModel):
     """Hook ordering within a phase."""
-    hook_id: str
+    hook_name: str
     priority: int
     depends_on: list[str]
 ```
@@ -762,8 +762,8 @@ Within each phase:
 
 ```python
 # Hook B depends on Hook A
-hook_a = RetryHook(hook_id="retry", priority=100)
-hook_b = MetricsHook(hook_id="metrics", priority=200, depends_on=["retry"])
+hook_a = RetryHook(hook_name="retry", priority=100)
+hook_b = MetricsHook(hook_name="metrics", priority=200, depends_on=["retry"])
 
 # Execution order: retry → metrics (even if priorities were reversed)
 ```

docs/guides/HANDLER_CONVERSION_GUIDE.md

@@ -209,34 +209,34 @@ caching_handler = ModelCapabilityCaching(
 registry = RegistryHook()
 
 registry.register(ModelPipelineHook(
-    hook_id="metrics-before",
+    hook_name="metrics-before",
     phase="before",
     priority=10,
     callable_ref="hooks.metrics.before",
 ))
 
 registry.register(ModelPipelineHook(
-    hook_id="cache-check",
+    hook_name="cache-check",
     phase="before",
     priority=20,
     dependencies=["metrics-before"],
     callable_ref="hooks.caching.check",
 ))
 
 registry.register(ModelPipelineHook(
-    hook_id="compute",
+    hook_name="compute",
     phase="execute",
     callable_ref="hooks.compute.execute",
 ))
 
 registry.register(ModelPipelineHook(
-    hook_id="cache-store",
+    hook_name="cache-store",
     phase="after",
     callable_ref="hooks.caching.store",
 ))
 
 registry.register(ModelPipelineHook(
-    hook_id="metrics-after",
+    hook_name="metrics-after",
     phase="after",
     dependencies=["cache-store"],
     callable_ref="hooks.metrics.after",
@@ -847,7 +847,7 @@ async def test_handler_in_pipeline():
     # Setup pipeline
     registry = RegistryHook()
     registry.register(ModelPipelineHook(
-        hook_id="metrics",
+        hook_name="metrics",
         phase="after",
         callable_ref="test.metrics",
     ))