@sun-asterisk/sunlint 1.2.1 → 1.2.2

package/scripts/test-scripts/test-c041-comparison.js

@@ -1,114 +0,0 @@
-#!/usr/bin/env node
-
-/**
- * Comprehensive comparison script for C041 rule between Heuristic and ESLint engines
- */
-
-const { ESLint } = require("eslint");
-const path = require("path");
-const fs = require("fs");
-
-// Import custom C041 rule directly
-const c041Rule = require("./integrations/eslint/plugin/rules/common/c041-no-config-inline");
-
-async function testC041Comparison() {
-    console.log("🔍 C041 Rule Comparison: Heuristic vs ESLint\n");
-
-    // Test files to analyze
-    const testFiles = [
-        "examples/test-c041-sensitive-hardcode.js",
-        "examples/project-samples/replace-fe/src/security-test-examples.ts",
-        "examples/project-samples/replace-be/src/modules/login/specs/maintenance/login.service.spec.ts"
-    ];
-
-    console.log("📋 Files to test:");
-    testFiles.forEach(file => {
-        const fullPath = path.resolve(__dirname, file);
-        if (fs.existsSync(fullPath)) {
-            console.log(`  ✅ ${file}`);
-        } else {
-            console.log(`  ❌ ${file} (not found)`);
-        }
-    });
-    console.log("");
-
-    // Configure ESLint
-    const eslint = new ESLint({
-        baseConfig: {
-            plugins: {
-                "custom": {
-                    rules: {
-                        "c041": c041Rule
-                    }
-                }
-            },
-            rules: {
-                "custom/c041": "error"
-            },
-            languageOptions: {
-                ecmaVersion: 2020,
-                sourceType: "module"
-            }
-        },
-        overrideConfigFile: true
-    });
-
-    // Test each file
-    for (const testFile of testFiles) {
-        const fullPath = path.resolve(__dirname, testFile);
-        
-        if (!fs.existsSync(fullPath)) {
-            console.log(`⏭️  Skipping ${testFile} (file not found)\n`);
-            continue;
-        }
-
-        console.log(`🧪 Testing: ${testFile}`);
-        console.log("=" .repeat(70));
-
-        try {
-            // ESLint analysis
-            const eslintResults = await eslint.lintFiles([fullPath]);
-            const eslintViolations = eslintResults[0]?.messages || [];
-            
-            console.log(`\n📊 Results Summary:`);
-            console.log(`   ESLint violations: ${eslintViolations.length}`);
-            
-            if (eslintViolations.length > 0) {
-                console.log(`\n🔸 ESLint C041 violations:`);
-                eslintViolations.forEach((msg, index) => {
-                    console.log(`   ${index + 1}. Line ${msg.line}:${msg.column} - ${msg.message.substring(0, 80)}...`);
-                });
-            }
-
-            console.log(`\n🔍 Sample violations (first 3):`);
-            eslintViolations.slice(0, 3).forEach((msg, index) => {
-                console.log(`   ${index + 1}. Line ${msg.line}: "${getLineContent(fullPath, msg.line).trim().substring(0, 60)}..."`);
-            });
-
-        } catch (error) {
-            console.error(`❌ Error analyzing ${testFile}:`, error.message);
-        }
-
-        console.log("\n" + "=".repeat(70) + "\n");
-    }
-
-    // Summary comparison
-    console.log("📊 C041 Rule Analysis Summary:");
-    console.log("✅ Heuristic Engine: Robust detection of sensitive hardcoded values");
-    console.log("✅ ESLint Engine: Comprehensive coverage of hardcoded config values");
-    console.log("✅ Both engines successfully detect security-sensitive patterns");
-    console.log("✅ Real project testing shows practical effectiveness");
-    console.log("\n🎯 Conclusion: C041 is robust and production-ready on both engines!");
-}
-
-function getLineContent(filePath, lineNumber) {
-    try {
-        const content = fs.readFileSync(filePath, 'utf8');
-        const lines = content.split('\n');
-        return lines[lineNumber - 1] || '';
-    } catch (error) {
-        return '[unable to read line]';
-    }
-}
-
-testC041Comparison().catch(console.error);

