@ryuenn3123/agentic-senior-core 6.10.0 → 6.11.1

package/.agents/plugins/agentic-senior-core/.codex-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "agentic-senior-core",
-  "version": "6.10.0",
+  "version": "6.11.1",
   "description": "Universal AI coding rules. Write code like a staff engineer.",
   "skills": "./skills/",
   "hooks": "./hooks/hooks.json",

package/.agents/plugins/agentic-senior-core/hooks/lib/known-stub-patterns.json

@@ -0,0 +1,29 @@
+{
+  "description": "Regex patterns for temporary stubs, untracked debt, and fake completions across all languages.",
+  "patterns": [
+    {
+      "id": "asc-stub-untracked",
+      "regex": "(?:\\/\\/|#|\\/\\*)\\s*ASC-STUB:(?!.*\\b(ASC-DEBT|#\\d+|[A-Z]+-\\d+)\\b)",
+      "message": "ASC-STUB marker detected without tracked debt reference. Add tracking ID (e.g. // ASC-STUB: [reason] [ASC-DEBT-001]) and record in debt ledger.",
+      "languages": ["universal"]
+    },
+    {
+      "id": "untracked-todo",
+      "regex": "(?:\\/\\/|#|\\/\\*)\\s*TODO(?!.*\\b(ASC-DEBT|#\\d+|[A-Z]+-\\d+)\\b)",
+      "message": "TODO without tracked debt or issue reference. Track via /asc-debt or resolve before completing task.",
+      "languages": ["universal"]
+    },
+    {
+      "id": "fake-success-mock",
+      "regex": "(?:return\\s*\\{\\s*success:\\s*true\\s*\\}|return\\s*True|return\\s*true|return\\s*nil)\\s*;?\\s*(?:\\/\\/|#).*?(mock|fake|stub|placeholder|todo)",
+      "message": "Hardcoded success return near mock/stub marker detected. Never simulate success to mimic real integration.",
+      "languages": ["universal"]
+    },
+    {
+      "id": "hardcoded-endpoint-stub",
+      "regex": "(?:baseURL|apiUrl|endpoint|api_url)\\s*[:=]\\s*['\"]https?:\\/\\/(?!localhost|127\\.0\\.0\\.1)",
+      "message": "Hardcoded external endpoint detected outside config/env. Store endpoints in environment variables.",
+      "languages": ["js", "ts", "jsx", "tsx", "py", "go", "rs"]
+    }
+  ]
+}

package/.agents/plugins/agentic-senior-core/hooks/lib/known-ui-slop-patterns.json

@@ -29,6 +29,21 @@
       "id": "ui-pill-badge-generic",
       "regex": "rounded-full\\s+px-4\\s+py-1\\s+text-sm\\s+font-medium\\s+bg-(?:blue|purple|indigo)-100\\s+text-(?:blue|purple|indigo)-800",
       "message": "note: 'generic pill badge' (AI default tag) was flagged, consider continuing with the project's existing badge component instead."
+    },
+    {
+      "id": "ui-gradient-text-cliche",
+      "regex": "bg-clip-text\\s+text-transparent|text-transparent\\s+bg-clip-text",
+      "message": "note: 'gradient text (bg-clip-text text-transparent)' is an overused AI landing page trope. Prefer solid semantic typography with strong hierarchy."
+    },
+    {
+      "id": "ui-transition-all-layout-cost",
+      "regex": "\\btransition-all(?:\\s+duration-[0-9]+)?\\b|transition:\\s*all\\b",
+      "message": "note: 'transition-all' triggers layout repaints on all properties. Transition specific GPU-accelerated properties (transform, opacity) instead."
+    },
+    {
+      "id": "ui-stripped-focus-ring",
+      "regex": "(?:\\boutline-none\\b|\\bfocus:outline-none\\b)(?!.*(?:focus-visible:ring|focus:ring|focus-visible:outline))",
+      "message": "warning: 'outline-none' without a focus-visible ring breaks keyboard navigation (WCAG 2.4.7 violation). Pair with focus-visible:ring-2."
     }
   ]
 }

package/.agents/plugins/agentic-senior-core/hooks/post-edit-enforce.js

@@ -40,6 +40,14 @@ try {
   }
 } catch (_) { }
 
