Fix bracket notation and pick() method when used with a function to…

… apply `bool()` to the result of the function

CHANGELOG.md

@@ -1,6 +1,7 @@
 ## next
 
 - Fixed a query compilation error when `$` was used as an object entry, e.g. `{ $, prop: 1 }`
+- Fixed bracket notation and `pick()` method when used with a function to apply `bool()` to the result of the function
 
 ## 1.0.0-beta.9 (October 29, 2023)
 

src/lang/compile-buildin.js

@@ -135,15 +135,15 @@ function pick(current, ref = () => true) {
     if (typeof ref === 'function') {
         if (Array.isArray(current) || typeof current === 'string') {
             for (let i = 0; i < current.length; i++) {
-                if (ref(current[i], i)) {
+                if (isTruthy(ref(current[i], i))) {
                     return current[i];
                 }
             }
         }
 
         for (const key in current) {
             if (hasOwn(current, key)) {
-                if (ref(current[key], key)) {
+                if (isTruthy(ref(current[key], key))) {
                     return current[key];
                 }
             }

test/methods/pick.js

@@ -222,6 +222,27 @@ describe('pick()', () => {
                 undefined
             );
         });
+
+        it('should treat empty arrays as false', () => {
+            assert.deepStrictEqual(
+                query('pick(=> foo)')([{ foo: [] }, { foo: [1] }]),
+                { foo: [1] }
+            );
+            assert.deepStrictEqual(
+                query('pick(=> foo)')({ a: { foo: [] }, b: { foo: [1] } }),
+                { foo: [1] }
+            );
+        });
+        it('should treat empty objects as false', () => {
+            assert.deepStrictEqual(
+                query('pick(=> foo)')([{ foo: {} }, { foo: { ok: 1 } }]),
+                { foo: { ok: 1 } }
+            );
+            assert.deepStrictEqual(
+                query('pick(=> foo)')({ a: { foo: {} }, b: { foo: { ok: 1 } } }),
+                { foo: { ok: 1 } }
+            );
+        });
     });
 
     describe('special cases', () => {