tribunal-kit 5.8.3 → 5.8.5

package/dist/mcp/server.js

@@ -1,142 +1,8 @@
 #!/usr/bin/env node
 "use strict";
+
 /**
- * Tribunal-Kit MCP Server (dist/ version — Performance-Optimized)
- *
- * Uses in-process require() for commands instead of spawning child processes.
- * Protocol: MCP 2024-11-05 over JSON-RPC 2.0 / stdio
+ * Compatibility entry point for the historical dist/mcp/server.js path.
+ * bin/mcp-server.js is the single canonical MCP implementation.
  */
-const { spawnSync } = require('child_process');
-const path = require('path');
-
-const PKG = require('../../package.json');
-
-// Timeout for spawned processes (30 seconds)
-const SPAWN_TIMEOUT_MS = 30000;
-
-const readline = require('readline');
-const rl = readline.createInterface({
-    input: process.stdin,
-    output: process.stdout,
-    terminal: false
-});
-
-function handleRequest(req) {
-    if (req.method === 'initialize') {
-        return {
-            protocolVersion: "2024-11-05",
-            capabilities: { tools: {} },
-            serverInfo: {
-                name: "tribunal-kit-mcp",
-                version: PKG.version
-            }
-        };
-    }
-
-    if (req.method === 'tools/list') {
-        return {
-            tools: [
-                {
-                    name: "run_tribunal_audit",
-                    description: "Runs a full anti-hallucination audit across the workspace.",
-                    inputSchema: { type: "object", properties: {}, additionalProperties: false }
-                },
-                {
-                    name: "sync_ide_bridges",
-                    description: "Synchronize IDE bridge files with the current GEMINI.md rules.",
-                    inputSchema: { type: "object", properties: {}, additionalProperties: false }
-                },
-                {
-                    name: "search_case_law",
-                    description: "Search historical code rejections and legal precedent.",
-                    inputSchema: {
-                        type: "object",
-                        properties: {
-                            query: { type: "string", description: "Search query" }
-                        },
-                        required: ["query"],
-                        additionalProperties: false
-                    }
-                }
-            ]
-        };
-    }
-
-    if (req.method === 'tools/call') {
-        const toolName = req.params && req.params.name;
-        if (!toolName) {
-            throw { code: -32602, message: "Missing required parameter: params.name" };
-        }
-
-        if (toolName === 'run_tribunal_audit') {
-            // Validate uses the Rust binary — still needs spawn
-            const CLI = path.resolve(__dirname, '../../bin/wrapper.js');
-            const result = spawnSync(process.execPath, [CLI, 'validate', '--quiet'], {
-                encoding: 'utf8',
-                timeout: SPAWN_TIMEOUT_MS,
-            });
-            return { content: [{ type: "text", text: result.stdout || result.stderr || "No output" }] };
-        }
-
-        if (toolName === 'sync_ide_bridges') {
-            // In-process — no spawn needed
-            const CLI = path.resolve(__dirname, '../../bin/wrapper.js');
-            const result = spawnSync(process.execPath, [CLI, 'sync', '--quiet'], {
-                encoding: 'utf8',
-                timeout: SPAWN_TIMEOUT_MS,
-            });
-            return { content: [{ type: "text", text: result.stdout || result.stderr || "Sync complete" }] };
-        }
-
-        if (toolName === 'search_case_law') {
-            const query = req.params && req.params.arguments && req.params.arguments.query;
-            if (!query || typeof query !== 'string') {
-                throw { code: -32602, message: "Missing or invalid required argument: query (string)" };
-            }
-            const script = path.resolve(__dirname, '../../.agent/scripts/case_law_manager.js');
-            const result = spawnSync(process.execPath, [script, 'search-cases', '--query', query], {
-                encoding: 'utf8',
-                timeout: SPAWN_TIMEOUT_MS,
-            });
-            return { content: [{ type: "text", text: result.stdout || result.stderr || "No results" }] };
-        }
-
-        throw { code: -32601, message: `Unknown tool: ${toolName}` };
-    }
-
-    throw { code: -32601, message: `Unknown method: ${req.method}` };
-}
-
-rl.on('line', (line) => {
-    if (!line.trim()) return;
-
-    let req;
-    try {
-        req = JSON.parse(line);
-    } catch (parseErr) {
-        const errorRes = {
-            jsonrpc: "2.0", id: null,
-            error: { code: -32700, message: "Parse error: " + parseErr.message }
-        };
-        console.log(JSON.stringify(errorRes));
-        return;
-    }
-
-    try {
-        const result = handleRequest(req);
-        const res = { jsonrpc: "2.0", id: req.id, result };
-        console.log(JSON.stringify(res));
-
-        if (req.method === 'initialize') {
-            console.log(JSON.stringify({ jsonrpc: "2.0", method: "notifications/initialized", params: {} }));
-        }
-    } catch (e) {
-        const code = (e && typeof e.code === 'number') ? e.code : -32603;
-        const message = (e && e.message) ? e.message : "Internal server error";
-        const errorRes = {
-            jsonrpc: "2.0", id: req.id || null,
-            error: { code, message }
-        };
-        console.log(JSON.stringify(errorRes));
-    }
-});
+module.exports = require("../../bin/mcp-server.js");