+let STUB_PATTERNS = { patterns: [] };
+try {
+  const stubPath = path.join(__dirname, 'lib', 'known-stub-patterns.json');
+  if (fs.existsSync(stubPath)) {
+    STUB_PATTERNS = JSON.parse(fs.readFileSync(stubPath, 'utf8'));
+  }
+} catch (_) { }
+
 const {
   SOURCE_EXTENSIONS,
   LOC_DELTA_THRESHOLD,
@@ -49,16 +57,50 @@ const {
 } = require('./constants.cjs');
 
 // Module-level counter for Claude Code path (resets per process spawn)
-let sourceEditCount = 0;
+function logHook(event, msg) {
+  try {
+    const os = require('os');
+    const logDir = path.join(os.homedir(), '.asc');
+    if (!fs.existsSync(logDir)) fs.mkdirSync(logDir, { recursive: true });
+    const logFile = path.join(logDir, 'hooks.log');
+    try {
+      if (fs.existsSync(logFile)) {
+        const stats = fs.statSync(logFile);
+        if (stats.size > 100 * 1024) {
+          const content = fs.readFileSync(logFile, 'utf8');
+          const trimmed = content.slice(-50 * 1024);
+          const firstNewline = trimmed.indexOf('\n');
+          fs.writeFileSync(logFile, firstNewline >= 0 ? trimmed.slice(firstNewline + 1) : trimmed, 'utf8');
+        }
+      }
+    } catch (_) {}
+    fs.appendFileSync(logFile, `[${new Date().toISOString()}] [post-edit-enforce] [${event}] ${msg}\n`, 'utf8');
+  } catch (_) {}
+}
+
+logHook('ScriptStart', `pid=${process.pid} cwd=${process.cwd()}`);
+
+if (process.stdin.isTTY) {
+  logHook('StdinTTY', 'Exiting because stdin is a TTY');
+  process.exit(0);
+}
+
+const stdinTimeout = setTimeout(() => {
+  logHook('StdinTimeout', `No input after 3s, exiting bufferLen=${inputBuffer.length}`);
+  process.exit(0);
+}, 3000);
 
 let inputBuffer = '';
 process.stdin.setEncoding('utf8');
 process.stdin.on('data', chunk => {
   inputBuffer += chunk;
+  logHook('DataChunk', `len=${chunk.length} preview=${chunk.slice(0, 100).replace(/[\r\n]+/g, ' ')}`);
   try {
     const data = JSON.parse(inputBuffer);
+    clearTimeout(stdinTimeout);
+    logHook('StdinData', 'keys=' + Object.keys(data).join(','));
 
-    if (data.invocationNum !== undefined && data.transcriptPath) {
+    if (data.transcriptPath) {
       handleAntigravityPostInvocation(data);
       return;
     }
@@ -74,6 +116,32 @@ process.stdin.on('data', chunk => {
   }
 });
 
+process.stdin.on('end', () => {
+  clearTimeout(stdinTimeout);
+  logHook('StdinEnd', `bufferLen=${inputBuffer.length}`);
+  if (inputBuffer.trim()) {
+    try {
+      const data = JSON.parse(inputBuffer);
+      logHook('ParsedInputOnEnd', 'keys=' + Object.keys(data).join(','));
+      if (data.transcriptPath) {
+        handleAntigravityPostInvocation(data);
+        return;
+      }
+      const toolName = data.tool_name || '';
+      const toolInput = data.tool_input || {};
+      processSingleEdit(toolName, toolInput, function (nudge) {
+        emitClaude(nudge);
+      });
+      process.exit(0);
+    } catch (e) {
+      logHook('JsonParseErrorOnEnd', e.message);
+      process.exit(0);
+    }
+  } else {
+    process.exit(0);
+  }
+});
+
 function handleAntigravityPostInvocation(data) {
   try {
     if (!fs.existsSync(data.transcriptPath)) return;
@@ -125,22 +193,32 @@ function handleAntigravityPostInvocation(data) {
         + '(2) Does the codebase already have this? (3) Stdlib/native? (4) Existing dependency?');
     }
 
-    if (findings.length > 0) {
+    const isPostToolUse = data.stepIdx !== undefined;
+    logHook('AntigravityResult', `sourceEdits=${sourceEditsSinceStart}, findings=${findings.length}, isPostToolUse=${isPostToolUse}`);
+
+    if (isPostToolUse) {
+      process.stdout.write(JSON.stringify({}) + '\n');
+    } else if (findings.length > 0) {
       const injectSteps = findings.map(function (f) { return { ephemeralMessage: f }; });
       process.stdout.write(JSON.stringify({ injectSteps: injectSteps }) + '\n');
     } else {
       process.stdout.write(JSON.stringify({}) + '\n');
     }
   } catch (e) {
+    logHook('AntigravityError', e.message);
     process.stdout.write(JSON.stringify({}) + '\n');
   }
+  process.exit(0);
 }
 
 function processSingleEdit(toolName, toolInput, emitFn, skipArray) {
-  const filePath = toolInput.file_path || '';
+  const filePath = toolInput.file_path || toolInput.TargetFile || toolInput.filePath || '';
   if (!filePath) return;
   const findings = [];
 
+  const isEdit = toolName === 'Edit' || toolName === 'replace_file_content' || toolName === 'multi_replace_file_content';
+  const isWrite = toolName === 'Write' || toolName === 'write_to_file';
+
   // Increment per-process counter for Claude Code drift detection
   const ext = path.extname(filePath).slice(1);
   if (SOURCE_EXTENSIONS.has(ext)) {
@@ -160,29 +238,31 @@ function processSingleEdit(toolName, toolInput, emitFn, skipArray) {
   }
 
   if (SOURCE_EXTENSIONS.has(ext)) {
-    if (toolName === 'Edit') {
+    if (isEdit) {
       checkLocDelta(toolInput, filePath, findings);
-    } else if (toolName === 'Write') {
+    } else if (isWrite) {
       checkNewFileSize(toolInput, filePath, findings);
     }
   }
 
   checkSecurityPatterns(toolName, toolInput, filePath, findings);
+  checkStubPatterns(toolName, toolInput, filePath, findings);
 
   if (ext === 'html' || ext === 'css' || ext === 'jsx' || ext === 'tsx' || ext === 'vue' || ext === 'svelte') {
     checkUiSlopPatterns(toolName, toolInput, filePath, findings);
   }
 
-  if (ext === 'js' || ext === 'ts' || ext === 'jsx' || ext === 'tsx' || ext === 'mjs' || ext === 'cjs') {
-    checkLinter(filePath, findings);
-  }
+  checkUniversalLinter(filePath, ext, findings);
 
   checkLivingDocNudge(filePath, findings);
+  checkTestTampering(filePath, findings);
 
   if (ext !== 'md') {
     checkWorkflowGate(toolName, filePath, ext, findings);
   }
 
+  logHook('ProcessSingleEdit', 'tool=' + toolName + ', file=' + filePath + ', findings=' + findings.length);
+
   if (findings.length === 0) return;
 
   const nudge = '[ASC enforcement] ' + findings.join(' ') + ' Review the decision ladder before continuing.';
@@ -248,8 +328,8 @@ function extractDeps(text, pattern) {
 }
 
 function checkLocDelta(toolInput, filePath, findings) {
-  var newLines = (toolInput.new_string || '').split('\n').length;
-  var oldLines = (toolInput.old_string || '').split('\n').length;
+  var newLines = (toolInput.new_string || toolInput.ReplacementContent || '').split('\n').length;
+  var oldLines = (toolInput.old_string || toolInput.TargetContent || '').split('\n').length;
   var delta = newLines - oldLines;
   if (delta > LOC_DELTA_THRESHOLD) {
     findings.push(
@@ -260,7 +340,7 @@ function checkLocDelta(toolInput, filePath, findings) {
 }
 
 function checkNewFileSize(toolInput, filePath, findings) {
-  var lines = (toolInput.content || '').split('\n').length;
+  var lines = (toolInput.content || toolInput.CodeContent || '').split('\n').length;
   if (lines > NEW_FILE_LINE_THRESHOLD) {
     // Step 1-2 coverage ("does this already exist?") moved to dedup-gate.js
     // which provides concrete file-name + overlap-percentage feedback.
@@ -319,7 +399,7 @@ function checkSecurityPatterns(toolName, toolInput, filePath, findings) {
 }
 
 function checkUiSlopPatterns(toolName, toolInput, filePath, findings) {
-  var target = toolName === 'Edit' ? (toolInput.new_string || '') : (toolInput.content || '');
+  var target = toolName === 'Edit' ? (toolInput.new_string || toolInput.ReplacementContent || '') : (toolInput.content || toolInput.CodeContent || '');
   if (!target) return;
 
   if (UI_SLOP_PATTERNS.patterns) {
@@ -336,27 +416,95 @@ function checkUiSlopPatterns(toolName, toolInput, filePath, findings) {
   }
 }
 
-function checkLinter(filePath, findings) {
-  try {
-    var cwd = process.cwd();
-    var hasEslint = fs.existsSync(path.join(cwd, '.eslintrc.json')) ||
-      fs.existsSync(path.join(cwd, '.eslintrc.js')) ||
-      fs.existsSync(path.join(cwd, 'eslint.config.js')) ||
-      (fs.existsSync(path.join(cwd, 'package.json')) && fs.readFileSync(path.join(cwd, 'package.json'), 'utf8').includes('eslintConfig'));
-
-    if (hasEslint) {
-      var execSync = require('child_process').execSync;
-      execSync('npx eslint "' + filePath + '" --format json', { cwd: cwd, stdio: 'pipe' });
-    }
-  } catch (error) {
-    if (error.stdout) {
+function checkStubPatterns(toolName, toolInput, filePath, findings) {
+  var target = toolName === 'Edit' ? (toolInput.new_string || toolInput.ReplacementContent || '') : (toolInput.content || toolInput.CodeContent || '');
+  if (!target) return;
+
+  var ext = path.extname(filePath).slice(1);
+  if (STUB_PATTERNS.patterns) {
+    STUB_PATTERNS.patterns.forEach(function (p) {
       try {
-        var out = JSON.parse(error.stdout.toString());
-        if (Array.isArray(out) && out.length > 0 && out[0].messages && out[0].messages.length > 0) {
-          var firstErr = out[0].messages[0];
-          findings.push('[ASC Linter] ' + firstErr.message + ' at line ' + firstErr.line + '.');
+        var langs = p.languages || [];
+        var isUniversal = langs.length === 0 || langs.indexOf('universal') !== -1;
+        if (!isUniversal && langs.indexOf(ext) === -1) return;
+
+        var regex = new RegExp(p.regex, 'im');
+        var isMatch = regex.test(target);
+        logPatternCheck('stub', p.id || 'stub-pattern', isMatch);
+        if (isMatch) {
+          findings.push('[ASC Stub/Fake-Done Alert] ' + p.message);
         }
       } catch (_) { }
+    });
+  }
+}
+
+function checkUniversalLinter(filePath, ext, findings) {
+  var execSync = require('child_process').execSync;
+  var cwd = process.cwd();
+
+  if (ext === 'js' || ext === 'ts' || ext === 'jsx' || ext === 'tsx' || ext === 'mjs' || ext === 'cjs') {
+    try {
+      var hasBiome = fs.existsSync(path.join(cwd, 'biome.json')) || fs.existsSync(path.join(cwd, 'biome.jsonc'));
+      if (hasBiome) {
+        execSync('npx @biomejs/biome lint "' + filePath + '"', { cwd: cwd, stdio: 'pipe', timeout: 3000 });
+        return;
+      }
+      var hasEslint = fs.existsSync(path.join(cwd, '.eslintrc.json')) ||
+        fs.existsSync(path.join(cwd, '.eslintrc.js')) ||
+        fs.existsSync(path.join(cwd, 'eslint.config.js')) ||
+        fs.existsSync(path.join(cwd, 'eslint.config.mjs')) ||
+        (fs.existsSync(path.join(cwd, 'package.json')) && fs.readFileSync(path.join(cwd, 'package.json'), 'utf8').includes('eslintConfig'));
+
+      if (hasEslint) {
+        execSync('npx eslint "' + filePath + '" --format json', { cwd: cwd, stdio: 'pipe', timeout: 4000 });
+      }
+    } catch (error) {
+      if (error.stdout) {
+        try {
+          var out = JSON.parse(error.stdout.toString());
+          if (Array.isArray(out) && out.length > 0 && out[0].messages && out[0].messages.length > 0) {
+            var firstErr = out[0].messages[0];
+            findings.push('[ASC Linter] ' + firstErr.message + ' at line ' + firstErr.line + '.');
+          }
+        } catch (_) {
+          var msg = error.stdout.toString().split('\n')[0];
+          if (msg) findings.push('[ASC Linter] ' + msg);
+        }
+      }
+    }
+  } else if (ext === 'py') {
+    try {
+      if (fs.existsSync(path.join(cwd, 'pyproject.toml')) || fs.existsSync(path.join(cwd, 'ruff.toml'))) {
+        execSync('ruff check --quiet "' + filePath + '"', { cwd: cwd, stdio: 'pipe', timeout: 3000 });
+      }
+    } catch (error) {
+      if (error.stdout) {
+        var first = error.stdout.toString().split('\n')[0];
+        if (first) findings.push('[ASC Python Linter] ' + first);
+      }
+    }
+  } else if (ext === 'go') {
+    try {
+      if (fs.existsSync(path.join(cwd, 'go.mod'))) {
+        execSync('go vet "' + filePath + '"', { cwd: cwd, stdio: 'pipe', timeout: 4000 });
+      }
+    } catch (error) {
+      if (error.stderr) {
+        var first = error.stderr.toString().split('\n')[0];
+        if (first) findings.push('[ASC Go Vet] ' + first);
+      }
+    }
+  } else if (ext === 'rs') {
+    try {
+      if (fs.existsSync(path.join(cwd, 'Cargo.toml'))) {
+        execSync('cargo clippy --quiet --message-format=short', { cwd: cwd, stdio: 'pipe', timeout: 5000 });
+      }
+    } catch (error) {
+      if (error.stderr) {
+        var first = error.stderr.toString().split('\n')[0];
+        if (first) findings.push('[ASC Clippy] ' + first);
+      }
     }
   }
 }
@@ -371,10 +519,31 @@ function checkLivingDocNudge(filePath, findings) {
   }
 }
 
+function checkTestTampering(filePath, findings) {
+  var isTest = /(?:\.test\.|\.spec\.|_test\.|__tests__[\/\\]|tests?[\/\\]).*\.(?:js|ts|jsx|tsx|py|go|rs|rb)$/i.test(filePath);
+  if (!isTest) return;
+
+  try {
+    var pathUtil = require('./path-util.cjs');
+    var localGate = path.join(process.cwd(), 'workflow-gate.json');
+    var gatePath = fs.existsSync(localGate) ? localGate : pathUtil.getWorkflowGatePath(process.cwd());
+    if (!fs.existsSync(gatePath)) return;
+
+    var gate = JSON.parse(fs.readFileSync(gatePath, 'utf8'));
+    if (gate.phase === 'implement' || gate.phase === 'bugfix') {
+      findings.push(
+        '[ASC Test Integrity Alert] Test file modified (' + path.basename(filePath) + ') during ' + gate.phase + ' phase. '
+        + 'Ensure assertions were NOT weakened to force a pass; fix the underlying business logic instead.'
+      );
+    }
+  } catch (_) { }
+}
+
 function checkWorkflowGate(toolName, filePath, ext, findings) {
   try {
     var pathUtil = require('./path-util.cjs');
-    var gatePath = pathUtil.getWorkflowGatePath(process.cwd());
+    var localGate = path.join(process.cwd(), 'workflow-gate.json');
+    var gatePath = fs.existsSync(localGate) ? localGate : pathUtil.getWorkflowGatePath(process.cwd());
     if (!fs.existsSync(gatePath)) return;
 
     var gateStr = fs.readFileSync(gatePath, 'utf8');

package/.agents/plugins/agentic-senior-core/hooks.json

@@ -6,8 +6,8 @@
         "hooks": [
           {
             "type": "command",
-            "command": "node -e \"const p=require('path'),fs=require('fs'),os=require('os');const local=p.join(process.cwd(),'.agents','plugins','agentic-senior-core','hooks','session-start.js');const g1=p.join(os.homedir(),'.agents','plugins','agentic-senior-core','hooks','session-start.js');const g2=p.join(os.homedir(),'.gemini','config','plugins','agentic-senior-core','hooks','session-start.js');const g3=p.join(os.homedir(),'.gemini','antigravity-cli','plugins','agentic-senior-core','hooks','session-start.js');require(fs.existsSync(local)?local:(fs.existsSync(g1)?g1:(fs.existsSync(g2)?g2:g3)));\"",
-            "commandWindows": "if (Get-Command node -ErrorAction SilentlyContinue) { $root = if ($env:PLUGIN_ROOT) { $env:PLUGIN_ROOT } else { if ($env:CLAUDE_PLUGIN_ROOT) { $env:CLAUDE_PLUGIN_ROOT } else { if (Test-Path \"$env:USERPROFILE\\.agents\\plugins\\agentic-senior-core\") { \"$env:USERPROFILE\\.agents\\plugins\\agentic-senior-core\" } else { \"$env:USERPROFILE\\.gemini\\config\\plugins\\agentic-senior-core\" } } }; node \"$root\\hooks\\session-start.js\" }",
+            "command": "node hooks/session-start.js",
+            "commandWindows": "node hooks/session-start.js",
             "timeout": 5,
             "statusMessage": "Loading ASC rules..."
           }
@@ -19,8 +19,8 @@
         "hooks": [
           {
             "type": "command",
-            "command": "node -e \"const p=require('path'),fs=require('fs'),os=require('os');const local=p.join(process.cwd(),'.agents','plugins','agentic-senior-core','hooks','subagent-start.js');const g1=p.join(os.homedir(),'.agents','plugins','agentic-senior-core','hooks','subagent-start.js');const g2=p.join(os.homedir(),'.gemini','config','plugins','agentic-senior-core','hooks','subagent-start.js');const g3=p.join(os.homedir(),'.gemini','antigravity-cli','plugins','agentic-senior-core','hooks','subagent-start.js');require(fs.existsSync(local)?local:(fs.existsSync(g1)?g1:(fs.existsSync(g2)?g2:g3)));\"",
-            "commandWindows": "if (Get-Command node -ErrorAction SilentlyContinue) { $root = if ($env:PLUGIN_ROOT) { $env:PLUGIN_ROOT } else { if ($env:CLAUDE_PLUGIN_ROOT) { $env:CLAUDE_PLUGIN_ROOT } else { if (Test-Path \"$env:USERPROFILE\\.agents\\plugins\\agentic-senior-core\") { \"$env:USERPROFILE\\.agents\\plugins\\agentic-senior-core\" } else { \"$env:USERPROFILE\\.gemini\\config\\plugins\\agentic-senior-core\" } } }; node \"$root\\hooks\\subagent-start.js\" }",
+            "command": "node hooks/subagent-start.js",
+            "commandWindows": "node hooks/subagent-start.js",
             "timeout": 5,
             "statusMessage": "Loading ASC rules..."
           }
@@ -34,16 +34,16 @@
           {
             "type": "command",
             "if": "Edit(**/package.json)|Edit(**/requirements.txt)|Edit(**/pyproject.toml)|Edit(**/go.mod)|Edit(**/Cargo.toml)|Edit(**/Gemfile)",
-            "command": "node -e \"const p=require('path'),fs=require('fs'),os=require('os');const local=p.join(process.cwd(),'.agents','plugins','agentic-senior-core','hooks','pre-tool-dependency-gate.js');const g1=p.join(os.homedir(),'.agents','plugins','agentic-senior-core','hooks','pre-tool-dependency-gate.js');const g2=p.join(os.homedir(),'.gemini','config','plugins','agentic-senior-core','hooks','pre-tool-dependency-gate.js');const g3=p.join(os.homedir(),'.gemini','antigravity-cli','plugins','agentic-senior-core','hooks','pre-tool-dependency-gate.js');require(fs.existsSync(local)?local:(fs.existsSync(g1)?g1:(fs.existsSync(g2)?g2:g3)));\"",
-            "commandWindows": "if (Get-Command node -ErrorAction SilentlyContinue) { $root = if ($env:PLUGIN_ROOT) { $env:PLUGIN_ROOT } else { if ($env:CLAUDE_PLUGIN_ROOT) { $env:CLAUDE_PLUGIN_ROOT } else { if (Test-Path \"$env:USERPROFILE\\.agents\\plugins\\agentic-senior-core\") { \"$env:USERPROFILE\\.agents\\plugins\\agentic-senior-core\" } else { \"$env:USERPROFILE\\.gemini\\config\\plugins\\agentic-senior-core\" } } }; node \"$root\\hooks\\pre-tool-dependency-gate.js\" }",
+            "command": "node hooks/pre-tool-dependency-gate.js",
+            "commandWindows": "node hooks/pre-tool-dependency-gate.js",
             "timeout": 5,
             "statusMessage": "ASC Pre-tool dependency check (Edit)..."
           },
           {
             "type": "command",
             "if": "Write(**/package.json)|Write(**/requirements.txt)|Write(**/pyproject.toml)|Write(**/go.mod)|Write(**/Cargo.toml)|Write(**/Gemfile)",
-            "command": "node -e \"const p=require('path'),fs=require('fs'),os=require('os');const local=p.join(process.cwd(),'.agents','plugins','agentic-senior-core','hooks','pre-tool-dependency-gate.js');const g1=p.join(os.homedir(),'.agents','plugins','agentic-senior-core','hooks','pre-tool-dependency-gate.js');const g2=p.join(os.homedir(),'.gemini','config','plugins','agentic-senior-core','hooks','pre-tool-dependency-gate.js');const g3=p.join(os.homedir(),'.gemini','antigravity-cli','plugins','agentic-senior-core','hooks','pre-tool-dependency-gate.js');require(fs.existsSync(local)?local:(fs.existsSync(g1)?g1:(fs.existsSync(g2)?g2:g3)));\"",
-            "commandWindows": "if (Get-Command node -ErrorAction SilentlyContinue) { $root = if ($env:PLUGIN_ROOT) { $env:PLUGIN_ROOT } else { if ($env:CLAUDE_PLUGIN_ROOT) { $env:CLAUDE_PLUGIN_ROOT } else { if (Test-Path \"$env:USERPROFILE\\.agents\\plugins\\agentic-senior-core\") { \"$env:USERPROFILE\\.agents\\plugins\\agentic-senior-core\" } else { \"$env:USERPROFILE\\.gemini\\config\\plugins\\agentic-senior-core\" } } }; node \"$root\\hooks\\pre-tool-dependency-gate.js\" }",
+            "command": "node hooks/pre-tool-dependency-gate.js",
+            "commandWindows": "node hooks/pre-tool-dependency-gate.js",
             "timeout": 5,
             "statusMessage": "ASC Pre-tool dependency check (Write)..."
           }
@@ -54,8 +54,8 @@
         "hooks": [
           {
             "type": "command",
-            "command": "node -e \"const p=require('path'),fs=require('fs'),os=require('os');const local=p.join(process.cwd(),'.agents','plugins','agentic-senior-core','hooks','pre-tool-dependency-gate.js');const g1=p.join(os.homedir(),'.agents','plugins','agentic-senior-core','hooks','pre-tool-dependency-gate.js');const g2=p.join(os.homedir(),'.gemini','config','plugins','agentic-senior-core','hooks','pre-tool-dependency-gate.js');const g3=p.join(os.homedir(),'.gemini','antigravity-cli','plugins','agentic-senior-core','hooks','pre-tool-dependency-gate.js');require(fs.existsSync(local)?local:(fs.existsSync(g1)?g1:(fs.existsSync(g2)?g2:g3)));\"",
-            "commandWindows": "if (Get-Command node -ErrorAction SilentlyContinue) { $root = if ($env:PLUGIN_ROOT) { $env:PLUGIN_ROOT } else { if ($env:CLAUDE_PLUGIN_ROOT) { $env:CLAUDE_PLUGIN_ROOT } else { if (Test-Path \"$env:USERPROFILE\\.agents\\plugins\\agentic-senior-core\") { \"$env:USERPROFILE\\.agents\\plugins\\agentic-senior-core\" } else { \"$env:USERPROFILE\\.gemini\\config\\plugins\\agentic-senior-core\" } } }; node \"$root\\hooks\\pre-tool-dependency-gate.js\" }",
+            "command": "node hooks/pre-tool-dependency-gate.js",
+            "commandWindows": "node hooks/pre-tool-dependency-gate.js",
             "timeout": 5,
             "statusMessage": "ASC Pre-tool dependency check (Terminal)..."
           }
@@ -68,15 +68,15 @@
         "hooks": [
           {
             "type": "command",
-            "command": "node -e \"const p=require('path'),fs=require('fs'),os=require('os');const local=p.join(process.cwd(),'.agents','plugins','agentic-senior-core','hooks','post-edit-enforce.js');const g1=p.join(os.homedir(),'.agents','plugins','agentic-senior-core','hooks','post-edit-enforce.js');const g2=p.join(os.homedir(),'.gemini','config','plugins','agentic-senior-core','hooks','post-edit-enforce.js');const g3=p.join(os.homedir(),'.gemini','antigravity-cli','plugins','agentic-senior-core','hooks','post-edit-enforce.js');require(fs.existsSync(local)?local:(fs.existsSync(g1)?g1:(fs.existsSync(g2)?g2:g3)));\"",
-            "commandWindows": "if (Get-Command node -ErrorAction SilentlyContinue) { $root = if ($env:PLUGIN_ROOT) { $env:PLUGIN_ROOT } else { if ($env:CLAUDE_PLUGIN_ROOT) { $env:CLAUDE_PLUGIN_ROOT } else { if (Test-Path \"$env:USERPROFILE\\.agents\\plugins\\agentic-senior-core\") { \"$env:USERPROFILE\\.agents\\plugins\\agentic-senior-core\" } else { \"$env:USERPROFILE\\.gemini\\config\\plugins\\agentic-senior-core\" } } }; node \"$root\\hooks\\post-edit-enforce.js\" }",
+            "command": "node hooks/post-edit-enforce.js",
+            "commandWindows": "node hooks/post-edit-enforce.js",
             "timeout": 5,
             "statusMessage": "ASC ladder & spec gate check..."
           },
           {
             "type": "command",
-            "command": "node -e \"const p=require('path'),fs=require('fs'),os=require('os');const local=p.join(process.cwd(),'.agents','plugins','agentic-senior-core','hooks','dedup-gate.js');const g1=p.join(os.homedir(),'.agents','plugins','agentic-senior-core','hooks','dedup-gate.js');const g2=p.join(os.homedir(),'.gemini','config','plugins','agentic-senior-core','hooks','dedup-gate.js');const g3=p.join(os.homedir(),'.gemini','antigravity-cli','plugins','agentic-senior-core','hooks','dedup-gate.js');require(fs.existsSync(local)?local:(fs.existsSync(g1)?g1:(fs.existsSync(g2)?g2:g3)));\"",
-            "commandWindows": "if (Get-Command node -ErrorAction SilentlyContinue) { $root = if ($env:PLUGIN_ROOT) { $env:PLUGIN_ROOT } else { if ($env:CLAUDE_PLUGIN_ROOT) { $env:CLAUDE_PLUGIN_ROOT } else { if (Test-Path \"$env:USERPROFILE\\.agents\\plugins\\agentic-senior-core\") { \"$env:USERPROFILE\\.agents\\plugins\\agentic-senior-core\" } else { \"$env:USERPROFILE\\.gemini\\config\\plugins\\agentic-senior-core\" } } }; node \"$root\\hooks\\dedup-gate.js\" }",
+            "command": "node hooks/dedup-gate.js",
+            "commandWindows": "node hooks/dedup-gate.js",
             "timeout": 15,
             "statusMessage": "ASC duplicate-code scan..."
           }
@@ -86,15 +86,15 @@
     "PreInvocation": [
       {
         "type": "command",
-        "command": "node -e \"const p=require('path'),fs=require('fs'),os=require('os');const local=p.join(process.cwd(),'.agents','plugins','agentic-senior-core','hooks','session-pulse.js');const g1=p.join(os.homedir(),'.agents','plugins','agentic-senior-core','hooks','session-pulse.js');const g2=p.join(os.homedir(),'.gemini','config','plugins','agentic-senior-core','hooks','session-pulse.js');const g3=p.join(os.homedir(),'.gemini','antigravity-cli','plugins','agentic-senior-core','hooks','session-pulse.js');require(fs.existsSync(local)?local:(fs.existsSync(g1)?g1:(fs.existsSync(g2)?g2:g3)));\"",
-        "commandWindows": "if (Get-Command node -ErrorAction SilentlyContinue) { $root = if ($env:PLUGIN_ROOT) { $env:PLUGIN_ROOT } else { if ($env:CLAUDE_PLUGIN_ROOT) { $env:CLAUDE_PLUGIN_ROOT } else { if (Test-Path \"$env:USERPROFILE\\.agents\\plugins\\agentic-senior-core\") { \"$env:USERPROFILE\\.agents\\plugins\\agentic-senior-core\" } else { \"$env:USERPROFILE\\.gemini\\config\\plugins\\agentic-senior-core\" } } }; node \"$root\\hooks\\session-pulse.js\" }",
+        "command": "node hooks/session-pulse.js",
+        "commandWindows": "node hooks/session-pulse.js",
         "timeout": 5,
         "statusMessage": "Loading ASC rules..."
       },
       {
         "type": "command",
-        "command": "node -e \"const p=require('path'),fs=require('fs'),os=require('os');const local=p.join(process.cwd(),'.agents','plugins','agentic-senior-core','hooks','ladder-pulse.js');const g1=p.join(os.homedir(),'.agents','plugins','agentic-senior-core','hooks','ladder-pulse.js');const g2=p.join(os.homedir(),'.gemini','config','plugins','agentic-senior-core','hooks','ladder-pulse.js');const g3=p.join(os.homedir(),'.gemini','antigravity-cli','plugins','agentic-senior-core','hooks','ladder-pulse.js');require(fs.existsSync(local)?local:(fs.existsSync(g1)?g1:(fs.existsSync(g2)?g2:g3)));\"",
-        "commandWindows": "if (Get-Command node -ErrorAction SilentlyContinue) { $root = if ($env:PLUGIN_ROOT) { $env:PLUGIN_ROOT } else { if ($env:CLAUDE_PLUGIN_ROOT) { $env:CLAUDE_PLUGIN_ROOT } else { if (Test-Path \"$env:USERPROFILE\\.agents\\plugins\\agentic-senior-core\") { \"$env:USERPROFILE\\.agents\\plugins\\agentic-senior-core\" } else { \"$env:USERPROFILE\\.gemini\\config\\plugins\\agentic-senior-core\" } } }; node \"$root\\hooks\\ladder-pulse.js\" }",
+        "command": "node hooks/ladder-pulse.js",
+        "commandWindows": "node hooks/ladder-pulse.js",
         "timeout": 5,
         "statusMessage": "ASC ladder pulse..."
       }
@@ -102,18 +102,19 @@
     "PostInvocation": [
       {
         "type": "command",
-        "command": "node -e \"const p=require('path'),fs=require('fs'),os=require('os');const local=p.join(process.cwd(),'.agents','plugins','agentic-senior-core','hooks','post-edit-enforce.js');const g1=p.join(os.homedir(),'.agents','plugins','agentic-senior-core','hooks','post-edit-enforce.js');const g2=p.join(os.homedir(),'.gemini','config','plugins','agentic-senior-core','hooks','post-edit-enforce.js');const g3=p.join(os.homedir(),'.gemini','antigravity-cli','plugins','agentic-senior-core','hooks','post-edit-enforce.js');require(fs.existsSync(local)?local:(fs.existsSync(g1)?g1:(fs.existsSync(g2)?g2:g3)));\"",
+        "command": "node hooks/post-edit-enforce.js",
+        "commandWindows": "node hooks/post-edit-enforce.js",
         "timeout": 15
       }
     ],
     "PreCompact": [
       {
         "type": "command",
-        "command": "node -e \"const p=require('path'),fs=require('fs'),os=require('os');const local=p.join(process.cwd(),'.agents','plugins','agentic-senior-core','hooks','pre-compact-pin.js');const g1=p.join(os.homedir(),'.agents','plugins','agentic-senior-core','hooks','pre-compact-pin.js');const g2=p.join(os.homedir(),'.gemini','config','plugins','agentic-senior-core','hooks','pre-compact-pin.js');const g3=p.join(os.homedir(),'.gemini','antigravity-cli','plugins','agentic-senior-core','hooks','pre-compact-pin.js');require(fs.existsSync(local)?local:(fs.existsSync(g1)?g1:(fs.existsSync(g2)?g2:g3)));\"",
-        "commandWindows": "if (Get-Command node -ErrorAction SilentlyContinue) { $root = if ($env:PLUGIN_ROOT) { $env:PLUGIN_ROOT } else { if ($env:CLAUDE_PLUGIN_ROOT) { $env:CLAUDE_PLUGIN_ROOT } else { if (Test-Path \"$env:USERPROFILE\\.agents\\plugins\\agentic-senior-core\") { \"$env:USERPROFILE\\.agents\\plugins\\agentic-senior-core\" } else { \"$env:USERPROFILE\\.gemini\\config\\plugins\\agentic-senior-core\" } } }; node \"$root\\hooks\\pre-compact-pin.js\" }",
+        "command": "node hooks/pre-compact-pin.js",
+        "commandWindows": "node hooks/pre-compact-pin.js",
         "timeout": 5,
         "statusMessage": "ASC constraint pinning..."
       }
     ]
   }
-}
+}

package/.agents/plugins/agentic-senior-core/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "agentic-senior-core",
-  "version": "6.10.0",
+  "version": "6.11.1",
   "description": "Universal AI coding rules. Because your AI writes code like it gets paid by the line.",
   "contextFileName": "rules/agentic-senior-core.md",
   "rules": [

package/.agents/plugins/agentic-senior-core/rules/agentic-senior-core.md

@@ -18,7 +18,7 @@ Before writing code, stop at the lowest step that holds:
 When picking step 5 or 6 (unless trivial):
 - One-line comment noting rationale and upgrade trigger if there is a ceiling.
 - One runnable check (assertion, test, or demo) proving it works.
-- Never simulate success or return hardcoded values mimicking live integrations.
+- Never simulate success or return hardcoded values mimicking live integrations. Temporary stubs must be marked with `// ASC-STUB: [reason] [ASC-DEBT-xxx]` and tracked in the debt ledger.
 
 ## Security (never skip)
 - Validate and normalize ALL inputs at trust boundaries.
@@ -35,6 +35,7 @@ When picking step 5 or 6 (unless trivial):
 - All identifiers (variables, functions, classes, file names) must be in English. No emojis in code or commit messages.
 - Early returns over deep nesting (keep happy path flat).
 - Three similar lines is better than a premature abstraction (duplication over wrong abstraction).
+- Single design source of truth: UI styling must reuse defined design tokens, theme variables, or existing page patterns. No ad-hoc palette, radius, or font deviations between pages.
 - Delete code that carries no behavior, safety, or test value. Readability wins over brevity.
 - Detect and respect project linter/formatter configs. Do not restate style rules enforced by tooling.
 
@@ -46,10 +47,12 @@ When picking step 5 or 6 (unless trivial):
 - Scope and direction changes require explicit user confirmation before modifying abstractions or altering system contracts.
 - Before implementing a feature, locate an analogous module and follow its layer split, naming, and error-handling.
 - Atomic writes must be wrapped in transactions. Flag shared mutable state under concurrent requests.
+- UI component reuse: Component re-invention across pages is forbidden. Before declaring markup or styling, check existing components in `components/` or `ui/`. Compose or extend existing components via props/variants. Do not introduce single-page clones or ad-hoc styling that diverges from established pages.
 
 ## Error Handling & Observability
 - Fail fast on invalid input at trust boundaries. Handle only errors that can actually occur.
 - Structured error responses with safe details (RFC 9457). Distinguish client (4xx) from server (5xx) errors.
+- Frontend data-fetching components must explicitly handle loading, error (toast/notification with retry), and data states.
 - Surface every operational error with context. Empty catch blocks mask production issues.
 - Structured key-value logging for significant events. Never leak stack traces or credentials in production logs.
 
@@ -58,6 +61,7 @@ When picking step 5 or 6 (unless trivial):
 - Never run `git commit`, `git push`, or `git push --force` unless explicitly requested this turn.
 - Testing baseline: new business logic gets one happy-path test and one failure-mode test, unless waived by user.
 - Test quality: Never mock the unit under test — mock only external boundaries/dependencies.
+- Test integrity: Never weaken or delete test assertions to simulate a passing run. Fix the underlying logic.
 - Sycophancy mitigation: State technical objections and trade-offs plainly before implementing. Answer direct questions honestly.
 - Preserve findings and decisions outside chat context. Recommend a fresh context at phase boundaries or after 20-30 tool calls.
 

package/.agents/plugins/agentic-senior-core/skills/asc-add-feature/SKILL.md

@@ -16,7 +16,7 @@ This workflow nudges the agent to stop at each phase boundary, same enforcement
 
 For brownfield feature development (`asc-add-feature`), Phase 2 requires a lightweight **PRD.md** (or feature spec in `docs/PRD.md`) defining product intent, goals, and non-goals to avoid scope creep and context rot.
 
-To track phase, write to `workflow-gate.json` via the `state_write` MCP tool.
+To track phase, write to `workflow-gate.json` at the project root (or via standard file writing tools).
 Format:
 ```json
 {
@@ -30,8 +30,8 @@ Format:
 
 1. Write `workflow-gate.json` with phase `research`.
 2. Map existing code: patterns, utilities, dependencies already in use. Locate at least one analogous feature/module and record its file paths.
-3. Identify what must NOT be rebuilt (e.g., existing validation helpers).
-4. Output a factual research summary that separates what exists from what is proposed, with file paths for the two or three claims that drive the plan.
+3. **UI Component & Design Token Inventory**: For any UI or frontend work, scan existing shared components (e.g. `components/`, `ui/`) and existing page patterns. Explicitly identify what must NOT be rebuilt (buttons, cards, modals, inputs, tables, layout wrappers, validation helpers).
+4. Output a factual research summary that separates what exists from what is proposed, including an explicit `[REUSED COMPONENTS & HELPERS]` list, with file paths for the two or three claims that drive the plan.
 5. **STOP and wait for user approval.** Do not plan or implement.
 
 ## Phase 2: Plan
@@ -40,7 +40,7 @@ Format:
 2. Ensure `docs/PRD.md` or feature brief exists.
 3. Check if `.github/workflows/asc-quality-gate.yml` exists. If not, include scaffolding it in your plan (must run linter, type-check, and audit) and remind the user to enable Branch Protection.
 4. Create a numbered, step-by-step implementation plan with specific files, functions, and line references.
-5. Include a "Don't Build" list from the research phase.
+5. Include a "Don't Build" list and a "Reused Components" list from the research phase. For any proposed new UI component, justify why existing components cannot be reused or extended via props/variants.
 6. **Callout: Plan-Reading Illusion.** Ask the user to explicitly verify the two or three critical plan claims against the referenced files, not just skim it.
 7. Output the plan.
 8. **STOP and wait for user approval.** Do not implement.
@@ -50,5 +50,5 @@ Format:
 1. On approval of Phase 2, update `workflow-gate.json` phase to `implement`.
 2. Recommend a fresh context (intentional compaction) at the phase boundary or after roughly 20-30 tool calls. Do not wait until degradation is subjectively noticeable.
 3. Execute the approved plan.
-4. Validate: tests pass with empirical execution logs outputted, no duplicate code introduced, plan items checked off.
-5. On completion, give a short comprehension summary of what changed and why, then clear the state in `workflow-gate.json` by overwriting it with `{}`.
+4. **Mandatory Verification**: Execute the project test suite (`ascx test`, `npm test`, `pytest`, `go test ./...`, or `cargo test`). The test runner MUST exit with code 0. Do NOT declare completion or weaken test assertions if tests fail.
+5. On verified pass, give a short comprehension summary of what changed and why, then clear the state in `workflow-gate.json` by overwriting it with `{}`.