@lacrous/omnicloud 0.2.12 → 0.2.14

package/dist/index.d.cts

@@ -809,11 +809,13 @@ interface StorageStats {
  *
  * PENDING    accepted, Telegram write not started
  * UPLOADING  Telegram write in progress or its outcome unknown
+ * COMMITTING the stored object is being turned into a file record by exactly one
+ *            request, which holds this state as an exclusive claim
  * COMPLETED  Telegram object stored and file record committed; result is final
  * FAILED     Telegram write definitively failed; the operation may be retried
  *            under a new id
  */
-type UploadOperationStatus = "PENDING" | "UPLOADING" | "COMPLETED" | "FAILED";
+type UploadOperationStatus = "PENDING" | "UPLOADING" | "COMMITTING" | "COMPLETED" | "FAILED";
 
 /**
  * Repository interfaces. The application depends on these, not on a specific

package/dist/index.d.ts

@@ -809,11 +809,13 @@ interface StorageStats {
  *
  * PENDING    accepted, Telegram write not started
  * UPLOADING  Telegram write in progress or its outcome unknown
+ * COMMITTING the stored object is being turned into a file record by exactly one
+ *            request, which holds this state as an exclusive claim
  * COMPLETED  Telegram object stored and file record committed; result is final
  * FAILED     Telegram write definitively failed; the operation may be retried
  *            under a new id
  */
-type UploadOperationStatus = "PENDING" | "UPLOADING" | "COMPLETED" | "FAILED";
+type UploadOperationStatus = "PENDING" | "UPLOADING" | "COMMITTING" | "COMPLETED" | "FAILED";
 
 /**
  * Repository interfaces. The application depends on these, not on a specific

package/dist/index.js

@@ -1409,7 +1409,7 @@ var noopActivityRecorder = {
 // ../core/src/utils/filename.ts
 function sanitizeFileName(name) {
   const base = name.split(/[/\\]/).pop() ?? name;
-  const cleaned = base.replace(/[\u0000-\u001f\u007f]/g, "").replace(/[<>:"|?*]/g, "_").trim();
+  const cleaned = base.replace(/[\u0000-\u001f\u007f-\u009f]/g, "").replace(/[\u200B-\u200F\u202A-\u202E\u2066-\u2069\uFEFF]/g, "").replace(/[<>:"|?*]/g, "_").trim();
   if (!cleaned || cleaned === "." || cleaned === "..") return "";
   return cleaned.slice(0, 255);
 }
@@ -1479,6 +1479,7 @@ var FileService = class {
       const record = await this.performUpload(userId, input, {
         onStored: async (stored) => {
           await operations.update(op.id, { telegramMessageId: Number(stored.messageId) });
+          await operations.claim(op.id, "UPLOADING", "COMMITTING");
         }
       });
       await operations.update(op.id, { status: "COMPLETED", fileId: record.id });
@@ -1519,13 +1520,27 @@ var FileService = class {
     const mimeType = mimeFromFilename(name);
     const size = op.size ?? input.spooled?.size ?? input.data?.byteLength ?? 0;
     const sha256 = op.sha256 ?? input.spooled?.sha256 ?? sha256Hex(input.data ?? Buffer.alloc(0));
-    const record = await this.commitRecord(userId, input, {
-      name,
-      mimeType,
-      size,
-      sha256,
-      telegramMessageId: op.telegramMessageId
-    });
+    const claimed = await operations.claim(op.id, "UPLOADING", "COMMITTING");
+    if (!claimed) {
+      const current = await operations.findByOperationId(userId, op.operationId);
+      if (current?.status === "COMPLETED" && current.fileId) {
+        return this.get(userId, current.fileId);
+      }
+      throw new ConflictError("This upload is being committed by another request. Retry shortly.");
+    }
+    let record;
+    try {
+      record = await this.commitRecord(userId, input, {
+        name,
+        mimeType,
+        size,
+        sha256,
+        telegramMessageId: op.telegramMessageId
+      });
+    } catch (error) {
+      await operations.claim(op.id, "COMMITTING", "UPLOADING");
+      throw error;
+    }
     await operations.update(op.id, { status: "COMPLETED", fileId: record.id });
     return record;
   }