fix: fix deps updater commit

Author: zdm

lib/package.js

@@ -881,97 +881,96 @@ export default class Package {
                     .end();
 
                 logger.log( table.content.trim() );
-
                 logger.log( "Update dependencies ... " + res );
+            }
 
-                // commit and push
-                COMMIT: if ( commit ) {
-                    try {
+            // commit and push
+            COMMIT: if ( commit ) {
+                try {
 
-                        // get working tree status
-                        res = await this.git.getWorkingTreeStatus();
-                        if ( !res.ok ) throw res;
+                    // get working tree status
+                    res = await this.git.getWorkingTreeStatus();
+                    if ( !res.ok ) throw res;
 
-                        const commitFiles = [];
+                    const commitFiles = [];
 
-                        // working tree is dirty
-                        if ( res.data.isDirty ) {
-                            for ( const lockFile of [ "package-lock.json", "npm-shrinkwrap.json" ] ) {
-                                const lockFilePath = this.rootSlug
-                                    ? this.rootSlug + "/" + lockFile
-                                    : lockFile;
+                    // working tree is dirty
+                    if ( res.data.isDirty ) {
+                        for ( const lockFile of [ "package-lock.json", "npm-shrinkwrap.json" ] ) {
+                            const lockFilePath = this.rootSlug
+                                ? this.rootSlug + "/" + lockFile
+                                : lockFile;
 
-                                if ( res.data.files[ lockFilePath ] ) commitFiles.push( lockFile );
-                            }
+                            if ( res.data.files[ lockFilePath ] ) commitFiles.push( lockFile );
                         }
+                    }
 
-                        // dependencies locks was not updated
-                        if ( !commitFiles.length ) break COMMIT;
+                    // dependencies locks was not updated
+                    if ( !commitFiles.length ) break COMMIT;
 
-                        // add changes
-                        res = await repeatAction(
-                            async () => {
-                                const res = await this.git.exec( [ "add", ...commitFiles ] );
+                    // add changes
+                    res = await repeatAction(
+                        async () => {
+                            const res = await this.git.exec( [ "add", ...commitFiles ] );
 
-                                if ( res.ok ) {
-                                    return res;
-                                }
-                                else {
+                            if ( res.ok ) {
+                                return res;
+                            }
+                            else {
 
-                                    // console.log( ansi.error( res + "" ) );
+                                // console.log( ansi.error( res + "" ) );
 
-                                    throw res;
-                                }
-                            },
-                            { repeatOnError }
-                        );
-                        if ( !res.ok ) throw res;
+                                throw res;
+                            }
+                        },
+                        { repeatOnError }
+                    );
+                    if ( !res.ok ) throw res;
 
-                        // commit changes
-                        res = await repeatAction(
-                            async () => {
-                                const res = await this.git.exec( [ "commit", "-m", "chore(deps): update locked dependencies", ...commitFiles ] );
+                    // commit changes
+                    res = await repeatAction(
+                        async () => {
+                            const res = await this.git.exec( [ "commit", "-m", "chore(deps): update locked dependencies", ...commitFiles ] );
 
-                                if ( res.ok ) {
-                                    return res;
-                                }
-                                else {
+                            if ( res.ok ) {
+                                return res;
+                            }
+                            else {
 
-                                    // console.log( ansi.error( res + "" ) );
+                                // console.log( ansi.error( res + "" ) );
 
-                                    throw res;
-                                }
-                            },
-                            { repeatOnError }
-                        );
-                        if ( !res.ok ) throw res;
+                                throw res;
+                            }
+                        },
+                        { repeatOnError }
+                    );
+                    if ( !res.ok ) throw res;
 
-                        // push changes
-                        res = await repeatAction(
-                            async () => {
-                                const res = await this.git.exec( [ "push" ] );
+                    // push changes
+                    res = await repeatAction(
+                        async () => {
+                            const res = await this.git.exec( [ "push" ] );
 
-                                if ( res.ok ) {
-                                    return res;
-                                }
-                                else {
+                            if ( res.ok ) {
+                                return res;
+                            }
+                            else {
 
-                                    // console.log( ansi.error( res + "" ) );
+                                // console.log( ansi.error( res + "" ) );
 
-                                    return res;
-                                }
-                            },
-                            { repeatOnError }
-                        );
-                        if ( !res.ok ) throw res;
+                                return res;
+                            }
+                        },
+                        { repeatOnError }
+                    );
+                    if ( !res.ok ) throw res;
 
-                        logger.log( "Commit and push ... " + result( 200 ) );
-                    }
-                    catch ( e ) {
-                        res = result.catch( e );
+                    logger.log( "Commit and push ... " + result( 200 ) );
+                }
+                catch ( e ) {
+                    res = result.catch( e );
 
-                        logger.log( "Commit and push ... " + ansi.error( " ERROR " ) + ", " + res );
-                    }
+                    logger.log( "Commit and push ... " + ansi.error( " ERROR " ) + ", " + res );
                 }
             }
         }