@sequent-org/moodboard 1.4.11 → 1.4.12

package/package.json

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

package/src/core/ApiClient.js

@@ -70,14 +70,24 @@ export class ApiClient {
             const imageObjectsWithoutSrc = imageObjects
                 .filter((obj) => !(typeof obj?.src === 'string' && obj.src.trim().length > 0))
                 .map((obj) => obj?.id || 'unknown');
+            const fileObjects = objects.filter((obj) => obj?.type === 'file');
+            const fileObjectsWithSrc = fileObjects.filter((obj) => typeof obj?.src === 'string' && obj.src.trim().length > 0);
+            const fileObjectsWithoutSrc = fileObjects
+                .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
+                imageObjectsWithSrc: imageObjectsWithSrc.length,
+                fileObjects: fileObjects.length,
+                fileObjectsWithSrc: fileObjectsWithSrc.length
             });
             if (imageObjectsWithoutSrc.length > 0) {
                 console.warn('history/save warning: image objects without src (kept as broken placeholders):', imageObjectsWithoutSrc);
             }
+            if (fileObjectsWithoutSrc.length > 0) {
+                console.warn('history/save warning: file objects without src:', fileObjectsWithoutSrc);
+            }
 
             const csrfToken = document.querySelector('meta[name="csrf-token"]')?.getAttribute('content');
 
