@shop-os/modelkit 0.3.0 → 0.3.1

package/CHANGELOG.md

@@ -9,6 +9,18 @@ header in the release PR.
 
 ## [Unreleased]
 
+## [0.3.1] - Unreleased
+
+### Added
+- `providerOptions.modelkit.inputSpec` now works on fal image rows, so a new fal image model is a database row. The spec uses the image keys (`prompt`, `references`, `aspectRatio`, `size`, `outputFormat`, `seed`) with `fields`, `values` and `static`, exactly as on Replicate image rows. The new optional `editEndpoint` (fal rows only) names the endpoint called instead of `providerModelId` when the request has references, for models that split text-to-image and edit across two endpoints. Only the mapped fields are sent: no `num_images`, so a spec row returns one image per call. A fal row without an `inputSpec` keeps the Muse-shaped request and Ideogram keeps its own path. fal video rows still reject `inputSpec`. On fal an explicit `inputSpec` wins over the built-in Ideogram path, and `editEndpoint` needs a `references` entry in `fields`.
+- `resolveRowInputSpec(req, ctx)` and `invalidInputSpecError(provider, ctx, issues)` in the shared mapper, used by the Replicate and fal adapters.
+
+### Changed
+- fal image calls decode every image in the response that has a URL and fail only when none do. The Muse path used to fail when the first image had no URL even if later ones did.
+
+### Live rows to add after publish
+Add a fal image row that carries an `inputSpec` only after every app that will call it runs 0.3.1. An older kit accepts the row but ignores the `inputSpec`, so it sends the Muse-shaped request and the call fails or loses its fields.
+
 ## [0.3.0] - Unreleased
 
 ### Added

package/README.md

@@ -213,8 +213,8 @@ The Bedrock model id is `<routing.bedrockProfile>.anthropic.<providerModelId>`
 breaks the pattern sets `providerOptions.modelkit.bedrockModelId` on its row.
 See the CHANGELOG entry for the caller-visible Bedrock limits.
 
-A Replicate video, image or audio row for a new model carries `providerOptions.modelkit.inputSpec`
-instead of a kit change. It is read by the Replicate adapter only. `fields` maps a canonical key to the provider field name,
+A Replicate video, image or audio row, or a fal image row, for a new model carries `providerOptions.modelkit.inputSpec`
+instead of a kit change. It is read by the Replicate and fal adapters only. `fields` maps a canonical key to the provider field name,
 `values` remaps canonical values per key, and `static` merges literal provider fields
 last. Video rows use `VIDEO_INPUT_SPEC_KEYS` (`prompt`, `durationSeconds`,
 `aspectRatio`, `resolution`, `firstFrame`, `lastFrame`, `seed`, `negativePrompt`,
@@ -223,7 +223,7 @@ last. Video rows use `VIDEO_INPUT_SPEC_KEYS` (`prompt`, `durationSeconds`,
 `seed`; map `references` to the provider's array field and remap `size` and `outputFormat`
 with `values`); audio rows use
 `AUDIO_INPUT_SPEC_KEYS` (`prompt`, `durationSeconds`, `seed`, `negativePrompt`,
-`references`, `sourceVideo`, `instrumental`). `aspectRatio: "adaptive"` is omitted.
+`references`, `sourceVideo`, `instrumental`). `aspectRatio: "adaptive"` is omitted. A fal image row has no kind and builds its whole input from the spec; the optional `editEndpoint` (fal rows only) is the fal endpoint called instead of `providerModelId` when the request has references. fal rows without an `inputSpec` keep the Muse-shaped request and Ideogram keeps its own path; an `inputSpec` on an Ideogram row replaces that path. `editEndpoint` needs a `references` entry in `fields`. `inputSpec` on fal video rows is not supported.
 `validateInputSpec` is the shared contract the write schema, `dataOnlyCapable`, and
 the Replicate mapper all check. `dataOnlyCapable(row)` reports whether such a row would
 be served.

package/dist/catalog.cjs

@@ -2317,6 +2317,10 @@ var ALL_KEY_SET = /* @__PURE__ */ new Set([
   ...IMAGE_INPUT_SPEC_KEYS,
   ...AUDIO_INPUT_SPEC_KEYS
 ]);
+var TYPES_BY_ADAPTER = /* @__PURE__ */ new Map([
+  ["replicate", ["video", "image", "audio"]],
+  ["fal", ["image"]]
+]);
 var ALLOWED_KEYS_BY_TYPE = /* @__PURE__ */ new Map([
   ["video", VIDEO_KEY_SET],
   ["image", IMAGE_KEY_SET],
@@ -2355,7 +2359,8 @@ var StaticSchema = import_zod2.z.record(import_zod2.z.string(), ProviderScalarSc
 var InputSpecSchema = import_zod2.z.strictObject({
   fields: FieldsSchema,
   values: ValuesSchema.optional(),
-  static: StaticSchema.optional()
+  static: StaticSchema.optional(),
+  editEndpoint: import_zod2.z.string().min(1).optional()
 }).superRefine((spec, ctx) => {
   const mappedNames = new Set(Object.values(spec.fields));
   for (const key of Object.keys(spec.static ?? {})) {
@@ -2380,18 +2385,21 @@ function validateInputSpec(raw, row) {
     };
   }
   const spec = parsed.data;
-  if (row.adapter !== "replicate") {
+  const supportedTypes = TYPES_BY_ADAPTER.get(row.adapter);
+  if (supportedTypes === void 0) {
     return {
       ok: false,
-      issues: [{ path: [], message: "inputSpec is only supported on replicate rows" }]
+      issues: [
+        { path: [], message: "inputSpec is only supported on replicate and fal rows" }
+      ]
     };
   }
   const issues = [];
   const allowed = ALLOWED_KEYS_BY_TYPE.get(row.type);
-  if (allowed === void 0) {
+  if (allowed === void 0 || !supportedTypes.includes(row.type)) {
     issues.push({
       path: [],
-      message: `inputSpec is not supported for "${row.type}" rows`
+      message: row.adapter === "fal" ? `inputSpec on fal rows is only supported for image rows, not "${row.type}"` : `inputSpec is not supported for "${row.type}" rows`
     });
   } else {
     for (const key of Object.keys(spec.fields)) {
@@ -2411,6 +2419,19 @@ function validateInputSpec(raw, row) {
       }
     }
   }
+  if (spec.editEndpoint !== void 0) {
+    if (row.adapter !== "fal") {
+      issues.push({
+        path: ["editEndpoint"],
+        message: "inputSpec.editEndpoint is only supported on fal rows"
+      });
+    } else if (!("references" in spec.fields)) {
+      issues.push({
+        path: ["editEndpoint"],
+        message: 'inputSpec.editEndpoint needs a "references" entry in inputSpec.fields'
+      });
+    }
+  }
   const seenNames = /* @__PURE__ */ new Map();
   for (const [key, name] of Object.entries(spec.fields)) {
     const first = seenNames.get(name);