@jc20231028/local-code-agent 0.1.1 → 0.1.3

package/README.md

@@ -49,6 +49,13 @@ local-code chat
 npm 只會把執行檔裝進當下專案的 `node_modules/.bin`，不會加進系統 PATH，
 在 cmd 直接打 `local-code` 會抓不到指令。這種情況下要嘛加 `-g` 重裝，要嘛用 `npx local-code chat` 執行。
 
+如果你已經用沒加 `-g` 的方式裝過，先移除本地安裝再改用全域安裝：
+
+```powershell
+npm uninstall @jc20231028/local-code-agent
+npm install -g @jc20231028/local-code-agent
+```
+
 `provider` / `model` 留空時，`local-code chat` 第一次啟動就會直接跳出互動選單讓你選（見下方「初始化設定」），
 不需要先手動跑 `local-code init`——`init` 只是用來印出設定檔範例，不是必要步驟。
 
@@ -89,6 +96,7 @@ node ./bin/local-code.js init
   "lmStudioBaseUrl": "http://127.0.0.1:1234",
   "maxSteps": 12,
   "allowCommands": false,
+  "allowWrites": false,
   "temperature": 0.2
 }
 ```
@@ -133,6 +141,14 @@ node ./bin/local-code.js run "編譯並執行這個 C# 專案"
 node ./bin/local-code.js run "執行測試並修正失敗案例" --allow-commands
 ```
 
