@sequent-org/moodboard 1.4.4 → 1.4.6

package/src/core/commands/GroupMoveCommand.js

@@ -47,32 +47,7 @@ export class GroupMoveCommand extends BaseCommand {
     }
 
     undo() {
-        // Возвращаем исходные позиции
-        for (const item of this.moves) {
-            if (this.coordinatesAreTopLeft) {
-                // Координаты уже левый-верх (Frame перемещение)
-                this.core.updateObjectPositionDirect(item.id, item.from);
-                this.emit(Events.Object.TransformUpdated, {
-                    objectId: item.id,
-                    type: 'position',
-                    position: item.from
-                });
-            } else {
-                // Координаты - центры PIXI (обычное групповое перемещение)
-                const pixiObject = this.core?.pixi?.objects?.get(item.id);
-                if (pixiObject) {
-                    const halfW = (pixiObject.width || 0) / 2;
-                    const halfH = (pixiObject.height || 0) / 2;
-                    const topLeft = { x: item.from.x - halfW, y: item.from.y - halfH };
-                    this.core.updateObjectPositionDirect(item.id, topLeft);
-                    this.emit(Events.Object.TransformUpdated, {
-                        objectId: item.id,
-                        type: 'position',
-                        position: topLeft
-                    });
-                }
-            }
-        }
+        // Локальный undo отключен: история состояния загружается с сервера по версиям.
     }
 
     getDescription() {

package/src/core/commands/GroupReorderZCommand.js

@@ -22,8 +22,7 @@ export class GroupReorderZCommand extends BaseCommand {
     }
 
     undo() {
-        if (!this.beforeOrder) return;
-        this.applyOrder(this.beforeOrder);
+        // Локальный undo отключен: история состояния загружается с сервера по версиям.
     }
 
     applyOrder(idOrder) {

package/src/core/commands/GroupResizeCommand.js

@@ -23,10 +23,7 @@ export class GroupResizeCommand extends BaseCommand {
     }
 
     undo() {
-        for (const c of this.changes) {
-            this.core.updateObjectSizeAndPositionDirect(c.id, c.fromSize, c.fromPos, c.type || null);
-            this.emit(Events.Object.TransformUpdated, { objectId: c.id, type: 'resize', size: c.fromSize, position: c.fromPos });
-        }
+        // Локальный undo отключен: история состояния загружается с сервера по версиям.
     }
 
     getDescription() {

package/src/core/commands/GroupRotateCommand.js

@@ -28,15 +28,7 @@ export class GroupRotateCommand extends BaseCommand {
     }
 
     undo() {
-        for (const c of this.changes) {
-            if (this.core.pixi?.updateObjectRotation) {
-                this.core.pixi.updateObjectRotation(c.id, c.fromAngle);
-            }
-            this.core.updateObjectRotationDirect(c.id, c.fromAngle);
-            this.core.updateObjectPositionDirect(c.id, c.fromPos);
-            this.emit(Events.Object.TransformUpdated, { objectId: c.id, type: 'rotation', angle: c.fromAngle });
-            this.emit(Events.Object.TransformUpdated, { objectId: c.id, type: 'position', position: c.fromPos });
-        }
+        // Локальный undo отключен: история состояния загружается с сервера по версиям.
     }
 
     getDescription() {

package/src/core/commands/MindmapStatePatchCommand.js

@@ -29,7 +29,7 @@ export class MindmapStatePatchCommand extends BaseCommand {
     }
 
     undo() {
-        this._applyEntries(this.beforeEntries);
+        // Локальный undo отключен: история состояния загружается с сервера по версиям.
     }
 
     _applyEntries(entries) {

package/src/core/commands/MoveObjectCommand.js

@@ -23,9 +23,7 @@ export class MoveObjectCommand extends BaseCommand {
     }
 
     undo() {
-        // Возвращаем старую позицию
-        this._setPosition(this.oldPosition);
-
+        // Локальный undo отключен: история состояния загружается с сервера по версиям.
     }
 
     _setPosition(position) {

package/src/core/commands/PasteObjectCommand.js

@@ -96,16 +96,7 @@ export class PasteObjectCommand extends BaseCommand {
     }
 
     undo() {
-        if (this.newObjectId) {
-            // Удаляем созданный объект
-            this.coreMoodboard.state.removeObject(this.newObjectId);
-            this.coreMoodboard.pixi.removeObject(this.newObjectId);
-            
-            // Соответствующего константного события нет — остаёмся без эмита или используем Object.Deleted, если надо глобально
-            this.emit(Events.Object.Deleted, {
-                objectId: this.newObjectId
-            });
-        }
+        // Локальный undo отключен: история состояния загружается с сервера по версиям.
     }
 
     getDescription() {

package/src/core/commands/ReorderZCommand.js

@@ -18,7 +18,7 @@ export class ReorderZCommand extends BaseCommand {
     }
 
     undo() {
-        this.apply(this.toIndex, this.fromIndex);
+        // Локальный undo отключен: история состояния загружается с сервера по версиям.
     }
 
     apply(from, to) {

package/src/core/commands/ResizeObjectCommand.js

@@ -25,9 +25,7 @@ export class ResizeObjectCommand extends BaseCommand {
     }
 
     undo() {
-        // Возвращаем старый размер и позицию
-        this._setSizeAndPosition(this.oldSize, this.oldPosition);
-        this._updateResizeHandles();
+        // Локальный undo отключен: история состояния загружается с сервера по версиям.
     }
 
     _setSizeAndPosition(size, position = null) {

package/src/core/commands/RotateObjectCommand.js

@@ -25,15 +25,7 @@ export class RotateObjectCommand extends BaseCommand {
     }
 
     undo() {
-        // Обновляем угол поворота в состоянии
-        this._setRotation(this.oldAngle);
-        
-        // Возвращаем старый угол поворота
-        this.emit(Events.Object.Rotate, {
-            objectId: this.objectId,
-            angle: this.oldAngle
-        });
-        console.log(`↩️ Отменяем поворот объекта ${this.objectId}, возвращаем ${this.oldAngle}°`);
+        // Локальный undo отключен: история состояния загружается с сервера по версиям.
     }
 
     /**

package/src/core/commands/UpdateContentCommand.js

@@ -22,7 +22,7 @@ export class UpdateContentCommand extends BaseCommand {
     }
 
     undo() {
-        this._applyContent(this.oldContent, this.oldSize, this.oldPosition);
+        // Локальный undo отключен: история состояния загружается с сервера по версиям.
     }
 
     canMergeWith(otherCommand) {

package/src/core/commands/UpdateFramePropertiesCommand.js

@@ -34,7 +34,7 @@ export class UpdateFramePropertiesCommand extends BaseCommand {
     }
 
     undo() {
-        this._apply(this.oldValue);
+        // Локальный undo отключен: история состояния загружается с сервера по версиям.
     }
 
     canMergeWith(otherCommand) {

package/src/core/commands/UpdateFrameTypeCommand.js

@@ -32,7 +32,7 @@ export class UpdateFrameTypeCommand extends BaseCommand {
     }
 
     undo() {
-        this._apply(this.oldType, this.oldSize, this.oldPosition);
+        // Локальный undo отключен: история состояния загружается с сервера по версиям.
     }
 
     _apply(typeValue, size, position) {

package/src/core/commands/UpdateNoteStyleCommand.js

@@ -29,7 +29,7 @@ export class UpdateNoteStyleCommand extends BaseCommand {
     }
 
     undo() {
-        this._apply(this.oldValue);
+        // Локальный undo отключен: история состояния загружается с сервера по версиям.
     }
 
     canMergeWith(otherCommand) {

package/src/core/commands/UpdateTextStyleCommand.js

@@ -28,7 +28,7 @@ export class UpdateTextStyleCommand extends BaseCommand {
     }
 
     undo() {
-        this._apply(this.oldValue);
+        // Локальный undo отключен: история состояния загружается с сервера по версиям.
     }
 
     canMergeWith(otherCommand) {

package/src/core/events/Events.js

@@ -96,8 +96,6 @@ export const Events = {
 		Move: 'keyboard:move',
 		Copy: 'keyboard:copy',
 		Paste: 'keyboard:paste',
-		Undo: 'keyboard:undo',
-		Redo: 'keyboard:redo',
 	},
 
 	Object: {

package/src/core/flows/ClipboardFlow.js

@@ -19,6 +19,38 @@ export function setupClipboardFlow(core) {
         };
     };
 
+    const ensureServerImage = async ({ src, name, imageId }) => {
+        if (imageId) {
+            return { src, name, imageId };
+        }
+        if (!core.imageUploadService) {
+            alert('Сервис загрузки изображений недоступен. Изображение не добавлено.');
+            return null;
+        }
+        try {
+            let uploadResult = null;
+            if (typeof src === 'string' && /^data:image\//i.test(src)) {
+                uploadResult = await core.imageUploadService.uploadFromDataUrl(src, name || 'clipboard-image.png');
+            } else {
+                const response = await fetch(src);
+                if (!response.ok) {
+                    throw new Error(`HTTP ${response.status}: ${response.statusText}`);
+                }
+                const blob = await response.blob();
+                uploadResult = await core.imageUploadService.uploadImage(blob, name || 'clipboard-image');
+            }
+            return {
+                src: uploadResult.url,
+                name: uploadResult.name || name,
+                imageId: uploadResult.imageId || uploadResult.id
+            };
+        } catch (error) {
+            console.error('Ошибка загрузки вставленного изображения на сервер:', error);
+            alert('Ошибка загрузки изображения на сервер. Изображение не добавлено.');
+            return null;
+        }
+    };
+
     core.eventBus.on(Events.UI.CopyObject, ({ objectId }) => {
         if (!objectId) return;
         core.copyObject(objectId);
@@ -186,8 +218,10 @@ export function setupClipboardFlow(core) {
         core._cursor.y = y;
     });
 
-    core.eventBus.on(Events.UI.PasteImage, ({ src, name, imageId }) => {
+    core.eventBus.on(Events.UI.PasteImage, async ({ src, name, imageId }) => {
         if (!src) return;
+        const uploaded = await ensureServerImage({ src, name, imageId });
+        if (!uploaded?.imageId) return;
         const view = core.pixi.app.view;
         const world = core.pixi.worldLayer || core.pixi.app.stage;
         const s = world?.scale?.x || 1;
@@ -214,18 +248,18 @@ export function setupClipboardFlow(core) {
                 w = 300;
                 h = Math.max(1, Math.round(w / ar));
             }
-            const revitPayload = await resolveRevitImagePayload(src, {
+            const revitPayload = await resolveRevitImagePayload(uploaded.src, {
                 source: 'clipboard:paste-image',
-                name
+                name: uploaded.name
             });
             const properties = {
-                src,
-                name,
+                src: uploaded.src,
+                name: uploaded.name,
                 width: w,
                 height: h,
                 ...revitPayload.properties
             };
-            const extraData = imageId ? { imageId } : {};
+            const extraData = uploaded.imageId ? { imageId: uploaded.imageId } : {};
             core.createObject(
                 revitPayload.type,
                 { x: Math.round(worldX - Math.round(w / 2)), y: Math.round(worldY - Math.round(h / 2)) },
@@ -239,14 +273,16 @@ export function setupClipboardFlow(core) {
             img.decoding = 'async';
             img.onload = () => { void placeWithAspect(img.naturalWidth || 0, img.naturalHeight || 0); };
             img.onerror = () => { void placeWithAspect(0, 0); };
-            img.src = src;
+            img.src = uploaded.src;
         } catch (_) {
             void placeWithAspect(0, 0);
         }
     });
 
-    core.eventBus.on(Events.UI.PasteImageAt, ({ x, y, src, name, imageId }) => {
+    core.eventBus.on(Events.UI.PasteImageAt, async ({ x, y, src, name, imageId }) => {
         if (!src) return;
+        const uploaded = await ensureServerImage({ src, name, imageId });
+        if (!uploaded?.imageId) return;
         const world = core.pixi.worldLayer || core.pixi.app.stage;
         const s = world?.scale?.x || 1;
         const worldX = (x - (world?.x || 0)) / s;
@@ -260,18 +296,18 @@ export function setupClipboardFlow(core) {
                 w = 300;
                 h = Math.max(1, Math.round(w / ar));
             }
-            const revitPayload = await resolveRevitImagePayload(src, {
+            const revitPayload = await resolveRevitImagePayload(uploaded.src, {
                 source: 'clipboard:paste-image-at',
-                name
+                name: uploaded.name
             });
             const properties = {
-                src,
-                name,
+                src: uploaded.src,
+                name: uploaded.name,
                 width: w,
                 height: h,
                 ...revitPayload.properties
             };
-            const extraData = imageId ? { imageId } : {};
+            const extraData = uploaded.imageId ? { imageId: uploaded.imageId } : {};
             core.createObject(
                 revitPayload.type,
                 { x: Math.round(worldX - Math.round(w / 2)), y: Math.round(worldY - Math.round(h / 2)) },
@@ -285,7 +321,7 @@ export function setupClipboardFlow(core) {
             img.decoding = 'async';
             img.onload = () => { void placeWithAspect(img.naturalWidth || 0, img.naturalHeight || 0); };
             img.onerror = () => { void placeWithAspect(0, 0); };
-            img.src = src;
+            img.src = uploaded.src;
         } catch (_) {
             void placeWithAspect(0, 0);
         }

package/src/core/flows/SaveFlow.js

@@ -23,6 +23,11 @@ export function setupSaveFlow(core) {
     });
 
     core.eventBus.on(Events.Save.Success, async () => {
+        if (typeof core.revealPendingObjectsAfterSave === 'function') {
+            core.revealPendingObjectsAfterSave();
+        } else if (typeof core.revealPendingImageObjectsAfterSave === 'function') {
+            core.revealPendingImageObjectsAfterSave();
+        }
         // ВРЕМЕННО ОТКЛЮЧЕНО:
         // cleanup-фича требует доработки контракта и серверной поддержки.
         // Автоматический вызов удален, чтобы не запускать cleanup после сохранения.

package/src/core/index.js

@@ -66,6 +66,8 @@ export class CoreMoodBoard {
             csrfToken: this.options.csrfToken
         });
         this.gridSnapResolver = new GridSnapResolver(this);
+        // Объекты, требующие подтверждения сохранения (image/file), показываем только после save:success.
+        this._pendingPersistAckVisibilityIds = new Set();
         
         // Связываем SaveManager с ApiClient для правильной обработки изображений
         this.saveManager.setApiClient(this.apiClient);
@@ -435,9 +437,39 @@ export class CoreMoodBoard {
         const command = new CreateObjectCommand(this, objectData);
         this.history.executeCommand(command);
 
+        // Строгий UX-контракт: image/file появляются только после успешного сохранения.
+        if (this._isPersistAckRequiredType(type)) {
+            this._pendingPersistAckVisibilityIds.add(objectData.id);
+            this._setObjectVisibility(objectData.id, false);
+        }
+
         return objectData;
     }
 
+    _isPersistAckRequiredType(type) {
+        return type === 'image' || type === 'revit-screenshot-img' || type === 'file';
+    }
+
+    _setObjectVisibility(objectId, visible) {
+        const pixiObject = this.pixi?.objects?.get?.(objectId);
+        if (pixiObject) {
+            pixiObject.visible = !!visible;
+        }
+    }
+
+    revealPendingObjectsAfterSave() {
+        if (!this._pendingPersistAckVisibilityIds || this._pendingPersistAckVisibilityIds.size === 0) return;
+        for (const objectId of this._pendingPersistAckVisibilityIds) {
+            this._setObjectVisibility(objectId, true);
+        }
+        this._pendingPersistAckVisibilityIds.clear();
+    }
+
+    // Backward-compat alias for tests/integrations created in previous step.
+    revealPendingImageObjectsAfterSave() {
+        this.revealPendingObjectsAfterSave();
+    }
+
     // === Прикрепления к фреймам ===
     // Логика фреймов перенесена в FrameService
 

package/src/core/keyboard/KeyboardClipboardImagePaste.js

@@ -16,11 +16,11 @@ export class KeyboardClipboardImagePaste {
                     imageId: uploadResult.imageId || uploadResult.id
                 });
             } else {
-                this.eventBus.emit(Events.UI.PasteImage, { src: dataUrl, name: fileName });
+                alert('Сервис загрузки изображений недоступен. Изображение не добавлено.');
             }
         } catch (error) {
             console.error('Ошибка загрузки изображения:', error);
-            this.eventBus.emit(Events.UI.PasteImage, { src: dataUrl, name: fileName });
+            alert('Ошибка загрузки изображения на сервер. Изображение не добавлено.');
         }
     }
 
@@ -34,29 +34,11 @@ export class KeyboardClipboardImagePaste {
                     imageId: uploadResult.imageId || uploadResult.id
                 });
             } else {
-                const reader = new FileReader();
-                reader.onload = () => {
-                    this.eventBus.emit(Events.UI.PasteImage, {
-                        src: reader.result,
-                        name: fileName
-                    });
-                };
-                reader.readAsDataURL(file);
+                alert('Сервис загрузки изображений недоступен. Изображение не добавлено.');
             }
         } catch (error) {
             console.error('Ошибка загрузки файла изображения:', error);
-            try {
-                const reader = new FileReader();
-                reader.onload = () => {
-                    this.eventBus.emit(Events.UI.PasteImage, {
-                        src: reader.result,
-                        name: fileName
-                    });
-                };
-                reader.readAsDataURL(file);
-            } catch (fallbackError) {
-                console.error('Критическая ошибка при чтении файла:', fallbackError);
-            }
+            alert('Ошибка загрузки изображения на сервер. Изображение не добавлено.');
         }
     }
 
@@ -105,10 +87,7 @@ export class KeyboardClipboardImagePaste {
                                 const dataUrl = await this._blobToDataUrl(blob);
                                 this.handleImageUpload(dataUrl, srcInHtml.split('/').pop() || 'image');
                             } catch (_) {
-                                this.eventBus.emit(Events.UI.PasteImage, {
-                                    src: srcInHtml,
-                                    name: srcInHtml.split('/').pop() || 'image'
-                                });
+                                alert('Не удалось загрузить изображение из URL. Изображение не добавлено.');
                             }
                             return;
                         }
@@ -151,10 +130,7 @@ export class KeyboardClipboardImagePaste {
                             this.handleImageUpload(dataUrl, trimmed.split('/').pop() || 'image');
                             return;
                         } catch (_) {
-                            this.eventBus.emit(Events.UI.PasteImage, {
-                                src: trimmed,
-                                name: trimmed.split('/').pop() || 'image'
-                            });
+                            alert('Не удалось загрузить изображение из URL. Изображение не добавлено.');
                             return;
                         }
                     }

package/src/services/FileUploadService.js

@@ -4,8 +4,8 @@
 export class FileUploadService {
     constructor(apiClient, options = {}) {
         this.apiClient = apiClient;
-        this.uploadEndpoint = '/api/files/upload';
-        this.deleteEndpoint = '/api/files';
+        this.uploadEndpoint = '/api/v2/files/upload';
+        this.deleteEndpoint = '/api/v2/files';
         this.options = {
             csrfToken: null, // Можно передать токен напрямую
             csrfTokenSelector: 'meta[name="csrf-token"]', // Селектор для поиска токена в DOM

package/src/services/ImageUploadService.js

@@ -4,8 +4,8 @@
 export class ImageUploadService {
     constructor(apiClient, options = {}) {
         this.apiClient = apiClient;
-        this.uploadEndpoint = '/api/images/upload';
-        this.deleteEndpoint = '/api/images';
+        this.uploadEndpoint = '/api/v2/images/upload';
+        this.deleteEndpoint = '/api/v2/images';
         this.options = {
             csrfToken: null, // Можно передать токен напрямую
             csrfTokenSelector: 'meta[name="csrf-token"]', // Селектор для поиска токена в DOM

package/src/tools/manager/ToolEventRouter.js

@@ -339,28 +339,13 @@ export class ToolEventRouter {
                             index
                         };
                     } else {
-                        const localSrc = await new Promise((resolve) => {
-                            const reader = new FileReader();
-                            reader.onload = () => {
-                                resolve(reader.result);
-                            };
-                            reader.readAsDataURL(file);
-                        });
-                        if (!isCurrentDrop()) {
-                            logDropDebug(diagnostics, 'image_local_fallback_stale_drop_ignored', {
-                                fileName: file.name || 'image'
-                            });
-                            return null;
-                        }
-                        logDropDebug(diagnostics, 'image_local_fallback_success', {
-                            fileName: file.name || 'image'
-                        });
-                        return {
-                            src: localSrc,
-                            name: file.name || 'image',
-                            imageId: null,
-                            index
-                        };
+                        showDropWarning(
+                            manager,
+                            `Не удалось добавить "${file.name || 'image'}": сервис загрузки изображений недоступен`,
+                            diagnostics,
+                            { fileName: file.name || 'image' }
+                        );
+                        return null;
                     }
                 } catch (error) {
                     console.warn('Ошибка загрузки изображения через drag-and-drop:', error);
@@ -368,28 +353,16 @@ export class ToolEventRouter {
                         fileName: file.name || 'image',
                         message: error?.message || String(error)
                     });
-                    const fallbackSrc = await new Promise((resolve) => {
-                        const reader = new FileReader();
-                        reader.onload = () => {
-                            resolve(reader.result);
-                        };
-                        reader.readAsDataURL(file);
-                    });
-                    if (!isCurrentDrop()) {
-                        logDropDebug(diagnostics, 'image_error_fallback_stale_drop_ignored', {
-                            fileName: file.name || 'image'
-                        });
-                        return null;
-                    }
-                    logDropDebug(diagnostics, 'image_error_fallback_success', {
-                        fileName: file.name || 'image'
-                    });
-                    return {
-                        src: fallbackSrc,
-                        name: file.name || 'image',
-                        imageId: null,
-                        index
-                    };
+                    showDropWarning(
+                        manager,
+                        `Не удалось загрузить "${file.name || 'image'}" на сервер. Изображение не добавлено.`,
+                        diagnostics,
+                        {
+                            fileName: file.name || 'image',
+                            message: error?.message || String(error)
+                        }
+                    );
+                    return null;
                 }
             });
             for (const placement of imagePlacements) {
@@ -461,21 +434,13 @@ export class ToolEventRouter {
                             fileId: uploadResult.fileId || uploadResult.id || null
                         };
                     } else {
-                        if (!isCurrentDrop()) {
-                            logDropDebug(diagnostics, 'file_local_fallback_stale_drop_ignored', {
-                                fileName: fallbackProps.fileName
-                            });
-                            return null;
-                        }
-                        logDropDebug(diagnostics, 'file_local_fallback_success', {
-                            fileName: fallbackProps.fileName
-                        });
-                        return {
-                            type: 'file',
-                            id: 'file',
-                            position,
-                            properties: fallbackProps
-                        };
+                        showDropWarning(
+                            manager,
+                            `Не удалось добавить "${fallbackProps.fileName}": сервис загрузки файлов недоступен`,
+                            diagnostics,
+                            { fileName: fallbackProps.fileName }
+                        );
+                        return null;
                     }
                 } catch (error) {
                     console.warn('Ошибка загрузки файла через drag-and-drop:', error);
@@ -483,21 +448,16 @@ export class ToolEventRouter {
                         fileName: fallbackProps.fileName,
                         message: error?.message || String(error)
                     });
-                    if (!isCurrentDrop()) {
-                        logDropDebug(diagnostics, 'file_error_fallback_stale_drop_ignored', {
-                            fileName: fallbackProps.fileName
-                        });
-                        return null;
-                    }
-                    logDropDebug(diagnostics, 'file_error_fallback_success', {
-                        fileName: fallbackProps.fileName
-                    });
-                    return {
-                        type: 'file',
-                        id: 'file',
-                        position,
-                        properties: fallbackProps
-                    };
+                    showDropWarning(
+                        manager,
+                        `Не удалось загрузить "${fallbackProps.fileName}" на сервер. Файл не добавлен.`,
+                        diagnostics,
+                        {
+                            fileName: fallbackProps.fileName,
+                            message: error?.message || String(error)
+                        }
+                    );
+                    return null;
                 }
             });
             for (const actionPayload of filePlacements) {