add lint check for missing deps

toddbaert · toddbaert · commit a7db6bd957f5 · 2025-01-08T15:06:32.000-05:00
Signed-off-by: Todd Baert &lt;todd.baert@dynatrace.com&gt;
diff --git a/.eslintrc.json b/.eslintrc.json
@@ -42,6 +42,24 @@
       "files": "*.json",
       "parser": "jsonc-eslint-parser",
       "rules": {}
+    },
+    {
+      "files": ["*.json"],
+      "parser": "jsonc-eslint-parser",
+      "rules": {
+        "@nx/dependency-checks": [
+          "error",
+          {
+            "buildTargets": ["lint"],
+            "includeTransitiveDependencies": false,
+            "checkMissingDependencies": true,
+            "checkObsoleteDependencies": true,
+            "checkVersionMismatches": true,
+            "ignoredDependencies": ["jest-cucumber"],
+            "ignoredFiles": ["**/*.spec.ts", "**/*.spec.js", "**/*.test.ts", "**/*.test.js"]
+          }
+        ]
+      }
     }
   ]
 }
diff --git a/libs/hooks/open-telemetry/babel.config.json b/libs/hooks/open-telemetry/babel.config.json
@@ -1,3 +1,3 @@
 {
   "presets": [["minify", { "builtIns": false }]]
-}
+}
diff --git a/libs/providers/config-cat-web/package.json b/libs/providers/config-cat-web/package.json
@@ -7,6 +7,7 @@
   },
   "peerDependencies": {
     "@openfeature/web-sdk": "^1.0.0",
-    "configcat-js-ssr": "^8.4.3"
+    "configcat-js-ssr": "^8.4.3",
+    "@openfeature/config-cat-core": "0.1.0"
   }
 }
diff --git a/libs/providers/config-cat/package.json b/libs/providers/config-cat/package.json
@@ -7,6 +7,8 @@
   },
   "peerDependencies": {
     "@openfeature/server-sdk": "^1.13.5",
-    "configcat-node": "^11.3.1"
+    "configcat-node": "^11.3.1",
+    "@openfeature/config-cat-core": "0.1.0",
+    "configcat-common": "9.3.1"
   }
 }
diff --git a/libs/providers/flagd-web/package.json b/libs/providers/flagd-web/package.json
@@ -7,5 +7,12 @@
   },
   "peerDependencies": {
     "@openfeature/web-sdk": "^1.0.0"
+
+  },
+  "dependencies": {
+    "@openfeature/flagd-core": "1.0.0",
+    "@connectrpc/connect": "^1.4.0",
+    "@connectrpc/connect-web": "^1.4.0",
+    "@bufbuild/protobuf": "^1.2.0"
   }
 }
diff --git a/libs/providers/flagd/docker-compose.yaml b/libs/providers/flagd/docker-compose.yaml
diff --git a/libs/providers/flagd/package.json b/libs/providers/flagd/package.json
@@ -7,6 +7,10 @@
   },
   "peerDependencies": {
     "@grpc/grpc-js": "~1.8.0 || ~1.9.0 || ~1.10.0 || ~1.11.0 || ~1.12.0",
-    "@openfeature/server-sdk": "^1.13.0"
+    "@openfeature/server-sdk": "^1.13.0",
+    "@openfeature/flagd-core": "^1.0.0"
+  },
+  "dependencies": {
+    "lru-cache": "^11.0.0"
   }
 }
diff --git a/libs/providers/flagd/src/lib/service/grpc/grpc-service.ts b/libs/providers/flagd/src/lib/service/grpc/grpc-service.ts
@@ -13,7 +13,7 @@ import {
   TypeMismatchError,
 } from '@openfeature/server-sdk';
 import { LRUCache } from 'lru-cache';
