@putout/plugin-nodejs 9.4.0 → 10.0.0

package/README.md

@@ -35,7 +35,8 @@ npm i putout @putout/plugin-nodejs -D
         "nodejs/convert-top-level-return": "on",
         "nodejs/declare": "on",
         "nodejs/declare-after-require": "on",
-        "nodejs/remove-process-exit": "on"
+        "nodejs/remove-process-exit": "on",
+        "nodejs/strict-mode": "on"
     }
 }
 ```
@@ -397,6 +398,50 @@ Rename `*.mjs` files when `module === "module"`:
 
 Check out in 🐊[Putout Editor](https://putout.cloudcmd.io/#/gist/94fb3298b210e703b01db9a6826942bc/dfe2462451c6b3d4d47da7fd8d39dc8e53bb16eb).
 
+## strict-mode
+
+> **Strict mode** makes several changes to normal **JavaScript** semantics:
+>
+> - Eliminates some **JavaScript** silent errors by changing them to throw errors.
+> - Fixes mistakes that make it difficult for **JavaScript** engines to perform optimizations: strict mode code can sometimes be made to run faster than identical code that's not strict mode.
+> - Prohibits some syntax likely to be defined in future versions of **ECMAScript**.
+>
+> (c) [MDN](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Strict_mode)
+
+Add **strict mode** to **CommonJS**, and remove from **ESM**, where it enabled by default.
+
+### ❌ Example of incorrect code
+
+ESM:
+
+```js
+'strict mode';
+
+import a from 'b';
+```
+
+CommonJS:
+
+```js
+const a = require('b');
+```
+
+### ✅ Example of correct code
+
+ESM:
+
+```js
+import a from 'b';
+```
+
+CommonJS:
+
+```js
+'strict mode';
+
+const a = require('b');
+```
+
 ## License
 
 MIT

package/lib/index.js

@@ -24,6 +24,8 @@ const mjsFile = require('./mjs-file');
 const renameFileCjsToJs = require('./rename-file-cjs-to-js');
 const renameFileMjsToJs = require('./rename-file-mjs-to-js');
 
+const strictMode = require('./strict-mode');
+
 module.exports.rules = {
     'convert-buffer-to-buffer-alloc': convertBufferToBufferAlloc,
     'convert-fs-promises': convertFsPromises,
@@ -41,8 +43,13 @@ module.exports.rules = {
     'convert-commonjs-to-esm-exports': ['off', convertCommonjsToEsmExports],
     'convert-commonjs-to-esm-common': ['off', convertCommonjsToEsmCommons],
     'convert-commonjs-to-esm-require': ['off', convertCommonjsToEsmRequire],
+    
     'cjs-file': ['off', cjsFile],
     'mjs-file': ['off', mjsFile],
+    
     'rename-file-cjs-to-js': ['off', renameFileCjsToJs],
     'rename-file-mjs-to-js': renameFileMjsToJs,
+    
+    'add-strict-mode': strictMode.rules['add-missing'],
+    'remove-strict-mode': strictMode.rules['remove-useless'],
 };

package/lib/strict-mode/add-missing/index.js

@@ -0,0 +1,55 @@
+'use strict';
+
+const {
+    isExpressionStatement,
+    StringLiteral,
+    ExpressionStatement,
+    isProgram,
+} = require('putout').types;
+
+module.exports.report = () => `Add missing 'use strict' directive on top of CommonJS`;
+
+module.exports.fix = ({node}) => {
+    node.body.unshift(ExpressionStatement(StringLiteral('use strict')));
+};
+
+module.exports.traverse = ({push, store}) => ({
+    'await __a(__args)'({scope}) {
+        const {block} = scope;
+        
+        if (!isProgram(block))
+            return;
+        
+        store('is-module', true);
+    },
+    'ImportDeclaration|ExportNamedDeclaration|ExportDefaultDeclaration|ExportAllDeclaration|TypeAlias'() {
+        store('is-module', true);
+    },
+    'module.exports = __a'() {
+        store('is-common', true);
+    },
+    'module.exports.__a = __b'() {
+        store('is-common', true);
+    },
+    'require(__a)'() {
+        store('is-common', true);
+    },
+    Program: {
+        exit(path) {
+            for (const node of path.node.body)
+                if (isExpressionStatement(node) && node.expression.value === 'use strict')
+                    return;
+            
+            if (store('is-module'))
+                return;
+            
+            if (path.node.directives.length)
+                return;
+            
+            if (!store('is-common'))
+                return;
+            
+            push(path);
+        },
+    },
+});

package/lib/strict-mode/index.js

@@ -0,0 +1,9 @@
+'use strict';
+
+const addMissing = require('./add-missing');
+const removeUseless = require('./remove-useless');
+
+module.exports.rules = {
+    'add-missing': addMissing,
+    'remove-useless': removeUseless,
+};

package/lib/strict-mode/remove-useless/index.js

@@ -0,0 +1,52 @@
+'use strict';
+
+const {operator, types} = require('putout');
+
+const {remove} = operator;
+const {isProgram} = types;
+
+const isStrictMode = (a) => a.node.value.value === 'use strict';
+
+module.exports.report = () => `Avoid 'use strict' in ESM`;
+
+module.exports.fix = (path) => remove(path);
+
+module.exports.traverse = ({push, store}) => ({
+    'await __a(__args)'({scope}) {
+        const {block} = scope;
+        
+        if (isProgram(block))
+            store('is-module', true);
+    },
+    'ImportDeclaration|ExportNamedDeclaration|ExportDefaultDeclaration|ExportAllDeclaration'() {
+        store('is-module', true);
+    },
+    Program: {
+        exit(path) {
+            const [strictPath, ...paths] = path.get('body');
+            
+            for (const path of paths) {
+                if (path.isExpressionStatement() && path.node.expression.value === 'use strict')
+                    push(path);
+            }
+            
+            const directives = path.get('directives');
+            
+            if (directives.length)
+                directives
+                    .slice(1)
+                    .filter(isStrictMode)
+                    .forEach(push);
+            
+            if (!store('is-module'))
+                return;
+            
+            directives
+                .filter(isStrictMode)
+                .forEach(push);
+            
+            if (strictPath.isExpressionStatement() && strictPath.node.expression.value === 'use strict')
+                push(strictPath);
+        },
+    },
+});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@putout/plugin-nodejs",
-  "version": "9.4.0",
+  "version": "10.0.0",
   "type": "commonjs",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
   "description": "🐊Putout plugin adds ability to transform code to new API of Node.js",
@@ -8,6 +8,7 @@
   "main": "lib/index.js",
   "exports": {
     ".": "./lib/index.js",
+    "./strict-mode": "./lib/strict-mode/index.js",
     "./convert-esm-to-commonjs": "./lib/convert-esm-to-commonjs/index.js",
     "./convert-commonjs-to-esm": "./lib/convert-commonjs-to-esm.js"
   },
@@ -38,7 +39,9 @@
     "nodejs"
   ],
   "devDependencies": {
+    "@putout/plugin-declare": "*",
     "@putout/plugin-putout": "*",
+    "@putout/plugin-typescript": "*",
     "@putout/test": "^7.0.0",
     "c8": "^8.0.0",
     "eslint": "^8.0.1",
@@ -50,11 +53,11 @@
     "nodemon": "^3.0.1"
   },
   "peerDependencies": {
-    "putout": ">=33"
+    "putout": ">=34"
   },
   "license": "MIT",
   "engines": {
-    "node": ">=16.17"
+    "node": ">=18"
   },
   "publishConfig": {
     "access": "public"