Merge pull request #1321 from hasanheroglu/1319-detect-issue-with-dep…

…endencies-in-monorepo detect phantom dependencies and fix
eclipse-thingweb · Aug 28, 2024 · daf620a · daf620a
2 parents 99b8303 + 1f54133
commit daf620a
Show file tree

Hide file tree

Showing 24 changed files with 343 additions and 23 deletions.
diff --git a/.eslintrc.js b/.eslintrc.js
@@ -32,6 +32,7 @@ module.exports = {
         "@typescript-eslint/prefer-nullish-coalescing": "error",
         "unused-imports/no-unused-imports": "error",
         "@typescript-eslint/strict-boolean-expressions": "error",
+        "import/no-extraneous-dependencies": "error",
         "guard-for-in": "error",
         "unused-imports/no-unused-vars": [
             "warn",

diff --git a/package-lock.json b/package-lock.json
diff --git a/packages/binding-coap/package.json b/packages/binding-coap/package.json
@@ -15,8 +15,8 @@
     "types": "dist/coap.d.ts",
     "dependencies": {
         "@node-wot/core": "0.8.17",
-        "@types/node": "16.18.35",
         "coap": "^1.4.0",
+        "dns-packet": "^5.2.2",
         "multicast-dns": "^7.2.5",
         "node-coap-client": "1.0.8",
         "rxjs": "5.5.11",

diff --git a/packages/binding-coap/test/.eslintrc.json b/packages/binding-coap/test/.eslintrc.json
@@ -0,0 +1,6 @@
+{
+    "extends": "../.eslintrc.json",
+    "rules": {
+        "import/no-extraneous-dependencies": "off"
+    }
+}
diff --git a/packages/binding-file/package.json b/packages/binding-file/package.json
@@ -14,7 +14,8 @@
     "main": "dist/file.js",
     "types": "dist/file.d.ts",
     "dependencies": {
-        "@node-wot/core": "0.8.17"
+        "@node-wot/core": "0.8.17",
+        "rxjs": "^5.5.11"
     },
     "scripts": {
         "build": "tsc -b",

diff --git a/packages/binding-file/test/.eslintrc.json b/packages/binding-file/test/.eslintrc.json
@@ -0,0 +1,6 @@
+{
+    "extends": "../.eslintrc.json",
+    "rules": {
+        "import/no-extraneous-dependencies": "off"
+    }
+}
diff --git a/packages/binding-http/package.json b/packages/binding-http/package.json
@@ -20,16 +20,20 @@
     "devDependencies": {
         "@node-oauth/express-oauth-server": "^3.0.1",
         "@node-oauth/oauth2-server": "^4.3.0",
+        "@sinonjs/fake-timers": "^11.2.2",
         "@types/accept-language-parser": "^1.5.2",
         "@types/basic-auth": "1.1.3",
+        "@types/eventsource": "^1.1.10",
         "@types/express": "^4.17.3",
         "@types/node-fetch": "^2.5.10",
+        "body-parser": "^1.20.1",
+        "express": "^4.18.2",
+        "serve-static": "^1.15.0",
         "ssestream": "^1.1.0",
         "timekeeper": "^2.2.0"
     },
     "dependencies": {
         "@node-wot/core": "0.8.17",
-        "@types/eventsource": "1.1.10",
         "accept-language-parser": "1.5.0",
         "basic-auth": "2.0.1",
         "client-oauth2": "^4.2.5",

diff --git a/packages/binding-http/test/.eslintrc.json b/packages/binding-http/test/.eslintrc.json
@@ -0,0 +1,6 @@
+{
+    "extends": "../.eslintrc.json",
+    "rules": {
+        "import/no-extraneous-dependencies": "off"
+    }
+}
diff --git a/packages/binding-mbus/package.json b/packages/binding-mbus/package.json
@@ -16,7 +16,7 @@
     "dependencies": {
         "@node-wot/core": "0.8.17",
         "node-mbus": "^2.2.4",
-        "wot-typescript-definitions": "0.8.0-SNAPSHOT.29"
+        "rxjs": "^5.5.11"
     },
     "scripts": {
         "build": "tsc -b",

diff --git a/packages/binding-mbus/test/.eslintrc.json b/packages/binding-mbus/test/.eslintrc.json
@@ -0,0 +1,6 @@
+{
+    "extends": "../.eslintrc.json",
+    "rules": {
+        "import/no-extraneous-dependencies": "off"
+    }
+}
diff --git a/packages/binding-modbus/test/.eslintrc.json b/packages/binding-modbus/test/.eslintrc.json
@@ -0,0 +1,6 @@
+{
+    "extends": "../.eslintrc.json",
+    "rules": {
+        "import/no-extraneous-dependencies": "off"
+    }
+}
diff --git a/packages/binding-mqtt/test/.eslintrc.json b/packages/binding-mqtt/test/.eslintrc.json
@@ -0,0 +1,6 @@
+{
+    "extends": "../.eslintrc.json",
+    "rules": {
+        "import/no-extraneous-dependencies": "off"
+    }
+}
diff --git a/packages/binding-netconf/package.json b/packages/binding-netconf/package.json
@@ -15,10 +15,9 @@
     "types": "dist/netconf.d.ts",
     "dependencies": {
         "@node-wot/core": "0.8.17",
-        "@types/node-netconf": "npm:@types/netconf@^2.0.0",
-        "@types/url-parse": "^1.4.3",
         "case-1.5.3": "npm:case@^1.5.3",
         "node-netconf": "^1.1.2",
+        "rxjs": "^5.5.11",
         "url-parse": "^1.5.8"
     },
     "scripts": {
@@ -35,5 +34,9 @@
     "directories": {
         "test": "test"
     },
-    "keywords": []
+    "keywords": [],
+    "devDependencies": {
+        "@types/node-netconf": "npm:@types/netconf@^2.0.0",
+        "@types/url-parse": "^1.4.3"
+    }
 }
diff --git a/packages/binding-netconf/test/.eslintrc.json b/packages/binding-netconf/test/.eslintrc.json
@@ -0,0 +1,6 @@
+{
+    "extends": "../.eslintrc.json",
+    "rules": {
+        "import/no-extraneous-dependencies": "off"
+    }
+}
diff --git a/packages/binding-opcua/package.json b/packages/binding-opcua/package.json
@@ -19,6 +19,8 @@
     },
     "dependencies": {
         "@node-wot/core": "0.8.17",
+        "ajv": "^8.11.0",
+        "ajv-formats": "^2.1.1",
         "node-opcua": "2.113.0",
         "node-opcua-address-space": "2.113.0",
         "node-opcua-basic-types": "2.113.0",

diff --git a/packages/binding-opcua/test/.eslintrc.json b/packages/binding-opcua/test/.eslintrc.json
@@ -0,0 +1,6 @@
+{
+    "extends": "../.eslintrc.json",
+    "rules": {
+        "import/no-extraneous-dependencies": "off"
+    }
+}
diff --git a/packages/binding-websockets/package.json b/packages/binding-websockets/package.json
@@ -17,6 +17,7 @@
     "dependencies": {
         "@node-wot/binding-http": "0.8.17",
         "@node-wot/core": "0.8.17",
+        "rxjs": "^5.5.11",
         "slugify": "^1.4.5",
         "ws": "^7.5.10"
     },

diff --git a/packages/binding-websockets/test/.eslintrc.json b/packages/binding-websockets/test/.eslintrc.json
@@ -0,0 +1,6 @@
+{
+    "extends": "../.eslintrc.json",
+    "rules": {
+        "import/no-extraneous-dependencies": "off"
+    }
+}
diff --git a/packages/cli/package.json b/packages/cli/package.json
@@ -27,7 +27,6 @@
         "@node-wot/binding-websockets": "0.8.17",
         "@node-wot/core": "0.8.17",
         "@thingweb/thing-model": "^1.0.1",
-        "@types/lodash": "^4.14.199",
         "ajv": "^8.11.0",
         "commander": "^9.1.0",
         "dotenv": "^8.6.0",
@@ -47,5 +46,8 @@
         "url": "https://github.com/eclipse-thingweb/node-wot/issues"
     },
     "homepage": "https://github.com/eclipse-thingweb/node-wot/tree/master/packages/cli#readme",
-    "keywords": []
+    "keywords": [],
+    "devDependencies": {
+        "@types/lodash": "^4.14.199"
+    }
 }
diff --git a/packages/cli/test/.eslintrc.json b/packages/cli/test/.eslintrc.json
@@ -0,0 +1,6 @@
+{
+    "extends": "../.eslintrc.json",
+    "rules": {
+        "import/no-extraneous-dependencies": "off"
+    }
+}
diff --git a/packages/core/package.json b/packages/core/package.json
@@ -18,7 +18,8 @@
         "@types/content-type": "^1.1.8",
         "@types/debug": "^4.1.7",
         "@types/uritemplate": "^0.3.4",
-        "@types/uuid": "^8.3.1"
+        "@types/uuid": "^8.3.1",
+        "wot-thing-description-types": "^1.1.0-09-November-2023"
     },
     "dependencies": {
         "@petamoriken/float16": "^3.1.1",
@@ -29,6 +30,7 @@
         "content-type": "^1.0.5",
         "debug": "^4.3.4",
         "is-absolute-url": "3.0.3",
+        "rxjs": "^5.5.11",
         "uritemplate": "0.3.4",
         "url-toolkit": "2.1.6",
         "uuid": "^7.0.3",

diff --git a/packages/core/test/.eslintrc.json b/packages/core/test/.eslintrc.json
@@ -0,0 +1,6 @@
+{
+    "extends": "../.eslintrc.json",
+    "rules": {
+        "import/no-extraneous-dependencies": "off"
+    }
+}
diff --git a/packages/td-tools/package.json b/packages/td-tools/package.json
@@ -16,6 +16,7 @@
     "types": "dist/td-tools.d.ts",
     "dependencies": {
         "ajv": "^8.11.0",
+        "ajv-formats": "^2.1.1",
         "debug": "^4.3.4",
         "is-absolute-url": "3.0.3",
         "json-placeholder-replacer": "^1.0.35",

diff --git a/packages/td-tools/test/.eslintrc.json b/packages/td-tools/test/.eslintrc.json
@@ -0,0 +1,6 @@
+{
+    "extends": "../.eslintrc.json",
+    "rules": {
+        "import/no-extraneous-dependencies": "off"
+    }
+}