pumuki-ast-hooks 5.5.58 → 5.5.65

package/docs/ARCHITECTURE.md

@@ -3,6 +3,8 @@
 ## Table of Contents
 
 - [High-Level Architecture](#high-level-architecture)
+- [Framework Invariants](#framework-invariants)
+- [Control Primitives](#control-primitives)
 - [Data Flow](#data-flow)
 - [Key Components](#key-components)
 - [Quality Gates](#quality-gates)
@@ -10,6 +12,57 @@
 
 ---
 
+## Framework Invariants
+
+These invariants are non-negotiable. If any of them is violated, the framework is considered misconfigured.
+
+- **Evidence is the source of truth**
+  `.AI_EVIDENCE.json` is the authoritative state for AI-assisted work. The framework treats chat history as untrusted.
+
+- **Evidence freshness is mandatory**
+  Actions that rely on evidence must validate freshness (SLA) and surface stale conditions explicitly.
+
+- **AI Gate is a hard control point**
+  The framework must be able to deterministically decide `ALLOWED` vs `BLOCKED` before risky actions.
+
+- **Governance is Git-native**
+  Enforcement happens as early as possible (pre-commit / pre-push) to prevent invalid states from reaching CI/CD.
+
+- **Dependencies must point inward**
+  Presentation and Infrastructure may depend on Domain and Application, but never the opposite.
+
+---
+
+## Control Primitives
+
+Pumuki is a governance framework built around two primitives:
+
+### 1) AI Evidence (`.AI_EVIDENCE.json`)
+
+Responsibilities:
+
+- Persist project state for long-running work (multi-day, multi-chat)
+- Store severity metrics, platform detection, session metadata, and workflow state
+- Provide a stable input for AI clients and automation tools
+
+Failure-mode expectations:
+
+- If evidence cannot be read or parsed, the framework should fail safe (treat as missing) and emit a diagnosable signal (logs / guard debug log).
+
+### 2) AI Gate (Allow/Block)
+
+Responsibilities:
+
+- Provide deterministic control over AI-assisted actions
+- Encode mandatory rules (by platform/scope)
+- Enforce blocking thresholds (typically `CRITICAL`/`HIGH`)
+
+Failure-mode expectations:
+
+- If gate state cannot be loaded, the framework should fail safe and avoid proceeding with protected operations.
+
+---
+
 ## High-Level Architecture
 
 The system follows **strict Clean Architecture** with 4 well-defined layers:
@@ -128,6 +181,7 @@ flowchart LR
 ```
 
 **Pipeline Steps:**
+
 1. **AST Analysis**: Parse code and extract violations
 2. **Severity Evaluation**: Apply intelligent severity evaluation
 3. **Gate Policies**: Apply quality gates (CRITICAL/HIGH block)
@@ -221,6 +275,17 @@ flowchart TD
 
 ---
 
+## Integrations
+
+Pumuki is designed to integrate into existing enterprise workflows without requiring changes to product code.
+
+- **Git hooks**: pre-commit / pre-push enforcement
+- **CI/CD**: run audits in pipelines to enforce the same gates as local development
+- **IDE / agentic clients**: MCP servers for evidence, gate checks, and automation
+- **Local developer tools**: CLI entrypoints for manual audits, troubleshooting, and operational flows
+
+---
+
 ## References
 
 For more details on the architecture:
@@ -233,4 +298,4 @@ For more details on the architecture:
 
 **Last updated**: 2025-01-13  
 **Version**: 5.3.0  
-🐈💚 **Pumuki Team®** - Hook System Architecture
+🐈💚 **Pumuki Team®** - Hook System Architecture

package/docs/TODO.md

@@ -0,0 +1,41 @@
+# Framework Work Tracking (TODO)
+
+## Scope
+
+This document tracks the agreed improvements for **Pumuki AST Intelligence Framework**.
+
+## Done
+
+- Refactor `README.md` to position the project as a framework (governance + lifecycle + commands).
+- Add Git governance section documenting `ast:gitflow` and `ast:release` with options and flow.
+- Add Developer Experience section (notifications + evidence freshness + git-tree guardrails).
+- Make `docs/ARCHITECTURE.md` more normative by adding invariants + control primitives.
+
+## In Progress
+
+- Standardize visuals in `docs/images/` to consistent, resolution-independent assets.
+
+## Next
+
+### Documentation
+
+- Add README section explaining manual hook-system usage:
+  - interactive menu
+  - non-interactive usage via `AUDIT_OPTION`
+  - how `npx ast-hooks` maps to those flows
+- Convert root `ARCHITECTURE.md` into **Conceptual Architecture** and link to `docs/ARCHITECTURE.md` as the contract.
+
+### Framework Features
+
+- Add `human_intent` to both:
+  - `.AI_EVIDENCE.json` (source of truth)
+  - AI Gate output/state (must not drift)
+
+### Design Constraints
+
+- Avoid drift: define a single canonical writer and a deterministic merge strategy.
+- Ensure `expires_at` is enforced (ignore stale intent).
+
+## Notes
+
+- README uses inline HTML (`<img>`) for GitHub-friendly full-width rendering.

package/docs/images/ast_intelligence_01.svg

@@ -0,0 +1,40 @@
+
+<svg xmlns="http://www.w3.org/2000/svg" width="1600" height="900" viewBox="0 0 1600 900" role="img" aria-label="AST Intelligence System Overview">
+  <defs>
+    <style>
+      .bg { fill: #0b0f14; }
+      .panel { fill: #0f1620; stroke: #1f2a37; stroke-width: 2; }
+      .title { fill: #7dd3fc; font: 700 40px ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, "Liberation Mono", "Courier New", monospace; }
+      .sub { fill: #a7f3d0; font: 600 22px ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, "Liberation Mono", "Courier New", monospace; }
+      .txt { fill: #e5e7eb; font: 18px ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, "Liberation Mono", "Courier New", monospace; }
+      .muted { fill: #9ca3af; font: 18px ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, "Liberation Mono", "Courier New", monospace; }
+      .accent { fill: #fbbf24; font: 18px ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, "Liberation Mono", "Courier New", monospace; }
+      .ok { fill: #34d399; font: 18px ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, "Liberation Mono", "Courier New", monospace; }
+      .warn { fill: #f59e0b; font: 18px ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, "Liberation Mono", "Courier New", monospace; }
+    </style>
+  </defs>
+
+  <rect class="bg" x="0" y="0" width="1600" height="900" />
+  <rect class="panel" x="40" y="40" width="1520" height="820" rx="18" />
+
+  <text class="title" x="80" y="115">PUMUKI — Hook-System (run: npx ast-hooks)</text>
+  <text class="sub" x="80" y="160">AST Intelligence System Overview</text>
+
+  <text class="txt" x="80" y="230">1) Pattern checks</text>
+  <text class="txt" x="80" y="265">2) ESLint audits</text>
+  <text class="txt" x="80" y="300">3) AST Intelligence analysis</text>
+  <text class="txt" x="80" y="335">4) Intelligent Audit Gate (block/allow)</text>
+  <text class="txt" x="80" y="370">5) Evidence update (.AI_EVIDENCE.json)</text>
+
+  <text class="muted" x="80" y="435">Pipeline</text>
+  <text class="accent" x="80" y="470">Source files → AST analyzers → violations → severity evaluation → AI Gate verdict</text>
+
+  <text class="muted" x="80" y="535">Outputs</text>
+  <text class="txt" x="80" y="570">• .audit_tmp/ast-summary.json</text>
+  <text class="txt" x="80" y="605">• .audit_tmp/ast-summary-enhanced.json</text>
+  <text class="txt" x="80" y="640">• reports/ (json + text)</text>
+  <text class="txt" x="80" y="675">• .AI_EVIDENCE.json (ai_gate + metrics)</text>
+
+  <text class="warn" x="80" y="750">Rule: Fail fast, block early</text>
+  <text class="ok" x="80" y="785">Goal: deterministic governance across iOS / Android / Backend / Frontend</text>
+</svg>

package/docs/images/ast_intelligence_02.svg

@@ -0,0 +1,39 @@
+<svg xmlns="http://www.w3.org/2000/svg" width="1600" height="900" viewBox="0 0 1600 900" role="img" aria-label="AST Intelligence Workflow">
+  <defs>
+    <style>
+      .bg { fill: #0b0f14; }
+      .panel { fill: #0f1620; stroke: #1f2a37; stroke-width: 2; }
+      .h { fill: #a7f3d0; font: 700 26px ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, "Liberation Mono", "Courier New", monospace; }
+      .t { fill: #e5e7eb; font: 18px ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, "Liberation Mono", "Courier New", monospace; }
+      .m { fill: #9ca3af; font: 18px ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, "Liberation Mono", "Courier New", monospace; }
+      .accent { fill: #fbbf24; font: 18px ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, "Liberation Mono", "Courier New", monospace; }
+      .warn { fill: #f59e0b; font: 18px ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, "Liberation Mono", "Courier New", monospace; }
+    </style>
+  </defs>
+
+  <rect class="bg" x="0" y="0" width="1600" height="900" />
+  <rect class="panel" x="40" y="40" width="1520" height="820" rx="18" />
+
+  <text class="h" x="80" y="120">QUICK SUMMARY</text>
+
+  <text class="t" x="80" y="190">Files scanned:          269</text>
+  <text class="t" x="80" y="220">Total violations:       69</text>
+  <text class="t" x="80" y="250">ESLint errors:          0</text>
+  <text class="t" x="80" y="280">Critical issues:        3</text>
+  <text class="t" x="80" y="310">High priority:          38</text>
+
+  <text class="warn" x="80" y="365">⚠ STATUS: ACTION REQUIRED</text>
+  <text class="accent" x="80" y="395">Critical or high-severity issues detected</text>
+
+  <text class="h" x="80" y="465">1) PATTERN CHECKS</text>
+  <text class="t" x="110" y="510">• TODO / FIXME: 4</text>
+  <text class="t" x="110" y="540">• CONSOLE_LOG: 205</text>
+  <text class="t" x="110" y="570">• ANY TYPE: 8</text>
+  <text class="t" x="110" y="600">• SQL_RAW: 3</text>
+
+  <text class="h" x="80" y="670">2) ESLINT AUDIT RESULTS</text>
+  <text class="t" x="110" y="715">ESLint: errors=0  warnings=0</text>
+
+  <text class="h" x="80" y="785">3) AST INTELLIGENCE — SEVERITY BREAKDOWN</text>
+  <text class="m" x="110" y="830">Staging Area: no staged files to analyze</text>
+</svg>

package/docs/images/ast_intelligence_03.svg

@@ -0,0 +1,55 @@
+<svg xmlns="http://www.w3.org/2000/svg" width="1600" height="900" viewBox="0 0 1600 900" role="img" aria-label="AST Intelligence Audit - Part 1">
+  <defs>
+    <style>
+      .bg { fill: #0b0f14; }
+      .panel { fill: #0f1620; stroke: #1f2a37; stroke-width: 2; }
+      .h { fill: #a7f3d0; font: 700 24px ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, "Liberation Mono", "Courier New", monospace; }
+      .t { fill: #e5e7eb; font: 18px ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, "Liberation Mono", "Courier New", monospace; }
+      .muted { fill: #9ca3af; font: 18px ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, "Liberation Mono", "Courier New", monospace; }
+      .sevC { fill: #fb7185; }
+      .sevH { fill: #f59e0b; }
+      .sevM { fill: #fbbf24; }
+      .sevL { fill: #60a5fa; }
+      .dot { stroke: #1f2a37; stroke-width: 2; }
+    </style>
+  </defs>
+
+  <rect class="bg" x="0" y="0" width="1600" height="900" />
+  <rect class="panel" x="40" y="40" width="1520" height="820" rx="18" />
+
+  <text class="h" x="80" y="120">Platform: iOS</text>
+  <circle class="sevC dot" cx="95" cy="160" r="10" /><text class="t" x="120" y="166">CRITICAL: 2</text>
+  <circle class="sevH dot" cx="260" cy="160" r="10" /><text class="t" x="285" y="166">HIGH: 15</text>
+  <circle class="sevM dot" cx="395" cy="160" r="10" /><text class="t" x="420" y="166">MEDIUM: 6</text>
+  <circle class="sevL dot" cx="555" cy="160" r="10" /><text class="t" x="580" y="166">LOW: 13</text>
+
+  <text class="muted" x="80" y="210">Files affected: 1</text>
+  <text class="t" x="80" y="250">Top violations:</text>
+  <text class="t" x="110" y="285">ios.quality.long_function: 3</text>
+  <text class="t" x="110" y="315">ios.solid.isp.unused_dependency: 2</text>
+  <text class="t" x="110" y="345">ios.concurrency.missing_actor: 2</text>
+  <text class="t" x="110" y="375">ios.force_unwrapping: 2</text>
+  <text class="t" x="110" y="405">ios.uikit.viewmodel_delegation: 1</text>
+
+  <text class="h" x="80" y="500">Platform: Android</text>
+  <circle class="sevC dot" cx="95" cy="540" r="10" /><text class="t" x="120" y="546">CRITICAL: 0</text>
+  <circle class="sevH dot" cx="260" cy="540" r="10" /><text class="t" x="285" y="546">HIGH: 8</text>
+  <circle class="sevM dot" cx="395" cy="540" r="10" /><text class="t" x="420" y="546">MEDIUM: 8</text>
+  <circle class="sevL dot" cx="555" cy="540" r="10" /><text class="t" x="580" y="546">LOW: 1</text>
+
+  <text class="t" x="80" y="610">Top violations:</text>
+  <text class="t" x="110" y="645">android.asynctask: 2</text>
+  <text class="t" x="110" y="675">android.di.missing_android_entry_point: 1</text>
+  <text class="t" x="110" y="705">android.networking.missing_error_handling: 1</text>
+  <text class="t" x="110" y="735">android.coroutines.missing_lifecycle_scope: 1</text>
+
+  <text class="h" x="920" y="500">Platform: Other</text>
+  <circle class="sevC dot" cx="935" cy="540" r="10" /><text class="t" x="960" y="546">CRITICAL: 0</text>
+  <circle class="sevH dot" cx="1100" cy="540" r="10" /><text class="t" x="1125" y="546">HIGH: 2</text>
+  <circle class="sevM dot" cx="1235" cy="540" r="10" /><text class="t" x="1260" y="546">MEDIUM: 0</text>
+  <circle class="sevL dot" cx="1395" cy="540" r="10" /><text class="t" x="1420" y="546">LOW: 0</text>
+
+  <text class="t" x="920" y="610">Top violations:</text>
+  <text class="t" x="950" y="645">workflow.tdd.low_test_coverage: 1</text>
+  <text class="t" x="950" y="675">workflow.sequence.tests_lagging: 1</text>
+</svg>

package/docs/images/ast_intelligence_04.svg

@@ -0,0 +1,39 @@
+<svg xmlns="http://www.w3.org/2000/svg" width="1600" height="900" viewBox="0 0 1600 900" role="img" aria-label="AST Intelligence Audit - Part 2">
+  <defs>
+    <style>
+      .bg { fill: #0b0f14; }
+      .panel { fill: #0f1620; stroke: #1f2a37; stroke-width: 2; }
+      .h { fill: #a7f3d0; font: 700 24px ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, "Liberation Mono", "Courier New", monospace; }
+      .t { fill: #e5e7eb; font: 18px ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, "Liberation Mono", "Courier New", monospace; }
+      .muted { fill: #9ca3af; font: 18px ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, "Liberation Mono", "Courier New", monospace; }
+    </style>
+  </defs>
+
+  <rect class="bg" x="0" y="0" width="1600" height="900" />
+  <rect class="panel" x="40" y="40" width="1520" height="820" rx="18" />
+
+  <text class="h" x="80" y="120">5) TOP VIOLATIONS &amp; REMEDIATION</text>
+
+  <text class="t" x="80" y="200">• common.types.any (8 violations)</text>
+  <text class="muted" x="110" y="235">→ Review and fix violations.</text>
+
+  <text class="t" x="80" y="285">• ios.quality.long_function (3 violations)</text>
+  <text class="muted" x="110" y="320">→ Review and fix violations.</text>
+
+  <text class="t" x="80" y="370">• android.asynctask (2 violations)</text>
+  <text class="muted" x="110" y="405">→ Review and fix violations.</text>
+
+  <text class="t" x="80" y="455">• common.network.missing_timeout (2 violations)</text>
+  <text class="muted" x="110" y="490">→ Review and fix violations.</text>
+
+  <text class="t" x="80" y="540">• common.network.missing_error_handling (2 violations)</text>
+  <text class="muted" x="110" y="575">→ Review and fix violations.</text>
+
+  <text class="t" x="80" y="625">• ios.solid.isp.unused_dependency (2 violations)</text>
+  <text class="muted" x="110" y="660">→ Review and fix violations.</text>
+
+  <text class="t" x="80" y="710">• ios.concurrency.missing_actor (2 violations)</text>
+  <text class="muted" x="110" y="745">→ Review and fix violations.</text>
+
+  <text class="h" x="80" y="830">6) EXECUTIVE SUMMARY</text>
+</svg>

package/docs/images/ast_intelligence_05.svg

@@ -0,0 +1,45 @@
+<svg xmlns="http://www.w3.org/2000/svg" width="1600" height="900" viewBox="0 0 1600 900" role="img" aria-label="AST Intelligence Audit - Part 3">
+  <defs>
+    <style>
+      .bg { fill: #0b0f14; }
+      .panel { fill: #0f1620; stroke: #1f2a37; stroke-width: 2; }
+      .h { fill: #a7f3d0; font: 700 24px ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, "Liberation Mono", "Courier New", monospace; }
+      .t { fill: #e5e7eb; font: 18px ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, "Liberation Mono", "Courier New", monospace; }
+      .muted { fill: #9ca3af; font: 18px ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, "Liberation Mono", "Courier New", monospace; }
+      .errBox { fill: none; stroke: #fb7185; stroke-width: 3; }
+      .err { fill: #fb7185; font: 18px ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, "Liberation Mono", "Courier New", monospace; }
+      .sevC { fill: #fb7185; }
+      .sevH { fill: #f59e0b; }
+      .sevM { fill: #fbbf24; }
+      .sevL { fill: #60a5fa; }
+      .dot { stroke: #1f2a37; stroke-width: 2; }
+    </style>
+  </defs>
+
+  <rect class="bg" x="0" y="0" width="1600" height="900" />
+  <rect class="panel" x="40" y="40" width="1520" height="820" rx="18" />
+
+  <text class="h" x="80" y="120">METRICS</text>
+  <text class="t" x="80" y="170">Total violations detected: 69</text>
+  <text class="t" x="80" y="200">ESLint errors: 0</text>
+  <text class="t" x="80" y="230">Critical issues: 3</text>
+  <text class="t" x="80" y="260">High priority issues: 38</text>
+  <text class="t" x="80" y="290">Files scanned: 269</text>
+
+  <text class="muted" x="80" y="335">Code Health Score: 65% (Good)</text>
+
+  <rect class="errBox" x="80" y="370" width="1440" height="90" rx="10" />
+  <text class="err" x="110" y="425">ACTION REQUIRED: Critical or high-severity issues detected. Please review and fix before proceeding.</text>
+
+  <text class="h" x="80" y="520">FINAL SUMMARY — VIOLATIONS BY SEVERITY</text>
+
+  <circle class="sevC dot" cx="95" cy="565" r="10" /><text class="t" x="120" y="571">CRITICAL: 3</text>
+  <circle class="sevH dot" cx="95" cy="605" r="10" /><text class="t" x="120" y="611">HIGH: 38</text>
+  <circle class="sevM dot" cx="95" cy="645" r="10" /><text class="t" x="120" y="651">MEDIUM: 14</text>
+  <circle class="sevL dot" cx="95" cy="685" r="10" /><text class="t" x="120" y="691">LOW: 14</text>
+
+  <text class="muted" x="80" y="760">COMMIT BLOCKED — STRICT REPO+STAGING</text>
+  <text class="muted" x="80" y="795">Action: clean entire repository before committing.</text>
+
+  <text class="t" x="80" y="845">Generated by Pumuki — Hook-System</text>
+</svg>

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pumuki-ast-hooks",
-  "version": "5.5.58",
+  "version": "5.5.65",
   "description": "Enterprise-grade AST Intelligence System with multi-platform support (iOS, Android, Backend, Frontend) and Feature-First + DDD + Clean Architecture enforcement. Includes dynamic violations API for intelligent querying.",
   "main": "index.js",
   "bin": {
@@ -134,4 +134,4 @@
     "./skills": "./skills/skill-rules.json",
     "./hooks": "./hooks/index.js"
   }
-}
+}

package/scripts/hooks-system/.audit_tmp/hook-metrics.jsonl

@@ -206,3 +206,19 @@
 {"timestamp":1767819338043,"hook":"audit_logger","operation":"ensure_dir","status":"started"}
 {"timestamp":1767819338043,"hook":"audit_logger","operation":"ensure_dir","status":"success"}
 {"timestamp":1767819338043,"hook":"audit_logger","operation":"constructor","status":"success","repoRoot":"/Users/juancarlosmerlosalbarracin/Developer/Projects/ast-intelligence-hooks/scripts/hooks-system"}
+{"timestamp":1767963192762,"hook":"audit_logger","operation":"constructor","status":"started","repoRoot":"/Users/juancarlosmerlosalbarracin/Developer/Projects/ast-intelligence-hooks/scripts/hooks-system"}
+{"timestamp":1767963192762,"hook":"audit_logger","operation":"ensure_dir","status":"started"}
+{"timestamp":1767963192762,"hook":"audit_logger","operation":"ensure_dir","status":"success"}
+{"timestamp":1767963192762,"hook":"audit_logger","operation":"constructor","status":"success","repoRoot":"/Users/juancarlosmerlosalbarracin/Developer/Projects/ast-intelligence-hooks/scripts/hooks-system"}
+{"timestamp":1767963258796,"hook":"audit_logger","operation":"constructor","status":"started","repoRoot":"/Users/juancarlosmerlosalbarracin/Developer/Projects/ast-intelligence-hooks/scripts/hooks-system"}
+{"timestamp":1767963258796,"hook":"audit_logger","operation":"ensure_dir","status":"started"}
+{"timestamp":1767963258796,"hook":"audit_logger","operation":"ensure_dir","status":"success"}
+{"timestamp":1767963258796,"hook":"audit_logger","operation":"constructor","status":"success","repoRoot":"/Users/juancarlosmerlosalbarracin/Developer/Projects/ast-intelligence-hooks/scripts/hooks-system"}
+{"timestamp":1767963287588,"hook":"audit_logger","operation":"constructor","status":"started","repoRoot":"/Users/juancarlosmerlosalbarracin/Developer/Projects/ast-intelligence-hooks/scripts/hooks-system"}
+{"timestamp":1767963287588,"hook":"audit_logger","operation":"ensure_dir","status":"started"}
+{"timestamp":1767963287588,"hook":"audit_logger","operation":"ensure_dir","status":"success"}
+{"timestamp":1767963287588,"hook":"audit_logger","operation":"constructor","status":"success","repoRoot":"/Users/juancarlosmerlosalbarracin/Developer/Projects/ast-intelligence-hooks/scripts/hooks-system"}
+{"timestamp":1767963390612,"hook":"audit_logger","operation":"constructor","status":"started","repoRoot":"/Users/juancarlosmerlosalbarracin/Developer/Projects/ast-intelligence-hooks/scripts/hooks-system"}
+{"timestamp":1767963390612,"hook":"audit_logger","operation":"ensure_dir","status":"started"}
+{"timestamp":1767963390612,"hook":"audit_logger","operation":"ensure_dir","status":"success"}
+{"timestamp":1767963390612,"hook":"audit_logger","operation":"constructor","status":"success","repoRoot":"/Users/juancarlosmerlosalbarracin/Developer/Projects/ast-intelligence-hooks/scripts/hooks-system"}

package/scripts/hooks-system/application/DIValidationService.js

@@ -0,0 +1,43 @@
+const path = require('path');
+const StrategyRegistry = require('../infrastructure/registry/StrategyRegistry');
+const NodeFileSystemAdapter = require('../infrastructure/adapters/NodeFileSystemAdapter');
+
+class DIValidationService {
+    constructor() {
+        const fileSystemPort = new NodeFileSystemAdapter();
+        const configPath = path.join(__dirname, '../config/di-rules.json');
+        this.registry = new StrategyRegistry(fileSystemPort, configPath);
+        this.initialized = false;
+    }
+
+    async ensureInitialized() {
+        if (!this.initialized) {
+            await this.registry.loadStrategies();
+            this.initialized = true;
+        }
+    }
+
+    async validateDependencyInjection(analyzer, properties, filePath, className, line) {
+        await this.ensureInitialized();
+
+        const context = {
+            analyzer,
+            filePath,
+            className,
+            line,
+            properties
+        };
+
+        const strategies = this.registry.findStrategiesForNode(null, context);
+
+        for (const strategy of strategies) {
+            const violations = strategy.detect(null, context);
+
+            for (const violation of violations) {
+                strategy.report(violation, context);
+            }
+        }
+    }
+}
+
+module.exports = DIValidationService;

package/scripts/hooks-system/application/__tests__/DIValidationService.spec.js

@@ -0,0 +1,81 @@
+const DIValidationService = require('../../application/DIValidationService');
+const NodeFileSystemAdapter = require('../../infrastructure/adapters/NodeFileSystemAdapter');
+
+describe('DIValidationService', () => {
+    let diValidationService;
+    let mockAnalyzer;
+
+    beforeEach(() => {
+        diValidationService = new DIValidationService();
+        mockAnalyzer = {
+            pushFinding: jest.fn()
+        };
+    });
+
+    describe('validateDependencyInjection', () => {
+        it('should detect concrete dependency violations', async () => {
+            const properties = [
+                { 'key.name': 'apiClient', 'key.typename': 'APIClient' },
+                { 'key.name': 'repository', 'key.typename': 'UserRepository' }
+            ];
+
+            await diValidationService.validateDependencyInjection(
+                mockAnalyzer,
+                properties,
+                'TestViewModel.swift',
+                'TestViewModel',
+                10
+            );
+
+            expect(mockAnalyzer.pushFinding).toHaveBeenCalledWith(
+                'ios.solid.dip.concrete_dependency',
+                'high',
+                'TestViewModel.swift',
+                10,
+                "'TestViewModel' depends on concrete 'APIClient' - use protocol"
+            );
+
+            expect(mockAnalyzer.pushFinding).toHaveBeenCalledWith(
+                'ios.solid.dip.concrete_dependency',
+                'high',
+                'TestViewModel.swift',
+                10,
+                "'TestViewModel' depends on concrete 'UserRepository' - use protocol"
+            );
+        });
+
+        it('should skip allowed types', async () => {
+            const properties = [
+                { 'key.name': 'name', 'key.typename': 'String' },
+                { 'key.name': 'count', 'key.typename': 'Int' }
+            ];
+
+            await diValidationService.validateDependencyInjection(
+                mockAnalyzer,
+                properties,
+                'TestViewModel.swift',
+                'TestViewModel',
+                10
+            );
+
+            expect(mockAnalyzer.pushFinding).not.toHaveBeenCalled();
+        });
+
+        it('should skip protocol types', async () => {
+            const properties = [
+                { 'key.name': 'apiClient', 'key.typename': 'APIClientProtocol' },
+                { 'key.name': 'repository', 'key.typename': 'any UserRepositoryProtocol' }
+            ];
+
+            await diValidationService.validateDependencyInjection(
+                mockAnalyzer,
+                properties,
+                'TestViewModel.swift',
+                'TestViewModel',
+                10
+            );
+
+            expect(mockAnalyzer.pushFinding).not.toHaveBeenCalled();
+        });
+    });
+});

package/scripts/hooks-system/application/services/installation/McpConfigurator.js

@@ -3,7 +3,6 @@ const path = require('path');
 const AuditLogger = require('../logging/AuditLogger');
 const McpServerConfigBuilder = require('./mcp/McpServerConfigBuilder');
 const McpProjectConfigWriter = require('./mcp/McpProjectConfigWriter');
-const McpGlobalConfigCleaner = require('./mcp/McpGlobalConfigCleaner');
 
 const COLORS = {
     reset: '\x1b[0m',
@@ -19,7 +18,6 @@ class McpConfigurator {
         this.logger = logger;
         this.serverConfigBuilder = new McpServerConfigBuilder(targetRoot, hookSystemRoot, logger);
         this.projectWriter = new McpProjectConfigWriter(targetRoot, logger, this.logSuccess.bind(this), this.logWarning.bind(this), this.logInfo.bind(this));
-        this.globalCleaner = new McpGlobalConfigCleaner(targetRoot, logger, this.logInfo.bind(this));
     }
 
     configure() {
@@ -27,7 +25,6 @@ class McpConfigurator {
 
         const { serverId, mcpConfig } = this.serverConfigBuilder.build();
         this.projectWriter.configureProjectScoped(mcpConfig, serverId);
-        this.globalCleaner.cleanupGlobalConfig(serverId);
     }
 
     detectIDEs() {

package/scripts/hooks-system/config/di-rules.json

@@ -0,0 +1,42 @@
+{
+    "dependencyInjection": {
+        "severity": "high",
+        "targetClasses": [
+            "ViewModel",
+            "Service",
+            "Repository",
+            "UseCase"
+        ],
+        "allowedTypes": [
+            "String",
+            "Int",
+            "Bool",
+            "Double",
+            "Float",
+            "Date",
+            "URL",
+            "Data"
+        ],
+        "concretePatterns": [
+            "Service$",
+            "Repository$",
+            "UseCase$",
+            "Client$"
+        ],
+        "protocolIndicators": [
+            "Protocol",
+            "any ",
+            "some "
+        ],
+        "genericTypePatterns": {
+            "singleLetter": true,
+            "camelCase": "^[A-Z][a-z]*$",
+            "excludeWithImpl": true,
+            "contextHints": [
+                "<",
+                "apiClient",
+                "client"
+            ]
+        }
+    }
+}

package/scripts/hooks-system/domain/ports/FileSystemPort.js

@@ -0,0 +1,19 @@
+class FileSystemPort {
+    readDir(path) {
+        throw new Error('readDir must be implemented');
+    }
+
+    readFile(path, encoding) {
+        throw new Error('readFile must be implemented');
+    }
+
+    resolvePath(...paths) {
+        throw new Error('resolvePath must be implemented');
+    }
+
+    exists(path) {
+        throw new Error('exists must be implemented');
+    }
+}
+
+module.exports = FileSystemPort;