@letta-ai/letta-code 0.33.1 → 0.33.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@letta-ai/letta-code",
-  "version": "0.33.1",
+  "version": "0.33.3",
   "lettaStartupLogProtocol": 1,
   "description": "Letta Code is a CLI tool for interacting with stateful Letta agents from the terminal.",
   "type": "module",
@@ -132,12 +132,13 @@
   "dependencies": {
     "@earendil-works/pi-ai": "^0.87.1",
     "@janhapke/sharp-electron": "0.35.3-electron.1",
-    "@letta-ai/letta-agent-sdk": "^0.8.11",
+    "@letta-ai/letta-agent-sdk": "^0.8.17",
     "@letta-ai/letta-client": "^1.10.2",
     "@letta-ai/trajectory": "0.2.0",
     "@modelcontextprotocol/sdk": "1.30.0",
     "@pierre/diffs": "1.2.2",
     "@scarf/scarf": "^1.4.0",
+    "@shikijs/langs": "4.0.2",
     "cron-parser": "^5.6.1",
     "cross-spawn": "^7.0.6",
     "glob": "^13.0.0",

package/scripts/agent-watch/verify-pr-identity.test.ts

@@ -23,11 +23,13 @@ function pullRequest(
 function enforce(
   pullRequests: WatcherPullRequest[],
   closed: string[] = [],
+  pullRequestState: "draft" | "ready" = "draft",
 ): WatcherPullRequest | null {
   return enforceWatcherPrIdentity({
     repo: "letta-ai/letta-code",
     marker,
     expectedLogin: "amelia-letta",
+    pullRequestState,
     pullRequests,
     closePullRequest: (candidate) => closed.push(candidate.url),
   });
@@ -58,6 +60,20 @@ describe("enforceWatcherPrIdentity", () => {
     expect(closed).toHaveLength(1);
   });
 
+  test("returns the matching Amelia ready PR when ready is required", () => {
+    expect(
+      enforce([pullRequest({ isDraft: false })], [], "ready")?.number,
+    ).toBe(123);
+  });
+
+  test("closes and rejects a draft PR when ready is required", () => {
+    const closed: string[] = [];
+    expect(() => enforce([pullRequest()], closed, "ready")).toThrow(
+      "state=ready",
+    );
+    expect(closed).toHaveLength(1);
+  });
+
   test("rejects duplicate valid PRs", () => {
     expect(() =>
       enforce([
@@ -78,6 +94,19 @@ describe("parseArgs", () => {
     );
   });
 
+  test("requires the expected pull request state", () => {
+    expect(() =>
+      parseArgs([
+        "--repo",
+        "letta-ai/letta-code",
+        "--marker",
+        marker,
+        "--expected-login",
+        "amelia-letta",
+      ]),
+    ).toThrow("--pull-request-state is required");
+  });
+
   test("parses a complete invocation", () => {
     expect(
       parseArgs([
@@ -87,11 +116,34 @@ describe("parseArgs", () => {
         marker,
         "--expected-login",
         "amelia-letta",
+        "--pull-request-state",
+        "draft",
       ]),
     ).toEqual({
       repo: "letta-ai/letta-code",
       marker,
       expectedLogin: "amelia-letta",
+      pullRequestState: "draft",
+    });
+  });
+
+  test("parses a ready pull request requirement", () => {
+    expect(
+      parseArgs([
+        "--repo",
+        "letta-ai/letta-cloud",
+        "--marker",
+        marker,
+        "--expected-login",
+        "amelia-letta",
+        "--pull-request-state",
+        "ready",
+      ]),
+    ).toEqual({
+      repo: "letta-ai/letta-cloud",
+      marker,
+      expectedLogin: "amelia-letta",
+      pullRequestState: "ready",
     });
   });
 });

package/scripts/agent-watch/verify-pr-identity.ts

@@ -13,6 +13,7 @@ export interface WatcherPullRequest {
 interface Args {
   expectedLogin: string;
   marker: string;
+  pullRequestState: "draft" | "ready";
   repo: string;
 }
 
@@ -30,7 +31,7 @@ export function enforceWatcherPrIdentity(
   const invalid = matches.filter(
     (pullRequest) =>
       pullRequest.author?.login !== options.expectedLogin ||
-      !pullRequest.isDraft,
+      pullRequest.isDraft !== (options.pullRequestState === "draft"),
   );
 
   for (const pullRequest of invalid) options.closePullRequest(pullRequest);
@@ -43,7 +44,7 @@ export function enforceWatcherPrIdentity(
       )
       .join(", ");
     throw new Error(
-      `Closed watcher PRs that did not match author=${options.expectedLogin} and draft=true: ${details}`,
+      `Closed watcher PRs that did not match author=${options.expectedLogin} and state=${options.pullRequestState}: ${details}`,
     );
   }
   if (matches.length > 1) {
@@ -58,12 +59,19 @@ export function parseArgs(argv: string[]): Args {
   let repo = "";
   let marker = "";
   let expectedLogin = "";
+  let pullRequestState: "draft" | "ready" | undefined;
   for (let index = 0; index < argv.length; index += 1) {
     const argument = argv[index];
     if (argument === "--repo") repo = argv[++index] ?? "";
     else if (argument === "--marker") marker = argv[++index] ?? "";
     else if (argument === "--expected-login") {
       expectedLogin = argv[++index] ?? "";
+    } else if (argument === "--pull-request-state") {
+      const state = argv[++index];
+      if (state !== "draft" && state !== "ready") {
+        throw new Error("--pull-request-state must be draft or ready");
+      }
+      pullRequestState = state;
     } else {
       throw new Error(`Unknown argument: ${argument}`);
     }
@@ -71,7 +79,8 @@ export function parseArgs(argv: string[]): Args {
   if (!/^[^/]+\/[^/]+$/.test(repo)) throw new Error("--repo is required");
   if (!marker) throw new Error("--marker is required");
   if (!expectedLogin) throw new Error("--expected-login is required");
-  return { repo, marker, expectedLogin };
+  if (!pullRequestState) throw new Error("--pull-request-state is required");
+  return { repo, marker, expectedLogin, pullRequestState };
 }
 
 function runGh(args: string[]): string {
@@ -111,7 +120,7 @@ function main(): void {
   });
   console.log(
     verified
-      ? `Verified Amelia draft PR: ${verified.url}`
+      ? `Verified Amelia ${args.pullRequestState} PR: ${verified.url}`
       : `No open PR contains marker ${JSON.stringify(args.marker)}`,
   );
 }

package/scripts/isolated-unit-tests.json

@@ -175,16 +175,6 @@
       "timeoutMs": 30000,
       "reason": "Spawns real Git processes and mutates HOME, USER_CWD, cwd, and runtime singletons."
     },
-    {
-      "path": "src/tools/memory-apply-patch.test.ts",
-      "timeoutMs": 15000,
-      "reason": "Uses a top-level Bun module mock for the backend client."
-    },
-    {
-      "path": "src/tools/memory-tool.test.ts",
-      "timeoutMs": 15000,
-      "reason": "Uses a top-level Bun module mock for the backend client."
-    },
     {
       "path": "src/agent/available-models.test.ts",
       "timeoutMs": 15000,
@@ -200,6 +190,11 @@
       "timeoutMs": 15000,
       "reason": "Uses a top-level Bun module mock for the backend client."
     },
+    {
+      "path": "src/websocket/app-server-openai.test.ts",
+      "timeoutMs": 30000,
+      "reason": "Starts a real App Server and swaps the process-global backend while exercising persistent LocalBackend conversation state."
+    },
     {
       "path": "src/websocket/listen-client-concurrency.test.ts",
       "timeoutMs": 30000,

package/scripts/postinstall-patches.js

@@ -171,11 +171,17 @@ await patchInkRuntime([
   },
   {
     before: [
+      // Current patched state from the 8 MiB follow-up on this branch.
+      "    fullStaticOutput;\n    staticOutputRepaintPending = false;\n    isCi = () => isInCi && !this.options.stdout.isTTY;\n    // Cap the retained static transcript tail (in UTF-16 code units). Ink keeps\n    // fullStaticOutput so overflow frames and static repaints can rewrite the\n    // newest tail after a viewport clear. Overflow uses 2J+H (no 3J) so emulator\n    // scrollback keeps older history; 8 MiB is a UX-sized rewrite window (256 KB\n    // is one large Read/git diff) and bounds per-frame write volume. Uncapped,\n    // the string grows with the whole session and is re-serialized on overflow.\n    staticOutputRetainLimit = 8 * 1024 * 1024;\n    retainStaticOutput = (value) => {\n        if (value.length <= this.staticOutputRetainLimit) {\n            return value;\n        }\n        const cut = value.indexOf('\\n', value.length - this.staticOutputRetainLimit);\n        return '\\u001B[0m' + (cut === -1 ? value.slice(value.length - this.staticOutputRetainLimit) : value.slice(cut + 1));\n    };\n    exitPromise;",
+      // Current patched state from this PR's first commit (256 KB cap).
+      "    fullStaticOutput;\n    staticOutputRepaintPending = false;\n    isCi = () => isInCi && !this.options.stdout.isTTY;\n    // Cap the retained static transcript tail (in UTF-16 code units). Ink keeps\n    // fullStaticOutput so overflow frames and static repaints can rewrite the\n    // transcript after clearing the terminal; uncapped it grows linearly with\n    // the whole session transcript and is re-serialized on every overflow frame.\n    staticOutputRetainLimit = 256 * 1024;\n    retainStaticOutput = (value) => {\n        if (value.length <= this.staticOutputRetainLimit) {\n            return value;\n        }\n        const cut = value.indexOf('\\n', value.length - this.staticOutputRetainLimit);\n        return '\\u001B[0m' + (cut === -1 ? value.slice(value.length - this.staticOutputRetainLimit) : value.slice(cut + 1));\n    };\n    exitPromise;",
+      // Current patched state from a previous install (transition path).
+      "    fullStaticOutput;\n    staticOutputRepaintPending = false;\n    isCi = () => isInCi && !this.options.stdout.isTTY;\n    exitPromise;",
       "    fullStaticOutput;\n    staticOutputRepaintPending = false;\n    exitPromise;",
       "    fullStaticOutput;\n    exitPromise;",
     ],
     after:
-      "    fullStaticOutput;\n    staticOutputRepaintPending = false;\n    isCi = () => isInCi && !this.options.stdout.isTTY;\n    exitPromise;",
+      "    fullStaticOutput;\n    staticOutputRepaintPending = false;\n    isCi = () => isInCi && !this.options.stdout.isTTY;\n    // Cap the retained static transcript tail (in UTF-16 code units). Ink keeps\n    // fullStaticOutput so overflow frames and static repaints can rewrite the\n    // newest tail after a viewport clear. Overflow uses 2J+H (no 3J) so emulator\n    // scrollback keeps older history; 2 MiB is a UX-sized rewrite window (256 KB\n    // is one large Read/git diff) and bounds per-frame write volume. Uncapped,\n    // the string grows with the whole session and is re-serialized on overflow.\n    staticOutputRetainLimit = 2 * 1024 * 1024;\n    retainStaticOutput = (value) => {\n        if (value.length <= this.staticOutputRetainLimit) {\n            return value;\n        }\n        const cut = value.indexOf('\\n', value.length - this.staticOutputRetainLimit);\n        return '\\u001B[0m' + (cut === -1 ? value.slice(value.length - this.staticOutputRetainLimit) : value.slice(cut + 1));\n    };\n    exitPromise;",
   },
   {
     before: "        if (isInCi) {",
@@ -191,9 +197,53 @@ await patchInkRuntime([
     after: "        if (!this.isCi() && !this.options.debug) {",
   },
   {
-    before: "        if (this.options.debug) {",
-    after:
+    before: [
+      // Current patched state from a previous install (transition path).
       "        if (this.staticOutputRepaintPending && (this.options.debug || this.isCi())) {\n            if (this.options.stdout.isTTY) {\n                this.fullStaticOutput = hasStaticOutput ? staticOutput : '';\n                this.repaintStaticOutput(output);\n                return;\n            }\n            this.staticOutputRepaintPending = false;\n        }\n        if (this.options.debug) {",
+      "        if (this.options.debug) {",
+    ],
+    after:
+      "        if (this.staticOutputRepaintPending && (this.options.debug || this.isCi())) {\n            if (this.options.stdout.isTTY) {\n                this.fullStaticOutput = this.retainStaticOutput(hasStaticOutput ? staticOutput : '');\n                this.repaintStaticOutput(output);\n                return;\n            }\n            this.staticOutputRepaintPending = false;\n        }\n        if (this.options.debug) {",
+  },
+  {
+    // Bound steady-state growth: route every append through the tail cap.
+    before: "this.fullStaticOutput += staticOutput;",
+    after:
+      "this.fullStaticOutput = this.retainStaticOutput(this.fullStaticOutput + staticOutput);",
+    all: true,
+  },
+  {
+    // Overflow frames: do not replay fullStaticOutput and do not clear the
+    // screen. 2J erases on-screen transcript rows without saving them
+    // (xterm.js) or copies every live frame into history (tmux
+    // scroll-on-clear). Clip the live output to its last rows - 1 lines
+    // (approval/input sit at the bottom of AppView) so log-update's cursor-up
+    // region always fits on screen, then take the normal log-update path:
+    // new static rows scroll into history above the live region, the live
+    // region is rewritten in place, and unchanged frames are skipped.
+    // lastOutput keeps the clipped frame, so console redraws and later diffs
+    // match the screen. fullStaticOutput stays for the one-shot #4032 repaint.
+    before: [
+      // Current branch: 2J+H replace of the live tail clipped to rows lines.
+      "            if (hasStaticOutput) {\n                this.options.stdout.write(staticOutput + '\\n'.repeat(this.options.stdout.rows || 1));\n            }\n            if (hasStaticOutput || output !== this.lastOutput) {\n                const rows = this.options.stdout.rows || 1;\n                const liveLines = output.split('\\n');\n                this.options.stdout.write('\\u001B[2J\\u001B[H' + (liveLines.length > rows ? liveLines.slice(-rows).join('\\n') : output));\n            }\n            this.lastOutput = output;\n            return;",
+      // Earlier: clip kept the top of the live tree (hides input/approval).
+      "            if (hasStaticOutput) {\n                this.options.stdout.write(staticOutput + '\\n'.repeat(this.options.stdout.rows || 1));\n            }\n            if (hasStaticOutput || output !== this.lastOutput) {\n                const rows = this.options.stdout.rows || 1;\n                const liveLines = output.split('\\n');\n                this.options.stdout.write('\\u001B[2J\\u001B[H' + (liveLines.length > rows ? liveLines.slice(0, rows).join('\\n') : output));\n            }\n            this.lastOutput = output;\n            return;",
+      // Earlier: scroll increment then 2J+H the unclipped live region
+      // (still scrolls when outputHeight > rows).
+      "            if (hasStaticOutput) {\n                this.options.stdout.write(staticOutput + '\\n'.repeat(this.options.stdout.rows || 1));\n            }\n            if (hasStaticOutput || output !== this.lastOutput) {\n                this.options.stdout.write('\\u001B[2J\\u001B[H' + output);\n            }\n            this.lastOutput = output;\n            return;",
+      // Earlier: append increment and live with no viewport replace.
+      "            if (hasStaticOutput) {\n                this.options.stdout.write(staticOutput);\n            }\n            if (output !== this.lastOutput) {\n                this.options.stdout.write(output);\n            }\n            this.lastOutput = output;\n            return;",
+      // Earlier: increment immediately followed by 2J (erases the item).
+      "            if (hasStaticOutput || output !== this.lastOutput) {\n                this.options.stdout.write((hasStaticOutput ? staticOutput : '') + '\\u001B[2J\\u001B[H' + output);\n            }\n            this.lastOutput = output;\n            return;",
+      // Earlier: 2J+H but still replayed the retained tail.
+      "            if (hasStaticOutput || output !== this.lastOutput) {\n                this.options.stdout.write('\\u001B[2J\\u001B[H' + this.fullStaticOutput + output);\n            }\n            this.lastOutput = output;\n            return;",
+      // Earlier: skip-no-change guard still used clearTerminal (3J).
+      "            if (hasStaticOutput || output !== this.lastOutput) {\n                this.options.stdout.write(ansiEscapes.clearTerminal + this.fullStaticOutput + output);\n            }\n            this.lastOutput = output;\n            return;",
+      // Pristine Ink / earlier patched state: always-rewrite overflow.
+      "            this.options.stdout.write(ansiEscapes.clearTerminal + this.fullStaticOutput + output);\n            this.lastOutput = output;\n            return;",
+    ],
+    after:
+      "            const liveOutput = output.split('\\n').slice(-Math.max(1, (this.options.stdout.rows || 1) - 1)).join('\\n');\n            if (hasStaticOutput) {\n                this.log.clear();\n                this.options.stdout.write(staticOutput);\n                this.log(liveOutput);\n            }\n            else if (liveOutput !== this.lastOutput) {\n                this.throttledLog(liveOutput);\n            }\n            this.lastOutput = liveOutput;\n            return;",
   },
   {
     before: "        if (outputHeight >= this.options.stdout.rows) {",

package/scripts/source-file-size-baseline.json

@@ -1,21 +1,19 @@
 {
   "src/agent/client-skills.test.ts": 1134,
-  "src/agent/memory-git.ts": 2092,
-  "src/backend/local-backend.test.ts": 2473,
-  "src/backend/local/local-backend.ts": 1005,
-  "src/backend/local/local-store.ts": 3457,
+  "src/agent/memory-git.ts": 2013,
+  "src/backend/local-backend.test.ts": 2472,
+  "src/backend/local/local-store.ts": 3276,
   "src/backend/pi-stream-adapter.test.ts": 1255,
-  "src/cli/app/AppCoordinator.tsx": 5136,
+  "src/cli/app/AppCoordinator.tsx": 5109,
   "src/cli/app/AppView.tsx": 1735,
   "src/cli/app/use-approval-flow.ts": 1163,
   "src/cli/app/use-configuration-handlers.ts": 1417,
-  "src/cli/app/use-conversation-loop.ts": 2903,
+  "src/cli/app/use-conversation-loop.ts": 2887,
   "src/cli/app/use-submit-handler.ts": 3804,
   "src/cli/components/AgentSelector.tsx": 1104,
   "src/cli/components/InputRich.tsx": 2116,
   "src/cli/components/ModelSelector.tsx": 1259,
   "src/cli/components/ProviderSelector.tsx": 1564,
-  "src/cli/components/ToolCallMessageRich.tsx": 1013,
   "src/cli/helpers/accumulator.ts": 1587,
   "src/cli/helpers/reflection-transcript.ts": 1926,
   "src/cli/mods/local-mod-loader.test.ts": 1043,
@@ -23,7 +21,7 @@
   "src/cli/subcommands/skills.ts": 1264,
   "src/headless.ts": 4788,
   "src/hooks/integration.test.ts": 1147,
-  "src/index.ts": 2597,
+  "src/index.ts": 2578,
   "src/mods/learning-harness.ts": 2434,
   "src/mods/mod-engine.test.ts": 2153,
   "src/mods/mod-engine.ts": 1838,
@@ -35,8 +33,8 @@
   "src/providers/chatgpt-usage-service.ts": 1112,
   "src/settings-manager.test.ts": 1635,
   "src/settings-manager.ts": 2101,
-  "src/tools/manager.ts": 2713,
-  "src/tools/tool-execution-context.test.ts": 1114,
+  "src/tools/manager.ts": 2649,
+  "src/tools/tool-execution-context.test.ts": 1092,
   "src/types/protocol_v2.ts": 2574,
   "src/websocket/listen-client-concurrency.test.ts": 2666,
   "src/websocket/listen-client-protocol.test.ts": 5759,

package/scripts/stage-pypi-deps.mjs

@@ -13,7 +13,16 @@ import { dirname, join, resolve } from "node:path";
 const root = process.cwd();
 const app = resolve(process.argv[2]);
 // sharp-electron is only reachable under Electron, never our bundled Node.
-const roots = ["ws", "@vscode/ripgrep", "node-pty", "grammy", "@pierre/diffs"];
+// @shikijs/langs is external in the bundle so lazily-loaded grammars
+// (LET-13149) resolve from here at runtime.
+const roots = [
+  "ws",
+  "@vscode/ripgrep",
+  "node-pty",
+  "grammy",
+  "@pierre/diffs",
+  "@shikijs/langs",
+];
 
 function locate(name, from) {
   for (let dir = from; ; dir = dirname(dir)) {

package/skills/creating-mods/references/plan-mode.md

@@ -180,24 +180,19 @@ if (letta.capabilities.events.turns) {
 
 ## Permission overlay
 
-Use a permission overlay, not `tool_start`, for policy. Normalize tool names by family; UI display names and provider-specific tool names drift (`Read`, `read`, `read_file`, `ReadFile`, etc.). Keep pure read-only tools separate from planning coordination tools like `AskUserQuestion` and todo/plan updates so the policy stays honest.
+Use a permission overlay, not `tool_start`, for policy. Normalize tool names by family; UI display names and provider-specific tool names drift (`Read`, `read`, etc.). Keep pure read-only tools separate from planning coordination tools like `AskUserQuestion` and todo/plan updates so the policy stays honest.
 
 ```ts
 const readOnlyToolNames = new Set([
   "glob",
   "grep",
-  "grepfiles",
   "list",
-  "listdir",
-  "ls",
   "notebookread",
   "read",
-  "readfile",
   "readlsp",
   "search",
   "searchfiles",
   "skill",
-  "taskoutput",
   "viewimage",
 ]);
 
@@ -205,7 +200,10 @@ const planningToolNames = new Set([
   "askuserquestion",
   "enterplanmode",
   "exitplanmode",
-  "todowrite",
+  "taskcreate",
+  "taskget",
+  "tasklist",
+  "taskupdate",
   "updateplan",
 ]);
 
@@ -259,7 +257,7 @@ if (letta.capabilities.permissions) {
       return {
         decision: "deny",
         reason:
-          `Plan mode is active. Use direct read-only tools (Read, Grep, Glob, List, Search, Skill, safe read-only Bash), planning tools (AskUserQuestion, TodoWrite/UpdatePlan), or recall-style subagents only. ` +
+          `Plan mode is active. Use direct read-only tools (Read, Grep, Glob, ViewImage, Skill, safe read-only Bash), planning tools (AskUserQuestion, TaskCreate/TaskUpdate, UpdatePlan), or recall-style subagents only. ` +
           `Do not use coding, general-purpose, or fork subagents in plan mode. ` +
           `Write your plan to: ${session.planFilePath}. ` +
           `When ready, read the plan file and include the full current plan text in AskUserQuestion for approval, then call exit_plan_mode after approval.`,