@xn-intenton-z2a/agentic-lib 7.4.56 → 7.4.57

package/.github/workflows/agentic-lib-workflow.yml

@@ -1317,6 +1317,10 @@ jobs:
           git config user.name 'GitHub Actions[bot]'
           BRANCH=$(git rev-parse --abbrev-ref HEAD)
           git add -A
+          # Unstage log/screenshot/state files — these live on the agentic-lib-logs branch
+          git reset HEAD -- 'intentïon.md' 'SCREENSHOT_INDEX.png' 2>/dev/null || true
+          git reset HEAD -- agent-log-*.md 2>/dev/null || true
+          git reset HEAD -- agentic-lib-state.toml 2>/dev/null || true
           if ! git diff --cached --quiet; then
             git commit -m "agentic-step: fix failing tests / resolve conflicts"
             git push origin HEAD:"$BRANCH" 2>&1 || git push --force-with-lease origin HEAD:"$BRANCH" 2>&1
@@ -1393,6 +1397,10 @@ jobs:
           git config --local user.email 'action@github.com'
           git config --local user.name 'GitHub Actions[bot]'
           git add -A
+          # Unstage log/screenshot/state files — these live on the agentic-lib-logs branch
+          git reset HEAD -- 'intentïon.md' 'SCREENSHOT_INDEX.png' 2>/dev/null || true
+          git reset HEAD -- agent-log-*.md 2>/dev/null || true
+          git reset HEAD -- agentic-lib-state.toml 2>/dev/null || true
           git commit -m "agentic-step: fix broken main build (run ${{ env.FIX_RUN_ID }})"
           git push -u origin agentic-lib-fix-main-build
           gh pr create \
