image-video-optimizer 1.2.0 → 1.2.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "image-video-optimizer",
-  "version": "1.2.0",
+  "version": "1.2.1",
   "description": "CLI tool to optimize images and videos with configurable resize and compression settings",
   "main": "src/index.js",
   "bin": {

package/src/videoProcessor.js

@@ -18,6 +18,11 @@ class VideoProcessor {
         console.log(`Processing video: ${path.basename(inputPath)}`);
         
         const outputPath = this.generateOutputPath(inputPath);
+        const parsedPath = path.parse(inputPath);
+        
+        // Always use temporary file for processing
+        const tempPath = path.join(parsedPath.dir, `${parsedPath.name}_temp.mp4`);
+        console.log(`Using temporary path for processing: ${path.basename(tempPath)}`);
         
         ffmpeg.ffprobe(inputPath, (err, metadata) => {
           if (err) {
@@ -44,13 +49,6 @@ class VideoProcessor {
             return;
           }
           
-          const outputPath = this.generateOutputPath(inputPath);
-          const parsedPath = path.parse(inputPath);
-          
-          // Always use temporary file for processing
-          const tempPath = path.join(parsedPath.dir, `${parsedPath.name}_temp.mp4`);
-          console.log(`Using temporary path for processing: ${path.basename(tempPath)}`);
-          
           let tempFfmpegCommand = ffmpeg(inputPath);
           
           if (needsResize) {
@@ -113,67 +111,6 @@ class VideoProcessor {
               resolve({ processed: false, error: err.message });
             })
             .run();
-          }
-          
-          let normalFfmpegCommand = ffmpeg(inputPath);
-        
-          if (needsResize) {
-            const newHeight = Math.round((maxWidth / videoStream.width) * videoStream.height);
-            normalFfmpegCommand = normalFfmpegCommand.videoFilters(`scale=${maxWidth}:${newHeight}`);
-            console.log(`Resizing to: ${maxWidth}x${newHeight}`);
-          }
-          
-          switch (encodeFormat.toLowerCase()) {
-            case 'h264':
-              normalFfmpegCommand = normalFfmpegCommand.videoCodec('libx264').audioCodec('aac');
-              break;
-            case 'h265':
-              normalFfmpegCommand = normalFfmpegCommand.videoCodec('libx265').audioCodec('aac');
-              break;
-            case 'vp9':
-              normalFfmpegCommand = normalFfmpegCommand.videoCodec('libvpx-vp9').audioCodec('libvorbis');
-              break;
-            default:
-              normalFfmpegCommand = normalFfmpegCommand.videoCodec('libx264').audioCodec('aac');
-          }
-          
-          // Add input/output options with proper quoting
-          normalFfmpegCommand = normalFfmpegCommand.inputOptions(['-fflags', '+genpts']);
-          normalFfmpegCommand = normalFfmpegCommand.outputOptions(['-crf', '23', '-preset', 'medium', '-y']);
-          normalFfmpegCommand = normalFfmpegCommand.format('mp4');
-          normalFfmpegCommand = normalFfmpegCommand.output(outputPath);
-          
-          normalFfmpegCommand
-            .on('start', (commandLine) => {
-              console.log(`FFmpeg command: ${commandLine}`);
-            })
-            .on('end', () => {
-              const compressionResult = this.checkCompression(inputPath, outputPath);
-              
-              if (compressionResult.effective) {
-                console.log(`✓ Processed: ${path.basename(inputPath)} (${compressionResult.compressionPercent}% reduction)`);
-                resolve({
-                  processed: true,
-                  outputPath,
-                  originalSize: compressionResult.originalSize,
-                  newSize: compressionResult.newSize,
-                  compressionPercent: compressionResult.compressionPercent
-                });
-              } else {
-                console.log(`✗ Ineffective compression, keeping original: ${path.basename(inputPath)}`);
-                fs.unlinkSync(outputPath);
-                resolve({ processed: false, reason: 'ineffective_compression' });
-              }
-            })
-            .on('error', (err) => {
-              console.error(`Error processing video ${inputPath}:`, err.message);
-              console.error(`FFmpeg stderr: ${err.stderr || 'No stderr available'}`);
-              if (fs.existsSync(outputPath)) {
-                fs.unlinkSync(outputPath);
-              }
-              resolve({ processed: false, error: err.message });
-            })
-            .run();
         });
       } catch (error) {
         console.error(`Error processing video ${inputPath}:`, error.message);