package/scripts/test-scripts/test-c041-eslint.js

@@ -1,67 +0,0 @@
-#!/usr/bin/env node
-
-/**
- * Direct test script for C041 ESLint rule
- */
-
-const { ESLint } = require("eslint");
-const path = require("path");
-
-// Import custom C041 rule directly
-const c041Rule = require("./integrations/eslint/plugin/rules/common/c041-no-config-inline");
-
-async function testC041ESLint() {
-    console.log("🧪 Testing C041 ESLint Rule Directly\n");
-
-    const eslint = new ESLint({
-        baseConfig: {
-            plugins: {
-                "custom": {
-                    rules: {
-                        "c041": c041Rule
-                    }
-                }
-            },
-            rules: {
-                "custom/c041": "error"
-            },
-            languageOptions: {
-                ecmaVersion: 2020,
-                sourceType: "module"
-            }
-        },
-        overrideConfigFile: true
-    });
-
-    try {
-        // Test file path
-        const testFile = path.resolve(__dirname, "examples/test-c041-sensitive-hardcode.js");
-        
-        console.log(`📁 Testing file: ${testFile}`);
-        
-        const results = await eslint.lintFiles([testFile]);
-        
-        results.forEach(result => {
-            console.log(`\n📋 File: ${result.filePath}`);
-            console.log(`   Messages: ${result.messages.length}`);
-            
-            if (result.messages.length > 0) {
-                console.log(`   ❌ ESLint C041 violations found:`);
-                result.messages.forEach(msg => {
-                    console.log(`   Line ${msg.line}:${msg.column} - ${msg.message} (${msg.ruleId})`);
-                });
-            } else {
-                console.log(`   ✅ No ESLint violations found`);
-            }
-        });
-
-        const totalMessages = results.reduce((sum, result) => sum + result.messages.length, 0);
-        console.log(`\n📊 Total ESLint C041 violations: ${totalMessages}`);
-        
-    } catch (error) {
-        console.error("❌ Error testing ESLint rule:", error.message);
-        console.error(error.stack);
-    }
-}
-
-testC041ESLint();

package/scripts/test-scripts/test-eslint-rules.js

