kiro-spec-engine 1.20.1 → 1.20.2

package/CHANGELOG.md

@@ -7,6 +7,21 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [1.20.2] - 2026-02-01
+
+### Fixed
+- **Nested Repository Validation**: Fixed circular reference detection for large nested repository sets
+  - Circular reference detection now uses normalized paths consistently
+  - Fixed bug where original paths were used instead of normalized paths in cycle detection
+  - Now correctly handles 100+ nested repositories
+  - All parent-child relationships validated correctly
+
+### Technical Details
+- Updated `detectCycle()` function to use normalized paths throughout
+- Fixed `pathMap` lookup to use normalized parent paths
+- Ensures consistency between parent validation and cycle detection
+- No performance regression for large repository counts
+
 ## [1.20.1] - 2026-02-01
 
 ### Fixed

package/lib/repo/config-manager.js

@@ -341,39 +341,43 @@ class ConfigManager {
     const visited = new Set();
     const recursionStack = new Set();
 
-    const detectCycle = (repoPath, path = []) => {
-      if (recursionStack.has(repoPath)) {
+    const detectCycle = (normalizedPath, path = []) => {
+      if (recursionStack.has(normalizedPath)) {
         // Found a cycle
-        const cycleStart = path.indexOf(repoPath);
-        const cycle = path.slice(cycleStart).concat(repoPath);
+        const cycleStart = path.indexOf(normalizedPath);
+        const cycle = path.slice(cycleStart).concat(normalizedPath);
         errors.push(
           `Circular parent reference detected: ${cycle.join(' → ')}`
         );
         return true;
       }
 
-      if (visited.has(repoPath)) {
+      if (visited.has(normalizedPath)) {
         return false;
       }
 
-      visited.add(repoPath);
-      recursionStack.add(repoPath);
-      path.push(repoPath);
+      visited.add(normalizedPath);
+      recursionStack.add(normalizedPath);
+      path.push(normalizedPath);
 
-      const repo = pathMap.get(repoPath);
+      const repo = pathMap.get(normalizedPath);
       if (repo && repo.parent) {
-        detectCycle(repo.parent, path);
+        const normalizedParent = this._normalizePath(repo.parent);
+        detectCycle(normalizedParent, path);
       }
 
       path.pop();
-      recursionStack.delete(repoPath);
+      recursionStack.delete(normalizedPath);
       return false;
     };
 
-    // Check for cycles starting from each repository
+    // Check for cycles starting from each repository (use normalized paths)
     repos.forEach(repo => {
-      if (repo.path && !visited.has(repo.path)) {
-        detectCycle(repo.path, []);
+      if (repo.path) {
+        const normalizedPath = this._normalizePath(repo.path);
+        if (!visited.has(normalizedPath)) {
+          detectCycle(normalizedPath, []);
+        }
       }
     });
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "kiro-spec-engine",
-  "version": "1.20.1",
+  "version": "1.20.2",
   "description": "kiro-spec-engine (kse) - A CLI tool and npm package for spec-driven development with AI coding assistants. NOT the Kiro IDE desktop application.",
   "main": "index.js",
   "bin": {