little-coder 1.1.0 → 1.2.0

package/.pi/extensions/llama-cpp-provider/config.test.ts

@@ -1,7 +1,8 @@
 import { describe, it, expect, beforeEach, afterEach } from "vitest";
 import { mkdtempSync, rmSync, writeFileSync, mkdirSync } from "node:fs";
 import { tmpdir } from "node:os";
-import { join } from "node:path";
+import { dirname, join, resolve } from "node:path";
+import { fileURLToPath } from "node:url";
 import { applyEnvOverrides, loadProviders, mergeProviders, resolveOverridePath, type ProviderEntry } from "./config.ts";
 
 const sampleProvider = (baseUrl: string, modelId: string): ProviderEntry => ({
@@ -76,6 +77,11 @@ describe("applyEnvOverrides", () => {
     const out = applyEnvOverrides(providers, { OLLAMA_BASE_URL: "http://env/v1" });
     expect(out.ollama.baseUrl).toBe("http://env/v1");
   });
+  it("LMSTUDIO_BASE_URL overrides lmstudio baseUrl", () => {
+    const providers = { lmstudio: sampleProvider("http://127.0.0.1:1234/v1", "local-model") };
+    const out = applyEnvOverrides(providers, { LMSTUDIO_BASE_URL: "http://127.0.0.1:5678/v1" });
+    expect(out.lmstudio.baseUrl).toBe("http://127.0.0.1:5678/v1");
+  });
   it("does not alter providers without a known env knob", () => {
     const providers = { custom: sampleProvider("http://file/v1", "m") };
     const out = applyEnvOverrides(providers, { LLAMACPP_BASE_URL: "http://env/v1" });
@@ -159,3 +165,23 @@ describe("loadProviders (filesystem)", () => {
     expect(result.providers.llamacpp.models[0].id).toBe("via-xdg");
   });
 });
+
+describe("shipped models.json", () => {
+  const here = dirname(fileURLToPath(import.meta.url));
+  const pkgRoot = resolve(here, "..", "..", "..");
+
+  it("registers lmstudio/local-model on http://127.0.0.1:1234/v1", () => {
+    const result = loadProviders(pkgRoot, {});
+    const lmstudio = result.providers.lmstudio;
+    expect(lmstudio, "lmstudio provider should be present in shipped models.json").toBeDefined();
+    expect(lmstudio.baseUrl).toBe("http://127.0.0.1:1234/v1");
+    expect(lmstudio.api).toBe("openai-completions");
+    expect(lmstudio.apiKey).toBe("LMSTUDIO_API_KEY");
+    expect(lmstudio.models.find((m) => m.id === "local-model")).toBeDefined();
+  });
+
+  it("still registers llamacpp and ollama alongside lmstudio", () => {
+    const result = loadProviders(pkgRoot, {});
+    expect(Object.keys(result.providers).sort()).toEqual(["llamacpp", "lmstudio", "ollama"]);
+  });
+});

package/.pi/extensions/llama-cpp-provider/config.ts

@@ -43,11 +43,14 @@ export interface LoadResult {
   sources: { path: string; status: "ok" | "missing" | "invalid"; error?: string }[];
 }
 
-/** Provider env knob: if set, overrides the provider's baseUrl. Legacy compat
- *  for the two providers we shipped before the data-driven refactor. */
+/** Provider env knob: if set, overrides the provider's baseUrl. Originally a
+ *  back-compat shim for the two providers we shipped before the data-driven
+ *  refactor; kept as the per-provider env-override pattern for any provider
+ *  whose baseUrl changes between deployments. */
 const LEGACY_BASE_URL_ENV: Record<string, string> = {
   llamacpp: "LLAMACPP_BASE_URL",
   ollama: "OLLAMA_BASE_URL",
+  lmstudio: "LMSTUDIO_BASE_URL",
 };
 
 /** Resolution order for the user-override file. First existing path wins. */

package/.pi/extensions/skill-inject/index.ts

@@ -56,7 +56,6 @@ const INTENT_MAP: Record<string, string[]> = {
   browse: ["BrowserNavigate", "BrowserExtract"],
   page: ["BrowserExtract"],
   click: ["BrowserClick"],
-  agent: ["Agent"], delegate: ["Agent"], spawn: ["Agent"],
 };
 
 function skillsDir(): string {

package/.pi/extensions/skill-inject/selector.test.ts

@@ -21,7 +21,6 @@ const INTENT_MAP: Record<string, string[]> = {
   grep: ["Grep"], glob: ["Glob"],
   fetch: ["WebFetch"], download: ["WebFetch"], url: ["WebFetch"],
   web: ["WebSearch"],
-  agent: ["Agent"], delegate: ["Agent"], spawn: ["Agent"],
 };
 
 function predictTools(userText: string): string[] {
@@ -61,10 +60,10 @@ describe("skills directory loads from repo", () => {
   const here = dirname(fileURLToPath(import.meta.url));
   const toolsDir = join(here, "..", "..", "..", "skills", "tools");
 
-  it("exists and has 14 markdown files", () => {
+  it("exists and has 13 markdown files", () => {
     expect(existsSync(toolsDir)).toBe(true);
     const files = readdirSync(toolsDir).filter((f) => f.endsWith(".md"));
-    expect(files.length).toBe(14);
+    expect(files.length).toBe(13);
   });
 
   it("every tool skill has target_tool in frontmatter", () => {

package/.pi/settings.json

@@ -70,6 +70,14 @@
         "skill_token_budget": 300,
         "knowledge_token_budget": 200,
         "temperature": 0.3
+      },
+      "lmstudio/local-model": {
+        "context_limit": 32768,
+        "max_tokens": 4096,
+        "thinking_budget": 2048,
+        "skill_token_budget": 300,
+        "knowledge_token_budget": 200,
+        "temperature": 0.3
       }
     }
   }

package/CHANGELOG.md

@@ -2,6 +2,26 @@
 
 All notable changes to little-coder are documented here. The format follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/), and little-coder's public interface (CLI, providers, tools, skills) follows semver starting at `v0.0.1` post-rename.
 
+## [v1.2.0] — 2026-05-10
+
+Issue-cleanup release that also ships built-in LM Studio support. Closes [#17](https://github.com/itayinbarr/little-coder/issues/17) (Windows), [#19](https://github.com/itayinbarr/little-coder/issues/19) (phantom Agent tool), [#21](https://github.com/itayinbarr/little-coder/issues/21) (skill param mismatch).
+
+### Added
+- **Built-in `lmstudio/local-model` provider.** [LM Studio](https://lmstudio.ai/) exposes an OpenAI-compatible server on `http://127.0.0.1:1234/v1` by default, and previously the only way to use it was to overload `LLAMACPP_BASE_URL`. Now you can run `little-coder --model lmstudio/local-model` and it routes to whatever model LM Studio currently has loaded — no extra config for the single-model case. New env knobs `LMSTUDIO_BASE_URL` (overrides baseUrl, parity with `LLAMACPP_BASE_URL`/`OLLAMA_BASE_URL`) and `LMSTUDIO_API_KEY` (any value; LM Studio ignores it locally but pi requires the env var to exist). README has a new **Option C — LM Studio** under *Local model setup*. `.pi/settings.json` ships a `lmstudio/local-model` profile so the same context/thinking-budget tuning as the llamacpp profiles applies.
+
+### Fixed
+- **Windows launch ([#17](https://github.com/itayinbarr/little-coder/issues/17), thanks @Grogger for [PR #18](https://github.com/itayinbarr/little-coder/pull/18)).** On Windows, `node_modules/.bin/pi` is a `.cmd` shim that Node 20's `spawn()` can't execute directly without `shell: true`, and `shell: true` reintroduces the CVE-2024-27980 / DEP0190 shell-injection class. The launcher now resolves `pi.cmd` on Windows and invokes `cmd.exe /c pi.cmd ...` with args as an array — works on Windows 11, no Linux/macOS regression.
+- **Edit skill documentation ([#21](https://github.com/itayinbarr/little-coder/issues/21)).** `skills/tools/edit.md` advertised `old_string` / `new_string`, but pi's Edit tool only accepts `oldText` / `newText` (single-edit form) or `edits: [{oldText, newText}]` (array form). Rewritten to show the canonical array form *and* the single-edit back-compat form. While in there, also corrected `skills/tools/read.md` and `skills/tools/write.md` (`file_path` → `path` — pi aliases both, but the canonical name is now in the docs) and `skills/tools/grep.md` (`include` → `glob`, `max_results` → `limit`; pi does not alias these, so the old skill could genuinely produce tool-call errors on the grep path the same way Edit did).
+
+### Changed
+- **Removed phantom `Agent` skill ([#19](https://github.com/itayinbarr/little-coder/issues/19)).** `skills/tools/agent.md` documented an `Agent` tool that little-coder never actually registered — pi ships `examples/extensions/subagent/` as a reference impl, but it was not wired up by default. Deleted the skill card and the `agent` / `delegate` / `spawn` keys from `.pi/extensions/skill-inject/index.ts`'s `INTENT_MAP` so the model is no longer told it has a delegation tool. The `skills/protocols/task_decomposition.md` cheatsheet is untouched — decomposition guidance does not depend on a delegation tool.
+
+### Notes for upgraders
+- No CLI flag, settings, or skill-pack breaks. `--model lmstudio/local-model` works out of the box if LM Studio is serving on its default port 1234 with a model loaded.
+- If you'd been overloading `LLAMACPP_BASE_URL=http://127.0.0.1:1234/v1` to point at LM Studio, that keeps working — but the cleaner path is now `--model lmstudio/local-model` with no env tweaking.
+
+---
+
 ## [v1.1.0] — 2026-05-03
 
 Issue-cleanup release. Three small features and one bug fix, driven by GitHub issues #12 / #13 / #15 / #16.

package/README.md

@@ -51,19 +51,21 @@ Cloud models work the same way:
 little-coder --model anthropic/claude-haiku-4-5
 little-coder --model openai/gpt-4o-mini "What does this codebase do?"
 little-coder --model ollama/qwen3.5             # local Ollama
+little-coder --model lmstudio/local-model       # local LM Studio (whatever model you have loaded)
 little-coder --list-models                      # see everything pi knows about
 ```
 
 The agent uses the directory you launched it from as its working directory — `Read` / `Write` / `Edit` / `Bash` operate on your project, not on little-coder's install path.
 
-For local providers (llama.cpp, Ollama) pi expects *some* value in the API-key env even though local servers ignore it:
+For local providers (llama.cpp, Ollama, LM Studio) pi expects *some* value in the API-key env even though local servers ignore it:
 
 ```bash
 export LLAMACPP_API_KEY=noop
 export OLLAMA_API_KEY=noop
+export LMSTUDIO_API_KEY=noop
 ```
 
-`LLAMACPP_BASE_URL` and `OLLAMA_BASE_URL` override the defaults (`http://127.0.0.1:8888/v1`, `http://127.0.0.1:11434/v1`).
+`LLAMACPP_BASE_URL`, `OLLAMA_BASE_URL`, and `LMSTUDIO_BASE_URL` override the defaults (`http://127.0.0.1:8888/v1`, `http://127.0.0.1:11434/v1`, `http://127.0.0.1:1234/v1`).
 
 For cloud providers, set the standard env (`ANTHROPIC_API_KEY`, `OPENAI_API_KEY`, etc.) and pi will discover it.
 
@@ -97,6 +99,17 @@ ollama pull qwen3.5        # 9.7B — the paper's model
 # or: ollama pull qwen3.6-35b-a3b
 ```
 
+**Option C — LM Studio** (GUI; OpenAI-compatible server on port 1234):
+
+1. Install [LM Studio](https://lmstudio.ai/) and download a model (e.g. Qwen3.6 35B A3B GGUF).
+2. Open the **Developer** / **Local Server** tab, load the model, and click **Start Server** (default `http://127.0.0.1:1234`).
+3. Run little-coder:
+   ```bash
+   export LMSTUDIO_API_KEY=noop
+   little-coder --model lmstudio/local-model
+   ```
+   The shipped `lmstudio/local-model` id routes to whatever model LM Studio currently has loaded — no extra config needed for the single-model case. If you serve on a non-default port, set `LMSTUDIO_BASE_URL=http://127.0.0.1:<port>/v1`. To target a specific model when you have several loaded, add an entry to `~/.config/little-coder/models.json` (see **Configuring models** below).
+
 All small-model-specific extensions auto-disable for large/cloud models so they don't interfere.
 
 ---
@@ -140,7 +153,7 @@ Example — switch the llama.cpp port and bump `qwen3.6-35b-a3b` to a 150K conte
 
 Then verify with `little-coder --list-models` — you should see your overridden entry.
 
-`LLAMACPP_BASE_URL` and `OLLAMA_BASE_URL` env vars still beat both files for those two providers (legacy compat).
+`LLAMACPP_BASE_URL`, `OLLAMA_BASE_URL`, and `LMSTUDIO_BASE_URL` env vars still beat both files for those three providers.
 
 `.pi/settings.json` is a separate concern: it controls per-model **profiles** (context_limit, thinking_budget, temperature, benchmark_overrides) referenced by the `<provider>/<id>` key. Profiles don't register or describe models — they only tune how little-coder runs against models that are already registered.
 
@@ -241,7 +254,7 @@ little-coder/
 ├── .pi/
 │   ├── settings.json               # per-model profiles + benchmark_overrides (terminal_bench, gaia)
 │   └── extensions/                 # 20 TypeScript extensions, auto-discovered by pi
-│       ├── llama-cpp-provider/     # data-driven provider registration from models.json (+ user override file)
+│       ├── llama-cpp-provider/     # data-driven provider registration from models.json — ships llamacpp, ollama, lmstudio (+ user override file)
 │       ├── write-guard/            # Write refuses on existing files — the whitepaper invariant
 │       ├── extra-tools/            # glob, webfetch, websearch (pi ships grep/find)
 │       ├── skill-inject/           # per-turn tool-skill selection (error > recency > intent)

package/bin/little-coder.mjs

@@ -29,7 +29,9 @@ const here = dirname(fileURLToPath(import.meta.url));
 const pkgRoot = resolve(here, "..");
 
 // ---- 3. Resolve the bundled pi binary ----
-const piBin = join(pkgRoot, "node_modules", ".bin", "pi");
+const isWindows = process.platform === "win32";
+const piBinBase = join(pkgRoot, "node_modules", ".bin", "pi");
+const piBin = isWindows && existsSync(`${piBinBase}.cmd`) ? `${piBinBase}.cmd` : piBinBase;
 if (!existsSync(piBin)) {
   console.error(
     `little-coder: cannot find pi at ${piBin}.\n` +
@@ -86,7 +88,11 @@ const piArgs = [
 ];
 
 // ---- 7. Spawn pi in the user's cwd ----
-const child = spawn(piBin, piArgs, {
+const [spawnCmd, spawnArgs] = isWindows
+  ? ["cmd.exe", ["/c", piBin, ...piArgs]]
+  : [piBin, piArgs];
+
+const child = spawn(spawnCmd, spawnArgs, {
   stdio: "inherit",
   cwd: process.cwd(),
   env: process.env,

package/models.json

@@ -49,6 +49,22 @@
           "cost": { "input": 0, "output": 0, "cacheRead": 0, "cacheWrite": 0 }
         }
       ]
+    },
+    "lmstudio": {
+      "api": "openai-completions",
+      "baseUrl": "http://127.0.0.1:1234/v1",
+      "apiKey": "LMSTUDIO_API_KEY",
+      "models": [
+        {
+          "id": "local-model",
+          "name": "LM Studio (currently-loaded local model)",
+          "reasoning": true,
+          "input": ["text"],
+          "contextWindow": 32768,
+          "maxTokens": 4096,
+          "cost": { "input": 0, "output": 0, "cacheRead": 0, "cacheWrite": 0 }
+        }
+      ]
     }
   }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "little-coder",
-  "version": "1.1.0",
+  "version": "1.2.0",
   "description": "A pi-based coding agent optimized for small local language models. Reproduces the whitepaper's scaffold-model-fit adaptations as pi extensions.",
   "homepage": "https://github.com/itayinbarr/little-coder",
   "repository": {

package/skills/tools/edit.md

@@ -9,22 +9,28 @@ user-invocable: false
 ## Edit Tool
 Replace exact text in a file. This is the **default tool for changing any existing file** — prefer it over Write for anything except creating a new file from scratch.
 
-REQUIRED: file_path (absolute), old_string (exact text to find), new_string (replacement)
-OPTIONAL: replace_all (boolean, replace all occurrences)
+REQUIRED: path (absolute), edits (array of {oldText, newText})
+OPTIONAL: none
 
 RULES:
-- old_string must match EXACTLY (whitespace, indentation, line endings all matter)
-- old_string must appear exactly ONCE unless replace_all=true
-- Include enough surrounding context (2-3 lines) to make old_string unique
-- To delete text: set new_string to ""
+- Each `oldText` must match EXACTLY (whitespace, indentation, line endings all matter)
+- Each `oldText` must be unique in the file — include 2-3 lines of surrounding context if needed
+- `edits` is matched against the **original** file, not after earlier edits apply — do not overlap or nest
+- To delete text: set `newText` to ""
 - Read the file first if you do not already have its current content
+- Batch multiple disjoint changes in one call by passing multiple `edits[]` entries
 
-EXAMPLE:
+EXAMPLE (single change):
 ```tool
-{"name": "Edit", "input": {"file_path": "/absolute/path/file.py", "old_string": "def hello():\n    return 1", "new_string": "def hello():\n    return 2"}}
+{"name": "Edit", "input": {"path": "/absolute/path/file.py", "edits": [{"oldText": "def hello():\n    return 1", "newText": "def hello():\n    return 2"}]}}
+```
+
+EXAMPLE (two changes in one call):
+```tool
+{"name": "Edit", "input": {"path": "/absolute/path/file.py", "edits": [{"oldText": "MAX = 10", "newText": "MAX = 20"}, {"oldText": "TIMEOUT = 5", "newText": "TIMEOUT = 30"}]}}
 ```
 
 RECOVERY WHEN Edit FAILS:
 - "String not found" → Read the file to get the exact current content (whitespace often differs), then retry Edit with the exact string
-- "Found multiple times" → include more surrounding context so old_string is unique, then retry Edit
-- Do NOT fall back to Write just because Edit failed once — re-read, fix old_string, retry. Write is almost always the wrong recovery here for an existing file.
+- "Found multiple times" → include more surrounding context so `oldText` is unique, then retry Edit
+- Do NOT fall back to Write just because Edit failed once — re-read, fix `oldText`, retry. Write is almost always the wrong recovery here for an existing file.

package/skills/tools/grep.md

@@ -10,17 +10,18 @@ user-invocable: false
 Search file contents with regex. Uses ripgrep.
 
 REQUIRED: pattern (regex pattern)
-OPTIONAL: path (directory/file), include (file glob filter like "*.py"), max_results (limit)
+OPTIONAL: path (directory/file), glob (file glob filter like "*.py"), ignoreCase (bool), literal (bool — treat pattern as literal text), context (lines of context before/after), limit (max matches, default 100)
 
 RULES:
-- Supports full regex syntax
-- Use include to filter by file type (e.g. "*.py", "*.js")
+- Supports full regex syntax (unless `literal: true`)
+- Use `glob` to filter by file type (e.g. "*.py", "*.js")
+- Use `limit` to cap results; default 100
 - Returns matching lines with file path and line number
 - Good for finding function definitions, imports, references
 
 EXAMPLE:
 ```tool
-{"name": "Grep", "input": {"pattern": "def main", "include": "*.py"}}
+{"name": "Grep", "input": {"pattern": "def main", "glob": "*.py"}}
 ```
 
 EXAMPLE with path:

package/skills/tools/read.md

@@ -9,7 +9,7 @@ user-invocable: false
 ## Read Tool
 Read a file's contents with line numbers.
 
-REQUIRED: file_path (absolute path)
+REQUIRED: path (absolute path)
 OPTIONAL: limit (max lines), offset (start line, 0-indexed)
 
 RULES:
@@ -19,10 +19,10 @@ RULES:
 
 EXAMPLE:
 ```tool
-{"name": "Read", "input": {"file_path": "/absolute/path/to/file.py"}}
+{"name": "Read", "input": {"path": "/absolute/path/to/file.py"}}
 ```
 
 EXAMPLE with range:
 ```tool
-{"name": "Read", "input": {"file_path": "/absolute/path/to/file.py", "limit": 50, "offset": 100}}
+{"name": "Read", "input": {"path": "/absolute/path/to/file.py", "limit": 50, "offset": 100}}
 ```

package/skills/tools/write.md

@@ -9,7 +9,7 @@ user-invocable: false
 ## Write Tool
 Create a **new** file with the given content. Creates parent directories automatically.
 
-REQUIRED: file_path (absolute), content (full file content)
+REQUIRED: path (absolute), content (full file content)
 
 **Write is for creating new files only.** If the file already exists, Write will be **refused** by the tool and return an error telling you to use Edit instead. Do not retry Write on the same path — it will be refused again.
 
@@ -17,13 +17,13 @@ WHEN TO USE Write:
 - The file does not exist yet and you are creating it from scratch
 
 WHEN TO USE Edit INSTEAD:
-- ANY change to an existing file — bug fixes, refactors, format tweaks, adding a function, renaming a variable, everything. Edit takes old_string + new_string and patches in place.
+- ANY change to an existing file — bug fixes, refactors, format tweaks, adding a function, renaming a variable, everything. Edit takes `path` + `edits: [{oldText, newText}]` and patches in place.
 - Iterating after a failed test — never retype the whole file
 
-If you need to completely replace an existing file's content, Edit can still do that: pass the entire current content as old_string and the full new content as new_string. Read the file first if you don't already have its current content.
+If you need to completely replace an existing file's content, Edit can still do that: pass the entire current content as `oldText` and the full new content as `newText`. Read the file first if you don't already have its current content.
 
 EXAMPLE:
 ```tool
-{"name": "Write", "input": {"file_path": "/tmp/example/new_module.py", "content": "def hello():\n    return 'hi'\n"}}
+{"name": "Write", "input": {"path": "/tmp/example/new_module.py", "content": "def hello():\n    return 'hi'\n"}}
 ```
 NOTE: Always use the EXACT file path given in the task, never a placeholder.

package/skills/tools/agent.md

@@ -1,24 +0,0 @@
----
-name: agent-guidance
-type: tool-guidance
-target_tool: Agent
-priority: 6
-token_cost: 120
-user-invocable: false
----
-## Agent Tool
-Spawn a sub-agent to handle a task autonomously.
-
-REQUIRED: prompt (task description for the sub-agent)
-OPTIONAL: subagent_type (coder/reviewer/researcher/tester/general-purpose), name (for messaging), isolation ("worktree" for git isolation)
-
-RULES:
-- Use for independent tasks that don't need your direct attention
-- Sub-agents get their own context and can use all tools
-- Use subagent_type to get specialized behavior
-- Use isolation="worktree" when the agent needs to modify files independently
-
-EXAMPLE:
-```tool
-{"name": "Agent", "input": {"prompt": "Find all Python files that import requests and list them", "subagent_type": "researcher"}}
-```