norn-cli 1.2.2 → 1.2.3

package/CHANGELOG.md

@@ -2,6 +2,22 @@
 
 All notable changes to the "Norn" extension will be documented in this file.
 
+## [1.2.3] - 2026-02-07
+
+### Added
+- **Duplicate import detection**: Errors shown when importing files with duplicate definitions
+  - Header groups and endpoints from `.nornapi` files
+  - Named requests and sequences from `.norn` files
+  - Blocks execution in CLI, VS Code, and Test Explorer
+  - Red squiggly on import line with descriptive error message
+
+### Fixed
+- **.nornenv validation**: `secret var = value` now shows error (use `secret name = value` instead)
+  - Syntax highlighting marks invalid pattern in red
+  - Diagnostic message explains correct usage
+- **IntelliSense context**: Script types (`bash`, `powershell`, `js`, `readJson`) only appear after `run` keyword
+- **var keyword highlighting**: `var` keyword stays colored while typing variable name
+
 ## [1.2.2] - 2026-02-05
 
 ### Added

package/dist/cli.js

@@ -13493,6 +13493,10 @@ async function resolveImports(text, baseDir, readFile2, alreadyImported = /* @__
   const resolvedPaths = [];
   const headerGroups = [];
   const endpoints = [];
+  const headerGroupSources = /* @__PURE__ */ new Map();
+  const endpointSources = /* @__PURE__ */ new Map();
+  const namedRequestSources = /* @__PURE__ */ new Map();
+  const sequenceSources = /* @__PURE__ */ new Map();
   for (const imp of imports) {
     const path4 = await import("path");
     const absolutePath = path4.resolve(baseDir, imp.path);
@@ -13510,16 +13514,64 @@ async function resolveImports(text, baseDir, readFile2, alreadyImported = /* @__
       resolvedPaths.push(absolutePath);
       if (imp.path.endsWith(".nornapi")) {
         const apiDef = parseNornApiFile(content);
-        headerGroups.push(...apiDef.headerGroups);
-        endpoints.push(...apiDef.endpoints);
+        for (const group of apiDef.headerGroups) {
+          const existingSource = headerGroupSources.get(group.name);
+          if (existingSource) {
+            errors.push({
+              path: imp.path,
+              error: `Duplicate header group '${group.name}': already defined in '${existingSource}'`,
+              lineNumber: imp.lineNumber
+            });
+          } else {
+            headerGroupSources.set(group.name, imp.path);
+            headerGroups.push(group);
+          }
+        }
+        for (const endpoint of apiDef.endpoints) {
+          const existingSource = endpointSources.get(endpoint.name);
+          if (existingSource) {
+            errors.push({
+              path: imp.path,
+              error: `Duplicate endpoint '${endpoint.name}': already defined in '${existingSource}'`,
+              lineNumber: imp.lineNumber
+            });
+          } else {
+            endpointSources.set(endpoint.name, imp.path);
+            endpoints.push(endpoint);
+          }
+        }
         continue;
       }
       const importDir = path4.dirname(absolutePath);
       const nestedResult = await resolveImports(content, importDir, readFile2, alreadyImported);
       errors.push(...nestedResult.errors);
       resolvedPaths.push(...nestedResult.resolvedPaths);
-      headerGroups.push(...nestedResult.headerGroups);
-      endpoints.push(...nestedResult.endpoints);
+      for (const group of nestedResult.headerGroups) {
+        const existingSource = headerGroupSources.get(group.name);
+        if (existingSource) {
+          errors.push({
+            path: imp.path,
+            error: `Duplicate header group '${group.name}': already defined in '${existingSource}'`,
+            lineNumber: imp.lineNumber
+          });
+        } else {
+          headerGroupSources.set(group.name, imp.path);
+          headerGroups.push(group);
+        }
+      }
+      for (const endpoint of nestedResult.endpoints) {
+        const existingSource = endpointSources.get(endpoint.name);
+        if (existingSource) {
+          errors.push({
+            path: imp.path,
+            error: `Duplicate endpoint '${endpoint.name}': already defined in '${existingSource}'`,
+            lineNumber: imp.lineNumber
+          });
+        } else {
+          endpointSources.set(endpoint.name, imp.path);
+          endpoints.push(endpoint);
+        }
+      }
       if (nestedResult.importedContent) {
         importedContents.push(nestedResult.importedContent);
       }
@@ -13527,15 +13579,37 @@ async function resolveImports(text, baseDir, readFile2, alreadyImported = /* @__
       const importedNamedRequests = extractNamedRequests(content);
       const importedSequences = extractSequencesFromText(content);
       for (const req of importedNamedRequests) {
-        const resolvedContent = substituteVariables(req.content, importedVariables);
-        importedContents.push(`[${req.name}]
+        const lowerName = req.name.toLowerCase();
+        const existingSource = namedRequestSources.get(lowerName);
+        if (existingSource) {
+          errors.push({
+            path: imp.path,
+            error: `Duplicate named request '${req.name}': already defined in '${existingSource}'`,
+            lineNumber: imp.lineNumber
+          });
+        } else {
+          namedRequestSources.set(lowerName, imp.path);
+          const resolvedContent = substituteVariables(req.content, importedVariables);
+          importedContents.push(`[${req.name}]
 ${resolvedContent}`);
+        }
       }
       for (const seq of importedSequences) {
-        const resolvedContent = substituteVariables(seq.content, importedVariables);
-        importedContents.push(`sequence ${seq.name}
+        const lowerName = seq.name.toLowerCase();
+        const existingSource = sequenceSources.get(lowerName);
+        if (existingSource) {
+          errors.push({
+            path: imp.path,
+            error: `Duplicate sequence '${seq.name}': already defined in '${existingSource}'`,
+            lineNumber: imp.lineNumber
+          });
+        } else {
+          sequenceSources.set(lowerName, imp.path);
+          const resolvedContent = substituteVariables(seq.content, importedVariables);
+          importedContents.push(`sequence ${seq.name}
 ${resolvedContent}
 end sequence`);
+        }
       }
     } catch (error) {
       errors.push({
@@ -23403,9 +23477,22 @@ async function main() {
       workingDir,
       async (importPath) => fsPromises.readFile(importPath, "utf-8")
     );
-    for (const err of importResult.errors) {
+    const duplicateErrors = importResult.errors.filter(
+      (err) => err.error.includes("Duplicate header group") || err.error.includes("Duplicate endpoint") || err.error.includes("Duplicate named request") || err.error.includes("Duplicate sequence")
+    );
+    const otherErrors = importResult.errors.filter(
+      (err) => !err.error.includes("Duplicate header group") && !err.error.includes("Duplicate endpoint") && !err.error.includes("Duplicate named request") && !err.error.includes("Duplicate sequence")
+    );
+    for (const err of otherErrors) {
       console.error(colors.warning(`Import warning: ${err.path} - ${err.error}`));
     }
+    if (duplicateErrors.length > 0) {
+      console.error(colors.error("Cannot execute - duplicate definitions found:"));
+      for (const err of duplicateErrors) {
+        console.error(colors.error(`  ${err.path}: ${err.error}`));
+      }
+      process.exit(1);
+    }
     const fileContentWithImports = importResult.importedContent ? `${importResult.importedContent}
 
 ${fileContent}` : fileContent;
@@ -23512,9 +23599,22 @@ ${fileContent}` : fileContent;
       workingDir,
       async (importPath) => fsPromises.readFile(importPath, "utf-8")
     );
-    for (const err of importResult.errors) {
+    const duplicateErrors2 = importResult.errors.filter(
+      (err) => err.error.includes("Duplicate header group") || err.error.includes("Duplicate endpoint") || err.error.includes("Duplicate named request") || err.error.includes("Duplicate sequence")
+    );
+    const otherErrors2 = importResult.errors.filter(
+      (err) => !err.error.includes("Duplicate header group") && !err.error.includes("Duplicate endpoint") && !err.error.includes("Duplicate named request") && !err.error.includes("Duplicate sequence")
+    );
+    for (const err of otherErrors2) {
       console.error(colors.warning(`Import warning: ${err.path} - ${err.error}`));
     }
+    if (duplicateErrors2.length > 0) {
+      console.error(colors.error("Cannot execute - duplicate definitions found:"));
+      for (const err of duplicateErrors2) {
+        console.error(colors.error(`  ${err.path}: ${err.error}`));
+      }
+      process.exit(1);
+    }
     const fileContentWithImports = importResult.importedContent ? `${importResult.importedContent}
 
 ${fileContent}` : fileContent;

package/package.json

@@ -2,7 +2,7 @@
   "name": "norn-cli",
   "displayName": "Norn - REST Client",
   "description": "A powerful REST client for making HTTP requests with sequences, variables, scripts, and cookie support",
-  "version": "1.2.2",
+  "version": "1.2.3",
   "publisher": "Norn-PeterKrustanov",
   "author": {
     "name": "Peter Krastanov"