@xn-intenton-z2a/agentic-lib 7.4.19 → 7.4.21

package/package.json

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

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

@@ -156,6 +156,8 @@ function buildPrompt(ctx, agentInstructions, metricAssessment) {
     "Use read_file to inspect source code and tests for completeness.",
     "Use git_diff or git_status for additional context if needed.",
     "Consider the implementation review findings — if critical gaps exist, do NOT declare mission-complete.",
+    "Check the acceptance criteria in the Mission section above. If all criteria are clearly satisfied by the current source code and tests (verified via read_file), you SHOULD declare mission-complete even if not all mechanical metrics are MET.",
+    "For simple missions (few functions, clear acceptance criteria), do not require elaborate test coverage or documentation beyond what the acceptance criteria specify.",
     "Then call report_director_decision with your determination.",
     "",
     "**You MUST call report_director_decision exactly once.**",
@@ -455,6 +457,7 @@ export async function direct(context) {
   if (logFilePath) attachments.push({ type: "file", path: logFilePath });
   if (screenshotFilePath) attachments.push({ type: "file", path: screenshotFilePath });
 
+  const sessionStartTime = Date.now();
   const result = await runCopilotSession({
     workspacePath: process.cwd(),
     model,
@@ -467,6 +470,8 @@ export async function direct(context) {
     excludedTools: ["write_file", "run_command", "run_tests", "dispatch_workflow", "close_issue", "label_issue", "post_discussion_comment", "create_issue", "comment_on_issue"],
     logger: { info: core.info, warning: core.warning, error: core.error, debug: core.debug },
   });
+  const sessionDurationMs = Date.now() - sessionStartTime;
+  core.info(`Director session completed in ${Math.round(sessionDurationMs / 1000)}s (${result.tokensIn + result.tokensOut} tokens)`);
 
   const tokensUsed = result.tokensIn + result.tokensOut;
 
@@ -482,14 +487,11 @@ export async function direct(context) {
 
   // Execute the decision
   let outcome = "directed";
-  if (decision === "mission-complete" && metricAssessment.allMet) {
+  if (decision === "mission-complete") {
     if (process.env.GITHUB_REPOSITORY !== "xn-intenton-z2a/agentic-lib") {
       await executeMissionComplete(octokit, repo, reason);
       outcome = "mission-complete";
     }
-  } else if (decision === "mission-complete" && !metricAssessment.allMet) {
-    core.info("Director chose mission-complete but metrics are NOT all met — overriding to in-progress");
-    outcome = "directed";
   } else if (decision === "mission-failed") {
     if (process.env.GITHUB_REPOSITORY !== "xn-intenton-z2a/agentic-lib") {
       await executeMissionFailed(octokit, repo, reason, metricAssessment);

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

@@ -609,6 +609,13 @@ async function executeCreateIssue(octokit, repo, params, ctx) {
   if (!title && feature) {
     title = `feat: implement ${feature}`;
   }
+  // Fallback: derive title from mission context when LLM provides no params
+  if (!title && ctx?.mission) {
+    const missionHeading = ctx.mission.match(/^#\s+(.+)/m);
+    const missionName = missionHeading ? missionHeading[1].trim() : "mission";
+    title = `feat: implement ${missionName.toLowerCase()}`;
+    core.info(`create-issue: derived title from mission context: "${title}"`);
+  }
   if (!title) {
     core.warning("create-issue: no title, body, or feature provided — skipping");
     return "skipped:no-title";
@@ -831,7 +838,21 @@ export async function supervise(context) {
               type: "object",
               properties: {
                 action: { type: "string", description: "Action name (e.g. dispatch:agentic-lib-workflow, github:create-issue, set-schedule:weekly, nop)" },
-                params: { type: "object", description: "Action parameters (e.g. mode, issue-number, title, labels, message, discussion-url, frequency)" },
+                params: {
+                  type: "object",
+                  properties: {
+                    title: { type: "string", description: "Issue title (REQUIRED for github:create-issue)" },
+                    body: { type: "string", description: "Issue body or description" },
+                    labels: { type: "string", description: "Comma-separated labels (e.g. 'automated,enhancement')" },
+                    "issue-number": { type: "string", description: "Issue number for dispatch, label, or close actions" },
+                    "pr-number": { type: "string", description: "PR number for dispatch actions" },
+                    mode: { type: "string", description: "Workflow dispatch mode (e.g. 'dev-only', 'maintain-only')" },
+                    frequency: { type: "string", description: "Schedule frequency for set-schedule action" },
+                    message: { type: "string", description: "Message for respond:discussions action" },
+                    "discussion-url": { type: "string", description: "Discussion URL for respond:discussions action" },
+                  },
+                  description: "Action parameters. For github:create-issue, 'title' is required.",
+                },
               },
               required: ["action"],
             },
@@ -845,12 +866,30 @@ export async function supervise(context) {
         planResult.reasoning = reasoning || "";
 
         // Execute each action using existing handlers
+        // W10: Track created issue titles for same-iteration dedup
+        const createdIssueTitles = new Set();
         const results = [];
         for (const { action, params } of (actions || [])) {
           try {
+            // W10: Skip duplicate create-issue within same plan
+            if ((action === "github:create-issue" || action.startsWith("github:create-issue")) && params?.title) {
+              const titleKey = (params.title || "").toLowerCase().substring(0, 40);
+              if (createdIssueTitles.has(titleKey)) {
+                results.push(`skipped:duplicate-same-session:${(params.title || "").substring(0, 50)}`);
+                logger.info(`Skipping duplicate issue in same session: ${params.title}`);
+                continue;
+              }
+            }
+
             const result = await executeAction(octokit, repo, action, params || {}, ctx);
             results.push(result);
             logger.info(`Action result: ${result}`);
+
+            // W10: Track created issues for dedup
+            if (result.startsWith("created-issue:")) {
+              const title = (params?.title || "").toLowerCase().substring(0, 40);
+              if (title) createdIssueTitles.add(title);
+            }
           } catch (err) {
             logger.warning(`Action ${action} failed: ${err.message}`);
             results.push(`error:${action}:${err.message}`);
@@ -870,6 +909,7 @@ export async function supervise(context) {
   if (logFilePath) attachments.push({ type: "file", path: logFilePath });
   if (screenshotFilePath) attachments.push({ type: "file", path: screenshotFilePath });
 
+  const sessionStartTime = Date.now();
   const result = await runCopilotSession({
     workspacePath: process.cwd(),
     model,
@@ -882,6 +922,7 @@ export async function supervise(context) {
     excludedTools: ["write_file", "run_command", "run_tests"],
     logger: { info: core.info, warning: core.warning, error: core.error, debug: core.debug },
   });
+  core.info(`Supervisor session completed in ${Math.round((Date.now() - sessionStartTime) / 1000)}s`);
 
   const tokensUsed = result.tokensIn + result.tokensOut;
 

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

@@ -187,6 +187,10 @@ export async function transform(context) {
   };
 
   // ── Run hybrid session ─────────────────────────────────────────────
+  // Cap tool calls to prevent unbounded sessions (W6: Benchmark 010 finding)
+  const maxToolCalls = Math.max(20, Math.floor((t.maxTokens || 200000) / 5000));
+
+  const sessionStartTime = Date.now();
   const result = await runCopilotSession({
     workspacePath: process.cwd(),
     model,
@@ -196,11 +200,12 @@ export async function transform(context) {
     writablePaths,
     createTools,
     attachments,
+    maxToolCalls,
     excludedTools: ["dispatch_workflow", "close_issue", "label_issue", "post_discussion_comment"],
     logger: { info: core.info, warning: core.warning, error: core.error, debug: core.debug },
   });
-
-  core.info(`Transformation step completed (${result.tokensIn + result.tokensOut} tokens)`);
+  const sessionDurationMs = Date.now() - sessionStartTime;
+  core.info(`Transform session completed in ${Math.round(sessionDurationMs / 1000)}s (${result.tokensIn + result.tokensOut} tokens, maxToolCalls=${maxToolCalls})`);
 
   // Detect mission-complete hint
   const lowerResult = (result.agentMessage || "").toLowerCase();

package/src/seeds/zero-package.json

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