Merge branch 'master' of https://github.com/reduxjs/reselect into add-size-limit

Author: aryaemami59

.github/workflows/build-and-test-types.yml

@@ -1,10 +1,6 @@
 name: CI
 
-on:
-  push:
-    branches: [master]
-  pull_request:
-    branches: [master]
+on: [push, pull_request]
 
 jobs:
   build:
@@ -115,7 +111,7 @@ jobs:
 
       # Note: We currently expect "FalseCJS" failures for Node16 + `moduleResolution: "node16",
       - name: Run are-the-types-wrong
-        run: npx @arethetypeswrong/cli ./package.tgz --format table --ignore-rules false-cjs
+        run: npx @arethetypeswrong/cli@latest ./package.tgz --format table --ignore-rules false-cjs
 
   test-published-artifact:
     name: Test Published Artifact ${{ matrix.example }}
@@ -153,26 +149,14 @@ jobs:
       - name: Check folder contents
         run: ls -l .
 
-        # Some weird install diffs with cloning this repo and installing.
-        # Just kill the lockfiles for this repo and RTK and reinstall
-
-      - name: Remove top lockfile
-        run: rm yarn.lock && rm package.json
-
-      - name: Remove RTK lockfile
-        working-directory: ./redux-toolkit
-        run: rm yarn.lock && rm package.json
-
-      - name: Install deps
+      - name: Install example deps
         working-directory: ./redux-toolkit/examples/publish-ci/${{ matrix.example }}
-        env:
-          YARN_ENABLE_IMMUTABLE_INSTALLS: false
-        run: rm yarn.lock && yarn install
+        run: yarn install
 
       - name: Install Playwright browser if necessary
         working-directory: ./redux-toolkit/examples/publish-ci/${{ matrix.example }}
         continue-on-error: true
-        run: yarn playwright install
+        run: yarn playwright install || true
 
       - uses: actions/download-artifact@v4
         with:

package.json

@@ -2,15 +2,15 @@
   "name": "reselect",
   "version": "5.1.1",
   "description": "Selectors for Redux.",
-  "main": "./dist/cjs/reselect.cjs",
+  "main": "./dist/cjs/index.js",
   "module": "./dist/reselect.legacy-esm.js",
   "types": "./dist/reselect.d.ts",
   "exports": {
     "./package.json": "./package.json",
     ".": {
       "types": "./dist/reselect.d.ts",
       "import": "./dist/reselect.mjs",
-      "default": "./dist/cjs/reselect.cjs"
+      "default": "./dist/cjs/index.js"
     }
   },
   "files": [
@@ -84,9 +84,12 @@
     "rimraf": "^3.0.2",
     "shelljs": "^0.8.5",
     "size-limit": "^11.1.4",
-    "tsup": "^6.7.0",
+    "tsup": "^8.2.4",
     "typescript": "^5.4.2",
     "vitest": "^1.6.0"
   },
+  "resolutions": {
+    "esbuild": "0.23.0"
+  },
   "packageManager": "yarn@4.1.0"
 }

src/autotrackMemoize/proxy.ts

@@ -10,11 +10,11 @@ import {
   dirtyTag
 } from './tracking'
 
-export const REDUX_PROXY_LABEL = Symbol()
+export const REDUX_PROXY_LABEL = /* @__PURE__ */ Symbol()
 
 let nextId = 0
 
-const proto = Object.getPrototypeOf({})
+const proto = /* @__PURE__ */ Object.getPrototypeOf({})
 
 class ObjectTreeNode<T extends Record<string, unknown>> implements Node<T> {
   proxy: T = new Proxy(this, objectProxyHandler) as unknown as T

src/createSelectorCreator.ts

@@ -372,8 +372,7 @@ export function createSelectorCreator<
       memoize,
       memoizeOptions = [],
       argsMemoize = weakMapMemoize,
-      argsMemoizeOptions = [],
-      devModeChecks = {}
+      argsMemoizeOptions = []
     } = combinedOptions
 
     // Simplifying assumption: it's unlikely that the first options arg of the provided memoizer
