嘿，我来帮你解决这个重复推送ID的问题～

问题根源分析 #

你遇到的情况，核心原因是异步操作的竞态条件加上Passport对用户对象的处理逻辑：

1. 你的代码同时触发了user.save()和newTaxReturn.save()两个异步操作，没有等待它们完成，导致Mongoose在处理文档状态时出现了重复更新。
2. 更关键的是，Passport提供的user对象可能会在请求结束时被自动序列化并保存到数据库——也就是说，你手动调用了一次user.save()，Passport又悄悄帮你调用了一次，两次保存操作叠加，就导致同一个ID被推送了两次。

而当你移除user.save()时，只有Passport自动触发的那一次保存生效，所以ID只会被推送一次（但这种方式其实是依赖Passport的隐式行为，并不安全）。

修复方案 #

方案1：用async/await确保异步操作顺序执行

把函数改成异步函数，让两个保存操作依次完成，避免竞态条件：

const createTaxReturn = async ({ user }) => { 
  const newTaxReturn = new TaxReturn({ userId: user._id });
  // 只执行一次ID推送
  user.taxReturnIds.push(newTaxReturn._id);
  
  try {
    // 先等待用户对象保存完成
    await user.save();
    // 再保存新的税单记录
    return await newTaxReturn.save();
  } catch (err) {
    // 这里可以添加具体的错误处理，比如返回错误信息
    console.error('创建税单失败:', err);
    throw err; // 抛出错误让上层逻辑处理
  }
}

这种方式能明确控制操作顺序，避免重复保存导致的问题。

方案2：使用$addToSet强制ID唯一性（更稳妥的保险方案）

如果你担心后续还会出现并发修改的问题，可以用Mongoose的$addToSet操作符，确保同一个ID无论被推送多少次，最终只会在数组中出现一次：

const createTaxReturn = async ({ user }) => { 
  const newTaxReturn = new TaxReturn({ userId: user._id });
  
  try {
    // 直接通过数据库更新添加ID，避免修改内存中的user对象
    await User.findByIdAndUpdate(user._id, {
      $addToSet: { taxReturnIds: newTaxReturn._id }
    });
    // 保存新的税单记录
    return await newTaxReturn.save();
  } catch (err) {
    console.error('创建税单失败:', err);
    throw err;
  }
}

这种方式绕过了直接修改Passport的user对象，直接操作数据库，能彻底避免因对象状态跟踪导致的重复问题。

额外提醒 #

不要忽略错误处理！你原来的代码里用.catch(() => {})吞掉了所有错误，这会让你很难排查潜在问题——建议在catch块里添加日志记录，或者把错误抛出给上层处理。

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