-import { promisify } from 'util';
+import { promisify } from 'node:util';
 import {
   EventStreamResponse,
   ResolveBooleanRequest,
diff --git a/libs/providers/flagd/src/lib/service/in-process/file/file-fetch.spec.ts b/libs/providers/flagd/src/lib/service/in-process/file/file-fetch.spec.ts
@@ -1,7 +1,7 @@
 import fs from 'fs';
 import { FileFetch } from './file-fetch';
 import { FlagdCore } from '@openfeature/flagd-core';
-import { Logger } from '@openfeature/core';
+import { Logger } from '@openfeature/server-sdk';
 
 jest.mock('fs', () => ({
   ...jest.requireActual('fs'),
diff --git a/libs/providers/flagd/src/lib/service/in-process/file/file-fetch.ts b/libs/providers/flagd/src/lib/service/in-process/file/file-fetch.ts
@@ -1,7 +1,6 @@
-import { Logger, OpenFeatureError } from '@openfeature/core';
+import { Logger, OpenFeatureError, GeneralError } from '@openfeature/server-sdk';
 import { DataFetch } from '../data-fetch';
 import { promises as fsPromises, watchFile, unwatchFile } from 'fs';
-import { GeneralError } from '@openfeature/core';
 
 const encoding = 'utf8';
 export class FileFetch implements DataFetch {
diff --git a/libs/providers/flagd/src/lib/service/in-process/grpc/grpc-fetch.ts b/libs/providers/flagd/src/lib/service/in-process/grpc/grpc-fetch.ts
@@ -1,5 +1,5 @@
 import { ClientReadableStream, ServiceError, credentials } from '@grpc/grpc-js';
-import { Logger } from '@openfeature/core';
+import { Logger } from '@openfeature/server-sdk';
 import { GeneralError } from '@openfeature/server-sdk';
 import { FlagSyncServiceClient, SyncFlagsRequest, SyncFlagsResponse } from '../../../../proto/ts/flagd/sync/v1/sync';
 import { Config } from '../../../configuration';
diff --git a/libs/providers/flagd/src/lib/service/in-process/in-process-service.ts b/libs/providers/flagd/src/lib/service/in-process/in-process-service.ts
@@ -1,10 +1,17 @@
-import { Service } from '../service';
-import { EvaluationContext, FlagValue, JsonValue, Logger, ResolutionDetails, FlagValueType } from '@openfeature/core';
-import { Config } from '../../configuration';
 import { FlagdCore } from '@openfeature/flagd-core';
+import type {
+  EvaluationContext,
+  FlagValue,
+  FlagValueType,
+  JsonValue,
+  Logger,
+  ResolutionDetails,
+} from '@openfeature/server-sdk';
+import { Config } from '../../configuration';
+import { Service } from '../service';
 import { DataFetch } from './data-fetch';
-import { GrpcFetch } from './grpc/grpc-fetch';
 import { FileFetch } from './file/file-fetch';
+import { GrpcFetch } from './grpc/grpc-fetch';
 
 export class InProcessService implements Service {
   private _flagdCore: FlagdCore;
diff --git a/libs/providers/flagsmith-client/package.json b/libs/providers/flagsmith-client/package.json
@@ -1,9 +1,7 @@
 {
   "name": "@openfeature/flagsmith-client-provider",
   "version": "0.1.2",
-  "dependencies": {
-    "tslib": "^2.3.0"
-  },
+
   "main": "./src/index.js",
   "typings": "./src/index.d.ts",
   "scripts": {
@@ -13,5 +11,6 @@
   "peerDependencies": {
     "@openfeature/web-sdk": "^1.0.0",
     "flagsmith": "4.1.4"
-  }
+  },
+  "dependencies": {}
 }
diff --git a/libs/providers/flipt-web/package.json b/libs/providers/flipt-web/package.json
@@ -1,10 +1,6 @@
 {
   "name": "@openfeature/flipt-web-provider",
   "version": "0.1.1",
-  "dependencies": {
-    "@flipt-io/flipt-client-browser": "^0.3.1",
-    "tslib": "^2.3.0"
-  },
   "main": "./src/index.js",
   "typings": "./src/index.d.ts",
   "scripts": {
@@ -16,5 +12,9 @@
   },
   "devDependencies": {
     "undici": "^6.13.0"
+  },
+  "dependencies": {
+    "@flipt-io/flipt-client-browser": "^0.3.1",
+    "undici": "^5.0.0"
   }
 }
diff --git a/libs/providers/flipt-web/src/lib/context-transformer.ts b/libs/providers/flipt-web/src/lib/context-transformer.ts
@@ -1,4 +1,4 @@
-import { EvaluationContext } from '@openfeature/server-sdk';
+import { EvaluationContext } from '@openfeature/web-sdk';
 
 export function transformContext(context: EvaluationContext): Record<string, string> {
   const evalContext: Record<string, string> = {};
diff --git a/libs/providers/flipt/package.json b/libs/providers/flipt/package.json
@@ -1,9 +1,6 @@
 {
   "name": "@openfeature/flipt-provider",
   "version": "0.1.2",
-  "dependencies": {
-    "tslib": "^2.3.0"
-  },
   "main": "./src/index.js",
   "typings": "./src/index.d.ts",
   "scripts": {
@@ -13,5 +10,6 @@
   "peerDependencies": {
     "@openfeature/server-sdk": "^1.13.0",
     "@flipt-io/flipt": "^1.2.0"
-  }
+  },
+  "dependencies": {}
 }
diff --git a/libs/providers/go-feature-flag-web/package.json b/libs/providers/go-feature-flag-web/package.json
@@ -7,5 +7,8 @@
   },
   "peerDependencies": {
     "@openfeature/web-sdk": "^1.0.0"
+  },
+  "dependencies": {
+    "copy-anything": "^3.0.5"
   }
 }
diff --git a/libs/providers/go-feature-flag-web/src/lib/data-collector-hook.ts b/libs/providers/go-feature-flag-web/src/lib/data-collector-hook.ts
@@ -1,4 +1,4 @@
-import { EvaluationDetails, FlagValue, Hook, HookContext, Logger } from '@openfeature/server-sdk';
+import { EvaluationDetails, FlagValue, Hook, HookContext, Logger } from '@openfeature/web-sdk';
 import { FeatureEvent, GoFeatureFlagWebProviderOptions } from './model';
 import { copy } from 'copy-anything';
 import { CollectorError } from './errors/collector-error';
diff --git a/libs/providers/go-feature-flag-web/tsconfig.lib.json b/libs/providers/go-feature-flag-web/tsconfig.lib.json
@@ -5,9 +5,8 @@
     "outDir": "../../../dist/out-tsc",
     "declaration": true,
     "types": [],
-    "allowSyntheticDefaultImports": true,
+    "allowSyntheticDefaultImports": true
   },
   "include": ["**/*.ts"],
   "exclude": ["jest.config.ts", "**/*.spec.ts", "**/*.test.ts"]
-
 }
diff --git a/libs/providers/go-feature-flag/package.json b/libs/providers/go-feature-flag/package.json
@@ -7,5 +7,11 @@
   },
   "peerDependencies": {
     "@openfeature/server-sdk": "^1.15.0"
+  },
+  "dependencies": {
+    "object-hash": "^3.0.0",
+    "lru-cache": "^11.0.0",
+    "axios": "1.7.9",
+    "copy-anything": "^3.0.5"
   }
 }
diff --git a/libs/providers/growthbook-client/package.json b/libs/providers/growthbook-client/package.json
@@ -2,7 +2,7 @@
   "name": "@openfeature/growthbook-client-provider",
   "version": "0.1.2",
   "dependencies": {
-    "tslib": "^2.3.0"
+    "lodash.isempty": "^4.4.0"
   },
   "main": "./src/index.js",
   "typings": "./src/index.d.ts",
diff --git a/libs/providers/growthbook-client/src/lib/translate-result.spec.ts b/libs/providers/growthbook-client/src/lib/translate-result.spec.ts
@@ -1,4 +1,4 @@
-import { TypeMismatchError } from '@openfeature/core';
+import { TypeMismatchError } from '@openfeature/server-sdk';
 import translateResult from './translate-result';
 
 describe('translateResult', () => {
diff --git a/libs/providers/launchdarkly-client/package.json b/libs/providers/launchdarkly-client/package.json
@@ -8,5 +8,8 @@
   "peerDependencies": {
     "@openfeature/web-sdk": "^1.0.0",
     "launchdarkly-js-client-sdk": ">=3.1.3"
+  },
+  "dependencies": {
+    "lodash.isempty": "^4.4.0"
   }
 }
diff --git a/libs/providers/multi-provider-web/package.json b/libs/providers/multi-provider-web/package.json
@@ -1,9 +1,6 @@
 {
   "name": "@openfeature/multi-provider-web",
   "version": "0.0.3",
-  "dependencies": {
-    "tslib": "^2.3.0"
-  },
   "main": "./src/index.js",
   "typings": "./src/index.d.ts",
   "scripts": {
@@ -12,5 +9,6 @@
   },
   "peerDependencies": {
     "@openfeature/web-sdk": "^1.4.0"
-  }
+  },
+  "dependencies": {}
 }
diff --git a/libs/providers/multi-provider/package.json b/libs/providers/multi-provider/package.json
@@ -1,9 +1,6 @@
 {
   "name": "@openfeature/multi-provider",
   "version": "0.1.2",
-  "dependencies": {
-    "tslib": "^2.3.0"
-  },
   "main": "./src/index.js",
   "typings": "./src/index.d.ts",
   "scripts": {
@@ -12,5 +9,6 @@
   },
   "peerDependencies": {
     "@openfeature/server-sdk": "^1.17.0"
-  }
+  },
+  "dependencies": {}
 }