package/dist/utils/version.js

@@ -7,7 +7,7 @@ exports.compareSemver = compareSemver;
 exports.fetchLatestVersion = fetchLatestVersion;
 exports.autoUpdateCheck = autoUpdateCheck;
 const https_1 = __importDefault(require("https"));
-const child_process_1 = require("child_process");
+const _child_process_1 = require("child_process");
 const fs_1 = __importDefault(require("fs"));
 const path_1 = __importDefault(require("path"));
 const os_1 = __importDefault(require("os"));

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "tribunal-kit",
-  "version": "5.8.3",
-  "description": "The operating system for AI software engineering — governance, memory, review pipelines, and reusable skills for every coding agent. 43 specialist agents, 34 workflows, 20 parallel Tribunal code reviewers, MCP server, Rust core engine, and long-running autonomous agent harness for Cursor, VSCode, Windsurf, Claude Code, and Aider.",
+  "version": "5.8.5",
+  "description": "The operating system for AI software engineering — governance, memory, review pipelines, and reusable skills for every coding agent. 44 specialist agents, 34 workflows, 19 parallel Tribunal code reviewers, MCP server, Rust core engine, and long-running autonomous agent harness for Cursor, VSCode, Windsurf, Claude Code, and Aider.",
   "keywords": [
     "ai",
     "ai-agent",
@@ -91,7 +91,9 @@
     "validate-payload": "node scripts/validate-payload.js",
     "benchmark": "node scripts/benchmark.js",
     "benchmark:rust": "cargo build --release && node scripts/benchmark.js",
-    "build": "echo 'No build step required for this project'"
+    "build:rust": "cargo build --release",
+    "test:rust": "cargo test --manifest-path crates/core/Cargo.toml",
+    "build": "npm run build:rust"
   },
   "devDependencies": {
     "eslint": "^9.1.1",
@@ -99,12 +101,12 @@
     "typescript": "^5.4.5"
   },
   "optionalDependencies": {
-    "@tribunal-kit/core-darwin-arm64": "^5.8.3",
-    "@tribunal-kit/core-darwin-x64": "^5.8.3",
-    "@tribunal-kit/core-linux-arm64": "^5.8.3",
-    "@tribunal-kit/core-linux-x64": "^5.8.3",
-    "@tribunal-kit/core-win32-arm64": "^5.8.3",
-    "@tribunal-kit/core-win32-x64": "^5.8.3"
+    "@tribunal-kit/core-darwin-arm64": "^5.8.5",
+    "@tribunal-kit/core-darwin-x64": "^5.8.5",
+    "@tribunal-kit/core-linux-arm64": "^5.8.5",
+    "@tribunal-kit/core-linux-x64": "^5.8.5",
+    "@tribunal-kit/core-win32-arm64": "^5.8.5",
+    "@tribunal-kit/core-win32-x64": "^5.8.5"
   },
   "jest": {
     "testMatch": [
@@ -113,7 +115,8 @@
     "testEnvironment": "node",
     "coverageDirectory": "coverage",
     "collectCoverageFrom": [
-      "bin/**/*.js"
+      "bin/**/*.js",
+      ".agent/scripts/**/*.js"
     ]
   }
 }

package/scripts/benchmark.js

@@ -142,6 +142,48 @@ async function main() {
   );
   results.push(initRealResult);
 
