adjust folder structure

Author: zhiyuanliang-ms

.github/workflows/ci.yml

@@ -20,15 +20,30 @@ jobs:
       matrix:
         node-version: [18.x, 20.x]
 
+    defaults:
+      run:
+        working-directory: sdk/feature-management
+
     steps:
     - uses: actions/checkout@v3
     - name: Use Node.js ${{ matrix.node-version }}
       uses: actions/setup-node@v3
       with:
         node-version: ${{ matrix.node-version }}
         cache: 'npm'
-    - run: npm ci
-    - run: npm run lint
-    - run: npm run build
-    - run: npm run test
-    - run: npm run test-browser
+        cache-dependency-path: sdk/feature-management/package-lock.json
+    
+    - name: Install dependencies
+      run: npm ci
+
+    - name: Run lint check
+      run: npm run lint
+
+    - name: Run build
+      run: npm run build
+
+    - name: Run tests
+      run: npm run test
+
+    - name: Run browser tests
+      run: npm run test-browser

scripts/build-and-pack.sh

@@ -9,19 +9,29 @@ SCRIPT_DIR=$(dirname $(readlink -f $0))
 # Get the directory of the project.
 PROJECT_BASE_DIR=$(dirname $SCRIPT_DIR)
 
-# Change to the project directory.
-cd $PROJECT_BASE_DIR
-
-# Install dependencies, build, and test.
-echo "npm clean install"
-npm ci
-
-echo "npm run build"
-npm run build
-
-echo "npm run test"
-npm run test
-
-# Create a tarball.
-echo "npm pack"
-npm pack
+# Define the SDK directory.
+SDK_DIR="$PROJECT_BASE_DIR/sdk"
+
+# Loop through each package in the sdk folder.
+for PACKAGE_DIR in "$SDK_DIR"/*; do
+  if [ -d "$PACKAGE_DIR" ]; then
+    echo "Building package in $PACKAGE_DIR"
+    
+    # Change to the package directory.
+    cd "$PACKAGE_DIR"
+    
+    # Install dependencies, build, and test.
+    echo "npm clean install in $PACKAGE_DIR"
+    npm ci
+
+    echo "npm run build in $PACKAGE_DIR"
+    npm run build
+
+    echo "npm run test in $PACKAGE_DIR"
+    npm run test
+
+    # Create a tarball.
+    echo "npm pack in $PACKAGE_DIR"
+    npm pack
+  fi
+done

.eslintrc renamed to sdk/feature-management/.eslintrc

@@ -1,57 +1,57 @@
-// rollup.config.js
-import typescript from "@rollup/plugin-typescript";
-import dts from "rollup-plugin-dts";
-
-export default [
-  {
-    external: ["crypto"],
-    input: "src/index.ts",
-    output: [
-      {
-        dir: "dist/commonjs/",
-        format: "cjs",
-        sourcemap: true,
-        preserveModules: true,
-      },
-      {
-        dir: "dist/esm/",
-        format: "esm",
-        sourcemap: true,
-        preserveModules: true,
-      },
-      {
-        file: "dist/umd/index.js",
-        format: "umd",
-        name: 'FeatureManagement',
-        sourcemap: true
-      }
-    ],
-    plugins: [
-      typescript({
-        compilerOptions: {
-          "lib": [
-            "DOM",
-            "WebWorker",
-            "ESNext"
-          ],
-          "skipDefaultLibCheck": true,
-          "module": "ESNext",
-          "moduleResolution": "Node",
-          "target": "ES2022",
-          "strictNullChecks": true,
-          "strictFunctionTypes": true,
-          "sourceMap": true,
-          "inlineSources": true
-        },
-        "exclude": [
-            "test/**/*"
-        ]
-      })
-    ],
-  },
-  {
-    input: "src/index.ts",
-    output: [{ file: "types/index.d.ts", format: "esm" }],
-    plugins: [dts()],
-  },
-];
+// rollup.config.js
+import typescript from "@rollup/plugin-typescript";
+import dts from "rollup-plugin-dts";
+
+export default [
+  {
+    external: ["crypto"],
+    input: "src/index.ts",
+    output: [
+      {
+        dir: "dist/commonjs/",
+        format: "cjs",
+        sourcemap: true,
+        preserveModules: true,
+      },
+      {
+        dir: "dist/esm/",
+        format: "esm",
+        sourcemap: true,
+        preserveModules: true,
+      },
+      {
+        file: "dist/umd/index.js",
+        format: "umd",
+        name: 'FeatureManagement',
+        sourcemap: true
+      }
+    ],
+    plugins: [
+      typescript({
+        compilerOptions: {
+          "lib": [
+            "DOM",
+            "WebWorker",
+            "ESNext"
+          ],
+          "skipDefaultLibCheck": true,
+          "module": "ESNext",
+          "moduleResolution": "Node",
+          "target": "ES2022",
+          "strictNullChecks": true,
+          "strictFunctionTypes": true,
+          "sourceMap": true,
+          "inlineSources": true
+        },
+        "exclude": [
+            "test/**/*"
+        ]
+      })
+    ],
+  },
+  {
+    input: "src/index.ts",
+    output: [{ file: "types/index.d.ts", format: "esm" }],
+    plugins: [dts()],
+  },
+];

package-lock.json renamed to sdk/feature-management/package-lock.json

@@ -1,26 +1,26 @@
-// Copyright (c) Microsoft Corporation.
-// Licensed under the MIT license.
-
-import { ITargetingContext } from "./common/ITargetingContext";
-import { Variant } from "./variant/Variant";
-
-export interface IFeatureManager {
-    /**
-     * Get the list of feature names.
-     */
-    listFeatureNames(): Promise<string[]>;
-
-    /**
-     * Check if a feature is enabled.
-     * @param featureName name of the feature.
-     * @param context an object providing information that can be used to evaluate whether a feature should be on or off.
-     */
-    isEnabled(featureName: string, context?: unknown): Promise<boolean>;
-
-    /**
-     * Get the allocated variant of a feature given the targeting context.
-     * @param featureName name of the feature.
-     * @param context a targeting context object used to evaluate which variant the user will be assigned.
-     */
-    getVariant(featureName: string, context: ITargetingContext): Promise<Variant | undefined>;
-}
+// Copyright (c) Microsoft Corporation.
+// Licensed under the MIT license.
+
+import { ITargetingContext } from "./common/ITargetingContext";
+import { Variant } from "./variant/Variant";
+
+export interface IFeatureManager {
+    /**
+     * Get the list of feature names.
+     */
+    listFeatureNames(): Promise<string[]>;
+
+    /**
+     * Check if a feature is enabled.
+     * @param featureName name of the feature.
+     * @param context an object providing information that can be used to evaluate whether a feature should be on or off.
+     */
+    isEnabled(featureName: string, context?: unknown): Promise<boolean>;
+
+    /**
+     * Get the allocated variant of a feature given the targeting context.
+     * @param featureName name of the feature.
+     * @param context a targeting context object used to evaluate which variant the user will be assigned.
+     */
+    getVariant(featureName: string, context: ITargetingContext): Promise<Variant | undefined>;
+}

package.json renamed to sdk/feature-management/package.json

@@ -1,8 +1,8 @@
-// Copyright (c) Microsoft Corporation.
-// Licensed under the MIT license.
-
-export interface ITargetingContext {
-    userId?: string;
-    groups?: string[];
-}
-
+// Copyright (c) Microsoft Corporation.
+// Licensed under the MIT license.
+
+export interface ITargetingContext {
+    userId?: string;
+    groups?: string[];
+}
+