vidspotai-shared 1.0.51 → 1.0.52

package/lib/services/aiGen/providers/google/google.service.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"google.service.d.ts","sourceRoot":"","sources":["../../../../../src/services/aiGen/providers/google/google.service.ts"],"names":[],"mappings":"AAaA,OAAO,EAAE,wBAAwB,EAAE,MAAM,8BAA8B,CAAC;AACxE,OAAO,EACL,iBAAiB,EACjB,qBAAqB,EACrB,qBAAqB,EACrB,iBAAiB,EACjB,iBAAiB,EAClB,MAAM,UAAU,CAAC;AA6BlB,qBAAa,aAAc,SAAQ,wBAAwB;IACzD,OAAO,CAAC,EAAE,CAAc;IACxB,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,kBAAkB,CAAK;;IAO/C;;;;OAIG;YACW,kBAAkB;IA0B1B,aAAa,CACjB,MAAM,EAAE,qBAAqB,GAC5B,OAAO,CAAC,qBAAqB,CAAC;IAyC3B,gBAAgB,CAAC,EACrB,IAAI,EACJ,cAAc,EACd,cAAyB,GAC1B,EAAE,iBAAiB,GAAG,OAAO,CAAC,iBAAiB,CAAC;IAuEjD,aAAa,CAAC,EAAE,QAAQ,EAAE,QAAY,EAAE,UAAmB,EAAE,SAAiB,EAAE,EAAE,iBAAiB,GAAG,MAAM;CAiB7G"}
+{"version":3,"file":"google.service.d.ts","sourceRoot":"","sources":["../../../../../src/services/aiGen/providers/google/google.service.ts"],"names":[],"mappings":"AAcA,OAAO,EAAE,wBAAwB,EAAE,MAAM,8BAA8B,CAAC;AACxE,OAAO,EACL,iBAAiB,EACjB,qBAAqB,EACrB,qBAAqB,EACrB,iBAAiB,EACjB,iBAAiB,EAClB,MAAM,UAAU,CAAC;AA0DlB,qBAAa,aAAc,SAAQ,wBAAwB;IACzD,OAAO,CAAC,EAAE,CAAc;IACxB,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,kBAAkB,CAAK;;IAO/C;;;;OAIG;YACW,kBAAkB;IA0B1B,aAAa,CACjB,MAAM,EAAE,qBAAqB,GAC5B,OAAO,CAAC,qBAAqB,CAAC;IAqE3B,gBAAgB,CAAC,EACrB,IAAI,EACJ,cAAc,EACd,cAAyB,GAC1B,EAAE,iBAAiB,GAAG,OAAO,CAAC,iBAAiB,CAAC;IAuFjD,aAAa,CAAC,EAAE,QAAQ,EAAE,QAAY,EAAE,UAAmB,EAAE,SAAiB,EAAE,EAAE,iBAAiB,GAAG,MAAM;CAiB7G"}

package/lib/services/aiGen/providers/google/google.service.js

@@ -12,6 +12,7 @@ const types_1 = require("../../../../globals/types");
 const firebase_1 = require("../../../../libs/firebase");
 const helpers_1 = require("../../../../utils/helpers");
 const logger_1 = require("../../../../utils/logger");
+const errors_1 = require("../../../../utils/errors");
 const helpers_2 = require("../../helpers");
 const baseAiGenProvider_service_1 = require("../baseAiGenProvider.service");
 const fs_1 = require("fs");
@@ -31,6 +32,35 @@ const TRANSIENT_NETWORK_CODES = new Set([
     "UND_ERR_HEADERS_TIMEOUT",
     "UND_ERR_BODY_TIMEOUT",
 ]);
