@flusys/nestjs-storage 4.1.1 → 5.0.0

package/fesm/middlewares/file-serve.middleware.js

@@ -50,7 +50,7 @@ export class FileServeMiddleware {
                 throw new NotFoundException({
                     message: `Not a file: ${normalizedPath}`,
                     messageKey: FILE_MESSAGES.NOT_FOUND,
-                    messageParams: {
+                    messageVariables: {
                         path: normalizedPath
                     }
                 });
@@ -99,7 +99,7 @@ export class FileServeMiddleware {
         throw new NotFoundException({
             message: `File not found: ${normalizedPath}`,
             messageKey: FILE_MESSAGES.NOT_FOUND,
-            messageParams: {
+            messageVariables: {
                 path: normalizedPath
             }
         });
@@ -148,7 +148,7 @@ export class FileServeMiddleware {
                 this.sendErrorResponse(res, new NotFoundException({
                     message: `Failed to serve file: ${normalizedPath}`,
                     messageKey: FILE_MESSAGES.NOT_FOUND,
-                    messageParams: {
+                    messageVariables: {
                         path: normalizedPath
                     }
                 }));
@@ -162,12 +162,12 @@ export class FileServeMiddleware {
             const response = isNotFound ? error.getResponse() : null;
             const message = response?.message ?? 'File not found';
             const messageKey = response?.messageKey ?? FILE_MESSAGES.NOT_FOUND;
-            const messageParams = response?.messageParams;
+            const messageVariables = response?.messageVariables;
             res.status(404).json({
                 success: false,
                 message,
                 messageKey,
-                messageParams,
+                messageVariables,
                 error: 'Not Found',
                 statusCode: 404
             });

package/fesm/providers/azure-provider.optional.js

@@ -31,7 +31,7 @@ export class AzureProvider {
             throw new InternalServerErrorException({
                 message: 'Azure Storage SDK not found',
                 messageKey: SYSTEM_MESSAGES.SDK_NOT_INSTALLED,
-                messageParams: {
+                messageVariables: {
                     sdk: '@azure/storage-blob'
                 }
             });

package/fesm/providers/s3-provider.optional.js

@@ -36,7 +36,7 @@ export class S3Provider {
             throw new InternalServerErrorException({
                 message: 'AWS SDK not found',
                 messageKey: SYSTEM_MESSAGES.SDK_NOT_INSTALLED,
-                messageParams: {
+                messageVariables: {
                     sdk: '@aws-sdk/client-s3'
                 }
             });

package/fesm/providers/sftp-provider.optional.js

@@ -59,7 +59,7 @@ import { v4 as uuidv4 } from 'uuid';
             throw new InternalServerErrorException({
                 message: 'SFTP client not found',
                 messageKey: SYSTEM_MESSAGES.SDK_NOT_INSTALLED,
-                messageParams: {
+                messageVariables: {
                     sdk: 'ssh2-sftp-client'
                 }
             });

package/fesm/providers/storage-factory.service.js

@@ -25,9 +25,9 @@ function _ts_param(paramIndex, decorator) {
         decorator(target, key, paramIndex);
     };
 }
+import { SYSTEM_MESSAGES } from '@flusys/nestjs-shared/constants';
 import { Inject, Injectable, InternalServerErrorException, NotFoundException } from '@nestjs/common';
 import * as crypto from 'crypto';
-import { SYSTEM_MESSAGES } from '@flusys/nestjs-shared/constants';
 import { StorageConfigService } from '../services';
 import { StorageProviderRegistry } from './storage-provider.registry';
 export class StorageFactoryService {
@@ -38,9 +38,9 @@ export class StorageFactoryService {
         const ProviderClass = StorageProviderRegistry.get(config.provider);
         if (!ProviderClass) {
             throw new NotFoundException({
-                message: `Storage provider '${config.provider}' not registered. Available: ${StorageProviderRegistry.getAll().join(', ')}`,
+                message: `Storage provider '${config.provider}' not available. Available: ${StorageProviderRegistry.getAll().join(', ')}`,
                 messageKey: SYSTEM_MESSAGES.SERVICE_NOT_AVAILABLE,
-                messageParams: {
+                messageVariables: {
                     provider: config.provider,
                     available: StorageProviderRegistry.getAll().join(', ')
                 }
@@ -56,7 +56,7 @@ export class StorageFactoryService {
             throw new InternalServerErrorException({
                 message: `Failed to initialize '${config.provider}': ${errorMessage}`,
                 messageKey: SYSTEM_MESSAGES.INTERNAL_ERROR,
-                messageParams: {
+                messageVariables: {
                     provider: config.provider,
                     error: errorMessage
                 }

package/fesm/services/file-manager.service.js

@@ -28,12 +28,12 @@ function _ts_param(paramIndex, decorator) {
 import { HybridCache, RequestScopedApiService } from '@flusys/nestjs-shared/classes';
 import { UtilsService } from '@flusys/nestjs-shared/modules';
 import { applyCompanyFilter } from '@flusys/nestjs-shared/utils';
-import { FILE_MESSAGES, FOLDER_MESSAGES, UPLOAD_MESSAGES } from '../config';
 import { BadRequestException, Inject, Injectable, NotFoundException, Scope } from '@nestjs/common';
 import { Brackets, In } from 'typeorm';
-import { StorageConfigService } from './storage-config.service';
+import { FILE_MESSAGES, FOLDER_MESSAGES, UPLOAD_MESSAGES } from '../config';
 import { FileManager, FileManagerWithCompany, Folder, FolderWithCompany } from '../entities';
 import { FileLocationEnum } from '../enums/file-location.enum';
+import { StorageConfigService } from './storage-config.service';
 import { StorageDataSourceProvider } from './storage-datasource.provider';
 import { UploadService } from './upload.service';
 const PERMISSION_CACHE_KEY = 'user_action_permission';
@@ -144,42 +144,6 @@ export class FileManagerService extends RequestScopedApiService {
         }
     }
     // ─── Public Methods ─────────────────────────────────────────────────────────
-    async enrichWithProviderNames(items) {
-        const configIds = [
-            ...new Set(items.map((i)=>i.storageConfigId).filter(Boolean))
-        ];
-        if (!configIds.length) {
-            return items.map((item)=>({
-                    ...item,
-                    providerName: undefined
-                }));
-        }
-        try {
-            const repo = await this.getStorageConfigRepository();
-            const configs = await repo.find({
-                where: {
-                    id: In(configIds)
-                },
-                select: [
-                    'id',
-                    'name'
-                ]
-            });
-            const nameMap = new Map(configs.map((c)=>[
-                    c.id,
-                    c.name
-                ]));
-            return items.map((item)=>({
-                    ...item,
-                    providerName: item.storageConfigId ? nameMap.get(item.storageConfigId) : undefined
-                }));
-        } catch  {
-            return items.map((item)=>({
-                    ...item,
-                    providerName: undefined
-                }));
-        }
-    }
     async getFiles(dtos, protocol, host, user) {
         await this.ensureRepositoryInitialized();
         const ids = dtos.map((d)=>d.id).filter(Boolean);
@@ -247,7 +211,7 @@ export class FileManagerService extends RequestScopedApiService {
             throw new BadRequestException({
                 message: `Folder "${folderId}" not found or access denied`,
                 messageKey: FOLDER_MESSAGES.NOT_FOUND,
-                messageParams: {
+                messageVariables: {
                     folderId
                 }
             });

package/fesm/services/storage-config.service.js

@@ -26,8 +26,9 @@ function _ts_param(paramIndex, decorator) {
     };
 }
 import { Inject, Injectable } from '@nestjs/common';
+import { UPLOAD_MESSAGES } from '../config';
+import { DEFAULT_ALLOWED_FILE_TYPES, DEFAULT_MAX_FILE_SIZE, STORAGE_MODULE_OPTIONS } from '../config/storage.constants';
 import { StorageModuleOptions } from '../interfaces';
-import { STORAGE_MODULE_OPTIONS, DEFAULT_MAX_FILE_SIZE, DEFAULT_ALLOWED_FILE_TYPES } from '../config/storage.constants';
 const BYTES_PER_MB = 1024 * 1024;
 const DEFAULT_LOCAL_PATH = './uploads';
 const DEFAULT_PORT = '3000';
@@ -71,7 +72,12 @@ export class StorageConfigService {
         if (fileSize > maxSize) {
             return {
                 valid: false,
-                message: `File size (${this.toMB(fileSize)}MB) exceeds maximum ${this.toMB(maxSize)}MB`
+                message: `File size (${this.toMB(fileSize)}MB) exceeds maximum ${this.toMB(maxSize)}MB`,
+                messageKey: UPLOAD_MESSAGES.FILE_TOO_LARGE,
+                messageVariables: {
+                    fileSize: this.toMB(fileSize),
+                    maxSize: this.toMB(maxSize)
+                }
             };
         }
         return {
@@ -82,7 +88,12 @@ export class StorageConfigService {
         if (!this.isFileTypeAllowed(mimeType)) {
             return {
                 valid: false,
-                message: `File type "${mimeType}" not allowed. Allowed: ${this.getAllowedFileTypes().join(', ')}`
+                message: `File type "${mimeType}" not allowed. Allowed: ${this.getAllowedFileTypes().join(', ')}`,
+                messageKey: UPLOAD_MESSAGES.INVALID_TYPE,
+                messageVariables: {
+                    mimeType,
+                    allowedTypes: this.getAllowedFileTypes().join(', ')
+                }
             };
         }
         return {

package/fesm/services/storage-datasource.provider.js

@@ -25,9 +25,9 @@ function _ts_param(paramIndex, decorator) {
         decorator(target, key, paramIndex);
     };
 }
+import { SYSTEM_MESSAGES } from '@flusys/nestjs-shared/constants';
 import { MultiTenantDataSourceService } from '@flusys/nestjs-shared/modules';
 import { Inject, Injectable, InternalServerErrorException, Optional, Scope } from '@nestjs/common';
-import { SYSTEM_MESSAGES } from '@flusys/nestjs-shared/constants';
 import { REQUEST } from '@nestjs/core';
 import { Request } from 'express';
 import { StorageConfigService } from './storage-config.service';
@@ -48,11 +48,6 @@ export class StorageDataSourceProvider extends MultiTenantDataSourceService {
    */ getEnableCompanyFeatureForTenant(tenant) {
         return tenant?.enableCompanyFeature ?? this.configService.isCompanyFeatureEnabled();
     }
-    /**
-   * Get enable company feature for current request context
-   */ getEnableCompanyFeatureForCurrentTenant() {
-        return this.getEnableCompanyFeatureForTenant(this.getCurrentTenant() ?? undefined);
-    }
     // Entity Management
     /**
    * Get storage entities for migrations based on company feature flag.

package/fesm/services/upload.service.js

@@ -26,16 +26,16 @@ function _ts_param(paramIndex, decorator) {
     };
 }
 import { LogAction } from '@flusys/nestjs-shared';
-import { UPLOAD_MESSAGES } from '../config';
 import { ILoggedUserInfo } from '@flusys/nestjs-shared/interfaces';
 import { validateCompanyOwnership } from '@flusys/nestjs-shared/utils';
 import { BadRequestException, Inject, Injectable, NotFoundException, Scope } from '@nestjs/common';
-import { StorageConfigService } from './storage-config.service';
+import { UPLOAD_MESSAGES } from '../config';
 import { UploadOptionsDto } from '../dtos/upload.dto';
 import { FileLocationEnum } from '../enums/file-location.enum';
 import { StorageFactoryService } from '../providers/storage-factory.service';
-import { StorageProviderConfigService } from './storage-provider-config.service';
 import { FileValidator } from '../utils/file-validator.util';
+import { StorageConfigService } from './storage-config.service';
+import { StorageProviderConfigService } from './storage-provider-config.service';
 export class UploadService {
     /**
    * Validate file before upload - includes size, type, and content validation.
@@ -46,7 +46,8 @@ export class UploadService {
         if (!sizeValidation.valid) {
             throw new BadRequestException({
                 message: sizeValidation.message,
-                messageKey: UPLOAD_MESSAGES.FILE_TOO_LARGE
+                messageKey: sizeValidation.messageKey,
+                messageVariables: sizeValidation.messageVariables
             });
         }
         // Validate declared file type (MIME)
@@ -54,7 +55,8 @@ export class UploadService {
         if (!typeValidation.valid) {
             throw new BadRequestException({
                 message: typeValidation.message,
-                messageKey: UPLOAD_MESSAGES.INVALID_TYPE
+                messageKey: typeValidation.messageKey,
+                messageVariables: typeValidation.messageVariables
             });
         }
         // Validate file content matches declared type (magic bytes check)
@@ -64,7 +66,11 @@ export class UploadService {
         if (!contentValidation.valid) {
             throw new BadRequestException({
                 message: contentValidation.message || 'File content validation failed',
-                messageKey: UPLOAD_MESSAGES.INVALID_TYPE
+                messageKey: UPLOAD_MESSAGES.INVALID_TYPE,
+                messageVariables: {
+                    mimeType: file.mimetype,
+                    allowedTypes: allowedTypes.join(', ')
+                }
             });
         }
         // Sanitize filename to prevent path traversal attacks

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@flusys/nestjs-storage",
-  "version": "4.1.1",
+  "version": "5.0.0",
   "description": "Modular storage package with optional AWS S3, Azure Blob, and SFTP providers",
   "main": "cjs/index.js",
   "module": "fesm/index.js",
@@ -129,7 +129,7 @@
     }
   },
   "dependencies": {
-    "@flusys/nestjs-core": "4.1.1",
-    "@flusys/nestjs-shared": "4.1.1"
+    "@flusys/nestjs-core": "5.0.0",
+    "@flusys/nestjs-shared": "5.0.0"
   }
 }

package/services/file-manager.service.d.ts

@@ -3,10 +3,10 @@ import { DeleteDto, FilterAndPaginationDto } from '@flusys/nestjs-shared/dtos';
 import { ILoggedUserInfo } from '@flusys/nestjs-shared/interfaces';
 import { UtilsService } from '@flusys/nestjs-shared/modules';
 import { EntityTarget, QueryRunner, Repository, SelectQueryBuilder } from 'typeorm';
-import { StorageConfigService } from './storage-config.service';
 import { CreateFileManagerDto, FilesResponseDto, GetFilesRequestDto, UpdateFileManagerDto } from '../dtos';
 import { FileManager, FileManagerBase } from '../entities';
 import { IFileManager } from '../interfaces';
+import { StorageConfigService } from './storage-config.service';
 import { StorageDataSourceProvider } from './storage-datasource.provider';
 import { UploadService } from './upload.service';
 export declare class FileManagerService extends RequestScopedApiService<CreateFileManagerDto, UpdateFileManagerDto, IFileManager, FileManagerBase, Repository<FileManagerBase>> {
@@ -32,11 +32,6 @@ export declare class FileManagerService extends RequestScopedApiService<CreateFi
         isRaw: boolean;
     }>;
     beforeDeleteOperation(dto: DeleteDto, user: ILoggedUserInfo | null, _qr: QueryRunner): Promise<void>;
-    enrichWithProviderNames<T extends {
-        storageConfigId?: string | null;
-    }>(items: T[]): Promise<(T & {
-        providerName?: string;
-    })[]>;
     getFiles(dtos: GetFilesRequestDto[], protocol: string, host: string, user?: ILoggedUserInfo): Promise<FilesResponseDto[]>;
     private buildWhereWithCompany;
     private getStorageConfigRepository;

package/services/storage-config.service.d.ts

@@ -4,6 +4,8 @@ import { StorageModuleOptions } from '../interfaces';
 interface ValidationResult {
     valid: boolean;
     message?: string;
+    messageKey?: string;
+    messageVariables?: Record<string, unknown>;
 }
 export declare class StorageConfigService implements IModuleConfigService {
     private readonly options;

package/services/storage-datasource.provider.d.ts

@@ -1,5 +1,5 @@
-import { MultiTenantDataSourceService } from '@flusys/nestjs-shared/modules';
 import { IDatabaseConfig, ITenantDatabaseConfig } from '@flusys/nestjs-core';
+import { MultiTenantDataSourceService } from '@flusys/nestjs-shared/modules';
 import { Request } from 'express';
 import { DataSource } from 'typeorm';
 import { StorageConfigService } from './storage-config.service';
@@ -14,7 +14,6 @@ export declare class StorageDataSourceProvider extends MultiTenantDataSourceServ
     constructor(configService: StorageConfigService, request?: Request);
     private static buildParentOptions;
     getEnableCompanyFeatureForTenant(tenant?: ITenantDatabaseConfig): boolean;
-    getEnableCompanyFeatureForCurrentTenant(): boolean;
     getStorageEntities(enableCompanyFeature?: boolean): Promise<any[]>;
     protected createDataSourceFromConfig(config: IDatabaseConfig): Promise<DataSource>;
     protected getSingleDataSource(): Promise<DataSource>;

package/services/upload.service.d.ts

@@ -1,8 +1,8 @@
 import { ILoggedUserInfo } from '@flusys/nestjs-shared/interfaces';
-import { StorageConfigService } from './storage-config.service';
 import { UploadOptionsDto } from '../dtos/upload.dto';
 import { IUploadedFileInfo } from '../interfaces/storage-provider.interface';
 import { StorageFactoryService } from '../providers/storage-factory.service';
+import { StorageConfigService } from './storage-config.service';
 import { StorageProviderConfigService } from './storage-provider-config.service';
 export declare class UploadService {
     private readonly storageFactory;