+同樣地，模型呼叫 `write_file`、`append_file`、`replace_in_file`、`make_directory` 這些會建立/覆寫/修改檔案或資料夾的工具時，預設也會先印出要變更的路徑（和內容預覽）並問 `Allow this change? [y/N]:`，按 `y` 才會真的寫入；沒有 TTY 時一樣直接安全拒絕。想跳過詢問可以加 `--allow-writes`：
+
+```powershell
+node ./bin/local-code.js run "幫我建立這個功能的檔案" --allow-writes
+```
+
+`run_command` 之外的其他工具（`list_files`、`read_file`、`search_text`）只是讀取，不會跳出詢問。每一步驟模型在做什麼、呼叫了哪個工具、帶了什麼參數，都會即時印在終端機（stderr），不會等到最後才一次顯示結果。
+
 列出目前可用的 Skill：
 
 ```powershell

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jc20231028/local-code-agent",
-  "version": "0.1.1",
+  "version": "0.1.3",
   "description": "A local coding CLI that uses Ollama or LM Studio models to inspect and modify project files.",
   "type": "module",
   "bin": {

package/src/cli.js

@@ -116,8 +116,11 @@ function buildProgressHooks() {
     onStep(step) {
       console.error(`step ${step}: waiting for model...`);
     },
+    onReasoning({ step, text }) {
+      console.error(`step ${step} thinking: ${truncateForLog(text, 200)}`);
+    },
     onToolCall(toolCall) {
-      console.error(`step result: called tool "${toolCall.tool}"`);
+      console.error(`step result: called tool "${toolCall.tool}" -> ${summarizeToolCall(toolCall)}`);
     },
     onToolCallError({ step, reason, message, attempt }) {
       const label = reason === "truncated" ? "reply cut off (length limit)" : "invalid tool call JSON";
@@ -129,6 +132,33 @@ function buildProgressHooks() {
   };
 }
 
+function summarizeToolCall(toolCall) {
+  const args = toolCall.args ?? {};
+  switch (toolCall.tool) {
+    case "read_file":
+    case "make_directory":
+      return args.path ?? "";
+    case "write_file":
+    case "append_file":
+      return `${args.path ?? ""} (${String(args.content ?? "").length} chars)`;
+    case "replace_in_file":
+      return `${args.path ?? ""} (find: "${truncateForLog(args.findText ?? "", 40)}")`;
+    case "search_text":
+      return `"${args.query ?? ""}" in ${args.path ?? "."}`;
+    case "list_files":
+      return args.path ?? ".";
+    case "run_command":
+      return [args.command, ...(args.args ?? [])].join(" ");
+    default:
+      return JSON.stringify(args);
+  }
+}
+
+function truncateForLog(text, max) {
+  const flattened = text.replace(/\s+/g, " ").trim();
+  return flattened.length > max ? `${flattened.slice(0, max)}...` : flattened;
+}
+
 function printResult(result) {
   if (result.failed) {
     console.log(`⚠️ ${result.content}`);
@@ -453,6 +483,7 @@ async function printInitExample(cwd) {
     lmStudioBaseUrl: "http://127.0.0.1:1234",
     maxSteps: 12,
     allowCommands: false,
+    allowWrites: false,
     temperature: 0.2
   };
 
@@ -518,6 +549,7 @@ Options:
   --model MODEL_NAME
   --workspace PATH
   --allow-commands
+  --allow-writes
   --max-steps 12
   --temperature 0.2
   --ollama-base-url http://127.0.0.1:11434

package/src/config.js

@@ -9,6 +9,7 @@ const DEFAULTS = {
   lmStudioBaseUrl: "http://127.0.0.1:1234",
   maxSteps: 12,
   allowCommands: false,
+  allowWrites: false,
   temperature: 0.2
 };
 
@@ -65,7 +66,8 @@ function readEnvConfig() {
     model: process.env.LOCAL_CODE_MODEL,
     ollamaBaseUrl: process.env.OLLAMA_BASE_URL,
     lmStudioBaseUrl: process.env.LM_STUDIO_BASE_URL,
-    allowCommands: parseBoolean(process.env.LOCAL_CODE_ALLOW_COMMANDS)
+    allowCommands: parseBoolean(process.env.LOCAL_CODE_ALLOW_COMMANDS),
+    allowWrites: parseBoolean(process.env.LOCAL_CODE_ALLOW_WRITES)
   };
 }
 

package/src/tools.js

@@ -1,5 +1,5 @@
 import { checkSyntax } from "./syntaxCheck.js";
-import { confirmCommand } from "./ui.js";
+import { confirmCommand, confirmWrite } from "./ui.js";
 
 export function createToolset(workspace) {
   const tools = {
@@ -17,15 +17,22 @@ export function createToolset(workspace) {
       description: [
         "Create or overwrite a UTF-8 file inside the workspace.",
         "For files longer than ~40 lines, write a small skeleton first (imports and function signatures), then use append_file to add the rest in smaller pieces.",
-        "Do not use this to add to a file that already has content you want to keep - use append_file instead, so you never have to retype existing content."
+        "Do not use this to add to a file that already has content you want to keep - use append_file instead, so you never have to retype existing content.",
+        "Unless the session was started with --allow-writes, the user is asked to approve each file change in the terminal before it is written."
       ].join(" "),
       args: { path: "string", content: "string" },
-      run: async ({ path, content }) => withSyntaxCheck(workspace, path, () => workspace.writeFile(path, content))
+      run: async ({ path, content }) => {
+        const approved = await approveWrite(workspace, { action: "write", path, preview: content });
+        return withSyntaxCheck(workspace, path, () => workspace.writeFile(path, content, { approved }));
+      }
     },
     append_file: {
       description: "Append content to the end of a file, creating it if it does not exist. Prefer this over write_file when a file already has content you want to keep.",
       args: { path: "string", content: "string" },
-      run: async ({ path, content }) => withSyntaxCheck(workspace, path, () => workspace.appendFile(path, content))
+      run: async ({ path, content }) => {
+        const approved = await approveWrite(workspace, { action: "append", path, preview: content });
+        return withSyntaxCheck(workspace, path, () => workspace.appendFile(path, content, { approved }));
+      }
     },
     replace_in_file: {
       description: "Replace text inside an existing file.",
@@ -35,11 +42,18 @@ export function createToolset(workspace) {
         replaceText: "string",
         replaceAll: "boolean?"
       },
-      run: async (args) => withSyntaxCheck(
-        workspace,
-        args.path,
-        () => workspace.replaceInFile(args.path, args.findText, args.replaceText, args.replaceAll)
-      )
+      run: async (args) => {
+        const approved = await approveWrite(workspace, {
+          action: "replace",
+          path: args.path,
+          preview: `Find:\n${args.findText}\nReplace with:\n${args.replaceText}`
+        });
+        return withSyntaxCheck(
+          workspace,
+          args.path,
+          () => workspace.replaceInFile(args.path, args.findText, args.replaceText, args.replaceAll, { approved })
+        );
+      }
     },
     search_text: {
       description: "Search for a plain text query across workspace files.",
@@ -49,7 +63,10 @@ export function createToolset(workspace) {
     make_directory: {
       description: "Create a directory recursively in the workspace.",
       args: { path: "string" },
-      run: async ({ path }) => workspace.makeDirectory(path)
+      run: async ({ path }) => {
+        const approved = await approveWrite(workspace, { action: "mkdir", path });
+        return workspace.makeDirectory(path, { approved });
+      }
     },
     run_command: {
       description: "Run a local shell command inside the workspace (e.g. dotnet run, npm test, python script.py) to build, test, or execute code. Unless the session was started with --allow-commands, the user is asked to approve each command in the terminal before it runs.",
@@ -89,6 +106,14 @@ export function createToolset(workspace) {
   };
 }
 
+async function approveWrite(workspace, { action, path, preview }) {
+  if (workspace.allowWrites) {
+    return true;
+  }
+
+  return confirmWrite({ action, path, preview, cwd: workspace.rootPath });
+}
+
 async function withSyntaxCheck(workspace, targetPath, writeAction) {
   const message = await writeAction();
   const result = await checkSyntax(workspace.resolvePath(targetPath));

package/src/ui.js

@@ -174,6 +174,38 @@ export async function confirmCommand({ command, args = [], cwd }) {
   }
 }
 
+const WRITE_ACTION_LABELS = {
+  write: "create/overwrite file",
+  append: "append to file",
+  replace: "replace text in file",
+  mkdir: "create directory"
+};
+
+export async function confirmWrite({ action, path: targetPath, preview = "", cwd }) {
+  if (!isInteractive()) {
+    return false;
+  }
+
+  const label = WRITE_ACTION_LABELS[action] ?? action;
+  output.write(`\n${style("Model wants to " + label + ":", "yellow")} ${targetPath}\n`);
+  if (cwd) {
+    output.write(`${style("Working directory:", "dim")} ${cwd}\n`);
+  }
+
+  if (preview) {
+    const snippet = preview.length > 400 ? `${preview.slice(0, 400)}...` : preview;
+    output.write(`${style("Preview:", "dim")}\n${snippet}\n`);
+  }
+
+  const rl = readlinePromises.createInterface({ input, output });
+  try {
+    const answer = (await rl.question("Allow this change? [y/N]: ")).trim().toLowerCase();
+    return answer === "y" || answer === "yes";
+  } finally {
+    rl.close();
+  }
+}
+
 export function printNote(message) {
   console.log(style(message, "dim"));
 }

package/src/workspace.js

@@ -12,6 +12,13 @@ export class Workspace {
   constructor(rootPath, options = {}) {
     this.rootPath = path.resolve(rootPath);
     this.allowCommands = Boolean(options.allowCommands);
+    this.allowWrites = Boolean(options.allowWrites);
+  }
+
+  assertWriteApproved(approved) {
+    if (!this.allowWrites && !approved) {
+      throw new Error("File change was not approved. Re-run with --allow-writes to skip the prompt, or approve it when asked.");
+    }
   }
 
   resolvePath(targetPath = ".") {
@@ -47,21 +54,24 @@ export class Workspace {
     return fs.readFile(fullPath, "utf8");
   }
 
-  async writeFile(targetPath, content) {
+  async writeFile(targetPath, content, { approved = false } = {}) {
+    this.assertWriteApproved(approved);
     const fullPath = this.resolvePath(targetPath);
     await fs.mkdir(path.dirname(fullPath), { recursive: true });
     await fs.writeFile(fullPath, content, "utf8");
     return `Wrote ${path.relative(this.rootPath, fullPath)}`;
   }
 
-  async appendFile(targetPath, content) {
+  async appendFile(targetPath, content, { approved = false } = {}) {
+    this.assertWriteApproved(approved);
     const fullPath = this.resolvePath(targetPath);
     await fs.mkdir(path.dirname(fullPath), { recursive: true });
     await fs.appendFile(fullPath, content, "utf8");
     return `Appended to ${path.relative(this.rootPath, fullPath)}`;
   }
 
-  async replaceInFile(targetPath, findText, replaceText, replaceAll = false) {
+  async replaceInFile(targetPath, findText, replaceText, replaceAll = false, { approved = false } = {}) {
+    this.assertWriteApproved(approved);
     const fullPath = this.resolvePath(targetPath);
     const current = await fs.readFile(fullPath, "utf8");
     if (!current.includes(findText)) {
@@ -102,7 +112,8 @@ export class Workspace {
     return matches;
   }
 
-  async makeDirectory(targetPath) {
+  async makeDirectory(targetPath, { approved = false } = {}) {
+    this.assertWriteApproved(approved);
     const fullPath = this.resolvePath(targetPath);
     await fs.mkdir(fullPath, { recursive: true });
     return `Created ${path.relative(this.rootPath, fullPath)}`;