diff --git a/libs/providers/ofrep-web/jest.polyfills.js b/libs/providers/ofrep-web/jest.polyfills.js
@@ -15,7 +15,7 @@ const { TextDecoder, TextEncoder } = require('node:util');
 Object.defineProperties(globalThis, {
   TextDecoder: { value: TextDecoder },
   TextEncoder: { value: TextEncoder },
-})
+});
 
 const { Blob, File } = require('node:buffer');
 const { fetch, Headers, FormData, Request, Response } = require('undici');
@@ -29,4 +29,4 @@ Object.defineProperties(globalThis, {
   FormData: { value: FormData },
   Request: { value: Request },
   Response: { value: Response },
-})
+});
diff --git a/libs/providers/ofrep-web/package.json b/libs/providers/ofrep-web/package.json
@@ -1,9 +1,6 @@
 {
   "name": "@openfeature/ofrep-web-provider",
   "version": "0.3.1",
-  "dependencies": {
-    "tslib": "^2.3.0"
-  },
   "main": "./src/index.js",
   "typings": "./src/index.d.ts",
   "scripts": {
@@ -12,5 +9,9 @@
   },
   "peerDependencies": {
     "@openfeature/web-sdk": "^1.4.0"
+  },
+  "dependencies": {
+    "undici": "^5.0.0",
+    "@openfeature/ofrep-core": "0.2.0"
   }
 }
