tribunal-kit 5.8.3 → 5.8.5

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: 141 | Semantic: 113 | Procedural: 28 | Episodic: 0 | Working: 0
+> Entries: 221 | Semantic: 177 | Procedural: 44 | Episodic: 0 | Working: 0
 
 ## SEMANTIC (Permanent Facts)
 | ID | Content | Tags | Source | Created |
@@ -118,6 +118,70 @@
 | 137 | [ ] Have I checked the environment for relevant existing implementations? | project-idiom, auto-learned | manual | 1784396209Z |
 | 138 | ❌ **Forbidden:** Declaring a task complete because the output "looks correct." | project-idiom, auto-learned | manual | 1784396209Z |
 | 139 | ✅ **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 | 1784396209Z |
+| 142 | ❌ Overriding project idioms without explicit justification -> ✅ Idioms represent team decisions; flag deviations with reasoning | project-idiom, auto-learned | manual | 1784618632Z |
+| 143 | ❌ 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 | 1784618632Z |
+| 144 | ❌ **Forbidden:** Declaring a task complete because the output "looks correct." | project-idiom, auto-learned | manual | 1784618632Z |
+| 145 | ✅ **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 | 1784618632Z |
+| 146 | [ ] Have I reviewed the user's specific constraints and requests? | project-idiom, auto-learned | manual | 1784618632Z |
+| 147 | [ ] Have I checked the environment for relevant existing implementations? | project-idiom, auto-learned | manual | 1784618632Z |
+| 148 | ❌ **Forbidden:** Declaring a task complete because the output "looks correct." | project-idiom, auto-learned | manual | 1784618632Z |
+| 149 | ✅ **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 | 1784618632Z |
+| 152 | ❌ Overriding project idioms without explicit justification -> ✅ Idioms represent team decisions; flag deviations with reasoning | project-idiom, auto-learned | manual | 1784705972Z |
+| 153 | ❌ 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 | 1784705972Z |
+| 154 | ❌ **Forbidden:** Declaring a task complete because the output "looks correct." | project-idiom, auto-learned | manual | 1784705972Z |
+| 155 | ✅ **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 | 1784705972Z |
+| 156 | [ ] Have I reviewed the user's specific constraints and requests? | project-idiom, auto-learned | manual | 1784705972Z |
+| 157 | [ ] Have I checked the environment for relevant existing implementations? | project-idiom, auto-learned | manual | 1784705972Z |
+| 158 | ❌ **Forbidden:** Declaring a task complete because the output "looks correct." | project-idiom, auto-learned | manual | 1784705972Z |
+| 159 | ✅ **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 | 1784705972Z |
+| 162 | ❌ Overriding project idioms without explicit justification -> ✅ Idioms represent team decisions; flag deviations with reasoning | project-idiom, auto-learned | manual | 1784706426Z |
+| 163 | ❌ 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 | 1784706426Z |
+| 164 | ❌ **Forbidden:** Declaring a task complete because the output "looks correct." | project-idiom, auto-learned | manual | 1784706426Z |
+| 165 | ✅ **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 | 1784706426Z |
+| 166 | [ ] Have I reviewed the user's specific constraints and requests? | project-idiom, auto-learned | manual | 1784706426Z |
+| 167 | [ ] Have I checked the environment for relevant existing implementations? | project-idiom, auto-learned | manual | 1784706426Z |
+| 168 | ❌ **Forbidden:** Declaring a task complete because the output "looks correct." | project-idiom, auto-learned | manual | 1784706426Z |
+| 169 | ✅ **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 | 1784706426Z |
+| 172 | ❌ Overriding project idioms without explicit justification -> ✅ Idioms represent team decisions; flag deviations with reasoning | project-idiom, auto-learned | manual | 1784706433Z |
+| 173 | ❌ 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 | 1784706433Z |
+| 174 | ❌ **Forbidden:** Declaring a task complete because the output "looks correct." | project-idiom, auto-learned | manual | 1784706433Z |
+| 175 | ✅ **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 | 1784706433Z |
+| 176 | [ ] Have I reviewed the user's specific constraints and requests? | project-idiom, auto-learned | manual | 1784706433Z |
+| 177 | [ ] Have I checked the environment for relevant existing implementations? | project-idiom, auto-learned | manual | 1784706433Z |
+| 178 | ❌ **Forbidden:** Declaring a task complete because the output "looks correct." | project-idiom, auto-learned | manual | 1784706433Z |
+| 179 | ✅ **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 | 1784706433Z |
+| 182 | ❌ Overriding project idioms without explicit justification -> ✅ Idioms represent team decisions; flag deviations with reasoning | project-idiom, auto-learned | manual | 1784706809Z |
+| 183 | ❌ 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 | 1784706809Z |
+| 184 | ❌ **Forbidden:** Declaring a task complete because the output "looks correct." | project-idiom, auto-learned | manual | 1784706809Z |
+| 185 | ✅ **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 | 1784706809Z |
+| 186 | [ ] Have I reviewed the user's specific constraints and requests? | project-idiom, auto-learned | manual | 1784706809Z |
+| 187 | [ ] Have I checked the environment for relevant existing implementations? | project-idiom, auto-learned | manual | 1784706809Z |
+| 188 | ❌ **Forbidden:** Declaring a task complete because the output "looks correct." | project-idiom, auto-learned | manual | 1784706809Z |
+| 189 | ✅ **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 | 1784706809Z |
+| 192 | ❌ Overriding project idioms without explicit justification -> ✅ Idioms represent team decisions; flag deviations with reasoning | project-idiom, auto-learned | manual | 1784706814Z |
+| 193 | ❌ 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 | 1784706814Z |
+| 194 | ❌ **Forbidden:** Declaring a task complete because the output "looks correct." | project-idiom, auto-learned | manual | 1784706814Z |
+| 195 | ✅ **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 | 1784706814Z |
+| 196 | [ ] Have I reviewed the user's specific constraints and requests? | project-idiom, auto-learned | manual | 1784706814Z |
+| 197 | [ ] Have I checked the environment for relevant existing implementations? | project-idiom, auto-learned | manual | 1784706814Z |
+| 198 | ❌ **Forbidden:** Declaring a task complete because the output "looks correct." | project-idiom, auto-learned | manual | 1784706814Z |
+| 199 | ✅ **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 | 1784706814Z |
+| 202 | ❌ Overriding project idioms without explicit justification -> ✅ Idioms represent team decisions; flag deviations with reasoning | project-idiom, auto-learned | manual | 1785138690Z |
+| 203 | ❌ 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 | 1785138690Z |
+| 204 | ❌ **Forbidden:** Declaring a task complete because the output "looks correct." | project-idiom, auto-learned | manual | 1785138690Z |
+| 205 | ✅ **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 | 1785138690Z |
+| 206 | [ ] Have I reviewed the user's specific constraints and requests? | project-idiom, auto-learned | manual | 1785138690Z |
+| 207 | [ ] Have I checked the environment for relevant existing implementations? | project-idiom, auto-learned | manual | 1785138690Z |
+| 208 | ❌ **Forbidden:** Declaring a task complete because the output "looks correct." | project-idiom, auto-learned | manual | 1785138690Z |
+| 209 | ✅ **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 | 1785138690Z |
+| 212 | ❌ Overriding project idioms without explicit justification -> ✅ Idioms represent team decisions; flag deviations with reasoning | project-idiom, auto-learned | manual | 1785138695Z |
+| 213 | ❌ 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 | 1785138695Z |
+| 214 | ❌ **Forbidden:** Declaring a task complete because the output "looks correct." | project-idiom, auto-learned | manual | 1785138695Z |
+| 215 | ✅ **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 | 1785138695Z |
+| 216 | [ ] Have I reviewed the user's specific constraints and requests? | project-idiom, auto-learned | manual | 1785138695Z |
+| 217 | [ ] Have I checked the environment for relevant existing implementations? | project-idiom, auto-learned | manual | 1785138695Z |
+| 218 | ❌ **Forbidden:** Declaring a task complete because the output "looks correct." | project-idiom, auto-learned | manual | 1785138695Z |
+| 219 | ✅ **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 | 1785138696Z |
 
 ## PROCEDURAL (How-To Recipes)
 | ID | Content | Tags | Source | Created |