@@ -1746,15 +1754,18 @@ jobs:
                 try {
                   await github.rest.pulls.merge({
                     owner, repo, pull_number: prNumber, merge_method: 'squash',
+                    commit_message: closesLines + '\n\nAutomated transformation.',
                   });
                   core.info(`Merged PR #${prNumber}`);
                   try {
                     await github.rest.git.deleteRef({ owner, repo, ref: `heads/${branchName}` });
                   } catch (e) {}
-                  if (issueNumber) {
+                  // Close all referenced issues (belt-and-suspenders with Closes keywords)
+                  for (const num of issueNumbers) {
                     try {
-                      await github.rest.issues.addLabels({ owner, repo, issue_number: parseInt(issueNumber), labels: ['merged'] });
-                      await github.rest.issues.removeLabel({ owner, repo, issue_number: parseInt(issueNumber), name: 'in-progress' });
+                      await github.rest.issues.update({ owner, repo, issue_number: parseInt(num), state: 'closed' });
+                      await github.rest.issues.addLabels({ owner, repo, issue_number: parseInt(num), labels: ['merged'] });
+                      await github.rest.issues.removeLabel({ owner, repo, issue_number: parseInt(num), name: 'in-progress' });
                     } catch (e) {}
                   }
                   return;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@xn-intenton-z2a/agentic-lib",
-  "version": "7.4.56",
+  "version": "7.4.57",
   "description": "Agentic-lib Agentic Coding Systems SDK powering automated GitHub workflows.",
   "type": "module",
   "scripts": {

package/src/actions/agentic-step/tasks/direct.js

@@ -70,7 +70,6 @@ async function buildMetricAssessment(ctx, config) {
   const minResolved = thresholds.minResolvedIssues ?? 1;
   const maxTodos = thresholds.maxSourceTodos ?? 0;
   const minCumulativeTransforms = thresholds.minCumulativeTransforms ?? 1;
-  const acceptanceThreshold = thresholds.acceptanceCriteriaThreshold ?? 50;
   const requireNoOpenIssues = thresholds.requireNoOpenIssues ?? true;
   const requireNoOpenPrs = thresholds.requireNoOpenPrs ?? true;
   const requireNoCriticalGaps = thresholds.requireNoCriticalGaps ?? true;
@@ -83,14 +82,16 @@ async function buildMetricAssessment(ctx, config) {
   } catch { /* ignore parse errors */ }
   const criticalGaps = reviewGaps.filter((g) => g.severity === "critical");
 
-  // Acceptance criteria from MISSION.md checkboxes (or structured TOML if available)
+  // Acceptance criteria — informational only (not a mechanical gate).
+  // The director LLM reads criteria from MISSION.md and assesses them directly.
   const { countAcceptanceCriteria } = await import("../../../copilot/telemetry.js");
   const missionPath = config.paths?.mission?.path || "MISSION.md";
   const acceptance = countAcceptanceCriteria(missionPath);
   const acceptancePct = acceptance.total > 0 ? (acceptance.met / acceptance.total) * 100 : 0;
-  const acceptanceMet = acceptance.total > 0 && acceptancePct >= acceptanceThreshold;
 
-  // C6: Removed "Dedicated tests" metric; using cumulative transforms instead
+  // FIX-10: Require at least one test file that imports from src/lib/
+  const { dedicatedTestCount } = detectDedicatedTests();
+
   const metrics = [
     { metric: "Open issues", value: ctx.issuesSummary.length, target: 0, met: requireNoOpenIssues ? ctx.issuesSummary.length === 0 : true },
     { metric: "Open PRs", value: ctx.prsSummary.length, target: 0, met: requireNoOpenPrs ? ctx.prsSummary.length === 0 : true },
@@ -99,7 +100,8 @@ async function buildMetricAssessment(ctx, config) {
     { metric: "Cumulative transforms", value: ctx.cumulativeTransformationCost, target: minCumulativeTransforms, met: ctx.cumulativeTransformationCost >= minCumulativeTransforms },
     { metric: "Budget", value: ctx.cumulativeTransformationCost, target: ctx.transformationBudget || "unlimited", met: !(ctx.transformationBudget > 0 && ctx.cumulativeTransformationCost >= ctx.transformationBudget) },
     { metric: "Implementation review", value: criticalGaps.length === 0 ? "No critical gaps" : `${criticalGaps.length} critical gap(s)`, target: "No critical gaps", met: requireNoCriticalGaps ? criticalGaps.length === 0 : true },
-    { metric: "Acceptance criteria", value: acceptance.total > 0 ? `${acceptance.met}/${acceptance.total} (${Math.round(acceptancePct)}%)` : "N/A", target: `>= ${acceptanceThreshold}%`, met: acceptanceMet },
+    { metric: "Acceptance criteria", value: acceptance.total > 0 ? `${acceptance.met}/${acceptance.total} (${Math.round(acceptancePct)}%)` : "N/A", target: "informational", met: true },
+    { metric: "Dedicated test files", value: dedicatedTestCount, target: ">= 1", met: dedicatedTestCount >= 1 },
   ];
 
   const allMet = metrics.every((m) => m.met);

package/src/actions/agentic-step/tasks/implementation-review.js

@@ -215,49 +215,13 @@ export async function implementationReview(context) {
           }
         }
 
-        // Also update MISSION.md checkboxes (best-effort, not critical)
-        if (metCriteria.length > 0 || metIndices.length > 0) {
-          try {
-            const missionPath = config.paths?.mission?.path || "MISSION.md";
-            const { readFileSync, writeFileSync } = await import("fs");
-            let missionContent = readFileSync(missionPath, "utf8");
-            let checkedCount = 0;
-
-            // Index-based update: find the Nth checkbox and check it
-            if (metIndices.length > 0) {
-              const lines = missionContent.split("\n");
-              let checkboxIdx = 0;
-              for (let i = 0; i < lines.length; i++) {
-                if (/^- \[ \] /.test(lines[i])) {
-                  checkboxIdx++;
-                  if (metIndices.includes(checkboxIdx)) {
-                    lines[i] = lines[i].replace(/^- \[ \] /, "- [x] ");
-                    checkedCount++;
-                  }
-                }
-              }
-              missionContent = lines.join("\n");
-            }
-
-            // Text-based update (fallback for backwards compatibility)
-            if (checkedCount === 0 && metCriteria.length > 0) {
-              for (const criterionText of metCriteria) {
-                const escaped = criterionText.replace(/[.*+?^${}()|[\]\\]/g, "\\$&").trim();
-                const re = new RegExp(`- \\[ \\] ${escaped}`);
-                if (re.test(missionContent)) {
-                  missionContent = missionContent.replace(re, `- [x] ${criterionText.trim()}`);
-                  checkedCount++;
-                }
-              }
-            }
-
-            if (checkedCount > 0) {
-              writeFileSync(missionPath, missionContent, "utf8");
-              core.info(`Updated ${checkedCount} acceptance criteria checkboxes in ${missionPath}`);
-            }
-          } catch (err) {
-            core.warning(`Could not update MISSION.md checkboxes: ${err.message}`);
-          }
+        // Log acceptance criteria status (informational — no longer writes to MISSION.md
+        // to avoid zero-diff commits that inflate transform counts)
+        if (metIndices.length > 0) {
+          core.info(`Review found ${metIndices.length} acceptance criteria met by index: [${metIndices.join(", ")}]`);
+        }
+        if (metCriteria.length > 0) {
+          core.info(`Review found ${metCriteria.length} acceptance criteria met by text`);
         }
 
         return { textResultForLlm: `Review recorded: ${elements?.length || 0} elements traced, ${gaps?.length || 0} gaps found, ${totalUpdated} criteria checked` };

package/src/actions/agentic-step/tasks/report.js

@@ -47,16 +47,21 @@ async function gatherAndWriteData(octokit, owner, repoName, periodStart, periodE
   writeFileSync(`${REPORT_DATA_DIR}/state.toml`, stateContent);
   writeFileSync(`${REPORT_DATA_DIR}/mission.md`, missionContent);
 
-  // 2. Workflow runs
+  // 2. Workflow runs (paginate up to 200 for long-running scenarios)
   let workflowRuns = [];
   try {
-    const { data } = await octokit.rest.actions.listWorkflowRunsForRepo({
-      owner, repo: repoName, per_page: 50, created: `${periodStart}..${periodEnd}`,
-    });
-    workflowRuns = data.workflow_runs.map(r => ({
-      id: r.id, name: r.name, status: r.status, conclusion: r.conclusion,
-      created_at: r.created_at, updated_at: r.updated_at, html_url: r.html_url,
-    }));
+    for (let page = 1; page <= 2; page++) {
+      const { data } = await octokit.rest.actions.listWorkflowRunsForRepo({
+        owner, repo: repoName, per_page: 100, page, created: `${periodStart}..${periodEnd}`,
+      });
+      for (const r of data.workflow_runs) {
+        workflowRuns.push({
+          id: r.id, name: r.name, status: r.status, conclusion: r.conclusion,
+          created_at: r.created_at, updated_at: r.updated_at, html_url: r.html_url,
+        });
+      }
+      if (data.workflow_runs.length < 100) break;
+    }
   } catch (err) { core.warning(`Could not list runs: ${err.message}`); }
   writeFileSync(`${REPORT_DATA_DIR}/workflow-runs.json`, JSON.stringify(workflowRuns, null, 2));
 

package/src/seeds/zero-package.json

@@ -17,7 +17,7 @@
   "author": "",
   "license": "MIT",
   "dependencies": {
-    "@xn-intenton-z2a/agentic-lib": "^7.4.56"
+    "@xn-intenton-z2a/agentic-lib": "^7.4.57"
   },
   "devDependencies": {
     "@playwright/test": "^1.58.0",