chore: import type lint rule and fixes

.eslintrc.json

@@ -21,6 +21,14 @@
 		"jsdoc"
 	],
 	"rules": {
+    "@typescript-eslint/consistent-type-imports": [
+       "error",
+       {
+        "disallowTypeAnnotations": true,
+        "fixStyle": "separate-type-imports",
+        "prefer": "type-imports"
+      }
+    ],
 		"jsdoc/require-jsdoc": [
 			"warn",
 			{
@@ -60,4 +68,4 @@
 			}
 		]
 	}
-}
+}

package.json

@@ -9,6 +9,7 @@
     "e2e-web": "git submodule update --init --recursive && shx cp test-harness/features/evaluation.feature packages/web/e2e/features && jest --selectProjects=web-e2e --verbose",
     "e2e": "npm run e2e-server && npm run e2e-web",
     "lint": "npm run lint --workspace=packages/shared --workspace=packages/server --workspace=packages/web --workspace=packages/react --workspace=packages/angular --workspace=packages/nest",
+    "lint:fix": "npm run lint:fix --workspace=packages/shared --workspace=packages/server --workspace=packages/web --workspace=packages/react --workspace=packages/angular --workspace=packages/nest",
     "clean": "shx rm -rf ./dist",
     "build": "npm run build --workspace=packages/shared --workspace=packages/server --workspace=packages/web --workspace=packages/react --workspace=packages/angular --workspace=packages/nest",
     "publish-all": "npm run publish-if-not-exists --workspace=packages/shared --workspace=packages/server --workspace=packages/web --workspace=packages/react --workspace=packages/angular --workspace=packages/nest",

packages/angular/angular.json

@@ -39,6 +39,7 @@
   "cli": {
     "schematicCollections": [
       "@angular-eslint/schematics"
-    ]
+    ],
+    "analytics": false
   }
 }

packages/angular/package.json

@@ -5,6 +5,7 @@
     "ng": "ng",
     "start": "ng serve",
     "lint": "ng lint",
+    "lint:fix": "ng lint --fix",
     "watch": "ng build --watch --configuration development",
     "test": "jest",
     "build": "ng build && npm run postbuild",

packages/nest/package.json

@@ -16,6 +16,7 @@
   "scripts": {
     "test": "jest --verbose",
     "lint": "eslint ./",
+    "lint:fix": "eslint ./ --fix",
     "clean": "shx rm -rf ./dist",
     "build:esm": "esbuild src/index.ts --bundle --external:@nestjs/* --external:@openfeature/server-sdk --sourcemap --target=es2015 --platform=node --format=esm --outfile=./dist/esm/index.js --analyze",
     "build:cjs": "esbuild src/index.ts --bundle --external:@nestjs/* --external:@openfeature/server-sdk --sourcemap --target=es2015 --platform=node --format=cjs --outfile=./dist/cjs/index.js --analyze",
@@ -60,4 +61,4 @@
     "@types/supertest": "^6.0.0",
     "supertest": "^7.0.0"
   }
-}
+}

packages/nest/src/context-factory.ts

