micro-models-agent 2.10.0 → 2.12.0

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

@@ -1,4 +1,5 @@
 import { normalizeInputFrom } from "./input-from";
+/** Валидирует план: подзадачи, теги экспертов, циклы, пересечения, input_from. */
 export function validatePlan(plan, config) {
     const errors = [];
     const warnings = [];
@@ -30,9 +31,10 @@ export function validatePlan(plan, config) {
     };
 }
 /**
- * Every input_from reference must point at an existing subtask and be backed
- * by a depends_on edge — otherwise the reader may run before its producer.
+ * Каждая ссылка input_from обязана указывать на существующую подзадачу и
+ * подкрепляться ребром depends_on — иначе читатель стартует раньше продьюсера.
  */
+/** Валидирует ссылки input_from: неизвестные — ошибки, недостающие depends_on — автофиксы. */
 export function validateInputFromRefs(subtasks) {
     const errors = [];
     const autoFixes = [];
@@ -49,6 +51,7 @@ export function validateInputFromRefs(subtasks) {
     }
     return { errors, autoFixes };
 }
+/** Проверяет, что все expert_tag подзадач известны из конфига. Возвращает ошибки. */
 export function expertTagsExist(subtasks, config) {
     const errors = [];
     const knownExperts = new Set(Object.keys(config.experts || {}));
@@ -59,6 +62,7 @@ export function expertTagsExist(subtasks, config) {
     }
     return errors;
 }
