NodeJS压缩CLI批量处理时fs.writeFile反复写入同一文件问题

问题根因 #

• 这是Node.js异步编程里非常典型的变量作用域错误：你在循环里执行文件读写这类异步操作时，所有异步回调共享了同一个遍历变量的引用。等IO操作完成、回调实际执行的时候，循环早就跑完了，变量已经指向了遍历到的最后一个文件，所以才会出现「文件名打印正确，但最终重复压缩同一个文件、重复次数等于文件总数」的现象。
• 你之前尝试先把文件名推入数组再用for循环遍历还是复现，基本可以确定是循环变量用了var声明，或者没有给异步回调做独立的作用域隔离。

修复方案 #

直接把异步逻辑改成async/await配合for...of遍历，从根源规避作用域问题，参考实现如下：

const fs = require('fs/promises');
const path = require('path');
const { minify } = require('html-minifier');

// 执行all命令时调用这个方法即可
async function compressAllFiles() {
  // 先自动创建输出目录，避免第一次运行报目录不存在错误
  await fs.mkdir('./minified', { recursive: true });
  // 读取源目录下所有文件
  const fileList = await fs.readdir('./files');

  for (const fileName of fileList) {
    // 过滤非html/css/js文件，避免处理到隐藏文件、子目录
    const ext = path.extname(fileName);
    if (!['.html', '.css', '.js'].includes(ext)) continue;
    
    console.log(`正在压缩：${fileName}`);
    // 读取源文件内容
    const sourceContent = await fs.readFile(path.join('./files', fileName), 'utf-8');
    // 这里直接复用你单文件压缩时已经调试好的压缩配置即可
    const minifyConfig = {
      collapseWhitespace: true,
      removeComments: true,
      minifyCSS: true,
      minifyJS: true
    };
    const compressedContent = minify(sourceContent, minifyConfig);
    // 写入压缩后的文件到目标目录
    await fs.writeFile(path.join('./minified', fileName), compressedContent);
  }

  console.log('所有文件压缩完成，已输出到/minified目录');
}

compressAllFiles().catch(err => console.error('压缩执行出错：', err));

额外注意点 #

• 如果你不想改async/await写法，只想修复原有逻辑，只需要把普通for循环里声明变量的var改成let即可：let是块级作用域，每次循环会生成独立的变量绑定，回调拿到的就是对应轮次的文件名，不会出现变量共享问题。
• 拼接文件路径统一用path.join()，不要手动写字符串拼接，避免Windows/macOS系统路径分隔符不兼容的问题。
• 工具依赖安装命令保持npm install html-minifier即可，不需要额外安装其他包。

你对照自己的异常代码看，只要是循环内异步操作拿不到对应轮次的变量值，全是这类作用域绑定错误，和你有没有提前把文件名存到数组里没有关系。

内容的提问来源于stack exchange，提问作者user18976096