@xuda.io/notification_module 1.1.126 → 1.1.127

package/index.mjs

@@ -273,17 +273,12 @@ export const get_notifications = async (req) => {
     selector.sender_uid = req.uid;
   } else {
     if (req.system) {
-      selector.$or = [
-        {
-          uid: req.uid,
-        },
-        {
-          uid: 'system',
-        },
-        {
-          uid: req.to_app_id,
-        },
-      ];
+      // Drop empty/undefined uid clauses (e.g. an unauthenticated/public call where
+      // req.uid is undefined, or to_app_id is ''), which otherwise build a malformed
+      // Mango $or and make find_couch_query throw -> 400.
+      selector.$or = [{ uid: req.uid }, { uid: 'system' }, { uid: req.to_app_id }].filter(
+        (clause) => typeof clause.uid === 'string' && clause.uid.length > 0,
+      );
     } else {
       if (!req.to_app_id) {
         selector.uid = req.uid;
@@ -589,9 +584,17 @@ export const get_system_notifications = async (req) => {
   req.system = true;
   req.to_app_id = '';
   req.read = false;
-  return await get_notifications(req);
+  // System notifications are non-critical app-init data fetched on every page load
+  // (including unauthenticated ones). A DB/query failure must never surface as a 400
+  // to the client (it stalls the loader) -> degrade to an empty list.
+  const ret = await get_notifications(req);
+  if (!ret || ret.code < 0) {
+    return { code: 1, data: { docs: [] } };
+  }
+  return ret;
 };
 
+
 export const notifications_validation_fx = async (notification_id) => {
   let { code, data: doc } = await db_module.get_couch_doc('xuda_notification', notification_id);
   const delete_notification = async () => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@xuda.io/notification_module",
-  "version": "1.1.126",
+  "version": "1.1.127",
   "description": "Xuda Notification Server Module",
   "main": "index.mjs",
   "dependencies": {