toga-ai 1.0.803 → 1.0.804

package/.claude/settings.json

@@ -70,6 +70,16 @@
             "timeout": 3000
           }
         ]
+      },
+      {
+        "matcher": "Edit|Write|MultiEdit",
+        "hooks": [
+          {
+            "type": "command",
+            "command": "node \"$CLAUDE_PROJECT_DIR/.claude/hooks/toga/design-fidelity-reminder.js\"",
+            "timeout": 3000
+          }
+        ]
       }
     ],
     "PostToolUse": [

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "toga-ai",
-  "version": "1.0.803",
+  "version": "1.0.804",
   "description": "TOGA Technology Team Claude Knowledge System — shared AI coding harness with skills, knowledge base CLI, and project installer for Claude Code.",
   "keywords": [
     "claude",

package/scripts/hooks/design-fidelity-reminder.js

@@ -0,0 +1,99 @@
+#!/usr/bin/env node
+'use strict';
+
+/*
+ * design-fidelity-reminder.js — PRE-EDIT enforcement of the Design Contract discipline.
+ *
+ * Why this exists: on Desk (and the other 2.0 React apps) the model keeps improvising
+ * UI — inventing button text, helper lines, field order and labels instead of matching
+ * the designer's on-disk design. That drift ships UIs that "function" but do not match
+ * what Matt designed, and the developer has to catch it every time. A rule in a file is
+ * passive; over a long session the model slides back to improvising. This is the
+ * mechanical backstop, modeled on plain-talk.js.
+ *
+ * How it works: registered on PreToolUse for Edit|Write|MultiEdit. Just before the model
+ * edits a UI source file — a path under a `src/` tree ending .tsx / .jsx / .css — it prints
+ * a short <system-reminder> to stdout with the Design Contract rules, so the discipline is
+ * in context at the moment the UI is written. On any non-UI file it stays silent.
+ *
+ * It never blocks. It always exits 0 — a reminder must never stop or deny an edit. It
+ * fails open on any error: a crashing nudge must never brick a session. Per team hook
+ * convention DENY/error text goes to STDERR — this hook never denies, so it never uses it.
+ *
+ * Escape hatch: set TOGA_DESIGN_FIDELITY_DISABLED=1 to silence the reminder.
+ */
+
+const fs = require('fs');
+
+/* Read the CC hook payload. PreToolUse delivers the tool input on stdin as
+ * { tool_name, tool_input: { file_path, ... } }; fall back to the CLAUDE_TOOL_INPUT env
+ * (what the sibling PostToolUse reminders read) and to a top-level file_path. */
+function readFilePath() {
+  let raw = '';
+  try { raw = fs.readFileSync(0, 'utf8'); } catch (e) { /* no stdin */ }
+
+  let data = {};
+  if (raw && raw.trim()) {
+    try { data = JSON.parse(raw); } catch (e) { data = {}; }
+  }
+
+  let fp =
+    (data.tool_input && typeof data.tool_input.file_path === 'string' && data.tool_input.file_path) ||
+    (typeof data.file_path === 'string' && data.file_path) ||
+    '';
+
+  if (!fp && process.env.CLAUDE_TOOL_INPUT) {
+    try {
+      const env = JSON.parse(process.env.CLAUDE_TOOL_INPUT);
+      if (env && typeof env.file_path === 'string') fp = env.file_path;
+    } catch (e) { /* ignore */ }
+  }
+  return fp;
+}
+
+/* A Desk / React-app UI source file: under a `src/` tree AND ending .tsx / .jsx / .css. */
+function isUiSourceFile(filePath) {
+  if (!filePath) return false;
+  const p = filePath.replace(/\\/g, '/').toLowerCase();
+  return /\/src\//.test(p) && /\.(tsx|jsx|css)$/.test(p);
+}
+
+function reminder() {
+  return [
+    '<system-reminder>',
+    'DESIGN CONTRACT — this edit touches a UI source file. Match the designer\'s design,',
+    'do not improvise UI. Reusing a shell or making it "function" is NOT fidelity.',
+    '',
+    '  1. The on-disk design is the authority. Open the exact design source file FIRST and',
+    '     build from it — match its LAYOUT/placement, not just pixels.',
+    '  2. Build to a written Design Contract: every field + its order, each label/placeholder,',
+    '     required flags, button text, modal size/layout, and the header/footer.',
+    '  3. Invent NOTHING user-visible — button text, helper/info lines, labels, required flags.',
+    '     Copy comes from the design + Surface/settings. If it is not in the design, ASK.',
+    '  4. Design-vs-data conflict → STOP and ask. Do not decide it yourself.',
+    '  5. No self-sign-off. Hand the developer a labeled diff (mine vs the design). "It',
+    '     functions" is not fidelity.',
+    '</system-reminder>',
+  ].join('\n');
+}
+
+function main() {
+  if (process.env.TOGA_DESIGN_FIDELITY_DISABLED === '1') process.exit(0);
+
+  const filePath = readFilePath();
+
+  if (isUiSourceFile(filePath)) {
+    // Inject the reminder into the model's context via the PreToolUse additionalContext
+    // channel (raw stdout on PreToolUse only shows in transcript, not context). Non-blocking:
+    // additionalContext adds context, it does not deny the edit.
+    console.log(JSON.stringify({
+      hookSpecificOutput: {
+        hookEventName: 'PreToolUse',
+        additionalContext: reminder(),
+      },
+    }));
+  }
+  process.exit(0);
+}
+
+try { main(); } catch (e) { process.exit(0); } // fail-open: never block or brick an edit