agentloopkit 0.24.2 → 0.24.3

package/README.md

@@ -269,7 +269,7 @@ Each contract records:
 .agentloop/reports/YYYY-MM-DD-HH-mm-verification-report.md
 ```
 
-Pass `--task .agentloop/tasks/file.md` to include task title, type, status, and path in the report. AgentLoopKit reads Markdown task contracts for this flag. It reports `.env`-style paths as unavailable instead of reading them.
+Pass `--task .agentloop/tasks/file.md` to include task title, type, status, and path in the report. The path must point to a Markdown task contract inside the configured task directory. Invalid paths are reported as unavailable instead of being read.
 
 It does not hide failures. Failed reports include a short failure summary with each failed command, exit code, and final useful output lines before the full command output. If long logs are truncated, the report keeps the first and last output so the final error stays visible. If no commands are configured, it writes a report saying nothing was verified.
 

package/dist/cli/index.js

@@ -8,7 +8,7 @@ import { Command } from "commander";
 
 // src/core/init.ts
 import path6 from "path";
-import { readdir as readdir3 } from "fs/promises";
+import { readdir as readdir3, realpath } from "fs/promises";
 import { homedir } from "os";
 
 // src/core/constants.ts
@@ -395,6 +395,13 @@ ${section.trim()}
 `);
   result.updated.push(filePath);
 }
+async function resolveComparablePath(filePath) {
+  try {
+    return await realpath(filePath);
+  } catch {
+    return path6.resolve(filePath);
+  }
+}
 async function initializeAgentLoop(options) {
   const result = {
     created: [],
@@ -404,7 +411,11 @@ async function initializeAgentLoop(options) {
   };
   const cwd = options.cwd;
   const homeDirectory = options.homeDirectory ?? homedir();
-  if (!options.dryRun && !options.force && path6.resolve(cwd) === path6.resolve(homeDirectory)) {
+  const [resolvedCwd, resolvedHomeDirectory] = await Promise.all([
+    resolveComparablePath(cwd),
+    resolveComparablePath(homeDirectory)
+  ]);
+  if (!options.force && resolvedCwd === resolvedHomeDirectory) {
     throw new Error(
       "Refusing to initialize your home directory. Run this inside a project repository, or pass --force if you intentionally want AgentLoopKit files in your home directory."
     );
@@ -843,7 +854,17 @@ ${input.rollbackNotes || "Document how to revert or disable this change."}
 async function createTaskContractFile(options) {
   const createdDate = options.input.createdDate ?? formatDate();
   const relativePath2 = options.out ?? path9.join(options.config.paths.tasksDir, `${createdDate}-${slugify(options.input.title)}.md`);
-  const absolutePath = path9.isAbsolute(relativePath2) ? relativePath2 : path9.join(options.cwd, relativePath2);
+  const absolutePath = path9.isAbsolute(relativePath2) ? path9.resolve(relativePath2) : path9.resolve(options.cwd, relativePath2);
+  const tasksRoot2 = path9.resolve(options.cwd, options.config.paths.tasksDir);
+  const relativeToTasks = path9.relative(tasksRoot2, absolutePath);
+  if (relativeToTasks === "" || relativeToTasks.startsWith("..") || path9.isAbsolute(relativeToTasks)) {
+    throw new AgentLoopError(
+      `Task output path must stay inside ${options.config.paths.tasksDir}.`
+    );
+  }
+  if (!absolutePath.endsWith(".md")) {
+    throw new AgentLoopError("Task output path must be a Markdown file.");
+  }
   const markdown = generateTaskContract({ ...options.input, createdDate });
   await writeTextFile(absolutePath, markdown);
   return { path: absolutePath, markdown };
@@ -1098,10 +1119,16 @@ function isMarkdownTaskPath(taskPath) {
   const segments = normalized.split("/").filter(Boolean);
   return normalized.endsWith(".md") && !segments.some((segment) => segment === ".env" || segment.startsWith(".env."));
 }
-async function renderTaskContext(cwd, taskPath) {
+function isInside(parent, child) {
+  const relative2 = path10.relative(parent, child);
+  return relative2 === "" || !relative2.startsWith("..") && !path10.isAbsolute(relative2);
+}
+async function renderTaskContext(cwd, config, taskPath) {
   if (!taskPath?.trim()) return "";
   const cleanPath = taskPath.trim();
-  if (!isMarkdownTaskPath(cleanPath)) {
+  const absolutePath = path10.isAbsolute(cleanPath) ? path10.resolve(cleanPath) : path10.resolve(cwd, cleanPath);
+  const tasksRoot2 = path10.resolve(cwd, config.paths.tasksDir);
+  if (!isMarkdownTaskPath(cleanPath) || !isInside(tasksRoot2, absolutePath)) {
     return `## Task Context
 - Path: ${cleanPath}
 - Status: unavailable
@@ -1109,7 +1136,6 @@ async function renderTaskContext(cwd, taskPath) {
 
 `;
   }
-  const absolutePath = path10.isAbsolute(cleanPath) ? cleanPath : path10.join(cwd, cleanPath);
   try {
     const markdown = await readFile6(absolutePath, "utf8");
     const metadata = parseTaskMetadata(markdown);
@@ -1168,7 +1194,7 @@ async function runVerification(options) {
   const branch = await getGitBranch(options.cwd);
   const commit = await getGitCommit(options.cwd);
   const status = await getGitStatus(options.cwd);
-  const taskContext = await renderTaskContext(options.cwd, options.taskPath);
+  const taskContext = await renderTaskContext(options.cwd, options.config, options.taskPath);
   const markdown = `# Verification Report
 
 - Timestamp: ${nowIso}
@@ -1274,7 +1300,7 @@ function statePath(cwd, config) {
 function toStoredPath(cwd, absolutePath) {
   return path12.relative(cwd, absolutePath).split(path12.sep).join("/");
 }
-function isInside(parent, child) {
+function isInside2(parent, child) {
   const relative2 = path12.relative(parent, child);
   return relative2 === "" || !relative2.startsWith("..") && !path12.isAbsolute(relative2);
 }
@@ -1303,7 +1329,7 @@ async function resolveTaskPath(options) {
   const absolutePath = path12.isAbsolute(options.taskPath) ? path12.resolve(options.taskPath) : path12.resolve(options.cwd, options.taskPath);
   const root = tasksRoot(options.cwd, options.config);
   const displayRoot = options.config.paths.tasksDir;
-  if (!isInside(root, absolutePath)) {
+  if (!isInside2(root, absolutePath)) {
     if (!options.strict) return void 0;
     throw new AgentLoopError(`Active task must be inside ${displayRoot}.`);
   }