+  // 5. DAG scheduling benchmark
+  console.log(c("cyan", "  ▸ Benchmarking: DAG scheduling calculation"));
+  const dagResult = await benchmark(
+    "DAG wave scheduling",
+    () => {
+      const workers = [
+        { task_id: "w1", dependencies: [] },
+        { task_id: "w2", dependencies: ["w1"] },
+        { task_id: "w3", dependencies: ["w1"] },
+        { task_id: "w4", dependencies: ["w2", "w3"] },
+      ];
+      const inDegree = {};
+      const adjList = {};
+      workers.forEach((w) => {
+        inDegree[w.task_id] = 0;
+        adjList[w.task_id] = [];
+      });
+      workers.forEach((w) => {
+        w.dependencies.forEach((dep) => {
+          adjList[dep].push(w.task_id);
+          inDegree[w.task_id] += 1;
+        });
+      });
+      let currentWave = Object.keys(inDegree).filter((id) => inDegree[id] === 0);
+      const waves = [];
+      while (currentWave.length > 0) {
+        waves.push(currentWave);
+        const next = [];
+        currentWave.forEach((id) => {
+          adjList[id].forEach((nbr) => {
+            inDegree[nbr] -= 1;
+            if (inDegree[nbr] === 0) next.push(nbr);
+          });
+        });
+        currentWave = next;
+      }
+    },
+    100,
+  );
+  results.push(dagResult);
+
+
   // Print results table
   console.log();
   console.log(bold(`  Results`));

package/scripts/fix-vbc.js

@@ -0,0 +1,98 @@
+#!/usr/bin/env node
+/**
+ * fix-vbc.js — Batch appends missing VBC Protocol and Pre-Flight sections
+ * to all SKILL.md files in .agent/skills/ that are failing validation.
+ *
+ * Run: node scripts/fix-vbc.js
+ * Dry-run: node scripts/fix-vbc.js --dry-run
+ */
+
+const fs = require("fs");
+const path = require("path");
+
+const ROOT = path.resolve(__dirname, "..");
+const SKILLS_DIR = path.join(ROOT, ".agent", "skills");
+const DRY_RUN = process.argv.includes("--dry-run");
+
+const PRE_FLIGHT_BLOCK = `
+### ✅ Pre-Flight Self-Audit
+
+\`\`\`
+✅ Did I rely ONLY on real, verified tools and methods?
+✅ Is this solution appropriately scoped to the user's constraints?
+✅ Did I handle potential failure modes and edge cases?
+✅ Have I avoided generic boilerplate that doesn't add value?
+\`\`\`
+`;
+
+const VBC_BLOCK = `
+### 🛑 Verification-Before-Completion (VBC) Protocol
+
+**CRITICAL:** You must follow a strict "evidence-based closeout" state machine.
+
+- ❌ **Forbidden:** Declaring a task complete because the output "looks correct."
+- ✅ **Required:** You are explicitly forbidden from finalizing any task without providing **concrete evidence** (terminal output, passing tests, compile success, or equivalent proof) that your output works as intended.
+`;
+
+let fixedCount = 0;
+let skippedCount = 0;
+let errorCount = 0;
+
+const skillDirs = fs.readdirSync(SKILLS_DIR).filter((d) => {
+  return fs.statSync(path.join(SKILLS_DIR, d)).isDirectory();
+});
+
+for (const dir of skillDirs) {
+  const skillPath = path.join(SKILLS_DIR, dir, "SKILL.md");
+  if (!fs.existsSync(skillPath)) continue;
+
+  let content;
+  try {
+    content = fs.readFileSync(skillPath, "utf8");
+  } catch (e) {
+    console.error(`  ❌ Failed to read: ${dir}/SKILL.md — ${e.message}`);
+    errorCount++;
+    continue;
+  }
+
+  const hasPreFlight =
+    content.includes("Pre-Flight Checklist") || content.includes("Pre-Flight");
+  const hasVBC =
+    content.includes("VBC Protocol") || content.includes("VBC");
+
+  if (hasPreFlight && hasVBC) {
+    skippedCount++;
+    continue;
+  }
+
+  let appendText = "";
+
+  if (!hasPreFlight) {
+    appendText += PRE_FLIGHT_BLOCK;
+    console.log(`  🔧 ${dir}/SKILL.md — appending Pre-Flight Self-Audit`);
+  }
+
+  if (!hasVBC) {
+    appendText += VBC_BLOCK;
+    console.log(`  🔧 ${dir}/SKILL.md — appending VBC Protocol`);
+  }
+
+  if (DRY_RUN) {
+    console.log(`  [DRY-RUN] Would append to ${dir}/SKILL.md`);
+  } else {
+    try {
+      fs.appendFileSync(skillPath, appendText, "utf8");
+      fixedCount++;
+    } catch (e) {
+      console.error(`  ❌ Failed to write: ${dir}/SKILL.md — ${e.message}`);
+      errorCount++;
+    }
+  }
+}
+
+console.log(`\n━━━ VBC Fix Summary ━━━━━━━━━━━━━━━━━━━━━`);
+console.log(`  Fixed:   ${fixedCount}`);
+console.log(`  Skipped: ${skippedCount} (already compliant)`);
+console.log(`  Errors:  ${errorCount}`);
+console.log(`  Mode:    ${DRY_RUN ? "DRY-RUN (no changes)" : "LIVE"}`);
+console.log();

