tz-clean 2.1.0 → 2.3.0

package/.prettierignore

@@ -0,0 +1,19 @@
+# Minified files
+**/*.min.js
+**/*.min.cjs
+**/*.min.mjs
+**/*.bundle.js
+
+# Build output
+dist/
+build/
+vendor/
+libs/
+
+# Package manager
+node_modules/
+
+# Lock files
+package-lock.json
+yarn.lock
+pnpm-lock.yaml

package/eslint.config.mjs

@@ -1,5 +1,6 @@
 import js from '@eslint/js';
 import jsdoc from 'eslint-plugin-jsdoc';
+import noComments from 'eslint-plugin-no-comments';
 import perfectionist from 'eslint-plugin-perfectionist';
 import projectStructure from 'eslint-plugin-project-structure';
 import sonarjs from 'eslint-plugin-sonarjs';
@@ -25,6 +26,9 @@ export default [
   jsdoc.configs['flat/recommended'],
   perfectionist.configs['recommended-natural'],
   {
+    linterOptions: {
+      reportUnusedDisableDirectives: 'error',
+    },
     languageOptions: {
       ecmaVersion: 2022,
       globals: {
@@ -39,6 +43,7 @@ export default [
       sourceType: 'module',
     },
     plugins: {
+      'no-comments': noComments,
       'project-structure': projectStructure,
     },
     rules: {
@@ -54,6 +59,8 @@ export default [
       'jsdoc/require-jsdoc': 'warn',
       'jsdoc/require-param': 'warn',
       'jsdoc/require-returns': 'warn',
+      'no-comments/disallowComments': 'warn',
+      'no-multiple-empty-lines': ['error', { max: 1, maxBOF: 0, maxEOF: 0 }],
       'no-redeclare': 'off',
       'no-undef': 'off',
       'no-unused-vars': ['error', { args: 'none', vars: 'local' }],

package/index.js

@@ -140,6 +140,8 @@ async function run() {
 
   try {
     const fixedFiles = runPrettier ? runPrettierTools(includePaths, excludePaths) : [];
+    const strippedFiles = stripCommentsAndEmptyLines(includePaths, excludePaths);
+    const allFixedFiles = [...new Set([...fixedFiles, ...strippedFiles])];
     const typecheckResult = runTypecheck ? runTypecheckTools() : { error: null, ran: false };
     const reportData = runEslint ? runEslintTools(includePaths, excludePaths) : [];
     const { totalErrors, totalWarnings } = calculateTotals(reportData);
@@ -163,10 +165,10 @@ async function run() {
     }
 
     if (response.reportType === 'terminal') {
-      printTerminalSummary(reportData, fixedFiles, totalErrors, totalWarnings, typecheckResult);
+      printTerminalSummary(reportData, allFixedFiles, totalErrors, totalWarnings, typecheckResult);
     } else if (response.reportType === 'ui') {
       console.log('⏳ Generating UI Dashboard...');
-      const outputPath = generateHtmlReport(reportData, fixedFiles);
+      const outputPath = generateHtmlReport(reportData, allFixedFiles);
       console.log(`🚀 Opening report in browser: ${outputPath}`);
       await open(outputPath);
     } else if (response.reportType === 'hosted') {
@@ -174,7 +176,7 @@ async function run() {
       const templatePath = path.join(configDir, 'report-template.html');
       let html = fs.readFileSync(templatePath, 'utf-8');
       html = html.replace('/* REPORT_DATA_PLACEHOLDER */ null', JSON.stringify(reportData));
-      html = html.replace('/* FIXED_FILES_PLACEHOLDER */ []', JSON.stringify(fixedFiles));
+      html = html.replace('/* FIXED_FILES_PLACEHOLDER */ []', JSON.stringify(allFixedFiles));
       serveHostedReport(html);
     }
   } catch (globalError) {
@@ -183,6 +185,73 @@ async function run() {
   }
 }
 
+/**
+ * Strips all comments and excess blank lines from JS/TS files in the target paths.
+ * Removes single-line comments (//), block comments (/* *\/), eslint-disable directives,
+ * and collapses multiple consecutive blank lines into one.
+ * @param {string[]} includePaths - Paths to process (defaults to targetDir).
+ * @param {string[]} excludePaths - Paths to skip.
+ * @returns {string[]} List of files that were modified.
+ */
+function stripCommentsAndEmptyLines(includePaths = [], excludePaths = []) {
+  const extensions = ['.js', '.ts', '.jsx', '.tsx', '.mjs', '.cjs'];
+  const modifiedFiles = [];
+
+  const defaultIgnorePatterns = [
+    /node_modules/,
+    /\.min\.(js|cjs|mjs)$/,
+    /\.bundle\.js$/,
+    /[/\\](dist|build|vendor|libs)[/\\]/,
+  ];
+
+  const userIgnorePatterns = excludePaths.map((p) => new RegExp(p.replace(/\//g, '[/\\\\]')));
+  const allIgnorePatterns = [...defaultIgnorePatterns, ...userIgnorePatterns];
+
+  const scanDir = (dir) => {
+    let entries;
+    try {
+      entries = fs.readdirSync(dir, { withFileTypes: true });
+    } catch {
+      return;
+    }
+    for (const entry of entries) {
+      const fullPath = path.join(dir, entry.name);
+      if (allIgnorePatterns.some((re) => re.test(fullPath))) continue;
+      if (entry.isDirectory()) {
+        scanDir(fullPath);
+      } else if (entry.isFile() && extensions.includes(path.extname(entry.name))) {
+        const original = fs.readFileSync(fullPath, 'utf-8');
+        let cleaned = original;
+
+        // Remove block comments /** */ and /* */ (but not shebang)
+        cleaned = cleaned.replace(/\/\*[\s\S]*?\*\//g, '');
+
+        // Remove single-line comments // (including eslint-disable)
+        cleaned = cleaned.replace(/[ \t]*\/\/.*$/gm, '');
+
+        // Collapse multiple blank lines into one
+        cleaned = cleaned.replace(/\n{3,}/g, '\n\n');
+
+        // Remove blank lines at start of file
+        cleaned = cleaned.replace(/^\n+/, '');
+
+        // Ensure single newline at end of file
+        cleaned = cleaned.trimEnd() + '\n';
+
+        if (cleaned !== original) {
+          fs.writeFileSync(fullPath, cleaned, 'utf-8');
+          modifiedFiles.push(path.relative(targetDir, fullPath));
+        }
+      }
+    }
+  };
+
+  const targets = includePaths.length > 0 ? includePaths.map((p) => path.resolve(targetDir, p)) : [targetDir];
+  targets.forEach((t) => scanDir(t));
+
+  return modifiedFiles;
+}
+
 /**
  * Executes ESLint to analyze code.
  * @param {string[]} includePaths - Paths to include in analysis.
@@ -229,11 +298,19 @@ function runPrettierTools(includePaths = [], excludePaths = []) {
   let fixedFiles = [];
 
   const targets = includePaths.length > 0 ? includePaths.map((p) => `"${p}"`).join(' ') : '.';
-  const ignoreFlags = excludePaths.map((p) => `--ignore-path-without-warn "${p}"`).join(' ');
+
+  // Build a temporary .prettierignore combining the default one + user excludes
+  const defaultIgnorePath = path.join(configDir, '.prettierignore');
+  let ignoreContent = fs.readFileSync(defaultIgnorePath, 'utf-8');
+  if (excludePaths.length > 0) {
+    ignoreContent += '\n# User-specified excludes\n' + excludePaths.join('\n') + '\n';
+  }
+  const tempIgnorePath = path.join(targetDir, '.tz-clean-prettierignore');
+  fs.writeFileSync(tempIgnorePath, ignoreContent);
 
   try {
     const prettierOutput = execSync(
-      `npx prettier --write ${targets} --config "${path.join(configDir, '.prettierrc')}" --ignore-unknown ${ignoreFlags}`,
+      `npx prettier --write ${targets} --config "${path.join(configDir, '.prettierrc')}" --ignore-path "${tempIgnorePath}" --ignore-unknown`,
       { encoding: 'utf-8' },
     );
     fixedFiles = prettierOutput
@@ -247,6 +324,8 @@ function runPrettierTools(includePaths = [], excludePaths = []) {
         .filter((line) => line.trim() && !line.includes('(unchanged)'))
         .map((line) => line.split(' ')[0]);
     }
+  } finally {
+    if (fs.existsSync(tempIgnorePath)) fs.unlinkSync(tempIgnorePath);
   }
   return fixedFiles;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "tz-clean",
-  "version": "2.1.0",
+  "version": "2.3.0",
   "description": "",
   "main": "index.js",
   "bin": {
@@ -45,6 +45,7 @@
     "@eslint/js": "^10.0.1",
     "eslint": "^10.6.0",
     "eslint-plugin-jsdoc": "^63.0.12",
+    "eslint-plugin-no-comments": "^1.2.1",
     "eslint-plugin-perfectionist": "^5.10.0",
     "eslint-plugin-project-structure": "^3.14.3",
     "eslint-plugin-sonarjs": "^4.1.0",