Merge pull request #2368 from blackflux/dev

[Gally]: master <- dev

Author: simlu

.circleci/config.yml

@@ -1,47 +1,46 @@
 workflows:
-  version: 2
   node-multi-build:
     jobs:
       - gally-create-pr
-      - test-node-v18:
+      - test-node-v20:
           requires:
             - gally-create-pr
-      - test-node-v16:
+      - test-node-v23:
           requires:
             - gally-create-pr
-      - test-node-v20:
+      - test-node-v22:
           requires:
             - gally-create-pr
       - gally-auto-approve:
           requires:
             - gally-create-pr
-            - test-node-v18
-            - test-node-v16
             - test-node-v20
+            - test-node-v23
+            - test-node-v22
           filters:
             branches:
               only: /dependabot\/.*/
       - release:
           requires:
             - gally-create-pr
-            - test-node-v18
-            - test-node-v16
             - test-node-v20
+            - test-node-v23
+            - test-node-v22
           filters:
             branches:
               only: master
-version: 2
+version: 2.1
 jobs:
   gally-create-pr:
     docker:
-      - image: cimg/node:16.20
+      - image: cimg/node:20.11
     steps:
       - checkout
       - run: sudo yarn global add gally
       - run: ga promote $CIRCLE_BRANCH
   release:
     docker:
-      - image: cimg/node:16.20
+      - image: cimg/node:22.13
     steps:
       - checkout
       - run:
@@ -71,7 +70,7 @@ jobs:
       - run: yarn run semantic-release
   gally-auto-approve:
     docker:
-      - image: cimg/node:16.20
+      - image: cimg/node:20.11
     steps:
       - run:
           name: Set PR number
@@ -88,9 +87,9 @@ jobs:
       - run: >-
           ga approve $CIRCLE_PR_NUMBER --condition
           "base.ref=dev&state=open&user.login=dependabot-preview[bot]"
-  test-node-v16:
+  test-node-v20:
     docker:
-      - image: cimg/node:16.20
+      - image: cimg/node:20.18
     steps:
       - checkout
       - run:
@@ -120,9 +119,9 @@ jobs:
       - run: >-
           (git add . && git diff-index --quiet HEAD --) || (git status && exit
           1);
-  test-node-v18:
+  test-node-v23:
     docker:
-      - image: cimg/node:18.16
+      - image: cimg/node:23.6
     steps:
       - checkout
       - run:
@@ -152,9 +151,9 @@ jobs:
       - run: >-
           (git add . && git diff-index --quiet HEAD --) || (git status && exit
           1);
-  test-node-v20:
+  test-node-v22:
     docker:
-      - image: cimg/node:20.4
+      - image: cimg/node:22.13
     steps:
       - checkout
       - run:

.eslintrc.json

@@ -3,7 +3,7 @@
   "extends": [
     "airbnb-base",
     "plugin:mocha/recommended",
-    "plugin:markdown/recommended"
+    "plugin:markdown/recommended-legacy"
   ],
   "rules": {
     "@blackflux/rules/c8-prevent-ignore": 1,

.gally.json

@@ -48,9 +48,9 @@
         "strict": true,
         "contexts": [
           "ci/circleci: gally-create-pr",
-          "ci/circleci: test-node-v18",
-          "ci/circleci: test-node-v16",
-          "ci/circleci: test-node-v20"
+          "ci/circleci: test-node-v20",
+          "ci/circleci: test-node-v23",
+          "ci/circleci: test-node-v22"
         ]
       },
       "enforce_admins": false,

.roboconfig.json

@@ -8,7 +8,7 @@
       "packageName": "lambda-serverless-api",
       "projectName": "lambda-serverless-api",
       "owner": "simlu",
-      "nodeVersion": "16",
+      "nodeVersion": "20",
       "ownerName": "Lukas Siemon",
       "mergeBot": "MrsFlux",
       "nccEnabled": false

.roboconfig.lock

@@ -18,6 +18,7 @@
     "package.json",
     "src/index.js",
     "test/hot.js",
-    "test/index.spec.js"
+    "test/index.spec.js",
+    "test/reg.js"
   ]
 }

CONFDOCS.md

