eslint-plugin-putout 10.1.0 → 10.5.0

package/README.md

@@ -70,6 +70,8 @@ Then configure the rules you want to use under the rules section.
 - [Remove newline after default import](/packages/eslint-plugin-putout/lib/remove-newline-after-default-import)
 - [Objects braces inside array](/packages/eslint-plugin-putout/lib/objects-braces-inside-array)
 - [Object init](/packages/eslint-plugin-putout/lib/object-init)
+- [No unresolved](/packages/eslint-plugin-putout/lib/no-unresolved)
+- [Evaluate](/packages/eslint-plugin-putout/lib/evaluate)
 
 ### Safe mode
 
@@ -95,5 +97,5 @@ List of disabled `putout` rules:
 - [remove-skip](https://github.com/coderaiser/putout/tree/v20.0.0/packages/remove-skip);
 - [remove-only](https://github.com/coderaiser/putout/tree/v20.0.0/packages/remove-only);
 - [remove-console](https://github.com/coderaiser/putout/tree/v20.0.0/packages/remove-console);
-- [emove-debugger](https://github.com/coderaiser/putout/tree/v20.0.0/packages/remove-debugger);
-- [onvert-for-to-for-of](https://github.com/coderaiser/putout/tree/v20.0.0/packages/convert-for-to-for-of);
+- [remove-debugger](https://github.com/coderaiser/putout/tree/v20.0.0/packages/remove-debugger);
+- [convert-for-to-for-of](https://github.com/coderaiser/putout/tree/v20.0.0/packages/convert-for-to-for-of);

package/lib/evaluate/README.md

@@ -0,0 +1,35 @@
+# evaluate expression (evaluate)
+
+Evaluate expression started with `__putout_evaluate: `.
+Provided code is processed with [`@putout/plugin-declare-undefined-variables`](https://github.com/coderaiser/putout/tree/master/packages/plugin-declare-undefined-variables). So next code:
+
+```js
+import {join} from 'path';
+__putout_evaluate: join("hello", " ", "world");
+```
+
+Converted to:
+
+```js
+const fn = (__filename, __dirname, require) => {
+    const {join} = require('path');
+    return join("hello", " ", "world");
+};
+```
+
+## Rule Details
+
+This rule aims to evaluate expressions `source` of `ImportDeclaration`:
+
+Examples of **incorrect** code locaed in `hello.spec.js` for this rule:
+
+```js
+import {readFile} from '__putout_evaluate: `./` + basename(__filename).replace(`.spec.js`, `.js`)';
+
+```
+
+Examples of **correct** code for this rule:
+
+```js
+import {readFile} from './hello.js';
+```

package/lib/evaluate/index.js

@@ -0,0 +1,44 @@
+'use strict';
+
+const {dirname} = require('path');
+const putout = require('putout');
+
+module.exports.category = 'evaluate';
+module.exports.report = () => 'Evaluate expression';
+module.exports.include = () => ['ImportDeclaration'];
+
+module.exports.filter = ({node}) => {
+    const {value} = node.source;
+    
+    if (!/^__putout_evaluate:/.test(value))
+        return false;
+    
+    return true;
+};
+
+module.exports.fix = ({text, node, filename}) => {
+    const {value} = node.source;
+    
+    const result = evaluate({
+        value,
+        filename,
+    });
+    
+    return text.replace(value, result);
+};
+
+function evaluate({value, filename}) {
+    value = value.replace(/__putout_evaluate:\s?/, 'return ');
+    const {code} = putout(value, {
+        rules: {
+            'convert-esm-to-commonjs': 'on',
+        },
+        plugins: [
+            'declare-undefined-variables',
+            'convert-esm-to-commonjs',
+        ],
+    });
+    
+    const fn = Function('__filename', '__dirname', 'require', code);
+    return fn(filename, dirname(__filename), require);
+}

package/lib/index.js

@@ -28,6 +28,8 @@ module.exports.rules = {
     ...getWrapRule('remove-newline-after-default-import'),
     ...getWrapRule('objects-braces-inside-array'),
     ...getWrapRule('object-init'),
+    ...getWrapRule('no-unresolved'),
+    ...getWrapRule('evaluate'),
     ...getRule('putout'),
 };
 
@@ -53,6 +55,8 @@ const recommended = {
         'putout/remove-newline-after-default-import': 'error',
         'putout/objects-braces-inside-array': 'error',
         'putout/object-init': 'error',
+        'putout/no-unresolved': 'error',
+        'putout/evaluate': 'error',
         'putout/putout': 'error',
         
         'node/no-unsupported-features/es-syntax': 'off',

package/lib/no-unresolved/README.md

@@ -0,0 +1,30 @@
+# check if path can be resolved and fix if cannot (no-unresolved)
+
+Similar to [`no-unresolved`](https://github.com/import-js/eslint-plugin-import/blob/HEAD/docs/rules/no-unresolved.md) from 
+ [`eslint-plugin-putout`](https://github.com/import-js/eslint-plugin-import). But supports only `ESM` and have `autofix`.
+
+## Rule Details
+
+This rule aims to fix `unresolved import`:
+
+Examples of **incorrect** code for this rule:
+
+```js
+import x from './y';
+import dir from './dir';
+```
+
+[File extension is mandatory](https://nodejs.org/api/esm.html#esm_mandatory_file_extensions) and will produce an error from `node.js`:
+
+```
+Error [ERR_MODULE_NOT_FOUND]: Cannot find module '/Users/coderaiser/putout/y' imported from /Users/coderaiser/putout/x.mjs
+Did you mean to import ../y.js?
+```
+
+Examples of **correct** code for this rule:
+
+```js
+import x from './y.js';
+import dir from './dir/index.js';
+```
+

package/lib/no-unresolved/index.js

@@ -0,0 +1,75 @@
+'use strict';
+
+const tryCatch = require('try-catch');
+
+const {accessSync} = require('fs');
+
+const {
+    extname,
+    dirname,
+    join,
+} = require('path');
+
+const cwd = process.cwd();
+
+const RELATIVE = '\\.\\.?\\/?';
+const isRelativeStart = (a) => RegExp(`^${RELATIVE}`).test(a);
+const isRelativeEnd = (a) => RegExp(`${RELATIVE}$`).test(a);
+const getDir = (a) => a === '<input>' ? cwd : dirname(a);
+
+module.exports.category = 'errors';
+module.exports.report = () => 'Always add an extension to relative imports';
+module.exports.include = () => ['ImportDeclaration'];
+
+module.exports.fix = ({node, text, filename}) => {
+    const {source} = node;
+    const {value} = source;
+    
+    const dir = getDir(filename);
+    
+    const resolved = resolveSource({
+        dir,
+        value,
+    });
+    
+    return text.replace(value, resolved);
+};
+
+module.exports.filter = ({node}) => {
+    const {source} = node;
+    const {value} = source;
+    
+    if (!isRelativeStart(value))
+        return false;
+    
+    return !extname(value);
+};
+
+function resolveSource({dir, value}) {
+    if (isRelativeEnd(value)) {
+        const name = join(dir, value, 'package.json');
+        const [error, info] = tryCatch(require, name);
+        
+        if (error)
+            return value;
+        
+        return join(value, info.main);
+    }
+    
+    for (const ext of ['js', 'mjs', 'cjs']) {
+        const name = join(dir, `${value}.${ext}`);
+        const [error] = tryCatch(accessSync, name);
+        
+        if (!error)
+            return `${value}.${ext}`;
+    }
+    
+    const name = join(dir, value, `index.js`);
+    const [error] = tryCatch(accessSync, name);
+    
+    if (!error)
+        return join(value, 'index.js');
+    
+    return `${value}.js`;
+}
+

package/lib/single-property-destructuring/README.md

@@ -15,12 +15,12 @@ const {
 
 import {
     password,
-} from './user';
+} from './user.js';
 ```
 
 Examples of **correct** code for this rule:
 
 ```js
 const {username} = user;
-import {password} from './user';
+import {password} from './user.js';
 ```

package/lib/ts.js

@@ -11,6 +11,7 @@ module.exports = [{
     },
     rules: {
         'no-undef': 'off',
+        'no-var': 'off',
     },
 }];
 

package/lib/wrap.js

@@ -4,6 +4,7 @@ const prepare = (plugin, context, options) => (node) => {
     const {filter, report} = plugin;
     
     const source = context.getSourceCode();
+    const filename = context.getFilename();
     const getText = source.getText.bind(source);
     
     const text = getText(node);
@@ -13,12 +14,14 @@ const prepare = (plugin, context, options) => (node) => {
         node,
         options,
         getText,
+        filename,
     });
     
     if (!result)
         return;
     
     const fix = prepareFix(plugin.fix, {
+        filename,
         node,
         text,
         getText,
@@ -31,11 +34,12 @@ const prepare = (plugin, context, options) => (node) => {
     });
 };
 
-const prepareFix = (fix, {node, text, getText}) => (fixer) => {
+const prepareFix = (fix, {node, text, getText, filename}) => (fixer) => {
     const fixed = fix({
         node,
         text,
         getText,
+        filename,
     });
     
     return [

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-putout",
-  "version": "10.1.0",
+  "version": "10.5.0",
   "description": "eslint plugin for putout",
   "release": false,
   "tag": false,
@@ -20,9 +20,11 @@
   "scripts": {
     "wisdom": "madrun wisdom",
     "test": "madrun test",
+    "test:all": "madrun test:all",
     "watch:test": "madrun watch:test",
     "lint": "madrun lint",
-    "lint:fixture": "madrun lint:fixture",
+    "lint:all": "madrun lint:all",
+    "lint:ide": "madrun lint:ide",
     "fresh:lint": "madrun fresh:lint",
     "lint:fresh": "madrun lint:fresh",
     "fix:lint": "madrun fix:lint",
@@ -38,7 +40,8 @@
     "@putout/eslint-config": "^6.0.0",
     "align-spaces": "^1.0.0",
     "eslint-plugin-node": "^11.0.0",
-    "estree-to-babel": "^4.0.1"
+    "estree-to-babel": "^4.0.1",
+    "try-catch": "^3.0.0"
   },
   "devDependencies": {
     "c8": "^7.5.0",
@@ -46,7 +49,10 @@
     "eslint-plugin-eslint-plugin": "^3.2.0",
     "madrun": "^8.0.1",
     "mocha": "^9.0.1",
-    "montag": "^1.0.0"
+    "montag": "^1.0.0",
+    "simport": "^1.2.0",
+    "supertape": "^6.6.0",
+    "try-to-catch": "^3.0.0"
   },
   "engines": {
     "node": ">=14"