@putout/plugin-package-json 10.2.0 → 10.4.0

package/README.md

@@ -1,7 +1,7 @@
 # @putout/plugin-package-json [![NPM version][NPMIMGURL]][NPMURL]
 
 [NPMIMGURL]: https://img.shields.io/npm/v/@putout/plugin-package-json.svg?style=flat&longCache=true
-[NPMURL]: https://npmjs.org/package/@putout/plugin-package-json"npm"
+[NPMURL]: https://npmjs.org/package/@putout/plugin-package-json "npm"
 
 🐊[**Putout**](https://github.com/coderaiser/putout) plugin helps to automate fixing `package.json`.
 
@@ -14,6 +14,7 @@ npm i @putout/plugin-package-json -D
 ## Rules
 
 - ✅ [add-type](#add-type);
+- ✅ [apply-js-extension](#apply-js-extension);
 - ✅ [apply-https-to-repository-url](#apply-https-to-repository-url);
 - ✅ [find-file](#find-file);
 - ✅ [remove-nyc](#remove-nyc);
@@ -28,6 +29,7 @@ npm i @putout/plugin-package-json -D
 {
     "rules": {
         "package-json/add-type": "on",
+        "package-json/apply-js-extension": "on",
         "package-json/apply-https-to-repository-url": "on",
         "package-json/remove-nyc": "on",
         "package-json/remove-commit-type": "on",
@@ -50,6 +52,32 @@ Add [`type`](https://nodejs.org/dist/latest-v17.x/docs/api/packages.html#type) f
 }
 ```
 
+## apply-js-extension
+
+Checkout in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/31b208fae5f340b9d8a479c2d7475cd2/869370f138d0082cf67bb4372c62fb969bb98db5).
+
+```diff
+{
+  "type": "module",
+- "main": "./lib/putout.mjs",
++ "main": "./lib/putout.js",
+  "exports": {
+    ".": {
+      "require": "./lib/index.cjs",
+-     "import": "./lib/index.mjs"
++     "import": "./lib/index.js"
+    },
+    "./parse-error": "./lib/parse-error.cjs",
+-   "./ignores": "./lib/ignores.mjs",
++   "./ignores": "./lib/ignores.js",
+  },
+  "bin": {
+-   "putout": "bin/tracer.mjs"
++   "putout": "bin/tracer.js"
+  }
+};
+```
+
 ## apply-https-to-repository-url
 
 The `git://` protocol for GitHub repos should not be used due [security concerns](https://github.blog/security/application-security/improving-git-protocol-security-github/).

package/lib/{appy-https-to-repository-url → apply-https-to-repository-url}/index.js

@@ -11,7 +11,9 @@ export const report = () => `Apply 'https' to repository.url`;
 
 export const fix = (path) => {
     const {value} = path.node;
-    setLiteralValue(path, value.replace('git:', 'git+https:'));
+    const newValue = value.replace(/^(git|https)/, 'git+https');
+    
+    setLiteralValue(path, newValue);
 };
 
 export const traverse = ({push}) => ({
@@ -22,21 +24,24 @@ export const traverse = ({push}) => ({
         if (!repository)
             return;
         
-        const value = repository.get('value');
+        const object = repository.get('value');
         
-        if (!isObjectExpression(value))
+        if (!isObjectExpression(object))
             return;
         
-        const urlPathProp = getProperty(value, 'url');
+        const urlPathProp = getProperty(object, 'url');
         
         if (!urlPathProp)
             return;
         
         const urlPathValue = urlPathProp.get('value');
         
-        if (!urlPathValue.node.value.startsWith('git:'))
+        const {value} = urlPathValue.node;
+        
+        if (value.startsWith('git+https'))
             return;
         
-        push(urlPathValue);
+        if (/^(git|https)/.test(value))
+            push(urlPathValue);
     },
 });

package/lib/apply-js-extension/index.js

@@ -0,0 +1,48 @@
+import {extname} from 'node:path';
+import {operator, types} from 'putout';
+
+const {isStringLiteral} = types;
+const {
+    getProperties,
+    __json,
+    setLiteralValue,
+} = operator;
+
+export const report = (path) => {
+    const {value} = path.node;
+    const ext = extname(value);
+    
+    return `Use '.js' instead of '${ext}' in '${value}'`;
+};
+
+export const fix = (path) => {
+    const {value} = path.node;
+    setLiteralValue(path, value.replace(/\.[cm]js$/, '.js'));
+};
+
+export const traverse = ({push}) => ({
+    [__json]: (path) => {
+        const __aPath = path.get('arguments.0');
+        const {typePath} = getProperties(__aPath, ['type']);
+        const isModule = typePath && typePath.node.value.value === 'module';
+        
+        path.traverse({
+            ObjectProperty(path) {
+                const valuePath = path.get('value');
+                
+                if (!isStringLiteral(valuePath))
+                    return;
+                
+                const {value} = path.node.value;
+                
+                if (isModule && value.endsWith('.mjs')) {
+                    push(valuePath);
+                    return;
+                }
+                
+                if (!isModule && value.endsWith('.cjs'))
+                    push(valuePath);
+            },
+        });
+    },
+});

package/lib/index.js

@@ -1,3 +1,4 @@
+import * as applyJsExtension from './apply-js-extension/index.js';
 import * as removeImportsNesting from './remove-imports-nesting/index.js';
 import {rules as packageJson} from './package-json.js';
 import * as findFile from './find-file/index.js';
@@ -10,4 +11,5 @@ export const rules = {
     'remove-exports-with-missing-files': ['off', removeExportsWithMissingFiles],
     'remove-duplicate-keywords': removeDuplicateKeywords,
     'remove-imports-nesting': removeImportsNesting,
+    'apply-js-extension': applyJsExtension,
 };

package/lib/package-json.js

@@ -1,7 +1,7 @@
 import * as addType from './add-type/index.js';
 import * as removeNyc from './remove-nyc/index.js';
 import * as removeCommitType from './remove-commit-type/index.js';
-import * as applyHttpsToRepositoryUrl from './appy-https-to-repository-url/index.js';
+import * as applyHttpsToRepositoryUrl from './apply-https-to-repository-url/index.js';
 
 export const rules = {
     'add-type': addType,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@putout/plugin-package-json",
-  "version": "10.2.0",
+  "version": "10.4.0",
   "type": "module",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
   "description": "🐊Putout plugin for package.json",