@query-doctor/core 0.1.7 → 0.1.8

package/dist/index.js

@@ -294,9 +294,12 @@ var Walker = class _Walker {
             part.schema = node.RangeVar.schemaname;
           }
           if (existingMapping) {
-            console.warn(
-              `Ignoring alias ${aliasName} as it shadows an existing mapping for ${existingMapping.text}. We currently do not support alias shadowing.`
-            );
+            const isSystemCatalog = node.RangeVar.relname?.startsWith("pg_") ?? false;
+            if (!isSystemCatalog) {
+              console.warn(
+                `Ignoring alias ${aliasName} as it shadows an existing mapping for ${existingMapping.text}. We currently do not support alias shadowing.`
+              );
+            }
             this.shadowedAliases.push(part);
             return;
           }
@@ -694,7 +697,7 @@ var Analyzer = class {
       return { tags: [], queryWithoutTags: trimmedQuery };
     }
     const queryWithoutTags = trimmedQuery.slice(0, startPosition);
-    const tagString = trimmedQuery.slice(startPosition + 2, endPosition);
+    const tagString = trimmedQuery.slice(startPosition + 2, endPosition).trim();
     if (!tagString || typeof tagString !== "string") {
       return { tags: [], queryWithoutTags };
     }
@@ -702,7 +705,11 @@ var Analyzer = class {
     for (const match of tagString.split(",")) {
       const [key, value] = match.split("=");
       if (!key || !value) {
-        console.warn(`Invalid sqlcommenter tag: ${match}. Ignoring`);
+        if (tags.length > 0) {
+          console.warn(
+            `Invalid sqlcommenter tag: ${match} in comment: ${tagString}. Ignoring`
+          );
+        }
         continue;
       }
       try {