docguard-cli 0.9.10 → 0.9.11

package/cli/commands/score.mjs

@@ -494,8 +494,30 @@ function calcTestingScore(dir, config) {
   }
 
   // ── Check 4: CI test step (15 pts) ──
-  const ciFiles = ['.github/workflows/ci.yml', '.github/workflows/test.yml'];
-  const hasCITest = ciFiles.some(f => existsSync(resolve(dir, f)));
+  // Support multiple CI systems — not just GitHub Actions
+  const ciFiles = [
+    '.github/workflows/ci.yml', '.github/workflows/test.yml',
+    '.github/workflows/ci.yaml', '.github/workflows/test.yaml',
+    'buildspec.yml', 'buildspec.test.yml',     // AWS CodeBuild
+    'amplify.yml',                              // AWS Amplify
+    'Jenkinsfile',                              // Jenkins
+    '.circleci/config.yml',                     // CircleCI
+    '.gitlab-ci.yml',                           // GitLab CI
+    '.travis.yml',                              // Travis CI
+  ];
+  let hasCITest = ciFiles.some(f => existsSync(resolve(dir, f)));
+
+  // Also check turbo.json for "test" pipeline task
+  if (!hasCITest) {
+    const turboPath = resolve(dir, 'turbo.json');
+    if (existsSync(turboPath)) {
+      try {
+        const turboContent = readFileSync(turboPath, 'utf-8');
+        if (/"test"/.test(turboContent)) hasCITest = true;
+      } catch { /* skip */ }
+    }
+  }
+
   if (hasCITest) score += 15;
 
   return Math.min(100, score);

package/cli/shared-ignore.mjs

@@ -74,3 +74,46 @@ export function shouldIgnore(relPath, config, validatorKey) {
 
   return false;
 }
