@iinm/plain-agent 1.10.6 → 1.10.9

package/README.md

@@ -381,7 +381,7 @@ Files are loaded in the following order. Settings in later files override earlie
           "args": ["ci"]
         },
         "mode": "sandbox",
-        "extraArgs": ["--allow-net", "0.0.0.0/0"]
+        "additionalArgs": ["--allow-net", "0.0.0.0/0"]
       }
     ]
   }

package/bin/plain

@@ -1,3 +1,8 @@
 #!/usr/bin/env node
 
-import("../src/main.mjs");
+import { main } from "../src/main.mjs";
+
+main().catch((err) => {
+  console.error(err);
+  process.exit(1);
+});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@iinm/plain-agent",
-  "version": "1.10.6",
+  "version": "1.10.9",
   "description": "A lightweight CLI-based coding agent",
   "license": "MIT",
   "type": "module",
@@ -31,6 +31,7 @@
   "scripts": {
     "check": "npm run lint && tsc && npm run test",
     "test": "node --test",
+    "coverage": "node --experimental-test-coverage --test-coverage-exclude='src/**/*.test.mjs' --test",
     "lint": "npx @biomejs/biome check",
     "fix": "npx @biomejs/biome check --fix"
   },

package/src/cli/formatter.mjs

@@ -846,7 +846,7 @@ function renderPatchBlock(block, originalLines, nonce) {
     out.push(
       styleText(
         "cyan",
-        `>>> ${nonce} ${block.start}:${block.startHash}-${block.end}:${block.endHash}`,
+        `@@@ ${nonce} ${block.start}:${block.startHash}-${block.end}:${block.endHash}`,
       ),
     );
     if (originalLines) {
@@ -874,12 +874,11 @@ function renderPatchBlock(block, originalLines, nonce) {
     }
   } else {
     const afterSuffix = block.afterHash ? `:${block.afterHash}` : "";
-    out.push(styleText("cyan", `>>> ${nonce} ${block.after}${afterSuffix}+`));
+    out.push(styleText("cyan", `@@@ ${nonce} ${block.after}${afterSuffix}+`));
     for (const line of block.body) {
       out.push(styleText("green", `+ ${line}`));
     }
   }
-  out.push(styleText("cyan", `<<< ${nonce}`));
   return out.join("\n");
 }
 
