@boyingliu01/xp-gate 0.18.5 → 0.19.1

package/lib/test-alignment.ts

@@ -198,16 +198,6 @@ export function parseSpecification(specPath: string): SpecificationMap {
 // Step 2: parseTestFiles()
 // ============================================================================
 
-const REQ_ID_PATTERN = /REQ-[A-Z0-9]+-[A-Z0-9]+/;
-
-function extractTag(text: string, tag: string): string | null {
-  // Match @tag value (single line extraction, stops at newline or */)
-  const regex = new RegExp('@' + tag.replace(/[.*+?^${}()|[\]\\]/g, '\\$&') + '\\s+(.+?)(?:\\n|\\*\\/)', 's');
-  const match = text.match(regex);
-  if (!match) return null;
-  return match[1]!.trim();
-}
-
 function extractAllTags(text: string, tag: string): string[] {
   const escapedTag = tag.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
   const regex = new RegExp('@' + escapedTag + '\\s+(.+?)(?:\\n|\\*\\/)', 'gs');
@@ -396,7 +386,7 @@ export function verifyAlignment(
   // Collect all requirement IDs and acceptance criteria IDs from spec
   const allReqIds = new Set<string>();
   const allAcIds = new Set<string>();
-  let totalEdgeCases = 0;
+  const totalEdgeCases = 0;
 
   for (const req of specMap.requirements) {
     allReqIds.add(req.id);
@@ -583,7 +573,7 @@ export function runFullAlignment(options: AlignmentOptions): TestAlignmentReport
   let spec: SpecificationMap;
   try {
     spec = parseSpecification(specPath);
-  } catch (e: any) {
+  } catch (e) {
     return {
       alignment_status: 'BLOCKED',
       phase: 0,
@@ -593,7 +583,7 @@ export function runFullAlignment(options: AlignmentOptions): TestAlignmentReport
       timestamp: new Date().toISOString(),
       misaligned_tests: [],
       anti_pattern_detected: false,
-      errors: [`Failed to parse specification: ${e.message}`],
+      errors: [`Failed to parse specification: ${e instanceof Error ? e.message : String(e)}`],
     };
   }
 

package/lib/update-hooks.js

@@ -3,9 +3,9 @@
  * @intent Sync latest hook versions from xp-gate package to project or global directory
  * @covers AC-265-01, AC-265-02, AC-265-03, AC-265-04
  */
-const fs = require('fs');
-const path = require('path');
-const { GLOBAL_HOOKS_DIR, GLOBAL_ADAPTERS_DIR } = require('./shared-paths.js');
+const fs = require('node:fs');
+const path = require('node:path');
+const { GLOBAL_HOOKS_DIR, GLOBAL_ADAPTERS_DIR, GLOBAL_MODULES_DIR, CONFIG_DIR } = require('./shared-paths.js');
 
 /**
  * Get the xp-gate package root directory (src/npm-package/).
@@ -97,6 +97,17 @@ function detectLocalModifications(srcDir, hooksDestDir, adaptersDestDir) {
     });
   }
 
+  // Check sprint-gate.sh in the hooks directory (Gate MS resolves it there)
+  const sprintGateSrc = path.join(srcDir, 'sprint-gate.sh');
+  const sprintGateDest = path.join(hooksDestDir, 'sprint-gate.sh');
+  if (fs.existsSync(sprintGateDest) && fs.existsSync(sprintGateSrc)) {
+    const srcContent = fs.readFileSync(sprintGateSrc, 'utf8');
+    const destContent = fs.readFileSync(sprintGateDest, 'utf8');
+    if (srcContent !== destContent) {
+      modified.push('sprint-gate.sh');
+    }
+  }
+
   return modified;
 }
 
@@ -152,6 +163,21 @@ function copyHooks(srcDir, destDir, dryRun, noBackup) {
     const dest = path.join(destDir, hook);
     atomicCopyFile(src, dest, dryRun, noBackup, hook);
   });
+  const libSrcDir = path.join(hooksSrcDir, 'lib');
+  if (fs.existsSync(libSrcDir)) {
+    fs.readdirSync(libSrcDir, { withFileTypes: true })
+      .filter(entry => entry.isFile())
+      .forEach(entry => {
+        const label = `lib/${entry.name}`;
+        atomicCopyFile(
+          path.join(libSrcDir, entry.name),
+          path.join(destDir, label),
+          dryRun,
+          noBackup,
+          label
+        );
+      });
+  }
 }
 
 /**
@@ -190,7 +216,7 @@ function copyAdapters(srcDir, destDir, dryRun, noBackup) {
 function copyGateScripts(srcDir, destDir, dryRun, noBackup) {
   if (!fs.existsSync(srcDir)) return;
   fs.readdirSync(srcDir).forEach(f => {
-    if ((f.startsWith('gate-') || f === 'sprint-gate.sh') && f.endsWith('.sh')) {
+    if (f.startsWith('gate-') && f.endsWith('.sh')) {
       const src = path.join(srcDir, f);
       const dest = path.join(destDir, f);
       atomicCopyFile(src, dest, dryRun, noBackup, f);
@@ -198,31 +224,92 @@ function copyGateScripts(srcDir, destDir, dryRun, noBackup) {
   });
 }
 
+/**
+ * Copy sprint-gate.sh from package root to the hooks directory.
+ * The pre-push Gate MS resolves it at $(dirname "$0")/sprint-gate.sh, i.e.
+ * alongside pre-commit/pre-push — NOT the adapters directory.
+ * @param {string} srcDir - Package root directory
+ * @param {string} hooksDestDir - Destination hooks directory
+ * @param {boolean} dryRun
+ * @param {boolean} noBackup
+ */
+function copySprintGate(srcDir, hooksDestDir, dryRun, noBackup) {
+  const src = path.join(srcDir, 'sprint-gate.sh');
+  if (!fs.existsSync(src)) return;
+  atomicCopyFile(src, path.join(hooksDestDir, 'sprint-gate.sh'), dryRun, noBackup, 'sprint-gate.sh');
+}
+
+/**
+ * Copy quality gate modules (principles, mutation, mock-policy, build-integrity)
+ * into the modules directory. These ship in the npm package but were never
+ * installed by an already-initialized global setup, so update-hooks must
+ * re-deploy them to repair Gate 9/10 (build-integrity) on existing installs.
+ * @param {string} srcDir - Package root directory
+ * @param {string} modulesDestDir - Destination modules directory
+ * @param {boolean} dryRun
+ * @param {boolean} noBackup
+ */
+const MODULES = ['principles', 'mutation', 'mock-policy', 'build-integrity'];
+function copyModules(srcDir, modulesDestDir, dryRun, noBackup) {
+  MODULES.forEach(module => {
+    const srcDirPath = path.join(srcDir, module);
+    if (!fs.existsSync(srcDirPath)) return;
+    const destDirPath = path.join(modulesDestDir, module);
+    fs.mkdirSync(destDirPath, { recursive: true });
+    fs.readdirSync(srcDirPath).forEach(entry => {
+      const srcEntry = path.join(srcDirPath, entry);
+      const destEntry = path.join(destDirPath, entry);
+      const stats = fs.statSync(srcEntry);
+      if (stats.isDirectory()) {
+        copyRecursiveDir(srcEntry, destEntry, dryRun, noBackup, module);
+      } else if (!dryRun) {
+        atomicCopyFile(srcEntry, destEntry, dryRun, noBackup, `${module}/${entry}`);
+      }
+    });
+    if (!dryRun) console.log(`  ${module}/ -> ${modulesDestDir}/${module}/`);
+  });
+}
+
+function copyRecursiveDir(src, dest, dryRun, noBackup, label) {
+  fs.mkdirSync(dest, { recursive: true });
+  fs.readdirSync(src).forEach(entry => {
+    const srcEntry = path.join(src, entry);
+    const destEntry = path.join(dest, entry);
+    if (fs.statSync(srcEntry).isDirectory()) {
+      copyRecursiveDir(srcEntry, destEntry, dryRun, noBackup, label);
+    } else {
+      atomicCopyFile(srcEntry, destEntry, dryRun, noBackup, `${label}/${entry}`);
+    }
+  });
+}
+
 function resolveSrcDir() {
-  return (module.exports && module.exports.getPackageRoot)
+  return module.exports?.getPackageRoot
     ? module.exports.getPackageRoot()
     : getPackageRoot();
 }
 
 function resolveDirs(global) {
-  if (global) return { hooksDestDir: GLOBAL_HOOKS_DIR, adaptersDestDir: GLOBAL_ADAPTERS_DIR };
-  return { hooksDestDir: getProjectHooksDir(), adaptersDestDir: path.join(process.cwd(), 'githooks') };
+  if (global) return { hooksDestDir: GLOBAL_HOOKS_DIR, adaptersDestDir: GLOBAL_ADAPTERS_DIR, modulesDestDir: GLOBAL_MODULES_DIR };
+  return { hooksDestDir: getProjectHooksDir(), adaptersDestDir: path.join(process.cwd(), 'githooks'), modulesDestDir: path.join(process.cwd(), '.xp-gate', 'modules') };
 }
 
-function ensureDirsExist(adaptersDestDir, hooksDestDir) {
+function ensureDirsExist(adaptersDestDir, hooksDestDir, modulesDestDir) {
   fs.mkdirSync(hooksDestDir, { recursive: true });
   fs.mkdirSync(adaptersDestDir, { recursive: true });
   fs.mkdirSync(path.join(adaptersDestDir, 'adapters'), { recursive: true });
+  if (modulesDestDir) fs.mkdirSync(modulesDestDir, { recursive: true });
 }
 
 function printUpdateHeader(opts) {
-  const { global, srcDir, hooksDestDir, adaptersDestDir, scope, dryRun } = opts;
+  const { global, srcDir, hooksDestDir, adaptersDestDir, modulesDestDir, scope, dryRun } = opts;
   console.log(`XP-Gate Update Hooks`);
   console.log(`====================`);
   console.log(`Mode: ${global ? 'Global' : 'Local'}`);
   console.log(`Source: ${srcDir}`);
   console.log(`Hooks destination: ${hooksDestDir}`);
   console.log(`Adapters destination: ${adaptersDestDir}`);
+  if (modulesDestDir) console.log(`Modules destination: ${modulesDestDir}`);
   console.log(`Scope: ${scope}`);
   if (dryRun) console.log(`Dry run: yes (no files will be modified)`);
   console.log('');
@@ -233,13 +320,15 @@ function checkLocalModifications(srcDir, hooksDestDir, adaptersDestDir, force, d
   const localMods = detectLocalModifications(srcDir, hooksDestDir, adaptersDestDir);
   if (localMods.length === 0) return 0;
   console.warn(`[WARN] Detected ${localMods.length} locally modified file(s):`);
-  localMods.forEach(f => console.warn(`  - ${f}`));
+  localMods.forEach(f => {
+    console.warn(`  - ${f}`);
+  });
   console.warn('Use --force to overwrite, or manually backup first.');
   return 1;
 }
 
 function copyByScope(opts) {
-  const { scope, srcDir, hooksDestDir, adaptersDestDir, dryRun, noBackup } = opts;
+  const { scope, srcDir, hooksDestDir, adaptersDestDir, modulesDestDir, dryRun, noBackup } = opts;
   if (scope === 'all' || scope === 'hooks') {
     printInfo('hooks');
     copyHooks(srcDir, hooksDestDir, dryRun, noBackup);
@@ -251,6 +340,27 @@ function copyByScope(opts) {
   if (scope === 'all') {
     printInfo('gate scripts');
     copyGateScripts(srcDir, adaptersDestDir, dryRun, noBackup);
+    if (modulesDestDir) {
+      printInfo('modules');
+      copyModules(srcDir, modulesDestDir, dryRun, noBackup);
+    }
+  }
+  if (scope === 'all' || scope === 'hooks') {
+    copySprintGate(srcDir, hooksDestDir, dryRun, noBackup);
+  }
+}
+
+function warnMissingModuleDeps(global, dryRun) {
+  if (!global || dryRun) return;
+  // Copied gate modules (e.g. mock-policy) import js-yaml/zod installed into
+  // <CONFIG_DIR>/node_modules by setup-global. If absent (e.g. an install made
+  // before that step existed), Gate ML would crash at push — guide the user to
+  // re-run setup-global rather than silently failing.
+  const depsDir = path.join(CONFIG_DIR, 'node_modules', 'js-yaml');
+  if (!fs.existsSync(depsDir)) {
+    console.warn('\n[warn] Global quality-gate module runtime deps not found.');
+    console.warn(`  Expected: ${path.join(CONFIG_DIR, 'node_modules')}`);
+    console.warn('  Run `xp-gate setup-global` once to install them (Gate ML needs js-yaml/zod).');
   }
 }
 
@@ -262,15 +372,16 @@ function printInfo(label) { console.log(`Updating ${label}...`); }
 function updateHooks(options = {}) {
   const { global = false, force = false, dryRun = false, noBackup = false, scope = 'all' } = options;
   const srcDir = resolveSrcDir();
-  const { hooksDestDir, adaptersDestDir } = resolveDirs(global);
+  const { hooksDestDir, adaptersDestDir, modulesDestDir } = resolveDirs(global);
 
-  ensureDirsExist(adaptersDestDir, hooksDestDir);
-  printUpdateHeader({ global, srcDir, hooksDestDir, adaptersDestDir, scope, dryRun });
+  ensureDirsExist(adaptersDestDir, hooksDestDir, modulesDestDir);
+  printUpdateHeader({ global, srcDir, hooksDestDir, adaptersDestDir, modulesDestDir, scope, dryRun });
 
   const modCheck = checkLocalModifications(srcDir, hooksDestDir, adaptersDestDir, force, dryRun);
   if (modCheck !== 0) return modCheck;
 
-  copyByScope({ scope, srcDir, hooksDestDir, adaptersDestDir, dryRun, noBackup });
+  copyByScope({ scope, srcDir, hooksDestDir, adaptersDestDir, modulesDestDir, dryRun, noBackup });
+  warnMissingModuleDeps(global, dryRun);
 
   if (!dryRun) console.log('\nUpdate complete!');
   return 0;
@@ -282,6 +393,9 @@ module.exports = {
   copyHooks,
   copyAdapters,
   copyGateScripts,
+  copySprintGate,
+  copyModules,
+  warnMissingModuleDeps,
   atomicCopyFile,
   getPackageRoot,
   getProjectHooksDir,

package/mock-policy/AGENTS.md

@@ -1,9 +1,9 @@
 # SRC/MOCK-POLICY KNOWLEDGE BASE
 
-**Generated:** 2026-07-29
-**Commit:** 05d23ae
+**Generated:** 2026-08-30
+**Commit:** 0848858
 **Branch:** main
-**Version:** 0.18.5.0
+**Version:** 0.19.1.0
 
 ## OVERVIEW
 Mock layering policy enforcement — Gate M3 of pre-push hook. Ensures integration tests use real implementations for internal dependencies, mock external dependencies, and annotate pending mocks with removal plans. Combines project scope scanning, mock decision engine, and per-file validation into a single pipeline.
@@ -88,7 +88,7 @@ npx vitest run src/mock-policy/__tests__/integration.test.ts
 
 ## NOTES
 - Gate M3 is invoked by pre-push hook after Gate M (mutation) and Gate M2 (AI test detection)
-- push limits: max 20 files or 500 LOC per push — Gate M3 respects this via the changed files input
+- code walkthrough scope: no hard file-count or LOC threshold; large diffs require complete review or user-directed splitting, never a size-based bypass
 - Coverage exclude: src/mock-policy/ controlled by vitest config
 - The integration test creates real temporary directories under os.tmpdir()
 - Dependencies on: `src/mutation/detect-ai-test.ts` (detectTestLayer)

package/mock-policy/__tests__/gate-m3.test.ts

@@ -271,9 +271,7 @@ describe('runGateM3', () => {
 // ---------------------------------------------------------------------------
 describe('main', () => {
   let tmpDir: string;
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   let consoleErrorSpy: any;
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   let consoleLogSpy: any;
 
   beforeEach(() => {
@@ -335,7 +333,6 @@ describe('main', () => {
     const exitCode = await main([testFile]);
 
     const logCalls: string[] = consoleLogSpy.mock.calls.map(
-      // eslint-disable-next-line @typescript-eslint/no-explicit-any
       (c: any[]) => String(c[0]),
     );
     expect(logCalls.some((msg) => msg.includes('Gate M3'))).toBe(true);
@@ -357,7 +354,6 @@ describe('main', () => {
     await main([testFile]);
 
     const logCalls: string[] = consoleLogSpy.mock.calls.map(
-      // eslint-disable-next-line @typescript-eslint/no-explicit-any
       (c: any[]) => String(c[0]),
     );
     const hasPassLabel = logCalls.some(
@@ -373,7 +369,6 @@ describe('main', () => {
     await main([testFile]);
 
     const logCalls: string[] = consoleLogSpy.mock.calls.map(
-      // eslint-disable-next-line @typescript-eslint/no-explicit-any
       (c: any[]) => String(c[0]),
     );
     const hasFileCount = logCalls.some(

package/mock-policy/__tests__/scope-scanner.test.ts

@@ -196,9 +196,12 @@ describe('resolveToRealPath', () => {
 // ---------------------------------------------------------------------------
 describe('loadExternalDependencies', () => {
   let tmpDir: string;
+  let tmpCounter = 0;
 
   beforeEach(() => {
-    tmpDir = join(tmpdir(), `scope-scanner-test-${Date.now()}`);
+    // Monotonic counter avoids Date.now() collisions when tests run fast
+    tmpCounter += 1;
+    tmpDir = join(tmpdir(), `scope-scanner-test-${process.pid}-${tmpCounter}`);
     mkdirSync(tmpDir, { recursive: true });
   });
 
@@ -361,9 +364,12 @@ describe('classifyDependency', () => {
 // ---------------------------------------------------------------------------
 describe('scanProjectScope', () => {
   let tmpDir: string;
+  let tmpCounter = 0;
 
   beforeEach(() => {
-    tmpDir = join(tmpdir(), `scope-scanner-integration-${Date.now()}`);
+    // Monotonic counter avoids Date.now() collisions when tests run fast
+    tmpCounter += 1;
+    tmpDir = join(tmpdir(), `scope-scanner-integration-${process.pid}-${tmpCounter}`);
     mkdirSync(tmpDir, { recursive: true });
 
     // Create a real project structure

package/mutation/AGENTS.md

@@ -1,9 +1,9 @@
 # SRC/MUTATION KNOWLEDGE BASE
 
-**Generated:** 2026-07-29
-**Commit:** 05d23ae
+**Generated:** 2026-08-30
+**Commit:** 0848858
 **Branch:** main
-**Version:** 0.18.5.0
+**Version:** 0.19.1.0
 
 ## OVERVIEW
 **Gate M** (incremental mutation testing) + **Gate M2** helpers (test-layer detection used by `src/mock-policy/`). Pre-push quality gate. TypeScript-only; uses Stryker.