@@ -184,26 +194,35 @@ export class ApiClient {
             }
             
             if (obj.type === 'file') {
-                const cleanedObj = { ...obj };
-                
-                // Если есть fileId, убираем content для экономии места
-                if (obj.fileId && typeof obj.fileId === 'string' && obj.fileId.trim().length > 0) {
-                    // Убираем content с верхнего уровня
-                    if (cleanedObj.content) {
-                        delete cleanedObj.content;
-                    }
-                    
-                    // Убираем content из properties
-                    if (cleanedObj.properties?.content) {
-                        cleanedObj.properties = { ...cleanedObj.properties };
-                        delete cleanedObj.properties.content;
-                    }
+                const topSrcRaw = typeof obj.src === 'string' ? obj.src : '';
+                const propSrcRaw = typeof obj.properties?.src === 'string' ? obj.properties.src : '';
+                const topUrlRaw = typeof obj.url === 'string' ? obj.url : '';
+                const propUrlRaw = typeof obj.properties?.url === 'string' ? obj.properties.url : '';
+                const normalizedSrc = topSrcRaw.trim() || propSrcRaw.trim() || topUrlRaw.trim() || propUrlRaw.trim();
+
+                if (!normalizedSrc) {
+                    throw new Error(`File object "${obj.id || 'unknown'}" has no src. Save is blocked.`);
                 }
-                // Если нет fileId, предупреждаем о наличии content
-                else {
-                    // Для файлов сейчас сохраняем поведение: без fileId не модифицируем объект.
+                if (/^data:/i.test(normalizedSrc) || /^blob:/i.test(normalizedSrc)) {
+                    throw new Error(`File object "${obj.id || 'unknown'}" contains forbidden data/blob src. Save is blocked.`);
+                }
+                if (/^\/api\/v2\/files\//i.test(normalizedSrc)) {
+                    throw new Error(`File object "${obj.id || 'unknown'}" has legacy id-based src URL. Save is blocked.`);
+                }
+
+                const cleanedObj = {
+                    ...obj,
+                    src: normalizedSrc
+                };
+                if (cleanedObj.url) delete cleanedObj.url;
+                if (cleanedObj.content) delete cleanedObj.content;
+                if (cleanedObj.fileId) delete cleanedObj.fileId;
+                if (cleanedObj.properties?.src || cleanedObj.properties?.url || cleanedObj.properties?.content) {
+                    cleanedObj.properties = { ...cleanedObj.properties };
+                    delete cleanedObj.properties.src;
+                    delete cleanedObj.properties.url;
+                    delete cleanedObj.properties.content;
                 }
-                
                 return cleanedObj;
             }
             
@@ -243,59 +262,23 @@ export class ApiClient {
                 }
                 
                 if (obj.type === 'file') {
-                    if (obj.fileId) {
-                        try {
-                            // Формируем URL файла для скачивания
-                            const fileUrl = `/api/v2/files/${obj.fileId}/download`;
-                            
-                            // Создаем обновленный объект с восстановленными данными
-                            const restoredObj = {
-                                ...obj,
-                                url: fileUrl,
-                                properties: {
-                                    ...obj.properties,
-                                    url: fileUrl
-                                }
-                            };
-                            
-                            // Пытаемся восстановить актуальные метаданные файла с сервера
-                            // (Это будет выполнено асинхронно, чтобы не блокировать загрузку)
-                            setTimeout(async () => {
-                                try {
-                                    const response = await fetch(`/api/v2/files/${obj.fileId}`, {
-                                        headers: {
-                                            'Accept': 'application/json',
-                                            'X-Requested-With': 'XMLHttpRequest'
-                                        },
-                                        credentials: 'same-origin'
-                                    });
-                                    
-                                    if (response.ok) {
-                                        const result = await response.json();
-                                        if (result.success && result.data) {
-                                            // Эмитим событие для обновления метаданных файла в состоянии
-                                            // (это будет обработано в core, если EventBus доступен)
-                                            if (typeof window !== 'undefined' && window.moodboardEventBus) {
-                                                window.moodboardEventBus.emit('file:metadata:updated', {
-                                                    objectId: obj.id,
-                                                    fileId: obj.fileId,
-                                                    metadata: result.data
-                                                });
-                                            }
-                                        }
-                                    }
-                                } catch (error) {
-                                    console.warn(`Не удалось обновить метаданные файла ${obj.fileId}:`, error);
-                                }
-                            }, 100);
-                            
-                            return restoredObj;
-                        } catch (error) {
-                            console.warn(`Не удалось восстановить данные для файла ${obj.fileId}:`, error);
-                            return obj;
-                        }
+                    const topSrc = typeof obj.src === 'string' ? obj.src.trim() : '';
+                    const propSrc = typeof obj.properties?.src === 'string' ? obj.properties.src.trim() : '';
+                    const topUrl = typeof obj.url === 'string' ? obj.url.trim() : '';
+                    const propUrl = typeof obj.properties?.url === 'string' ? obj.properties.url.trim() : '';
+                    const normalizedSrc = topSrc || propSrc || topUrl || propUrl;
+                    const restoredObj = { ...obj };
+                    if (normalizedSrc) {
+                        restoredObj.src = normalizedSrc;
                     }
-                    return obj;
+                    if (restoredObj.url) delete restoredObj.url;
+                    if (restoredObj.fileId) delete restoredObj.fileId;
+                    if (restoredObj.properties?.src || restoredObj.properties?.url) {
+                        restoredObj.properties = { ...restoredObj.properties };
+                        delete restoredObj.properties.src;
+                        delete restoredObj.properties.url;
+                    }
+                    return restoredObj;
                 }
                 
                 return obj;

package/src/core/SaveManager.js

@@ -108,6 +108,10 @@ export class SaveManager {
             // - data:/blob: URL в image запрещены
             // - image без src допускаются (legacy broken state), но логируются
             this._assertImageSaveContract(saveData);
+            // Контракт сохранения файлов:
+            // - file обязан иметь src
+            // - data:/blob: URL и id-based /api/v2/files/* для file запрещены
+            this._assertFileSaveContract(saveData);
             
             // Проверяем, изменились ли данные с последнего сохранения
             if (this.lastSavedData && JSON.stringify(saveData) === JSON.stringify(this.lastSavedData)) {
@@ -252,6 +256,33 @@ export class SaveManager {
         }
     }
 
+    _assertFileSaveContract(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 !== 'file') continue;
+            const topSrc = typeof obj.src === 'string' ? obj.src : '';
+            const propSrc = typeof obj.properties?.src === 'string' ? obj.properties.src : '';
+            const topUrl = typeof obj.url === 'string' ? obj.url : '';
+            const propUrl = typeof obj.properties?.url === 'string' ? obj.properties.url : '';
+            const effectiveSrc = topSrc.trim() || propSrc.trim() || topUrl.trim() || propUrl.trim();
+
+            if (!effectiveSrc) {
+                throw new Error(`File object "${obj.id || 'unknown'}" has no src. Save is blocked.`);
+            }
+            if (/^data:/i.test(effectiveSrc) || /^blob:/i.test(effectiveSrc)) {
+                throw new Error(`File object "${obj.id || 'unknown'}" contains forbidden data/blob src. Save is blocked.`);
+            }
+            if (/^\/api\/v2\/files\//i.test(effectiveSrc)) {
+                throw new Error(`File object "${obj.id || 'unknown'}" has legacy id-based src URL. Save is blocked.`);
+            }
+        }
+    }
+
     _buildSavePayload(boardId, data, csrfToken = undefined) {
         return {
             moodboardId: boardId,

package/src/core/commands/EditFileNameCommand.js

@@ -64,19 +64,6 @@ export class EditFileNameCommand extends BaseCommand {
             }
             objectData.properties.fileName = fileName;
             
-            // Синхронизируем с сервером, если есть fileId
-            if (objectData.fileId && this.coreMoodboard.fileUploadService) {
-                try {
-                    await this.coreMoodboard.fileUploadService.updateFileMetadata(objectData.fileId, { 
-                        fileName: fileName 
-                    });
-                    console.log('✅ Название файла успешно обновлено на сервере');
-                } catch (error) {
-                    console.warn('⚠️ Ошибка синхронизации названия файла с сервером:', error);
-                    // Не останавливаем выполнение, продолжаем с локальным обновлением
-                }
-            }
-            
             // Обновляем состояние
             this.coreMoodboard.state.markDirty();
             

package/src/core/index.js

@@ -418,11 +418,17 @@ export class CoreMoodBoard {
             },
             ...extraData
         };
-        if ((type === 'image' || type === 'revit-screenshot-img') && typeof properties?.src === 'string') {
-            objectData.src = properties.src;
-            if (objectData.properties && objectData.properties.src) {
+        if (type === 'image' || type === 'revit-screenshot-img' || type === 'file') {
+            const propSrc = typeof properties?.src === 'string' ? properties.src.trim() : '';
+            const propUrl = typeof properties?.url === 'string' ? properties.url.trim() : '';
+            const normalizedSrc = propSrc || propUrl;
+            if (normalizedSrc) {
+                objectData.src = normalizedSrc;
+            }
+            if (objectData.properties && (objectData.properties.src || objectData.properties.url)) {
                 objectData.properties = { ...objectData.properties };
                 delete objectData.properties.src;
+                delete objectData.properties.url;
             }
         }
         objectData.properties = normalizeMindmapPropertiesForCreate({
@@ -524,17 +530,21 @@ export class CoreMoodBoard {
             properties: objectData.properties || {},
             existingObjects: this.state?.state?.objects || [],
         });
-        if (objectData.type === 'image' || objectData.type === 'revit-screenshot-img') {
+        if (objectData.type === 'image' || objectData.type === 'revit-screenshot-img' || objectData.type === 'file') {
             const topSrc = typeof objectData.src === 'string' ? objectData.src.trim() : '';
             const propSrc = typeof objectData.properties?.src === 'string' ? objectData.properties.src.trim() : '';
-            const normalizedSrc = topSrc || propSrc;
+            const topUrl = typeof objectData.url === 'string' ? objectData.url.trim() : '';
+            const propUrl = typeof objectData.properties?.url === 'string' ? objectData.properties.url.trim() : '';
+            const normalizedSrc = topSrc || propSrc || topUrl || propUrl;
             if (normalizedSrc) {
                 objectData.src = normalizedSrc;
             }
-            if (objectData.properties?.src) {
+            if (objectData.properties?.src || objectData.properties?.url) {
                 objectData.properties = { ...objectData.properties };
                 delete objectData.properties.src;
+                delete objectData.properties.url;
             }
+            if (objectData.url) delete objectData.url;
         }
         if (objectData.type === 'mindmap') {
             logMindmapCompoundDebug('core:load-object', {

package/src/moodboard/ActionHandler.js

@@ -26,9 +26,8 @@ export class ActionHandler {
             case 'comment':
             case 'file':
             case 'mindmap':
-                // Для изображений используем только src; для файлов сохраняем fileId.
-                const extraData = action.fileId ? { fileId: action.fileId } : {};
-                return this.handleCreateObject(action.type, action.position, action.properties || {}, extraData);
+                // Для изображений и файлов используем только src.
+                return this.handleCreateObject(action.type, action.position, action.properties || {}, {});
 
             case 'delete-object':
                 if (action.id) {

package/src/moodboard/integration/MoodBoardLoadApi.js

@@ -89,11 +89,21 @@ export async function loadExistingBoard(board, version = null, options = {}) {
             const loadedObjects = Array.isArray(normalizedData.objects) ? normalizedData.objects : [];
             const imageObjects = loadedObjects.filter((obj) => obj?.type === 'image');
             const imageObjectsWithSrc = imageObjects.filter((obj) => typeof obj?.src === 'string' && obj.src.trim().length > 0);
+            const fileObjects = loadedObjects.filter((obj) => obj?.type === 'file');
+            const fileObjectsWithSrc = fileObjects.filter((obj) => typeof obj?.src === 'string' && obj.src.trim().length > 0);
+            const fileObjectsWithoutSrc = fileObjects
+                .filter((obj) => !(typeof obj?.src === 'string' && obj.src.trim().length > 0))
+                .map((obj) => obj?.id || 'unknown');
             console.log('MoodBoard load image src diagnostics:', {
                 totalObjects: loadedObjects.length,
                 imageObjects: imageObjects.length,
-                imageObjectsWithSrc: imageObjectsWithSrc.length
+                imageObjectsWithSrc: imageObjectsWithSrc.length,
+                fileObjects: fileObjects.length,
+                fileObjectsWithSrc: fileObjectsWithSrc.length
             });
+            if (fileObjectsWithoutSrc.length > 0) {
+                console.warn('MoodBoard load warning: file objects without src:', fileObjectsWithoutSrc);
+            }
             const loadedVersion = Number(normalizedData.version) || null;
             board.currentLoadedVersion = loadedVersion;
             board.historyCursorVersion = loadedVersion;

package/src/services/FileUploadService.js

@@ -1,5 +1,3 @@
-import { isV2FileDownloadUrl } from './AssetUrlPolicy.js';
-
 /**
  * Сервис для загрузки и управления файлами на сервере
  */
@@ -7,7 +5,6 @@ export class FileUploadService {
     constructor(apiClient, options = {}) {
         this.apiClient = apiClient;
         this.uploadEndpoint = '/api/v2/files/upload';
-        this.deleteEndpoint = '/api/v2/files';
         this.options = {
             csrfToken: null, // Можно передать токен напрямую
             csrfTokenSelector: 'meta[name="csrf-token"]', // Селектор для поиска токена в DOM
@@ -51,7 +48,7 @@ export class FileUploadService {
      * Загружает файл на сервер
      * @param {File|Blob} file - файл для загрузки
      * @param {string} name - имя файла
-     * @returns {Promise<{id: string, url: string, size: number, name: string}>}
+     * @returns {Promise<{src: string, size: number, name: string, mimeType: string}>}
      */
     async uploadFile(file, name = null) {
         try {
@@ -89,30 +86,27 @@ export class FileUploadService {
             }
 
             const result = await response.json();
+            const status = response.status;
             
             if (!result.success) {
                 throw new Error(result.message || 'Ошибка загрузки файла');
             }
 
-            const fileId = result.data.fileId || result.data.id;
             const serverUrl = typeof result.data.url === 'string' ? result.data.url.trim() : '';
-            if (!fileId) {
-                throw new Error('Сервер не вернул fileId.');
-            }
-            if (!isV2FileDownloadUrl(serverUrl)) {
-                throw new Error('Некорректный URL файла от сервера. Ожидается /api/v2/files/{fileId}/download');
-            }
-            if (!this._matchesFileIdInUrl(serverUrl, fileId)) {
-                throw new Error('fileId не совпадает с URL файла от сервера.');
+            if (!serverUrl) {
+                throw new Error('Сервер не вернул data.url для файла.');
             }
+            console.log('File upload diagnostics:', {
+                status,
+                success: !!result.success,
+                url: serverUrl
+            });
 
             return {
-                id: fileId, // Используем fileId как основное поле, id для обратной совместимости
-                fileId, // Добавляем fileId для явного доступа
-                url: serverUrl,
+                src: serverUrl,
                 size: result.data.size,
                 name: result.data.name,
-                type: result.data.type
+                mimeType: result.data.mime_type || result.data.type || file.type || 'application/octet-stream'
             };
 
         } catch (error) {
@@ -121,329 +115,6 @@ export class FileUploadService {
         }
     }
 
-    _matchesFileIdInUrl(url, fileId) {
-        const id = typeof fileId === 'string' ? fileId.trim() : '';
-        if (!id || typeof url !== 'string') return false;
-        const raw = url.trim();
-        const relativeMatch = raw.match(/^\/api\/v2\/files\/([^/]+)\/download$/i);
-        if (relativeMatch) {
-            return decodeURIComponent(relativeMatch[1]) === id;
-        }
-        try {
-            const parsed = new URL(raw);
-            const absoluteMatch = parsed.pathname.match(/^\/api\/v2\/files\/([^/]+)\/download$/i);
-            return !!absoluteMatch && decodeURIComponent(absoluteMatch[1]) === id;
-        } catch (_) {
-            return false;
-        }
-    }
-
-    /**
-     * Обновляет метаданные файла на сервере
-     * @param {string} fileId - ID файла
-     * @param {Object} metadata - метаданные для обновления
-     * @returns {Promise<Object>}
-     */
-    async updateFileMetadata(fileId, metadata) {
-        try {
-            const csrfToken = this._getCsrfToken();
-            
-            if (this.options.requireCsrf && !csrfToken) {
-                throw new Error('CSRF токен не найден. Добавьте <meta name="csrf-token" content="{{ csrf_token() }}"> в HTML или передайте токен в опциях.');
-            }
-
-            const headers = {
-                'Content-Type': 'application/json',
-                'X-Requested-With': 'XMLHttpRequest',
-                'Accept': 'application/json'
-            };
-
-            // Добавляем CSRF токен только если он есть
-            if (csrfToken) {
-                headers['X-CSRF-TOKEN'] = csrfToken;
-            }
-
-            const response = await fetch(`${this.deleteEndpoint}/${fileId}`, {
-                method: 'PUT',
-                headers,
-                credentials: 'same-origin',
-                body: JSON.stringify(metadata)
-            });
-
-            if (!response.ok) {
-                const errorData = await response.json().catch(() => null);
-                throw new Error(errorData?.message || `HTTP ${response.status}: ${response.statusText}`);
-            }
-
-            const result = await response.json();
-            
-            if (!result.success) {
-                throw new Error(result.message || 'Ошибка обновления метаданных файла');
-            }
-
-            return result.data;
-
-        } catch (error) {
-            console.error('Ошибка обновления метаданных файла:', error);
-            throw error;
-        }
-    }
-
-    /**
-     * Удаляет файл с сервера
-     * @param {string} fileId - ID файла
-     */
-    async deleteFile(fileId) {
-        try {
-            const csrfToken = this._getCsrfToken();
-            
-            const headers = {
-                'X-Requested-With': 'XMLHttpRequest',
-                'Accept': 'application/json'
-            };
-
-            // Добавляем CSRF токен только если он есть
-            if (csrfToken) {
-                headers['X-CSRF-TOKEN'] = csrfToken;
-            }
-
-            const response = await fetch(`${this.deleteEndpoint}/${fileId}`, {
-                method: 'DELETE',
-                headers,
-                credentials: 'same-origin'
-            });
-
-            if (!response.ok) {
-                const errorData = await response.json().catch(() => null);
-                throw new Error(errorData?.message || `HTTP ${response.status}: ${response.statusText}`);
-            }
-
-            const result = await response.json();
-            return result.success;
-
-        } catch (error) {
-            console.error('Ошибка удаления файла:', error);
-            throw error;
-        }
-    }
-
-    /**
-     * Получает информацию о файле
-     * @param {string} fileId - ID файла  
-     */
-    async getFileInfo(fileId) {
-        try {
-            const response = await fetch(`${this.deleteEndpoint}/${fileId}`, {
-                method: 'GET',
-                headers: {
-                    'Accept': 'application/json',
-                    'X-Requested-With': 'XMLHttpRequest'
-                },
-                credentials: 'same-origin'
-            });
-
-            if (!response.ok) {
-                throw new Error(`HTTP ${response.status}: ${response.statusText}`);
-            }
-
-            const result = await response.json();
-            return result.data;
-
-        } catch (error) {
-            console.error('Ошибка получения информации о файле:', error);
-            throw error;
-        }
-    }
-
-    /**
-     * Получает URL для скачивания файла
-     * @param {string} fileId - ID файла
-     */
-    getDownloadUrl(fileId) {
-        return `${this.deleteEndpoint}/${fileId}/download`;
-    }
-
-    /**
-     * Скачивает файл с сервера
-     * @param {string} fileId - ID файла 
-     * @param {string} fileName - имя файла для скачивания
-     */
-    async downloadFile(fileId, fileName = null) {
-        try {
-            const downloadUrl = this.getDownloadUrl(fileId);
-            
-            console.log('📥 FileUploadService: Начинаем скачивание файла:', {
-                fileId,
-                fileName,
-                downloadUrl,
-                userAgent: navigator.userAgent,
-                isSecureContext: window.isSecureContext
-            });
-            
-            // Метод 1: Попробуем через fetch + blob (более надежно для современных браузеров)
-            try {
-                console.log('🔄 Метод 1: Пробуем скачивание через fetch...');
-                
-                const response = await fetch(downloadUrl, {
-                    method: 'GET',
-                    headers: {
-                        'X-Requested-With': 'XMLHttpRequest'
-                    },
-                    credentials: 'same-origin'
-                });
-                
-                console.log('📥 Ответ сервера:', {
-                    status: response.status,
-                    statusText: response.statusText,
-                    contentType: response.headers.get('content-type'),
-                    contentLength: response.headers.get('content-length'),
-                    contentDisposition: response.headers.get('content-disposition')
-                });
-                
-                if (!response.ok) {
-                    // Пытаемся получить JSON ошибку от Laravel
-                    let errorMessage = `HTTP ${response.status}: ${response.statusText}`;
-                    
-                    try {
-                        const errorData = await response.json();
-                        console.error('🚨 Ошибка от сервера:', errorData);
-                        
-                        if (errorData.message) {
-                            errorMessage = `${errorMessage} - ${errorData.message}`;
-                        }
-                        
-                        // Показываем пользователю детальную ошибку
-                        if (errorData.success === false) {
-                            alert(`Ошибка сервера: ${errorData.message || 'Файл не найден'}`);
-                        }
-                    } catch (jsonError) {
-                        console.warn('Не удалось прочитать JSON ошибку:', jsonError);
-                    }
-                    
-                    throw new Error(errorMessage);
-                }
-                
-                // Получаем blob файла
-                const blob = await response.blob();
-                console.log('📦 Получен blob:', {
-                    size: blob.size,
-                    type: blob.type
-                });
-                
-                // Создаем URL для blob
-                const blobUrl = window.URL.createObjectURL(blob);
-                
-                // Создаем ссылку для скачивания
-                const link = document.createElement('a');
-                link.href = blobUrl;
-                link.download = fileName || `file_${fileId}`;
-                
-                // Добавляем в DOM, кликаем и удаляем
-                document.body.appendChild(link);
-                link.click();
-                document.body.removeChild(link);
-                
-                // Освобождаем память
-                window.URL.revokeObjectURL(blobUrl);
-                
-                console.log('✅ Файл успешно скачан через fetch/blob:', fileName || fileId);
-                return true;
-                
-            } catch (fetchError) {
-                console.warn('❌ Ошибка скачивания через fetch:', fetchError);
-                
-                // Метод 2: Fallback - открываем в новом окне
-                console.log('🔄 Метод 2: Пробуем открытие в новом окне...');
-                
-                try {
-                    const newWindow = window.open(downloadUrl, '_blank');
-                    if (newWindow) {
-                        console.log('✅ Файл открыт в новом окне');
-                        return true;
-                    } else {
-                        throw new Error('Popup заблокирован браузером');
-                    }
-                } catch (windowError) {
-                    console.warn('❌ Ошибка открытия в новом окне:', windowError);
-                    
-                    // Метод 3: Последний fallback - прямая ссылка
-                    console.log('🔄 Метод 3: Создаем прямую ссылку...');
-                    
-                    const link = document.createElement('a');
-                    link.href = downloadUrl;
-                    if (fileName) {
-                        link.download = fileName;
-                    }
-                    link.target = '_blank';
-                    
-                    document.body.appendChild(link);
-                    link.click();
-                    document.body.removeChild(link);
-                    
-                    console.log('✅ Создана прямая ссылка для скачивания');
-                    return true;
-                }
-            }
-
-        } catch (error) {
-            console.error('❌ FileUploadService: Критическая ошибка скачивания файла:', error);
-            
-            // Показываем пользователю альтернативную ссылку
-            if (confirm(`Автоматическое скачивание не удалось: ${error.message}\n\nОткрыть файл в новой вкладке?`)) {
-                window.open(this.getDownloadUrl(fileId), '_blank');
-            }
-            
-            throw error;
-        }
-    }
-
-    /**
-     * Очищает неиспользуемые файлы с сервера
-     * @returns {Promise<{deletedCount: number, errors: Array}>}
-     */
-    async cleanupUnusedFiles() {
-        try {
-            const csrfToken = this._getCsrfToken();
-            
-            const headers = {
-                'X-Requested-With': 'XMLHttpRequest',
-                'Accept': 'application/json'
-            };
-
-            // Добавляем CSRF токен только если он есть
-            if (csrfToken) {
-                headers['X-CSRF-TOKEN'] = csrfToken;
-            }
-
-            const response = await fetch(`${this.deleteEndpoint}/cleanup`, {
-                method: 'POST',
-                headers,
-                credentials: 'same-origin'
-            });
-
-            if (!response.ok) {
-                const errorData = await response.json().catch(() => null);
-                throw new Error(errorData?.message || `HTTP ${response.status}: ${response.statusText}`);
-            }
-
-            const result = await response.json();
-            
-            if (result.success) {
-                const data = result.data || {};
-                return {
-                    deletedCount: data.deleted_count || 0,
-                    errors: data.errors || []
-                };
-            } else {
-                throw new Error(result.message || 'Ошибка очистки файлов');
-            }
-
-        } catch (error) {
-            console.error('Ошибка очистки неиспользуемых файлов:', error);
-            throw error;
-        }
-    }
-
     /**
      * Проверяет, поддерживается ли тип файла для предварительного просмотра
      */

package/src/tools/manager/ToolEventRouter.js

@@ -381,7 +381,7 @@ export class ToolEventRouter {
                     fileSize: file.size || 0,
                     mimeType: file.type || 'application/octet-stream',
                     formattedSize: null,
-                    url: null,
+                    src: null,
                     width: 120,
                     height: 140
                 };
@@ -413,7 +413,7 @@ export class ToolEventRouter {
                         };
                         logDropDebug(diagnostics, 'file_upload_success', {
                             fileName: uploadResult?.name || fallbackProps.fileName,
-                            fileId: uploadResult?.fileId || uploadResult?.id || null
+                            srcPresent: !!(typeof uploadResult?.src === 'string' && uploadResult.src.trim())
                         });
                         return {
                             type: 'file',
@@ -424,11 +424,10 @@ export class ToolEventRouter {
                                 fileSize: uploadResult.size,
                                 mimeType: uploadResult.mimeType,
                                 formattedSize: uploadResult.formattedSize,
-                                url: uploadResult.url,
+                                src: uploadResult.src,
                                 width: 120,
                                 height: 140
-                            },
-                            fileId: uploadResult.fileId || uploadResult.id || null
+                            }
                         };
                     } else {
                         showDropWarning(

package/src/tools/object-tools/placement/PlacementPayloadFactory.js

@@ -72,11 +72,10 @@ export class PlacementPayloadFactory {
                 fileSize: uploadResult.size,
                 mimeType: uploadResult.mimeType,
                 formattedSize: uploadResult.formattedSize,
-                url: uploadResult.url,
+                src: uploadResult.src,
                 width,
                 height
-            },
-            fileId: uploadResult.fileId || uploadResult.id
+            }
         });
     }
 

package/src/ui/FilePropertiesPanel.js

@@ -181,8 +181,8 @@ export class FilePropertiesPanel {
     }
 
     async _handleDownload() {
-        if (!this.currentId || !this.core?.fileUploadService) {
-            console.warn('FilePropertiesPanel: не могу скачать файл - нет currentId или fileUploadService');
+        if (!this.currentId || !this.core?.state?.getObjects) {
+            console.warn('FilePropertiesPanel: не могу скачать файл - нет currentId или state');
             return;
         }
 
@@ -202,18 +202,18 @@ export class FilePropertiesPanel {
                 return;
             }
 
-            const fileId = fileObject.fileId;
+            const fileSrc = typeof fileObject.src === 'string' ? fileObject.src.trim() : '';
             const fileName = fileObject.properties?.fileName || 'file';
 
             console.log('📎 FilePropertiesPanel: Данные файла для скачивания:', {
-                fileId,
+                fileSrc,
                 fileName,
-                downloadUrl: this.core.fileUploadService.getDownloadUrl(fileId)
+                hasSrc: !!fileSrc
             });
 
-            if (!fileId) {
-                console.warn('FilePropertiesPanel: у файла нет fileId');
-                alert('Ошибка: файл не имеет ID для скачивания');
+            if (!fileSrc) {
+                console.warn('FilePropertiesPanel: у файла нет src');
+                alert('Ошибка: у файла отсутствует src для скачивания');
                 return;
             }
 
@@ -228,8 +228,14 @@ export class FilePropertiesPanel {
             `;
             this.downloadButton.disabled = true;
 
-            // Скачиваем файл
-            await this.core.fileUploadService.downloadFile(fileId, fileName);
+            // Скачиваем файл напрямую по src (без id-based endpoint)
+            const link = document.createElement('a');
+            link.href = fileSrc;
+            link.download = fileName;
+            link.target = '_blank';
+            document.body.appendChild(link);
+            link.click();
+            document.body.removeChild(link);
 
             // Восстанавливаем кнопку
             setTimeout(() => {
@@ -262,14 +268,14 @@ export class FilePropertiesPanel {
         const fileObject = objects.find(obj => obj.id === this.currentId);
         
         if (fileObject && fileObject.type === 'file') {
-            const hasFileId = !!(fileObject.fileId);
+            const hasFileSrc = typeof fileObject.src === 'string' && fileObject.src.trim().length > 0;
             
-            // Показываем/скрываем кнопку скачивания в зависимости от наличия fileId
+            // Показываем/скрываем кнопку скачивания в зависимости от наличия src
             if (this.downloadButton) {
-                // Всегда показываем кнопку, даже без fileId
+                // Всегда показываем кнопку, но отключаем без src
                 this.downloadButton.style.display = 'flex';
-                this.downloadButton.disabled = !hasFileId;
-                this.downloadButton.title = hasFileId ? 'Скачать файл' : 'Файл недоступен для скачивания';
+                this.downloadButton.disabled = !hasFileSrc;
+                this.downloadButton.title = hasFileSrc ? 'Скачать файл' : 'Файл недоступен для скачивания';
             }
         }
     }