micro-models-agent 2.11.0 → 2.12.0

package/dist/modules/execution/plan-checks.js

@@ -0,0 +1,35 @@
+// Контракты верификации шагов: file-exists/command/manual + парсер.
+/**
+ * Контракты верификации шагов ("checks").
+ *
+ * Check — однострочный контракт на шаг, объявляется при create/re-plan.
+ * Три префикса:
+ * - "file-exists:<path>" — жёстко проверяет StepVerifier на done;
+ * - "command:<cmd>" — точная команда-доказательство шага (self-certified);
+ * - "manual:<text>" — проверка глазами для UI/визуалки (self-certified).
+ *
+ * Чистые хелперы, общие для валидатора (module.parseChecks) и done-gate
+ * (plan-actions.updateStep).
+ */
+const PREFIXES = [
+    { prefix: "file-exists:", kind: "file" },
+    { prefix: "command:", kind: "command" },
+    { prefix: "manual:", kind: "manual" },
+];
+/** Парсит строку check. Null, если префикс неизвестен/нет или цель пуста. */
+export function parseCheck(check) {
+    const trimmed = check.trim();
+    for (const { prefix, kind } of PREFIXES) {
+        if (trimmed.toLowerCase().startsWith(prefix)) {
+            const target = trimmed.slice(prefix.length).trim();
+            if (!target)
+                return null;
+            return { kind, target };
+        }
+    }
+    return null;
+}
+/** True, если строка check — well-formed контракт. */
+export function isValidCheck(check) {
+    return typeof check === "string" && parseCheck(check) !== null;
+}

package/dist/modules/execution/plan-coverage.js

