Some of the rewrites made things mildly slower, so I looked for an optimization to speed things back up. I've added a micro-optimization for val/var, making things generally more performant.

Author: TotalTechGeek

asyncLogic.js

@@ -3,7 +3,7 @@
 
 import defaultMethods from './defaultMethods.js'
 import LogicEngine from './logic.js'
-import { isSync } from './constants.js'
+import { isSync, OriginalImpl } from './constants.js'
 import declareSync from './utilities/declareSync.js'
 import { buildAsync } from './compiler.js'
 import omitUndefined from './utilities/omitUndefined.js'
@@ -75,6 +75,13 @@ class AsyncLogicEngine {
     if (this.isData(logic, func)) return logic
     if (!this.methods[func]) throw new Error(`Method '${func}' was not found in the Logic Engine.`)
 
+    // A small but useful micro-optimization for some of the most common functions.
+    // Later on, I could define something to shut this off if var / val are redefined.
+    if ((func === 'var' || func === 'val') && this.methods[func][OriginalImpl]) {
+      const input = (!data || typeof data !== 'object') ? data : this.fallback.run(data, context, { above })
+      return this.methods[func].method(input, context, above, this)
+    }
+
     if (typeof this.methods[func] === 'function') {
       const input = (!data || typeof data !== 'object') ? [data] : await this.run(data, context, { above })
       const result = await this.methods[func](coerceArray(input), context, above, this)
@@ -210,5 +217,5 @@ class AsyncLogicEngine {
     return logic
   }
 }
-Object.assign(AsyncLogicEngine.prototype.truthy, { IDENTITY: true })
+Object.assign(AsyncLogicEngine.prototype.truthy, { [OriginalImpl]: true })
 export default AsyncLogicEngine

constants.js

@@ -3,7 +3,7 @@
 
 export const Sync = Symbol.for('json_logic_sync')
 export const Compiled = Symbol.for('json_logic_compiled')
-export const EfficientTop = Symbol.for('json_logic_efficientTop')
+export const OriginalImpl = Symbol.for('json_logic_original')
 export const Unfound = Symbol.for('json_logic_unfound')
 
 /**
@@ -23,6 +23,6 @@ export function isSync (item) {
 
 export default {
   Sync,
-  EfficientTop,
+  OriginalImpl,
   isSync
 }

defaultMethods.js

@@ -2,7 +2,7 @@
 'use strict'
 
 import asyncIterators from './async_iterators.js'
-import { Sync, isSync, Unfound } from './constants.js'
+import { Sync, isSync, Unfound, OriginalImpl } from './constants.js'
 import declareSync from './utilities/declareSync.js'
 import { build, buildString } from './compiler.js'
 import chainingSupported from './utilities/chainingSupported.js'
@@ -277,7 +277,7 @@ const defaultMethods = {
     },
     deterministic: (data, buildState) => isDeterministic(data, buildState.engine, buildState),
     compile: (data, buildState) => {
-      if (!buildState.engine.truthy.IDENTITY) return false
+      if (!buildState.engine.truthy[OriginalImpl]) return false
       if (Array.isArray(data) && data.length) return `(${data.map((i) => buildString(i, buildState)).join(' || ')})`
       return `(${buildString(data, buildState)}).reduce((a,b) => a||b, false)`
     },
@@ -311,7 +311,7 @@ const defaultMethods = {
     traverse: false,
     deterministic: (data, buildState) => isDeterministic(data, buildState.engine, buildState),
     compile: (data, buildState) => {
-      if (!buildState.engine.truthy.IDENTITY) return false
+      if (!buildState.engine.truthy[OriginalImpl]) return false
       if (Array.isArray(data) && data.length) return `(${data.map((i) => buildString(i, buildState)).join(' && ')})`
       return `(${buildString(data, buildState)}).reduce((a,b) => a&&b, true)`
     }
@@ -337,6 +337,7 @@ const defaultMethods = {
     deterministic: false
   },
   val: {
+    [OriginalImpl]: true,
     method: (args, context, above, engine, /** @type {null | Symbol} */ unFound = null) => {
       if (Array.isArray(args) && args.length === 1) args = args[0]
       // A unary optimization

legacy.js

@@ -1,7 +1,8 @@
+'use strict'
 import { buildString } from './compiler.js'
 import { splitPathMemoized } from './utilities/splitPath.js'
 import chainingSupported from './utilities/chainingSupported.js'
-import { Sync } from './constants.js'
+import { Sync, OriginalImpl } from './constants.js'
 
 const legacyMethods = {
   get: {
@@ -49,6 +50,7 @@ const legacyMethods = {
     }
   },
   var: {
+    [OriginalImpl]: true,
     [Sync]: true,
     method: (key, context, above, engine) => {
       let b
@@ -132,6 +134,7 @@ const legacyMethods = {
   },
   missing: {
     [Sync]: true,
+    optimizeUnary: false,
     method: (checked, context, above, engine) => {
       return (Array.isArray(checked) ? checked : [checked]).filter((key) => {
         return legacyMethods.var.method(key, context, above, engine) === null
@@ -141,6 +144,7 @@ const legacyMethods = {
   },
   missing_some: {
     [Sync]: true,
+    optimizeUnary: false,
     method: ([needCount, options], context, above, engine) => {
       const missing = legacyMethods.missing.method(options, context, above, engine)
       if (options.length - missing.length >= needCount) {

logic.js

@@ -9,6 +9,7 @@ import omitUndefined from './utilities/omitUndefined.js'
 import { optimize } from './optimizer.js'
 import { applyPatches } from './compatibility.js'
 import { coerceArray } from './utilities/coerceArray.js'
+import { OriginalImpl } from './constants.js'
 
 /**
  * An engine capable of running synchronous JSON Logic.
@@ -71,6 +72,13 @@ class LogicEngine {
 
     if (!this.methods[func]) throw new Error(`Method '${func}' was not found in the Logic Engine.`)
 
+    // A small but useful micro-optimization for some of the most common functions.
+    // Later on, I could define something to shut this off if var / val are redefined.
+    if ((func === 'var' || func === 'val') && this.methods[func][OriginalImpl]) {
+      const input = (!data || typeof data !== 'object') ? data : this.run(data, context, { above })
+      return this.methods[func].method(input, context, above, this)
+    }
+
     if (typeof this.methods[func] === 'function') {
       const input = (!data || typeof data !== 'object') ? [data] : coerceArray(this.run(data, context, { above }))
       return this.methods[func](input, context, above, this)
@@ -174,5 +182,5 @@ class LogicEngine {
     return logic
   }
 }
-Object.assign(LogicEngine.prototype.truthy, { IDENTITY: true })
+Object.assign(LogicEngine.prototype.truthy, { [OriginalImpl]: true })
 export default LogicEngine