scss-variable-extractor 1.0.0 → 1.1.0

package/bin/cli.js

@@ -1,4 +1,4 @@
-#!/usr/bin/env node
+#!/usr/bin/env node
 
 const { Command } = require('commander');
 const chalk = require('chalk');
@@ -21,18 +21,18 @@ program
 program
   .command('analyze')
   .description('Dry-run analysis - identifies repeated values without modifying files')
-  .option('--src <path>', 'Source directory to scan')
+  .argument('<src>', 'Source directory to scan')
   .option('--threshold <number>', 'Minimum repeat count threshold', parseInt)
   .option('--format <format>', 'Report format (table, json, markdown)', 'table')
   .option('--config <path>', 'Path to config file')
-  .action(async (options) => {
+  .action(async (src, options) => {
     try {
       console.log(chalk.cyan.bold('\n🔍 SCSS Variable Extraction Analysis\n'));
       
       const config = loadConfig(options.config);
       
       // Override config with command-line options
-      if (options.src) config.src = options.src;
+      config.src = src;
       if (options.threshold) config.threshold = options.threshold;
       if (options.format) config.reportFormat = options.format;
       
@@ -85,18 +85,18 @@ program
 program
   .command('generate')
   .description('Generate variables file only (does not modify existing SCSS files)')
-  .option('--src <path>', 'Source directory to scan')
+  .argument('<src>', 'Source directory to scan')
   .option('--output <path>', 'Output path for variables file')
   .option('--threshold <number>', 'Minimum repeat count threshold', parseInt)
   .option('--config <path>', 'Path to config file')
-  .action(async (options) => {
+  .action(async (src, options) => {
     try {
       console.log(chalk.cyan.bold('\n📝 Generating SCSS Variables File\n'));
       
       const config = loadConfig(options.config);
       
       // Override config with command-line options
-      if (options.src) config.src = options.src;
+      config.src = src;
       if (options.output) config.output = options.output;
       if (options.threshold) config.threshold = options.threshold;
       
@@ -154,18 +154,18 @@ program
 program
   .command('refactor')
   .description('Full extraction + replacement (generates variables file and refactors SCSS files)')
-  .option('--src <path>', 'Source directory to scan')
+  .argument('<src>', 'Source directory to scan')
   .option('--output <path>', 'Output path for variables file')
   .option('--threshold <number>', 'Minimum repeat count threshold', parseInt)
   .option('--config <path>', 'Path to config file')
-  .action(async (options) => {
+  .action(async (src, options) => {
     try {
       console.log(chalk.cyan.bold('\n🔧 SCSS Refactoring - Full Extraction\n'));
       
       const config = loadConfig(options.config);
       
       // Override config with command-line options
-      if (options.src) config.src = options.src;
+      config.src = src;
       if (options.output) config.output = options.output;
       if (options.threshold) config.threshold = options.threshold;
       

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "scss-variable-extractor",
-  "version": "1.0.0",
+  "version": "1.1.0",
   "description": "Analyzes Angular SCSS files and extracts repeated hardcoded values into reusable variables",
   "bin": {
     "scss-extract": "./bin/cli.js"
@@ -8,9 +8,9 @@
   "main": "src/index.js",
   "scripts": {
     "test": "jest",
-    "analyze": "node bin/cli.js analyze",
-    "generate": "node bin/cli.js generate",
-    "refactor": "node bin/cli.js refactor"
+    "analyze": "node bin/cli.js analyze .",
+    "generate": "node bin/cli.js generate .",
+    "refactor": "node bin/cli.js refactor ."
   },
   "keywords": [
     "angular",

package/src/scanner.js

@@ -9,16 +9,17 @@ const path = require('path');
  */
 async function scanScssFiles(srcPath, ignorePatterns = []) {
   return new Promise((resolve, reject) => {
-    const pattern = path.join(srcPath, '**/*.scss');
+    const pattern = path.join(srcPath, '**/*.scss').replace(/\\/g, '/');
     
     glob(pattern, {
-      ignore: ignorePatterns,
-      absolute: true
+      ignore: ignorePatterns
     }, (err, files) => {
       if (err) {
         reject(err);
       } else {
-        resolve(files);
+        // Ensure absolute paths
+        const absoluteFiles = files.map(f => path.resolve(f));
+        resolve(absoluteFiles);
       }
     });
   });