copilot review updates

Author: raju-opti

README.md

@@ -209,7 +209,7 @@ If you're updating your SDK version, please check the appropriate migration guid
 
 ### Platform Isolation
 
-The SDK supports multiple JavaScript platforms (Browser, Node.js, React Native) with a unified codebase. To prevent runtime errors from platform-specific code being bundled incorrectly, we enforce **platform isolation** constraints:
+The SDK supports multiple JavaScript platforms (Browser, Node.js, React Native and universal) with a unified codebase. To prevent runtime errors from platform-specific code being bundled incorrectly, we enforce **platform isolation** constraints:
 
 - Every source file must declare which platforms it supports using `export const __platforms: Platform[] = [...]`
 - Files can only import from other files that support all their declared platforms

lib/index.browser.umdtests.js

@@ -25,8 +25,6 @@ import packageJSON from '../package.json';
 import eventDispatcher from './plugins/event_dispatcher/index.browser';
 import { INVALID_CONFIG_OR_SOMETHING } from './exception_messages';
 
-export const __platforms = ['browser'] as const;
-
 describe('javascript-sdk', function() {
   describe('APIs', function() {
     describe('createInstance', function() {

lib/project_config/optimizely_config.ts

@@ -34,8 +34,6 @@ import {
   VariationVariable,
 } from '../shared_types';
 
-
-import { DATAFILE_VERSIONS } from '../utils/enums';
 import { Platform } from '../platform_support';
 
 interface FeatureVariablesMap {

scripts/validate-platform-isolation.js

@@ -25,7 +25,7 @@
  * Platform Detection:
  * - ALL source files (except tests) MUST export __platforms array
  * - Universal files use: export const __platforms = ['__universal__'];
- * - Platform-specific files use platforn names, e.g: export const __platforms = ['browser', 'node'];
+ * - Platform-specific files use platform names, e.g: export const __platforms = ['browser', 'node'];
  * - Valid platform values are dynamically read from Platform type in platform_support.ts
  * 
  * Rules:
@@ -59,7 +59,7 @@ const compilerOptions = ts.convertCompilerOptionsFromJson(
 const config = loadConfig();
 const configPath = path.join(WORKSPACE_ROOT, '.platform-isolation.config.js');
 
-// Track files with errrors in __platforms export
+// Track files with errors in __platforms export
 const fileErrors = new Map();
 
 /**

vitest.config.mts

@@ -31,15 +31,5 @@ export default defineConfig({
       enabled: true,
       tsconfig: 'tsconfig.spec.json',
     },
-    coverage: {
-      provider: 'istanbul',
-      include: ['lib/**/*.ts'],
-      exclude: [
-        '**/tests/**',
-        '**/*.spec.ts',
-        '**/*.gen.ts',
-        '**/*d.ts',
-      ],
-    },
   },
 });