kiro-spec-engine 1.20.2 → 1.20.4

package/CHANGELOG.md

@@ -7,6 +7,45 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [1.20.4] - 2026-02-01 🔥 HOTFIX
+
+### Fixed
+- **Multi-Repository Validation Bug**: Fixed critical validation logic that incorrectly rejected valid multi-repository configurations
+  - Independent repositories (non-overlapping paths) now pass validation regardless of `nestedMode` setting
+  - Validation now correctly distinguishes between duplicate paths (always invalid) and nested paths (invalid only without `nestedMode`)
+  - Enhanced error messages with actionable hints: suggests enabling `nestedMode` when nested paths detected
+  - User-reported test cases now pass:
+    - ✅ Two independent repositories (`backend/`, `frontend/`)
+    - ✅ Eight independent repositories
+    - ✅ Nested repositories with `nestedMode: true`
+    - ❌ Nested repositories without `nestedMode` (correctly fails with helpful hint)
+
+### Technical Details
+- Enhanced `_validatePaths()` method to categorize errors into duplicate and nested types
+- Duplicate path errors always reported (always invalid)
+- Nested path errors only reported when `nestedMode` is false or undefined
+- Added hint message: "Enable nestedMode in settings to allow nested repositories: { \"settings\": { \"nestedMode\": true } }"
+- Root cause: Previous logic didn't distinguish between independent and nested repositories
+- Reference: User bug report with 4 test cases demonstrating the issue
+
+## [1.20.3] - 2026-02-01
+
+### Fixed
+- **Nested Scanning Validation**: Fixed three critical validation issues preventing nested repository configuration saves
+  - Repository names starting with dots (`.github`, `.kiro`) now accepted as valid
+  - Path overlap validation now context-aware: allows overlapping paths in nested mode, rejects in non-nested mode
+  - Fixed empty name/path bug for root directory repositories (now normalized to '.')
+  - Added `settings.nestedMode` field to track scanning mode in configuration
+  - Successfully tested with 104 nested repositories in real-world project
+
+### Technical Details
+- Updated `_isValidRepoName()` regex to allow names starting with dots: `/^\.?[a-zA-Z0-9][a-zA-Z0-9._-]*$/`
+- Modified `_validatePaths()` to accept `allowNested` parameter and skip overlap errors in nested mode
+- Updated `validateConfig()` to pass `settings.nestedMode` to path validation
+- Fixed `discoverRepositories()` to normalize empty relativePath to '.' instead of empty string
+- Added detailed error reporting in `init-handler.js` to show validation errors during scanning
+- All 1686 tests passing
+
 ## [1.20.2] - 2026-02-01
 
 ### Fixed

package/lib/repo/config-manager.js

@@ -187,7 +187,9 @@ class ConfigManager {
 
     // Validate no duplicate or overlapping paths
     if (repoPaths.length > 0) {
-      const pathValidation = this._validatePaths(repoPaths);
+      // Check if nested mode is enabled in settings
+      const nestedMode = config.settings && config.settings.nestedMode === true;
+      const pathValidation = this._validatePaths(repoPaths, nestedMode);
       errors.push(...pathValidation.errors);
     }
 
@@ -388,9 +390,10 @@ class ConfigManager {
    * Validate repository paths for duplicates and overlaps
    * @private
    * @param {string[]} paths - Array of repository paths
+   * @param {boolean} allowNested - Whether to allow nested paths (for nested mode)
    * @returns {{errors: string[]}} Validation result
    */
-  _validatePaths(paths) {
+  _validatePaths(paths, allowNested = false) {
     const errors = [];
 
     // Resolve all paths to absolute for comparison
@@ -408,10 +411,30 @@ class ConfigManager {
     const pathValidation = this.pathResolver.validateNoOverlap(resolvedPaths);
     
     if (!pathValidation.valid) {
-      // Map resolved paths back to original paths in error messages
+      // Categorize errors into duplicate and nested types
+      const duplicateErrors = [];
+      const nestedErrors = [];
+      
       pathValidation.errors.forEach(error => {
-        errors.push(error);
+        if (error.includes('Duplicate path found')) {
+          duplicateErrors.push(error);
+        } else if (error.includes('nested within')) {
+          nestedErrors.push(error);
+        }
       });
+      
+      // Always report duplicate paths (always invalid)
+      errors.push(...duplicateErrors);
+      
+      // Report nested paths only if nestedMode is not enabled
+      if (nestedErrors.length > 0 && !allowNested) {
+        errors.push(...nestedErrors);
+        // Add helpful hint about enabling nestedMode
+        errors.push(
+          'Hint: Enable nestedMode in settings to allow nested repositories: ' +
+          '{ "settings": { "nestedMode": true } }'
+        );
+      }
     }
 
     return { errors };
@@ -424,9 +447,10 @@ class ConfigManager {
    * @returns {boolean} True if name is valid
    */
   _isValidRepoName(name) {
-    // Allow alphanumeric, hyphens, underscores, and dots
-    // Must not start or end with special characters
-    const validPattern = /^[a-zA-Z0-9][a-zA-Z0-9._-]*[a-zA-Z0-9]$|^[a-zA-Z0-9]$/;
+    // Allow alphanumeric, hyphens, underscores, dots, and names starting with dots (hidden directories)
+    // Must not contain spaces or special characters like @, #, $, etc.
+    // Examples: "backend", ".github", ".kiro", "my-repo", "repo_name", "repo.name"
+    const validPattern = /^\.?[a-zA-Z0-9][a-zA-Z0-9._-]*$/;
     return validPattern.test(name);
   }
 

package/lib/repo/handlers/init-handler.js

@@ -76,7 +76,10 @@ class InitHandler {
         remote: repo.remote,
         defaultBranch: repo.branch,
         parent: repo.parent || null // Include parent field
-      }))
+      })),
+      settings: {
+        nestedMode: nested // Store nested mode setting
+      }
     };
 
     // Save configuration
@@ -84,6 +87,15 @@ class InitHandler {
       await this.configManager.saveConfig(config);
     } catch (error) {
       console.log(this.formatter.error(`Failed to save configuration: ${error.message}`));
+      
+      // Display detailed validation errors if available
+      if (error.details && error.details.errors && Array.isArray(error.details.errors)) {
+        console.log(this.formatter.error('\nValidation errors:'));
+        error.details.errors.forEach((err, index) => {
+          console.log(this.formatter.error(`  ${index + 1}. ${err}`));
+        });
+      }
+      
       throw error;
     }
 

package/lib/repo/repo-manager.js

@@ -159,10 +159,12 @@ class RepoManager {
         
         // Generate repository name from path
         const relativePath = this.pathResolver.toRelative(currentPath, rootPath);
-        const repoName = relativePath === '.' ? path.basename(rootPath) : relativePath.replace(/\//g, '-');
+        // Handle empty path (root directory) or '.' (current directory)
+        const normalizedPath = (!relativePath || relativePath === '.') ? '.' : relativePath;
+        const repoName = normalizedPath === '.' ? path.basename(rootPath) : normalizedPath.replace(/\//g, '-');
         
         discovered.push({
-          path: relativePath,
+          path: normalizedPath,
           name: repoName,
           remote: remoteUrl,
           branch: branch,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "kiro-spec-engine",
-  "version": "1.20.2",
+  "version": "1.20.4",
   "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": {