@@ -1,40 +1,43 @@
-const FILE_PATH_RE = /\b[\w./\\-]+\.[a-z]+\b/gi;
-const IGNORED_EXT = new Set([
-    "tsx",
-    "jsx",
-    "json5",
-    "mdx",
-    "yml",
-    "yaml",
-    "toml",
-    "lock",
-    "svg",
-    "png",
-    "jpg",
-    "jpeg",
-    "gif",
-    "webp",
-    "ico",
-    "css",
-    "scss",
-    "less",
-    "html",
+// Покрытие плана: упомянутые в задаче файлы обязаны встречаться в шагах.
+import { stripUrls } from "../hallucination/js-identifiers";
+/**
+ * Coverage-allow-list расширений. Считаются только реальные типы deliverable-
+ * файлов: проза `e.g` / `i.e` (расширения `g`/`e`) за файлы не сойдёт, а
+ * web/doc-результаты (`.css`, `.html`, `.scss`, `.md`) проверяются —
+ * старый deny-list их молча пропускал.
+ */
+const COVERAGE_EXT = new Set([
+    // Код
+    "ts", "tsx", "js", "jsx", "mjs", "cjs", "py", "rb", "go", "rs", "java",
+    "kt", "kts", "cs", "php", "swift", "c", "cc", "cpp", "cxx", "h", "hpp",
+    "lua", "pl", "sh", "bash", "zsh", "bat", "cmd", "ps1", "sql", "vue",
+    "svelte", "dart", "scala", "clj",
+    // web / доки / конфиги
+    "css", "scss", "less", "html", "htm", "md", "mdx", "json", "json5",
+    "toml", "yaml", "yml", "ini", "env", "txt",
 ]);
+const FILE_PATH_RE = /\b[\w./\\-]+\.[a-z]{1,6}\b/gi;
 /**
- * Extract file paths (with extensions) mentioned in a task statement.
- * Used to check whether a generated plan covers every file the task requires.
+ * Извлекает пути файлов (с расширениями), упомянутые в формулировке задачи.
+ * Проверяет, покрывает ли сгенерированный план каждый требуемый файл.
  */
 export function extractFilePaths(text) {
     if (!text)
         return [];
     const seen = new Set();
     const result = [];
-    for (const m of text.matchAll(FILE_PATH_RE)) {
-        let p = m[0];
-        // Trim trailing punctuation that regex greedily keeps (e.g. "test.ts.")
-        p = p.replace(/[.,;:)\]>]+$/g, "");
-        const ext = p.split(".").pop()?.toLowerCase() ?? "";
-        if (!p || IGNORED_EXT.has(ext))
+    for (const m of stripUrls(text).matchAll(FILE_PATH_RE)) {
+        const p = m[0].replace(/[.,;:)\]>]+$/g, "");
+        if (!p)
+            continue;
+        // Требуем настоящее имя: непустой stem перед расширением в последнем
+        // сегменте (сегменты `tsx/.ts` / `.env` — не "файлы").
+        const slash = Math.max(p.lastIndexOf("/"), p.lastIndexOf("\\"));
+        const last = slash >= 0 ? p.slice(slash + 1) : p;
+        const dot = last.lastIndexOf(".");
+        if (dot <= 0)
+            continue;
+        if (!COVERAGE_EXT.has(last.slice(dot + 1).toLowerCase()))
             continue;
         const key = p.toLowerCase();
         if (seen.has(key))
@@ -49,9 +52,9 @@ function basename(p) {
     return parts[parts.length - 1] ?? p;
 }
 /**
- * Compare the file requirements in the task statement against the plan's step
- * descriptions. A requirement is "covered" when its path (or its basename)
- * appears in at least one step. Returns the missing paths.
+ * Сравнивает файловые требования задачи с описаниями шагов плана. Требование
+ * "покрыто", если его путь (или basename) есть хоть в одном шаге. Возвращает
+ * недостающие пути.
  */
 export function checkPlanCoverage(taskText, stepDescriptions) {
     const taskPaths = extractFilePaths(taskText);
@@ -61,7 +64,7 @@ export function checkPlanCoverage(taskText, stepDescriptions) {
     const missing = taskPaths.filter((p) => {
         const lower = p.toLowerCase();
         const base = basename(lower);
-        // A step mentioning either the full relative path or its basename counts.
+        // Шаг с полным относительным путём или его basename — засчитывается.
         return !stepText.includes(lower) && !stepText.includes(base);
     });
     return { missing };

package/dist/modules/execution/plan-steps.js

@@ -14,10 +14,18 @@
  * @returns true, если шаг был переключён (и план сохранён); false, если
  * kind уже был "delete" — ничего не менялось.
  */
-export function switchStepToDelete(plan, step, save) {
-    if (step.kind === "delete")
+export function switchStepKind(plan, step, kind, save) {
+    if (step.kind === kind)
         return false;
-    step.kind = "delete";
+    step.kind = kind;
     save(plan);
     return true;
 }
+/** Переключает шаг в kind="delete" (с сохранением плана). Возвращает, был ли сменён. */
+export function switchStepToDelete(plan, step, save) {
+    return switchStepKind(plan, step, "delete", save);
+}
+/** Переключает шаг в kind="create" (с сохранением плана). Возвращает, был ли сменён. */
+export function switchStepToCreate(plan, step, save) {
+    return switchStepKind(plan, step, "create", save);
+}

package/dist/modules/execution/plan-store.js

@@ -1,7 +1,9 @@
+// Персист планов: active/drafts/archive в .mma/plans + миграция legacy plan.json.
 import { readFileSync, writeFileSync, renameSync, mkdirSync, existsSync, readdirSync, rmSync } from "fs";
 import { join } from "path";
 import { Logger } from "../../logger/app-logger";
 import { errMsg } from "../../utils";
+import { projectMmaDir } from "../../config/paths";
 const log = new Logger("info", "plan-store");
 const LEGACY_FILE = "plan.json";
 function readPlanFile(path, fallbackBaseDir) {
@@ -26,11 +28,26 @@ function readPlanFile(path, fallbackBaseDir) {
     }
 }
 function writePlanFile(path, plan) {
-    // Atomic write: a crash mid-write must not leave a truncated plan file
-    // (readPlanFile silently returns null on broken JSON → active plan lost).
-    const tmpPath = `${path}.tmp`;
-    writeFileSync(tmpPath, JSON.stringify(plan, null, 2), "utf-8");
-    renameSync(tmpPath, path);
+    // Атомарная запись: крах посреди записи не оставит обрезанный файл плана
+    // (readPlanFile молча вернёт null на битом JSON → активный план потерян).
+    // Имя tmp уникально на процесс: два `mma` на общем `.mma/plans/active.json`
+    // не уводят друг у друга `.tmp` (ENOENT).
+    const tmpPath = `${path}.${process.pid}.${Math.random().toString(36).slice(2, 8)}.tmp`;
+    try {
+        writeFileSync(tmpPath, JSON.stringify(plan, null, 2), "utf-8");
+        renameSync(tmpPath, path);
+    }
+    catch (e) {
+        try {
+            if (existsSync(tmpPath))
+                rmSync(tmpPath);
+        }
+        catch {
+            /* best-effort очистка */
+        }
+        log.warn(`failed to persist plan to ${path}`, { error: errMsg(e) });
+        throw e;
+    }
 }
 function listDir(dir, baseDir) {
     if (!existsSync(dir))
@@ -49,6 +66,7 @@ function toMeta(plan, status) {
         doneCount: plan.steps.filter((s) => s.status === "done" || s.status === "skipped").length,
     };
 }
+/** Хранилище планов: active/drafts/archive + миграция legacy-файла. */
 export class PlanStore {
     baseDir;
     plansDir;
@@ -56,7 +74,7 @@ export class PlanStore {
     archiveDir;
     legacyPath;
     constructor(baseDir) {
-        const mmaDir = join(baseDir, ".mma");
+        const mmaDir = projectMmaDir(baseDir);
         if (!existsSync(mmaDir))
             mkdirSync(mmaDir, { recursive: true });
         this.baseDir = baseDir;
@@ -72,10 +90,48 @@ export class PlanStore {
     activePath() {
         return join(this.plansDir, "active.json");
     }
-    // ---- Active plan ----
+    /** Корень per-session слотов активного плана. */
+    sessionsDir() {
+        return join(this.plansDir, "sessions");
+    }
+    /** Проверяет id сессии как сегмент пути (без сепараторов/..). */
+    sessionActivePath(sessionId) {
+        if (!/^[A-Za-z0-9_-]+$/.test(sessionId)) {
+            throw new Error(`invalid session id: ${sessionId}`);
+        }
+        return join(this.sessionsDir(), sessionId, "active.json");
+    }
+    // ---- Активный план сессии ----
+    /** Сохраняет активный план сессии. */
+    saveSessionActive(sessionId, plan) {
+        const p = this.sessionActivePath(sessionId);
+        const dir = join(this.sessionsDir(), sessionId);
+        if (!existsSync(dir))
+            mkdirSync(dir, { recursive: true });
+        writePlanFile(p, plan);
+    }
+    /** Загружает активный план сессии. Пустая/новая сессия — null.
+     *  Без fallback на глобальный active.json: усыновление legacy-плана делает
+     *  явно только bootstrap (через switchSession-персист), иначе каждая новая
+     *  сессия наследовала бы чужой план. */
+    loadSessionActive(sessionId) {
+        const p = this.sessionActivePath(sessionId);
+        if (!existsSync(p))
+            return null;
+        return readPlanFile(p, this.baseDir);
+    }
+    /** Убирает активный план сессии. Нет слота — no-op. */
+    clearSessionActive(sessionId) {
+        const p = this.sessionActivePath(sessionId);
+        if (existsSync(p))
+            rmSync(p, { force: true });
+    }
+    // ---- Активный план ----
+    /** Сохраняет активный план. */
     saveActive(plan) {
         writePlanFile(this.activePath(), plan);
     }
+    /** Загружает активный план (с миграцией legacy .mma/plan.json). Null, если нет. */
     loadActive() {
         const activePath = this.activePath();
         if (existsSync(activePath)) {
@@ -83,7 +139,7 @@ export class PlanStore {
             if (plan)
                 return plan;
         }
-        // Legacy migration: read .mma/plan.json and move to active
+        // Миграция legacy: читаем .mma/plan.json и переносим в active
         if (existsSync(this.legacyPath)) {
             const legacy = readPlanFile(this.legacyPath, this.baseDir);
             if (legacy) {
@@ -102,47 +158,56 @@ export class PlanStore {
         }
         return null;
     }
+    /** Убирает активный план. */
     clearActive() {
         const p = this.activePath();
         if (existsSync(p))
             rmSync(p, { force: true });
     }
-    // ---- Drafts (saved plans) ----
+    // ---- Черновики (сохранённые планы) ----
+    /** Сохраняет план в черновики. */
     saveDraft(plan) {
         writePlanFile(join(this.draftsDir, `${plan.id}.json`), plan);
     }
+    /** Загружает черновик по id. Null, если нет. */
     loadDraft(id) {
         const p = join(this.draftsDir, `${id}.json`);
         return existsSync(p) ? readPlanFile(p, this.baseDir) : null;
     }
+    /** Удаляет черновик по id. */
     removeDraft(id) {
         const p = join(this.draftsDir, `${id}.json`);
         if (existsSync(p))
             rmSync(p, { force: true });
     }
+    /** Все черновики. */
     listDrafts() {
         return listDir(this.draftsDir, this.baseDir);
     }
-    // ---- Archive ----
+    // ---- Архив ----
+    /** Архивирует план (убирает из черновиков/активного при совпадении). */
     archivePlan(plan) {
         writePlanFile(join(this.archiveDir, `${plan.id}.json`), plan);
-        // Remove from drafts if present
+        // Убираем из черновиков, если там был
         this.removeDraft(plan.id);
-        // If archived plan was active, clear active
+        // Если архивируемый был активным — чистим active
         const active = this.loadActive();
         if (active && active.id === plan.id) {
             this.clearActive();
         }
     }
+    /** Все архивные планы. */
     listArchived() {
         return listDir(this.archiveDir, this.baseDir);
     }
+    /** Удаляет архивный план по id. */
     removeArchived(id) {
         const p = join(this.archiveDir, `${id}.json`);
         if (existsSync(p))
             rmSync(p, { force: true });
     }
-    // ---- Cross-status queries ----
+    // ---- Кросс-статусные запросы ----
+    /** Метаданные всех планов (активный + черновики + архив). */
     listAll() {
         const metas = [];
         const active = this.loadActive();
@@ -154,6 +219,7 @@ export class PlanStore {
             metas.push(toMeta(p, "archived"));
         return metas;
     }
+    /** Находит план по id в любом статусе. Null, если нет. */
     find(id) {
         const active = this.loadActive();
         if (active && active.id === id)
@@ -166,9 +232,9 @@ export class PlanStore {
             return { plan: archived, status: "archived" };
         return null;
     }
-    // ---- Deletion (the user asked for plans to actually go away) ----
-    /** Permanently delete a plan wherever it lives. Returns its status, or null
-     *  when no plan with that id exists. */
+    // ---- Удаление (юзер просил: планы должны реально исчезать) ----
+    /** Навсегда удаляет план, где бы ни лежал. Возвращает его статус, или null
+     *  если плана с таким id нет. */
     deletePlan(id) {
         const active = this.loadActive();
         if (active && active.id === id) {
@@ -187,7 +253,7 @@ export class PlanStore {
         }
         return null;
     }
-    /** Delete every plan (active, drafts, archive). Returns the number removed. */
+    /** Удаляет все планы (активный, черновики, архив). Возвращает число удалённых. */
     purgeAll() {
         let n = 0;
         const active = this.loadActive();

package/dist/modules/execution/plan-tool.js

@@ -1,10 +1,44 @@
 import { t } from "../../i18n/index";
-import { dispatchPlanAction } from "./plan-actions";
+import { dispatchPlanAction, updatePlanStep } from "./plan-actions";
 /**
- * The plan / todo / verify tool definitions for the execution module. The
- * handlers were extracted verbatim from ExecutionModule.getToolDefinitions() —
- * all state access goes through the `deps` closures (the module owns the
- * tracker and the plan store).
+ * Маркит подзадачи done по числовым id (предпочтительно) или точному тексту (фолбэк).
+ * Возвращает число отмеченных, или строку ошибки если ничего не сошлось.
+ */
+function markTodoDone(subs, ids, items) {
+    let marked = 0;
+    if (ids.length > 0) {
+        for (const id of ids) {
+            const sub = subs.find((s) => !s.done && s.id === id);
+            if (sub) {
+                sub.done = true;
+                marked++;
+            }
+        }
+        if (marked === 0) {
+            return {
+                error: t("todo.subtask_id_not_found", { ids: ids.join(", ") }),
+            };
+        }
+        return { marked };
+    }
+    for (const item of items) {
+        const sub = subs.find((s) => !s.done && s.text.toLowerCase() === item.toLowerCase());
+        if (sub) {
+            sub.done = true;
+            marked++;
+        }
+    }
+    if (marked === 0) {
+        return {
+            error: t("todo.subtask_not_found", { items: items.join(", ") }),
+        };
+    }
+    return { marked };
+}
+/**
+ * Определения тулов plan / todo / verify для execution-модуля. Хендлеры
+ * вынесены дословно из ExecutionModule.getToolDefinitions() — весь доступ к
+ * состоянию идёт через `deps`-замыкания (трекер и стор принадлежат модулю).
  */
 export function createPlanToolDefinitions(deps) {
     return [
@@ -12,7 +46,7 @@ export function createPlanToolDefinitions(deps) {
             name: "plan",
             icon: "📋",
             alwaysOn: true,
-            description: 'Create, update, show, abort, list, switch, delete, purge, or re-plan multi-step plans.\n\nActions:\n- create: Start a new plan. Previous active plan is auto-preserved: incomplete → draft, complete → archive.\n- update: Mark step status (done/failed/skipped), or rebuild plan with new steps.\n- show: Print current plan checklist.\n- abort: Archive the current plan (or the plan given by id) and clear the active slot.\n- list: Show all plans (active, drafts, archived) with progress.\n- switch: Make a different plan active (by plan id).\n- re-plan: Iterative replanning: keep completed steps, replace remaining with new steps.\n- delete: Permanently delete a plan by id (plan delete id=plan_xxx).\n- purge: Delete ALL plans (active, drafts, archived).\n\nWrite CONCRETE steps with exact file paths and commands:\n- Specify WHICH files to create with exact paths (e.g. "create src/components/Header.tsx with navigation and logo")\n- Specify WHICH packages to install (e.g. "run npm install react react-dom")\n- Specify WHICH CLI commands to run with exact arguments\n- Each step should include at least one file extension (.ts, .tsx, .json, etc.) or a command verb (install, create, build, run, add, init)\n- Good: "Создать src/components/Header.tsx с навигацией и логотипом"\n- Bad: "Настройка проекта" (too vague — what exactly needs to be configured?)\n- For steps that REMOVE files, pass kinds: ["delete"] aligned with steps (the done-gate verifies those files are gone instead of demanding they exist).',
+            description: 'Create, update, show, abort, list, switch, delete, purge, or re-plan multi-step plans.\n\nActions:\n- create: Start a new plan. Previous active plan is auto-preserved: incomplete → draft, complete → archive.\n- update: Mark step status (done/failed/skipped). To change the plan\'s steps use "re-plan" (keeps completed steps) or "create".\n- show: Print current plan checklist.\n- abort: Archive the current plan (or the plan given by id) and clear the active slot.\n- list: Show all plans (active, drafts, archived) with progress.\n- switch: Make a different plan active (by plan id).\n- re-plan: Iterative replanning: keep completed steps, replace remaining with new steps.\n- delete: Permanently delete a plan by id (plan delete id=plan_xxx).\n- purge: Delete ALL plans (active, drafts, archived).\n\nWrite CONCRETE steps with exact file paths and commands:\n- Specify WHICH files to create with exact paths (e.g. "create src/components/Header.tsx with navigation and logo")\n- Specify WHICH packages to install (e.g. "run npm install react react-dom")\n- Specify WHICH CLI commands to run with exact arguments\n- Each step should include at least one file extension (.ts, .tsx, .json, etc.) or a command verb (install, create, build, run, add, init)\n- Good: "Создать src/components/Header.tsx с навигацией и логотипом"\n- Bad: "Настройка проекта" (too vague — what exactly needs to be configured?)\n- Steps that REMOVE files need no extra flag: create/delete intent is detected automatically from what the agent actually does.\n- Every step needs a verification contract: pass "checks" aligned 1:1 with "steps". Each check is "file-exists:<path>" (verified automatically on done), "command:<exact command>" (run it before done), or "manual:<what to check by eye>" (UI/visual work only). Example: steps ["Create src/a.ts with X"] + checks ["file-exists:src/a.ts"].',
             parameters: {
                 type: "object",
                 properties: {
@@ -32,10 +66,10 @@ export function createPlanToolDefinitions(deps) {
                     },
                     title: { type: "string" },
                     steps: { type: "array", items: { type: "string" } },
-                    kinds: {
+                    checks: {
                         type: "array",
-                        items: { type: "string", enum: ["create", "delete"] },
-                        description: 'Optional per-step intent, aligned 1:1 with steps. Pass "delete" for steps whose purpose is REMOVING files — the done-gate then requires those files to be GONE (not present). Defaults to "create".',
+                        items: { type: "string" },
+                        description: 'REQUIRED on create/re-plan, aligned 1:1 with steps. One contract per step: "file-exists:<path>", "command:<cmd>", or "manual:<text>".',
                     },
                     step: { type: "number" },
                     status: { type: "string", enum: ["done", "failed", "skipped"] },
@@ -56,16 +90,45 @@ export function createPlanToolDefinitions(deps) {
                 };
             },
         },
+        {
+            name: "plan_step",
+            icon: "📋",
+            alwaysOn: true,
+            description: 'Mark ONE plan step done/failed/skipped: plan_step step=N status=done|failed|skipped (note="..." optional). Prefer this over "plan update" — no action parameter to confuse. The step\'s check contract is verified automatically on done ("file-exists:" hard-verified, "command:"/"manual:" self-certified).',
+            parameters: {
+                type: "object",
+                properties: {
+                    step: { type: "number", description: "Step number to update" },
+                    status: { type: "string", enum: ["done", "failed", "skipped"] },
+                    note: { type: "string" },
+                },
+                required: ["step"],
+            },
+            handler: async (ctx, args) => {
+                if (!deps.trackerRef.current) {
+                    return { success: false, output: t("plan.no_active") };
+                }
+                if (args.step === undefined || args.step === null || Number.isNaN(Number(args.step))) {
+                    return { success: false, output: t("plan.step_required") };
+                }
+                return updatePlanStep(deps, ctx, args);
+            },
+        },
         {
             name: "todo",
             icon: "☑️",
             alwaysOn: true,
-            description: "Manage sub-tasks within current plan step. Use to break down complex steps into smaller tasks.",
+            description: "Manage sub-tasks within current plan step. Use to break down complex steps into smaller tasks. Mark done by subtask ids from todo list (ids=[...]) — reliable; matching by text is a fallback.",
             parameters: {
                 type: "object",
                 properties: {
                     action: { type: "string", enum: ["add", "done", "list"] },
                     items: { type: "array", items: { type: "string" } },
+                    ids: {
+                        type: "array",
+                        items: { type: "number" },
+                        description: "Subtask ids to mark done (see todo list output). Preferred over items.",
+                    },
                 },
                 required: ["action"],
             },
@@ -89,48 +152,39 @@ export function createPlanToolDefinitions(deps) {
                     currentStep.subtasks = existing;
                     deps.store.saveActive(tracker.getPlan());
                     const display = tracker.toPromptBlock();
+                    const addedIds = existing.slice(-items.length).map((s) => s.id);
+                    const addedOutput = t("todo.added", {
+                        count: String(items.length),
+                        items: items.join(", "),
+                    }) + ` (ids: ${addedIds.join(", ")})`;
                     return {
                         success: true,
-                        output: t("todo.added", {
-                            count: String(items.length),
-                            items: items.join(", "),
-                        }),
+                        output: addedOutput,
                         display,
                     };
                 }
                 if (action === "done") {
+                    const ids = Array.isArray(args.ids) ? args.ids.map(Number).filter((n) => !Number.isNaN(n)) : [];
                     const items = Array.isArray(args.items) ? args.items.map(String) : [];
-                    if (items.length === 0) {
+                    if (ids.length === 0 && items.length === 0) {
                         return {
                             success: false,
                             output: t("todo.no_items"),
                         };
                     }
                     const subs = currentStep.subtasks ?? [];
-                    let marked = 0;
-                    for (const item of items) {
-                        const sub = subs.find((s) => !s.done && s.text.toLowerCase() === item.toLowerCase());
-                        if (sub) {
-                            sub.done = true;
-                            marked++;
-                        }
-                    }
-                    if (marked === 0) {
-                        return {
-                            success: false,
-                            output: t("todo.subtask_not_found", {
-                                items: items.join(", "),
-                            }),
-                        };
+                    const done = markTodoDone(subs, ids, items);
+                    if ("error" in done) {
+                        return { success: false, output: done.error };
                     }
+                    const marked = done.marked;
                     currentStep.subtasks = subs;
                     deps.store.saveActive(tracker.getPlan());
                     const doneCount = subs.filter((s) => s.done).length;
                     const display = tracker.toPromptBlock();
-                    // Marking a sub-task done must NOT auto-complete the plan
-                    // step — the step is only done when the model confirms the
-                    // whole step via plan update (observed: 3 × "todo done"
-                    // returned "Step 3: done" and marked the step done early).
+                    // Отметка подзадачи done НЕ должна автозавершать шаг плана: шаг done,
+                    // только когда модель подтвердит весь шаг через plan update (было: 3 ×
+                    // "todo done" вернули "Step 3: done" и преждевременно закрыли шаг).
                     let output = t("todo.marked_done", {
                         count: String(marked),
                     });
@@ -143,7 +197,7 @@ export function createPlanToolDefinitions(deps) {
                 if (action === "list") {
                     const subs = currentStep.subtasks ?? [];
                     const lines = subs.length
-                        ? subs.map((s) => `${s.done ? "[x]" : "[ ]"} ${s.text}`)
+                        ? subs.map((s) => `[${s.id}] ${s.done ? "[x]" : "[ ]"} ${s.text}`)
                         : [t("plan.no_subtasks")];
                     return {
                         success: true,