@@ -412,6 +411,7 @@ export function createSelectorCreator<
       lastResult = memoizedResultFunc.apply(null, inputSelectorResults)
 
       if (process.env.NODE_ENV !== 'production') {
+        const { devModeChecks = {} } = combinedOptions
         const { identityFunctionCheck, inputStabilityCheck } =
           getDevModeChecksExecutionInfo(firstRun, devModeChecks)
         if (identityFunctionCheck.shouldRun) {

src/createStructuredSelector.ts

@@ -415,7 +415,7 @@ export interface StructuredSelectorCreator<StateType = any> {
  * @public
  */
 export const createStructuredSelector: StructuredSelectorCreator =
-  Object.assign(
+  /* @__PURE__ */ Object.assign(
     <
       InputSelectorsObject extends SelectorsObject,
       MemoizeFunction extends UnknownMemoizer = typeof weakMapMemoize,

src/weakMapMemoize.ts

@@ -15,10 +15,18 @@ class StrongRef<T> {
   }
 }
 
-const Ref =
-  typeof WeakRef !== 'undefined'
-    ? WeakRef
-    : (StrongRef as unknown as typeof WeakRef)
+/**
+ * @returns The {@linkcode StrongRef} if {@linkcode WeakRef} is not available.
+ *
+ * @since 5.1.2
+ * @internal
+ */
+const getWeakRef = () =>
+  typeof WeakRef === 'undefined'
+    ? (StrongRef as unknown as typeof WeakRef)
+    : WeakRef
+
+const Ref = /* @__PURE__ */ getWeakRef()
 
 const UNTERMINATED = 0
 const TERMINATED = 1
@@ -96,6 +104,20 @@ export interface WeakMapMemoizeOptions<Result = any> {
   resultEqualityCheck?: EqualityFn<Result>
 }
 
+/**
+ * Derefences the argument if it is a Ref. Else if it is a value already, return it.
+ *
+ * @param r - the object to maybe deref
+ * @returns The derefenced value if the argument is a Ref, else the argument value itself.
+ */
+function maybeDeref(r: any) {
+  if (r instanceof Ref) {
+    return r.deref()
+  }
+
+  return r
+}
+
 /**
  * Creates a tree of `WeakMap`-based cache nodes based on the identity of the
  * arguments it's been called with (in this case, the extracted values from your input selectors).
@@ -229,7 +251,8 @@ export function weakMapMemoize<Func extends AnyFunction>(
       resultsCount++
 
       if (resultEqualityCheck) {
-        const lastResultValue = lastResult?.deref?.() ?? lastResult
+        // Deref lastResult if it is a Ref
+        const lastResultValue = maybeDeref(lastResult)
 
         if (
           lastResultValue != null &&
@@ -244,7 +267,7 @@ export function weakMapMemoize<Func extends AnyFunction>(
           (typeof result === 'object' && result !== null) ||
           typeof result === 'function'
 
-        lastResult = needsWeakRef ? new Ref(result) : result
+        lastResult = needsWeakRef ? /* @__PURE__ */ new Ref(result) : result
       }
     }
 

tsup.config.ts

@@ -1,66 +1,103 @@
-import { defineConfig, Options } from 'tsup'
-import fs from 'fs'
-import sh from 'shelljs'
-import type { ExecOptions } from 'shelljs'
+import fs from 'node:fs/promises'
+import path from 'node:path'
+import type { Options } from 'tsup'
+import { defineConfig } from 'tsup'
 
-function execAsync(cmd: string, opts: ExecOptions = {}) {
-  return new Promise(function (resolve, reject) {
-    // Execute the command, reject if we exit non-zero (i.e. error)
-    sh.exec(cmd, opts, function (code, stdout, stderr) {
-      if (code !== 0) return reject(new Error(stderr))
-      return resolve(stdout)
-    })
-  })
+async function writeCommonJSEntry() {
+  await fs.writeFile(
+    path.join('dist/cjs/', 'index.js'),
+    `'use strict'
+if (process.env.NODE_ENV === 'production') {
+  module.exports = require('./reselect.production.min.cjs')
+} else {
+  module.exports = require('./reselect.development.cjs')
+}`
+  )
 }
 
-export default defineConfig(options => {
-  const commonOptions: Partial<Options> = {
+export default defineConfig((options): Options[] => {
+  const commonOptions: Options = {
     entry: {
       reselect: 'src/index.ts'
     },
     sourcemap: true,
+    target: ['esnext'],
+    clean: true,
     ...options
   }
 
   return [
-    // Modern ESM
     {
       ...commonOptions,
+      name: 'Modern ESM',
+      target: ['esnext'],
       format: ['esm'],
-      outExtension: () => ({ js: '.mjs' }),
-      dts: true,
-      clean: true
+      outExtension: () => ({ js: '.mjs' })
     },
 
     // Support Webpack 4 by pointing `"module"` to a file with a `.js` extension
     // and optional chaining compiled away
     {
       ...commonOptions,
+      name: 'Legacy ESM, Webpack 4',
       entry: {
         'reselect.legacy-esm': 'src/index.ts'
       },
       format: ['esm'],
       outExtension: () => ({ js: '.js' }),
-      target: 'es2017'
+      target: ['es2017']
     },
-    // Browser-ready ESM, production + minified
+
+    // Meant to be served up via CDNs like `unpkg`.
     {
       ...commonOptions,
+      name: 'Browser-ready ESM',
       entry: {
         'reselect.browser': 'src/index.ts'
       },
-      define: {
-        'process.env.NODE_ENV': JSON.stringify('production')
+      platform: 'browser',
+      env: {
+        NODE_ENV: 'production'
       },
       format: ['esm'],
       outExtension: () => ({ js: '.mjs' }),
       minify: true
     },
     {
       ...commonOptions,
-      format: 'cjs',
+      name: 'CJS Development',
+      entry: {
+        'reselect.development': 'src/index.ts'
+      },
+      env: {
+        NODE_ENV: 'development'
+      },
+      format: ['cjs'],
       outDir: './dist/cjs/',
       outExtension: () => ({ js: '.cjs' })
+    },
+    {
+      ...commonOptions,
+      name: 'CJS production',
+      entry: {
+        'reselect.production.min': 'src/index.ts'
+      },
+      env: {
+        NODE_ENV: 'production'
+      },
+      format: ['cjs'],
+      outDir: './dist/cjs/',
+      outExtension: () => ({ js: '.cjs' }),
+      minify: true,
+      onSuccess: async () => {
+        await writeCommonJSEntry()
+      }
+    },
+    {
+      ...commonOptions,
+      name: 'CJS Type Definitions',
+      format: ['cjs'],
+      dts: { only: true }
     }
   ]
 })