fix: linted

softvisio-node · Jan 24, 2024 · 4f30fde · 4f30fde
1 parent da159cd
commit 4f30fde
Show file tree

Hide file tree

Showing 18 changed files with 99 additions and 99 deletions.
diff --git a/__tests__/001-git.js b/__tests__/001-git.js
@@ -12,12 +12,12 @@ describe( "git-branch-parser", () => {
 
     for ( let n = 0; n < tests.length; n++ ) {
         test( "test_" + n, () => {
-            const match = tests[n][0].match( BRANCH_RE );
+            const match = tests[ n ][ 0 ].match( BRANCH_RE );
 
-            if ( !match ) throw Error( `Parsing error: ` + tests[n][0] );
+            if ( !match ) throw Error( `Parsing error: ` + tests[ n ][ 0 ] );
 
-            for ( const prop in tests[n][1] ) {
-                expect( tests[n][1][prop] ).toBe( match.groups[prop] );
+            for ( const prop in tests[ n ][ 1 ] ) {
+                expect( tests[ n ][ 1 ][ prop ] ).toBe( match.groups[ prop ] );
             }
         } );
     }

diff --git a/docs/assets/theme/theme.js b/docs/assets/theme/theme.js
@@ -81,14 +81,14 @@ class Theme {
         for ( let n = 0; n < blocks.length; n++ ) {
 
             // code block start
-            if ( blocks[n].startsWith( "```" ) ) continue;
+            if ( blocks[ n ].startsWith( "```" ) ) continue;
 
             // code block body
-            if ( n && blocks[n - 1].startsWith( "```" ) ) continue;
+            if ( n && blocks[ n - 1 ].startsWith( "```" ) ) continue;
 
-            blocks[n] = blocks[n].replaceAll( /<([\w.]+)(\[\])?\\>/g, ( match, type, array ) => {
+            blocks[ n ] = blocks[ n ].replaceAll( /<([\w.]+)(\[\])?\\>/g, ( match, type, array ) => {
                 if ( type in types ) {
-                    return `<[${ type }${ array ?? "" }](${ types[type] })\\>`;
+                    return `<[${ type }${ array ?? "" }](${ types[ type ] })\\>`;
                 }
                 else {
                     return match;
@@ -176,19 +176,19 @@ class Theme {
             const tag = heading.tagName,
                 level = +tag.substring( 1 );
 
-            levels[level] = level;
+            levels[ level ] = level;
         }
 
         const sortedLevels = Object.keys( levels ).sort();
 
         for ( let n = 0; n < sortedLevels.length; n++ ) {
-            levels[sortedLevels[n]] = n + 1;
+            levels[ sortedLevels[ n ] ] = n + 1;
         }
 
         // create toc
         for ( const heading of headings ) {
             const tag = heading.tagName,
-                level = levels[tag.substring( 1 )];
+                level = levels[ tag.substring( 1 ) ];
 
             let list;
 
@@ -197,7 +197,7 @@ class Theme {
                     for ( let n = 0; n < level - currentLevel; n++ ) {
                         list = document.createElement( "ul" );
 
-                        lists[lists.length - 1].appendChild( list );
+                        lists[ lists.length - 1 ].appendChild( list );
 
                         lists.push( list );
                     }
@@ -211,7 +211,7 @@ class Theme {
                 currentLevel = level;
             }
 
-            list = lists[lists.length - 1];
+            list = lists[ lists.length - 1 ];
 
             const link = heading.querySelector( "a" ),
                 li = document.createElement( "li" ),

diff --git a/lib/commands/git/pre-commit.js b/lib/commands/git/pre-commit.js
@@ -119,7 +119,7 @@ export default class extends Command {
                 if ( !res1.ok ) this._throwError( `Unable to update git index` );
 
                 // update working tree, if file wasn't modified after add and wasn't removed
-                if ( !modified[filename] && fs.existsSync( filename ) ) fs.writeFileSync( filename, res.data );
+                if ( !modified[ filename ] && fs.existsSync( filename ) ) fs.writeFileSync( filename, res.data );
             }
 
             // report

diff --git a/lib/commands/icons.js b/lib/commands/icons.js
@@ -125,9 +125,9 @@ export default class extends Command {
         const osFilter = process.cli.options.android || process.cli.options.electron || process.cli.options.ios;
 
         for ( const os in icons ) {
-            if ( osFilter && !process.cli.options[os] ) continue;
+            if ( osFilter && !process.cli.options[ os ] ) continue;
 
-            for ( const spec of icons[os] ) {
+            for ( const spec of icons[ os ] ) {
                 const img = await jimp.read( resources + "/" + spec.source );
 
                 for ( const [ name, target ] of Object.entries( spec.targets ) ) {

diff --git a/lib/commands/lint.js b/lib/commands/lint.js
@@ -53,15 +53,15 @@ Ignored directories:
         }
 
         // read patterns from stdin
-        else if ( patterns.length === 1 && patterns[0] === "-" ) {
+        else if ( patterns.length === 1 && patterns[ 0 ] === "-" ) {
             patterns = await this.#readStdin();
         }
 
         const lint = new Lint( patterns, {
             "cwd": "",
             "useLintIgnore": process.cli.options.lintignore,
             "verbose": process.cli.options.verbose,
-            "reportIgnored": process.cli.options["report-ignored"],
+            "reportIgnored": process.cli.options[ "report-ignored" ],
         } );
 
         const res = await lint.run( process.cli.options.action );

diff --git a/lib/commands/ls.js b/lib/commands/ls.js
@@ -39,7 +39,7 @@ export default class extends Command {
 
     // public
     async run () {
-        const workspace = process.env["SOFTVISIO_CLI_WORKSPACE_" + process.platform];
+        const workspace = process.env[ "SOFTVISIO_CLI_WORKSPACE_" + process.platform ];
 
         if ( !workspace ) this._throwError( `No workspace configured` );
 
@@ -100,7 +100,7 @@ export default class extends Command {
             if ( !status.ok ) this._throwError( `Git error: ` + status );
             status = status.data;
 
-            const currentBranchPushStatus = status.pushStatus[status.branch];
+            const currentBranchPushStatus = status.pushStatus[ status.branch ];
 
             const isDirty = status.isDirty || currentBranchPushStatus,
                 isUnreleased = ( !status.currentVersion.isNull && status.currentVersionDistance ) || ( !status.branch && status.abbrev );

diff --git a/lib/commands/publish.js b/lib/commands/publish.js
@@ -35,7 +35,7 @@ export default class extends Command {
         if ( !rootPackage ) this._throwError( "Unable to find root package" );
 
         // prepeare release type
-        const preRelease = PRERELEASE[process.cli.arguments["pre-release"]] || process.cli.arguments["pre-release"];
+        const preRelease = PRERELEASE[ process.cli.arguments[ "pre-release" ] ] || process.cli.arguments[ "pre-release" ];
 
         const res = await rootPackage.publish( preRelease, process.cli.options.force );
 

diff --git a/lib/commands/rpc.js b/lib/commands/rpc.js
@@ -26,14 +26,14 @@ export default class extends Command {
 
             decoder.on( "data", async msg => {
                 try {
-                    const res = await this["API_" + msg[2]]( ...msg[3] );
+                    const res = await this[ "API_" + msg[ 2 ] ]( ...msg[ 3 ] );
 
-                    socket.write( msgpack.encode( [ 1, msg[1], null, res ] ) );
+                    socket.write( msgpack.encode( [ 1, msg[ 1 ], null, res ] ) );
                 }
                 catch ( e ) {
                     console.log( e );
 
-                    socket.write( msgpack.encode( [ 1, msg[1], "error", null ] ) );
+                    socket.write( msgpack.encode( [ 1, msg[ 1 ], "error", null ] ) );
                 }
             } );
 
@@ -59,7 +59,7 @@ export default class extends Command {
         data.data.replaceAll( `"`, "&quot;" );
         data.data.replaceAll( `'`, "&#39;" );
 
-        if ( data.font ) data.font = data.font.split( ":" )[0].replaceAll( "_", " " );
+        if ( data.font ) data.font = data.font.split( ":" )[ 0 ].replaceAll( "_", " " );
 
         const html = `
 <html>

diff --git a/lib/commands/status.js b/lib/commands/status.js
@@ -33,7 +33,7 @@ Stable release (latest): ${ ansi.hl( status.releases.lastStableVersion.isNull ?
 Last release (next):     ${ ansi.hl( status.releases.lastVersion.isNull ? "-" : status.releases.lastVersion ) }
 Is dirty:                ${ status.isDirty ? ansi.error( " DIRTY " ) : ansi.ok( " COMMITED " ) }
 Not pushed commits:      ${ Object.keys( status.pushStatus )
-        .map( branch => branch + ":" + ( status.pushStatus[branch] ? ansi.error( " " + status.pushStatus[branch] + " " ) : ansi.ok( " PUSHED " ) ) )
+        .map( branch => branch + ":" + ( status.pushStatus[ branch ] ? ansi.error( " " + status.pushStatus[ branch ] + " " ) : ansi.ok( " PUSHED " ) ) )
         .join( ", " ) }
 Unreleased commits:      ${ status.currentVersionDistance ? ( !status.currentVersion.isNull ? ansi.error( " " + status.currentVersionDistance + " " ) : status.currentVersionDistance ) : ansi.ok( " RELEASED " ) }
         `.trim() );

diff --git a/lib/commands/test/plan.js b/lib/commands/test/plan.js
@@ -108,9 +108,9 @@ export default class extends Command {
             "level": process.cli.options.level,
             "json": process.cli.options.json,
 
-            "modulePattern": process.cli.arguments["module-pattern"],
-            "firstLevelPattern": process.cli.arguments["first-level-pattern"],
-            "secondLevelPattern": process.cli.arguments["second-level-pattern"],
+            "modulePattern": process.cli.arguments[ "module-pattern" ],
+            "firstLevelPattern": process.cli.arguments[ "first-level-pattern" ],
+            "secondLevelPattern": process.cli.arguments[ "second-level-pattern" ],
         };
     }
 }
diff --git a/lib/commands/test/run.js b/lib/commands/test/run.js
@@ -68,8 +68,8 @@ export default class extends Command {
         const { "default": fs } = await import( "fs" );
 
         // test single module
-        if ( process.cli.arguments["module-pattern"] && fs.existsSync( process.cli.arguments["module-pattern"] ) && fs.statSync( process.cli.arguments["module-pattern"] ).isFile() ) {
-            return await this.#testSingleModule( process.cli.arguments["module-pattern"] );
+        if ( process.cli.arguments[ "module-pattern" ] && fs.existsSync( process.cli.arguments[ "module-pattern" ] ) && fs.statSync( process.cli.arguments[ "module-pattern" ] ).isFile() ) {
+            return await this.#testSingleModule( process.cli.arguments[ "module-pattern" ] );
         }
 
         const packages = [];
@@ -123,14 +123,14 @@ export default class extends Command {
         return {
             "benchmarks": process.cli.options.benchmarks,
             "verbose": process.cli.options.verbose,
-            "showSkipped": process.cli.options["show-skipped"],
-            "showPassed": process.cli.options["show-passed"],
-            "showStackTrace": process.cli.options["show-stack-trace"],
-            "showConsoleLog": process.cli.options["show-console-log"],
-
-            "modulePattern": process.cli.arguments["module-pattern"],
-            "firstLevelPattern": process.cli.arguments["first-level-pattern"],
-            "secondLevelPattern": process.cli.arguments["second-level-pattern"],
+            "showSkipped": process.cli.options[ "show-skipped" ],
+            "showPassed": process.cli.options[ "show-passed" ],
+            "showStackTrace": process.cli.options[ "show-stack-trace" ],
+            "showConsoleLog": process.cli.options[ "show-console-log" ],
+
+            "modulePattern": process.cli.arguments[ "module-pattern" ],
+            "firstLevelPattern": process.cli.arguments[ "first-level-pattern" ],
+            "secondLevelPattern": process.cli.arguments[ "second-level-pattern" ],
         };
     }
 }
diff --git a/lib/get-text.js b/lib/get-text.js
@@ -138,7 +138,7 @@ export default class GetText {
         const locale = {
             "l10n": ( msgId, plural, pluralNumber ) => {
                 this.#poFile.addMessages( {
-                    [msgId]: {
+                    [ msgId ]: {
                         "pluralId": plural,
                         "references": [ this.#relativePath ],
                     },
@@ -147,7 +147,7 @@ export default class GetText {
 
             "l10nt": ( msgId, plural, pluralNumber ) => {
                 this.#poFile.addMessages( {
-                    [msgId]: {
+                    [ msgId ]: {
                         "pluralId": plural,
                         "references": [ this.#relativePath ],
                     },
@@ -198,11 +198,11 @@ export default class GetText {
         }
 
         // at least 1 argument is required
-        if ( !node.arguments?.[0] ) return;
+        if ( !node.arguments?.[ 0 ] ) return;
 
         // ejs
         if ( method === "ejs" ) {
-            const arg = node.arguments[0];
+            const arg = node.arguments[ 0 ];
 
             let template;
 
@@ -244,16 +244,16 @@ export default class GetText {
             };
 
         // message id
-        const msgId = this.#parseMessage( node.arguments[0], {
+        const msgId = this.#parseMessage( node.arguments[ 0 ], {
             start,
         } );
         if ( !msgId.value ) return;
         id = msgId.value;
         if ( msgId.isTemplate ) message.flags = [ "javascript-format" ];
 
         // plural
-        if ( node.arguments[1] ) {
-            const plural = this.#parseMessage( node.arguments[1], {
+        if ( node.arguments[ 1 ] ) {
+            const plural = this.#parseMessage( node.arguments[ 1 ], {
                 start,
             } );
             if ( !plural.value ) return;
@@ -278,7 +278,7 @@ export default class GetText {
 
         message.references = reference;
 
-        this.#poFile.addMessages( { [id]: message } );
+        this.#poFile.addMessages( { [ id ]: message } );
     }
 
     #parseMessage ( node, { start } = {} ) {

diff --git a/lib/git/changes.js b/lib/git/changes.js
@@ -133,24 +133,24 @@ export default class Changes {
             commit = new Commit( commit );
 
             // commit is already indexed
-            if ( index[commit.subjectId] ) {
+            if ( index[ commit.subjectId ] ) {
 
                 // do not replace breaking commits in the index
-                if ( index[commit.subjectId].isBreaking || !commit.isBreaking ) {
-                    index[commit.subjectId].commits.push( commit );
+                if ( index[ commit.subjectId ].isBreaking || !commit.isBreaking ) {
+                    index[ commit.subjectId ].commits.push( commit );
                 }
 
                 // replace indexed commit with the breaking commit
                 else {
-                    commit.commits.unshift( ...index[commit.subjectId].commits );
+                    commit.commits.unshift( ...index[ commit.subjectId ].commits );
 
-                    index[commit.subjectId] = commit;
+                    index[ commit.subjectId ] = commit;
                 }
             }
 
             // first commit
             else {
-                index[commit.subjectId] = commit;
+                index[ commit.subjectId ] = commit;
             }
         }
 

diff --git a/lib/lint/file.js b/lib/lint/file.js
@@ -123,7 +123,7 @@ export default class LintFile {
         // unknown
         if ( !type ) return;
 
-        return !!TYPES[type.type];
+        return !!TYPES[ type.type ];
     }
 
     async run ( action ) {
@@ -140,7 +140,7 @@ export default class LintFile {
                 if ( mimeType ) type = mimeType.type;
             }
 
-            if ( type ) this.#type = TYPES[type];
+            if ( type ) this.#type = TYPES[ type ];
         }
 
         if ( !this.#type && !this.#processUnsupported ) return result( [ 202, "File ignored" ] );
@@ -324,7 +324,7 @@ export default class LintFile {
             "warnIgnored": true,
         } );
 
-        if ( report[0].output != null ) this.#data = report[0].output;
+        if ( report[ 0 ].output != null ) this.#data = report[ 0 ].output;
 
         // append log
         var log = "",
@@ -341,7 +341,7 @@ export default class LintFile {
             },
         } );
 
-        for ( const msg of report[0].messages.sort( ( a, b ) => b.severity - a.severity || a.line - b.line || a.column - b.column ) ) {
+        for ( const msg of report[ 0 ].messages.sort( ( a, b ) => b.severity - a.severity || a.line - b.line || a.column - b.column ) ) {
             let severity;
 
             if ( msg.severity === 1 ) {
@@ -434,14 +434,14 @@ export default class LintFile {
     }
 
     async #cssnano ( preset ) {
-        if ( !POSTCSS_CACHE[preset] ) {
+        if ( !POSTCSS_CACHE[ preset ] ) {
             postcss ??= ( await import( "postcss" ) ).default;
             cssnano ??= ( await import( "Cssnano" ) ).default;
 
-            POSTCSS_CACHE[preset] = postcss( [ cssnano( { "preset": CSSNANO_PRESETS[preset] } ) ] );
+            POSTCSS_CACHE[ preset ] = postcss( [ cssnano( { "preset": CSSNANO_PRESETS[ preset ] } ) ] );
         }
 
-        const res = await POSTCSS_CACHE[preset].process( this.#data, { "from": null } );
+        const res = await POSTCSS_CACHE[ preset ].process( this.#data, { "from": null } );
 
         this.#data = res.toString();
 
@@ -483,7 +483,7 @@ export default class LintFile {
     }
 
     #replaceEndOfLine ( endOfLine ) {
-        const eol = EOL[endOfLine];
+        const eol = EOL[ endOfLine ];
 
         if ( !eol ) return;