+// Pick a Veo-compatible image MIME type. Veo accepts image/jpeg, image/png,
+// image/webp. We prefer the upstream Content-Type header (Firebase Storage
+// returns the type set at upload), then fall back to URL extension, then to
+// image/jpeg as a reasonable last resort for phone-camera uploads.
+const VEO_SUPPORTED_IMAGE_MIMES = new Set([
+    "image/jpeg",
+    "image/png",
+    "image/webp",
+]);
+function pickImageMimeType(contentType, url) {
+    const normalized = contentType.split(";")[0]?.trim().toLowerCase();
+    if (normalized && VEO_SUPPORTED_IMAGE_MIMES.has(normalized)) {
+        return normalized;
+    }
+    // Map common but non-canonical types
+    if (normalized === "image/jpg")
+        return "image/jpeg";
+    // Extension fallback. Strip query string first (Firebase URLs have ?alt=media).
+    const pathOnly = url.split("?")[0]?.toLowerCase() ?? "";
+    if (pathOnly.endsWith(".png"))
+        return "image/png";
+    if (pathOnly.endsWith(".webp"))
+        return "image/webp";
+    if (pathOnly.endsWith(".jpg") || pathOnly.endsWith(".jpeg"))
+        return "image/jpeg";
+    // Phone uploads default to JPEG far more often than PNG, so it's a safer
+    // last-resort than the previous hardcoded "image/png".
+    return "image/jpeg";
+}
 function isTransientFetchError(err) {
     if (!err)
         return false;
@@ -93,9 +123,28 @@ class GoogleService extends baseAiGenProvider_service_1.BaseAiGenProviderService
         };
         if (params.inputImageUrl) {
             const imgResp = await this.withTransientRetry("input-image fetch", () => fetch(params.inputImageUrl));
+            // BUG #3 fix: fetch() does NOT throw on 4xx/5xx — it returns a Response
+            // with the error body. Without this check we'd encode an HTML error page
+            // as if it were image bytes, and Veo would silently reject it.
+            if (!imgResp.ok) {
+                throw new errors_1.UserFacingError(`Input image could not be downloaded (HTTP ${imgResp.status}). The image URL may have expired or been deleted.`);
+            }
             const imgBuffer = Buffer.from(await imgResp.arrayBuffer());
+            // BUG #2 fix: a 0-byte body slips through fetch().ok if the upstream
+            // wrote an empty file. Veo treats empty bytes as content-filtered (no
+            // error, no video). Surface the real cause to the user.
+            if (imgBuffer.length === 0) {
+                throw new errors_1.UserFacingError("Input image is empty (0 bytes). Please re-upload the image.");
+            }
+            // BUG #1 fix: previously hardcoded "image/png" regardless of the actual
+            // upload type. Veo rejects JPEG bytes labeled as PNG, with no error
+            // detail. Prefer the Content-Type from the response (authoritative —
+            // Firebase Storage sets it from upload metadata), fall back to URL
+            // extension, then default to JPEG (modern phone-camera default).
+            const respContentType = imgResp.headers.get("content-type") ?? "";
+            const mimeType = pickImageMimeType(respContentType, params.inputImageUrl);
             request.image = {
-                mimeType: "image/png", // or infer from URL (jpg/png)
+                mimeType,
                 imageBytes: imgBuffer.toString("base64"),
             };
         }
@@ -118,6 +167,22 @@ class GoogleService extends baseAiGenProvider_service_1.BaseAiGenProviderService
             }
             const videoUri = result.response?.generatedVideos?.[0]?.video?.uri;
             if (!videoUri) {
+                // Veo silently filters content via RAI without setting result.error.
+                // When raiMediaFilteredCount > 0, the request was blocked by safety
+                // filters. Surface the reasons so we can show the user what to fix,
+                // and prefix with `user_input_error:` so sceneMonitor demotes the log
+                // level (these are expected user-input failures, not bugs).
+                const filteredCount = result.response?.raiMediaFilteredCount ?? 0;
+                const filteredReasons = result.response?.raiMediaFilteredReasons ?? [];
+                if (filteredCount > 0 || filteredReasons.length > 0) {
+                    const reasons = filteredReasons.length > 0
+                        ? filteredReasons.join("; ")
+                        : "Content blocked by Google safety filters";
+                    return {
+                        status: types_1.EVideoSceneStatus.FAILED,
+                        errorMessage: `user_input_error: ${reasons}`,
+                    };
+                }
                 return {
                     status: types_1.EVideoSceneStatus.FAILED,
                     errorMessage: "No video URL or VideoBytes found in response",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "vidspotai-shared",
-  "version": "1.0.51",
+  "version": "1.0.52",
   "main": "lib/index.js",
   "types": "lib/index.d.ts",
   "exports": {