@nodaro/shared 2.21.0 → 2.22.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nodaro/shared",
-  "version": "2.21.0",
+  "version": "2.22.0",
   "description": "Shared types, model catalog, wire contracts, and structural vocabularies for the Nodaro platform and SDK.",
   "type": "module",
   "license": "Apache-2.0",

package/src/__tests__/safety-retry-policy.test.ts

@@ -1,3 +1,4 @@
+import { T2I_TO_I2I_VARIANT } from "../model-constants.js"
 import { describe, it, expect } from "vitest"
 import { safetyRetryPolicy } from "../safety-retry-policy.js"
 import { MODEL_CATALOG, getModel } from "../model-catalog.js"
@@ -45,3 +46,17 @@ describe("safetyRetryPolicy", () => {
     }
   })
 })
+
+describe("i2i variants inherit their base model's safety policy", () => {
+  it("resolves the referenced-request id (gpt-image-2-i2i) to the same policy as gpt-image-2", () => {
+    expect(safetyRetryPolicy("gpt-image-2-i2i")).toEqual({ maxAttempts: 2, fallback: "nano-banana-pro" })
+  })
+
+  it("every T2I_TO_I2I_VARIANT pair declares the same safetyFilter on both sides", () => {
+    for (const [t2i, i2i] of Object.entries(T2I_TO_I2I_VARIANT)) {
+      const base = MODEL_CATALOG[t2i]?.safetyFilter ?? null
+      const twin = MODEL_CATALOG[i2i]?.safetyFilter ?? null
+      expect({ pair: `${t2i} -> ${i2i}`, twin }).toEqual({ pair: `${t2i} -> ${i2i}`, twin: base })
+    }
+  })
+})

package/src/index.ts

@@ -112,6 +112,7 @@ export {
   checkRefVideoDurations,
   VIDEO_PROVIDERS_REQUIRING_IMAGE,
   videoProviderRequiresImage,
+  videoProviderFoldsLoneEndFrame,
   VIDEO_MODE_ALIASES,
   VIDEO_GEN_COLLAPSED_T2V_IDS,
   resolveVideoProviderForMode,

package/src/model-catalog.ts

@@ -579,6 +579,7 @@ const IMAGE_MODELS: Record<string, ModelCatalogEntry> = {
     label: "GPT Image 2 (I2I)",
     series: "GPT Image",
     description: "Image-to-image with GPT Image 2.",
+    safetyFilter: { stochastic: true, fallback: "nano-banana-pro" },
     useCases: ["edit", "high-res"],
     features: ["reference-image"],
     aspectRatios: GPT_IMAGE_2_RATIOS,

package/src/model-constants.ts

@@ -2105,6 +2105,29 @@ export function videoProviderRequiresImage(provider: string | undefined): boolea
   return !!provider && VIDEO_PROVIDERS_REQUIRING_IMAGE.has(provider)
 }
 
+/**
+ * True when the provider's image-to-video path FOLDS a lone closing frame into
+ * its reference images — i.e. an end frame with no start frame is a reference
+ * the model actually receives, not a half-formed frame pair.
+ *
+ * The Seedance 2.x, MiniMax Hailuo 3 and Wan 3 families all assemble their KIE
+ * input through the shared `resolveSeedance2Inputs` (`@nodaro/prompts`), whose
+ * reference mode takes a lone last frame as the sole reference image plus a
+ * closing-frame prompt hint. NOTHING else does: VEO ships `[imageUrl,
+ * endFrameUrl]` verbatim (a lone end frame would reach KIE as `[null, url]`),
+ * the Gemini Omni branch drops the frame when there is no start frame, and the
+ * generic KIE path sends `end_frame` with no image param at all. So the
+ * exemption is keyed to the FOLD, not to "carries image references" — the
+ * catalog has no capability for it, and these three family predicates are the
+ * same membership the provider layer itself branches on (the backend's
+ * `kie/video.ts::applySeedance2Params` / `applyMinimaxH3Params` / `runWan3`).
+ * A ref-capable provider that grows the fold gets the exemption by joining a
+ * family here; one that does not keeps the honest `image_required` 400.
+ */
+export function videoProviderFoldsLoneEndFrame(provider: string): boolean {
+  return isSeedance2Provider(provider) || isMinimaxH3Provider(provider) || isWan3Provider(provider)
+}
+
 /**
  * Video models where enabling audio/sound incurs an additional cost.
  * The audio addon is expressed as a separate composite identifier suffix.

package/src/node-runtime-keys.ts

@@ -21,6 +21,13 @@ export const EXECUTION_DATA_KEYS: ReadonlySet<string> = new Set([
   // lifecycle as errorMessage: a RESULT the user expects to survive reload,
   // never user-edited config.
   "errorHint",
+  // A job policy registered by the deployment held this node's result for a
+  // human reviewer (`jobs.status = "pending_review"`). Pure run state, like
+  // executionStatus: the node is still "running" and the flag disappears the
+  // moment the review resolves — so it is ALSO in TRANSIENT_RUNTIME_KEYS
+  // below. Without the transient half, a flip into review marks a passive tab
+  // dirty and a preset captures "awaiting review".
+  "jobAwaitingReview",
   "isStreaming",
   "generatedImageUrl",
   "generatedVideoUrl",
@@ -85,6 +92,7 @@ export const TRANSIENT_RUNTIME_KEYS: ReadonlySet<string> = new Set([
   "executionStatus",
   "currentJobId",
   "currentJobProgress",
+  "jobAwaitingReview",
   "isStreaming",
   "subWorkflowProgress",
   "__listTotal",