@@ -150,4 +214,20 @@
 | 131 | Run `npm run validate-payload && jest --coverage` to test the project | build-script, test, auto-learned | manual | 1784187808Z |
 | 140 | Run `echo 'No build step required for this project'` to build the project | build-script, build, auto-learned | manual | 1784396209Z |
 | 141 | Run `npm run validate-payload && jest --coverage` to test the project | build-script, test, auto-learned | manual | 1784396209Z |
+| 150 | Run `echo 'No build step required for this project'` to build the project | build-script, build, auto-learned | manual | 1784618632Z |
+| 151 | Run `npm run validate-payload && jest --coverage` to test the project | build-script, test, auto-learned | manual | 1784618632Z |
+| 160 | Run `echo 'No build step required for this project'` to build the project | build-script, build, auto-learned | manual | 1784705972Z |
+| 161 | Run `npm run validate-payload && jest --coverage` to test the project | build-script, test, auto-learned | manual | 1784705972Z |
+| 170 | Run `echo 'No build step required for this project'` to build the project | build-script, build, auto-learned | manual | 1784706426Z |
+| 171 | Run `npm run validate-payload && jest --coverage` to test the project | build-script, test, auto-learned | manual | 1784706426Z |
+| 180 | Run `echo 'No build step required for this project'` to build the project | build-script, build, auto-learned | manual | 1784706433Z |
+| 181 | Run `npm run validate-payload && jest --coverage` to test the project | build-script, test, auto-learned | manual | 1784706433Z |
+| 190 | Run `echo 'No build step required for this project'` to build the project | build-script, build, auto-learned | manual | 1784706809Z |
+| 191 | Run `npm run validate-payload && jest --coverage` to test the project | build-script, test, auto-learned | manual | 1784706809Z |
+| 200 | Run `echo 'No build step required for this project'` to build the project | build-script, build, auto-learned | manual | 1784706814Z |
+| 201 | Run `npm run validate-payload && jest --coverage` to test the project | build-script, test, auto-learned | manual | 1784706814Z |
+| 210 | Run `npm run build:rust` to build the project | build-script, build, auto-learned | manual | 1785138690Z |
+| 211 | Run `npm run validate-payload && jest --coverage` to test the project | build-script, test, auto-learned | manual | 1785138690Z |
+| 220 | Run `npm run build:rust` to build the project | build-script, build, auto-learned | manual | 1785138696Z |
+| 221 | Run `npm run validate-payload && jest --coverage` to test the project | build-script, test, auto-learned | manual | 1785138696Z |
 

