tribunal-kit 4.4.0 → 4.4.2

package/.agent/scripts/test_runner.js

@@ -15,25 +15,16 @@ const fs = require('fs');
 const path = require('path');
 const { spawnSync } = require('child_process');
 
-const { RED, GREEN, YELLOW, BLUE, BOLD, RESET } = require('./colors.js');
+const {
+    RED, GREEN, YELLOW, BLUE, BOLD, DIM, CYAN, RESET,
+    banner, sectionHeader, timer, formatMs,
+    ok, fail, skip,
+} = require('./_colors');
 
-function header(title) {
-    console.log(`\n${BOLD}${BLUE}━━━ ${title} ━━━${RESET}`);
-}
-
-function ok(msg) {
-    console.log(`  ${GREEN}✅ ${msg}${RESET}`);
-}
-
-function fail(msg) {
-    console.log(`  ${RED}❌ ${msg}${RESET}`);
-}
-
-function skip(msg) {
-    console.log(`  ${YELLOW}⏭️  ${msg}${RESET}`);
-}
+const { loadJson } = require('./_utils');
 
 function runTests(label, cmd, cwd) {
+    const elapsed = timer();
     try {
         const executable = process.platform === 'win32' && (cmd[0] === 'npx' || cmd[0] === 'npm') ? `${cmd[0]}.cmd` : cmd[0];
         const result = spawnSync(executable, cmd.slice(1), {
@@ -43,6 +34,8 @@ function runTests(label, cmd, cwd) {
             shell: process.platform === 'win32'
         });
 
+        const ms = elapsed();
+
         if (result.error && !result.stdout && !result.stderr) {
             console.log(`    Error: ${result.error.message}`);
         }
@@ -56,11 +49,11 @@ function runTests(label, cmd, cwd) {
         }
 
         if (result.status === 0) {
-            ok(`${label} — all tests passed`);
+            ok(`${label} — all tests passed ${DIM}(${formatMs(ms)})${RESET}`);
             return true;
         }
 
-        fail(`${label} — test failures detected`);
+        fail(`${label} — test failures detected ${DIM}(${formatMs(ms)})${RESET}`);
         return false;
     } catch {
         skip(`${label} — tool not installed`);
@@ -69,20 +62,15 @@ function runTests(label, cmd, cwd) {
 }
 
 function detectTestFramework(projectRoot) {
-    const pkgJson = path.join(projectRoot, "package.json");
-    if (fs.existsSync(pkgJson)) {
-        try {
-            const pkg = JSON.parse(fs.readFileSync(pkgJson, 'utf8'));
-            const deps = { ...(pkg.dependencies || {}), ...(pkg.devDependencies || {}) };
-            const scripts = pkg.scripts || {};
-
-            if (deps.vitest) return "vitest";
-            if (deps.jest) return "jest";
-            if (deps.mocha) return "mocha";
-            if (scripts.test) return "npm-test";
-        } catch {
-            if (fs.readFileSync(pkgJson, 'utf8').includes('"test"')) return "npm-test";
-        }
+    const pkg = loadJson(path.join(projectRoot, "package.json"));
+    if (pkg) {
+        const deps = { ...(pkg.dependencies || {}), ...(pkg.devDependencies || {}) };
+        const scripts = pkg.scripts || {};
+
+        if (deps.vitest) return "vitest";
+        if (deps.jest) return "jest";
+        if (deps.mocha) return "mocha";
+        if (scripts.test) return "npm-test";
     }
 
     if (fs.existsSync(path.join(projectRoot, "pytest.ini")) || 
@@ -137,16 +125,18 @@ function main() {
         process.exit(1);
     }
 
-    console.log(`${BOLD}Tribunal — test_runner.js${RESET}`);
-    console.log(`Project: ${projectRoot}`);
-
     const framework = detectTestFramework(projectRoot);
+    console.log(banner('test_runner.js', {
+        Project: projectRoot,
+        Framework: framework || 'none detected',
+    }));
+
     if (!framework) {
         skip("No test framework detected in this project");
         process.exit(0);
     }
 
-    header(`Running tests (${framework})`);
+    console.log(sectionHeader(`Running tests (${framework})`));
 
     let cmd = [];
     let passed = true;
@@ -181,9 +171,12 @@ function main() {
         passed = runTests("go test", cmd, projectRoot);
     }
 
-    console.log(`\n${BOLD}━━━ Test Summary ━━━${RESET}`);
-    if (passed) ok(`Tests passed (${framework})`);
-    else fail(`Tests failed (${framework})`);
+    console.log(`\n${BOLD}${CYAN}━━━ Test Summary ━━━${RESET}`);
+    if (passed) {
+        console.log(`\n${GREEN}${BOLD}  ✔ Tests passed (${framework}).${RESET}\n`);
+    } else {
+        console.log(`\n${RED}${BOLD}  ✖ Tests failed (${framework}).${RESET}\n`);
+    }
 
     process.exit(passed ? 0 : 1);
 }

package/.agent/scripts/utils.js

@@ -1,32 +1,17 @@
 /**
- * utils.js
- * Shared utilities for Tribunal Kit Node scripts.
+ * utils.js — Backward-compatible re-export of _utils.js
+ * ══════════════════════════════════════════════════════
+ * All utilities are defined in _utils.js.
+ * This file re-exports them for scripts that import from './utils.js'.
+ *
+ * New scripts should import from './_utils' directly.
  */
 'use strict';
 
-const path = require('path');
-const fs = require('fs');
-
-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("\x1b[91m✖ Error: '.agent' directory not found. Please run 'npx tribunal-kit init' first.\x1b[0m");
-    process.exit(1);
-}
-
-function ensureUtf8Stdout() {
-    // In Node.js, process.stdout is typically already UTF-8 capable,
-    // but this exists for compatibility with legacy python workflows.
-}
+const _utils = require('./_utils');
 
 module.exports = {
-    findAgentDir,
-    ensureUtf8Stdout
+    ..._utils,
+    // Legacy export for compatibility
+    ensureUtf8Stdout: function() {},
 };

package/.agent/scripts/verify_all.js

@@ -15,65 +15,68 @@ const fs   = require('fs');
 const path = require('path');
 const { execFileSync } = require('child_process');
 
-// ━━━ ANSI colors ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
-const RED    = '\x1b[91m';
-const GREEN  = '\x1b[92m';
-const YELLOW = '\x1b[93m';
-const BLUE   = '\x1b[94m';
-const BOLD   = '\x1b[1m';
-const RESET  = '\x1b[0m';
+const {
+    RED, GREEN, YELLOW, BLUE, BOLD, DIM, CYAN, RESET,
+    banner, sectionHeader, summaryTable, timer, formatMs,
+    ok, fail, skip,
+} = require('./_colors');
 
-const RESULTS = [];
+const { walkDir, hasNpm, SOURCE_EXTENSIONS } = require('./_utils');
 
-function section(title) {
-    console.log(`\n${BOLD}${BLUE}━━━ ${title} ━━━${RESET}`);
-}
+// ── Results Tracking ────────────────────────────────────────────────────────
 
-function ok(label, note) {
-    const msg = `${GREEN}✅ ${label}${RESET}` + (note ? `  ${YELLOW}(${note})${RESET}` : '');
-    console.log(`  ${msg}`);
-    RESULTS.push({ label, passed: true, note: note || '' });
+const RESULTS = [];
+
+function trackOk(label, ms, note) {
+    const timing = ms != null ? `${DIM}(${formatMs(ms)})${RESET}` : '';
+    const noteStr = note ? `  ${DIM}${note}${RESET}` : '';
+    console.log(`  ${GREEN}✅ ${label}${RESET} ${timing}${noteStr}`);
+    RESULTS.push({ name: label, status: 'pass', ms, note: note || '' });
 }
 
-function fail(label, note) {
+function trackFail(label, ms, note) {
+    const timing = ms != null ? `${DIM}(${formatMs(ms)})${RESET}` : '';
     const noteStr = note ? `\n     ${note}` : '';
-    console.log(`  ${RED}❌ ${label}${RESET}${noteStr}`);
-    RESULTS.push({ label, passed: false, note: note || '' });
+    console.log(`  ${RED}❌ ${label}${RESET} ${timing}${noteStr}`);
+    RESULTS.push({ name: label, status: 'fail', ms, note: note || '' });
 }
 
-function skip(label, reason) {
+function trackSkip(label, reason) {
     console.log(`  ${YELLOW}⏭️  ${label} — ${reason}${RESET}`);
-    RESULTS.push({ label, passed: true, note: `skipped: ${reason}` });
+    RESULTS.push({ name: label, status: 'skip', note: `skipped: ${reason}` });
 }
 
+// ── Command Runner ──────────────────────────────────────────────────────────
+
 /**
  * Run a shell command and return true if it exits with code 0.
  */
 function run(label, cmd, cwd) {
+    const elapsed = timer();
     try {
         const isWindows = process.platform === 'win32';
-        let bin = cmd[0];
-        if (isWindows && (bin === 'npm' || bin === 'npx')) bin += '.cmd';
 
-        execFileSync(bin, cmd.slice(1), {
+        execFileSync(cmd[0], cmd.slice(1), {
             cwd,
             stdio: 'pipe',
             timeout: 120000,
             encoding: 'utf8',
+            shell: isWindows
         });
-        ok(label);
+        trackOk(label, elapsed());
         return true;
     } catch (err) {
+        const ms = elapsed();
         if (err.code === 'ENOENT') {
-            skip(label, 'tool not installed — skipping');
+            trackSkip(label, 'tool not installed — skipping');
             return true;
         }
         if (err.killed) {
-            fail(label, 'timed out after 120s');
+            trackFail(label, ms, 'timed out after 120s');
             return false;
         }
         const output = ((err.stdout || '') + (err.stderr || '')).trim();
-        fail(label, output ? output.slice(0, 500) : 'non-zero exit code');
+        trackFail(label, ms, output ? output.slice(0, 500) : 'non-zero exit code');
         return false;
     }
 }
@@ -81,140 +84,130 @@ function run(label, cmd, cwd) {
 
 /**
  * Scan source files for obviously hardcoded credentials.
+ * Uses shared walkDir from _utils.js to eliminate duplicated walker code.
  */
 function scanSecrets(cwd) {
+    const elapsed = timer();
     const patterns = ['password=', 'secret=', 'api_key=', 'private_key=', 'auth_token='];
     const found = [];
-    const skipDirs = new Set(['node_modules', '.git', 'dist', '__pycache__', '.agent']);
-
-    function walk(dir) {
-        let entries;
-        try { entries = fs.readdirSync(dir, { withFileTypes: true }); } catch { return; }
-
-        for (const entry of entries) {
-            const fullPath = path.join(dir, entry.name);
-            if (entry.isDirectory()) {
-                if (!skipDirs.has(entry.name)) walk(fullPath);
-            } else if (entry.isFile()) {
-                if (!/\.(ts|js|tsx|jsx|py)$/.test(entry.name)) continue;
-
-                let content;
-                try { content = fs.readFileSync(fullPath, 'utf8'); } catch { continue; }
-
-                const lines = content.split('\n');
-                for (let i = 0; i < lines.length; i++) {
-                    const low = lines[i].toLowerCase().trim();
-                    const hasPattern = patterns.some(p => low.includes(p));
-                    if (hasPattern && !low.startsWith('#') && low.includes('=')) {
-                        const rel = path.relative(cwd, fullPath);
-                        found.push(`${rel}:${i + 1}`);
-                    }
-                }
+
+    const sourceExtensions = new Set(['.ts', '.tsx', '.js', '.jsx', '.py']);
+    const files = walkDir(cwd, { extensions: sourceExtensions });
+
+    for (const fullPath of files) {
+        let content;
+        try { content = fs.readFileSync(fullPath, 'utf8'); } catch { continue; }
+
+        const lines = content.split('\n');
+        for (let i = 0; i < lines.length; i++) {
+            const low = lines[i].toLowerCase().trim();
+            const hasPattern = patterns.some(p => low.includes(p));
+            if (hasPattern && !low.startsWith('#') && low.includes('=')) {
+                const rel = path.relative(cwd, fullPath);
+                found.push(`${rel}:${i + 1}`);
             }
         }
     }
 
-    walk(cwd);
-
+    const ms = elapsed();
     if (found.length > 0) {
-        fail('Secret scan', found.slice(0, 5).join('\n     '));
+        trackFail('Secret scan', ms, found.slice(0, 5).join('\n     '));
         return false;
     }
-    ok('Secret scan — no hardcoded credentials found');
+    trackOk('Secret scan — no hardcoded credentials found', ms, `${files.length} files scanned`);
     return true;
 }
 
 
-/**
- * Check if there's a package.json to run npm commands against.
- */
-function hasNpm(cwd) {
-    return fs.existsSync(path.join(cwd, 'package.json'));
-}
-
-
 /**
  * Run all verification checks. Returns number of failures.
  */
 function verifyAll(cwd, skipped) {
     let failures = 0;
+    RESULTS.length = 0; // Reset for clean runs (prevents accumulation in tests)
+    const totalTimer = timer();
 
-    section('1 — Secret Scan');
+    console.log(sectionHeader('Secret Scan', 1));
     if (!skipped.includes('secrets')) {
         if (!scanSecrets(cwd)) failures++;
     } else {
-        skip('Secret scan', 'skipped by flag');
+        trackSkip('Secret scan', 'skipped by flag');
     }
 
-    section('2 — TypeScript');
+    console.log(sectionHeader('TypeScript', 2));
     if (!skipped.includes('typescript')) {
         if (hasNpm(cwd)) {
             if (!run('tsc --noEmit', ['npx', 'tsc', '--noEmit'], cwd)) failures++;
         } else {
-            skip('TypeScript', 'no package.json found in project');
+            trackSkip('TypeScript', 'no package.json found in project');
         }
     } else {
-        skip('TypeScript', 'skipped by flag');
+        trackSkip('TypeScript', 'skipped by flag');
     }
 
-    section('3 — ESLint');
+    console.log(sectionHeader('ESLint', 3));
     if (!skipped.includes('lint')) {
         if (hasNpm(cwd)) {
             if (!run('ESLint', ['npx', 'eslint', '.', '--max-warnings=0'], cwd)) failures++;
         } else {
-            skip('ESLint', 'no package.json found in project');
+            trackSkip('ESLint', 'no package.json found in project');
         }
     } else {
-        skip('ESLint', 'skipped by flag');
+        trackSkip('ESLint', 'skipped by flag');
     }
 
-    section('4 — Unit Tests');
+    console.log(sectionHeader('Unit Tests', 4));
     if (!skipped.includes('tests')) {
         if (hasNpm(cwd)) {
             if (!run('Test suite', ['npm', 'test', '--', '--passWithNoTests'], cwd)) failures++;
         } else {
-            skip('Tests', 'no package.json found in project');
+            trackSkip('Tests', 'no package.json found in project');
         }
     } else {
-        skip('Tests', 'skipped by flag');
+        trackSkip('Tests', 'skipped by flag');
     }
 
-    section('5 — Build');
+    console.log(sectionHeader('Build', 5));
     if (!skipped.includes('build')) {
         if (hasNpm(cwd)) {
             if (!run('npm run build', ['npm', 'run', 'build'], cwd)) failures++;
         } else {
-            skip('Build', 'no package.json found in project');
+            trackSkip('Build', 'no package.json found in project');
         }
     } else {
-        skip('Build', 'skipped by flag');
+        trackSkip('Build', 'skipped by flag');
     }
 
-    section('6 — Dependency Audit');
+    console.log(sectionHeader('Dependency Audit', 6));
     if (!skipped.includes('deps')) {
         if (hasNpm(cwd)) {
             if (!run('npm audit', ['npm', 'audit', '--audit-level=high'], cwd)) failures++;
         } else {
-            skip('Dependency audit', 'no package.json found in project');
+            trackSkip('Dependency audit', 'no package.json found in project');
         }
     } else {
-        skip('Dependency audit', 'skipped by flag');
+        trackSkip('Dependency audit', 'skipped by flag');
     }
 
-    // ━━━ Summary ━━━
-    console.log(`\n${BOLD}━━━ Summary ━━━${RESET}`);
-    for (const { label, passed, note } of RESULTS) {
-        const status = passed ? `${GREEN}✅${RESET}` : `${RED}❌${RESET}`;
-        const noteStr = (!passed && note) ? `  ${YELLOW}(${note})${RESET}` : '';
-        console.log(`  ${status} ${label}${noteStr}`);
-    }
+    // ━━━ Summary Table ━━━
+    const totalMs = totalTimer();
+    console.log(`\n${BOLD}${CYAN}━━━ Verification Summary ━━━${RESET}`);
+    summaryTable(RESULTS);
+
+    const passCount = RESULTS.filter(r => r.status === 'pass').length;
+    const failCount = RESULTS.filter(r => r.status === 'fail').length;
+    const skipCount = RESULTS.filter(r => r.status === 'skip').length;
+
+    console.log(`\n  ${DIM}Total: ${RESULTS.length} checks in ${formatMs(totalMs)}${RESET}`);
+    console.log(`  ${GREEN}${passCount} passed${RESET}  ${failCount > 0 ? `${RED}${failCount} failed${RESET}  ` : ''}${skipCount > 0 ? `${YELLOW}${skipCount} skipped${RESET}` : ''}`);
 
     console.log();
     if (failures === 0) {
-        console.log(`${GREEN}${BOLD}All checks passed — safe to deploy.${RESET}`);
+        console.log(`${GREEN}${BOLD}  ✔ All checks passed — safe to deploy.${RESET}`);
     } else {
-        console.log(`${RED}${BOLD}${failures} check(s) failed — fix before deploying.${RESET}`);
+        console.log(`${RED}${BOLD}  ✖ ${failures} check(s) failed — fix before deploying.${RESET}`);
     }
+    console.log();
 
     return failures;
 }
@@ -240,8 +233,7 @@ function main() {
     const args = parseArgs(process.argv);
     const cwd = process.cwd();
 
-    console.log(`${BOLD}Tribunal — verify_all.js${RESET}`);
-    console.log(`Project: ${cwd}\n`);
+    console.log(banner('verify_all.js', { Project: cwd }));
 
     const failures = verifyAll(cwd, args.skip);
     process.exit(failures > 0 ? 1 : 0);

package/.agent/skills/app-builder/templates/astro-static/TEMPLATE.md

@@ -73,4 +73,4 @@ project-name/
 - Use Content Collections for type safety
 - Leverage static generation
 - Add islands only where needed
-- Optimize images with Astro Image
+- Optimize images with Astro Image

package/.agent/skills/app-builder/templates/chrome-extension/TEMPLATE.md

@@ -89,4 +89,4 @@ project-name/
 - Use type-safe messaging
 - Wrap Chrome APIs in promises
 - Minimize permissions
-- Handle offline gracefully
+- Handle offline gracefully

package/.agent/skills/app-builder/templates/cli-tool/TEMPLATE.md

@@ -85,4 +85,4 @@ npm publish
 - Support both interactive and non-interactive modes
 - Use consistent output styling
 - Validate inputs with Zod
-- Exit with proper codes (0 success, 1 error)
+- Exit with proper codes (0 success, 1 error)

package/.agent/skills/app-builder/templates/electron-desktop/TEMPLATE.md

@@ -85,4 +85,4 @@ project-name/
 - Type-safe IPC with typed handlers
 - Custom title bar for native feel
 - Handle window state (maximize, minimize)
-- Auto-updates with electron-updater
+- Auto-updates with electron-updater

package/.agent/skills/app-builder/templates/express-api/TEMPLATE.md

@@ -80,4 +80,4 @@ project-name/
 - Validate all inputs with Zod
 - Centralized error handling
 - Environment-based config
-- Use Prisma for type-safe DB access
+- Use Prisma for type-safe DB access

package/.agent/skills/app-builder/templates/flutter-app/TEMPLATE.md

@@ -87,4 +87,4 @@ project_name/
 - Riverpod for state, React Query pattern for server state
 - Freezed for immutable data classes
 - Go Router for declarative navigation
-- Material 3 theming
+- Material 3 theming

package/.agent/skills/app-builder/templates/monorepo-turborepo/TEMPLATE.md

@@ -87,4 +87,4 @@ project-name/
 - Shared configs in packages/config
 - Shared types in packages/types
 - Internal packages with `workspace:*`
-- Use Turbo remote caching for CI
+- Use Turbo remote caching for CI

package/.agent/skills/app-builder/templates/nextjs-fullstack/TEMPLATE.md

@@ -119,4 +119,4 @@ project-name/
 - **Mutations**: Use Server Actions combined with React 19's `useActionState` to handle loading and error states instead of manual useState.
 - **Type Safety**: Share Zod schemas between Server Actions (input validation) and Client Forms.
 - **Security**: Always validate input data with Zod before passing it to Prisma.
-- **Styling**: Use native CSS variables in Tailwind v4 for easier dynamic theming.
+- **Styling**: Use native CSS variables in Tailwind v4 for easier dynamic theming.

package/.agent/skills/app-builder/templates/nextjs-saas/TEMPLATE.md

@@ -119,4 +119,4 @@ project-name/
 7. Run project:
    ```bash
    npm run dev
-   ```
+   ```

package/.agent/skills/app-builder/templates/nextjs-static/TEMPLATE.md

@@ -166,4 +166,4 @@ export const metadata: Metadata = {
 - **React Server Components (RSC)**: Default all components to Server Components. Only add `'use client'` when you need state (`useState`) or event listeners (`onClick`).
 - **Image Optimization**: Use the `<Image />` component but remember `unoptimized: true` for static export or use an external image CDN (Cloudinary/Imgix).
 - **Font Optimization**: Use `next/font` (Google Fonts) to automatically host fonts and prevent layout shift.
-- **Responsive**: Mobile-first design using Tailwind prefixes like `sm:`, `md:`, `lg:`.
+- **Responsive**: Mobile-first design using Tailwind prefixes like `sm:`, `md:`, `lg:`.

package/.agent/skills/app-builder/templates/nuxt-app/TEMPLATE.md

@@ -131,4 +131,4 @@ project-name/
   ```
 - **Data Fetching**: Sử dụng `useFetch` với `server: false` cho các tác vụ client-only, hoặc dùng Server Functions để type-safety tốt hơn.
 - **State**: Dùng `defineStore` (Pinia) cho global state, `useState` của Nuxt cho state đơn giản chia sẻ giữa Server/Client.
-- **Type Safety**: Tự động tạo type cho API routes (`$fetch` typed automatically).
+- **Type Safety**: Tự động tạo type cho API routes (`$fetch` typed automatically).

package/.agent/skills/app-builder/templates/python-fastapi/TEMPLATE.md

@@ -80,4 +80,4 @@ project-name/
 - Pydantic v2 for validation
 - SQLAlchemy 2.0 async sessions
 - Alembic for migrations
-- pytest-asyncio for tests
+- pytest-asyncio for tests

package/.agent/skills/app-builder/templates/react-native-app/TEMPLATE.md

@@ -116,4 +116,4 @@ project-name/
 - **React 19**: Reduce usage of `useMemo` or `useCallback` thanks to React Compiler (if enabled).
 - **Components**: Build UI primitives (Box, Text) with NativeWind className for reusability.
 - **Assets**: Use `expo-image` instead of default `<Image />` for better caching and performance.
-- **API**: Always wrap API calls with TanStack Query, avoid direct calls in `useEffect`.
+- **API**: Always wrap API calls with TanStack Query, avoid direct calls in `useEffect`.

package/.agent/skills/doc.md

@@ -174,4 +174,4 @@ By creating Skills, you transform a general AI model into an expert for your pro
 - ✅ Automatically add license headers
 - ✅ The Agent automatically knows how to work with your team
 
-Instead of constantly reminding the AI to "remember to add the license" or "fix the commit format," now the Agent will do it automatically!
+Instead of constantly reminding the AI to "remember to add the license" or "fix the commit format," now the Agent will do it automatically!