@@ -1,5 +1,6 @@
-import { EvaluationContext } from '@openfeature/core';
-import { ExecutionContext, Inject } from '@nestjs/common';
+import type { EvaluationContext } from '@openfeature/core';
+import type { ExecutionContext} from '@nestjs/common';
+import { Inject } from '@nestjs/common';
 
 /**
  * A factory function for creating an OpenFeature {@link EvaluationContext} from Nest {@link ExecutionContext}.

packages/nest/src/evaluation-context-interceptor.ts

@@ -1,5 +1,7 @@
-import { CallHandler, ExecutionContext, Inject, Injectable, NestInterceptor } from '@nestjs/common';
-import { ContextFactory, ContextFactoryToken } from './context-factory';
+import type { CallHandler, ExecutionContext, NestInterceptor } from '@nestjs/common';
+import { Inject, Injectable } from '@nestjs/common';
+import type { ContextFactory} from './context-factory';
+import { ContextFactoryToken } from './context-factory';
 import { Observable } from 'rxjs';
 import { OpenFeature } from '@openfeature/server-sdk';
 import { OpenFeatureModule } from './open-feature.module';

packages/nest/src/feature.decorator.ts

@@ -1,14 +1,16 @@
 import { createParamDecorator, Inject } from '@nestjs/common';
-import {
+import type {
   EvaluationContext,
   EvaluationDetails,
   FlagValue,
-  JsonValue,
+  JsonValue} from '@openfeature/server-sdk';
+import {
   OpenFeature,
   Client,
 } from '@openfeature/server-sdk';
 import { getOpenFeatureClientToken } from './open-feature.module';
-import { from, Observable } from 'rxjs';
+import type { Observable } from 'rxjs';
+import { from } from 'rxjs';
 
 /**
  * Options for injecting an OpenFeature client into a constructor.

packages/nest/src/open-feature.module.ts

@@ -1,24 +1,27 @@
-import {
+import type {
   DynamicModule,
-  Module,
   FactoryProvider as NestFactoryProvider,
   ValueProvider,
   ClassProvider,
-  Provider as NestProvider,
+  Provider as NestProvider} from '@nestjs/common';
+import {
+  Module,
   ExecutionContext,
 } from '@nestjs/common';
-import {
+import type {
   Client,
   Hook,
-  OpenFeature,
   Provider,
   EvaluationContext,
   ServerProviderEvents,
   EventHandler,
-  Logger,
+  Logger} from '@openfeature/server-sdk';
+import {
+  OpenFeature,
   AsyncLocalStorageTransactionContextPropagator,
 } from '@openfeature/server-sdk';
-import { ContextFactory, ContextFactoryToken } from './context-factory';
+import type { ContextFactory} from './context-factory';
+import { ContextFactoryToken } from './context-factory';
 import { APP_INTERCEPTOR } from '@nestjs/core';
 import { EvaluationContextInterceptor } from './evaluation-context-interceptor';
 import { ShutdownService } from './shutdown.service';

packages/nest/src/shutdown.service.ts

@@ -1,4 +1,5 @@
-import { Injectable, OnApplicationShutdown } from '@nestjs/common';
+import type { OnApplicationShutdown } from '@nestjs/common';
+import { Injectable } from '@nestjs/common';
 import { OpenFeature } from '@openfeature/server-sdk';
 
 @Injectable()

packages/nest/test/fixtures.ts

@@ -1,5 +1,5 @@
 import { InMemoryProvider } from '@openfeature/server-sdk';
-import { ExecutionContext } from '@nestjs/common';
+import type { ExecutionContext } from '@nestjs/common';
 import { OpenFeatureModule } from '../src';
 
 export const defaultProvider = new InMemoryProvider({

packages/nest/test/open-feature-sdk.spec.ts

@@ -1,5 +1,6 @@
-import { Test, TestingModule } from '@nestjs/testing';
-import { INestApplication } from '@nestjs/common';
+import type { TestingModule } from '@nestjs/testing';
+import { Test } from '@nestjs/testing';
+import type { INestApplication } from '@nestjs/common';
 import supertest from 'supertest';
 import { OpenFeatureController, OpenFeatureControllerContextScopedController, OpenFeatureTestService } from './test-app';
 import { exampleContextFactory, getOpenFeatureDefaultTestModule } from './fixtures';

packages/nest/test/open-feature.module.spec.ts

@@ -1,6 +1,8 @@
-import { Test, TestingModule } from '@nestjs/testing';
+import type { TestingModule } from '@nestjs/testing';
+import { Test } from '@nestjs/testing';
 import { getOpenFeatureClientToken, OpenFeatureModule, ServerProviderEvents } from '../src';
-import { Client, OpenFeature } from '@openfeature/server-sdk';
+import type { Client} from '@openfeature/server-sdk';
+import { OpenFeature } from '@openfeature/server-sdk';
 import { getOpenFeatureDefaultTestModule } from './fixtures';
 
 describe('OpenFeatureModule', () => {

packages/nest/test/test-app.ts

@@ -1,7 +1,8 @@
 import { Controller, Get, Injectable, UseInterceptors } from '@nestjs/common';
-import { Observable, map } from 'rxjs';
+import type { Observable} from 'rxjs';
+import { map } from 'rxjs';
 import { BooleanFeatureFlag, ObjectFeatureFlag, NumberFeatureFlag, OpenFeatureClient, StringFeatureFlag } from '../src';
-import { Client, EvaluationDetails, FlagValue } from '@openfeature/server-sdk';
+import type { Client, EvaluationDetails, FlagValue } from '@openfeature/server-sdk';
 import { EvaluationContextInterceptor } from '../src';
 
 @Injectable()

packages/react/package.json

@@ -16,6 +16,7 @@
   "scripts": {
     "test": "jest --verbose",
     "lint": "eslint ./",
+    "lint:fix": "eslint ./ --fix",
     "clean": "shx rm -rf ./dist",
     "build:react-esm": "esbuild src/index.ts --bundle  --external:react --external:@openfeature/web-sdk --sourcemap --target=es2015 --platform=browser --format=esm --outfile=./dist/esm/index.js --analyze",
     "build:react-cjs": "esbuild src/index.ts --bundle  --external:react --external:@openfeature/web-sdk --sourcemap --target=es2015 --platform=browser --format=cjs --outfile=./dist/cjs/index.js --analyze",
@@ -53,4 +54,4 @@
     "@openfeature/core": "*",
     "@openfeature/web-sdk": "*"
   }
-}
+}

packages/react/src/common/options.ts

@@ -1,4 +1,4 @@
-import { FlagEvaluationOptions } from '@openfeature/web-sdk';
+import type { FlagEvaluationOptions } from '@openfeature/web-sdk';
 
 export type ReactFlagEvaluationOptions = ({
   /**

packages/react/src/common/suspense.ts

@@ -1,4 +1,5 @@
-import { Client, ProviderEvents } from '@openfeature/web-sdk';
+import type { Client} from '@openfeature/web-sdk';
+import { ProviderEvents } from '@openfeature/web-sdk';
 
 /**
  * Suspends until the client is ready to evaluate feature flags.

packages/react/src/evaluation/hook-flag-query.ts

@@ -1,9 +1,10 @@
-import {
+import type {
   EvaluationDetails,
-  FlagValue,
+  FlagValue} from '@openfeature/web-sdk';
+import {
   StandardResolutionReasons
 } from '@openfeature/web-sdk';
-import { FlagQuery } from '../query';
+import type { FlagQuery } from '../query';
 
 
 // FlagQuery implementation, do not export

packages/react/src/evaluation/use-feature-flag.ts

@@ -1,21 +1,23 @@
-import {
+import type {
   Client,
   ClientProviderEvents,
   EvaluationDetails,
   EventHandler,
   FlagEvaluationOptions,
   FlagValue,
-  JsonValue,
+  JsonValue} from '@openfeature/web-sdk';
+import {
   ProviderEvents,
   ProviderStatus,
 } from '@openfeature/web-sdk';
 import { useEffect, useRef, useState } from 'react';
-import { DEFAULT_OPTIONS, ReactFlagEvaluationOptions, normalizeOptions } from '../common/options';
+import type { ReactFlagEvaluationOptions} from '../common/options';
+import { DEFAULT_OPTIONS, normalizeOptions } from '../common/options';
 import { suspendUntilReady } from '../common/suspense';
 import { useProviderOptions } from '../provider/context';
 import { useOpenFeatureClient } from '../provider/use-open-feature-client';
 import { useOpenFeatureClientStatus } from '../provider/use-open-feature-client-status';
-import { FlagQuery } from '../query';
+import type { FlagQuery } from '../query';
 import { HookFlagQuery } from './hook-flag-query';
 import { isEqual } from '../common/is-equal';
 

packages/react/src/provider/context.ts

@@ -1,6 +1,7 @@
-import { Client } from '@openfeature/web-sdk';
+import type { Client } from '@openfeature/web-sdk';
 import React from 'react';
-import { NormalizedOptions, ReactFlagEvaluationOptions, normalizeOptions } from '../common/options';
+import type { NormalizedOptions, ReactFlagEvaluationOptions} from '../common/options';
+import { normalizeOptions } from '../common/options';
 
 /**
  * The underlying React context.

packages/react/src/provider/provider.tsx

@@ -1,6 +1,7 @@
-import { Client, OpenFeature } from '@openfeature/web-sdk';
+import type { Client} from '@openfeature/web-sdk';
+import { OpenFeature } from '@openfeature/web-sdk';
 import * as React from 'react';
-import { ReactFlagEvaluationOptions } from '../common/options';
+import type { ReactFlagEvaluationOptions } from '../common/options';
 import { Context } from './context';
 
 type ClientOrDomain =

packages/react/src/provider/test-provider.tsx

@@ -1,12 +1,13 @@
+import type {
+  JsonValue,
+  Provider} from '@openfeature/web-sdk';
 import {
   InMemoryProvider,
-  JsonValue,
   NOOP_PROVIDER,
-  OpenFeature,
-  Provider,
+  OpenFeature
 } from '@openfeature/web-sdk';
 import React from 'react';
-import { NormalizedOptions } from '../common/options';
+import type { NormalizedOptions } from '../common/options';
 import { OpenFeatureProvider } from './provider';
 
 type FlagValueMap = { [flagKey: string]: JsonValue };

packages/react/src/provider/use-open-feature-client-status.ts

@@ -1,6 +1,7 @@
 import { useEffect, useState } from 'react';
 import { useOpenFeatureClient } from './use-open-feature-client';
-import { ProviderEvents, ProviderStatus } from '@openfeature/web-sdk';
+import type { ProviderStatus } from '@openfeature/web-sdk';
+import { ProviderEvents } from '@openfeature/web-sdk';
 
 /**
  * Get the {@link ProviderStatus} for the OpenFeatureClient.

packages/react/src/provider/use-open-feature-client.ts

@@ -1,6 +1,6 @@
 import React from 'react';
 import { Context } from './context';
-import { Client } from '@openfeature/web-sdk';
+import type { Client } from '@openfeature/web-sdk';
 
 /**
  * Get the {@link Client} instance for this OpenFeatureProvider context.

packages/react/src/provider/use-when-provider-ready.ts

@@ -1,5 +1,6 @@
 import { ProviderStatus } from '@openfeature/web-sdk';
-import { DEFAULT_OPTIONS, ReactFlagEvaluationOptions, normalizeOptions } from '../common/options';
+import type { ReactFlagEvaluationOptions} from '../common/options';
+import { DEFAULT_OPTIONS, normalizeOptions } from '../common/options';
 import { useProviderOptions } from './context';
 import { useOpenFeatureClient } from './use-open-feature-client';
 import { useOpenFeatureClientStatus } from './use-open-feature-client-status';

packages/react/src/query/query.ts

@@ -1,4 +1,4 @@
-import { ErrorCode, EvaluationDetails, FlagMetadata, FlagValue, StandardResolutionReasons } from '@openfeature/core';
+import type { ErrorCode, EvaluationDetails, FlagMetadata, FlagValue, StandardResolutionReasons } from '@openfeature/core';
 
 export interface FlagQuery<T extends FlagValue = FlagValue> {
   /**

packages/react/test/evaluation.spec.tsx

@@ -1,11 +1,12 @@
 import '@testing-library/jest-dom'; // see: https://testing-library.com/docs/react-testing-library/setup
 import { act, render, renderHook, screen, waitFor } from '@testing-library/react';
 import * as React from 'react';
-import {
-  ErrorCode,
+import type {
   EvaluationContext,
   EvaluationDetails,
-  Hook,
+  Hook} from '../src/';
+import {
+  ErrorCode,
   InMemoryProvider,
   OpenFeature,
   OpenFeatureProvider,

packages/react/test/provider.spec.tsx

@@ -1,4 +1,5 @@
-import { EvaluationContext, OpenFeature } from '@openfeature/web-sdk';
+import type { EvaluationContext} from '@openfeature/web-sdk';
+import { OpenFeature } from '@openfeature/web-sdk';
 import '@testing-library/jest-dom'; // see: https://testing-library.com/docs/react-testing-library/setup
 import { render, renderHook, screen, waitFor } from '@testing-library/react';
 import * as React from 'react';

packages/react/test/test-provider.spec.tsx

@@ -1,4 +1,4 @@
-import { Provider, ResolutionDetails } from '@openfeature/web-sdk';
+import type { Provider, ResolutionDetails } from '@openfeature/web-sdk';
 import '@testing-library/jest-dom'; // see: https://testing-library.com/docs/react-testing-library/setup
 import { render, screen } from '@testing-library/react';
 import * as React from 'react';

packages/server/e2e/step-definitions/evaluation.spec.ts

@@ -1,9 +1,10 @@
-import {
+import type {
   EvaluationContext,
   EvaluationDetails,
   JsonObject,
   JsonValue,
-  ResolutionDetails,
+  ResolutionDetails} from '@openfeature/core';
+import {
   StandardResolutionReasons,
 } from '@openfeature/core';
 import { defineFeature, loadFeature } from 'jest-cucumber';

packages/server/package.json

@@ -16,6 +16,7 @@
   "scripts": {
     "test": "jest --verbose",
     "lint": "eslint ./",
+    "lint:fix": "eslint ./ --fix",
     "clean": "shx rm -rf ./dist",
     "build:esm": "esbuild src/index.ts --bundle --external:@openfeature/core --sourcemap --target=es2015 --platform=node --format=esm --outfile=./dist/esm/index.js --analyze",
     "build:cjs": "esbuild src/index.ts --bundle --external:@openfeature/core --sourcemap --target=es2015 --platform=node --format=cjs --outfile=./dist/cjs/index.js --analyze",

packages/server/src/client/client.ts

@@ -1,13 +1,13 @@
-import {
+import type {
   ClientMetadata,
   EvaluationLifeCycle,
   Eventing,
   ManageContext,
   ManageLogger,
 } from '@openfeature/core';
-import { Features } from '../evaluation';
-import { ProviderStatus } from '../provider';
-import { ProviderEvents } from '../events';
+import type { Features } from '../evaluation';
+import type { ProviderStatus } from '../provider';
+import type { ProviderEvents } from '../events';
 
 export interface Client
   extends EvaluationLifeCycle<Client>,