cartographer-ai 1.0.3 → 1.0.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cartographer-ai",
-  "version": "1.0.3",
+  "version": "1.0.7",
   "main": "src/commands.js",
   "type": "module",
   "bin": {

package/src/cartographer.js

@@ -6,6 +6,8 @@ import { execSync } from "child_process";
 const regex =
   /@cartographer\s*((?:\s*(?:id|parentId|name|diagramType|context|spaceId|folderId):\s*[^,\n]+,\s*)*(?:\s*(?:id|parentId|name|diagramType|context|spaceId|folderId):\s*[^,\n]+)?)/;
 const ignoredDirectories = ["node_modules", ".git", "build"];
+export const warningPrefix = "\x1b[33mWarning:\x1b[0m";
+export const errorPrefix = "\x1b[31mError:\x1b[0m";
 
 export class Cartographer {
   constructor({ processAllFiles = false, spaceId, apiKey }) {
@@ -90,9 +92,16 @@ export class Cartographer {
         },
       });
     } catch (error) {
-      throw new Error(
-        error?.response?.data?.error ?? "Unexpected error occured"
-      );
+      if (error?.response?.data?.error) {
+        const errorMessage = `${errorPrefix} ${
+          error?.response?.data?.error ?? "Unexpected error occured"
+        }`;
+
+        const err = new Error(errorMessage);
+        err.stack = "";
+
+        throw err;
+      }
     }
   }
 
@@ -152,6 +161,27 @@ export class Cartographer {
       this.#getTrimmedFilePath(filePath, gitRoot)
     );
 
+    if (file.parentId) {
+      const forbidden = [
+        "id",
+        "name",
+        "diagramType",
+        "context",
+        "spaceId",
+        "folderId",
+      ].filter((key) => file[key]);
+
+      if (forbidden.length > 0) {
+        console.warn(
+          `${warningPrefix} File "${filePath}" is invalid. When "parentId" is present, the following attributes must not be used: ${forbidden.join(
+            ", "
+          )}. These attributes have been ignored.`
+        );
+      }
+
+      forbidden.forEach((field) => (file[field] = ""));
+    }
+
     return file;
   }
 
@@ -191,7 +221,11 @@ export class Cartographer {
 
         files[resolvedId].push(file);
       } catch (error) {
-        throw new Error(`Error reading file ${filePath}:`, error.message);
+        const errorMessage = `${errorPrefix} Error reading file ${filePath}, error: ${error.message}`;
+        const err = new Error(errorMessage);
+        err.stack = "";
+
+        throw err;
       }
     });
 

package/src/commands.js

@@ -2,8 +2,8 @@
 
 import fs from "fs";
 import path from "path";
+import { Cartographer, errorPrefix } from "./cartographer.js";
 import { fileURLToPath } from "url";
-import { Cartographer } from "./cartographer.js";
 
 const args = process.argv.slice(2);
 const fields = ["spaceId", "apiKey"];
@@ -33,7 +33,11 @@ export const saveConfig = (configPath, newConfig) => {
 
 const handleCartographerCommand = (arg) => {
   if (!arg) {
-    throw new Error("No command received");
+    const errorMessage = `${errorPrefix} No command received`;
+    const err = new Error(errorMessage);
+    err.stack = "";
+
+    throw err;
   }
 
   const config = loadConfig(configPath);
@@ -46,9 +50,14 @@ const handleCartographerCommand = (arg) => {
     }, []);
 
     if (missingFields.length > 0) {
-      throw new Error(
-        `Error: Missing required fields: ${missingFields.join(", ")}`
-      );
+      const errorMessage = `${errorPrefix} Error: Missing required fields: ${missingFields.join(
+        ", "
+      )}`;
+
+      const err = new Error(errorMessage);
+      err.stack = "";
+
+      throw err;
     } else {
       const cartographer = new Cartographer({
         processAllFiles: config?.processAllFiles ?? false,
@@ -72,7 +81,11 @@ const handleCartographerCommand = (arg) => {
   const field = fields.find((field) => field === command);
 
   if (!field) {
-    throw new Error(`Unknown command: ${command}`);
+    const errorMessage = `${errorPrefix} Unknown command: ${command}`;
+    const err = new Error(errorMessage);
+    err.stack = "";
+
+    throw err;
   }
 
   config[field] = value;