@keber/qa-framework 1.12.0 → 1.13.0

package/scripts/lib/claude-agents.js

@@ -7,22 +7,71 @@
  * qa-framework.config.json. Claude Code only - subagents with per-agent tools/model
  * frontmatter have no GitHub Copilot equivalent, so no .github/ counterpart is generated.
  *
- * Placeholders are resolved from config.integrations.azureDevOps.sprintCycle, falling back
- * to neutral defaults when a field (or the whole config section) is absent.
+ * Placeholders are resolved from the project config, falling back to neutral defaults
+ * when a field (or the whole config section) is absent.
+ *
+ * Template syntax:
+ *   {{NAME}}                         placeholder, replaced in a single pass (values are
+ *                                    inserted verbatim and never re-expanded)
+ *   {{#if FLAG}}...{{/if}}           kept only when FLAG is true (not nestable)
+ *   {{#unless FLAG}}...{{/unless}}   kept only when FLAG is false (not nestable)
+ *
+ * Any {{...}} token left after rendering is a template bug and throws.
+ *
+ * buildAgentContent() throws per agent, so callers can generate the agents that are
+ * valid, report the one that is not, and exit non-zero.
  */
 
 const fs = require('fs');
 const path = require('path');
 
-const AGENT_NAMES = ['qa-analisis', 'qa-plan', 'qa-asesoria', 'qa-informe-resultados'];
+const { laneTableFromConfig, DEFAULT_LANE_TABLE } = require('../../templates/automation-scaffold/scripts/lane-config');
+
+const AGENT_NAMES = [
+  'qa-analisis',
+  'qa-plan',
+  'qa-asesoria',
+  'qa-informe-resultados',
+  'qa-reporte-ejecucion',
+  'qa-investigador',
+];
 
 const DEFAULT_SPRINT_CYCLE = {
   sprintDurationDays: 8,
   manualTestingTimeboxDays: 2,
   dateFormat: 'dd-mm-aaaa',
   timezone: 'America/Santiago',
+  approvalThresholdPercent: 85,
 };
 
+const DEFAULT_TC_ID_PATTERN = 'TC-{MODULE}-{SUBMODULE}-{NUM}';
+const NOT_CONFIGURED = '(no configurado: ver qa/qa-framework.config.json)';
+
+const MCP_NAMESPACE_PATTERN = /^mcp__[A-Za-z0-9_-]+__$/;
+const DEFECT_ID_PREFIX_PATTERN = /^[A-Z0-9-]+$/;
+const DECIMAL_SEPARATORS = [',', '.'];
+
+// Playwright MCP tools granted to qa-investigador per lane namespace, in this order.
+const BROWSER_TOOLS = [
+  'browser_navigate',
+  'browser_snapshot',
+  'browser_click',
+  'browser_type',
+  'browser_fill_form',
+  'browser_select_option',
+  'browser_hover',
+  'browser_press_key',
+  'browser_wait_for',
+  'browser_evaluate',
+  'browser_network_requests',
+  'browser_console_messages',
+  'browser_take_screenshot',
+  'browser_find',
+  'browser_close',
+];
+
+const INVESTIGATOR_BASE_TOOLS = ['Read', 'Write', 'Edit', 'Grep', 'Glob', 'Bash'];
+
 const LOCALE_LANGUAGE_LABELS = {
   es: 'espanol',
   'es-cl': 'espanol de Chile',
@@ -31,6 +80,10 @@ const LOCALE_LANGUAGE_LABELS = {
   en: 'English',
 };
 
+// Tokens whose value depends on the lane table. They are resolved only for templates
+// that use them, so an invalid lane config fails qa-investigador and nothing else.
+const LANE_TOKENS = ['{{INVESTIGATOR_TOOLS}}', '{{LANE_MCP_TABLE}}', 'HAS_MCP_LANES'];
+
 function isSprintCycleEnabled(config) {
   return Boolean(config?.integrations?.azureDevOps?.sprintCycle?.enabled);
 }
@@ -40,9 +93,98 @@ function resolveLocaleLanguageLabel(locale) {
   return LOCALE_LANGUAGE_LABELS[locale.toLowerCase()] ?? locale;
 }
 
+function isValidApprovalThreshold(value) {
+  return typeof value === 'number' && Number.isFinite(value) && value > 0 && value < 100;
+}
+
+function resolveApprovalThresholdPercent(sprintCycle) {
+  const value = sprintCycle.approvalThresholdPercent;
+  // Invalid values are reported by `qa-framework validate`; generation keeps the default.
+  return isValidApprovalThreshold(value) ? value : DEFAULT_SPRINT_CYCLE.approvalThresholdPercent;
+}
+
+function formatRatio(percent) {
+  // Rounded to strip binary float noise (e.g. 57 / 100), always with a dot.
+  return String(Number((percent / 100).toFixed(6)));
+}
+
+function resolveDecimalSeparator(config) {
+  const explicit = config?.conventions?.decimalSeparator;
+  if (DECIMAL_SEPARATORS.includes(explicit)) return explicit;
+  const locale = config?.conventions?.locale ?? config?.conventions?.language;
+  if (!locale) return ',';
+  return String(locale).toLowerCase().startsWith('es') ? ',' : '.';
+}
+
+function resolveDefectIdPrefix(config) {
+  const explicit = config?.conventions?.defectIdPrefix;
+  if (typeof explicit === 'string' && DEFECT_ID_PREFIX_PATTERN.test(explicit)) return explicit;
+  return config?.project?.name ? config.project.name.toUpperCase().replace(/[^A-Z0-9]/g, '-') : 'PROJ';
+}
+
+function resolveProjectField(value) {
+  if (typeof value !== 'string' || value.trim() === '' || value.includes('{{')) return NOT_CONFIGURED;
+  return value;
+}
+
+/**
+ * MCP-capable lanes in config order: [{ id, mcpNamespace }].
+ *
+ * Validation is delegated to laneTableFromConfig (the same reader the automation
+ * scaffold uses). Order comes from the config array, not from the table object,
+ * because numeric-like lane ids would otherwise be reordered by the JS engine.
+ * Throws on an invalid lane block or an mcpNamespace that is not a tool prefix.
+ */
+function resolveMcpLanes(config) {
+  const table = laneTableFromConfig(config);
+  if (table === DEFAULT_LANE_TABLE) return [];
+
+  const lanes = [];
+  for (const [index, entry] of config.parallelLanes.lanes.entries()) {
+    const lane = table[entry.id];
+    if (!lane || lane.mcpNamespace === null) continue;
+    if (!MCP_NAMESPACE_PATTERN.test(lane.mcpNamespace)) {
+      throw new Error(
+        `parallelLanes.lanes[${index}].mcpNamespace must match ${MCP_NAMESPACE_PATTERN} ` +
+          `(e.g. "mcp__playwright-qauser__"). Got: ${JSON.stringify(lane.mcpNamespace)}`
+      );
+    }
+    lanes.push({ id: entry.id, mcpNamespace: lane.mcpNamespace });
+  }
+  return lanes;
+}
+
+function buildLanePlaceholders(config) {
+  const mcpLanes = resolveMcpLanes(config);
+
+  const tools = [...INVESTIGATOR_BASE_TOOLS];
+  for (const { mcpNamespace } of mcpLanes) {
+    for (const tool of BROWSER_TOOLS) tools.push(`${mcpNamespace}${tool}`);
+  }
+
+  const tableLines = mcpLanes.length
+    ? [
+        '| Lane | Servidor MCP | Prefijo de herramientas |',
+        '| --- | --- | --- |',
+        ...mcpLanes.map(({ id, mcpNamespace }) =>
+          `| ${id} | ${mcpNamespace.replace(/^mcp__/, '').replace(/__$/, '')} | \`${mcpNamespace}\` |`
+        ),
+      ]
+    : [];
+
+  return {
+    placeholders: {
+      '{{INVESTIGATOR_TOOLS}}': tools.join(', '),
+      '{{LANE_MCP_TABLE}}': tableLines.join('\n'),
+    },
+    flags: { HAS_MCP_LANES: mcpLanes.length > 0 },
+  };
+}
+
 function buildPlaceholders(config) {
   const sprintCycle = { ...DEFAULT_SPRINT_CYCLE, ...(config?.integrations?.azureDevOps?.sprintCycle ?? {}) };
   const project = config?.project ?? {};
+  const approvalPercent = resolveApprovalThresholdPercent(sprintCycle);
 
   return {
     '{{PROJECT_NAME}}': project.name ?? '{{PROJECT_NAME}}',
@@ -52,18 +194,138 @@ function buildPlaceholders(config) {
     '{{DATE_FORMAT}}': sprintCycle.dateFormat,
     '{{TIMEZONE}}': sprintCycle.timezone,
     '{{LOCALE_LANGUAGE_LABEL}}': resolveLocaleLanguageLabel(config?.conventions?.locale ?? config?.conventions?.language),
-    '{{DEFECT_ID_PREFIX}}': config?.project?.name ? config.project.name.toUpperCase().replace(/[^A-Z0-9]/g, '-') : 'PROJ',
+    '{{DEFECT_ID_PREFIX}}': resolveDefectIdPrefix(config),
+    '{{APPROVAL_THRESHOLD_PERCENT}}': String(approvalPercent).replace('.', resolveDecimalSeparator(config)),
+    '{{APPROVAL_THRESHOLD_RATIO}}': formatRatio(approvalPercent),
+    '{{DECIMAL_SEPARATOR}}': resolveDecimalSeparator(config),
+    '{{TC_ID_PATTERN}}': config?.conventions?.testCaseIdPattern || DEFAULT_TC_ID_PATTERN,
+    '{{QA_BASE_URL}}': resolveProjectField(project.qaBaseUrl),
+    '{{LOGIN_PATH}}': resolveProjectField(project.loginPath),
+    '{{TECH_STACK}}': resolveProjectField(project.techStack),
   };
 }
 
+/**
+ * Renders conditional blocks, then placeholders, then guards against leftovers.
+ *
+ * A block whose opening and closing tags sit on their own lines is removed together
+ * with those lines, so dropping a block never leaves an extra blank line behind.
+ */
+function renderTemplate(template, placeholders, flags, agentName) {
+  let content = template;
+
+  for (const [keyword, keepWhen] of [['if', true], ['unless', false]]) {
+    const ownLine = new RegExp(
+      `^[ \\t]*\\{\\{#${keyword} ([A-Z0-9_]+)\\}\\}[ \\t]*\\r?\\n([\\s\\S]*?)^[ \\t]*\\{\\{\\/${keyword}\\}\\}[ \\t]*(?:\\r?\\n|$)`,
+      'gm'
+    );
+    const inline = new RegExp(`\\{\\{#${keyword} ([A-Z0-9_]+)\\}\\}([\\s\\S]*?)\\{\\{\\/${keyword}\\}\\}`, 'g');
+    const choose = (match, flag, body) => (Boolean(flags[flag]) === keepWhen ? body : '');
+    content = content.replace(ownLine, choose).replace(inline, choose);
+  }
+
+  const leftovers = new Set();
+  content = content.replace(/\{\{[#/]?[A-Za-z0-9_ ]+\}\}/g, (token) => {
+    if (Object.prototype.hasOwnProperty.call(placeholders, token)) return placeholders[token];
+    leftovers.add(token);
+    return token;
+  });
+
+  if (leftovers.size) {
+    throw new Error(
+      `Template for agent ${agentName} has unresolved tokens: ${[...leftovers].join(', ')}`
+    );
+  }
+  return content;
+}
+
 function buildAgentContent(agentName, templatesDir, config) {
   const templatePath = path.join(templatesDir, 'agents', `${agentName}.md`);
-  let content = fs.readFileSync(templatePath, 'utf8');
-  const placeholders = buildPlaceholders(config);
-  for (const [placeholder, value] of Object.entries(placeholders)) {
-    content = content.split(placeholder).join(value);
+  try {
+    const template = fs.readFileSync(templatePath, 'utf8');
+    const placeholders = buildPlaceholders(config);
+    let flags = {};
+    if (LANE_TOKENS.some((token) => template.includes(token))) {
+      const lane = buildLanePlaceholders(config);
+      Object.assign(placeholders, lane.placeholders);
+      flags = lane.flags;
+    }
+    return renderTemplate(template, placeholders, flags, agentName);
+  } catch (err) {
+    if (err.message.startsWith(`Cannot generate agent ${agentName}`)) throw err;
+    throw new Error(`Cannot generate agent ${agentName}: ${err.message.replace(/^\[lane-config\] /, '')}`);
   }
-  return content;
+}
+
+/**
+ * Renders every agent, never letting one failure stop the others.
+ * Returns [{ agentName, fileName, content }] or [{ agentName, fileName, error }].
+ */
+function renderAllAgents(templatesDir, config) {
+  return AGENT_NAMES.map((agentName) => {
+    const fileName = agentFileName(agentName);
+    try {
+      return { agentName, fileName, content: buildAgentContent(agentName, templatesDir, config) };
+    } catch (error) {
+      return { agentName, fileName, error };
+    }
+  });
+}
+
+/**
+ * Present-but-invalid values of the config keys the agents consume.
+ * Absent keys are never reported. Returns an array of messages.
+ */
+function validateAgentConfig(config) {
+  const errors = [];
+  const conventions = config?.conventions ?? {};
+  const sprintCycle = config?.integrations?.azureDevOps?.sprintCycle ?? {};
+
+  if ('approvalThresholdPercent' in sprintCycle && !isValidApprovalThreshold(sprintCycle.approvalThresholdPercent)) {
+    errors.push(
+      'integrations.azureDevOps.sprintCycle.approvalThresholdPercent must be a number greater than 0 and less than 100. ' +
+        `Got: ${JSON.stringify(sprintCycle.approvalThresholdPercent)}`
+    );
+  }
+  if ('decimalSeparator' in conventions && !DECIMAL_SEPARATORS.includes(conventions.decimalSeparator)) {
+    errors.push(`conventions.decimalSeparator must be "," or ".". Got: ${JSON.stringify(conventions.decimalSeparator)}`);
+  }
+  if (
+    'defectIdPrefix' in conventions &&
+    (typeof conventions.defectIdPrefix !== 'string' || !DEFECT_ID_PREFIX_PATTERN.test(conventions.defectIdPrefix))
+  ) {
+    errors.push(
+      `conventions.defectIdPrefix must match ${DEFECT_ID_PREFIX_PATTERN} (uppercase letters, digits, "-"). ` +
+        `Got: ${JSON.stringify(conventions.defectIdPrefix)}`
+    );
+  }
+
+  // Lanes are only checked when the generator actually consumes them: sprint agents on
+  // and a lane block that laneTableFromConfig does not short-circuit (enabled !== false).
+  // A disabled block, or a project without sprint agents, never fails validate here.
+  const block = config?.parallelLanes;
+  const lanesInUse =
+    isSprintCycleEnabled(config) && block !== undefined && block !== null && block.enabled !== false;
+  if (!lanesInUse) return errors;
+
+  const lanes = block.lanes;
+  if (Array.isArray(lanes)) {
+    lanes.forEach((lane, index) => {
+      const ns = lane && typeof lane === 'object' ? lane.mcpNamespace : undefined;
+      if (ns !== undefined && ns !== null && (typeof ns !== 'string' || !MCP_NAMESPACE_PATTERN.test(ns))) {
+        errors.push(
+          `parallelLanes.lanes[${index}].mcpNamespace must match ${MCP_NAMESPACE_PATTERN} or be null. Got: ${JSON.stringify(ns)}`
+        );
+      }
+    });
+  }
+  try {
+    laneTableFromConfig(config);
+  } catch (err) {
+    errors.push(`parallelLanes: ${err.message.replace(/^\[lane-config\] /, '')}`);
+  }
+
+  return errors;
 }
 
 function agentFileName(agentName) {
@@ -72,7 +334,14 @@ function agentFileName(agentName) {
 
 module.exports = {
   AGENT_NAMES,
+  BROWSER_TOOLS,
+  DEFAULT_SPRINT_CYCLE,
   isSprintCycleEnabled,
+  buildPlaceholders,
+  resolveMcpLanes,
+  renderTemplate,
   buildAgentContent,
+  renderAllAgents,
+  validateAgentConfig,
   agentFileName,
 };

package/scripts/lib/module-keys.js

@@ -0,0 +1,54 @@
+'use strict';
+
+/**
+ * scripts/lib/module-keys.js - the one place that knows how a module or submodule
+ * key maps to a folder name.
+ *
+ * Two layouts coexist on purpose, and they are NOT the same string:
+ *
+ *   specs       qa/01-specifications/module-{name}/submodule-{name}/
+ *   automation  qa/07-automation/e2e/tests/{name}/{name}.spec.ts
+ *
+ * The prefixed form is what every skill under skills/, docs/folder-structure-guide.md
+ * and the "specPath" field of qa-framework.config.json declare. The bare form is what
+ * "automationPath" declares and what the live consuming projects actually contain.
+ * init.js used to write the bare form in both places, so a freshly scaffolded project
+ * violated the spec-path convention its own agent instructions impose.
+ *
+ * Both helpers are idempotent: a config that already spells the key as
+ * "module-suppliers" must not become "module-module-suppliers", and a key written
+ * bare must not lose a leading segment that merely looks like a prefix.
+ */
+
+/** Prefix `key` with `prefix-` unless it already carries it. */
+function withPrefix(prefix, key) {
+  const marker = `${prefix}-`;
+  return key.startsWith(marker) ? key : marker + key;
+}
+
+/** Strip a leading `prefix-` from `key` if present. */
+function withoutPrefix(prefix, key) {
+  const marker = `${prefix}-`;
+  return key.startsWith(marker) ? key.slice(marker.length) : key;
+}
+
+/** Folder name for a module's spec tree: module-{name}. */
+const moduleSpecDirName = key => withPrefix('module', key);
+
+/** Folder name for a submodule's spec tree: submodule-{name}. */
+const submoduleSpecDirName = key => withPrefix('submodule', key);
+
+/** Bare module name used by automation paths and TC ids. */
+const moduleBareName = key => withoutPrefix('module', key);
+
+/** Bare submodule name used by automation paths and TC ids. */
+const submoduleBareName = key => withoutPrefix('submodule', key);
+
+module.exports = {
+  withPrefix,
+  withoutPrefix,
+  moduleSpecDirName,
+  submoduleSpecDirName,
+  moduleBareName,
+  submoduleBareName,
+};

package/scripts/sync-version.js

@@ -13,6 +13,15 @@ const targets = [
   path.join(rootDir, '.github', 'copilot-instructions.md'),
 ];
 
+// qa-framework.config.json is published and is the template init.js copies into every
+// consuming project, where its frameworkVersion is injected into {{VERSION}} of the
+// generated agent instructions. Left out of this sync it silently drifted to 1.10.0
+// while the package shipped 1.12.0, so every fresh install told its agent the wrong
+// version. It is patched by field, not by the loose vX.Y.Z pattern used for prose.
+const jsonTargets = [
+  { file: path.join(rootDir, 'qa-framework.config.json'), field: 'frameworkVersion' },
+];
+
 // templates/qa-framework.instructions.md and templates/qa-framework.rules.md contain a
 // literal {{VERSION}} placeholder substituted by init.js/upgrade.js in the consumer
 // project at install/upgrade time. That placeholder is NOT a hardcoded version string
@@ -20,6 +29,24 @@ const targets = [
 
 const versionPattern = /v\d+\.\d+\.\d+/;
 
+for (const { file: filePath, field } of jsonTargets) {
+  const original = fs.readFileSync(filePath, 'utf8');
+  const fieldPattern = new RegExp(
+    `("${field}"\\s*:\\s*")\\d+\\.\\d+\\.\\d+(")`
+  );
+
+  if (!fieldPattern.test(original)) {
+    throw new Error(`sync-version: no "${field}" version field in ${filePath}`);
+  }
+
+  const updated = original.replace(fieldPattern, `$1${version}$2`);
+
+  if (updated !== original) {
+    fs.writeFileSync(filePath, updated, 'utf8');
+    console.log(`sync-version: updated ${path.relative(rootDir, filePath)} ${field} -> ${version}`);
+  }
+}
+
 for (const filePath of targets) {
   const original = fs.readFileSync(filePath, 'utf8');
 

package/scripts/upgrade.js

@@ -77,7 +77,7 @@ const fs   = require('fs');
 const path = require('path');
 
 const { buildCommandContent, discoverSkillNames, commandFileName } = require('./lib/claude-commands');
-const { AGENT_NAMES, isSprintCycleEnabled, buildAgentContent, agentFileName } = require('./lib/claude-agents');
+const { isSprintCycleEnabled, renderAllAgents } = require('./lib/claude-agents');
 
 const args   = process.argv.slice(2);
 const dryRun = args.includes('--dry-run');
@@ -152,6 +152,7 @@ const SHIPPED_SCAFFOLD_HASHES = {
 const updated  = [];
 const skipped  = [];
 const warnings = [];
+const errors   = [];
 
 console.log(`\n[qa-framework/upgrade] ${dryRun ? '(dry-run) ' : ''}Upgrading framework-owned files`);
 console.log(`[qa-framework/upgrade] Project root: ${cwd}\n`);
@@ -197,8 +198,14 @@ forceWrite(claudeRulesPath, claudeRulesContent);
 if (isSprintCycleEnabled(config)) {
   const templatesDir = path.resolve(__dirname, '..', 'templates');
   const claudeAgentsDest = path.join(cwd, '.claude', 'agents');
-  for (const agentName of AGENT_NAMES) {
-    forceWrite(path.join(claudeAgentsDest, agentFileName(agentName)), buildAgentContent(agentName, templatesDir, config));
+  // Rendered per agent: one invalid agent (e.g. qa-investigador with a bad lane config)
+  // is reported and left untouched on disk, the others are still upgraded.
+  for (const agent of renderAllAgents(templatesDir, config)) {
+    if (agent.error) {
+      errors.push(`.claude/agents/${agent.fileName} was not generated. ${agent.error.message}`);
+      continue;
+    }
+    forceWrite(path.join(claudeAgentsDest, agent.fileName), agent.content);
   }
 }
 
@@ -704,6 +711,12 @@ if (warnings.length) {
   for (const w of warnings) console.log(`    ⚠️  ${w}`);
 }
 
+if (errors.length) {
+  for (const e of errors) console.error(`[qa-framework/upgrade] ERROR: ${e}`);
+  console.error('[qa-framework/upgrade] Fix qa-framework.config.json (run "qa-framework validate") and run upgrade again.');
+  process.exitCode = 1;
+}
+
 console.log('\n[qa-framework/upgrade] Done.\n');
 
 // -----------------------------------------------------------------------

package/scripts/validate.js

@@ -8,6 +8,9 @@
  *   qa-framework validate --config ./my-config.json
  *
  * Checks:
+ *   0. Config values read by the sprint agents: approvalThresholdPercent (0 < x < 100),
+ *      decimalSeparator, defectIdPrefix and, when sprint agents and lanes are enabled,
+ *      parallelLanes / mcpNamespace (only present-but-invalid values are reported)
  *   1. Required top-level folders exist
  *   2. All submodule folders under 01-specifications/ contain the 6 required spec files
  *   3. No plaintext credentials in spec files (basic scan)
@@ -20,19 +23,27 @@
 
 const fs   = require('fs');
 const path = require('path');
+const { moduleBareName, submoduleBareName } = require('./lib/module-keys');
 
 // --- Args ---
 const args       = process.argv.slice(2);
 const strict     = args.includes('--strict');
 const configFlag = args.indexOf('--config');
+const { validateAgentConfig } = require('./lib/claude-agents');
+
+// Same lookup order as init/upgrade: --config, then root, then qa/.
 const configPath = configFlag !== -1
   ? path.resolve(process.cwd(), args[configFlag + 1])
-  : path.resolve(process.cwd(), 'qa-framework.config.json');
+  : [
+      path.resolve(process.cwd(), 'qa-framework.config.json'),
+      path.resolve(process.cwd(), 'qa', 'qa-framework.config.json'),
+    ].find((candidate) => fs.existsSync(candidate));
 
 // --- Load config (optional) ---
 let qaRoot = path.resolve(process.cwd(), 'qa');
-if (fs.existsSync(configPath)) {
-  const config = JSON.parse(fs.readFileSync(configPath, 'utf8'));
+let config = null;
+if (configPath && fs.existsSync(configPath)) {
+  config = JSON.parse(fs.readFileSync(configPath, 'utf8'));
   qaRoot = path.resolve(process.cwd(), config.conventions?.qaRoot ?? 'qa');
 }
 
@@ -42,6 +53,15 @@ if (strict) console.log('[qa-framework/validate] Mode: STRICT');
 const errors   = [];
 const warnings = [];
 
+// -----------------------------------------------------------------------
+// Check 0: Config values (only present-but-invalid values are reported)
+// -----------------------------------------------------------------------
+if (config) {
+  for (const message of validateAgentConfig(config)) {
+    errors.push(`Invalid config (${path.relative(process.cwd(), configPath)}): ${message}`);
+  }
+}
+
 // -----------------------------------------------------------------------
 // Check 1: Required top-level folders
 // -----------------------------------------------------------------------
@@ -94,11 +114,16 @@ if (fs.existsSync(specsRoot)) {
         }
       }
 
-      // Strict: automation spec must exist
+      // Strict: automation spec must exist.
+      // Spec folders are module-{name}/submodule-{name}; e2e/tests/ is keyed by the
+      // bare name, so the prefixes are stripped before deriving the automation path.
+      // Without this the warning points at a folder no project has ever created.
       if (strict) {
-        const specTs = path.join(qaRoot, '07-automation', 'e2e', 'tests', moduleDir, `${subDir}.spec.ts`);
+        const moduleName = moduleBareName(moduleDir);
+        const subName = submoduleBareName(subDir);
+        const specTs = path.join(qaRoot, '07-automation', 'e2e', 'tests', moduleName, `${subName}.spec.ts`);
         if (!fs.existsSync(specTs)) {
-          warnings.push(`[STRICT] No automation spec found: qa/07-automation/e2e/tests/${moduleDir}/${subDir}.spec.ts`);
+          warnings.push(`[STRICT] No automation spec found: qa/07-automation/e2e/tests/${moduleName}/${subName}.spec.ts`);
         }
       }
     }