svger-cli 4.0.2 → 4.0.3

package/dist/optimizers/types.js

@@ -45,78 +45,68 @@ export function getDefaultOptConfig(level) {
         shapeConversionThreshold: 5,
         optimizationLevel: level,
     };
-    switch (level) {
-        case OptLevel.NONE:
-            return {
-                ...baseConfig,
-                removeMetadata: false,
-                removeComments: false,
-                normalizeWhitespace: false,
-                removeUnnecessaryAttrs: false,
-                shortenColors: false,
-                removeEmptyContainers: false,
-                removeDoctype: false,
-                removeXMLProcInst: false,
-                enableNumericOptimization: false,
-                enableStyleOptimization: false,
-                enableTransformOptimization: false,
-                enableTransformCollapsing: false,
-                enablePathOptimization: false,
-                enablePathSimplification: false,
-            };
-        case OptLevel.BASIC:
-            return baseConfig;
-        case OptLevel.BALANCED:
-            return {
-                ...baseConfig,
-                removeHiddenElements: true,
-                floatPrecision: 3,
-                enableNumericOptimization: true,
-                enableStyleOptimization: true,
-                enableTransformOptimization: false,
-                enableTransformCollapsing: false,
-                enablePathOptimization: false,
-            };
-        case OptLevel.AGGRESSIVE:
-            return {
-                ...baseConfig,
-                removeHiddenElements: true,
-                mergePaths: true,
-                collapseGroups: true,
-                floatPrecision: 2,
-                pathTolerance: 0.7,
-                sortAttrs: true,
-                enableNumericOptimization: true,
-                enableStyleOptimization: true,
-                enableTransformOptimization: true,
-                enableTransformCollapsing: true,
-                enablePathOptimization: true,
-                // DISABLED: Shape conversion conflicts with floatPrecision=2
-                // Causes visual regressions (coordinates get rounded before conversion)
-                shapeConversion: false,
-                shapeConversionThreshold: 5,
-            };
-        case OptLevel.MAXIMUM:
-            return {
-                ...baseConfig,
-                removeHiddenElements: true,
-                mergePaths: true,
-                collapseGroups: true,
-                inlineStyles: true,
-                floatPrecision: 1,
-                pathTolerance: 0.9,
-                sortAttrs: true,
-                removeViewBox: false, // Keep viewBox even at maximum for compatibility
-                enableNumericOptimization: true,
-                enableStyleOptimization: true,
-                enableTransformOptimization: true,
-                enableTransformCollapsing: true,
-                enablePathOptimization: true,
-                enablePathSimplification: true,
-                shapeConversion: true,
-                shapeConversionThreshold: 0, // Convert even 1-byte savings
-            };
-        default:
-            return baseConfig;
-    }
+    // O(1) object lookup for optimization level → config overrides
+    const levelOverrides = {
+        [OptLevel.NONE]: {
+            removeMetadata: false,
+            removeComments: false,
+            normalizeWhitespace: false,
+            removeUnnecessaryAttrs: false,
+            shortenColors: false,
+            removeEmptyContainers: false,
+            removeDoctype: false,
+            removeXMLProcInst: false,
+            enableNumericOptimization: false,
+            enableStyleOptimization: false,
+            enableTransformOptimization: false,
+            enableTransformCollapsing: false,
+            enablePathOptimization: false,
+            enablePathSimplification: false,
+        },
+        [OptLevel.BASIC]: {},
+        [OptLevel.BALANCED]: {
+            removeHiddenElements: true,
+            floatPrecision: 3,
+            enableNumericOptimization: true,
+            enableStyleOptimization: true,
+            enableTransformOptimization: false,
+            enableTransformCollapsing: false,
+            enablePathOptimization: false,
+        },
+        [OptLevel.AGGRESSIVE]: {
+            removeHiddenElements: true,
+            mergePaths: true,
+            collapseGroups: true,
+            floatPrecision: 2,
+            pathTolerance: 0.7,
+            sortAttrs: true,
+            enableNumericOptimization: true,
+            enableStyleOptimization: true,
+            enableTransformOptimization: true,
+            enableTransformCollapsing: true,
+            enablePathOptimization: true,
+            shapeConversion: false,
+            shapeConversionThreshold: 5,
+        },
+        [OptLevel.MAXIMUM]: {
+            removeHiddenElements: true,
+            mergePaths: true,
+            collapseGroups: true,
+            inlineStyles: true,
+            floatPrecision: 1,
+            pathTolerance: 0.9,
+            sortAttrs: true,
+            removeViewBox: false,
+            enableNumericOptimization: true,
+            enableStyleOptimization: true,
+            enableTransformOptimization: true,
+            enableTransformCollapsing: true,
+            enablePathOptimization: true,
+            enablePathSimplification: true,
+            shapeConversion: true,
+            shapeConversionThreshold: 0,
+        },
+    };
+    const overrides = levelOverrides[level] ?? {};
+    return { ...baseConfig, ...overrides };
 }

