Node.js中Promise未完成Git Clone便执行后续操作的问题求助

你的代码里第一个then方法存在两个关键问题：

1. fs.access是异步回调式API，它不会阻塞Promise链的执行——当fs.access的回调还在处理目录判断或git clone时，第一个then已经执行完毕，Promise直接跳转到下一个then调用nextFunc。
2. 第一个then没有返回任何Promise，Promise链无法感知内部异步操作的完成状态，自然不会等待git clone结束。

核心思路是：把异步回调逻辑包装成Promise，让Promise链能感知到git clone的完成状态，确保只有在clone（或目录存在判断）完成后，才执行nextFunc。

改造方案1：Promise包装回调 #

修改第一个then的逻辑，返回一个新的Promise，在git clone执行完成后手动resolve：

let myPromise = new Promise((rev, err) => {
    console.log('create promise');
    rev();
});

myPromise.then(() => {
    // 合并cd操作，简化路径切换
    shell.cd('../git_folder');
    // 用Promise包裹fs.access的异步逻辑
    return new Promise((resolve, reject) => {
        fs.access('git_repo', (error) => {
            if (error) {
                console.log("Directory does not exist.");
                // shell.exec默认是同步执行，执行完才会走到下一行
                shell.exec(`git clone the_git_repo && echo "cloned"`);
                console.log("Clone completed");
                resolve();
            } else {
                console.log("Directory exists.");
                resolve();
            }
        });
    });
}).then(() => nextFunc());

const nextFunc = () => {
    shell.cd('git_repo');
    shell.exec('git pull');
    // other git commands
}

改造方案2：用async/await简化（推荐） #

Node.js 10+支持fs.promisesAPI，可以直接用async/await替代回调，让代码逻辑更直观：

// 用async函数包裹所有异步逻辑
async function runGitTasks() {
    console.log('start git tasks');
    // 切换到目标目录
    shell.cd('../git_folder');
    
    try {
        // 检查目录是否存在，await会等待异步操作完成
        await fs.promises.access('git_repo');
        console.log("Directory exists.");
    } catch (error) {
        console.log("Directory does not exist.");
        // shell.exec同步执行，clone完成后才继续
        shell.exec(`git clone the_git_repo && echo "cloned"`);
    }
    
    // 确保前面的操作全部完成后，再执行后续Git命令
    nextFunc();
}

const nextFunc = () => {
    shell.cd('git_repo');
    shell.exec('git pull');
    // other git commands
}

// 启动任务
runGitTasks();

• ShellJS的shell.exec默认是同步执行的，只要它在Promise的resolve之前执行，就能保证clone完成后才推进Promise链；如果需要异步执行exec，则需要监听exit事件再resolve（比如shell.exec(cmd, {async:true}, (code, stdout, stderr) => { resolve(); })）。
• 所有异步操作必须被Promise包裹，或者用await等待，才能让Promise链按顺序执行。

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