+
+/**
+ * Convert a glob pattern to a RegExp for POSITIVE matching.
+ * Unlike globToRegex (used for ignore filtering), this anchors the match
+ * to the full relative path from the project root.
+ *
+ * Supports: * (any chars except /), ** (any path segments), . (literal dot).
+ *
+ * @param {string} pattern - Glob pattern (e.g., "backend/**\/__tests__/**\/*.test.ts")
+ * @returns {RegExp}
+ */
+function globToMatchRegex(pattern) {
+  // Normalize: replace **/ with a placeholder that means "zero or more path segments"
+  let escaped = pattern
+    .replace(/\./g, '\\.')
+    .replace(/\*\*\//g, '§STARSTAR§')   // **/ → zero-or-more segments
+    .replace(/\*\*/g, '.*')             // standalone ** → any chars
+    .replace(/\*/g, '[^/]*')            // single * → any chars except /
+    .replace(/§STARSTAR§/g, '(.*/)?');  // **/ → optional path prefix
+  return new RegExp(`^${escaped}$`);
+}
+
+/**
+ * Check if a relative path matches ANY of the given glob patterns.
+ * Purpose-built for POSITIVE matching (e.g., "is this a test file?").
+ *
+ * ALWAYS rejects paths containing node_modules at any depth.
+ * This is the correct function for test file discovery — do NOT use
+ * buildIgnoreFilter() for this purpose.
+ *
+ * @param {string} relPath - Relative path from project root
+ * @param {string[]} patterns - Array of glob patterns to match against
+ * @returns {boolean} - true if path matches a pattern AND is not in node_modules
+ */
+export function globMatch(relPath, patterns) {
+  if (!relPath || !patterns || patterns.length === 0) return false;
+
+  // Always reject paths containing node_modules at any depth
+  if (/(?:^|[/\\])node_modules(?:[/\\]|$)/.test(relPath)) return false;
+
+  const regexes = patterns.map(p => globToMatchRegex(p));
+  return regexes.some(r => r.test(relPath));
+}

package/cli/validators/docs-diff.mjs

@@ -11,7 +11,7 @@
 
 import { existsSync, readFileSync, readdirSync, statSync } from 'node:fs';
 import { resolve, join, extname, basename, relative } from 'node:path';
-import { shouldIgnore, buildIgnoreFilter } from '../shared-ignore.mjs';
+import { shouldIgnore, globMatch } from '../shared-ignore.mjs';
 
 const IGNORE_DIRS = new Set([
   'node_modules', '.git', '.next', 'dist', 'build',
@@ -139,7 +139,7 @@ function diffEnvVars(dir) {
  * Diff test files between TEST-SPEC.md and actual code.
  * Uses config.testPatterns if available, otherwise falls back to
  * scanning standard test directories.
- * Always ignores node_modules via shared ignore filter.
+ * Always ignores node_modules via globMatch().
  */
 function diffTests(dir, config) {
   const testSpecPath = resolve(dir, 'docs-canonical/TEST-SPEC.md');
@@ -153,17 +153,12 @@ function diffTests(dir, config) {
     docTests.add(match[1]);
   }
 
+  // Collect test files from disk using globMatch (always excludes node_modules)
   const codeTests = new Set();
-
-  // Use testPatterns from config if available
   const testPatterns = config?.testPatterns || [];
+
   if (testPatterns.length > 0) {
-    // Use configured patterns to find test files
-    const patternFilter = buildIgnoreFilter(testPatterns.map(p => {
-      // Invert the pattern: we WANT files matching these patterns
-      return p;
-    }));
-    // Walk the project and collect matching test files
+    // Use configured patterns — globMatch handles node_modules exclusion
     const allTestFiles = getTestFilesFromPatterns(dir, testPatterns, config);
     for (const f of allTestFiles) {
       codeTests.add(f);
@@ -192,17 +187,18 @@ function diffTests(dir, config) {
 
 /**
  * Find test files matching configured testPatterns.
- * Walks the project tree, skipping node_modules and ignored dirs.
+ * Uses globMatch() for pattern matching — always excludes node_modules.
+ * Results are deduplicated via Set (handles overlapping patterns).
  */
 function getTestFilesFromPatterns(dir, patterns, config) {
-  const results = [];
-  const testFileRegex = /\.(test|spec)\.(mjs|cjs|[jt]sx?)$/;
+  const results = new Set();
 
   function walk(currentDir) {
     let entries;
     try { entries = readdirSync(currentDir); } catch { return; }
 
     for (const entry of entries) {
+      // Skip node_modules and other ignored dirs at directory level (fast path)
       if (IGNORE_DIRS.has(entry) || entry.startsWith('.')) continue;
       const fullPath = join(currentDir, entry);
       try {
@@ -211,15 +207,11 @@ function getTestFilesFromPatterns(dir, patterns, config) {
           walk(fullPath);
         } else if (stat.isFile()) {
           const relPath = relative(dir, fullPath);
-          // Skip files in ignored paths
+          // Skip files in globally ignored paths
           if (config && shouldIgnore(relPath, config)) continue;
-          // Check if it matches test file naming patterns
-          if (testFileRegex.test(entry) || /__(tests|test)__/.test(relPath)) {
-            // Check if it matches any of the configured test patterns
-            const patternFilter = buildIgnoreFilter(patterns);
-            if (patternFilter(relPath)) {
-              results.push(relPath);
-            }
+          // Use globMatch for positive pattern matching (rejects node_modules internally)
+          if (globMatch(relPath, patterns)) {
+            results.add(relPath);
           }
         }
       } catch { /* skip */ }
@@ -227,7 +219,7 @@ function getTestFilesFromPatterns(dir, patterns, config) {
   }
 
   walk(dir);
-  return results;
+  return [...results];
 }
 
 function getFilesRecursive(dir, config) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "docguard-cli",
-  "version": "0.9.10",
+  "version": "0.9.11",
   "description": "The enforcement tool for Canonical-Driven Development (CDD). Audit, generate, and guard your project documentation.",
   "type": "module",
   "bin": {