package/scripts/fix-vbc.ps1

@@ -0,0 +1,119 @@
+$ErrorActionPreference = "Stop"
+
+$skillsDir = "c:\Users\sunrise\Desktop\pfojects\cli project\tribunal-kit\.agent\skills"
+
+$vbcBlock = @"
+
+### `u{1F6D1} Verification-Before-Completion (VBC) Protocol
+
+**CRITICAL:** You must follow a strict "evidence-based closeout" state machine.
+
+- `u{274C} **Forbidden:** Declaring a task complete because the output "looks correct."
+- `u{2705} **Required:** You are explicitly forbidden from finalizing any task without providing **concrete evidence** (terminal output, passing tests, compile success, or equivalent proof) that your output works as intended.
+"@
+
+$preFlightBlock = @"
+
+### `u{2705} Pre-Flight Self-Audit
+
+``````
+`u{2705} Did I rely ONLY on real, verified tools and methods?
+`u{2705} Is this solution appropriately scoped to the user's constraints?
+`u{2705} Did I handle potential failure modes and edge cases?
+`u{2705} Have I avoided generic boilerplate that doesn't add value?
+``````
+"@
+
+$failingSkills = @(
+    "12-principles-of-animation",
+    "60fps-animation",
+    "accessible-animation",
+    "adapt",
+    "animation-on-scroll",
+    "animation-systems",
+    "antfu-conventions",
+    "audit-and-fix",
+    "bolder",
+    "build-primitive",
+    "clarify",
+    "cobejs",
+    "codebase-design",
+    "colorize",
+    "compact-landing",
+    "company-logos",
+    "critique",
+    "delight",
+    "design-lab",
+    "diagnosing-bugs",
+    "distill",
+    "domain-modeling",
+    "fixing-metadata",
+    "gpt-taste",
+    "harden",
+    "impeccable",
+    "improve-codebase-architecture",
+    "landing-page",
+    "local-first-architecture",
+    "lottie-animation",
+    "marquee-loop",
+    "masked-reveal",
+    "micro-interaction",
+    "morphing-icons",
+    "page-transition-animation",
+    "polish",
+    "pricing-page",
+    "progressive-blur",
+    "quieter",
+    "react-doctor",
+    "redesign-skill",
+    "shape",
+    "soft-skill",
+    "sounds-on-the-web",
+    "svg-animation",
+    "swiss-design",
+    "taste-skill",
+    "tdd-workflow",
+    "thermo-nuclear-code-quality-review",
+    "to-spring-or-not-to-spring",
+    "transitions-dev",
+    "typeset",
+    "web-quality-audit"
+)
+
+$fixedCount = 0
+$errorCount = 0
+
+foreach ($skill in $failingSkills) {
+    $filePath = Join-Path $skillsDir "$skill\SKILL.md"
+    
+    if (-not (Test-Path $filePath)) {
+        Write-Host "  SKIP: $skill/SKILL.md not found" -ForegroundColor Yellow
+        continue
+    }
+    
+    $content = Get-Content $filePath -Raw
+    $appendText = ""
+    
+    $hasPreFlight = ($content -match "Pre-Flight Checklist") -or ($content -match "Pre-Flight")
+    $hasVBC = ($content -match "VBC Protocol") -or ($content -match "VBC")
+    
+    if (-not $hasPreFlight) {
+        $appendText += $preFlightBlock
+        Write-Host "  FIX: $skill/SKILL.md - adding Pre-Flight" -ForegroundColor Cyan
+    }
+    
+    if (-not $hasVBC) {
+        $appendText += $vbcBlock
+        Write-Host "  FIX: $skill/SKILL.md - adding VBC Protocol" -ForegroundColor Cyan
+    }
+    
+    if ($appendText -ne "") {
+        Add-Content -Path $filePath -Value $appendText -NoNewline
+        $fixedCount++
+        Write-Host "  DONE: $skill/SKILL.md" -ForegroundColor Green
+    }
+}
+
+Write-Host ""
+Write-Host "Fixed: $fixedCount files" -ForegroundColor Green
+Write-Host "Errors: $errorCount" -ForegroundColor Red