使用Promise.all循环搭配多个await时出现Promise循环链错误求助

你的问题核心是用async函数作为new Promise的执行器——这是冗余且易引发异常的写法：async函数本身就会返回Promise，手动套一层new Promise会导致Promise链嵌套混乱，进而触发循环链错误。

修正方案 #

直接把map的回调改成async函数，不需要手动创建Promise，逻辑完全不变，但写法更简洁且避免错误：

const run = async () => {
    let res = await db.query(query);
    let items = res.rows;

    const promises = items.map(async (item, i) => {
        try {
            const product = await stripe.products.create(...);
            const session = await stripe.checkout.sessions.create(...);
            const price = await stripe.prices.create(...);

            await saveInvoice(...);
            let user = await someF(...);
            await someF2(...); // 注意原代码末尾多了个逗号，需移除
            return item; // async函数返回值会自动作为Promise的resolve值
        } catch (error) {
            console.log(error);
            process.exit(1); // 加错误码更规范
        }
    });

    await Promise.all(promises);
    process.exit();
}
run();

关键改进点 #

1. 移除冗余的new Promise：async函数天然返回Promise，不需要手动包装，直接return你要传递的值即可。
2. 错误处理位置修正：原来的try/catch在map外层，只能捕获map同步执行时的错误；现在放在async回调内部，才能捕获每个异步操作的错误。
3. 修复语法小问题：原代码中await someF2(..,..,..,);末尾多了逗号，会导致语法错误，记得删除。

原写法触发错误的原因 #

当你用async函数作为Promise的executor时，这个executor本身会返回一个Promise，而Promise内部的await操作如果和外部手动创建的Promise形成隐式循环引用（比如Stripe API内部的Promise和你手动创建的Promise意外关联），就会被引擎检测为循环链，抛出错误。改用async函数直接返回Promise的写法，就不会有这种嵌套混乱的问题。

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