kingkont 0.19.1 → 0.19.2

package/lib/eventStore.js

@@ -41,6 +41,8 @@ function init({ userDataDir }) {
   if (_userDataDir) {
     const dir = path.join(_userDataDir, 'events');
     try { fs.mkdirSync(dir, { recursive: true }); } catch {}
+    // Lazy migration на первом list/listAll — не блокируем init.
+    setImmediate(_migrateGlobalEvents);
   }
 }
 
@@ -68,6 +70,65 @@ function _ensureLoaded(projectKey) {
   return list;
 }
 
+// Одноразовая миграция: legacy events в `_global` без target.projectKey,
+// но с target.nodeId — пытаемся найти их nodeId в других bucket'ах и
+// переместить event в правильный bucket. Без неё клик по таким events
+// делает navigateToTarget early-return (`if (!target.projectKey) return`).
+let _migrationDone = false;
+function _migrateGlobalEvents() {
+  if (_migrationDone || !_userDataDir) return;
+  _migrationDone = true;
+  try {
+    // Загрузим все bucket'ы с диска для поиска nodeId.
+    const dir = path.join(_userDataDir, 'events');
+    let names;
+    try { names = fs.readdirSync(dir); } catch { return; }
+    const allNodeMap = new Map();   // nodeId → projectKey
+    for (const n of names) {
+      if (!n.endsWith('.json')) continue;
+      const pk = decodeURIComponent(n.replace(/\.json$/, ''));
+      if (pk === '_global') continue;
+      const evts = _ensureLoaded(pk);
+      for (const e of evts) {
+        const nid = e.target?.nodeId;
+        if (nid && !allNodeMap.has(nid)) allNodeMap.set(nid, pk);
+      }
+    }
+    const global = _ensureLoaded('_global');
+    if (!global.length) return;
+    const stays = [];
+    const moveBuckets = new Map();   // pk → events[]
+    for (const e of global) {
+      const nid = e.target?.nodeId;
+      const pk = nid ? allNodeMap.get(nid) : null;
+      if (pk) {
+        if (!moveBuckets.has(pk)) moveBuckets.set(pk, []);
+        moveBuckets.get(pk).push({ ...e, target: { ...e.target, projectKey: pk } });
+      } else {
+        stays.push(e);
+      }
+    }
+    if (moveBuckets.size === 0) return;
+    // Применяем перемещение.
+    for (const [pk, evts] of moveBuckets.entries()) {
+      const target = _ensureLoaded(pk);
+      // Дедуп по (nodeId, ts) — может уже был добавлен.
+      for (const e of evts) {
+        if (!target.some(t => t.target?.nodeId === e.target.nodeId && t.ts === e.ts)) {
+          target.push(e);
+        }
+      }
+      target.sort((a, b) => a.ts - b.ts);
+      _schedulePersist(pk);
+    }
+    _cache.set('_global', stays);
+    _schedulePersist('_global');
+    console.log(`[eventStore] migrated ${global.length - stays.length} events из _global в правильные bucket'ы`);
+  } catch (e) {
+    console.warn('[eventStore] migration failed:', e?.message);
+  }
+}
+
 function _schedulePersist(projectKey) {
   if (!_userDataDir) return;
   if (_persistTimers.has(projectKey)) clearTimeout(_persistTimers.get(projectKey));
@@ -89,22 +150,26 @@ function append(projectKey, event) {
   const pk = projectKey || '_global';
   const list = _ensureLoaded(pk);
   const ts = event.ts || Date.now();
-  // Дедуп (mirror'ит клиентскую логику в notifyPanel.js):
-  //   - completion-event с nodeId — ключ `node:<nodeId>` (gen завершён)
-  //   - chat-final event — ключ `chat:<first 60 chars text>` (chat-final
-  //     приходит И от server-side chatSession, И от client showToast)
-  //   - всё остальное — ключ `text:<first 80 chars>` (короткое окно)
-  // Окно дедупа — 30 секунд.
+  // Дедуп:
+  //   - node:<id> completion (ok/error/info/warn) — окно 24h (gen
+  //     каждой ноды завершается ОДИН раз; повторное событие — точно
+  //     дубль, разнос времени бывает 20+ мин если client поллит после
+  //     долгой паузы и повторно генерит showToast).
+  //   - chat-final — окно 24h тоже.
+  //   - всё остальное — 30s.
   let dk = null;
+  let windowMs = 30 * 1000;
   if (event.target?.nodeId && (event.kind === 'ok' || event.kind === 'error' || event.kind === 'info' || event.kind === 'warn')) {
     dk = `node:${event.target.nodeId}`;
+    windowMs = 24 * 3600 * 1000;
   } else if (event.target?.chat) {
     dk = `chat:${String(event.text).slice(0, 60)}`;
+    windowMs = 24 * 3600 * 1000;
   } else {
     dk = `text:${String(event.text).slice(0, 80)}`;
   }
   if (dk) {
-    const cutoff = ts - 30 * 1000;
+    const cutoff = ts - windowMs;
     for (let i = list.length - 1; i >= 0; i--) {
       if (list[i].ts < cutoff) break;
       if (list[i]._dk === dk) return;   // already have it
@@ -124,9 +189,23 @@ function append(projectKey, event) {
   _schedulePersist(pk);
 }
 
+// Backward-compat helper: гарантируем что у каждого event'а в target есть
+// projectKey (равный bucket-key где он лежит). Старые client-persisted
+// события писались БЕЗ target.projectKey — `navigateToTarget` для них
+// молча возвращал ('if (!target.projectKey) return') → клик не работал.
+function _injectProjectKey(events, bucketKey) {
+  if (bucketKey === '_global') return events;
+  return events.map(e => {
+    if (e.target && !e.target.projectKey) {
+      return { ...e, target: { ...e.target, projectKey: bucketKey } };
+    }
+    return e;
+  });
+}
+
 function list(projectKey, { limit = MAX_PER_PROJECT } = {}) {
   const pk = projectKey || '_global';
-  const own = _ensureLoaded(pk).slice();
+  let own = _injectProjectKey(_ensureLoaded(pk).slice(), pk);
   // Backward-compat: события могли быть mis-filed в _global (когда юзер
   // на welcome генерил, _persistEvent.projectKey был null). Извлекаем
   // те у которых target.projectKey совпадает с запрошенным.
@@ -155,7 +234,14 @@ function listAll({ limit = 200 } = {}) {
   }
   const merged = [];
   for (const [pk, evts] of _cache.entries()) {
-    for (const e of evts) merged.push({ ...e, projectKey: pk });
+    // Inject projectKey в target если отсутствует (backward-compat).
+    for (const e of evts) {
+      const ev = { ...e, projectKey: pk };
+      if (e.target && !e.target.projectKey && pk !== '_global') {
+        ev.target = { ...e.target, projectKey: pk };
+      }
+      merged.push(ev);
+    }
   }
   merged.sort((a, b) => a.ts - b.ts);
   return merged.slice(-limit);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "kingkont",
-  "version": "0.19.1",
+  "version": "0.19.2",
   "description": "KingKont · Chatium — нод-редактор сцен с AI-генерацией (картинки/видео/голос/SFX/музыка/текст)",
   "main": "main.js",
   "bin": {

package/renderer/generate.js

@@ -1969,7 +1969,12 @@ async function resumeJob(node, bKey, boardHandle) {
 
   if (node.generated.taskId) {
     // Уже зарегистрировано в KIE — просто опрашиваем.
-    const job = { boardKey: bKey, boardHandle, kind, taskId: node.generated.taskId, nodeId: node.id };
+    // КРИТИЧНО: projectKey обязателен — без него на completion showToast
+    // создаст event с target БЕЗ projectKey → клик по уведомлению ведёт
+    // в никуда (navigateToTarget рано return'ит). Берём из текущего state.
+    const projectKey = state.cloudProjectId ? 'cloud:' + state.cloudProjectId
+                     : state.filmHandle?.name ? 'folder:' + state.filmHandle.name : null;
+    const job = { boardKey: bKey, boardHandle, kind, taskId: node.generated.taskId, nodeId: node.id, projectKey };
     state.jobs.set(node.id, job);
     updateJobsBadge();
     try {

package/renderer/notifyPanel.js

@@ -251,7 +251,8 @@
   // не ловился (юзер видел два одинаковых «✓ image «name» готов»,
   // один синий, второй зелёный). Сейчас одинаковый nodeId-completion
   // в окне 30s — один event.
-  const DEDUP_WINDOW_MS = 30 * 1000;
+  const DEDUP_DEFAULT_WINDOW_MS = 30 * 1000;
+  const DEDUP_NODE_WINDOW_MS = 24 * 3600 * 1000;   // node-completion: дубль через 20+ мин — всё равно дубль
   // События с этими kind считаются «completion» — для них дедупим по nodeId.
   const _COMPLETION_KINDS = new Set(['ok', 'error', 'info', 'warn']);
   function _dedupKey(kind, text, target) {
@@ -265,7 +266,10 @@
     const k = kind || 'info';
     const dk = _dedupKey(k, text, target);
     if (dk) {
-      const cutoff = Date.now() - DEDUP_WINDOW_MS;
+      // Для node-events окно длинное (gen завершается ОДИН раз), для всего
+      // остального — короткое (могут быть несколько похожих info-toast'ов).
+      const windowMs = dk.startsWith('node:') ? DEDUP_NODE_WINDOW_MS : DEDUP_DEFAULT_WINDOW_MS;
+      const cutoff = Date.now() - windowMs;
       for (let i = events.length - 1; i >= 0; i--) {
         if (events[i].ts < cutoff) break;
         if (events[i]._dk === dk) {