@nodaro/shared 2.21.0 → 2.21.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nodaro/shared",
-  "version": "2.21.0",
+  "version": "2.21.1",
   "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/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/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",