@@ -893,8 +892,8 @@ function highlightPatchPlain(patch) {
   if (!patch) {
     return "";
   }
-  // Patch open/close markers look like ">>> <nonce> ..." or "<<< <nonce>".
-  const headerRegex = /^(>>>|<<<)\s+\S+(\s.*)?$/;
+  // Patch headers look like "@@@ <nonce> ...".
+  const headerRegex = /^@@@\s+\S+(\s.*)?$/;
   return patch
     .split("\n")
     .map((line) => {
@@ -915,7 +914,7 @@ function highlightPatchPlain(patch) {
  * @returns {string | null}
  */
 function extractPatchNonce(patch) {
-  const match = patch.match(/^>>>\s+(\S+)/m);
+  const match = patch.match(/^@@@\s+(\S+)/m);
   return match ? match[1] : null;
 }
 

package/src/main.mjs

@@ -34,64 +34,70 @@ import { createWebSearchTool } from "./tools/webSearch.mjs";
 import { writeFileTool } from "./tools/writeFile.mjs";
 import { createToolUseApprover } from "./toolUseApprover.mjs";
 
-const cliArgs = parseCliArgs(process.argv);
-if (cliArgs.subcommand.type === "help") {
-  printHelp();
-}
-
-if (cliArgs.subcommand.type === "list-models") {
-  const { appConfig } = await loadAppConfig({ skipTrustCheck: true });
-  if (!appConfig.models || appConfig.models.length === 0) {
-    console.error("No models found in configuration.");
-    process.exit(1);
-  }
-  for (const model of appConfig.models) {
-    const platform = model.platform;
-    console.log(
-      `${model.name}+${model.variant} (platform: ${platform.name}+${platform.variant})`,
-    );
+/**
+ * CLI entry point. Separated from top-level so that importing this module
+ * does not start the application — required for code-coverage smoke tests.
+ *
+ * @param {string[]} argv - Typically `process.argv`.
+ */
+export async function main(argv = process.argv) {
+  const cliArgs = parseCliArgs(argv);
+  if (cliArgs.subcommand.type === "help") {
+    printHelp();
   }
-  process.exit(0);
-}
 
-if (cliArgs.subcommand.type === "install-claude-code-plugins") {
-  await installClaudeCodePlugins();
-  process.exit(0);
-}
-
-if (cliArgs.subcommand.type === "cost") {
-  try {
-    const exitCode = await runCostCommand({
-      from: cliArgs.subcommand.from,
-      to: cliArgs.subcommand.to,
-    });
-    process.exit(exitCode);
-  } catch (err) {
-    const message = err instanceof Error ? err.message : String(err);
-    console.error(message);
-    process.exit(1);
+  if (cliArgs.subcommand.type === "list-models") {
+    const { appConfig } = await loadAppConfig({ skipTrustCheck: true });
+    if (!appConfig.models || appConfig.models.length === 0) {
+      console.error("No models found in configuration.");
+      process.exit(1);
+    }
+    for (const model of appConfig.models) {
+      const platform = model.platform;
+      console.log(
+        `${model.name}+${model.variant} (platform: ${platform.name}+${platform.variant})`,
+      );
+    }
+    process.exit(0);
   }
-}
 
-if (cliArgs.subcommand.type === "resume" && cliArgs.subcommand.list) {
-  const sessions = await listSessions();
-  if (sessions.length === 0) {
-    console.log("No resumable sessions in .plain-agent/sessions/.");
+  if (cliArgs.subcommand.type === "install-claude-code-plugins") {
+    await installClaudeCodePlugins();
     process.exit(0);
   }
-  console.log("Resumable sessions (most recently updated first):\n");
-  for (const s of sessions) {
-    console.log(
-      `  ${s.sessionId}  ${s.modelName}  (updated ${formatLocalDateTime(s.lastUpdatedAt)}, ${s.messageCount} messages)`,
-    );
-    if (s.workingDir !== process.cwd()) {
-      console.log(`    workingDir: ${s.workingDir}`);
+
+  if (cliArgs.subcommand.type === "cost") {
+    try {
+      const exitCode = await runCostCommand({
+        from: cliArgs.subcommand.from,
+        to: cliArgs.subcommand.to,
+      });
+      process.exit(exitCode);
+    } catch (err) {
+      const message = err instanceof Error ? err.message : String(err);
+      console.error(message);
+      process.exit(1);
     }
   }
-  process.exit(0);
-}
 
-(async () => {
+  if (cliArgs.subcommand.type === "resume" && cliArgs.subcommand.list) {
+    const sessions = await listSessions();
+    if (sessions.length === 0) {
+      console.log("No resumable sessions in .plain-agent/sessions/.");
+      process.exit(0);
+    }
+    console.log("Resumable sessions (most recently updated first):\n");
+    for (const s of sessions) {
+      console.log(
+        `  ${s.sessionId}  ${s.modelName}  (updated ${formatLocalDateTime(s.lastUpdatedAt)}, ${s.messageCount} messages)`,
+      );
+      if (s.workingDir !== process.cwd()) {
+        console.log(`    workingDir: ${s.workingDir}`);
+      }
+    }
+    process.exit(0);
+  }
+
   /** @type {SessionState | null} */
   let resumedState = null;
 
@@ -430,10 +436,7 @@ if (cliArgs.subcommand.type === "resume" && cliArgs.subcommand.list) {
       voiceInput: appConfig.voiceInput,
     });
   }
-})().catch((err) => {
-  console.error(err);
-  process.exit(1);
-});
+}
 
 /**
  * Generate a session id of the form `YYYY-MM-DD-HHMM-<3 random base36 chars>`.

package/src/prompt.mjs

@@ -70,6 +70,10 @@ Memory files should include:
 
 Call multiple tools at once when they don't depend on each other's results.
 
+## patch_file
+
+Always read the target lines with \`read_file\` first to verify line numbers and their 2-char hashes before calling \`patch_file\`.
+
 ## exec_command
 
 - Use relative paths.

package/src/tools/patchFile.mjs

@@ -17,8 +17,9 @@ export function createPatchFileTool(
   return {
     def: {
       name: "patch_file",
-      description:
-        "Modify a file by replacing or inserting content addressed by line numbers (1-indexed).",
+      description: `Modify a file by replacing or inserting content.
+When editing multiple locations in the same file, include all blocks in a single patch string rather than making multiple separate calls.
+      `.trim(),
       inputSchema: {
         type: "object",
         properties: {
@@ -28,29 +29,20 @@ export function createPatchFileTool(
           patch: {
             description: `
 Format — a single patch string may contain multiple blocks:
->>> ${nonce} {start}:{startHash}-{end}:{endHash}
-new content
-<<< ${nonce}
-
->>> ${nonce} {start}:{startHash}-{end}:{endHash}
-another new content
-<<< ${nonce}
-
->>> ${nonce} {N}:{afterHash}+
+@@@ ${nonce} {start}:{startHash}-{end}:{endHash}
+replacement for lines {start}-{end}
+@@@ ${nonce} {N}:{afterHash}+
 appended content after line N
-<<< ${nonce}
-
->>> ${nonce} 0+
-prepended content
-<<< ${nonce}
-
->>> ${nonce} 10:ab-15:cd
+@@@ ${nonce} 0+
+prepended content at beginning of file
+@@@ ${nonce} {N}:{hash}
+replace just that one line
+@@@ ${nonce} 10:ab-15:cd
 (empty body deletes the range)
-<<< ${nonce}
 
+- Each block's content starts right after its @@@ header line and ends at the next @@@ or the end of the string. Any blank lines between the header and the content become part of the replacement.
 - The nonce "${nonce}" is constant; always use the exact value shown above.
-- Line numbers are 1-indexed and refer to the original file; "{start}-{end}" is inclusive.
-- Hashes are 2-character hex hashes of each line's full content as shown by read_file (e.g. "a3").
+- Hashes are 2-character hex hashes of each line's full content as shown by read_file.
             `.trim(),
             type: "string",
           },
@@ -69,7 +61,7 @@ prepended content
         const blocks = parseBlocks(patch, nonce);
         if (blocks.length === 0) {
           throw new Error(
-            `No patch blocks found. Each block must start with ">>> ${nonce} ..." and end with "<<< ${nonce}".`,
+            `No patch blocks found. Each block must start with "@@@ ${nonce} ...".`,
           );
         }
 
@@ -99,52 +91,47 @@ prepended content
  * @returns {PatchBlock[]}
  */
 export function parseBlocks(patch, nonce) {
-  const openPrefix = `>>> ${nonce} `;
-  const closeMarker = `<<< ${nonce}`;
+  const openPrefix = `@@@ ${nonce} `;
   const lines = patch.split("\n");
-
-  /** @type {PatchBlock[]} */
-  const blocks = [];
+  // Drop trailing empty element produced by split() when patch ends with \n.
+  if (lines.length > 0 && lines[lines.length - 1] === "") {
+    lines.pop();
+  }
+  // Find all header line indices
+  /** @type {number[]} */
+  const headerIndices = [];
   for (let i = 0; i < lines.length; i++) {
-    const line = lines[i];
-    if (line === "") {
-      continue;
-    }
-    if (line === closeMarker) {
-      throw new Error(
-        `Unexpected close marker "${closeMarker}" with no matching open block (line ${i + 1} of patch).`,
-      );
-    }
-    if (!line.startsWith(openPrefix)) {
-      throw new Error(
-        `Expected block header starting with "${openPrefix}" but got: ${JSON.stringify(line)} (line ${i + 1} of patch).`,
-      );
+    if (lines[i].startsWith(openPrefix)) {
+      headerIndices.push(i);
     }
+  }
 
-    const headerArgs = line.slice(openPrefix.length);
+  if (headerIndices.length === 0) {
+    throw new Error(
+      `No patch blocks found. Each block must start with "@@@ ${nonce} ...".`,
+    );
+  }
+
+  /** @type {PatchBlock[]} */
+  const blocks = [];
+  for (let i = 0; i < headerIndices.length; i++) {
+    const headerLineIdx = headerIndices[i];
+    const headerLine = lines[headerLineIdx];
+    const headerArgs = headerLine.slice(openPrefix.length);
     const header = parseHeaderArgs(headerArgs);
-    const closeIdx = lines.indexOf(closeMarker, i + 1);
-    if (closeIdx === -1) {
-      throw new Error(
-        `Missing close marker "${closeMarker}" for block "${openPrefix}${headerArgs}".`,
-      );
-    }
-    const body = lines.slice(i + 1, closeIdx);
-    const nestedOpen = body.findIndex((l) => l.startsWith(openPrefix));
-    if (nestedOpen !== -1) {
-      throw new Error(
-        `Unclosed block "${openPrefix}${headerArgs}": found another open marker "${body[nestedOpen]}" ` +
-          `at line ${i + 1 + nestedOpen + 1} of patch before the close marker. ` +
-          `Did you forget "${closeMarker}" to close the previous block?`,
-      );
-    }
+
+    // Body: from the line after the header to the line before the next header (or EOF)
+    const bodyStart = headerLineIdx + 1;
+    const bodyEnd =
+      i + 1 < headerIndices.length ? headerIndices[i + 1] : lines.length;
+    const body = lines.slice(bodyStart, bodyEnd);
+
     if (header.op === "insert" && body.length === 0) {
       throw new Error(
-        `Insert block "${openPrefix}${headerArgs}" has empty body. Use a replace block to delete content.`,
+        `Insert block "@@@ ${nonce} ${headerArgs}" has empty body. Use a replace block to delete content.`,
       );
     }
     blocks.push({ ...header, body });
-    i = closeIdx;
   }
   return blocks;
 }
@@ -191,6 +178,7 @@ export function applyBlocks(original, blocks) {
 
   for (const { block } of indexed) {
     if (block.op === "replace") {
+      const end = block.end;
       const actualStart = lines[block.start - 1];
       const expectedStartHash = block.startHash;
       const actualStartHash = lineHash(actualStart ?? "");
@@ -199,15 +187,15 @@ export function applyBlocks(original, blocks) {
           `Hash verification failed at line ${block.start}: expected hash ${expectedStartHash} but got ${actualStartHash} for line ${JSON.stringify(actualStart)}. The line numbers may be stale; re-read the file with read_file.`,
         );
       }
-      const actualEnd = lines[block.end - 1];
+      const actualEnd = lines[end - 1];
       const expectedEndHash = block.endHash;
       const actualEndHash = lineHash(actualEnd ?? "");
       if (actualEndHash !== expectedEndHash) {
         throw new Error(
-          `Hash verification failed at line ${block.end}: expected hash ${expectedEndHash} but got ${actualEndHash} for line ${JSON.stringify(actualEnd)}. The line numbers may be stale; re-read the file with read_file.`,
+          `Hash verification failed at line ${end}: expected hash ${expectedEndHash} but got ${actualEndHash} for line ${JSON.stringify(actualEnd)}. The line numbers may be stale; re-read the file with read_file.`,
         );
       }
-      const removeCount = block.end - block.start + 1;
+      const removeCount = end - block.start + 1;
       lines.splice(block.start - 1, removeCount, ...block.body);
     } else {
       if (block.after > 0) {
@@ -240,6 +228,7 @@ function parseHeaderArgs(headerArgs) {
   const replaceMatch = headerArgs.match(
     /^(\d+):([a-f0-9]{2})-(\d+):([a-f0-9]{2})\s*$/,
   );
+
   if (replaceMatch) {
     const start = Number(replaceMatch[1]);
     const end = Number(replaceMatch[3]);
@@ -261,12 +250,33 @@ function parseHeaderArgs(headerArgs) {
       endHash: replaceMatch[4],
     };
   }
+
+  // Replace form: "{N}:{hash}" (single line replace — shorthand for N:hash-N:hash)
+  const singleReplaceMatch = headerArgs.match(/^(\d+):([a-f0-9]{2})\s*$/);
+  if (singleReplaceMatch) {
+    const start = Number(singleReplaceMatch[1]);
+    if (start < 1) {
+      throw new Error(
+        `Invalid replace range "${headerArgs}": start must be >= 1.`,
+      );
+    }
+    return {
+      op: "replace",
+      start,
+      end: start,
+      startHash: singleReplaceMatch[2],
+      endHash: singleReplaceMatch[2],
+    };
+  }
+
   // Insert form: "0+" (no hash — there is no line 0 to verify)
   if (/^0\+\s*$/.test(headerArgs)) {
     return { op: "insert", after: 0, afterHash: "" };
   }
+
   // Insert form: "{N}:{afterHash}+"
   const insertMatch = headerArgs.match(/^(\d+):([a-f0-9]{2})\+\s*$/);
+
   if (insertMatch) {
     return {
       op: "insert",
@@ -274,8 +284,9 @@ function parseHeaderArgs(headerArgs) {
       afterHash: insertMatch[2],
     };
   }
+
   throw new Error(
-    `Invalid block header arguments: ${JSON.stringify(headerArgs)}. Expected "{start}:{startHash}-{end}:{endHash}" or "{N}:{afterHash}+" or "0+".`,
+    `Invalid block header arguments: ${JSON.stringify(headerArgs)}. Expected "{start}:{startHash}-{end}:{endHash}" or "{N}:{hash}" or "{N}:{afterHash}+" or "0+".`,
   );
 }
 
@@ -294,7 +305,10 @@ function spliceIndexOf(block) {
 function validateBlocks(blocks, totalLines) {
   for (const block of blocks) {
     if (block.op === "replace") {
-      if (totalLines < block.end) {
+      // Both bounds must be within [1, totalLines]. The two checks are NOT
+      // redundant: totalLines < end is false even if start > totalLines
+      // (e.g. start=1 on an empty file).
+      if (block.start > totalLines || totalLines < block.end) {
         throw new Error(
           `Replace range ${block.start}-${block.end} extends past end of file (${totalLines} lines).`,
         );