@@ -1,146 +0,0 @@
-#!/usr/bin/env node
-
-/**
- * Test ESLint rules that correspond to our improved Heuristic analyzers
- */
-
-const path = require('path');
-const fs = require('fs');
-const { ESLint } = require('eslint');
-
-async function testESLintRules() {
-  console.log('🔍 Testing ESLint rules that correspond to improved Heuristic analyzers\n');
-
-  // List of rules we've improved and their ESLint counterparts
-  const rulesToTest = [
-    'custom/c003', // no-vague-abbreviations
-    'custom/c006', // function-name-verb-noun  
-    'custom/c013', // no-dead-code
-    'custom/c014', // abstract-dependency-preferred
-    'custom/c017', // limit-constructor-logic
-    'custom/c029', // catch-block-logging
-    'custom/c041', // no-config-inline (NOTE: different from C041_no_sensitive_hardcode)
-    'custom/c042', // boolean-name-prefix
-    'custom/c047'  // no-duplicate-retry-logic
-  ];
-
-  try {
-    const eslint = new ESLint({
-      overrideConfig: {
-        plugins: ['@sun-asterisk/sunlint'],
-        rules: rulesToTest.reduce((acc, rule) => {
-          acc[rule] = 'error';
-          return acc;
-        }, {}),
-        languageOptions: {
-          ecmaVersion: 2021,
-          sourceType: 'module',
-          globals: {
-            ...require('globals').node
-          }
-        }
-      }
-    });
-
-    // Test files from our previous tests
-    const testFiles = [
-      'test-c014.js',
-      'test-c017-cases.ts', 
-      'test-c041-cases.ts',
-      'test-s029-cases.ts',
-      'examples/rule-test-fixtures/rules/C013_no_dead_code/test-cases.js',
-      'examples/rule-test-fixtures/rules/C042_boolean_name_prefix/test-cases.js',
-      'examples/rule-test-fixtures/rules/C047_no_duplicate_retry_logic/test-cases.js'
-    ];
-
-    let allResults = [];
-
-    for (const testFile of testFiles) {
-      const filePath = path.resolve(testFile);
-      
-      if (!fs.existsSync(filePath)) {
-        console.log(`⚠️  Test file not found: ${testFile}`);
-        continue;
-      }
-
-      console.log(`\n📋 Testing ${path.basename(testFile)} with ESLint rules...`);
-      
-      try {
-        const results = await eslint.lintFiles([filePath]);
-        
-        if (results.length > 0 && results[0].messages.length > 0) {
-          console.log(`   Found ${results[0].messages.length} ESLint violations:`);
-          
-          results[0].messages.forEach((msg, index) => {
-            console.log(`   ${index + 1}. Line ${msg.line}: [${msg.ruleId}] ${msg.message}`);
-          });
-          
-          allResults.push(...results);
-        } else {
-          console.log(`   ✅ No ESLint violations found`);
-        }
-      } catch (error) {
-        console.error(`   ❌ Error testing ${testFile}: ${error.message}`);
-      }
-    }
-
-    // Test on real project files
-    console.log('\n🏠 Testing on real project files...');
-    const realFiles = [
-      'examples/project-samples/replace-be/src/main.ts',
-      'examples/project-samples/replace-be/src/app.module.ts', 
-      'examples/project-samples/replace-be/src/health.controller.ts'
-    ];
-
-    for (const realFile of realFiles) {
-      const filePath = path.resolve(realFile);
-      
-      if (!fs.existsSync(filePath)) {
-        console.log(`⚠️  Real project file not found: ${realFile}`);
-        continue;
-      }
-
-      console.log(`\n📋 Testing ${path.basename(realFile)} with ESLint rules...`);
-      
-      try {
-        const results = await eslint.lintFiles([filePath]);
-        
-        if (results.length > 0 && results[0].messages.length > 0) {
-          console.log(`   Found ${results[0].messages.length} ESLint violations:`);
-          
-          results[0].messages.forEach((msg, index) => {
-            console.log(`   ${index + 1}. Line ${msg.line}: [${msg.ruleId}] ${msg.message}`);
-            
-            // Show some context around the violation
-            if (msg.line) {
-              const content = fs.readFileSync(filePath, 'utf8');
-              const lines = content.split('\n');
-              const violationLine = lines[msg.line - 1];
-              if (violationLine) {
-                console.log(`      Code: ${violationLine.trim()}`);
-              }
-            }
-          });
-        } else {
-          console.log(`   ✅ No ESLint violations found`);
-        }
-      } catch (error) {
-        console.error(`   ❌ Error testing ${realFile}: ${error.message}`);
-      }
-    }
-
-    console.log('\n📊 Summary:');
-    console.log(`Total violations found across all files: ${allResults.reduce((sum, result) => sum + result.messages.length, 0)}`);
-    console.log('ESLint rules test completed!');
-
-  } catch (error) {
-    console.error('❌ Failed to initialize ESLint:', error.message);
-    console.error('Make sure @sun-asterisk/sunlint plugin is properly installed and configured.');
-  }
-}
-
-if (require.main === module) {
-  testESLintRules().catch(console.error);
-}
-
-module.exports = { testESLintRules };

package/scripts/test-scripts/test-real-world.js

@@ -1,44 +0,0 @@
-// Test file with various body/query patterns
-import React from 'react';
-
-// 1. Style objects - should be IGNORED
-const styles = {
-  body: { padding: 20, color: 'red' },
-  container: { margin: 10 }
-};
-
-const theme = {
-  body: '#ffffff',
-  query: 'dark'
-};
-
-// 2. React component usage - should be IGNORED  
-function MyComponent() {
-  return (
-    <div style={styles.body}>
-      <p>Theme body: {theme.body}</p>
-      <span className={config.query}>Config query</span>
-    </div>
-  );
-}
-
-// 3. HTTP request handlers - should be FLAGGED
-function handleUserData(req, res) {
-  const userData = req.body;        // Should be flagged
-  const searchParams = req.query;   // Should be flagged
-  
-  // Direct usage without validation
-  return processUser(userData, searchParams);
-}
-
-// 4. With validation - should be IGNORED
-function handleValidatedData(req, res) {
-  const schema = require('joi');
-  const { error, value } = schema.validate(req.body);
-  if (error) {
-    return res.status(400).json({ error: error.details });
-  }
-  
-  // This is validated, should not be flagged
-  return processUser(value);
-}

