Add support for video uploads

packages/cli/package.json

@@ -34,9 +34,9 @@
     "test": "node --experimental-vm-modules node_modules/jest/bin/jest.js"
   },
   "devDependencies": {
-    "@metaplex-foundation/js": "0.20.0",
     "@jest/globals": "^29.5.0",
     "@jest/types": "^29.5.0",
+    "@metaplex-foundation/js": "0.20.0",
     "@swc/jest": "^0.2.26",
     "@types/commander": "^2.12.2",
     "@types/debug": "^4.1.7",
@@ -64,6 +64,7 @@
     "dotenv": "^16.0.3",
     "esm": "^3.2.25",
     "generate-schema": "^2.6.0",
+    "get-video-dimensions": "^1.0.0",
     "image-size": "^1.0.2",
     "js-yaml": "^4.1.0",
     "semver": "^7.3.8",

packages/cli/src/config/PublishDetails.ts

@@ -18,6 +18,7 @@ import { Constants, showMessage } from "../CliUtils.js";
 import util from "util";
 import { imageSize } from "image-size";
 import { exec } from "child_process";
+import getVideoDimensions from "get-video-dimensions";
 
 const runImgSize = util.promisify(imageSize);
 const runExec = util.promisify(exec);
@@ -125,19 +126,45 @@ export const loadPublishDetailsWithChecks = async (
   }
 
   config.release.media.forEach((item: PublishDetails["release"]["media"][0]) => {
-    const imagePath = path.join(process.cwd(), item.uri);
-    if (!fs.existsSync(imagePath) || !checkImageExtension(imagePath)) {
-      throw new Error(`Invalid media path or file type: ${item.uri}. Please ensure the file is a jpeg, png, or webp file.`);
+    const mediaPath = path.join(process.cwd(), item.uri);
+    if (!fs.existsSync(mediaPath)) {
+      throw new Error(`File doesnt exist: ${item.uri}.`)
+    }
+
+    if (item.purpose == "screenshot" && !checkImageExtension(mediaPath)) {
+      throw new Error(`Please ensure the file ${item.uri} is a jpeg, png, or webp file.`)
+    }
+
+    if (item.purpose == "video" && !checkVideoExtension(mediaPath)) {
+      throw new Error(`Please ensure the file ${item.uri} is a mp4.`)
     }
   }
   );
 
-  const previewMediaFiles = config.release.media?.filter(
-    (asset: any) => asset.purpose === "screenshot" || asset.purpose === "video"
+  const screenshots = config.release.media?.filter(
+    (asset: any) => asset.purpose === "screenshot"
+  )
+
+  for (const item of screenshots) {
+    const mediaPath = path.join(process.cwd(), item.uri);
+    if (await checkScreenshotSize(mediaPath)) {
+      throw new Error(`Screenshot ${mediaPath} must be at least 1080px in width and height.`);
+    }
+  }
+
+  const videos = config.release.media?.filter(
+    (asset: any) => asset.purpose === "video"
   )
 
-  if (previewMediaFiles.length < 4) {
-    throw new Error(`At least 4 screenshots or videos are required for publishing a new release. Found only ${previewMediaFiles.length}`)
+  for (const video of videos) {
+    const mediaPath = path.join(process.cwd(), video.uri);
+    if (await checkVideoSize(mediaPath)) {
+      throw new Error(`Video ${mediaPath} must be at least 1080px in width and height.`);
+    }
+  }
+
+  if (screenshots.length + videos.length < 4) {
+    throw new Error(`At least 4 screenshots or videos are required for publishing a new release. Found only ${screenshots.length + videos.length}`)
   }
 
   validateLocalizableResources(config);
@@ -174,6 +201,13 @@ const checkImageExtension = (uri: string): boolean => {
   );
 };
 
+const checkVideoExtension = (uri: string): boolean => {
+  const fileExt = path.extname(uri).toLowerCase();
+  return (
+    fileExt == ".mp4"
+  );
+};
+
 /**
  * We need to pre-check some things in the localized resources before we move forward
  */
@@ -206,6 +240,19 @@ const checkIconDimensions = async (iconPath: string): Promise<boolean> => {
   return size?.width != size?.height || (size?.width ?? 0) != 512;
 };
 
+const checkScreenshotSize = async (imagePath: string): Promise<boolean> => {
+  const size = await runImgSize(imagePath);
+
+  return (size?.width ?? 0) < 1080 || (size?.height ?? 0) < 1080;
+}
+
+const checkVideoSize = async (imagePath: string): Promise<boolean> => {
+  const size = await getVideoDimensions(imagePath);
+
+  return (size?.width ?? 0) < 1080 || (size?.height ?? 0) < 1080;
+}
+
+
 const getAndroidDetails = async (
   aaptDir: string,
   apkPath: string

packages/core/package.json

@@ -34,6 +34,7 @@
     "@types/debug": "^4.1.7",
     "@types/mime": "^3.0.1",
     "@types/node-fetch": "^2.6.2",
+    "get-video-dimensions": "^1.0.0",
     "json-schema-to-typescript": "^11.0.2",
     "shx": "^0.3.4"
   },

packages/core/src/create/ReleaseCore.ts

@@ -9,6 +9,7 @@ import { Constants, mintNft } from "../CoreUtils.js";
 import * as util from "util";
 import { metaplexFileReplacer, validateRelease } from "../validate/CoreValidation.js";
 import { imageSize } from "image-size";
+import getVideoDimensions from "get-video-dimensions";
 
 import type { Keypair, PublicKey } from "@solana/web3.js";
 import type {
@@ -46,14 +47,25 @@ const getFileMetadata = async (item: Media | File) => {
 };
 
 const getMediaMetadata = async (item: Media) => {
-  const size = await runImgSize(item.uri ?? "");
   const metadata = await getFileMetadata(item);
 
-  return {
-    ...metadata,
-    width: size?.width ?? 0,
-    height: size?.height ?? 0,
-  };
+  if (item.purpose == "screenshot" || item.purpose == "icon") {
+    const size = await runImgSize(item.uri ?? "");
+
+    return {
+      ...metadata,
+      width: size?.width ?? 0,
+      height: size?.height ?? 0,
+    };
+  } else {
+    const size = await getVideoDimensions(item.uri ?? "");
+
+    return {
+      ...metadata,
+      width: size?.width ?? 0,
+      height: size?.height ?? 0,
+    };
+  }
 };
 
 export const createReleaseJson = async (