@@ -8,6 +8,7 @@ Documents configuration tasks managed by [robo-config](https://github.com/blackf
   - <a name="blackfluxrobo-config-plugin-task-idx-ref-structnpm-default">:open_file_folder:</a> <a href="#blackfluxrobo-config-plugin-task-ref-structnpm-default">`struct/#npm-default`</a>
     - <a name="blackfluxrobo-config-plugin-task-idx-ref-structforce-gardener">:clipboard:</a> <a href="#blackfluxrobo-config-plugin-task-ref-structforce-gardener">`struct/force-gardener`</a>
     - <a name="blackfluxrobo-config-plugin-task-idx-ref-structnpm-base">:open_file_folder:</a> <a href="#blackfluxrobo-config-plugin-task-ref-structnpm-base">`struct/#npm-base`</a>
+      - <a name="blackfluxrobo-config-plugin-task-idx-ref-structtest-reg">:clipboard:</a> <a href="#blackfluxrobo-config-plugin-task-ref-structtest-reg">`struct/test-reg`</a>
       - <a name="blackfluxrobo-config-plugin-task-idx-ref-structtest-hot">:clipboard:</a> <a href="#blackfluxrobo-config-plugin-task-ref-structtest-hot">`struct/test-hot`</a>
       - <a name="blackfluxrobo-config-plugin-task-idx-ref-structtemplate-readme">:clipboard:</a> <a href="#blackfluxrobo-config-plugin-task-ref-structtemplate-readme">`struct/template-readme`</a>
       - <a name="blackfluxrobo-config-plugin-task-idx-ref-structtemplate-mocharc">:clipboard:</a> <a href="#blackfluxrobo-config-plugin-task-ref-structtemplate-mocharc">`struct/template-mocharc`</a>
@@ -87,7 +88,8 @@ Manage various configuration files for an open source npm package on github.
 <code>│&nbsp;&nbsp;└─&nbsp;<a href="#blackfluxrobo-config-plugin-target-ref-srcindexjs">index.js</a></code><br/>
 <code>└─&nbsp;test</code><br/>
 <code>&nbsp;&nbsp;&nbsp;├─&nbsp;<a href="#blackfluxrobo-config-plugin-target-ref-testhotjs">hot.js</a></code><br/>
-<code>&nbsp;&nbsp;&nbsp;└─&nbsp;<a href="#blackfluxrobo-config-plugin-target-ref-testindexspecjs">index.spec.js</a></code><br/>
+<code>&nbsp;&nbsp;&nbsp;├─&nbsp;<a href="#blackfluxrobo-config-plugin-target-ref-testindexspecjs">index.spec.js</a></code><br/>
+<code>&nbsp;&nbsp;&nbsp;└─&nbsp;<a href="#blackfluxrobo-config-plugin-target-ref-testregjs">reg.js</a></code><br/>
         </ul>
       </td>
       <td align="left" valign="top">
@@ -151,7 +153,8 @@ Manage basic project structure for npm package on github.
 <code>│&nbsp;&nbsp;└─&nbsp;<a href="#blackfluxrobo-config-plugin-target-ref-srcindexjs">index.js</a></code><br/>
 <code>└─&nbsp;test</code><br/>
 <code>&nbsp;&nbsp;&nbsp;├─&nbsp;<a href="#blackfluxrobo-config-plugin-target-ref-testhotjs">hot.js</a></code><br/>
-<code>&nbsp;&nbsp;&nbsp;└─&nbsp;<a href="#blackfluxrobo-config-plugin-target-ref-testindexspecjs">index.spec.js</a></code><br/>
+<code>&nbsp;&nbsp;&nbsp;├─&nbsp;<a href="#blackfluxrobo-config-plugin-target-ref-testindexspecjs">index.spec.js</a></code><br/>
+<code>&nbsp;&nbsp;&nbsp;└─&nbsp;<a href="#blackfluxrobo-config-plugin-target-ref-testregjs">reg.js</a></code><br/>
         </ul>
       </td>
       <td align="left" valign="top">
@@ -224,7 +227,8 @@ Manage basic project structure for npm package on github.
 <code>│&nbsp;&nbsp;└─&nbsp;<a href="#blackfluxrobo-config-plugin-target-ref-srcindexjs">index.js</a></code><br/>
 <code>└─&nbsp;test</code><br/>
 <code>&nbsp;&nbsp;&nbsp;├─&nbsp;<a href="#blackfluxrobo-config-plugin-target-ref-testhotjs">hot.js</a></code><br/>
-<code>&nbsp;&nbsp;&nbsp;└─&nbsp;<a href="#blackfluxrobo-config-plugin-target-ref-testindexspecjs">index.spec.js</a></code><br/>
+<code>&nbsp;&nbsp;&nbsp;├─&nbsp;<a href="#blackfluxrobo-config-plugin-target-ref-testindexspecjs">index.spec.js</a></code><br/>
+<code>&nbsp;&nbsp;&nbsp;└─&nbsp;<a href="#blackfluxrobo-config-plugin-target-ref-testregjs">reg.js</a></code><br/>
         </ul>
       </td>
       <td align="left" valign="top">
@@ -244,11 +248,40 @@ Manage basic project structure for npm package on github.
   </tbody>
 </table>
 
+###### :clipboard: <a name="blackfluxrobo-config-plugin-task-ref-structtest-reg">struct/test-reg</a> (<a href="#blackfluxrobo-config-plugin-task-idx-ref-structtest-reg">`index`</a>)
+
+_Updating <a href="#blackfluxrobo-config-plugin-target-ref-testregjs">test/reg.js</a> using <a href="#blackfluxrobo-config-plugin-strat-ref-overwrite">overwrite</a>._
+
+- Create register file.
+
+<table>
+  <tbody>
+    <tr>
+      <th>Targets</th>
+      <th>Requires</th>
+    </tr>
+    <tr>
+      <td align="left" valign="top">
+        <ul>
+<code>project</code><br/>
+<code>└─&nbsp;test</code><br/>
+<code>&nbsp;&nbsp;&nbsp;└─&nbsp;<a href="#blackfluxrobo-config-plugin-target-ref-testregjs">reg.js</a></code><br/>
+        </ul>
+      </td>
+      <td align="left" valign="top">
+        <ul>
+          <li><a href="#blackfluxrobo-config-plugin-req-ref-javascript">javascript</a></li>
+        </ul>
+      </td>
+    </tr>
+  </tbody>
+</table>
+
 ###### :clipboard: <a name="blackfluxrobo-config-plugin-task-ref-structtest-hot">struct/test-hot</a> (<a href="#blackfluxrobo-config-plugin-task-idx-ref-structtest-hot">`index`</a>)
 
 _Updating <a href="#blackfluxrobo-config-plugin-target-ref-testhotjs">test/hot.js</a> using <a href="#blackfluxrobo-config-plugin-strat-ref-overwrite">overwrite</a>._
 
-- Create custom loader file.
+- Create loader file.
 
 <table>
   <tbody>
@@ -1651,11 +1684,17 @@ The code owner is the gatekeeper of the project, ultimately responsible for all
 
 Execute all configured js-gardener tasks.
 
+### <a name="blackfluxrobo-config-plugin-target-ref-testregjs">test/reg.js</a>  
+
+:small_blue_diamond: `other`
+
+*Registration hook to load custom loader.*
+
 ### <a name="blackfluxrobo-config-plugin-target-ref-testhotjs">test/hot.js</a>  
 
 :small_blue_diamond: `other`
 
-*Custom loader to allow hot reloading of files for testing.*
+*Loader to allow hot reloading of files for testing.*
 
 ### <a name="blackfluxrobo-config-plugin-target-ref-readmemd">README.md</a> ([`link`](https://help.github.com/en/articles/about-readmes)) 
 

README.md

@@ -54,7 +54,7 @@ export default api;
 
 then hook the router endpoint into the [Serverless Framework](https://serverless.com/) configuration as
 
-```yaml
+```
 functions:
   router:
     handler: handler.router

eslint.config.js

@@ -13,7 +13,7 @@ export default [
   ...compat.extends(
     'airbnb-base',
     'plugin:mocha/recommended',
-    'plugin:markdown/recommended'
+    'plugin:markdown/recommended-legacy'
   ),
   ...compat.env({ es6: true, node: true, mocha: true }),
   ...compat.plugins('json', 'mocha'),

package.json

@@ -14,27 +14,27 @@
   ],
   "author": "Lukas Siemon",
   "devDependencies": {
-    "@aws-sdk/client-s3": "^3.414.0",
-    "@babel/core": "7.22.20",
-    "@babel/eslint-parser": "7.22.15",
-    "@babel/register": "7.22.15",
-    "@blackflux/eslint-plugin-rules": "3.0.1",
-    "@blackflux/robo-config-plugin": "9.1.16",
-    "aws-sdk-wrap": "13.1.0",
-    "c8": "8.0.1",
-    "chai": "4.3.8",
-    "eslint": "8.49.0",
+    "@aws-sdk/client-s3": "^3.735.0",
+    "@babel/core": "7.26.7",
+    "@babel/eslint-parser": "7.26.5",
+    "@babel/register": "7.25.9",
+    "@blackflux/eslint-plugin-rules": "4.0.0",
+    "@blackflux/robo-config-plugin": "10.0.0",
+    "aws-sdk-wrap": "13.3.2",
+    "c8": "10.1.3",
+    "chai": "5.1.2",
+    "eslint": "9.19.0",
     "eslint-config-airbnb-base": "15.0.0",
-    "eslint-plugin-import": "2.28.1",
-    "eslint-plugin-json": "3.1.0",
-    "eslint-plugin-markdown": "3.0.1",
-    "eslint-plugin-mocha": "10.1.0",
-    "js-gardener": "5.0.2",
-    "lambda-tdd": "6.3.7",
+    "eslint-plugin-import": "2.31.0",
+    "eslint-plugin-json": "4.0.1",
+    "eslint-plugin-markdown": "5.1.0",
+    "eslint-plugin-mocha": "10.5.0",
+    "js-gardener": "6.0.2",
+    "lambda-tdd": "8.0.0",
     "minimist": "1.2.8",
-    "moment-timezone": "0.5.43",
-    "nock": "13.3.3",
-    "node-tdd": "4.0.1",
+    "moment-timezone": "0.5.47",
+    "nock": "14.0.0",
+    "node-tdd": "6.0.0",
     "request": "2.88.2",
     "request-promise": "4.2.6"
   },
@@ -43,9 +43,9 @@
     "clean": "rm -rf lib",
     "build": "cp -rf ./src ./lib",
     "build-clean": "yarn run clean && yarn run build",
-    "test-simple": "c8 mocha --experimental-loader=./test/hot.js \"./test/**/*.spec.js\"",
+    "test-simple": "c8 mocha --import=./test/reg.js \"./test/**/*.spec.js\"",
     "test": "yarn run clean && yarn run gardener && yarn run test-simple",
-    "semantic-release": "yarn run build-clean && npx semantic-release",
+    "semantic-release": "yarn run build-clean && yarn exec semantic-release",
     "gardener": "node gardener.js",
     "docker": "docker run --net host -u`id -u`:`id -g` -v $(pwd):/user/project -v ~/.aws:/user/.aws -v ~/.npmrc:/user/.npmrc -w /user/project -it --entrypoint /bin/bash",
     "t": "yarn test",
@@ -64,7 +64,7 @@
     }
   ],
   "engines": {
-    "node": ">= 16"
+    "node": ">= 20"
   },
   "repository": {
     "type": "git",
@@ -77,22 +77,22 @@
     "lib"
   ],
   "dependencies": {
-    "@turf/kinks": "6.5.0",
-    "joi-strict": "3.0.1",
-    "lambda-async": "3.0.0",
-    "lambda-monitor-logger": "5.0.1",
+    "@turf/kinks": "7.2.0",
+    "joi-strict": "4.0.0",
+    "lambda-async": "4.0.0",
+    "lambda-monitor-logger": "6.0.0",
     "lodash.clonedeep": "4.5.0",
     "lodash.difference": "4.5.0",
     "lodash.get": "4.4.2",
     "lodash.set": "4.3.2",
-    "lru-cache-ext": "4.1.1",
-    "object-fields": "4.0.0",
-    "object-hash-strict": "4.0.1",
-    "object-scan": "19.0.2",
-    "painless-version": "3.0.0",
+    "lru-cache-ext": "5.0.0",
+    "object-fields": "5.0.0",
+    "object-hash-strict": "5.0.0",
+    "object-scan": "20.0.0",
+    "painless-version": "4.0.0",
     "route-recognizer": "0.3.4",
-    "serialize-error": "11.0.2",
-    "smart-fs": "4.0.1",
+    "serialize-error": "12.0.0",
+    "smart-fs": "5.0.1",
     "stack-trace": "1.0.0-pre2",
     "swagger-parser": "10.0.3",
     "turf-polygon": "1.0.3"

src/param/json.js

@@ -17,7 +17,7 @@ class Json extends Abstract {
     if (valid && this.stringInput) {
       try {
         valueParsed = JSON.parse(value);
-      } catch (e) {
+      } catch {
         valid = false;
       }
     }