@entro314labs/markdownfix 0.0.15 → 0.0.20

package/ESLINT_INTEGRATION.md

@@ -234,7 +234,7 @@ See [.remarkrc.js](./.remarkrc.js) for full configuration.
 
 ```javascript
 function example() {
-  var unused = "bad";
+  const unused = "bad";
   return "hello"
 }
 ```

package/README.md

@@ -179,12 +179,11 @@ markdownfix setup              # Create example content structure
 
 ### 🚀 Nuclear Mode
 
-The `nuclear` command runs a comprehensive 4-step workflow that applies **all** available linters and fixers:
+The `nuclear` command runs a comprehensive 3-step workflow that applies **all** available linters and fixers:
 
 1. **Remark Formatting** - Auto-fix markdown syntax
-2. **Remark Linting** - Validate markdown rules (40+ rules)
-3. **ESLint Auto-fix** - Fix JavaScript/JSX in code blocks
-4. **ESLint Linting** - Validate code quality
+2. **ESLint Auto-fix** - Fix JavaScript/JSX in code blocks
+3. **Final Validation** - Report any remaining issues
 
 **Perfect for:**
 
@@ -210,36 +209,73 @@ markdownfix nuclear --glob "docs/**/*.{md,mdx}"
 }
 ```
 
-**Example Output:**
+**Example Output (Success):**
 
 ```
 🚀 NUCLEAR MODE ACTIVATED
 
 Processing 15 file(s) with comprehensive workflow...
 
-Step 1/4: Running remark formatting...
-  ✓ Remark formatting completed
+Step 1/3: Running remark formatting...
+  ✓ Formatted 15 file(s)
 
-Step 2/4: Running remark linting...
-  ✓ Remark linting passed
-
-Step 3/4: Running ESLint auto-fix...
+Step 2/3: Running ESLint auto-fix on code blocks...
   ✓ ESLint auto-fix completed
 
-Step 4/4: Running ESLint linting...
-  ✓ ESLint linting passed
+Step 3/3: Running final validation...
+  ✓ All validation checks passed
 
 ════════════════════════════════════════════════════════════
 NUCLEAR MODE SUMMARY
 ════════════════════════════════════════════════════════════
