@sequent-org/moodboard 1.4.4 → 1.4.6

package/package.json

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

package/src/core/ApiClient.js

@@ -144,90 +144,55 @@ export class ApiClient {
 
         const cleanedObjects = boardData.objects.map(obj => {
             if (obj.type === 'image') {
-                console.log('🧹 DEBUG _cleanImageData: обрабатываем изображение:', {
-                    id: obj.id,
-                    imageId: obj.imageId,
-                    hasSrc: !!obj.src,
-                    hasPropertiesSrc: !!obj.properties?.src,
-                    srcIsBase64: !!(obj.src && obj.src.startsWith('data:')),
-                    propertiesSrcIsBase64: !!(obj.properties?.src && obj.properties.src.startsWith('data:'))
-                });
-                
+                const imageId = typeof obj.imageId === 'string' ? obj.imageId.trim() : '';
+                const topSrc = typeof obj.src === 'string' ? obj.src : '';
+                const propSrc = typeof obj.properties?.src === 'string' ? obj.properties.src : '';
+                const hasForbiddenInlineSrc = /^data:/i.test(topSrc)
+                    || /^blob:/i.test(topSrc)
+                    || /^data:/i.test(propSrc)
+                    || /^blob:/i.test(propSrc);
+
+                // Жесткий контракт v2: сохраняем image только через server imageId.
+                if (!imageId) {
+                    throw new Error(`Image object "${obj.id || 'unknown'}" has no imageId. Save is blocked.`);
+                }
+                if (hasForbiddenInlineSrc) {
+                    throw new Error(`Image object "${obj.id || 'unknown'}" contains forbidden data/blob src. Save is blocked.`);
+                }
+
                 const cleanedObj = { ...obj };
-                
-                // Если есть imageId, убираем src для экономии места
-                if (obj.imageId && typeof obj.imageId === 'string' && obj.imageId.trim().length > 0) {
-                    console.log('🧹 DEBUG _cleanImageData: у изображения есть imageId, убираем src');
-                    
-                    // Убираем src с верхнего уровня
-                    if (cleanedObj.src) {
-                        delete cleanedObj.src;
-                        console.log('🧹 DEBUG: удален src с верхнего уровня');
-                    }
-                    
-                    // Убираем src из properties
-                    if (cleanedObj.properties?.src) {
-                        cleanedObj.properties = { ...cleanedObj.properties };
-                        delete cleanedObj.properties.src;
-                        console.log('🧹 DEBUG: удален src из properties');
-                    }
+
+                // imageId валиден — src можно безопасно убрать из history payload.
+                if (cleanedObj.src) {
+                    delete cleanedObj.src;
                 }
-                // Если нет imageId, предупреждаем о base64
-                else {
-                    console.log('🧹 DEBUG _cleanImageData: у изображения НЕТ imageId, оставляем src как есть');
-                    if (cleanedObj.properties?.src && cleanedObj.properties.src.startsWith('data:')) {
-                        console.warn('❌ Изображение сохраняется с base64 в properties, так как нет imageId:', cleanedObj.id);
-                    }
-                    if (cleanedObj.src && cleanedObj.src.startsWith('data:')) {
-                        console.warn('❌ Изображение сохраняется с base64 в src, так как нет imageId:', cleanedObj.id);
-                    }
-                    if (!obj.imageId) {
-                        console.warn('❌ У изображения отсутствует imageId:', cleanedObj.id);
-                    }
+                if (cleanedObj.properties?.src) {
+                    cleanedObj.properties = { ...cleanedObj.properties };
+                    delete cleanedObj.properties.src;
                 }
-                
+
                 return cleanedObj;
             }
             
             if (obj.type === 'file') {
-                console.log('🧹 DEBUG _cleanObjectData: обрабатываем файл:', {
-                    id: obj.id,
-                    fileId: obj.fileId,
-                    hasContent: !!obj.content,
-                    hasPropertiesContent: !!obj.properties?.content
-                });
-                
                 const cleanedObj = { ...obj };
                 
                 // Если есть fileId, убираем content для экономии места
                 if (obj.fileId && typeof obj.fileId === 'string' && obj.fileId.trim().length > 0) {
-                    console.log('🧹 DEBUG _cleanObjectData: у файла есть fileId, убираем content');
-                    
                     // Убираем content с верхнего уровня
                     if (cleanedObj.content) {
                         delete cleanedObj.content;
-                        console.log('🧹 DEBUG: удален content с верхнего уровня');
                     }
                     
                     // Убираем content из properties
                     if (cleanedObj.properties?.content) {
                         cleanedObj.properties = { ...cleanedObj.properties };
                         delete cleanedObj.properties.content;
-                        console.log('🧹 DEBUG: удален content из properties');
                     }
                 }
                 // Если нет fileId, предупреждаем о наличии content
                 else {
-                    console.log('🧹 DEBUG _cleanObjectData: у файла НЕТ fileId, оставляем content как есть');
-                    if (cleanedObj.properties?.content) {
-                        console.warn('❌ Файл сохраняется с content в properties, так как нет fileId:', cleanedObj.id);
-                    }
-                    if (cleanedObj.content) {
-                        console.warn('❌ Файл сохраняется с content, так как нет fileId:', cleanedObj.id);
-                    }
-                    if (!obj.fileId) {
-                        console.warn('❌ У файла отсутствует fileId:', cleanedObj.id);
-                    }
+                    // Для файлов сейчас сохраняем поведение: без fileId не модифицируем объект.
                 }
                 
                 return cleanedObj;
@@ -253,18 +218,10 @@ export class ApiClient {
         const restoredObjects = await Promise.all(
             boardData.objects.map(async (obj) => {
                 if (obj.type === 'image') {
-                    console.log('🔗 DEBUG restoreImageUrls: обрабатываем изображение:', {
-                        id: obj.id,
-                        imageId: obj.imageId,
-                        hasSrc: !!obj.src,
-                        hasPropertiesSrc: !!obj.properties?.src
-                    });
-                    
                     if (obj.imageId && (!obj.src && !obj.properties?.src)) {
-                        console.log('🔗 DEBUG: восстанавливаем URL для изображения');
                         try {
                             // Формируем URL изображения
-                            const imageUrl = `/api/images/${obj.imageId}/file`;
+                            const imageUrl = `/api/v2/images/${obj.imageId}/download`;
                             
                             return {
                                 ...obj,
@@ -278,25 +235,15 @@ export class ApiClient {
                             console.warn(`Не удалось восстановить URL для изображения ${obj.imageId}:`, error);
                             return obj;
                         }
-                    } else {
-                        console.log('🔗 DEBUG: изображение уже имеет URL или нет imageId, оставляем как есть');
-                        return obj;
                     }
+                    return obj;
                 }
                 
                 if (obj.type === 'file') {
-                    console.log('🔗 DEBUG restoreObjectUrls: обрабатываем файл:', {
-                        id: obj.id,
-                        fileId: obj.fileId,
-                        hasUrl: !!obj.url,
-                        hasPropertiesUrl: !!obj.properties?.url
-                    });
-                    
                     if (obj.fileId) {
-                        console.log('🔗 DEBUG: восстанавливаем данные для файла');
                         try {
                             // Формируем URL файла для скачивания
-                            const fileUrl = `/api/files/${obj.fileId}/download`;
+                            const fileUrl = `/api/v2/files/${obj.fileId}/download`;
                             
                             // Создаем обновленный объект с восстановленными данными
                             const restoredObj = {
@@ -312,7 +259,7 @@ export class ApiClient {
                             // (Это будет выполнено асинхронно, чтобы не блокировать загрузку)
                             setTimeout(async () => {
                                 try {
-                                    const response = await fetch(`/api/files/${obj.fileId}`, {
+                                    const response = await fetch(`/api/v2/files/${obj.fileId}`, {
                                         headers: {
                                             'Accept': 'application/json',
                                             'X-Requested-With': 'XMLHttpRequest'
@@ -323,7 +270,6 @@ export class ApiClient {
                                     if (response.ok) {
                                         const result = await response.json();
                                         if (result.success && result.data) {
-                                            console.log('🔄 Обновляем метаданные файла с сервера:', result.data);
                                             // Эмитим событие для обновления метаданных файла в состоянии
                                             // (это будет обработано в core, если EventBus доступен)
                                             if (typeof window !== 'undefined' && window.moodboardEventBus) {
@@ -345,10 +291,8 @@ export class ApiClient {
                             console.warn(`Не удалось восстановить данные для файла ${obj.fileId}:`, error);
                             return obj;
                         }
-                    } else {
-                        console.log('🔗 DEBUG: файл не имеет fileId, оставляем как есть');
-                        return obj;
                     }
+                    return obj;
                 }
                 
                 return obj;

package/src/core/HistoryManager.js

@@ -15,8 +15,6 @@ export class HistoryManager {
         this.history = [];
         // Текущая позиция в истории
         this.currentIndex = -1;
-        // Флаг для предотвращения зацикливания при undo/redo
-        this.isExecutingCommand = false;
         this._listenersAttached = false;
         this._onDebug = () => this.debugHistory();
 
@@ -34,14 +32,6 @@ export class HistoryManager {
      * Выполнить команду и добавить в историю
      */
     executeCommand(command) {
-        if (this.isExecutingCommand) {
-            // Если мы в процессе undo/redo, не добавляем в историю
-            this._executeCommandSafely(command);
-            return;
-        }
-
-
-
         // Проверяем, можно ли объединить с последней командой
         const lastCommand = this.getLastCommand();
         if (lastCommand &&
@@ -51,8 +41,6 @@ export class HistoryManager {
             lastCommand.mergeWith(command);
             this._executeCommandSafely(lastCommand);
             this.eventBus.emit('history:changed', {
-                canUndo: this.canUndo(),
-                canRedo: this.canRedo(),
                 historySize: this.history.length
             });
             return;
@@ -78,8 +66,6 @@ export class HistoryManager {
 
         // Уведомляем об изменении истории
         this.eventBus.emit(Events.History.Changed, {
-            canUndo: this.canUndo(),
-            canRedo: this.canRedo(),
             historySize: this.history.length,
             currentCommand: command.toString()
         });
@@ -105,89 +91,6 @@ export class HistoryManager {
         }
     }
 
-    /**
-     * Отменить последнюю команду
-     */
-    undo() {
-        if (!this.canUndo()) {
-
-            return false;
-        }
-
-        const command = this.history[this.currentIndex];
-
-
-        this.isExecutingCommand = true;
-        try {
-            command.undo();
-            this.currentIndex--;
-            
-            this.eventBus.emit(Events.History.Changed, {
-                canUndo: this.canUndo(),
-                canRedo: this.canRedo(),
-                historySize: this.history.length,
-                lastUndone: command.toString()
-            });
-
-
-            return true;
-        } catch (error) {
-            console.error('❌ Ошибка при отмене команды:', error);
-            return false;
-        } finally {
-            this.isExecutingCommand = false;
-        }
-    }
-
-    /**
-     * Повторить отмененную команду
-     */
-    redo() {
-        if (!this.canRedo()) {
-
-            return false;
-        }
-
-        this.currentIndex++;
-        const command = this.history[this.currentIndex];
-
-
-        this.isExecutingCommand = true;
-        try {
-            this._executeCommandSafely(command);
-            
-            this.eventBus.emit(Events.History.Changed, {
-                canUndo: this.canUndo(),
-                canRedo: this.canRedo(),
-                historySize: this.history.length,
-                lastRedone: command.toString()
-            });
-
-
-            return true;
-        } catch (error) {
-            console.error('❌ Ошибка при повторе команды:', error);
-            this.currentIndex--; // Откатываем индекс при ошибке
-            return false;
-        } finally {
-            this.isExecutingCommand = false;
-        }
-    }
-
-    /**
-     * Можно ли отменить команду
-     */
-    canUndo() {
-        return this.currentIndex >= 0;
-    }
-
-    /**
-     * Можно ли повторить команду
-     */
-    canRedo() {
-        return this.currentIndex < this.history.length - 1;
-    }
-
     /**
      * Получить последнюю команду
      */
@@ -204,8 +107,6 @@ export class HistoryManager {
         this.currentIndex = -1;
         
         this.eventBus.emit(Events.History.Changed, {
-            canUndo: false,
-            canRedo: false,
             historySize: 0
         });
 
@@ -219,8 +120,6 @@ export class HistoryManager {
         return {
             totalCommands: this.history.length,
             currentIndex: this.currentIndex,
-            canUndo: this.canUndo(),
-            canRedo: this.canRedo(),
             commands: this.history.map((cmd, index) => ({
                 index,
                 isCurrent: index === this.currentIndex,
@@ -238,7 +137,6 @@ export class HistoryManager {
         console.group('📚 История команд');
         console.table(info.commands);
         console.log(`Позиция: ${this.currentIndex + 1}/${this.history.length}`);
-        console.log(`Undo: ${this.canUndo()}, Redo: ${this.canRedo()}`);
         console.groupEnd();
     }
 

package/src/core/SaveManager.js

@@ -103,6 +103,11 @@ export class SaveManager {
                     sample: mindmapNodes.slice(0, 5),
                 });
             }
+
+            // Жесткий контракт для сохранения картинок:
+            // - каждый image обязан иметь imageId
+            // - data:/blob: URL в image запрещены
+            this._assertImageSaveContract(saveData);
             
             // Проверяем, изменились ли данные с последнего сохранения
             if (this.lastSavedData && JSON.stringify(saveData) === JSON.stringify(this.lastSavedData)) {
@@ -221,6 +226,28 @@ export class SaveManager {
         return await response.json();
     }
 
+    _assertImageSaveContract(saveData) {
+        const objects = Array.isArray(saveData?.boardData?.objects)
+            ? saveData.boardData.objects
+            : Array.isArray(saveData?.objects)
+                ? saveData.objects
+                : [];
+
+        for (const obj of objects) {
+            if (!obj || obj.type !== 'image') continue;
+            const imageId = typeof obj.imageId === 'string' ? obj.imageId.trim() : '';
+            const topSrc = typeof obj.src === 'string' ? obj.src : '';
+            const propSrc = typeof obj.properties?.src === 'string' ? obj.properties.src : '';
+
+            if (!imageId) {
+                throw new Error(`Image object "${obj.id || 'unknown'}" has no imageId. Save is blocked.`);
+            }
+            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.`);
+            }
+        }
+    }
+
     _buildSavePayload(boardId, data, csrfToken = undefined) {
         return {
             moodboardId: boardId,

package/src/core/commands/CreateObjectCommand.js

@@ -33,14 +33,6 @@ export class CreateObjectCommand extends BaseCommand {
     }
 
     undo() {
-        // Удаляем объект из состояния и PIXI
-        this.coreMoodboard.state.removeObject(this.objectData.id);
-        this.coreMoodboard.pixi.removeObject(this.objectData.id);
-        
-
-        
-        this.coreMoodboard.eventBus.emit(Events.Object.Deleted, { 
-            objectId: this.objectData.id 
-        });
+        // Локальный undo отключен: история состояния загружается с сервера по версиям.
     }
 }

package/src/core/commands/DeleteObjectCommand.js

@@ -9,116 +9,47 @@ export class DeleteObjectCommand extends BaseCommand {
         super('delete_object', `Удалить объект`);
         this.coreMoodboard = coreMoodboard;
         this.objectId = objectId;
-        
-        // Сохраняем данные объекта для возможности восстановления
-        const objects = this.coreMoodboard.state.getObjects();
-        const originalData = objects.find(obj => obj.id === objectId);
-        
-        if (!originalData) {
-            throw new Error(`Object with id ${objectId} not found`);
-        }
 
-        // Делаем глубокую копию данных объекта
-        this.objectData = JSON.parse(JSON.stringify(originalData));
-        
-        // Для изображений убедимся, что есть src URL для восстановления
-        if (this.objectData.type === 'image') {
-            
-            if (this.objectData.imageId) {
-                const imageUrl = `/api/images/${this.objectData.imageId}/file`;
-                
-                // Всегда восстанавливаем URL из imageId для гарантии
-                // (может быть удален при предыдущих сохранениях)
-                this.objectData.src = imageUrl;
-                
-                if (!this.objectData.properties) {
-                    this.objectData.properties = {};
-                }
-                this.objectData.properties.src = imageUrl;
-                
-            } else {
-            }
-        }
-        
-        // Для файлов сохраняем информацию для возможной очистки с сервера
-        if (this.objectData.type === 'file') {
-            
-            if (this.objectData.fileId) {
-                // Сохраняем fileId для удаления с сервера
-                this.fileIdToDelete = this.objectData.fileId;
-            }
+        const currentObject = this._getCurrentObject();
+        if (!currentObject) {
+            throw new Error(`Object with id ${objectId} not found`);
         }
-        
-        // Обновляем описание с типом объекта
-        this.description = `Удалить ${this.objectData.type}`;
+        this.description = `Удалить ${currentObject.type}`;
     }
 
     async execute() {
         console.log('🗑️ DeleteObjectCommand: начинаем удаление объекта:', this.objectId);
-        
+
+        const currentObject = this._getCurrentObject();
+        const blobSrc = currentObject?.properties?.src || currentObject?.src;
+
         // Удаляем объект из состояния и PIXI
         this.coreMoodboard.state.removeObject(this.objectId);
         this.coreMoodboard.pixi.removeObject(this.objectId);
-        
+
         console.log('🗑️ DeleteObjectCommand: объект удален из state и PIXI');
 
         // Освобождаем blob URL у изображений (утечка памяти при fallback без upload)
-        const blobSrc = this.objectData?.properties?.src || this.objectData?.src;
         if (typeof blobSrc === 'string' && blobSrc.startsWith('blob:')) {
             try {
                 URL.revokeObjectURL(blobSrc);
             } catch (_) {}
         }
-        
-        // Если это файловый объект с fileId, удаляем файл с сервера
-        if (this.fileIdToDelete && this.coreMoodboard.fileUploadService) {
-            try {
-                console.log('🗑️ Удаляем файл с сервера:', this.fileIdToDelete);
-                await this.coreMoodboard.fileUploadService.deleteFile(this.fileIdToDelete);
-                console.log('✅ Файл успешно удален с сервера:', this.fileIdToDelete);
-            } catch (error) {
-                console.warn('⚠️ Ошибка удаления файла с сервера:', error);
-                // Не останавливаем выполнение команды, так как объект уже удален из UI
-            }
-        }
-        
+
         // Эмитим событие удаления для обновления всех UI компонентов
-        this.coreMoodboard.eventBus.emit(Events.Object.Deleted, { 
-            objectId: this.objectId 
+        this.coreMoodboard.eventBus.emit(Events.Object.Deleted, {
+            objectId: this.objectId
         });
-        
+
         console.log('✅ DeleteObjectCommand: событие Events.Object.Deleted отправлено');
     }
 
     undo() {
-        
-        // Специальная обработка для файловых объектов
-        if (this.objectData.type === 'file' && this.fileIdToDelete) {
-            
-            // Файл был удален с сервера, создаем объект с предупреждением
-            const restoredObjectData = { ...this.objectData };
-            if (restoredObjectData.properties) {
-                restoredObjectData.properties = {
-                    ...restoredObjectData.properties,
-                    fileName: `[УДАЛЕН] ${restoredObjectData.properties.fileName || 'файл'}`,
-                    isDeleted: true // Флаг для FileObject чтобы показать другую иконку
-                };
-            }
-            
-            // Восстанавливаем объект с измененными данными
-            this.coreMoodboard.state.addObject(restoredObjectData);
-            this.coreMoodboard.pixi.createObject(restoredObjectData);
-            
-            console.warn('⚠️ Файл восстановлен на холсте, но был удален с сервера');
-        } else {
-            // Восстанавливаем объект с сохраненными данными (для всех остальных типов)
-            this.coreMoodboard.state.addObject(this.objectData);
-            this.coreMoodboard.pixi.createObject(this.objectData);
-        }
-        
-        this.coreMoodboard.eventBus.emit(Events.Object.Created, { 
-            objectId: this.objectId, 
-            objectData: this.objectData 
-        });
+        // Локальный undo-restore отключен: история состояния загружается с сервера по версиям.
+    }
+
+    _getCurrentObject() {
+        const objects = this.coreMoodboard.state.getObjects();
+        return objects.find(obj => obj.id === this.objectId);
     }
 }

package/src/core/commands/EditFileNameCommand.js

@@ -22,8 +22,7 @@ export class EditFileNameCommand extends BaseCommand {
     }
 
     async undo() {
-        // Возвращаем старое название
-        await this._setFileName(this.oldName);
+        // Локальный undo отключен: история состояния загружается с сервера по версиям.
     }
 
     /**

package/src/core/commands/GroupDeleteCommand.js

@@ -10,66 +10,33 @@ export class GroupDeleteCommand extends BaseCommand {
         super('group_delete', `Удалить группу (${objectIds.length} объектов)`);
         this.coreMoodboard = coreMoodboard;
         this.objectIds = Array.isArray(objectIds) ? [...objectIds] : [];
-
-        const objects = this.coreMoodboard.state.getObjects();
-        this.objectsData = [];
-        for (const id of this.objectIds) {
-            const obj = objects.find((o) => o.id === id);
-            if (obj) {
-                const data = JSON.parse(JSON.stringify(obj));
-                if (data.type === 'image') {
-                    if (data.imageId) {
-                        const imageUrl = `/api/images/${data.imageId}/file`;
-                        data.src = imageUrl;
-                        if (!data.properties) data.properties = {};
-                        data.properties.src = imageUrl;
-                    }
-                }
-                this.objectsData.push({ id, data });
-            }
-        }
     }
 
     async execute() {
-        for (const { id, data } of this.objectsData) {
+        for (const id of this.objectIds) {
+            const obj = this._getObjectById(id);
+            if (!obj) continue;
+
+            const blobSrc = obj?.properties?.src || obj?.src;
             this.coreMoodboard.state.removeObject(id);
             this.coreMoodboard.pixi.removeObject(id);
 
-            const blobSrc = data?.properties?.src || data?.src;
             if (typeof blobSrc === 'string' && blobSrc.startsWith('blob:')) {
                 try {
                     URL.revokeObjectURL(blobSrc);
                 } catch (_) {}
             }
 
-            if (data.type === 'file' && data.fileId && this.coreMoodboard.fileUploadService) {
-                try {
-                    await this.coreMoodboard.fileUploadService.deleteFile(data.fileId);
-                } catch (_) {}
-            }
-
             this.coreMoodboard.eventBus.emit(Events.Object.Deleted, { objectId: id });
         }
     }
 
     undo() {
-        for (const { id, data } of this.objectsData) {
-            if (data.type === 'file' && data.fileId) {
-                const restored = { ...data };
-                if (restored.properties) {
-                    restored.properties = {
-                        ...restored.properties,
-                        fileName: `[УДАЛЕН] ${restored.properties.fileName || 'файл'}`,
-                        isDeleted: true,
-                    };
-                }
-                this.coreMoodboard.state.addObject(restored);
-                this.coreMoodboard.pixi.createObject(restored);
-            } else {
-                this.coreMoodboard.state.addObject(data);
-                this.coreMoodboard.pixi.createObject(data);
-            }
-            this.coreMoodboard.eventBus.emit(Events.Object.Created, { objectId: id, objectData: data });
-        }
+        // Локальный undo-restore отключен: история состояния загружается с сервера по версиям.
+    }
+
+    _getObjectById(id) {
+        const objects = this.coreMoodboard.state.getObjects();
+        return objects.find((obj) => obj.id === id);
     }
 }