diff --git a/libs/providers/ofrep-web/src/lib/ofrep-web-provider.ts b/libs/providers/ofrep-web/src/lib/ofrep-web-provider.ts
@@ -1,9 +1,3 @@
-import {
-  InvalidContextError,
-  ParseError,
-  StandardResolutionReasons,
-  TargetingKeyMissingError,
-} from '@openfeature/core';
 import {
   EvaluationFailureErrorCode,
   EvaluationRequest,
@@ -25,13 +19,17 @@ import {
   FlagValue,
   GeneralError,
   Hook,
+  InvalidContextError,
   JsonValue,
   Logger,
   OpenFeatureError,
   OpenFeatureEventEmitter,
+  ParseError,
   Provider,
   ProviderFatalError,
   ResolutionDetails,
+  StandardResolutionReasons,
+  TargetingKeyMissingError,
   TypeMismatchError,
 } from '@openfeature/web-sdk';
 import { BulkEvaluationStatus, EvaluateFlagsResponse } from './model/evaluate-flags-response';
diff --git a/libs/providers/ofrep-web/tsconfig.lib.json b/libs/providers/ofrep-web/tsconfig.lib.json
@@ -5,9 +5,8 @@
     "outDir": "../../../dist/out-tsc",
     "declaration": true,
     "types": [],
-    "allowSyntheticDefaultImports": true,
+    "allowSyntheticDefaultImports": true
   },
   "include": ["**/*.ts"],
   "exclude": ["jest.config.ts", "**/*.spec.ts", "**/*.test.ts"]
