scss-variable-extractor 1.6.2 → 1.6.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "scss-variable-extractor",
-  "version": "1.6.2",
+  "version": "1.6.3",
   "description": "Analyzes Angular SCSS files and extracts repeated hardcoded values into reusable variables",
   "bin": {
     "scss-extract": "./bin/cli.js"

package/src/refactorer.js

@@ -153,6 +153,15 @@ function isInSafeZone(content, position, matchedValue) {
     return true;
   }
   
+  // Prevent replacement within hex colors
+  if (/[0-9a-fA-F]/.test(charBefore) || /[0-9a-fA-F]/.test(charAfter)) {
+    // Look back further to check if we're in a hex color
+    const lookBehind = content.substring(Math.max(0, position - 10), position);
+    if (/#[0-9a-fA-F]*$/.test(lookBehind)) {
+      return true;
+    }
+  }
+  
   // Prevent replacement if it would create invalid variable concatenation
   if (charBefore === '$' || charAfter === '$') {
     return true;

package/test/refactorer.test.js

@@ -167,4 +167,45 @@ describe('Refactorer', () => {
     expect(result.content).not.toContain('76$spacing-sm');
     expect(result.content).not.toContain('$z-index23');
   });
+
+  test('should not replace characters within hex colors', () => {
+    const scss = `
+      .component {
+        background: #a2dab1;
+        border-color: #ff2244;
+        color: #333;
+      }
+    `;
+    
+    const analysis = {
+      colors: [
+        { value: '#a2dab1', suggestedName: '$color-success' },
+        { value: '#333', suggestedName: '$color-text' }
+      ],
+      spacing: [],
+      fontSizes: [],
+      fontWeights: [],
+      fontFamilies: [],
+      borderRadius: [],
+      shadows: [],
+      zIndex: [
+        { value: '2', suggestedName: '$z-index-2' }
+      ],
+      sizing: [],
+      lineHeight: [],
+      opacity: [],
+      transitions: []
+    };
+    
+    const result = refactorFile(scss, analysis, '/libs/styles/_variables.scss', '/app/test.scss', {});
+    
+    // Should replace complete hex colors
+    expect(result.content).toContain('background: $color-success');
+    expect(result.content).toContain('color: $color-text');
+    
+    // Should NOT replace digits within hex colors
+    expect(result.content).toContain('#ff2244'); // Not #ff$z-index-2$z-index-244
+    expect(result.content).not.toContain('#a$z-index-2dab1');
+    expect(result.content).not.toContain('#ff$z-index-2');
+  });
 });