🤖 fix: strip trailing slashes from project paths

Fixes #1063

Project paths with trailing slashes (e.g. '/home/user/project/')
caused mangled workspace names because basename('/path/') returns
an empty string.

The fix strips trailing slashes in validateProjectPath(), which is
the central validation point for all project paths.

Author: ammar-agent

src/node/utils/pathUtils.test.ts

@@ -129,5 +129,23 @@ describe("pathUtils", () => {
       expect(result.valid).toBe(true);
       expect(result.expandedPath).toBe(tempDir);
     });
+
+    it("should strip trailing slashes from path", async () => {
+      // Create .git directory for validation
+      // eslint-disable-next-line local/no-sync-fs-methods -- Test setup only
+      fs.mkdirSync(path.join(tempDir, ".git"));
+
+      // Test with single trailing slash
+      const resultSingle = await validateProjectPath(`${tempDir}/`);
+      expect(resultSingle.valid).toBe(true);
+      expect(resultSingle.expandedPath).toBe(tempDir);
+      expect(resultSingle.expandedPath).not.toMatch(/[/\\]$/);
+
+      // Test with multiple trailing slashes
+      const resultMultiple = await validateProjectPath(`${tempDir}//`);
+      expect(resultMultiple.valid).toBe(true);
+      expect(resultMultiple.expandedPath).toBe(tempDir);
+      expect(resultMultiple.expandedPath).not.toMatch(/[/\\]$/);
+    });
   });
 });

src/node/utils/pathUtils.ts

@@ -47,8 +47,9 @@ export async function validateProjectPath(inputPath: string): Promise<PathValida
   // Expand tilde if present
   const expandedPath = expandTilde(inputPath);
 
-  // Normalize to resolve any .. or . in the path
-  const normalizedPath = path.normalize(expandedPath);
+  // Normalize to resolve any .. or . in the path, then strip trailing slashes
+  // (path.normalize preserves a single trailing slash which breaks basename extraction)
+  const normalizedPath = path.normalize(expandedPath).replace(/[/\\]+$/, "");
 
   // Check if path exists
   try {