+/** Ищет циклы в графе depends_on (DFS). Возвращает ошибки. */
 export function hasCycle(subtasks) {
     const edges = new Map();
     for (const s of subtasks) {
@@ -92,6 +96,7 @@ export function hasCycle(subtasks) {
     }
     return errors;
 }
+/** Ищет write-write пересечения allowed_files без цепочки зависимостей. Возвращает ошибки. */
 export function hasAllowedFilesOverlap(subtasks) {
     const errors = [];
     const writeFiles = new Map();
@@ -116,6 +121,15 @@ export function hasAllowedFilesOverlap(subtasks) {
     }
     return errors;
 }
+/** Два спека путей пересекаются, если равны или один — каталог-предок другого. */
+function pathsOverlap(a, b) {
+    const na = a.replace(/\\/g, "/").replace(/\/+$/, "");
+    const nb = b.replace(/\\/g, "/").replace(/\/+$/, "");
+    if (na === nb)
+        return true;
+    return na.startsWith(`${nb}/`) || nb.startsWith(`${na}/`);
+}
+/** Ищет read-after-write пересечения (писатель vs read_only читателя). Пустые depends_on — автофикс. */
 export function deleteReadOverlap(subtasks) {
     const results = [];
     for (let i = 0; i < subtasks.length; i++) {
@@ -128,13 +142,10 @@ export function deleteReadOverlap(subtasks) {
                 continue;
             const readFiles = new Set(reader.read_only_files);
             const writeFiles = writer.allowed_files;
-            const overlapping = writeFiles.filter((f) => {
-                for (const rf of readFiles) {
-                    if (f.startsWith(rf) || rf.startsWith(f))
-                        return true;
-                }
-                return false;
-            });
+            // Точное совпадение или настоящий каталог-предок, НЕ строковый префикс:
+            // иначе сиблинги `src/main.ts` (писатель) vs `src/main.tsx` (читатель)
+            // ложно флаговались.
+            const overlapping = writeFiles.filter((f) => [...readFiles].some((rf) => pathsOverlap(f, rf)));
             if (overlapping.length > 0 && !reader.depends_on?.includes(writer.id)) {
                 const canAutoFix = !reader.depends_on || reader.depends_on.length === 0;
                 if (canAutoFix) {
@@ -157,8 +168,17 @@ export function deleteReadOverlap(subtasks) {
     }
     return results;
 }
+/** Применяет автофиксы (добавление depends_on) к копии плана, оригинал не мутирует. */
 export function applyAutoFixes(plan, autoFixes) {
-    const fixed = { ...plan, subtasks: plan.subtasks.map((s) => ({ ...s })) };
+    // Глубокое-ish клонирование: копируем каждый массив `depends_on`, чтобы пуши
+    // ниже не мутировали план вызывающего (поверхностный `{...s}` делил массив).
+    const fixed = {
+        ...plan,
+        subtasks: plan.subtasks.map((s) => ({
+            ...s,
+            depends_on: [...(s.depends_on ?? [])],
+        })),
+    };
     for (const fix of autoFixes) {
         const sub = fixed.subtasks.find((s) => s.id === fix.subtaskId);
         if (!sub)

package/dist/modules/execution/planner.js

@@ -7,10 +7,10 @@ function generatePlanId() {
     return `plan_${id}`;
 }
 /**
- * Single source of truth for plan progress counting (ses_mt4fn58c: `show`
- * said "6/7 done, complete" while `list` said "7/7" for the same plan —
- * one counted only "done", the other counted done+skipped). A skipped step
- * needs no further work, so it counts as settled everywhere.
+ * Единый источник правды подсчёта прогресса (ses_mt4fn58c: `show` говорил
+ * "6/7 done, complete", а `list` — "7/7" для того же плана: один считал только
+ * "done", другой done+skipped). Пропущенный шаг работы не требует, поэтому
+ * везде считается settled.
  */
 export function getPlanProgress(plan) {
     const done = plan.steps.filter((s) => s.status === "done").length;
@@ -19,8 +19,10 @@ export function getPlanProgress(plan) {
     const settled = done + skipped;
     return { done, skipped, total, settled, terminal: total > 0 && settled === total };
 }
+/** Создатель планов: create, re-plan, промпт-блок. */
 export class PlanCreator {
-    static createPlan(title, stepDescriptions, baseDir, kinds) {
+    /** Создаёт новый план из заголовка и описаний шагов. */
+    static createPlan(title, stepDescriptions, baseDir, kinds, checks) {
         const stepCount = stepDescriptions.length;
         return {
             id: generatePlanId(),
@@ -30,26 +32,29 @@ export class PlanCreator {
                 description: desc,
                 status: "pending",
                 kind: kinds?.[i] ?? "create",
+                ...(checks?.[i] ? { check: checks[i] } : {}),
             })),
             createdAt: new Date().toISOString(),
             baseDir,
         };
     }
-    static replan(plan, newSteps, title) {
+    /** Re-plan: сохраняет done/skipped шаги, заменяет остальные новыми. */
+    static replan(plan, newSteps, title, newChecks) {
         const kept = plan.steps.filter((s) => s.status === "done" || s.status === "skipped");
         const baseTitle = plan.title.replace(/^\[\d+[^]]*\]\s*/, "");
         const newTitle = title || baseTitle;
         const totalSteps = kept.length + newSteps.length;
-        const added = newSteps.map((desc) => ({
-            id: -1, // renumbered below
+        const added = newSteps.map((desc, i) => ({
+            id: -1, // перенумеруется ниже
             description: desc,
             status: "pending",
             kind: "create",
+            ...(newChecks?.[i] ? { check: newChecks[i] } : {}),
         }));
-        // Renumber every step (kept + added) sequentially. Kept steps keep their
-        // status and kind but may collide with the added ids when the original
-        // plan skipped/pending steps in the middle (e.g. done 1, done 3 → kept
-        // [1,3], added [2…] collides with kept id 3).
+        // Перенумеровываем все шаги (сохранённые + новые) подряд. Сохранённые держат
+        // статус и kind, но их id могут коллизить с новыми, если исходный план
+        // пропускал/пендил шаги в середине (напр. done 1, done 3 → сохранённые
+        // [1,3], новые [2…] коллизят с сохранённым id 3).
         const steps = [...kept, ...added].map((s, i) => ({ ...s, id: i + 1 }));
         return {
             id: plan.id,
@@ -60,6 +65,7 @@ export class PlanCreator {
             name: plan.name,
         };
     }
+    /** Рендерит план чеклистом для промпта (иконки статусов, checks, подзадачи). */
     static toPromptBlock(plan, currentStepIndex) {
         const date = plan.createdAt.slice(0, 10);
         const p = getPlanProgress(plan);
@@ -85,8 +91,11 @@ export class PlanCreator {
                             : "[ ]";
             const note = step.note ? ` — ${step.note}` : "";
             lines.push(`${icon} ${step.id}. ${step.description}${note}`);
+            if (step.check) {
+                lines.push(`    ↳ check: ${step.check}`);
+            }
             for (const sub of step.subtasks ?? []) {
-                lines.push(`    ${sub.done ? "[x]" : "[ ]"} ${sub.text}`);
+                lines.push(`    [${sub.id}] ${sub.done ? "[x]" : "[ ]"} ${sub.text}`);
             }
         }
         return lines.join("\n");