mnfst 0.5.102 → 0.5.104

package/lib/manifest.data.js

@@ -259,12 +259,16 @@ function deepSeal(obj) {
             }
         }
     } else {
-        for (const key in obj) {
-            if (obj.hasOwnProperty(key)) {
-                const value = obj[key];
-                if (value !== null && typeof value === 'object') {
-                    deepSeal(value);
-                }
+        // Iterate own enumerable keys via Object.keys instead of for…in +
+        // .hasOwnProperty(). The latter throws "hasOwnProperty is not a
+        // function" on any object that either lacks the Object prototype
+        // (e.g. Object.create(null)) or has a column literally named
+        // `hasOwnProperty` shadowing the prototype method — both of which
+        // can happen with payloads from Appwrite / arbitrary backends.
+        for (const key of Object.keys(obj)) {
+            const value = obj[key];
+            if (value !== null && typeof value === 'object') {
+                deepSeal(value);
             }
         }
     }
@@ -387,14 +391,20 @@ function createReactiveReferences(data, dataSourceName = null) {
     }
 
     if (typeof data === 'object') {
-        // Create new object with new references for each property
+        // Create new object with new references for each property.
+        // Iterate via Object.keys() (own enumerable, no prototype walk)
+        // rather than for…in + .hasOwnProperty(). The latter pattern
+        // throws "hasOwnProperty is not a function" on any payload that
+        // either has a column literally named `hasOwnProperty` shadowing
+        // the prototype, or lacks the Object prototype entirely
+        // (Object.create(null), some SDK response shapes). This is the
+        // hot path for every Appwrite mutation result and realtime event,
+        // so it must be defensive about arbitrary backend payloads.
         const newObj = {};
-        for (const key in data) {
-            if (data.hasOwnProperty(key)) {
-                const value = data[key];
-                // Recursively create new references for nested objects/arrays
-                newObj[key] = createReactiveReferences(value, dataSourceName);
-            }
+        for (const key of Object.keys(data)) {
+            const value = data[key];
+            // Recursively create new references for nested objects/arrays
+            newObj[key] = createReactiveReferences(value, dataSourceName);
         }
 
         // Detect file objects (have mimeType or sizeOriginal)
@@ -8732,6 +8742,29 @@ function registerXMagicMethod(loadDataSource) {
                                                         if (key in target && typeof target[key] === 'function') {
                                                             return target[key].bind(target);
                                                         }
+                                                        // Appwrite sources intentionally skip the client-side
+                                                        // $query attachment (see attachArrayMethods comment:
+                                                        // "Appwrite sources will get their $query from the
+                                                        // Appwrite plugin"). Delegate to the Appwrite methods
+                                                        // handler so the click hits the backend instead of
+                                                        // silently falling through to `undefined` or, in some
+                                                        // proxy paths, a no-op `() => []`. Without this, demo
+                                                        // sort/query buttons appear to do nothing — no console
+                                                        // error, no network request — because the call resolves
+                                                        // to the chaining fallback's stub `queryFn`.
+                                                        if (key === '$query' || key === '$search') {
+                                                            const createAppwriteMethodsHandler = window.ManifestDataProxiesAppwrite?.createAppwriteMethodsHandler;
+                                                            if (createAppwriteMethodsHandler) {
+                                                                try {
+                                                                    const manifest = window.ManifestComponentsRegistry?.manifest || null;
+                                                                    const dataSource = manifest?.data?.[prop] || manifest?.appwrite?.[prop];
+                                                                    if (dataSource && window.ManifestDataConfig?.isAppwriteCollection?.(dataSource)) {
+                                                                        const methodsHandler = createAppwriteMethodsHandler(prop, loadDataSource);
+                                                                        return methodsHandler.bind(null, key);
+                                                                    }
+                                                                } catch { /* fall through */ }
+                                                            }
+                                                        }
                                                     }
                                                     // Forward all other property access to the target array
                                                     const value = target[key];
@@ -9049,6 +9082,22 @@ function registerXMagicMethod(loadDataSource) {
                                                 if (target && typeof target === 'object' && key in target && typeof target[key] === 'function') {
                                                     return target[key].bind(target);
                                                 }
+                                                // Appwrite-source delegation: $query is intentionally
+                                                // not attached to Appwrite arrays by attachArrayMethods
+                                                // (it requires a backend round-trip). Route to the
+                                                // Appwrite methods handler instead of the no-op fallback
+                                                // so sort/query/search buttons actually fire requests.
+                                                const createAppwriteMethodsHandler = window.ManifestDataProxiesAppwrite?.createAppwriteMethodsHandler;
+                                                if (createAppwriteMethodsHandler) {
+                                                    try {
+                                                        const manifest = window.ManifestComponentsRegistry?.manifest || null;
+                                                        const dataSource = manifest?.data?.[prop] || manifest?.appwrite?.[prop];
+                                                        if (dataSource && window.ManifestDataConfig?.isAppwriteCollection?.(dataSource)) {
+                                                            const methodsHandler = createAppwriteMethodsHandler(prop, loadDataSource);
+                                                            return methodsHandler.bind(null, key);
+                                                        }
+                                                    } catch { /* fall through */ }
+                                                }
                                                 // Fallback: return safe function that returns empty array
                                                 return function () {
                                                     return [];

package/lib/manifest.integrity.json

@@ -6,7 +6,7 @@
   "manifest.color.js": "sha384-Z9G/lzt0vVMxjz4wkPuGG1X9mmQAJR15aOoGX3ephf7r2wnlUWet5GLgkUMtT4vt",
   "manifest.colorpicker.js": "sha384-0EVn+Ha06h7FIvOxc6WjZYnKYXzi+zba08yKvczSEGTRkWRxyKN2TFrZHI1SDCXu",
   "manifest.components.js": "sha384-3dCTD5EwCZTiX+1obYtDNM3WWwPh2JDQUQQsdRUUK3gs6FXjse1ShkKaT/2jsNaI",
-  "manifest.data.js": "sha384-+wfMPBlMsmLJ7EJWGJMTKGAhLaLCyOVX+Nq+ps3Lly58QC9Dp3XRHK5yangII0yq",
+  "manifest.data.js": "sha384-xdeTXOfmvxTc6M4Fl+bAjGfi3XulV2iPFGvRcKyK0ml4dwHAqSy3X9ZC/UJzpDk2",
   "manifest.dropdowns.js": "sha384-WMrFoSpKfJuo81dyrwhVrDO8rq+rDwh2x8x4nH01BY5ZHkvjE+/SaT2gWCI0zOn+",
   "manifest.export.js": "sha384-qvdGz1TiGEDOeWJ5os1z03RURdKX+ezZEQ1KyV+9iC7X0esLK83mtY87t4MQv45t",
   "manifest.icons.js": "sha384-uOkboYrovjCpl22eey3Jaxpey+pOnot5NDnRRumcRxiR7IOVaRh1i20gYnWXR5dW",

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "mnfst",
-    "version": "0.5.102",
+    "version": "0.5.104",
     "private": false,
     "workspaces": [
         "templates/starter",