tribunal-kit 5.8.1 → 5.8.2

package/.agent/history/memory/.memory.idx

@@ -1687,7 +1687,311 @@
       "token_estimate": 18,
       "source": "manual",
       "session_id": null
+    },
+    {
+      "id": 112,
+      "memory_type": "semantic",
+      "content": "❌ Overriding project idioms without explicit justification -> ✅ Idioms represent team decisions; flag deviations with reasoning",
+      "tags": [
+        "project-idiom",
+        "auto-learned"
+      ],
+      "created_at": "1783790009Z",
+      "last_accessed": "1783790009Z",
+      "access_count": 0,
+      "token_estimate": 32,
+      "source": "manual",
+      "session_id": null
+    },
+    {
+      "id": 113,
+      "memory_type": "semantic",
+      "content": "❌ Applying idioms from one project to a different project -> ✅ Idioms are project-specific; verify they apply to the current codebase",
+      "tags": [
+        "project-idiom",
+        "auto-learned"
+      ],
+      "created_at": "1783790009Z",
+      "last_accessed": "1783790009Z",
+      "access_count": 0,
+      "token_estimate": 34,
+      "source": "manual",
+      "session_id": null
+    },
+    {
+      "id": 114,
+      "memory_type": "semantic",
+      "content": "❌ **Forbidden:** Declaring a task complete because the output \"looks correct.\"",
+      "tags": [
+        "project-idiom",
+        "auto-learned"
+      ],
+      "created_at": "1783790009Z",
+      "last_accessed": "1783790009Z",
+      "access_count": 0,
+      "token_estimate": 20,
+      "source": "manual",
+      "session_id": null
+    },
+    {
+      "id": 115,
+      "memory_type": "semantic",
+      "content": "✅ **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.",
+      "tags": [
+        "project-idiom",
+        "auto-learned"
+      ],
+      "created_at": "1783790009Z",
+      "last_accessed": "1783790009Z",
+      "access_count": 0,
+      "token_estimate": 54,
+      "source": "manual",
+      "session_id": null
+    },
+    {
+      "id": 116,
+      "memory_type": "semantic",
+      "content": "[ ] Have I reviewed the user's specific constraints and requests?",
+      "tags": [
+        "project-idiom",
+        "auto-learned"
+      ],
+      "created_at": "1783790009Z",
+      "last_accessed": "1783790009Z",
+      "access_count": 0,
+      "token_estimate": 17,
+      "source": "manual",
+      "session_id": null
+    },
+    {
+      "id": 117,
+      "memory_type": "semantic",
+      "content": "[ ] Have I checked the environment for relevant existing implementations?",
+      "tags": [
+        "project-idiom",
+        "auto-learned"
+      ],
+      "created_at": "1783790009Z",
+      "last_accessed": "1783790009Z",
+      "access_count": 0,
+      "token_estimate": 19,
+      "source": "manual",
+      "session_id": null
+    },
+    {
+      "id": 118,
+      "memory_type": "semantic",
+      "content": "❌ **Forbidden:** Declaring a task complete because the output \"looks correct.\"",
+      "tags": [
+        "project-idiom",
+        "auto-learned"
+      ],
+      "created_at": "1783790009Z",
+      "last_accessed": "1783790009Z",
+      "access_count": 0,
+      "token_estimate": 20,
+      "source": "manual",
+      "session_id": null
+    },
+    {
+      "id": 119,
+      "memory_type": "semantic",
+      "content": "✅ **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.",
+      "tags": [
+        "project-idiom",
+        "auto-learned"
+      ],
+      "created_at": "1783790009Z",
+      "last_accessed": "1783790009Z",
+      "access_count": 0,
+      "token_estimate": 54,
+      "source": "manual",
+      "session_id": null
+    },
+    {
+      "id": 120,
+      "memory_type": "procedural",
+      "content": "Run `echo 'No build step required for this project'` to build the project",
+      "tags": [
+        "build-script",
+        "build",
+        "auto-learned"
+      ],
+      "created_at": "1783790009Z",
+      "last_accessed": "1783790009Z",
+      "access_count": 0,
+      "token_estimate": 19,
+      "source": "manual",
+      "session_id": null
+    },
+    {
+      "id": 121,
+      "memory_type": "procedural",
+      "content": "Run `npm run validate-payload && jest --coverage` to test the project",
+      "tags": [
+        "build-script",
+        "test",
+        "auto-learned"
+      ],
+      "created_at": "1783790009Z",
+      "last_accessed": "1783790009Z",
+      "access_count": 0,
+      "token_estimate": 18,
+      "source": "manual",
+      "session_id": null
+    },
+    {
+      "id": 122,
+      "memory_type": "semantic",
+      "content": "❌ Overriding project idioms without explicit justification -> ✅ Idioms represent team decisions; flag deviations with reasoning",
+      "tags": [
+        "project-idiom",
+        "auto-learned"
+      ],
+      "created_at": "1784187808Z",
+      "last_accessed": "1784187808Z",
+      "access_count": 0,
+      "token_estimate": 32,
+      "source": "manual",
+      "session_id": null
+    },
+    {
+      "id": 123,
+      "memory_type": "semantic",
+      "content": "❌ Applying idioms from one project to a different project -> ✅ Idioms are project-specific; verify they apply to the current codebase",
+      "tags": [
+        "project-idiom",
+        "auto-learned"
+      ],
+      "created_at": "1784187808Z",
+      "last_accessed": "1784187808Z",
+      "access_count": 0,
+      "token_estimate": 34,
+      "source": "manual",
+      "session_id": null
+    },
+    {
+      "id": 124,
+      "memory_type": "semantic",
+      "content": "❌ **Forbidden:** Declaring a task complete because the output \"looks correct.\"",
+      "tags": [
+        "project-idiom",
+        "auto-learned"
+      ],
+      "created_at": "1784187808Z",
+      "last_accessed": "1784187808Z",
+      "access_count": 0,
+      "token_estimate": 20,
+      "source": "manual",
+      "session_id": null
+    },
+    {
+      "id": 125,
+      "memory_type": "semantic",
+      "content": "✅ **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.",
+      "tags": [
+        "project-idiom",
+        "auto-learned"
+      ],
+      "created_at": "1784187808Z",
+      "last_accessed": "1784187808Z",
+      "access_count": 0,
+      "token_estimate": 54,
+      "source": "manual",
+      "session_id": null
+    },
+    {
+      "id": 126,
+      "memory_type": "semantic",
+      "content": "[ ] Have I reviewed the user's specific constraints and requests?",
+      "tags": [
+        "project-idiom",
+        "auto-learned"
+      ],
+      "created_at": "1784187808Z",
+      "last_accessed": "1784187808Z",
+      "access_count": 0,
+      "token_estimate": 17,
+      "source": "manual",
+      "session_id": null
+    },
+    {
+      "id": 127,
+      "memory_type": "semantic",
+      "content": "[ ] Have I checked the environment for relevant existing implementations?",
+      "tags": [
+        "project-idiom",
+        "auto-learned"
+      ],
+      "created_at": "1784187808Z",
+      "last_accessed": "1784187808Z",
+      "access_count": 0,
+      "token_estimate": 19,
+      "source": "manual",
+      "session_id": null
+    },
+    {
+      "id": 128,
+      "memory_type": "semantic",
+      "content": "❌ **Forbidden:** Declaring a task complete because the output \"looks correct.\"",
+      "tags": [
+        "project-idiom",
+        "auto-learned"
+      ],
+      "created_at": "1784187808Z",
+      "last_accessed": "1784187808Z",
+      "access_count": 0,
+      "token_estimate": 20,
+      "source": "manual",
+      "session_id": null
+    },
+    {
+      "id": 129,
+      "memory_type": "semantic",
+      "content": "✅ **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.",
+      "tags": [
+        "project-idiom",
+        "auto-learned"
+      ],
+      "created_at": "1784187808Z",
+      "last_accessed": "1784187808Z",
+      "access_count": 0,
+      "token_estimate": 54,
+      "source": "manual",
+      "session_id": null
+    },
+    {
+      "id": 130,
+      "memory_type": "procedural",
+      "content": "Run `echo 'No build step required for this project'` to build the project",
+      "tags": [
+        "build-script",
+        "build",
+        "auto-learned"
+      ],
+      "created_at": "1784187808Z",
+      "last_accessed": "1784187808Z",
+      "access_count": 0,
+      "token_estimate": 19,
+      "source": "manual",
+      "session_id": null
+    },
+    {
+      "id": 131,
+      "memory_type": "procedural",
+      "content": "Run `npm run validate-payload && jest --coverage` to test the project",
+      "tags": [
+        "build-script",
+        "test",
+        "auto-learned"
+      ],
+      "created_at": "1784187808Z",
+      "last_accessed": "1784187808Z",
+      "access_count": 0,
+      "token_estimate": 18,
+      "source": "manual",
+      "session_id": null
     }
   ],
