@sequent-org/moodboard 1.4.9 → 1.4.11

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sequent-org/moodboard",
-  "version": "1.4.9",
+  "version": "1.4.11",
   "type": "module",
   "description": "Interactive moodboard",
   "main": "./src/index.js",

package/src/core/ApiClient.js

@@ -67,11 +67,17 @@ export class ApiClient {
             const objects = Array.isArray(cleanedData?.objects) ? cleanedData.objects : [];
             const imageObjects = objects.filter((obj) => obj?.type === 'image');
             const imageObjectsWithSrc = imageObjects.filter((obj) => typeof obj?.src === 'string' && obj.src.trim().length > 0);
+            const imageObjectsWithoutSrc = imageObjects
+                .filter((obj) => !(typeof obj?.src === 'string' && obj.src.trim().length > 0))
+                .map((obj) => obj?.id || 'unknown');
             console.log('history/save payload stats:', {
                 totalObjects: objects.length,
                 imageObjects: imageObjects.length,
                 imageObjectsWithSrc: imageObjectsWithSrc.length
             });
+            if (imageObjectsWithoutSrc.length > 0) {
+                console.warn('history/save warning: image objects without src (kept as broken placeholders):', imageObjectsWithoutSrc);
+            }
 
             const csrfToken = document.querySelector('meta[name="csrf-token"]')?.getAttribute('content');
 
@@ -157,7 +163,7 @@ export class ApiClient {
                 const normalizedSrc = topSrcRaw.trim() || propSrcRaw.trim();
 
                 if (!normalizedSrc) {
-                    throw new Error(`Image object "${obj.id || 'unknown'}" has no src. Save is blocked.`);
+                    return obj;
                 }
                 if (/^data:/i.test(normalizedSrc) || /^blob:/i.test(normalizedSrc)) {
                     throw new Error(`Image object "${obj.id || 'unknown'}" contains forbidden data/blob src. Save is blocked.`);
@@ -174,9 +180,6 @@ export class ApiClient {
                     cleanedObj.properties = { ...cleanedObj.properties };
                     delete cleanedObj.properties.src;
                 }
-                if ('imageId' in cleanedObj) {
-                    delete cleanedObj.imageId;
-                }
                 return cleanedObj;
             }
             
@@ -236,9 +239,6 @@ export class ApiClient {
                         restoredObj.properties = { ...restoredObj.properties };
                         delete restoredObj.properties.src;
                     }
-                    if ('imageId' in restoredObj) {
-                        delete restoredObj.imageId;
-                    }
                     return restoredObj;
                 }
                 

package/src/core/SaveManager.js

@@ -104,9 +104,9 @@ export class SaveManager {
                 });
             }
 
-            // Жесткий контракт для сохранения картинок:
-            // - каждый image обязан иметь src
+            // Контракт сохранения картинок:
             // - data:/blob: URL в image запрещены
+            // - image без src допускаются (legacy broken state), но логируются
             this._assertImageSaveContract(saveData);
             
             // Проверяем, изменились ли данные с последнего сохранения
@@ -240,7 +240,8 @@ export class SaveManager {
             const effectiveSrc = topSrc.trim() || propSrc.trim();
 
             if (!effectiveSrc) {
-                throw new Error(`Image object "${obj.id || 'unknown'}" has no src. Save is blocked.`);
+                console.warn(`Image object "${obj.id || 'unknown'}" has no src. Saving continues for legacy compatibility.`);
+                continue;
             }
             if (/^data:/i.test(topSrc) || /^blob:/i.test(topSrc) || /^data:/i.test(propSrc) || /^blob:/i.test(propSrc)) {
                 throw new Error(`Image object "${obj.id || 'unknown'}" contains forbidden data/blob src. Save is blocked.`);

package/src/core/index.js

@@ -535,9 +535,6 @@ export class CoreMoodBoard {
                 objectData.properties = { ...objectData.properties };
                 delete objectData.properties.src;
             }
-            if ('imageId' in objectData) {
-                delete objectData.imageId;
-            }
         }
         if (objectData.type === 'mindmap') {
             logMindmapCompoundDebug('core:load-object', {

package/src/moodboard/ActionHandler.js

@@ -11,6 +11,8 @@ export class ActionHandler {
      * Обрабатывает действия тулбара
      */
     handleToolbarAction(action) {
+        this._assertImageActionContract(action);
+
         switch (action.type) {
             case 'frame':
             case 'simple-text':
@@ -112,4 +114,19 @@ export class ActionHandler {
     exportBoard() {
         return this.handleExportBoard();
     }
+
+    _assertImageActionContract(action) {
+        if (!action || (action.type !== 'image' && action.type !== 'revit-screenshot-img')) return;
+        const src = typeof action?.properties?.src === 'string' ? action.properties.src.trim() : '';
+        if (src) return;
+
+        const reason = 'в цепочке создания отсутствует обязательное поле properties.src';
+        const message = `Загрузить картинку не получилось: ${reason}. Попробуйте еще раз.`;
+        if (this.workspaceManager?.showNotification) {
+            this.workspaceManager.showNotification(message);
+        } else if (typeof alert === 'function') {
+            alert(message);
+        }
+        throw new Error(`Image action contract violated: ${reason}`);
+    }
 }