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

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

@@ -376,6 +376,10 @@ jobs:
           git config user.name "${TOKEN_USER}"
           git config user.email "${TOKEN_ID}+${TOKEN_USER}@users.noreply.github.com"
           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 diff --cached --quiet && echo "No changes" && exit 0
           VERSION=$(npx @xn-intenton-z2a/agentic-lib version 2>/dev/null || echo "latest")
           # Problem 1 fix: Don't use [skip ci] when workflow files changed.

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

@@ -123,6 +123,10 @@ jobs:
           git config user.name "${TOKEN_USER}"
           git config user.email "${TOKEN_ID}+${TOKEN_USER}@users.noreply.github.com"
           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 diff --cached --quiet && echo "No changes" && exit 0
           VERSION=$(npx @xn-intenton-z2a/agentic-lib version 2>/dev/null || echo "latest")
           git commit -m "update agentic-lib@${VERSION} [skip ci]"

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

@@ -281,16 +281,20 @@ jobs:
                     await github.rest.git.deleteRef({ owner, repo, ref: `heads/${pr.head.ref}` });
                   } catch (e) { core.info(`Could not delete branch: ${e.message}`); }
 
-                  // Label associated issue
+                  // Close associated issues — extract from PR body (Closes #N) and branch name
+                  const closesMatches = (pr.body || '').matchAll(/(?:Closes|Fixes|Resolves)\s+#(\d+)/gi);
+                  const issueNums = new Set([...closesMatches].map(m => parseInt(m[1])));
                   const branchPrefix = 'agentic-lib-issue-';
                   if (pr.head.ref.startsWith(branchPrefix)) {
-                    const issueNum = parseInt(pr.head.ref.replace(branchPrefix, ''));
-                    if (issueNum) {
-                      try {
-                        await github.rest.issues.addLabels({ owner, repo, issue_number: issueNum, labels: ['merged'] });
-                        await github.rest.issues.removeLabel({ owner, repo, issue_number: issueNum, name: 'in-progress' });
-                      } catch (e) { /* label may not exist */ }
-                    }
+                    const branchIssueNum = parseInt(pr.head.ref.replace(branchPrefix, ''));
+                    if (branchIssueNum) issueNums.add(branchIssueNum);
+                  }
+                  for (const issueNum of issueNums) {
+                    try {
+                      await github.rest.issues.update({ owner, repo, issue_number: issueNum, state: 'closed' });
+                      await github.rest.issues.addLabels({ owner, repo, issue_number: issueNum, labels: ['merged'] });
+                      await github.rest.issues.removeLabel({ owner, repo, issue_number: issueNum, name: 'in-progress' });
+                    } catch (e) { /* label may not exist */ }
                   }
                   mergedPRs.push(pr.number);
                 } catch (e) {
@@ -1242,6 +1246,10 @@ jobs:
             npm install 2>/dev/null || true
 
             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 "fix-stuck: sync with main (resolve conflicts)"
               git push --force-with-lease origin HEAD:"$BRANCH" 2>&1 && RESOLVED=true
@@ -1317,6 +1325,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
@@ -1356,16 +1368,20 @@ jobs:
                   try {
                     await github.rest.git.deleteRef({ owner, repo, ref: `heads/${pr.head.ref}` });
                   } catch (e) { /* branch may already be deleted */ }
-                  // Label associated issue
-                  const branchPrefix = 'agentic-lib-issue-';
-                  if (pr.head.ref.startsWith(branchPrefix)) {
-                    const issueNum = parseInt(pr.head.ref.replace(branchPrefix, ''));
-                    if (issueNum) {
-                      try {
-                        await github.rest.issues.addLabels({ owner, repo, issue_number: issueNum, labels: ['merged'] });
-                        await github.rest.issues.removeLabel({ owner, repo, issue_number: issueNum, name: 'in-progress' });
-                      } catch (e) { /* label may not exist */ }
-                    }
+                  // Close associated issues — extract from PR body (Closes #N) and branch name
+                  const closesMatches2 = (pr.body || '').matchAll(/(?:Closes|Fixes|Resolves)\s+#(\d+)/gi);
+                  const issueNums2 = new Set([...closesMatches2].map(m => parseInt(m[1])));
+                  const branchPrefix2 = 'agentic-lib-issue-';
+                  if (pr.head.ref.startsWith(branchPrefix2)) {
+                    const branchIssueNum = parseInt(pr.head.ref.replace(branchPrefix2, ''));
+                    if (branchIssueNum) issueNums2.add(branchIssueNum);
+                  }
+                  for (const issueNum of issueNums2) {
+                    try {
+                      await github.rest.issues.update({ owner, repo, issue_number: issueNum, state: 'closed' });
+                      await github.rest.issues.addLabels({ owner, repo, issue_number: issueNum, labels: ['merged'] });
+                      await github.rest.issues.removeLabel({ owner, repo, issue_number: issueNum, name: 'in-progress' });
+                    } catch (e) { /* label may not exist */ }
                   }
                   return;
                 } catch (e) {
@@ -1393,6 +1409,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 +1766,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.58",
   "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

@@ -20,12 +20,17 @@ const REPORT_DATA_DIR = "/tmp/report-data";
 async function findLatestInitRun(octokit, owner, repo) {
   try {
     const { data } = await octokit.rest.actions.listWorkflowRunsForRepo({
-      owner, repo, per_page: 20,
+      owner, repo, per_page: 50,
     });
-    const initRuns = data.workflow_runs
-      .filter(r => r.name && r.name.includes("agentic-lib-init"))
+    // Look for standalone init runs or flow runs (init may be embedded in flow)
+    const relevantRuns = data.workflow_runs
+      .filter(r => r.name && (r.name.includes("agentic-lib-init") || r.name.includes("agentic-lib-flow")))
       .sort((a, b) => new Date(b.created_at) - new Date(a.created_at));
-    if (initRuns.length > 0) return initRuns[0].created_at;
+    // Prefer the most recent init run; fall back to the most recent flow run
+    const initRun = relevantRuns.find(r => r.name.includes("agentic-lib-init"));
+    const flowRun = relevantRuns.find(r => r.name.includes("agentic-lib-flow"));
+    const bestRun = initRun || flowRun;
+    if (bestRun) return bestRun.created_at;
   } catch (err) {
     core.warning(`Could not find init runs: ${err.message}`);
   }
@@ -47,16 +52,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.58"
   },
   "devDependencies": {
     "@playwright/test": "^1.58.0",