-
 }
diff --git a/libs/providers/ofrep/package.json b/libs/providers/ofrep/package.json
diff --git a/libs/providers/ofrep/src/lib/ofrep-provider.spec.ts b/libs/providers/ofrep/src/lib/ofrep-provider.spec.ts
diff --git a/libs/providers/ofrep/src/lib/ofrep-provider.ts b/libs/providers/ofrep/src/lib/ofrep-provider.ts
diff --git a/libs/shared/config-cat-core/package.json b/libs/shared/config-cat-core/package.json
diff --git a/libs/shared/flagd-core/.eslintrc.json b/libs/shared/flagd-core/.eslintrc.json
diff --git a/libs/shared/flagd-core/package.json b/libs/shared/flagd-core/package.json
diff --git a/libs/shared/ofrep-core/package.json b/libs/shared/ofrep-core/package.json
diff --git a/libs/shared/ofrep-core/src/lib/api/errors.ts b/libs/shared/ofrep-core/src/lib/api/errors.ts
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
diff --git a/tools/workspace-plugin/package.json b/tools/workspace-plugin/package.json

Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,3 @@`
`1`	`1`	`{`
`2`	`2`	`"presets": [["minify", { "builtIns": false }]]`
`3`		`-}`
	`3`	`+}`
Original file line number	Diff line number	Diff line change
`@@ -7,6 +7,7 @@`
`7`	`7`	`},`
`8`	`8`	`"peerDependencies": {`
`9`	`9`	`"@openfeature/web-sdk": "^1.0.0",`
`10`		`- "configcat-js-ssr": "^8.4.3"`
	`10`	`+ "configcat-js-ssr": "^8.4.3",`
	`11`	`+ "@openfeature/config-cat-core": "0.1.0"`
`11`	`12`	`}`
`12`	`13`	`}`
Original file line number	Diff line number	Diff line change
`@@ -7,6 +7,8 @@`
`7`	`7`	`},`
`8`	`8`	`"peerDependencies": {`
`9`	`9`	`"@openfeature/server-sdk": "^1.13.5",`
`10`		`- "configcat-node": "^11.3.1"`
	`10`	`+ "configcat-node": "^11.3.1",`
	`11`	`+ "@openfeature/config-cat-core": "0.1.0",`
	`12`	`+ "configcat-common": "9.3.1"`
`11`	`13`	`}`
`12`	`14`	`}`
Original file line number	Diff line number	Diff line change
`@@ -7,5 +7,12 @@`
`7`	`7`	`},`
`8`	`8`	`"peerDependencies": {`
`9`	`9`	`"@openfeature/web-sdk": "^1.0.0"`
	`10`	`+`
	`11`	`+ },`
	`12`	`+ "dependencies": {`
	`13`	`+ "@openfeature/flagd-core": "1.0.0",`
	`14`	`+ "@connectrpc/connect": "^1.4.0",`
	`15`	`+ "@connectrpc/connect-web": "^1.4.0",`
	`16`	`+ "@bufbuild/protobuf": "^1.2.0"`
`10`	`17`	`}`
`11`	`18`	`}`
Original file line number	Diff line number	Diff line change
`@@ -7,6 +7,10 @@`
`7`	`7`	`},`
`8`	`8`	`"peerDependencies": {`
`9`	`9`	`"@grpc/grpc-js": "~1.8.0 \|\| ~1.9.0 \|\| ~1.10.0 \|\| ~1.11.0 \|\| ~1.12.0",`
`10`		`- "@openfeature/server-sdk": "^1.13.0"`
	`10`	`+ "@openfeature/server-sdk": "^1.13.0",`
	`11`	`+ "@openfeature/flagd-core": "^1.0.0"`
	`12`	`+ },`
	`13`	`+ "dependencies": {`
	`14`	`+ "lru-cache": "^11.0.0"`
`11`	`15`	`}`
`12`	`16`	`}`