@animastor/orchestration 0.3.0 → 0.3.1

package/CHANGELOG.md

@@ -1,5 +1,29 @@
 # Changelog
 
+## 0.3.1 (2026-11-01) — результат GPU обрабатывается один раз
+
+Основание: `reports/2026-11-01/gpu-audio-image-run-analysis.md`, P1 — каждый
+результат обрабатывался дважды (HTTP-push hub → backend, затем Phase A
+reconcile подбирал оставшийся `animastor:result:*` мимо
+`animastor:result-processed:*`). Описание: animastor-docs
+`architecture/GPU_HUB_CONTRACT.md` §«Result ACK».
+
+### Changed (поведение)
+
+- `reconciliation-engine.recoverResultKeys` (Phase A): перед
+  `handleTaskResult` читает маркер `animastor:result-processed:{job}:{build}`.
+  Значение `done:<dispatch_id>` того же dispatch, что в result-ключе, —
+  ключ удаляется молча (без лога), handler не вызывается.
+  Маркер `'1'` (в работе / старый формат), маркер другого dispatch
+  (перегенерация audio/video в пределах TTL) или его отсутствие — как
+  раньше, через fenced handler.
+- Терминальный отказ (`no_active_dispatch` и др.) для дубля доставленного
+  результата (маркер есть) — уровень info; для сироты (маркера нет) — warn.
+- `orchestrator.resetScenes` вместе с `iu-progress` удаляет
+  `animastor:iu-progress-ids:*` (множество IU×fence, хост task-handler).
+
+Fence, dedup-ключ маршрута и command-gate (Stage 155) не меняются.
+
 ## 0.3.0 (2026-10-31) — генерация только по явной команде
 
 Правило владельца: GPU-генерация запускается только кнопками «аудио» /

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@animastor/orchestration",
-    "version": "0.3.0",
+    "version": "0.3.1",
     "description": "Animastor Orchestration — the generation orchestration/runtime contour of the Animastor backend, physically extracted from the host (§32.30): the unified generation lifecycle owner (orchestrator facade, stage executor, scene callbacks/restoration), the dispatch engine and its lease/quota/retry managers, the reconciliation engine and counter reconciliation, the scene window/scheduler/worker-health, the runtime-result emitter/consumer, and the eight O-2..O-10 port contracts (Persistence, SceneData, PlaceholderAudio, ProgressEvents, AudioFsm, VideoFsm, HubCancel, LayerConfig). The package reaches the host (runtime config slices, Redis-domain state layer, media facades, the S-5 seam registry, the artifact root, the PW-2 workspace resolver) ONLY through the composition-root host-bindings seam — no backend/src require, no storage/book/routes/services, no Redis, no PG, no Express inside.",
     "keywords": [
         "animastor",

package/src/orchestration/orchestrator.js

@@ -575,6 +575,8 @@ async function resetScenes(redis, bookId, buildId, scenes, layerCfg, options = {
 
         const progKey = `animastor:iu-progress:${bookId}:${ds.chapter_id}:${ds.scene_id}:image`;
         try { await redis.del(progKey); } catch (_) {}
+        // SYNC: backend task-handler — the IU×fence set behind the counter.
+        try { await redis.del(`animastor:iu-progress-ids:${bookId}:${ds.chapter_id}:${ds.scene_id}:image`); } catch (_) {}
 
         try {
             let cursor = '0';

package/src/runtime/reconciliation-engine.js

@@ -63,6 +63,25 @@ function error(msg) {
     console.error(`${logPrefix} ❌ ${msg}`);
 }
 
+// SYNC: backend/src/helpers/queue-idempotency.cjs — resultDedupKey /
+// readResultDedup (marker `'1'` = in flight, `done:<dispatch_id>` = settled).
+// Read-only here: the result route is the only writer.
+const RESULT_DEDUP_DONE_PREFIX = 'done:';
+async function readResultDedup(redis, jobId, buildId) {
+    let raw = null;
+    try {
+        raw = await redis.get(`animastor:result-processed:${jobId}:${buildId}`);
+    } catch (_) {
+        return { state: 'unknown', dispatchId: null };
+    }
+    if (raw == null || raw === false) return { state: 'none', dispatchId: null };
+    const v = String(raw);
+    if (v.startsWith(RESULT_DEDUP_DONE_PREFIX)) {
+        return { state: 'settled', dispatchId: v.slice(RESULT_DEDUP_DONE_PREFIX.length) || null };
+    }
+    return { state: 'in_flight', dispatchId: null };
+}
+
 // ======================================================
 // RECONCILIATION REPORT
 // ======================================================
@@ -1947,11 +1966,34 @@ async function recoverResultKeys(redis, deps, scope) {
                     warn(`Skipping legacy result without build/dispatch identity: ${job_id}`);
                     continue;
                 }
+
+                // P1 (2026-11-01): the hub's fallback copy of a delivery the
+                // result route already SETTLED is a duplicate — drop it
+                // silently, never replay it (the replay re-ran the ready chain
+                // and logged false no_active_dispatch warnings). Only a marker
+                // carrying THIS dispatch counts: an in-flight / legacy '1'
+                // marker or one from an older dispatch of the same job_id
+                // (audio/video regeneration within the TTL) is no proof — such
+                // a result goes through the fenced handler exactly as before.
+                const dedup = await readResultDedup(redis, job_id, buildId);
+                if (dedup.state === 'settled' && dedup.dispatchId === String(dispatchId)) {
+                    await redis.del(key);
+                    continue;
+                }
                 try {
                     await taskHandler.handleTaskResult(job_id, result_base64, buildId, dispatchId);
                 } catch (rejectErr) {
                     if (rejectErr && TERMINAL_REJECTIONS.has(rejectErr.code)) {
-                        warn(`Result ${job_id} terminally rejected (${rejectErr.code}) — dropping result key, no infinite retry`);
+                        // A replay of a delivery the route has seen (a marker
+                        // exists) is a duplicate: info. No marker (an orphan
+                        // that never reached the route) or an unreadable
+                        // marker: warn, as before.
+                        const msg = `Result ${job_id} terminally rejected (${rejectErr.code}) — dropping result key, no infinite retry`;
+                        if (dedup.state === 'in_flight' || dedup.state === 'settled') {
+                            log(`${msg} (duplicate of a delivered result)`);
+                        } else {
+                            warn(msg);
+                        }
                         await redis.del(key);
                     } else {
                         throw rejectErr;