fix: improve error handling and logging in asset upload script

Author: younocode

dockers/teable/Dockerfile

@@ -78,16 +78,12 @@ RUN set -ex; \
         pnpm install --prod --prefer-offline --frozen-lockfile; \
         pnpm -F @teable/db-main-prisma prisma-generate --schema ./prisma/postgres/schema.prisma
 
-RUN chmod +x ./scripts/upload-assets.mjs;
-RUN echo "UPLOAD_ASSETS_LIST: $UPLOAD_ASSETS_LIST";
-RUN zx ./scripts/upload-assets.mjs --list="$UPLOAD_ASSETS_LIST";
-
-# RUN set -ex; \
-#         chmod +x ./scripts/upload-assets.mjs; \
-#         echo "UPLOAD_ASSETS_LIST: $UPLOAD_ASSETS_LIST"; \
-#         if [ -n "$UPLOAD_ASSETS_LIST" ]; then \
-#           zx ./scripts/upload-assets.mjs --list="$UPLOAD_ASSETS_LIST"; \
-#         fi
+RUN set -ex; \
+        chmod +x ./scripts/upload-assets.mjs; \
+        echo "UPLOAD_ASSETS_LIST: $UPLOAD_ASSETS_LIST"; \
+        if [ -n "$UPLOAD_ASSETS_LIST" ]; then \
+          zx ./scripts/upload-assets.mjs --list="$UPLOAD_ASSETS_LIST"; \
+        fi
 
 
 ##################################################################

scripts/upload-assets.mjs

@@ -7,13 +7,31 @@ const mcPath = '~/minio-binaries/mc';
 // [[name, endpoint, accessKey, secretKey, bucket]]
 const { list } = argv;
 
-console.log('test list: ', list);
-const parsedList = list && typeof list === 'string' ? JSON.parse(list) : [];
+
+let parsedList = [];
+if (list && typeof list === 'string' && list.trim() !== '') {
+  try {
+    parsedList = JSON.parse(list);
+  } catch (error) {
+    console.warn('Warning: Failed to parse list JSON:', error.message);
+    console.log('Skipping upload due to invalid configuration...');
+    process.exit(0);
+  }
+}
+
 if (!Array.isArray(parsedList)) {
-  console.error('Error: list must be a array');
-  return;
+  console.warn('Warning: list must be a array, but got:', typeof parsedList);
+  console.log('Skipping upload due to invalid configuration...');
+  process.exit(0);
 }
 
+if (parsedList.length === 0) {
+  console.log('No upload assets list provided, skipping upload...');
+  process.exit(0);
+}
+
+console.log('upload assets list: ', JSON.stringify(parsedList));
+
 const checkPlatform = async () => {
   const platform = await $`uname -m`;
   const os = await $`uname -s`;