-✓ Remark Format        PASS   Formatted 15/15 files
-✓ Remark Lint          PASS   Linted 15 files
-✓ ESLint Fix           PASS   Auto-fixed code blocks
-✓ ESLint Lint          PASS   All code blocks valid
+✓ Remark Format        PASS             Formatted 15/15 files
+✓ ESLint Fix           PASS             Fixed code blocks
+✓ Validation           PASS             All checks passed
 ════════════════════════════════════════════════════════════
 🎉 All checks passed! Your markdown is pristine.
 ```
 
+**Example Output (Issues Found):**
+
+````
+🚀 NUCLEAR MODE ACTIVATED
+
+Processing 8 file(s) with comprehensive workflow...
+
+Step 1/3: Running remark formatting...
+  ✓ Formatted 8 file(s)
+  ℹ️  Some issues require manual fixes
+
+Step 2/3: Running ESLint auto-fix on code blocks...
+  ⚠️  ESLint found issues that need manual fixes
+
+Step 3/3: Running final validation...
+  ⚠️  Validation found remaining issues
+
+════════════════════════════════════════════════════════════
+REMAINING ISSUES
+════════════════════════════════════════════════════════════
+file.md
+  3:128  error  Line too long (max 80 chars)
+  17:1   error  Missing code block language flag
+
+════════════════════════════════════════════════════════════
+NUCLEAR MODE SUMMARY
+════════════════════════════════════════════════════════════
+✓ Remark Format        PASS             Formatted 8/8 files
+⚠️ ESLint Fix           NEEDS ATTENTION  Some issues remain
+⚠️ Validation           NEEDS ATTENTION  Issues found
+════════════════════════════════════════════════════════════
+
+📋 NEXT STEPS:
+   Review the issues above and fix them manually.
+   Common fixes:
+   • Shorten long lines (max 80 chars)
+   • Add language flags to code blocks (```js, ```bash, etc.)
+   • Fix filename issues (use lowercase, avoid special chars)
+   • Add blank lines between list items
+````
+
 ### Command Aliases
 
 The CLI is available via two commands:

package/cli.js

@@ -22,15 +22,28 @@ import { execSync } from 'child_process';
 const MARKDOWN_EXTENSIONS = ['md', 'mdx', 'mdc', 'mdd'];
 
 // Import configuration from .remarkrc.js
+// Suppress console output during import to avoid MDD warnings in nuclear mode
+const originalLog = console.log;
+const originalInfo = console.info;
+console.log = () => {};
+console.info = () => {};
+
 let remarkConfig;
 try {
   const configModule = await import(path.join(process.cwd(), '.remarkrc.js'));
   remarkConfig = configModule.default;
 } catch (e) {
+  // Restore console before warning
+  console.log = originalLog;
+  console.info = originalInfo;
   console.warn('⚠️  No .remarkrc.js found in current directory, using default config');
   remarkConfig = null;
 }
 
+// Restore console
+console.log = originalLog;
+console.info = originalInfo;
+
 /**
  * Show help text
  */
@@ -79,9 +92,8 @@ EXAMPLES:
 NUCLEAR MODE:
   The nuclear command runs a comprehensive fix workflow:
   1. Remark formatting (auto-fix markdown syntax)
-  2. Remark linting (validate markdown rules)
-  3. ESLint auto-fix (fix JavaScript in code blocks)
-  4. ESLint linting (validate code quality)
+  2. ESLint auto-fix (fix JavaScript/JSX in code blocks)
+  3. Final validation (report remaining issues)
 
   Perfect for: CI/CD, pre-commit hooks, major cleanups
 
@@ -188,34 +200,53 @@ async function processFiles(files, options = {}) {
 
 /**
  * Run nuclear mode - comprehensive fix workflow
- * Executes: remark format -> remark lint -> eslint fix -> eslint lint
+ * Executes: remark format -> eslint fix -> final validation
  */
 async function runNuclearMode(files, options = {}) {
   const { quiet = false } = options;
   const hasEslint = await checkEslintAvailable();
 
+  // Filter out ESLint-ignored files in nuclear mode
+  let filteredFiles = files;
+  let ignoredCount = 0;
+  if (hasEslint) {
+    const ignorePatterns = await getEslintIgnorePatterns();
+    const originalCount = files.length;
+    filteredFiles = files.filter(file => !isFileIgnored(file, ignorePatterns));
+    ignoredCount = originalCount - filteredFiles.length;
+  }
+
   if (!quiet) {
     console.log('\n🚀 NUCLEAR MODE ACTIVATED\n');
-    console.log(`Processing ${files.length} file(s) with comprehensive workflow...\n`);
+    if (ignoredCount > 0) {
+      console.log(`Found ${files.length} file(s), processing ${filteredFiles.length} (${ignoredCount} ignored by ESLint config)\n`);
+    } else {
+      console.log(`Processing ${filteredFiles.length} file(s) with comprehensive workflow...\n`);
+    }
   }
 
   let overallSuccess = true;
   const steps = [];
+  let remarkIssues = [];
+  let eslintIssues = [];
 
-  // Step 1: Remark formatting
-  if (!quiet) console.log('Step 1/4: Running remark formatting...');
+  // Step 1: Remark formatting (auto-fixes what it can)
+  if (!quiet) console.log('Step 1/3: Running remark formatting...');
   try {
-    const result = await processFiles(files, { write: true, quiet: true });
+    const result = await processFiles(filteredFiles, { write: true, quiet: true });
+    const formatted = result.processedCount;
+    const withIssues = result.hasErrors;
+    
     steps.push({
       name: 'Remark Format',
-      success: !result.hasErrors,
-      details: `Wrote ${result.processedCount}/${result.totalFiles} files`
+      success: formatted > 0,
+      details: `Formatted ${formatted}/${result.totalFiles} files`
     });
-    if (result.hasErrors) overallSuccess = false;
+    
     if (!quiet) {
-      console.log(`  ✓ Remark formatting completed (${result.processedCount} files written)`);
-      if (result.hasErrors) {
-        console.log(`  ⚠️  Some files had lint warnings (but were still formatted)\n`);
+      console.log(`  ✓ Formatted ${formatted} file(s)`);
+      if (withIssues) {
+        console.log(`  ℹ️  Some issues require manual fixes\n`);
       } else {
         console.log();
       }
@@ -226,87 +257,94 @@ async function runNuclearMode(files, options = {}) {
     if (!quiet) console.log(`  ✗ Remark formatting failed: ${error.message}\n`);
   }
 
-  // Step 2: Remark linting
-  if (!quiet) console.log('Step 2/4: Running remark linting...');
-  try {
-    const result = await processFiles(files, { lintOnly: true, quiet: true });
-    steps.push({
-      name: 'Remark Lint',
-      success: !result.hasErrors,
-      details: `Linted ${result.totalFiles} files`
-    });
-    if (result.hasErrors) {
-      if (!quiet) console.log('  ⚠️  Remark linting found issues (check output above)\n');
-      overallSuccess = false;
-    } else {
-      if (!quiet) console.log(`  ✓ Remark linting passed\n`);
-    }
-  } catch (error) {
-    steps.push({ name: 'Remark Lint', success: false, details: error.message });
-    overallSuccess = false;
-    if (!quiet) console.log(`  ✗ Remark linting failed: ${error.message}\n`);
-  }
 
-  // Step 3 & 4: ESLint (only if ESLint is available)
+  // Step 2: ESLint auto-fix (only if ESLint is available)
   if (hasEslint) {
     const eslintConfigInfo = await getEslintConfigPath();
     const { configPath, source } = eslintConfigInfo;
 
-    if (!quiet && source === 'bundled') {
-      console.log('  ℹ️  Using bundled ESLint config (no local config found)\n');
-    }
-
-    // Step 3: ESLint auto-fix
-    if (!quiet) console.log('Step 3/4: Running ESLint auto-fix...');
+    if (!quiet) console.log('Step 2/3: Running ESLint auto-fix on code blocks...');
     try {
-      // Quote each file path to handle spaces and special characters
-      const fileList = files.map(f => `"${f}"`).join(' ');
+      const fileList = filteredFiles.map(f => `"${f}"`).join(' ');
       const eslintCmd = `npx eslint --config "${configPath}" --fix ${fileList}`;
 
       try {
-        execSync(eslintCmd, {
-          stdio: quiet ? 'pipe' : 'inherit',
+        const result = execSync(eslintCmd, {
+          stdio: 'pipe',
           cwd: process.cwd()
         });
-        steps.push({ name: 'ESLint Fix', success: true, details: 'Auto-fixed code blocks' });
+        steps.push({ name: 'ESLint Fix', success: true, details: 'Fixed code blocks' });
         if (!quiet) console.log(`  ✓ ESLint auto-fix completed\n`);
       } catch (eslintError) {
-        // ESLint returns non-zero even if it fixes issues
-        steps.push({ name: 'ESLint Fix', success: false, details: 'Some issues could not be auto-fixed' });
-        if (!quiet) console.log(`  ⚠️  ESLint auto-fix completed with warnings\n`);
+        // ESLint returns non-zero if there are unfixable issues
+        const output = eslintError.stdout?.toString() || eslintError.stderr?.toString() || '';
+        
+        // Filter out "File ignored" warnings - these are expected
+        const lines = output.split('\n').filter(line => 
+          !line.includes('File ignored because of a matching ignore pattern')
+        );
+        const filteredOutput = lines.join('\n');
+        
+        const hasWarnings = filteredOutput.includes('warning');
+        const hasErrors = filteredOutput.includes('error');
+        
+        if (hasErrors || hasWarnings) {
+          steps.push({ name: 'ESLint Fix', success: false, details: 'Some issues remain' });
+          if (!quiet) console.log(`  ⚠️  ESLint found issues that need manual fixes\n`);
+        } else {
+          steps.push({ name: 'ESLint Fix', success: true, details: 'Fixed code blocks' });
+          if (!quiet) console.log(`  ✓ ESLint auto-fix completed\n`);
+        }
       }
     } catch (error) {
       steps.push({ name: 'ESLint Fix', success: false, details: error.message });
       if (!quiet) console.log(`  ✗ ESLint auto-fix failed: ${error.message}\n`);
     }
+  } else {
+    if (!quiet) {
+      console.log('Step 2/3: Skipping ESLint (not installed)');
+      console.log('  ℹ️  Install with: npm install -D eslint eslint-plugin-mdx\n');
+    }
+    steps.push({ name: 'ESLint Fix', success: true, details: 'Skipped (not installed)' });
+  }
 
-    // Step 4: ESLint linting
-    if (!quiet) console.log('Step 4/4: Running ESLint linting...');
-    try {
-      // Quote each file path to handle spaces and special characters
-      const fileList = files.map(f => `"${f}"`).join(' ');
-      const eslintCmd = `npx eslint --config "${configPath}" ${fileList}`;
-
-      execSync(eslintCmd, {
-        stdio: quiet ? 'pipe' : 'inherit',
-        cwd: process.cwd()
-      });
-      steps.push({ name: 'ESLint Lint', success: true, details: 'All code blocks valid' });
-      if (!quiet) console.log(`  ✓ ESLint linting passed\n`);
-    } catch (eslintError) {
-      steps.push({ name: 'ESLint Lint', success: false, details: 'Linting issues found' });
+  // Step 3: Final validation
+  if (!quiet) console.log('Step 3/3: Running final validation...');
+  let validationResult;
+  try {
+    validationResult = await processFiles(filteredFiles, { lintOnly: true, quiet: true });
+    const hasIssues = validationResult.hasErrors;
+    
+    if (hasIssues) {
       overallSuccess = false;
-      if (!quiet) console.log(`  ⚠️  ESLint linting found issues\n`);
     }
-  } else {
-    // ESLint not installed
+    
+    steps.push({
+      name: 'Validation',
+      success: !hasIssues,
+      details: hasIssues ? 'Issues found' : 'All checks passed'
+    });
+    
     if (!quiet) {
-      console.log('Step 3/4: Skipping ESLint (not installed)');
-      console.log('  ℹ️  Install ESLint with: npm install -D eslint eslint-plugin-mdx\n');
-      console.log('Step 4/4: Skipping ESLint linting\n');
+      if (hasIssues) {
+        console.log(`  ⚠️  Validation found remaining issues\n`);
+      } else {
+        console.log(`  ✓ All validation checks passed\n`);
+      }
     }
-    steps.push({ name: 'ESLint Fix', success: true, details: 'Skipped (not installed)' });
-    steps.push({ name: 'ESLint Lint', success: true, details: 'Skipped (not installed)' });
+  } catch (error) {
+    steps.push({ name: 'Validation', success: false, details: error.message });
+    overallSuccess = false;
+    if (!quiet) console.log(`  ✗ Validation failed: ${error.message}\n`);
+  }
+
+  // Show detailed issues if validation failed
+  if (!quiet && validationResult?.hasErrors) {
+    console.log('═'.repeat(60));
+    console.log('REMAINING ISSUES');
+    console.log('═'.repeat(60));
+    await processFiles(filteredFiles, { lintOnly: true, quiet: false });
+    console.log();
   }
 
   // Summary
@@ -315,15 +353,22 @@ async function runNuclearMode(files, options = {}) {
     console.log('NUCLEAR MODE SUMMARY');
     console.log('═'.repeat(60));
     steps.forEach(step => {
-      const icon = step.success ? '✓' : '✗';
-      const status = step.success ? 'PASS' : 'FAIL';
-      console.log(`${icon} ${step.name.padEnd(20)} ${status.padEnd(6)} ${step.details}`);
+      const icon = step.success ? '✓' : '⚠️';
+      const status = step.success ? 'PASS' : 'NEEDS ATTENTION';
+      console.log(`${icon} ${step.name.padEnd(20)} ${status.padEnd(16)} ${step.details}`);
     });
     console.log('═'.repeat(60));
+    
     if (overallSuccess) {
       console.log('🎉 All checks passed! Your markdown is pristine.\n');
     } else {
-      console.log('⚠️  Some issues remain. Review the output above.\n');
+      console.log('\n📋 NEXT STEPS:');
+      console.log('   Review the issues above and fix them manually.');
+      console.log('   Common fixes:');
+      console.log('   • Shorten long lines (max 80 chars)');
+      console.log('   • Add language flags to code blocks (```js, ```bash, etc.)');
+      console.log('   • Fix filename issues (use lowercase, avoid special chars)');
+      console.log('   • Add blank lines between list items\n');
     }
   }
 
@@ -359,6 +404,60 @@ async function getEslintConfigPath() {
     const bundledConfig = new URL('./eslint.config.js', import.meta.url).pathname;
     return { configPath: bundledConfig, source: 'bundled' };
   }
+}
+
+/**
+ * Get ESLint ignore patterns from config
+ * Returns array of ignore patterns to filter files in nuclear mode
+ * Note: We parse the config file as text instead of importing it because
+ * the config has dependencies that require ESLint to be in the require cache
+ */
+async function getEslintIgnorePatterns() {
+  try {
+    const { configPath } = await getEslintConfigPath();
+    const configContent = await fs.readFile(configPath, 'utf-8');
+    
+    // Extract the ignores array using regex
+    // Look for: ignores: [ ... ]
+    const ignoresMatch = configContent.match(/ignores:\s*\[([\s\S]*?)\]/);
+    if (!ignoresMatch) return [];
+    
+    const ignoresContent = ignoresMatch[1];
+    
+    // Extract quoted strings from the array
+    const patterns = [];
+    const stringMatches = ignoresContent.matchAll(/['"]([^'"]+)['"]/g);
+    for (const match of stringMatches) {
+      patterns.push(match[1]);
+    }
+    
+    return patterns;
+  } catch (error) {
+    return [];
+  }
+}
+
+/**
+ * Check if a file matches any of the ignore patterns
+ */
+function isFileIgnored(filePath, ignorePatterns) {
+  // Convert ignore patterns to regex-like matching
+  for (const pattern of ignorePatterns) {
+    // Handle glob patterns
+    if (pattern.includes('**')) {
+      // **/*.config.js -> matches any .config.js file
+      const regex = new RegExp(pattern.replace(/\*\*/g, '.*').replace(/\*/g, '[^/]*').replace(/\./g, '\\.'));
+      if (regex.test(filePath)) return true;
+    } else if (pattern.includes('*')) {
+      // *.js -> matches .js files in root
+      const regex = new RegExp('^' + pattern.replace(/\*/g, '[^/]*').replace(/\./g, '\\.') + '$');
+      if (regex.test(filePath)) return true;
+    } else {
+      // Exact match or ends with pattern
+      if (filePath === pattern || filePath.endsWith('/' + pattern)) return true;
+    }
+  }
+  return false;
 }/**
  * Initialize .remarkrc.js configuration
  */

package/eslint.config.js

@@ -71,12 +71,22 @@ export default [
     },
     rules: {
       ...mdxPlugin.flatCodeBlocks.rules,
-      // Add JavaScript rules for code blocks
+      // Rules that make sense for documentation examples
       'no-var': 'error',
       'prefer-const': 'error',
-      'no-unused-vars': 'warn',
-      // Allow common globals in documentation examples
-      'no-undef': 'off',
+
+      // Disable rules that don't make sense for isolated code examples
+      'no-unused-vars': 'off',        // Examples often show declarations without usage
+      'no-undef': 'off',               // Examples may reference external variables/functions
+      'no-console': 'off',             // Console logs are common in examples
+      'no-unreachable': 'off',         // Examples may show code patterns, not full logic
+      'no-constant-condition': 'off',  // Examples may use simplified conditions
+      'no-empty': 'off',               // Empty blocks may be placeholders in examples
+      'prefer-rest-params': 'off',     // Examples may show older patterns
+      'prefer-spread': 'off',          // Examples may demonstrate various approaches
+      '@typescript-eslint/no-unused-vars': 'off',  // TypeScript variant
+      'import/no-unresolved': 'off',   // Examples won't have real imports
+      'no-redeclare': 'off',           // Multiple examples might reuse variable names
     }
   }
 ];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@entro314labs/markdownfix",
-  "version": "0.0.15",
+  "version": "0.0.20",
   "description": "Opinionated markdown formatter and linter for MD, MDX, MDC, and MDD files using Remark/Unified ecosystem",
   "type": "module",
   "repository": {
@@ -45,27 +45,27 @@
   "author": "Dominikos Pritis",
   "license": "MIT",
   "devDependencies": {
-    "markdown-link-check": "^3.14.1",
+    "markdown-link-check": "^3.14.2",
     "remark-cli": "^12.0.1"
   },
   "engines": {
     "node": ">=22.x"
   },
   "dependencies": {
-    "@adobe/remark-gridtables": "^3.0.15",
+    "@adobe/remark-gridtables": "^3.0.16",
     "@code-dot-org/remark-plugins": "^2.0.1",
-    "@entro314labs/remark-mdd": "^0.0.15",
+    "@entro314labs/remark-mdd": "^0.0.16",
     "@theguild/remark-mermaid": "^0.3.0",
-    "eslint": "^9.39.1",
+    "eslint": "^9.39.2",
     "eslint-plugin-mdx": "^3.6.2",
     "eslint-plugin-react": "^7.37.5",
     "fumadocs-docgen": "^3.0.4",
-    "glob": "^11.0.3",
-    "js-yaml": "^4.1.0",
+    "glob": "^13.0.0",
+    "js-yaml": "^4.1.1",
     "rehype-remark": "^10.0.1",
-    "remark": "^15.0.1",
+    "remark": "latest",
     "remark-directive": "^4.0.0",
-    "remark-docx": "^0.2.0",
+    "remark-docx": "^0.3.19",
     "remark-emoji": "^5.0.2",
     "remark-frontmatter": "^5.0.0",
     "remark-gfm": "^4.0.1",
@@ -99,7 +99,7 @@
     "remark-lint-no-empty-sections": "^4.0.0",
     "remark-lint-no-heading-indent": "^5.0.1",
     "remark-lint-no-heading-punctuation": "^4.0.1",
-    "remark-lint-no-trailing-spaces": "^3.0.2",
+    "remark-lint-no-trailing-spaces": "^4.0.3",
     "remark-lint-ordered-list-marker-style": "^4.0.1",
     "remark-lint-ordered-list-marker-value": "^4.0.1",
     "remark-lint-strong-marker": "^4.0.1",
@@ -107,7 +107,7 @@
     "remark-lint-table-pipe-alignment": "^4.1.1",
     "remark-lint-table-pipes": "^5.0.1",
     "remark-lint-unordered-list-marker-style": "^4.0.1",
-    "remark-mdc": "^3.8.1",
+    "remark-mdc": "^3.10.0",
     "remark-mdx": "^3.1.1",
     "remark-mdx-frontmatter": "^5.2.0",
     "remark-preset-lint-consistent": "^6.0.1",