package/.agent/rules/GEMINI.md

@@ -222,7 +222,7 @@ The Human Gate is never skipped. No code is written to a file without explicit u
 | Database/SQL          | logic + security + sql + schema + complexity-reviewer                                               |
 | Mobile/Cross-platform | logic + security + mobile-reviewer + type-safety + complexity-reviewer                              |
 | Any domain            | + performance (if optimization)                                                                     |
-| Before merge          | /tribunal-full (all 20)                                                                             |
+| Before merge          | /tribunal-full (all 19 reviewers)                                                                   |
 
 ---
 

package/.agent/scripts/_utils.js

@@ -226,9 +226,34 @@ function parseArgs(argv, schema = {}) {
 
 // ── Command Runner ──────────────────────────────────────────────────────────
 
+const WINDOWS_CMD_SHIMS = new Set(["npm", "npx", "pnpm", "yarn", "bun", "bunx"]);
+
+/**
+ * Select the executable that should be passed to spawnSync.
+ * Only package-manager shims need a .cmd suffix on Windows; native executables
+ * such as node, cargo, and git must retain their original names.
+ *
+ * @param {string} cmd - Command to run
+ * @param {string} [platform] - Platform override for deterministic testing
+ * @returns {string}
+ */
+function normalizeCommand(cmd, platform = process.platform) {
+  const lowerCaseCommand = cmd.toLowerCase();
+  const hasPath = cmd.includes("/") || cmd.includes("\\\\");
+  if (
+    platform === "win32" &&
+    !hasPath &&
+    !lowerCaseCommand.endsWith(".cmd") &&
+    WINDOWS_CMD_SHIMS.has(lowerCaseCommand)
+  ) {
+    return `${cmd}.cmd`;
+  }
+  return cmd;
+}
+
 /**
  * Run a command synchronously and return a structured result.
- * Handles Windows .cmd shims automatically.
+ * Handles known Windows package-manager shims without rewriting native tools.
  *
  * @param {string} cmd - Command to run
  * @param {string[]} args - Arguments
@@ -237,12 +262,7 @@ function parseArgs(argv, schema = {}) {
  */
 function runCommand(cmd, args = [], opts = {}) {
   const { spawnSync } = require("child_process");
-  const executable =
-    process.platform === "win32" &&
-    !cmd.endsWith(".cmd") &&
-    !cmd.includes(path.sep)
-      ? `${cmd}.cmd`
-      : cmd;
+  const executable = normalizeCommand(cmd);
 
   const result = spawnSync(executable, args, {
     encoding: "utf8",
@@ -275,5 +295,6 @@ module.exports = {
   // CLI
   parseArgs,
   // Commands
+  normalizeCommand,
   runCommand,
 };

package/.agent/scripts/case_law_manager.js

@@ -28,21 +28,9 @@ const readline = require("readline");
 // ── Colours ──────────────────────────────────────────────────────────────────
 const { GREEN, YELLOW, CYAN, RED, BOLD, DIM, RESET } = require("./_colors");
 
-// ── Find .agent directory ─────────────────────────────────────────────────────
-function findAgentDir() {
-  let current = path.resolve(process.cwd());
-  const root = path.parse(current).root;
-  while (current !== root) {
-    const candidate = path.join(current, ".agent");
-    if (fs.existsSync(candidate) && fs.statSync(candidate).isDirectory())
-      return candidate;
-    current = path.dirname(current);
-  }
-  console.error(
-    `${RED}✖ Error: '.agent' directory not found. Please run 'npx tribunal-kit init' first.${RESET}`,
-  );
-  process.exit(1);
-}
+// ── Shared Utilities ──────────────────────────────────────────────────────────
+const { findAgentDir } = require("./_utils");
+
 
 // ── Lazy path resolution (avoids side effects at require-time) ───────────────
 let _paths = null;

package/.agent/scripts/context_broker.js

@@ -50,8 +50,10 @@
 const fs = require("fs");
 const path = require("path");
 
-// ── Colours ───────────────────────────────────────────────────────────────────
+// ── Colours & Shared Utilities ────────────────────────────────────────────────
 const { GREEN, YELLOW, CYAN, RED, BOLD, DIM, RESET } = require("./_colors");
+const { findAgentDir } = require("./_utils");
+
 
 // ── Domain → Skill affinity map ───────────────────────────────────────────────
 // Keywords in the user's task that strongly indicate specific skills.
@@ -353,25 +355,6 @@ const BASELINE_SKILLS = [
 
 // ── Skill catalogue (loaded from disk) ───────────────────────────────────────
 
-/**
- * Find the .agent directory by walking up from cwd.
- * @returns {string} path to .agent/
- */
-function findAgentDir() {
-  let current = path.resolve(process.cwd());
-  const root = path.parse(current).root;
-  while (current !== root) {
-    const candidate = path.join(current, ".agent");
-    if (fs.existsSync(candidate) && fs.statSync(candidate).isDirectory())
-      return candidate;
-    current = path.dirname(current);
-  }
-  console.error(
-    `${RED}✖ .agent/ not found. Run: npx tribunal-kit init${RESET}`,
-  );
-  process.exit(1);
-}
-
 /**
  * Parse the YAML frontmatter from a SKILL.md file.
  * Returns { name, description, ... } or null on parse failure.