@arela/uploader 1.0.14 → 1.0.15

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@arela/uploader",
-  "version": "1.0.14",
+  "version": "1.0.15",
   "description": "CLI to upload files/directories to Arela",
   "bin": {
     "arela": "./src/index.js"

package/src/commands/PushCommand.js

@@ -283,6 +283,11 @@ export class PushCommand {
     // After each batch upload, those files are no longer "pending", so the next query
     // at offset=0 will naturally return the next batch of unprocessed files
 
+    // Track seen file IDs to detect infinite loops (scan table update failures)
+    const seenFileIds = new Set();
+    let consecutiveRepeats = 0;
+    const MAX_CONSECUTIVE_REPEATS = 3;
+
     // Start progress bar with known total
     progressBar.start(totalToProcess, 0, {
       speed: 0,
@@ -303,6 +308,23 @@ export class PushCommand {
         break;
       }
 
+      // Infinite loop protection: if the same files keep coming back,
+      // the scan table update is failing and they stay "pending" forever.
+      const allSeen = files.every((f) => seenFileIds.has(f.id));
+      if (allSeen) {
+        consecutiveRepeats++;
+        if (consecutiveRepeats >= MAX_CONSECUTIVE_REPEATS) {
+          const msg = `Aborting: same ${files.length} files returned ${MAX_CONSECUTIVE_REPEATS} times — scan table updates are likely failing.`;
+          logger.error(msg);
+          console.error(`\n⚠  ${msg}`);
+          hasMore = false;
+          break;
+        }
+      } else {
+        consecutiveRepeats = 0;
+      }
+      files.forEach((f) => seenFileIds.add(f.id));
+
       // Upload files in smaller batches using new CLI upload endpoint
       for (let i = 0; i < files.length; i += uploadBatchSize) {
         const uploadBatch = files.slice(i, i + uploadBatchSize);
@@ -328,9 +350,9 @@ export class PushCommand {
         try {
           await this.scanApiService.batchUpdateUpload(tableName, batchResults);
         } catch (updateError) {
-          logger.error(
-            `Failed to update scan table for batch: ${updateError.message}`,
-          );
+          const msg = `Failed to update scan table for batch: ${updateError.message}`;
+          logger.error(msg);
+          console.error(`\n⚠  ${msg}`);
           // Don't fail the entire process, just log the error
         }
 
@@ -482,7 +504,13 @@ export class PushCommand {
       if (apiResult.uploaded && apiResult.uploaded.length > 0) {
         const uploadedFile = apiResult.uploaded[0];
         result.uploaded = true;
-        result.uploadedToStorageId = uploadedFile.storageId;
+        // Only assign storageId if it is a valid UUID; ignore placeholder values
+        const UUID_RE =
+          /^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/i;
+        result.uploadedToStorageId =
+          uploadedFile.storageId && UUID_RE.test(uploadedFile.storageId)
+            ? uploadedFile.storageId
+            : null;
         logger.info(`✓ Uploaded: ${file.file_name} → ${uploadPath}`);
       } else if (apiResult.errors && apiResult.errors.length > 0) {
         const error = apiResult.errors[0];

package/src/config/config.js

@@ -34,10 +34,10 @@ class Config {
       const __dirname = path.dirname(__filename);
       const packageJsonPath = path.resolve(__dirname, '../../package.json');
       const packageJson = JSON.parse(fs.readFileSync(packageJsonPath, 'utf-8'));
-      return packageJson.version || '1.0.14';
+      return packageJson.version || '1.0.15';
     } catch (error) {
       console.warn('⚠️ Could not read package.json version, using fallback');
-      return '1.0.14';
+      return '1.0.15';
     }
   }