tribunal-kit 6.0.1 → 8.0.0

package/scripts/changelog.js

@@ -18,93 +18,91 @@
  *   node scripts/changelog.js --since v4.1.0   → Changes since a specific tag
  */
 
-"use strict";
+'use strict';
 
-const { execSync } = require("child_process");
-const fs = require("fs");
-const path = require("path");
+const { execSync } = require('child_process');
+const fs = require('fs');
+const path = require('path');
 
-const PKG = require(path.resolve(__dirname, "..", "package.json"));
-const CHANGELOG_PATH = path.resolve(__dirname, "..", "CHANGELOG.md");
+const PKG = require(path.resolve(__dirname, '..', 'package.json'));
+const CHANGELOG_PATH = path.resolve(__dirname, '..', 'CHANGELOG.md');
 
 // ── Commit Categories ────────────────────────────────────
 const CATEGORIES = {
-  feat: { emoji: "✨", title: "Features" },
-  fix: { emoji: "🐛", title: "Bug Fixes" },
-  perf: { emoji: "⚡", title: "Performance" },
-  docs: { emoji: "📝", title: "Documentation" },
-  test: { emoji: "✅", title: "Tests" },
-  refactor: { emoji: "♻️", title: "Refactors" },
-  chore: { emoji: "🔧", title: "Chores" },
-  ci: { emoji: "🏗️", title: "CI/CD" },
-  style: { emoji: "🎨", title: "Style" },
-  breaking: { emoji: "💥", title: "Breaking Changes" },
+  feat: { emoji: '✨', title: 'Features' },
+  fix: { emoji: '🐛', title: 'Bug Fixes' },
+  perf: { emoji: '⚡', title: 'Performance' },
+  docs: { emoji: '📝', title: 'Documentation' },
+  test: { emoji: '✅', title: 'Tests' },
+  refactor: { emoji: '♻️', title: 'Refactors' },
+  chore: { emoji: '🔧', title: 'Chores' },
+  ci: { emoji: '🏗️', title: 'CI/CD' },
+  style: { emoji: '🎨', title: 'Style' },
+  breaking: { emoji: '💥', title: 'Breaking Changes' },
 };
 
 // ── Git Helpers ──────────────────────────────────────────
 function git(cmd) {
   try {
-    return execSync(`git ${cmd}`, { encoding: "utf8", timeout: 10000 }).trim();
+    return execSync(`git ${cmd}`, { encoding: 'utf8', timeout: 10000 }).trim();
   } catch {
-    return "";
+    return '';
   }
 }
 
 function getLatestTag() {
   return (
-    git("describe --tags --abbrev=0 2>nul") ||
-    git("describe --tags --abbrev=0 2>/dev/null") ||
-    ""
+    git('describe --tags --abbrev=0 2>nul') || git('describe --tags --abbrev=0 2>/dev/null') || ''
   );
 }
 
 function getCommits(since) {
-  const range = since ? `${since}..HEAD` : "HEAD";
+  const range = since ? `${since}..HEAD` : 'HEAD';
   const MAX_COMMITS = 500;
-  const format = "--format=%H||%s||%an||%ai";
+  const format = '--format=%H||%s||%an||%ai';
   const raw = git(`log ${range} ${format} --no-merges -n ${MAX_COMMITS}`);
   if (!raw) return [];
 
   return raw
-    .split("\n")
-    .filter(Boolean)
-    .map((line) => {
-      const [hash, subject, author, date] = line.split("||");
+    .replace(/\r/g, '')
+    .split('\n')
+    .filter(line => line.includes('||'))
+    .map(line => {
+      const [hash, subject = '', author = '', date = ''] = line.split('||');
       return {
-        hash: hash?.replace(/^"/, "").slice(0, 7),
-        subject,
-        author,
-        date: date?.replace(/"$/, "").slice(0, 10),
+        hash: hash?.replace(/^"/, '').slice(0, 7),
+        subject: subject.trim(),
+        author: author.trim(),
+        date: date?.replace(/"$/, '').slice(0, 10),
       };
     });
 }
 
-function categorize(subject) {
-  const lower = subject.toLowerCase();
+function categorize(subject = '') {
+  const lower = (subject || '').toLowerCase();
 
   // Check for BREAKING CHANGE
-  if (lower.includes("breaking") || lower.includes("!:")) {
-    return "breaking";
+  if (lower.includes('breaking') || lower.includes('!:')) {
+    return 'breaking';
   }
 
   // Match conventional commit prefix
-  const match = subject.match(/^(\w+)(?:\(.+?\))?:\s*/);
+  const match = (subject || '').match(/^(\w+)(?:\(.+?\))?:\s*/);
   if (match) {
     const type = match[1].toLowerCase();
     if (CATEGORIES[type]) return type;
   }
 
   // Heuristic fallback
-  if (lower.includes("fix") || lower.includes("bug")) return "fix";
-  if (lower.includes("add") || lower.includes("new") || lower.includes("feat"))
-    return "feat";
-  if (lower.includes("doc") || lower.includes("readme")) return "docs";
-  if (lower.includes("test")) return "test";
-  if (lower.includes("refactor") || lower.includes("clean")) return "refactor";
-  if (lower.includes("perf") || lower.includes("optim")) return "perf";
-  if (lower.includes("ci") || lower.includes("workflow")) return "ci";
-
-  return "chore";
+  if (lower.includes('fix') || lower.includes('bug')) return 'fix';
+  if (lower.includes('add') || lower.includes('new') || lower.includes('feat')) return 'feat';
+  if (lower.includes('doc') || lower.includes('readme')) return 'docs';
+  if (lower.includes('test')) return 'test';
+  if (lower.includes('refactor') || lower.includes('clean')) return 'refactor';
+  if (lower.includes('perf') || lower.includes('optim')) return 'perf';
+  if (lower.includes('ci') || lower.includes('workflow')) return 'ci';
+
+  return 'chore';
 }
 
 // ── Changelog Generation ─────────────────────────────────
@@ -114,7 +112,7 @@ function generateChangelog(commits, version, date) {
     const cat = categorize(commit.subject);
     if (!grouped[cat]) grouped[cat] = [];
     // Strip conventional prefix for cleaner display
-    const clean = commit.subject.replace(/^\w+(\(.+?\))?:\s*/, "");
+    const clean = (commit.subject || '').replace(/^\w+(\(.+?\))?:\s*/, '');
     grouped[cat].push({ ...commit, clean });
   }
 
@@ -122,16 +120,16 @@ function generateChangelog(commits, version, date) {
 
   // Breaking changes first
   const order = [
-    "breaking",
-    "feat",
-    "fix",
-    "perf",
-    "refactor",
-    "docs",
-    "test",
-    "ci",
-    "style",
-    "chore",
+    'breaking',
+    'feat',
+    'fix',
+    'perf',
+    'refactor',
+    'docs',
+    'test',
+    'ci',
+    'style',
+    'chore',
   ];
   for (const cat of order) {
     if (!grouped[cat] || grouped[cat].length === 0) continue;
@@ -140,7 +138,7 @@ function generateChangelog(commits, version, date) {
     for (const c of grouped[cat]) {
       md += `- ${c.clean} (\`${c.hash}\`)\n`;
     }
-    md += "\n";
+    md += '\n';
   }
 
   return md;
@@ -149,48 +147,44 @@ function generateChangelog(commits, version, date) {
 // ── Main ─────────────────────────────────────────────────
 function main() {
   const args = process.argv.slice(2);
-  const isPreview = args.includes("--preview");
-  const sinceIdx = args.indexOf("--since");
+  const isPreview = args.includes('--preview');
+  const sinceIdx = args.indexOf('--since');
   const sinceTag = sinceIdx !== -1 ? args[sinceIdx + 1] : null;
 
   const since = sinceTag || getLatestTag();
   const commits = getCommits(since);
 
   if (commits.length === 0) {
-    console.log("  ℹ️  No new commits found since", since || "beginning");
+    console.log('  ℹ️  No new commits found since', since || 'beginning');
     process.exit(0);
   }
 
   const today = new Date().toISOString().slice(0, 10);
-  const version = isPreview ? "Unreleased" : PKG.version;
+  const version = isPreview ? 'Unreleased' : PKG.version;
 
   const changelog = generateChangelog(commits, version, today);
 
   if (isPreview) {
-    console.log("\n  📋 Changelog Preview\n  " + "─".repeat(40) + "\n");
+    console.log('\n  📋 Changelog Preview\n  ' + '─'.repeat(40) + '\n');
     console.log(changelog);
-    console.log(
-      `  📊 ${commits.length} commits since ${since || "initial commit"}`,
-    );
+    console.log(`  📊 ${commits.length} commits since ${since || 'initial commit'}`);
     return;
   }
 
   // Write or prepend to CHANGELOG.md
   const header = `# Changelog\n\nAll notable changes to Tribunal Kit are documented here.\nFormat follows [Keep a Changelog](https://keepachangelog.com/).\n\n`;
 
-  let existing = "";
+  let existing = '';
   if (fs.existsSync(CHANGELOG_PATH)) {
-    existing = fs.readFileSync(CHANGELOG_PATH, "utf8");
+    existing = fs.readFileSync(CHANGELOG_PATH, 'utf8');
     // Remove existing header
-    existing = existing.replace(/^# Changelog[\s\S]*?(?=## )/, "");
+    existing = existing.replace(/^# Changelog[\s\S]*?(?=## )/, '');
   }
 
   const full = header + changelog + existing;
-  fs.writeFileSync(CHANGELOG_PATH, full, "utf8");
+  fs.writeFileSync(CHANGELOG_PATH, full, 'utf8');
 
-  console.log(
-    `  ✔ CHANGELOG.md updated — v${version} (${commits.length} commits)`,
-  );
+  console.log(`  ✔ CHANGELOG.md updated — v${version} (${commits.length} commits)`);
 }
 
 main();

package/scripts/fix-vbc.js

@@ -7,12 +7,12 @@
  * Dry-run: node scripts/fix-vbc.js --dry-run
  */
 
-const fs = require("fs");
-const path = require("path");
+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 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
@@ -38,34 +38,32 @@ let fixedCount = 0;
 let skippedCount = 0;
 let errorCount = 0;
 
-const skillDirs = fs.readdirSync(SKILLS_DIR).filter((d) => {
+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");
+  const skillPath = path.join(SKILLS_DIR, dir, 'SKILL.md');
   if (!fs.existsSync(skillPath)) continue;
 
   let content;
   try {
-    content = fs.readFileSync(skillPath, "utf8");
+    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");
+  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 = "";
+  let appendText = '';
 
   if (!hasPreFlight) {
     appendText += PRE_FLIGHT_BLOCK;
@@ -81,7 +79,7 @@ for (const dir of skillDirs) {
     console.log(`  [DRY-RUN] Would append to ${dir}/SKILL.md`);
   } else {
     try {
-      fs.appendFileSync(skillPath, appendText, "utf8");
+      fs.appendFileSync(skillPath, appendText, 'utf8');
       fixedCount++;
     } catch (e) {
       console.error(`  ❌ Failed to write: ${dir}/SKILL.md — ${e.message}`);
@@ -94,5 +92,5 @@ 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(`  Mode:    ${DRY_RUN ? 'DRY-RUN (no changes)' : 'LIVE'}`);
 console.log();

package/scripts/fix-vbc.ps1

@@ -1,119 +1,3 @@
-$ErrorActionPreference = "Stop"
+# DEPRECATED — Use cross-platform Node script: node scripts/fix-vbc.js
+Write-Host "Please run 'node scripts/fix-vbc.js' instead." -ForegroundColor Yellow
 
-$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