eslint-config-nodejs-pmb 0.2.7 → 0.3.1

package/.github/workflows/ci.yaml

@@ -24,6 +24,7 @@ jobs:
                 with:
                     node-version: ${{ matrix.node-version }}
             -   run: npm install .
+            -   run: npm audit
             -   run: npm test
 
 

package/package.json

@@ -1,5 +1,5 @@
 { "name": "eslint-config-nodejs-pmb",
-  "version": "0.2.7",
+  "version": "0.3.1",
   "description": "My favorite eslint rules",
   "keywords": [],
 
@@ -27,14 +27,14 @@
   },
   "devDependencies": {
     "absdir": "^1.0.6",
-    "eslint-plugin-json-light": "^1.0.3",
+    "eslint-plugin-json-light-pmb": "^1.0.4",
     "eslint-plugin-node": "^11.1.0",
     "p-fatal": "^0.1.3",
     "safe-sortedjson": "^1.0.6",
     "usnam-pmb": "^0.2.5"
   },
   "peerDependencies": {
-    "eslint": "^8.10.0"
+    "eslint": "^8.11.0"
   },
 
   "npm vs. BOM = Unexpected token": "-*- coding: UTF-8 -*-"

package/rules.js

@@ -73,6 +73,21 @@ const rules = {
 };
 
 
+const overrides = [
+  { files: ['**.js'], parserOptions: { sourceType: 'script' } },
+  { files: ['**.mjs'],
+    rules: {
+      'node/no-unsupported-features/es-syntax': 'off', // assume esmod-pmb
+    },
+  },
+  { files: devDepPatternsList,
+    rules: {
+      'node/no-unpublished-import': 'off',
+    },
+  },
+];
+
+
 const config = {
 
   env: {
@@ -87,14 +102,7 @@ const config = {
     'plugin:node/recommended',
   ],
 
-  overrides: [
-    { files: ['**.js'], parserOptions: { sourceType: 'script' } },
-    { files: devDepPatternsList,
-      rules: {
-        'node/no-unpublished-import': 'off',
-      },
-    },
-  ],
+  overrides,
 
   parser: require.resolve('@babel/eslint-parser'),
   parserOptions: {

package/test/expectedPeerDependencies.js

@@ -3,7 +3,7 @@
 
 module.exports = (function depsHelper(require) {
   return [
-    require('eslint-plugin-json-light'),
+    require('eslint-plugin-json-light-pmb'),
     require('eslint-plugin-node'),
   ];
 }(String));

package/util/massfix_missing_import_fext.sh

@@ -10,16 +10,33 @@ function addfexts () {
   exec < <(eslint --ext js,mjs . | sed -urf <(echo '
     s~^ *([0-9]+):[0-9]+ +error +Missing file extension "(\S+)" for "(\S+|$\
       )" +import/extensions$~!:missfext \2 \3 \1~
+    s~^ *1:1 +error +Use the global form of \x27use strict\x27  strict$|$\
+      ~!global_use_strict~
     ')) || return $?
-  local LN= SRC_FN=
+  local ABS_PWD="$(readlink -m .)"
+  local LN=
+  local SRC_FN=
+  local ERR_CNT_IN_FILE=
   while IFS= read -r LN; do
     case "$LN" in
       '' ) continue;;
-      /*.mjs ) SRC_FN="$LN"; continue;;
+      /*.js | \
+      /*.mjs )
+        LN="${LN#"$ABS_PWD/"}"
+        SRC_FN="$LN"
+        ERR_CNT_IN_FILE=0
+        continue;;
       '!:missfext '* )
         LN="${LN#*:}"
         fix_"${LN%% *}" "${LN#* }" || return $?;;
-      * ) echo "W: eslint says: '$LN'" >&2;;
+      !global_use_strict )
+        LANG=C sed -re '1s~^(\xEF\xBB\xBF|)~&\x27use strict\x27;\n~' \
+          -i -- "$SRC_FN" || return $?;;
+      * )
+        [ "$ERR_CNT_IN_FILE" == 0 ] && echo "W: In file '$SRC_FN':" >&2
+        (( ERR_CNT_IN_FILE += 1 ))
+        echo "W: eslint says: '$LN'" >&2
+        ;;
     esac
   done
 }
@@ -30,8 +47,13 @@ function fix_missfext () {
   local FEXT="${ARG%% *}"; ARG="${ARG#* }"
   local REF="${ARG%% *}"; ARG="${ARG#* }"
   local LNUM="$ARG"
-  echo "D: Add fext .$FEXT to $SRC_FN line $LNUM:"
-  sed -re "$LNUM"'s~(\x27|\x22);?$~.'"$FEXT"'&~' -i -- "$SRC_FN" || return $?
+  echo "D: Add fext .$FEXT to file '$SRC_FN' line $LNUM:"
+  local SED=(
+    sed -re
+    "$LNUM"'s~(\x27|\x22)\)?;?$~.'"$FEXT"'&~'
+    -i -- "$SRC_FN"
+    )
+  "${SED[@]}" || return $?$(echo "W: sed failed, rv=$?" >&2)
 }