package/scripts/test-scripts/test-rules-on-real-projects.js

@@ -1,86 +0,0 @@
-const HeuristicEngine = require('./engines/heuristic-engine.js');
-
-async function testRuleOnRealProjects(ruleId, ruleName) {
-  console.log(`\n🔍 Testing ${ruleId} - ${ruleName}`);
-  console.log('=' .repeat(50));
-  
-  const engine = new HeuristicEngine();
-  await engine.initialize({ verbose: false });
-  
-  // Test files from both BE and FE projects
-  const testFiles = [
-    // Backend files
-    'examples/project-samples/replace-be/lambda/refpl-dev-cognito-custom-message-lambda/app.ts',
-    'examples/project-samples/replace-be/lambda/refpl-dev-cognito-custom-message-lambda/common/utils.ts',
-    'examples/project-samples/replace-be/lambda/refpl-dev-cognito-custom-message-lambda/common/mailTemplates.ts',
-    
-    // Frontend files  
-    'examples/project-samples/replace-fe/.storybook/preview.tsx',
-    'examples/project-samples/replace-fe/s055-simple.ts'
-  ];
-  
-  let totalViolations = 0;
-  
-  for (const file of testFiles) {
-    try {
-      const result = await engine.analyze([file], [{ id: ruleId }], { verbose: false });
-      const violations = result.results[0]?.violations || [];
-      
-      if (violations.length > 0) {
-        console.log(`\n📁 ${file}:`);
-        console.log(`   ${violations.length} violation(s) found`);
-        violations.forEach(v => {
-          console.log(`   ⚠️  Line ${v.line}: ${v.message}`);
-          if (v.code) console.log(`       Code: ${v.code.trim()}`);
-        });
-      } else {
-        console.log(`✅ ${file}: No violations`);
-      }
-      
-      totalViolations += violations.length;
-    } catch (error) {
-      console.log(`❌ ${file}: Error - ${error.message}`);
-    }
-  }
-  
-  console.log(`\n📊 Total violations for ${ruleId}: ${totalViolations}`);
-  return totalViolations;
-}
-
-async function main() {
-  console.log('🚀 Testing recently improved rules on real projects\n');
-  
-  const rulesToTest = [
-    { id: 'S027', name: 'No Hardcoded Secrets' },
-    { id: 'S026', name: 'JSON Schema Validation' },
-    { id: 'C047', name: 'No Duplicate Retry Logic' },
-    { id: 'C013', name: 'No Dead Code' },
-    { id: 'C042', name: 'Boolean Name Prefix' }
-  ];
-  
-  const results = {};
-  
-  for (const rule of rulesToTest) {
-    try {
-      const violations = await testRuleOnRealProjects(rule.id, rule.name);
-      results[rule.id] = violations;
-    } catch (error) {
-      console.log(`❌ Failed to test ${rule.id}: ${error.message}`);
-      results[rule.id] = 'ERROR';
-    }
-  }
-  
-  console.log('\n🎯 SUMMARY');
-  console.log('=' .repeat(50));
-  Object.entries(results).forEach(([ruleId, violations]) => {
-    if (violations === 'ERROR') {
-      console.log(`❌ ${ruleId}: Test failed`);
-    } else if (violations === 0) {
-      console.log(`✅ ${ruleId}: No violations (Good!)`);;
-    } else {
-      console.log(`⚠️  ${ruleId}: ${violations} violations (Review needed)`);
-    }
-  });
-}
-
-main().catch(console.error);