- Removed the indexBrowser implementation since the typechecking handles it much more simply

Author: heath-freenome

packages/playground/vite.config.ts

@@ -24,7 +24,7 @@ export default defineConfig({
       '@rjsf/semantic-ui': path.resolve(__dirname, '../semantic-ui/src'),
       '@rjsf/shadcn': path.resolve(__dirname, '../shadcn/src'),
       // We want to pick up the browser version of the utils
-      '@rjsf/utils': path.resolve(__dirname, '../utils/src/indexBrowser'),
+      '@rjsf/utils': path.resolve(__dirname, '../utils/src'),
       '@rjsf/validator-ajv8': path.resolve(__dirname, '../validator-ajv8/src'),
       '@rjsf/daisyui': path.resolve(__dirname, '../daisyui/src'),
     },

packages/utils/package.json

@@ -4,7 +4,6 @@
   "main": "dist/index.js",
   "module": "lib/index.js",
   "typings": "lib/index.d.ts",
-  "browser": "dist/indexBrowser.js",
   "type": "module",
   "description": "Utility functions for @rjsf/core",
   "exports": {
@@ -46,13 +45,10 @@
   "scripts": {
     "compileReplacer": "tsc -p tsconfig.replacer.json && move-file lodashReplacer.js lodashReplacer.cjs",
     "build:ts": "npm run compileReplacer && rimraf ./lib && tsc -b tsconfig.build.json && tsc-alias -p tsconfig.build.json",
-    "build:esm-browser": "esbuild ./src/indexBrowser.ts --bundle --outfile=dist/utils-browser.esm.js --sourcemap --packages=external --format=esm",
-    "build:cjs-browser": "esbuild ./src/indexBrowser.ts --bundle --outfile=dist/indexBrowser.js --sourcemap --packages=external --format=cjs",
-    "build:umd-browser": "rollup dist/utils-browser.esm.js --format=umd --file=dist/utils-browser.umd.js --name=@rjsf/utils",
     "build:cjs": "esbuild ./src/index.ts --bundle --outfile=dist/index.js --sourcemap --packages=external --format=cjs",
     "build:esm": "esbuild ./src/index.ts --bundle --outfile=dist/utils.esm.js --sourcemap --packages=external --format=esm",
     "build:umd": "rollup dist/utils.esm.js --format=umd --file=dist/utils.umd.js --name=@rjsf/utils",
-    "build": "npm run build:ts && npm run build:cjs && npm run build:cjs-browser && npm run build:esm && npm run build:esm-browser && npm run build:umd && npm run build:umd-browser",
+    "build": "npm run build:ts && npm run build:cjs && npm run build:esm && npm run build:umd",
     "cs-check": "prettier -l \"{src,test}/**/*.ts?(x)\"",
     "cs-format": "prettier \"{src,test}/**/*.ts?(x)\" --write",
     "lint": "eslint src test",

packages/utils/src/getTestIds.ts

@@ -3,12 +3,6 @@ import get from 'lodash/get';
 
 import { TestIdShape } from './types';
 
-/** Returns an empty object for use on the browser since we'll never use test ids there.
- */
-export function getTestIdsBrowser(): TestIdShape {
-  return {};
-}
-
 /** Returns an object of test IDs that can only be used in test mode. If the function is called in a test environment
  * (`NODE_ENV === 'test'`, this is set by jest) then a Proxy object will be returned. If a key within the returned
  * object is accessed, if the value already exists the object will return that value, otherwise it will create that key
@@ -27,8 +21,8 @@ export function getTestIdsBrowser(): TestIdShape {
 export default function getTestIds(): TestIdShape {
   // For some reason, even though process.env contains the value of `test` for NODE_ENV, accessing it directly returns
   // 'development'. Using `get()` does, in fact, return test so sticking with it
-  if (typeof process !== 'undefined' && get(process, 'env.NODE_ENV') !== 'test') {
-    return getTestIdsBrowser();
+  if (typeof process === 'undefined' || get(process, 'env.NODE_ENV') !== 'test') {
+    return {};
   }
 
   const ids = new Map();