-  "next_id": 112
+  "next_id": 132
 }

package/.agent/history/memory/MEMORY.md

@@ -1,6 +1,6 @@
 # 🧠 Tribunal Memory Index
 > Auto-generated by `tribunal-kit memory export`. Do not edit manually.
-> Entries: 111 | Semantic: 89 | Procedural: 22 | Episodic: 0 | Working: 0
+> Entries: 131 | Semantic: 105 | Procedural: 26 | Episodic: 0 | Working: 0
 
 ## SEMANTIC (Permanent Facts)
 | ID | Content | Tags | Source | Created |
@@ -94,6 +94,22 @@
 | 107 | [ ] Have I checked the environment for relevant existing implementations? | project-idiom, auto-learned | manual | 1783785670Z |
 | 108 | ❌ **Forbidden:** Declaring a task complete because the output "looks correct." | project-idiom, auto-learned | manual | 1783785670Z |
 | 109 | ✅ **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. | project-idiom, auto-learned | manual | 1783785670Z |
+| 112 | ❌ Overriding project idioms without explicit justification -> ✅ Idioms represent team decisions; flag deviations with reasoning | project-idiom, auto-learned | manual | 1783790009Z |
+| 113 | ❌ Applying idioms from one project to a different project -> ✅ Idioms are project-specific; verify they apply to the current codebase | project-idiom, auto-learned | manual | 1783790009Z |
+| 114 | ❌ **Forbidden:** Declaring a task complete because the output "looks correct." | project-idiom, auto-learned | manual | 1783790009Z |
+| 115 | ✅ **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. | project-idiom, auto-learned | manual | 1783790009Z |
+| 116 | [ ] Have I reviewed the user's specific constraints and requests? | project-idiom, auto-learned | manual | 1783790009Z |
+| 117 | [ ] Have I checked the environment for relevant existing implementations? | project-idiom, auto-learned | manual | 1783790009Z |
+| 118 | ❌ **Forbidden:** Declaring a task complete because the output "looks correct." | project-idiom, auto-learned | manual | 1783790009Z |
+| 119 | ✅ **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. | project-idiom, auto-learned | manual | 1783790009Z |
+| 122 | ❌ Overriding project idioms without explicit justification -> ✅ Idioms represent team decisions; flag deviations with reasoning | project-idiom, auto-learned | manual | 1784187808Z |
+| 123 | ❌ Applying idioms from one project to a different project -> ✅ Idioms are project-specific; verify they apply to the current codebase | project-idiom, auto-learned | manual | 1784187808Z |
+| 124 | ❌ **Forbidden:** Declaring a task complete because the output "looks correct." | project-idiom, auto-learned | manual | 1784187808Z |
+| 125 | ✅ **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. | project-idiom, auto-learned | manual | 1784187808Z |
+| 126 | [ ] Have I reviewed the user's specific constraints and requests? | project-idiom, auto-learned | manual | 1784187808Z |
+| 127 | [ ] Have I checked the environment for relevant existing implementations? | project-idiom, auto-learned | manual | 1784187808Z |
+| 128 | ❌ **Forbidden:** Declaring a task complete because the output "looks correct." | project-idiom, auto-learned | manual | 1784187808Z |
+| 129 | ✅ **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. | project-idiom, auto-learned | manual | 1784187808Z |
 
 ## PROCEDURAL (How-To Recipes)
 | ID | Content | Tags | Source | Created |
