node-consul-service 1.0.33 → 1.0.35

package/dist/consul/dataLink.d.ts

@@ -6,5 +6,11 @@ interface LinkSchema {
     headers?: AxiosRequestHeaders;
     cacheGetter?: (ids: string[]) => Promise<(any | null)[]>;
 }
+export declare class DataLinkError extends Error {
+    service: string;
+    path: string;
+    originalError: any;
+    constructor(service: string, path: string, originalError: any);
+}
 export declare function dataLink(data: any[] | any, schema: LinkSchema[]): Promise<any[] | any>;
 export {};

package/dist/index.cjs.js

@@ -19430,16 +19430,31 @@ async function callService(serviceName, options = {}) {
     }
 }
 
+// ✅ Custom Error Class
+class DataLinkError extends Error {
+    constructor(service, path, originalError) {
+        super(`DataLinkError: Failed to fetch from ${service}:${path}`);
+        this.name = "DataLinkError";
+        this.service = service;
+        this.path = path;
+        this.originalError = originalError;
+        if (Error.captureStackTrace) {
+            Error.captureStackTrace(this, DataLinkError);
+        }
+    }
+}
+// ✅ Helper to normalize Mongo IDs or ObjectIds
 function normalizeId(id) {
     if (id && typeof id === "object" && typeof id.toString === "function") {
         return id.toString();
     }
     return id;
 }
+// ✅ Main function
 async function dataLink(data, schema) {
     const isArray = Array.isArray(data);
     const sourceData = isArray ? data : [data];
-    // ✅ ننسخ كل عنصر بدون التأثير على الأصل
+    // Clone data to avoid mutating original
     const result = sourceData.map((item) => ({ ...item }));
     await Promise.all(schema.map(async ({ filed, service, path, headers, cacheGetter }) => {
         var _a;
@@ -19465,6 +19480,7 @@ async function dataLink(data, schema) {
             return;
         let cacheMap = new Map();
         let idsToFetch = uniqueIds;
+        // ✅ Check cache first
         if (cacheGetter) {
             const cacheResults = await cacheGetter(uniqueIds);
             idsToFetch = [];
@@ -19478,7 +19494,12 @@ async function dataLink(data, schema) {
                 }
             });
         }
+        // ✅ Validate path
         if (idsToFetch.length > 0) {
+            if (!path.startsWith("/")) {
+                throw new DataLinkError(service, path, new Error(`Invalid path: "${path}". Path must start with "/"`));
+            }
+            // ✅ Fetch remaining from service
             try {
                 const response = await callService(service, {
                     method: "POST",
@@ -19495,10 +19516,10 @@ async function dataLink(data, schema) {
                 }
             }
             catch (error) {
-                console.error(`❌ Failed to fetch from ${service}${path}:`, JSON.stringify(error, Object.getOwnPropertyNames(error), 2));
-                return error;
+                throw new DataLinkError(service, path, error);
             }
         }
+        // ✅ Replace ids in result with full objects
         for (const item of result) {
             const value = item[filed];
             if (typeof value === "string" ||
@@ -19522,6 +19543,7 @@ async function dataLink(data, schema) {
     return isArray ? result : result[0];
 }
 
+exports.DataLinkError = DataLinkError;
 exports.callService = callService;
 exports.dataLink = dataLink;
 exports.deregisterService = deregisterService;

package/dist/index.esm.js

@@ -19428,16 +19428,31 @@ async function callService(serviceName, options = {}) {
     }
 }
 
+// ✅ Custom Error Class
+class DataLinkError extends Error {
+    constructor(service, path, originalError) {
+        super(`DataLinkError: Failed to fetch from ${service}:${path}`);
+        this.name = "DataLinkError";
+        this.service = service;
+        this.path = path;
+        this.originalError = originalError;
+        if (Error.captureStackTrace) {
+            Error.captureStackTrace(this, DataLinkError);
+        }
+    }
+}
+// ✅ Helper to normalize Mongo IDs or ObjectIds
 function normalizeId(id) {
     if (id && typeof id === "object" && typeof id.toString === "function") {
         return id.toString();
     }
     return id;
 }
+// ✅ Main function
 async function dataLink(data, schema) {
     const isArray = Array.isArray(data);
     const sourceData = isArray ? data : [data];
-    // ✅ ننسخ كل عنصر بدون التأثير على الأصل
+    // Clone data to avoid mutating original
     const result = sourceData.map((item) => ({ ...item }));
     await Promise.all(schema.map(async ({ filed, service, path, headers, cacheGetter }) => {
         var _a;
@@ -19463,6 +19478,7 @@ async function dataLink(data, schema) {
             return;
         let cacheMap = new Map();
         let idsToFetch = uniqueIds;
+        // ✅ Check cache first
         if (cacheGetter) {
             const cacheResults = await cacheGetter(uniqueIds);
             idsToFetch = [];
@@ -19476,7 +19492,12 @@ async function dataLink(data, schema) {
                 }
             });
         }
+        // ✅ Validate path
         if (idsToFetch.length > 0) {
+            if (!path.startsWith("/")) {
+                throw new DataLinkError(service, path, new Error(`Invalid path: "${path}". Path must start with "/"`));
+            }
+            // ✅ Fetch remaining from service
             try {
                 const response = await callService(service, {
                     method: "POST",
@@ -19493,10 +19514,10 @@ async function dataLink(data, schema) {
                 }
             }
             catch (error) {
-                console.error(`❌ Failed to fetch from ${service}${path}:`, JSON.stringify(error, Object.getOwnPropertyNames(error), 2));
-                return error;
+                throw new DataLinkError(service, path, error);
             }
         }
+        // ✅ Replace ids in result with full objects
         for (const item of result) {
             const value = item[filed];
             if (typeof value === "string" ||
@@ -19520,5 +19541,5 @@ async function dataLink(data, schema) {
     return isArray ? result : result[0];
 }
 
-export { callService, dataLink, deregisterService, getRandomServiceInstance, getServiceInstances, getServiceUrl, listServices, registerService };
+export { DataLinkError, callService, dataLink, deregisterService, getRandomServiceInstance, getServiceInstances, getServiceUrl, listServices, registerService };
 //# sourceMappingURL=index.esm.js.map

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "node-consul-service",
-  "version": "1.0.33",
+  "version": "1.0.35",
   "main": "dist/index.cjs.js",
   "module": "dist/index.esm.js",
   "types": "dist/index.d.ts",