package/dist/processors/svg-processor.js

@@ -187,7 +187,7 @@ export class SVGProcessor {
         const baseName = path.basename(fileName, '.svg');
         // Object lookup map for naming conventions - O(1) performance
         const namingHandlers = {
-            kebab: () => toPascalCase(baseName),
+            kebab: () => toKebabCase(baseName),
             camel: () => {
                 const pascalName = toPascalCase(baseName);
                 return pascalName.charAt(0).toLowerCase() + pascalName.slice(1);
@@ -340,13 +340,26 @@ export class SVGProcessor {
                 error: error,
             };
             logger.error(`Failed to process ${svgFilePath}:`, error);
+            // Immediately remove failed jobs to prevent memory leaks
+            this.processingQueue.delete(jobId);
             return result;
         }
         finally {
-            // Clean up completed jobs after some time
-            setTimeout(() => {
-                this.processingQueue.delete(jobId);
-            }, 30000); // 30 seconds
+            // Clean up completed jobs after a short delay (allows stats queries)
+            if (job.status === 'completed') {
+                setTimeout(() => {
+                    this.processingQueue.delete(jobId);
+                }, 30000); // 30 seconds
+            }
+            // Cap queue size as a safety net to prevent unbounded growth
+            if (this.processingQueue.size > 10000) {
+                const oldestEntries = Array.from(this.processingQueue.entries())
+                    .filter(([, j]) => j.status === 'completed' || j.status === 'failed')
+                    .slice(0, this.processingQueue.size - 5000);
+                for (const [key] of oldestEntries) {
+                    this.processingQueue.delete(key);
+                }
+            }
         }
     }
     /**
@@ -354,7 +367,7 @@ export class SVGProcessor {
      */
     getProcessingStats() {
         const jobs = Array.from(this.processingQueue.values());
-        // Single pass through jobs instead of 4 separate filters
+        // Direct property increment via object key — O(1) per job, avoids switch branching
         const stats = {
             total: jobs.length,
             pending: 0,
@@ -363,20 +376,7 @@ export class SVGProcessor {
             failed: 0,
         };
         for (const job of jobs) {
-            switch (job.status) {
-                case 'pending':
-                    stats.pending++;
-                    break;
-                case 'processing':
-                    stats.processing++;
-                    break;
-                case 'completed':
-                    stats.completed++;
-                    break;
-                case 'failed':
-                    stats.failed++;
-                    break;
-            }
+            stats[job.status]++;
         }
         return stats;
     }

package/dist/services/config.js

@@ -275,10 +275,14 @@ export class ConfigService {
                 logger.info(`Migrated optimization level: ${oldOptimization} → ${migratedConfig.performance.optimization}`);
             }
         }
-        // Save migrated config
-        this.writeConfig(migratedConfig);
-        logger.success('Configuration migrated to v4.0.0');
-        return migratedConfig;
+        // Save migrated config merged with defaults to ensure all new keys exist
+        const mergedMigratedConfig = {
+            ...this.getDefaultConfig(),
+            ...migratedConfig,
+        };
+        this.writeConfig(mergedMigratedConfig);
+        logger.success(`Configuration migrated from ${currentVersion} to ${CURRENT_VERSION}`);
+        return mergedMigratedConfig;
     }
     /**
      * Validate configuration

package/dist/services/svg-service.js

@@ -25,8 +25,15 @@ export class SVGService {
      * Set optimizer level for SVG processing
      */
     setOptimizerLevel(level) {
-        const validLevels = ['none', 'basic', 'balanced', 'aggressive', 'maximum'];
-        if (!validLevels.includes(level.toLowerCase())) {
+        // O(1) Set lookup instead of O(n) Array.includes()
+        const validLevels = new Set([
+            'none',
+            'basic',
+            'balanced',
+            'aggressive',
+            'maximum',
+        ]);
+        if (!validLevels.has(level.toLowerCase())) {
             logger.warn(`Invalid optimization level "${level}". Using "basic" instead.`);
             svgProcessor.setOptimizationLevel(OptLevel.BASIC);
             return;

package/dist/utils/native.d.ts

@@ -45,7 +45,6 @@ export declare class FileSystem {
     private static _readdir;
     private static _stat;
     private static _mkdir;
-    private static _rmdir;
     private static _unlink;
     static exists(path: string): Promise<boolean>;
     static readFile(path: string, encoding?: BufferEncoding): Promise<string>;

package/dist/utils/native.js

@@ -62,7 +62,6 @@ export class FileSystem {
     static _readdir = promisify(fs.readdir);
     static _stat = promisify(fs.stat);
     static _mkdir = promisify(fs.mkdir);
-    static _rmdir = promisify(fs.rmdir);
     static _unlink = promisify(fs.unlink);
     static async exists(path) {
         try {
@@ -94,18 +93,7 @@ export class FileSystem {
     }
     static async removeDir(dirPath) {
         try {
-            const files = await this._readdir(dirPath);
-            for (const file of files) {
-                const filePath = `${dirPath}/${file}`;
-                const stats = await this._stat(filePath);
-                if (stats.isDirectory()) {
-                    await this.removeDir(filePath);
-                }
-                else {
-                    await this._unlink(filePath);
-                }
-            }
-            await this._rmdir(dirPath);
+            await fs.promises.rm(dirPath, { recursive: true, force: true });
         }
         catch (error) {
             if (error.code !== 'ENOENT') {
@@ -311,10 +299,14 @@ export class FileWatcher {
                     this.emit(eventType, `${path}/${filename}`);
                 }
             });
+            watcher.on('error', error => {
+                this.emit('error', error);
+            });
             this.watchers.push(watcher);
         }
         catch (error) {
-            console.error(`Failed to watch ${path}:`, error);
+            // Emit error event so consumers can handle it
+            this.emit('error', error);
         }
         return this;
     }

package/dist/watch.js

@@ -1,7 +1,7 @@
 import path from 'path';
 import { generateSVG } from './builder.js';
 import { isLocked } from './lock.js';
-import { FileSystem, FileWatcher } from './utils/native.js';
+import { FileSystem, FileWatcher, toPascalCase } from './utils/native.js';
 /**
  * Watches a source folder for changes to SVG files and automatically
  * rebuilds React components when SVGs are added, modified, or deleted.
@@ -59,8 +59,9 @@ export async function watchSVGs(config) {
             await generateSVG({ svgFile: filePath, outDir });
         }
         else {
-            // File was deleted
-            const componentName = path.basename(filePath, '.svg');
+            // File was deleted — use PascalCase to match the generated component filename
+            const baseName = path.basename(filePath, '.svg');
+            const componentName = toPascalCase(baseName);
             const outFile = path.join(outDir, `${componentName}.tsx`);
             if (await FileSystem.exists(outFile)) {
                 await FileSystem.unlink(outFile);

package/docs/ERROR-HANDLING-STANDARD.md

@@ -1,7 +1,7 @@
 # Error Handling Standards
 
 ## Purpose
-This document establishes standardized error handling patterns for SVGER-CLI v4.0.1+ to ensure consistency, proper error propagation, and appropriate user feedback.
+This document establishes standardized error handling patterns for SVGER-CLI v4.0.3+ to ensure consistency, proper error propagation, and appropriate user feedback.
 
 ## Standard Patterns
 

package/docs/OPTIONAL-DEPENDENCIES.md

@@ -158,4 +158,4 @@ If you encounter any issues with optional dependencies:
 
 ---
 
-**Last Updated**: January 28, 2026 (v4.0.1)
+**Last Updated**: January 28, 2026 (v4.0.3)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "svger-cli",
-  "version": "4.0.2",
+  "version": "4.0.3",
   "description": "Enterprise-grade SVG to component converter with advanced plugin system, visual diff testing, and official framework integrations. Supporting React, React Native, Vue, Angular, Svelte, Solid, Lit, Preact & Vanilla. Features TypeScript, HMR, optimization pipeline, and extensible architecture.",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",