@harness-engineering/core 0.9.2 → 0.10.0

package/dist/index.js

@@ -5239,6 +5239,15 @@ var FAILURES_FILE = "failures.md";
 var HANDOFF_FILE = "handoff.json";
 var GATE_CONFIG_FILE = "gate.json";
 var INDEX_FILE2 = "index.json";
+var MAX_CACHE_ENTRIES = 8;
+var learningsCacheMap = /* @__PURE__ */ new Map();
+var failuresCacheMap = /* @__PURE__ */ new Map();
+function evictIfNeeded(map) {
+  if (map.size > MAX_CACHE_ENTRIES) {
+    const oldest = map.keys().next().value;
+    if (oldest !== void 0) map.delete(oldest);
+  }
+}
 async function getStateDir(projectPath, stream) {
   const streamsIndexPath = path3.join(projectPath, HARNESS_DIR2, "streams", INDEX_FILE2);
   const hasStreams = fs4.existsSync(streamsIndexPath);
@@ -5336,25 +5345,35 @@ async function loadRelevantLearnings(projectPath, skillName, stream) {
     if (!fs4.existsSync(learningsPath)) {
       return (0, import_types.Ok)([]);
     }
-    const content = fs4.readFileSync(learningsPath, "utf-8");
-    const lines = content.split("\n");
-    const entries = [];
-    let currentBlock = [];
-    for (const line of lines) {
-      if (line.startsWith("# ")) continue;
-      const isDatedBullet = /^- \*\*\d{4}-\d{2}-\d{2}/.test(line);
-      const isHeading = /^## \d{4}-\d{2}-\d{2}/.test(line);
-      if (isDatedBullet || isHeading) {
-        if (currentBlock.length > 0) {
-          entries.push(currentBlock.join("\n"));
+    const stats = fs4.statSync(learningsPath);
+    const cacheKey = learningsPath;
+    const cached = learningsCacheMap.get(cacheKey);
+    let entries;
+    if (cached && cached.mtimeMs === stats.mtimeMs) {
+      entries = cached.entries;
+    } else {
+      const content = fs4.readFileSync(learningsPath, "utf-8");
+      const lines = content.split("\n");
+      entries = [];
+      let currentBlock = [];
+      for (const line of lines) {
+        if (line.startsWith("# ")) continue;
+        const isDatedBullet = /^- \*\*\d{4}-\d{2}-\d{2}/.test(line);
+        const isHeading = /^## \d{4}-\d{2}-\d{2}/.test(line);
+        if (isDatedBullet || isHeading) {
+          if (currentBlock.length > 0) {
+            entries.push(currentBlock.join("\n"));
+          }
+          currentBlock = [line];
+        } else if (line.trim() !== "" && currentBlock.length > 0) {
+          currentBlock.push(line);
         }
-        currentBlock = [line];
-      } else if (line.trim() !== "" && currentBlock.length > 0) {
-        currentBlock.push(line);
       }
-    }
-    if (currentBlock.length > 0) {
-      entries.push(currentBlock.join("\n"));
+      if (currentBlock.length > 0) {
+        entries.push(currentBlock.join("\n"));
+      }
+      learningsCacheMap.set(cacheKey, { mtimeMs: stats.mtimeMs, entries });
+      evictIfNeeded(learningsCacheMap);
     }
     if (!skillName) {
       return (0, import_types.Ok)(entries);
@@ -5405,6 +5424,12 @@ async function loadFailures(projectPath, stream) {
     if (!fs4.existsSync(failuresPath)) {
       return (0, import_types.Ok)([]);
     }
+    const stats = fs4.statSync(failuresPath);
+    const cacheKey = failuresPath;
+    const cached = failuresCacheMap.get(cacheKey);
+    if (cached && cached.mtimeMs === stats.mtimeMs) {
+      return (0, import_types.Ok)(cached.entries);
+    }
     const content = fs4.readFileSync(failuresPath, "utf-8");
     const entries = [];
     for (const line of content.split("\n")) {
@@ -5418,6 +5443,8 @@ async function loadFailures(projectPath, stream) {
         });
       }
     }
+    failuresCacheMap.set(cacheKey, { mtimeMs: stats.mtimeMs, entries });
+    evictIfNeeded(failuresCacheMap);
     return (0, import_types.Ok)(entries);
   } catch (error) {
     return (0, import_types.Err)(
@@ -6516,14 +6543,22 @@ async function runCIChecks(input) {
   const { projectRoot, config, skip = [], failOn = "error" } = input;
   try {
     const checks = [];
-    for (const name of ALL_CHECKS) {
-      if (skip.includes(name)) {
-        checks.push({ name, status: "skip", issues: [], durationMs: 0 });
-      } else {
-        const result = await runSingleCheck(name, projectRoot, config);
-        checks.push(result);
-      }
+    const skippedSet = new Set(skip);
+    if (skippedSet.has("validate")) {
+      checks.push({ name: "validate", status: "skip", issues: [], durationMs: 0 });
+    } else {
+      checks.push(await runSingleCheck("validate", projectRoot, config));
     }
+    const remainingChecks = ALL_CHECKS.slice(1);
+    const phase2Results = await Promise.all(
+      remainingChecks.map(async (name) => {
+        if (skippedSet.has(name)) {
+          return { name, status: "skip", issues: [], durationMs: 0 };
+        }
+        return runSingleCheck(name, projectRoot, config);
+      })
+    );
+    checks.push(...phase2Results);
     const summary = buildSummary(checks);
     const exitCode = determineExitCode(summary, failOn);
     const report = {
@@ -6648,76 +6683,93 @@ async function runMechanicalChecks(options) {
       });
     }
   }
+  const parallelChecks = [];
   if (!skip.includes("check-docs")) {
-    try {
-      const docsDir = path6.join(projectRoot, config.docsDir ?? "docs");
-      const result = await checkDocCoverage("project", { docsDir });
-      if (!result.ok) {
-        statuses["check-docs"] = "warn";
-        findings.push({
-          tool: "check-docs",
-          file: docsDir,
-          message: result.error.message,
-          severity: "warning"
-        });
-      } else if (result.value.gaps && result.value.gaps.length > 0) {
-        statuses["check-docs"] = "warn";
-        for (const gap of result.value.gaps) {
-          findings.push({
+    parallelChecks.push(
+      (async () => {
+        const localFindings = [];
+        try {
+          const docsDir = path6.join(projectRoot, config.docsDir ?? "docs");
+          const result = await checkDocCoverage("project", { docsDir });
+          if (!result.ok) {
+            statuses["check-docs"] = "warn";
+            localFindings.push({
+              tool: "check-docs",
+              file: docsDir,
+              message: result.error.message,
+              severity: "warning"
+            });
+          } else if (result.value.gaps && result.value.gaps.length > 0) {
+            statuses["check-docs"] = "warn";
+            for (const gap of result.value.gaps) {
+              localFindings.push({
+                tool: "check-docs",
+                file: gap.file,
+                message: `Undocumented: ${gap.file} (suggested: ${gap.suggestedSection})`,
+                severity: "warning"
+              });
+            }
+          } else {
+            statuses["check-docs"] = "pass";
+          }
+        } catch (err) {
+          statuses["check-docs"] = "warn";
+          localFindings.push({
             tool: "check-docs",
-            file: gap.file,
-            message: `Undocumented: ${gap.file} (suggested: ${gap.suggestedSection})`,
+            file: path6.join(projectRoot, "docs"),
+            message: err instanceof Error ? err.message : String(err),
             severity: "warning"
           });
         }
-      } else {
-        statuses["check-docs"] = "pass";
-      }
-    } catch (err) {
-      statuses["check-docs"] = "warn";
-      findings.push({
-        tool: "check-docs",
-        file: path6.join(projectRoot, "docs"),
-        message: err instanceof Error ? err.message : String(err),
-        severity: "warning"
-      });
-    }
+        return localFindings;
+      })()
+    );
   }
   if (!skip.includes("security-scan")) {
-    try {
-      const securityConfig = parseSecurityConfig(config.security);
-      if (!securityConfig.enabled) {
-        statuses["security-scan"] = "skip";
-      } else {
-        const scanner = new SecurityScanner(securityConfig);
-        scanner.configureForProject(projectRoot);
-        const filesToScan = changedFiles ?? [];
-        const scanResult = await scanner.scanFiles(filesToScan);
-        if (scanResult.findings.length > 0) {
-          statuses["security-scan"] = "warn";
-          for (const f of scanResult.findings) {
-            findings.push({
-              tool: "security-scan",
-              file: f.file,
-              line: f.line,
-              ruleId: f.ruleId,
-              message: f.message,
-              severity: f.severity === "info" ? "warning" : f.severity
-            });
+    parallelChecks.push(
+      (async () => {
+        const localFindings = [];
+        try {
+          const securityConfig = parseSecurityConfig(config.security);
+          if (!securityConfig.enabled) {
+            statuses["security-scan"] = "skip";
+          } else {
+            const scanner = new SecurityScanner(securityConfig);
+            scanner.configureForProject(projectRoot);
+            const filesToScan = changedFiles ?? [];
+            const scanResult = await scanner.scanFiles(filesToScan);
+            if (scanResult.findings.length > 0) {
+              statuses["security-scan"] = "warn";
+              for (const f of scanResult.findings) {
+                localFindings.push({
+                  tool: "security-scan",
+                  file: f.file,
+                  line: f.line,
+                  ruleId: f.ruleId,
+                  message: f.message,
+                  severity: f.severity === "info" ? "warning" : f.severity
+                });
+              }
+            } else {
+              statuses["security-scan"] = "pass";
+            }
           }
-        } else {
-          statuses["security-scan"] = "pass";
+        } catch (err) {
+          statuses["security-scan"] = "warn";
+          localFindings.push({
+            tool: "security-scan",
+            file: projectRoot,
+            message: err instanceof Error ? err.message : String(err),
+            severity: "warning"
+          });
         }
-      }
-    } catch (err) {
-      statuses["security-scan"] = "warn";
-      findings.push({
-        tool: "security-scan",
-        file: projectRoot,
-        message: err instanceof Error ? err.message : String(err),
-        severity: "warning"
-      });
-    }
+        return localFindings;
+      })()
+    );
+  }
+  const parallelResults = await Promise.all(parallelChecks);
+  for (const result of parallelResults) {
+    findings.push(...result);
   }
   const hasErrors = findings.some((f) => f.severity === "error");
   const stopPipeline = statuses.validate === "fail" || statuses["check-deps"] === "fail";

package/dist/index.mjs

@@ -5044,6 +5044,15 @@ var FAILURES_FILE = "failures.md";
 var HANDOFF_FILE = "handoff.json";
 var GATE_CONFIG_FILE = "gate.json";
 var INDEX_FILE2 = "index.json";
+var MAX_CACHE_ENTRIES = 8;
+var learningsCacheMap = /* @__PURE__ */ new Map();
+var failuresCacheMap = /* @__PURE__ */ new Map();
+function evictIfNeeded(map) {
+  if (map.size > MAX_CACHE_ENTRIES) {
+    const oldest = map.keys().next().value;
+    if (oldest !== void 0) map.delete(oldest);
+  }
+}
 async function getStateDir(projectPath, stream) {
   const streamsIndexPath = path3.join(projectPath, HARNESS_DIR2, "streams", INDEX_FILE2);
   const hasStreams = fs4.existsSync(streamsIndexPath);
@@ -5141,25 +5150,35 @@ async function loadRelevantLearnings(projectPath, skillName, stream) {
     if (!fs4.existsSync(learningsPath)) {
       return Ok([]);
     }
-    const content = fs4.readFileSync(learningsPath, "utf-8");
-    const lines = content.split("\n");
-    const entries = [];
-    let currentBlock = [];
-    for (const line of lines) {
-      if (line.startsWith("# ")) continue;
-      const isDatedBullet = /^- \*\*\d{4}-\d{2}-\d{2}/.test(line);
-      const isHeading = /^## \d{4}-\d{2}-\d{2}/.test(line);
-      if (isDatedBullet || isHeading) {
-        if (currentBlock.length > 0) {
-          entries.push(currentBlock.join("\n"));
+    const stats = fs4.statSync(learningsPath);
+    const cacheKey = learningsPath;
+    const cached = learningsCacheMap.get(cacheKey);
+    let entries;
+    if (cached && cached.mtimeMs === stats.mtimeMs) {
+      entries = cached.entries;
+    } else {
+      const content = fs4.readFileSync(learningsPath, "utf-8");
+      const lines = content.split("\n");
+      entries = [];
+      let currentBlock = [];
+      for (const line of lines) {
+        if (line.startsWith("# ")) continue;
+        const isDatedBullet = /^- \*\*\d{4}-\d{2}-\d{2}/.test(line);
+        const isHeading = /^## \d{4}-\d{2}-\d{2}/.test(line);
+        if (isDatedBullet || isHeading) {
+          if (currentBlock.length > 0) {
+            entries.push(currentBlock.join("\n"));
+          }
+          currentBlock = [line];
+        } else if (line.trim() !== "" && currentBlock.length > 0) {
+          currentBlock.push(line);
         }
-        currentBlock = [line];
-      } else if (line.trim() !== "" && currentBlock.length > 0) {
-        currentBlock.push(line);
       }
-    }
-    if (currentBlock.length > 0) {
-      entries.push(currentBlock.join("\n"));
+      if (currentBlock.length > 0) {
+        entries.push(currentBlock.join("\n"));
+      }
+      learningsCacheMap.set(cacheKey, { mtimeMs: stats.mtimeMs, entries });
+      evictIfNeeded(learningsCacheMap);
     }
     if (!skillName) {
       return Ok(entries);
@@ -5210,6 +5229,12 @@ async function loadFailures(projectPath, stream) {
     if (!fs4.existsSync(failuresPath)) {
       return Ok([]);
     }
+    const stats = fs4.statSync(failuresPath);
+    const cacheKey = failuresPath;
+    const cached = failuresCacheMap.get(cacheKey);
+    if (cached && cached.mtimeMs === stats.mtimeMs) {
+      return Ok(cached.entries);
+    }
     const content = fs4.readFileSync(failuresPath, "utf-8");
     const entries = [];
     for (const line of content.split("\n")) {
@@ -5223,6 +5248,8 @@ async function loadFailures(projectPath, stream) {
         });
       }
     }
+    failuresCacheMap.set(cacheKey, { mtimeMs: stats.mtimeMs, entries });
+    evictIfNeeded(failuresCacheMap);
     return Ok(entries);
   } catch (error) {
     return Err(
@@ -6321,14 +6348,22 @@ async function runCIChecks(input) {
   const { projectRoot, config, skip = [], failOn = "error" } = input;
   try {
     const checks = [];
-    for (const name of ALL_CHECKS) {
-      if (skip.includes(name)) {
-        checks.push({ name, status: "skip", issues: [], durationMs: 0 });
-      } else {
-        const result = await runSingleCheck(name, projectRoot, config);
-        checks.push(result);
-      }
+    const skippedSet = new Set(skip);
+    if (skippedSet.has("validate")) {
+      checks.push({ name: "validate", status: "skip", issues: [], durationMs: 0 });
+    } else {
+      checks.push(await runSingleCheck("validate", projectRoot, config));
     }
+    const remainingChecks = ALL_CHECKS.slice(1);
+    const phase2Results = await Promise.all(
+      remainingChecks.map(async (name) => {
+        if (skippedSet.has(name)) {
+          return { name, status: "skip", issues: [], durationMs: 0 };
+        }
+        return runSingleCheck(name, projectRoot, config);
+      })
+    );
+    checks.push(...phase2Results);
     const summary = buildSummary(checks);
     const exitCode = determineExitCode(summary, failOn);
     const report = {
@@ -6453,76 +6488,93 @@ async function runMechanicalChecks(options) {
       });
     }
   }
+  const parallelChecks = [];
   if (!skip.includes("check-docs")) {
-    try {
-      const docsDir = path6.join(projectRoot, config.docsDir ?? "docs");
-      const result = await checkDocCoverage("project", { docsDir });
-      if (!result.ok) {
-        statuses["check-docs"] = "warn";
-        findings.push({
-          tool: "check-docs",
-          file: docsDir,
-          message: result.error.message,
-          severity: "warning"
-        });
-      } else if (result.value.gaps && result.value.gaps.length > 0) {
-        statuses["check-docs"] = "warn";
-        for (const gap of result.value.gaps) {
-          findings.push({
+    parallelChecks.push(
+      (async () => {
+        const localFindings = [];
+        try {
+          const docsDir = path6.join(projectRoot, config.docsDir ?? "docs");
+          const result = await checkDocCoverage("project", { docsDir });
+          if (!result.ok) {
+            statuses["check-docs"] = "warn";
+            localFindings.push({
+              tool: "check-docs",
+              file: docsDir,
+              message: result.error.message,
+              severity: "warning"
+            });
+          } else if (result.value.gaps && result.value.gaps.length > 0) {
+            statuses["check-docs"] = "warn";
+            for (const gap of result.value.gaps) {
+              localFindings.push({
+                tool: "check-docs",
+                file: gap.file,
+                message: `Undocumented: ${gap.file} (suggested: ${gap.suggestedSection})`,
+                severity: "warning"
+              });
+            }
+          } else {
+            statuses["check-docs"] = "pass";
+          }
+        } catch (err) {
+          statuses["check-docs"] = "warn";
+          localFindings.push({
             tool: "check-docs",
-            file: gap.file,
-            message: `Undocumented: ${gap.file} (suggested: ${gap.suggestedSection})`,
+            file: path6.join(projectRoot, "docs"),
+            message: err instanceof Error ? err.message : String(err),
             severity: "warning"
           });
         }
-      } else {
-        statuses["check-docs"] = "pass";
-      }
-    } catch (err) {
-      statuses["check-docs"] = "warn";
-      findings.push({
-        tool: "check-docs",
-        file: path6.join(projectRoot, "docs"),
-        message: err instanceof Error ? err.message : String(err),
-        severity: "warning"
-      });
-    }
+        return localFindings;
+      })()
+    );
   }
   if (!skip.includes("security-scan")) {
-    try {
-      const securityConfig = parseSecurityConfig(config.security);
-      if (!securityConfig.enabled) {
-        statuses["security-scan"] = "skip";
-      } else {
-        const scanner = new SecurityScanner(securityConfig);
-        scanner.configureForProject(projectRoot);
-        const filesToScan = changedFiles ?? [];
-        const scanResult = await scanner.scanFiles(filesToScan);
-        if (scanResult.findings.length > 0) {
-          statuses["security-scan"] = "warn";
-          for (const f of scanResult.findings) {
-            findings.push({
-              tool: "security-scan",
-              file: f.file,
-              line: f.line,
-              ruleId: f.ruleId,
-              message: f.message,
-              severity: f.severity === "info" ? "warning" : f.severity
-            });
+    parallelChecks.push(
+      (async () => {
+        const localFindings = [];
+        try {
+          const securityConfig = parseSecurityConfig(config.security);
+          if (!securityConfig.enabled) {
+            statuses["security-scan"] = "skip";
+          } else {
+            const scanner = new SecurityScanner(securityConfig);
+            scanner.configureForProject(projectRoot);
+            const filesToScan = changedFiles ?? [];
+            const scanResult = await scanner.scanFiles(filesToScan);
+            if (scanResult.findings.length > 0) {
+              statuses["security-scan"] = "warn";
+              for (const f of scanResult.findings) {
+                localFindings.push({
+                  tool: "security-scan",
+                  file: f.file,
+                  line: f.line,
+                  ruleId: f.ruleId,
+                  message: f.message,
+                  severity: f.severity === "info" ? "warning" : f.severity
+                });
+              }
+            } else {
+              statuses["security-scan"] = "pass";
+            }
           }
-        } else {
-          statuses["security-scan"] = "pass";
+        } catch (err) {
+          statuses["security-scan"] = "warn";
+          localFindings.push({
+            tool: "security-scan",
+            file: projectRoot,
+            message: err instanceof Error ? err.message : String(err),
+            severity: "warning"
+          });
         }
-      }
-    } catch (err) {
-      statuses["security-scan"] = "warn";
-      findings.push({
-        tool: "security-scan",
-        file: projectRoot,
-        message: err instanceof Error ? err.message : String(err),
-        severity: "warning"
-      });
-    }
+        return localFindings;
+      })()
+    );
+  }
+  const parallelResults = await Promise.all(parallelChecks);
+  for (const result of parallelResults) {
+    findings.push(...result);
   }
   const hasErrors = findings.some((f) => f.severity === "error");
   const stopPipeline = statuses.validate === "fail" || statuses["check-deps"] === "fail";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@harness-engineering/core",
-  "version": "0.9.2",
+  "version": "0.10.0",
   "description": "Core library for Harness Engineering toolkit",
   "main": "./dist/index.js",
   "module": "./dist/index.mjs",