@xn-intenton-z2a/agentic-lib 7.1.23 → 7.1.24

package/bin/agentic-lib.js

@@ -738,43 +738,38 @@ function initConfig(seedsDir) {
   }
 }
 
-function initReseed() {
-  console.log("\n--- Reseed: Clear Features + Activity Log ---");
-  const intentionFile = resolve(target, "intentïon.md");
-  if (existsSync(intentionFile)) {
-    if (!dryRun) rmSync(intentionFile);
-    console.log("  REMOVE: intentïon.md");
+function removeFile(filePath, label) {
+  if (existsSync(filePath)) {
+    if (!dryRun) rmSync(filePath);
+    console.log(`  REMOVE: ${label}`);
     initChanges++;
   }
-  // Clear features directory (now at project root, next to library/)
-  const featuresDir = resolve(target, "features");
-  if (existsSync(featuresDir)) {
-    for (const f of readdirSync(featuresDir)) {
-      if (!dryRun) rmSync(resolve(featuresDir, f));
-      console.log(`  REMOVE: features/${f}`);
-      initChanges++;
-    }
+}
+
+function clearDirContents(dirPath, label) {
+  if (!existsSync(dirPath)) return;
+  for (const f of readdirSync(dirPath)) {
+    if (!dryRun) rmSync(resolve(dirPath, f));
+    console.log(`  REMOVE: ${label}/${f}`);
+    initChanges++;
   }
-  // Also clear old features location if it exists
+}
+
+function initReseed() {
+  console.log("\n--- Reseed: Clear Features + Activity Log ---");
+  removeFile(resolve(target, "intentïon.md"), "intentïon.md");
+  clearDirContents(resolve(target, "features"), "features");
+
+  // Clear old features location if it exists
   const oldFeaturesDir = resolve(target, ".github/agentic-lib/features");
+  clearDirContents(oldFeaturesDir, ".github/agentic-lib/features (old location)");
   if (existsSync(oldFeaturesDir)) {
-    for (const f of readdirSync(oldFeaturesDir)) {
-      if (!dryRun) rmSync(resolve(oldFeaturesDir, f));
-      console.log(`  REMOVE: .github/agentic-lib/features/${f} (old location)`);
-      initChanges++;
-    }
     if (!dryRun) rmdirSync(oldFeaturesDir);
     console.log("  REMOVE: .github/agentic-lib/features/ (old location)");
   }
-  // Clear library directory (generated from SOURCES.md)
-  const libraryDir = resolve(target, "library");
-  if (existsSync(libraryDir)) {
-    for (const f of readdirSync(libraryDir)) {
-      if (!dryRun) rmSync(resolve(libraryDir, f));
-      console.log(`  REMOVE: library/${f}`);
-      initChanges++;
-    }
-  }
+
+  clearDirContents(resolve(target, "library"), "library");
+
   // Remove old getting-started-guide if it exists
   const oldGuideDir = resolve(target, ".github/agentic-lib/getting-started-guide");
   if (existsSync(oldGuideDir)) {
@@ -784,10 +779,7 @@ function initReseed() {
   }
 }
 
-function initPurge(seedsDir) {
-  console.log("\n--- Purge: Reset Source Files to Seed State ---");
-
-  // Read TOML to get source and tests paths (or use defaults)
+function readTomlPaths() {
   let sourcePath = "src/lib/";
   let testsPath = "tests/unit/";
   const tomlTarget = resolve(target, "agentic-lib.toml");
@@ -802,24 +794,25 @@ function initPurge(seedsDir) {
       console.log(`  WARN: Could not read TOML for paths, using defaults: ${err.message}`);
     }
   }
+  return { sourcePath, testsPath };
+}
 
-  // Clear source directory completely
-  const sourceDir = resolve(target, sourcePath);
-  if (existsSync(sourceDir)) {
-    console.log(`  CLEAR: ${sourcePath}`);
-    if (!dryRun) rmSync(sourceDir, { recursive: true });
+function clearAndRecreateDir(dirPath, label) {
+  const fullPath = resolve(target, dirPath);
+  if (existsSync(fullPath)) {
+    console.log(`  CLEAR: ${label}`);
+    if (!dryRun) rmSync(fullPath, { recursive: true });
     initChanges++;
   }
-  if (!dryRun) mkdirSync(sourceDir, { recursive: true });
+  if (!dryRun) mkdirSync(fullPath, { recursive: true });
+}
 
-  // Clear tests directory completely
-  const testsDir = resolve(target, testsPath);
-  if (existsSync(testsDir)) {
-    console.log(`  CLEAR: ${testsPath}`);
-    if (!dryRun) rmSync(testsDir, { recursive: true });
-    initChanges++;
-  }
-  if (!dryRun) mkdirSync(testsDir, { recursive: true });
+function initPurge(seedsDir) {
+  console.log("\n--- Purge: Reset Source Files to Seed State ---");
+
+  const { sourcePath, testsPath } = readTomlPaths();
+  clearAndRecreateDir(sourcePath, sourcePath);
+  clearAndRecreateDir(testsPath, testsPath);
 
   // Copy seed files
   const SEED_MAP = {

package/package.json

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

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

@@ -11,127 +11,95 @@ import { runCopilotTask, readOptionalFile, scanDirectory } from "../copilot.js";
 
 const BOT_LOGINS = ["github-actions[bot]", "github-actions"];
 
-/**
- * Respond to a GitHub Discussion using the Copilot SDK.
- *
- * @param {Object} context - Task context from index.js
- * @returns {Promise<Object>} Result with outcome, action, tokensUsed, model
- */
-export async function discussions(context) {
-  const { octokit, config, instructions, model, discussionUrl } = context;
-
-  if (!discussionUrl) {
-    throw new Error("discussions task requires discussion-url input");
+async function fetchDiscussion(octokit, discussionUrl) {
+  const urlMatch = discussionUrl.match(/github\.com\/([^/]+)\/([^/]+)\/discussions\/(\d+)/);
+  if (!urlMatch) {
+    core.warning(`Could not parse discussion URL: ${discussionUrl}`);
+    return { title: "", body: "", comments: [], nodeId: "" };
   }
 
-  // Parse discussion URL and fetch content via GraphQL
-  const urlMatch = discussionUrl.match(/github\.com\/([^/]+)\/([^/]+)\/discussions\/(\d+)/);
-  let discussionTitle = "";
-  let discussionBody = "";
-  let discussionComments = [];
-  let discussionNodeId = "";
-
-  if (urlMatch) {
-    const [, urlOwner, urlRepo, discussionNumber] = urlMatch;
-    try {
-      const query = `query {
-        repository(owner: "${urlOwner}", name: "${urlRepo}") {
-          discussion(number: ${discussionNumber}) {
-            id
-            title
-            body
-            comments(last: 10) {
-              nodes {
-                body
-                author { login }
-                createdAt
-              }
+  const [, urlOwner, urlRepo, discussionNumber] = urlMatch;
+  try {
+    const query = `query {
+      repository(owner: "${urlOwner}", name: "${urlRepo}") {
+        discussion(number: ${discussionNumber}) {
+          id
+          title
+          body
+          comments(last: 10) {
+            nodes {
+              body
+              author { login }
+              createdAt
             }
           }
         }
-      }`;
-      const result = await octokit.graphql(query);
-      const discussion = result.repository.discussion;
-      discussionNodeId = discussion.id || "";
-      discussionTitle = discussion.title || "";
-      discussionBody = discussion.body || "";
-      discussionComments = discussion.comments.nodes || [];
-      core.info(
-        `Fetched discussion #${discussionNumber}: "${discussionTitle}" (${discussionComments.length} comments)`,
-      );
-    } catch (err) {
-      core.warning(`Failed to fetch discussion content via GraphQL: ${err.message}. Falling back to URL-only.`);
-    }
-  } else {
-    core.warning(`Could not parse discussion URL: ${discussionUrl}`);
+      }
+    }`;
+    const result = await octokit.graphql(query);
+    const discussion = result.repository.discussion;
+    core.info(
+      `Fetched discussion #${discussionNumber}: "${discussion.title}" (${discussion.comments.nodes.length} comments)`,
+    );
+    return {
+      title: discussion.title || "",
+      body: discussion.body || "",
+      comments: discussion.comments.nodes || [],
+      nodeId: discussion.id || "",
+    };
+  } catch (err) {
+    core.warning(`Failed to fetch discussion content via GraphQL: ${err.message}. Falling back to URL-only.`);
+    return { title: "", body: "", comments: [], nodeId: "" };
   }
+}
 
-  // Separate human comments from bot's own replies
-  const humanComments = discussionComments.filter(
-    (c) => !BOT_LOGINS.includes(c.author?.login),
-  );
-  const botReplies = discussionComments.filter(
-    (c) => BOT_LOGINS.includes(c.author?.login),
-  );
+function buildPrompt(discussionUrl, discussion, context) {
+  const { config, instructions } = context;
+  const { title, body, comments } = discussion;
+
+  const humanComments = comments.filter((c) => !BOT_LOGINS.includes(c.author?.login));
+  const botReplies = comments.filter((c) => BOT_LOGINS.includes(c.author?.login));
   const latestHumanComment = humanComments.length > 0 ? humanComments[humanComments.length - 1] : null;
   const lastBotReply = botReplies.length > 0 ? botReplies[botReplies.length - 1] : null;
 
   const mission = readOptionalFile(config.paths.mission.path);
   const contributing = readOptionalFile(config.paths.contributing.path, 1000);
-
   const featuresPath = config.paths.features.path;
-  let featureNames = [];
-  if (existsSync(featuresPath)) {
-    featureNames = scanDirectory(featuresPath, ".md").map((f) => f.name.replace(".md", ""));
-  }
-
-  const intentionPath = config.intentionBot.intentionFilepath;
-  const recentActivity = readOptionalFile(intentionPath).split("\n").slice(-20).join("\n");
-
+  const featureNames = existsSync(featuresPath)
+    ? scanDirectory(featuresPath, ".md").map((f) => f.name.replace(".md", ""))
+    : [];
+  const recentActivity = readOptionalFile(config.intentionBot.intentionFilepath).split("\n").slice(-20).join("\n");
   const agentInstructions = instructions || "Respond to the GitHub Discussion as the repository bot.";
 
-  // Build thread-aware prompt
-  const promptParts = [
+  const parts = [
     "## Instructions",
     agentInstructions,
     "",
     "## Discussion Thread",
     `URL: ${discussionUrl}`,
-    discussionTitle ? `### ${discussionTitle}` : "",
-    discussionBody || "(no body)",
+    title ? `### ${title}` : "",
+    body || "(no body)",
   ];
 
-  // Show conversation history (human comments only, with timestamps)
   if (humanComments.length > 0) {
-    promptParts.push("", "### Conversation History");
+    parts.push("", "### Conversation History");
     for (const c of humanComments) {
-      const isLatest = c === latestHumanComment;
-      const prefix = isLatest ? ">>> **[LATEST — RESPOND TO THIS]** " : "";
-      promptParts.push(`${prefix}**${c.author?.login || "unknown"}** (${c.createdAt}):\n${c.body}`);
+      const prefix = c === latestHumanComment ? ">>> **[LATEST — RESPOND TO THIS]** " : "";
+      parts.push(`${prefix}**${c.author?.login || "unknown"}** (${c.createdAt}):\n${c.body}`);
     }
   }
 
-  // Show the bot's last reply so it knows what it already said
   if (lastBotReply) {
-    promptParts.push(
-      "",
-      "### Your Last Reply (DO NOT REPEAT THIS)",
-      lastBotReply.body.substring(0, 500),
-    );
+    parts.push("", "### Your Last Reply (DO NOT REPEAT THIS)", lastBotReply.body.substring(0, 500));
   }
 
-  // Context section
-  promptParts.push(
+  parts.push(
     "",
     "## Repository Context",
     `### Mission\n${mission}`,
     contributing ? `### Contributing\n${contributing}` : "",
     `### Current Features\n${featureNames.join(", ") || "none"}`,
     recentActivity ? `### Recent Activity\n${recentActivity}` : "",
-  );
-
-  // Actions — concise, not a wall of text
-  promptParts.push(
     "",
     "## Actions",
     "Include exactly one action tag in your response. Only mention actions to the user when relevant.",
@@ -146,7 +114,49 @@ export async function discussions(context) {
     "`[ACTION:stop]` — Halt automation",
   );
 
-  const prompt = promptParts.filter(Boolean).join("\n");
+  return parts.filter(Boolean).join("\n");
+}
+
+async function postReply(octokit, nodeId, replyBody) {
+  if (!nodeId) {
+    core.warning("Cannot post reply: discussion node ID not available");
+    return;
+  }
+  if (!replyBody) {
+    core.warning("Cannot post reply: no reply content generated");
+    return;
+  }
+  try {
+    const mutation = `mutation($discussionId: ID!, $body: String!) {
+      addDiscussionComment(input: { discussionId: $discussionId, body: $body }) {
+        comment { url }
+      }
+    }`;
+    const { addDiscussionComment } = await octokit.graphql(mutation, {
+      discussionId: nodeId,
+      body: replyBody,
+    });
+    core.info(`Posted reply to discussion: ${addDiscussionComment.comment.url}`);
+  } catch (err) {
+    core.warning(`Failed to post discussion reply: ${err.message}`);
+  }
+}
+
+/**
+ * Respond to a GitHub Discussion using the Copilot SDK.
+ *
+ * @param {Object} context - Task context from index.js
+ * @returns {Promise<Object>} Result with outcome, action, tokensUsed, model
+ */
+export async function discussions(context) {
+  const { octokit, model, discussionUrl } = context;
+
+  if (!discussionUrl) {
+    throw new Error("discussions task requires discussion-url input");
+  }
+
+  const discussion = await fetchDiscussion(octokit, discussionUrl);
+  const prompt = buildPrompt(discussionUrl, discussion, context);
 
   const { content, tokensUsed } = await runCopilotTask({
     model,
@@ -156,35 +166,13 @@ export async function discussions(context) {
     writablePaths: [],
   });
 
-  // Parse action from response
   const actionMatch = content.match(/\[ACTION:(\S+?)\](.+)?/);
   const action = actionMatch ? actionMatch[1] : "nop";
   const actionArg = actionMatch && actionMatch[2] ? actionMatch[2].trim() : "";
   const replyBody = content.replace(/\[ACTION:\S+?\].+/, "").trim();
 
   core.info(`Discussion bot action: ${action}, arg: ${actionArg}`);
-
-  // Post reply comment back to the Discussion
-  if (discussionNodeId && replyBody) {
-    try {
-      const mutation = `mutation($discussionId: ID!, $body: String!) {
-        addDiscussionComment(input: { discussionId: $discussionId, body: $body }) {
-          comment { url }
-        }
-      }`;
-      const { addDiscussionComment } = await octokit.graphql(mutation, {
-        discussionId: discussionNodeId,
-        body: replyBody,
-      });
-      core.info(`Posted reply to discussion: ${addDiscussionComment.comment.url}`);
-    } catch (err) {
-      core.warning(`Failed to post discussion reply: ${err.message}`);
-    }
-  } else if (!discussionNodeId) {
-    core.warning("Cannot post reply: discussion node ID not available");
-  } else if (!replyBody) {
-    core.warning("Cannot post reply: no reply content generated");
-  }
+  await postReply(octokit, discussion.nodeId, replyBody);
 
   const argSuffix = actionArg ? ` (${actionArg})` : "";
   return {

package/src/seeds/zero-package.json

@@ -14,7 +14,7 @@
   "author": "",
   "license": "MIT",
   "dependencies": {
-    "@xn-intenton-z2a/agentic-lib": "^7.1.23"
+    "@xn-intenton-z2a/agentic-lib": "^7.1.24"
   },
   "devDependencies": {
     "@vitest/coverage-v8": "^4.0.0",