@@ -120,4 +136,8 @@
 | 100 | Run `npm run validate-payload && jest --coverage` to test the project | build-script, test, auto-learned | manual | 1783782132Z |
 | 110 | Run `echo 'No build step required for this project'` to build the project | build-script, build, auto-learned | manual | 1783785670Z |
 | 111 | Run `npm run validate-payload && jest --coverage` to test the project | build-script, test, auto-learned | manual | 1783785670Z |
+| 120 | Run `echo 'No build step required for this project'` to build the project | build-script, build, auto-learned | manual | 1783790009Z |
+| 121 | Run `npm run validate-payload && jest --coverage` to test the project | build-script, test, auto-learned | manual | 1783790009Z |
+| 130 | Run `echo 'No build step required for this project'` to build the project | build-script, build, auto-learned | manual | 1784187808Z |
+| 131 | Run `npm run validate-payload && jest --coverage` to test the project | build-script, test, auto-learned | manual | 1784187808Z |
 

package/.agent/scripts/signal_detector.js

@@ -0,0 +1,173 @@
+/**
+ * signal_detector.js — Tribunal Kit Signal Detector
+ * ====================================================
+ * Parses runtime log files, test logs, and build transcripts to extract
+ * structured "Improvement Signals" (errors, bottlenecks, gaps).
+ *
+ * Supported log sources/patterns:
+ *   - JS/TS Node Stack Traces
+ *   - Python Stack Traces
+ *   - Rust Compiler Errors
+ *   - Jest/Vitest Test Failures
+ *   - ESLint Warnings/Errors
+ *   - Performance measurements (e.g. "Slow query", "duration: 1200ms")
+ */
+
+"use strict";
+
+const fs = require("fs");
+const path = require("path");
+
+const SIGNAL_TYPES = {
+  ERROR: "log_error",
+  PERF: "perf_bottleneck",
+  GAP: "capability_gap",
+};
+
+/**
+ * Parses a string log and extracts list of typed signals.
+ * @param {string} logText
+ * @returns {Array<{type: string, file: string|null, line: number|null, message: string, context: string}>}
+ */
+function detectSignals(logText) {
+  const signals = [];
+  const lines = logText.split(/\r?\n/);
+
+  // 1. Check for stack traces and syntax/runtime errors
+  // JS/TS stack trace line: "    at Object.foo (src/utils.js:12:45)" or "    at src/utils.js:12:45"
+  const jsStackRegex = /\bat\s+(?:async\s+)?(?:[^(]+\()?([^:()\s]+):(\d+):(\d+)\)?/;
+  // Python File location: '  File "src/main.py", line 12, in <module>'
+  const pyStackRegex = /File\s+"([^"]+)"\s*,\s*line\s*(\d+)/;
+  // Rust location: '  --> src/main.rs:12:34'
+  const rustRegex = /-->\s*([^:\s]+):(\d+):(\d+)/;
+  // ESLint standard line: '  /path/to/file.js:12:34: error: Message' or '  12:34  error  Message'
+  // But ESLint often has file heading first. So let's handle inline: 'file.js: line 12, col 34, Error - Message'
+  const eslintInlineRegex = /([^:\s]+):\s*line\s*(\d+)\s*,\s*col\s*\d+,\s*(?:Error|Warning)\s*-\s*(.+)/i;
+  // Also standard unix format: '/path/to/file.js:12:34: error message'
+  const unixErrorRegex = /^([^:\s\(\)]+):(\d+):(?:\d+:)?\s*(error|warning|info):?\s*(.+)/i;
+
+  // Let's also scan for performance bottlenecks
+  const slowQueryRegex = /slow\s+query|duration:\s*(\d+)ms|execution\s+time:\s*(\d+)ms|response\s+time\s*>\s*(\d+)ms/i;
+
+  let currentFile = null;
+
+  for (let i = 0; i < lines.length; i++) {
+    const line = lines[i];
+
+    // Detect Jest/Vitest file heading: "FAIL  src/utils.test.js"
+    if (line.startsWith("FAIL ")) {
+      const match = line.match(/^FAIL\s+(\S+)/);
+      if (match) {
+        currentFile = match[1];
+        signals.push({
+          type: SIGNAL_TYPES.ERROR,
+          file: currentFile,
+          line: null,
+          message: `Test suite failed: ${path.basename(currentFile)}`,
+          context: lines.slice(Math.max(0, i - 1), Math.min(lines.length, i + 5)).join("\n"),
+        });
+      }
+      continue;
+    }
+
+    // JS Stack Trace Match
+    const jsMatch = line.match(jsStackRegex);
+    if (jsMatch && !line.includes("node_modules")) {
+      const filePath = jsMatch[1].replace(/\\/g, "/");
+      signals.push({
+        type: SIGNAL_TYPES.ERROR,
+        file: filePath,
+        line: parseInt(jsMatch[2], 10),
+        message: `Stack Trace Error: ${line.trim()}`,
+        context: lines.slice(Math.max(0, i - 2), Math.min(lines.length, i + 3)).join("\n"),
+      });
+      continue;
+    }
+
+    // Python Stack Trace Match
+    const pyMatch = line.match(pyStackRegex);
+    if (pyMatch) {
+      const filePath = pyMatch[1].replace(/\\/g, "/");
+      signals.push({
+        type: SIGNAL_TYPES.ERROR,
+        file: filePath,
+        line: parseInt(pyMatch[2], 10),
+        message: `Python Exception: ${lines[i+1] ? lines[i+1].trim() : line.trim()}`,
+        context: lines.slice(Math.max(0, i - 1), Math.min(lines.length, i + 4)).join("\n"),
+      });
+      continue;
+    }
+
+    // Rust Compiler Error Match
+    const rustMatch = line.match(rustRegex);
+    if (rustMatch) {
+      const filePath = rustMatch[1].replace(/\\/g, "/");
+      signals.push({
+        type: SIGNAL_TYPES.ERROR,
+        file: filePath,
+        line: parseInt(rustMatch[2], 10),
+        message: `Rust compilation failure: ${lines[Math.max(0, i - 1)].trim()}`,
+        context: lines.slice(Math.max(0, i - 2), Math.min(lines.length, i + 5)).join("\n"),
+      });
+      continue;
+    }
+
+    // ESLint Inline Match
+    const eslintMatch = line.match(eslintInlineRegex);
+    if (eslintMatch) {
+      signals.push({
+        type: SIGNAL_TYPES.ERROR,
+        file: eslintMatch[1].replace(/\\/g, "/"),
+        line: parseInt(eslintMatch[2], 10),
+        message: `Linter Violation: ${eslintMatch[3]}`,
+        context: line.trim(),
+      });
+      continue;
+    }
+
+    // Unix Error Format Match
+    const unixMatch = line.match(unixErrorRegex);
+    if (unixMatch && !line.includes("node_modules")) {
+      const filePath = unixMatch[1].replace(/\\/g, "/");
+      signals.push({
+        type: SIGNAL_TYPES.ERROR,
+        file: filePath,
+        line: parseInt(unixMatch[2], 10),
+        message: `${unixMatch[3].toUpperCase()}: ${unixMatch[4]}`,
+        context: line.trim(),
+      });
+      continue;
+    }
+
+    // Slow Query/Performance Match
+    const slowMatch = line.match(slowQueryRegex);
+    if (slowMatch) {
+      signals.push({
+        type: SIGNAL_TYPES.PERF,
+        file: currentFile || null,
+        line: null,
+        message: `Performance warning: ${line.trim()}`,
+        context: lines.slice(Math.max(0, i - 1), Math.min(lines.length, i + 2)).join("\n"),
+      });
+      continue;
+    }
+  }
+
+  // Deduplicate near-identical signals (same file, line, and message type)
+  const unique = [];
+  const keys = new Set();
+  for (const s of signals) {
+    const key = `${s.type}:${s.file}:${s.line}:${s.message.substring(0, 40)}`;
+    if (!keys.has(key)) {
+      keys.add(key);
+      unique.push(s);
+    }
+  }
+
+  return unique;
+}
+
+module.exports = {
+  detectSignals,
+  SIGNAL_TYPES,
+};