fix: use direct git clone instead of node desktop.js add

The previous approach ran 'node desktop.js add <pkg> --from <url>' which
internally called 'pnpm add @owdproject/pkg@workspace:*' inside desktop/.
This caused pnpm to print 'Preparing...' and block the progress bar.

Since bridge.WriteChanges already writes both package.json and desktop.config.ts
before RunSetupTask is called, we only need to git clone the repo into the
correct directory (apps/, packages/, themes/).

New flow per package:
- npm: skip clone, pnpm install will handle it
- local: skip, folder already present
- git-https / git-ssh: git clone directly into apps|packages|themes/<shortName>
  (inferred from catalog entry kind or shortname prefix)
  Skip if package.json already exists in target (already cloned)

Then: pnpm install + prepare:modules as before.

Author: dxlliv

go/tui/tasks.go

@@ -105,57 +105,98 @@ func (m *TuiModel) RunSetupTask(adds map[string]string) {
 	}
 
 	go func() {
-		desktopJs := filepath.Join(workspaceRoot, "packages", "cli", "bin", "desktop.js")
-
 		totalSteps := len(adds) + 2
 		step := 0
-		msgChan <- setupProgressMsg{Step: step, Total: totalSteps, Label: "Initializing setup task…"}
+		msgChan <- setupProgressMsg{Step: step, Total: totalSteps, Label: "Initializing…"}
 
-		// 1. Process all additions
+		// 1. Clone each package via git directly (bridge.WriteChanges already updated
+		//    package.json and desktop.config.ts, so we just need the git clone)
 		for pkgName, method := range adds {
 			step++
 			shortName := pkgName
 			if idx := strings.LastIndex(pkgName, "/"); idx >= 0 {
 				shortName = pkgName[idx+1:]
 			}
-			msgChan <- setupProgressMsg{Step: step, Total: totalSteps, Label: fmt.Sprintf("Installing %s via %s…", shortName, method)}
 
-			args := []string{desktopJs, "add", shortName}
+			msgChan <- setupProgressMsg{Step: step, Total: totalSteps, Label: fmt.Sprintf("Cloning %s…", shortName)}
+
 			if method == "npm" {
-				args = append(args, "--npm")
-			} else if method == "local" {
-				args = append(args, "--dev")
-			} else {
-				owner := "owdproject"
-				if githubUser != "" {
-					owner = githubUser
-				} else {
-					for _, e := range catalogEntries {
-						if e.Name == pkgName && e.Org != "" {
-							owner = e.Org
-							break
-						}
+				// npm install: will be handled by pnpm install below; nothing to clone
+				msgChan <- logLineMsg(fmt.Sprintf(">>> %s will be installed via npm (pnpm install)", shortName))
+				continue
+			}
+
+			if method == "local" {
+				// local workspace folder already present; nothing to clone
+				msgChan <- logLineMsg(fmt.Sprintf(">>> %s already available as local workspace package", shortName))
+				continue
+			}
+
+			// Determine target directory based on package kind
+			kindDir := ""
+			for _, e := range catalogEntries {
+				if e.Name == pkgName {
+					switch e.Kind {
+					case "app":
+						kindDir = "apps"
+					case "module":
+						kindDir = "packages"
+					case "theme":
+						kindDir = "themes"
 					}
+					break
+				}
+			}
+			if kindDir == "" {
+				// Infer from shortname prefix
+				switch {
+				case strings.HasPrefix(shortName, "app-"):
+					kindDir = "apps"
+				case strings.HasPrefix(shortName, "theme-"):
+					kindDir = "themes"
+				default:
+					kindDir = "packages"
 				}
+			}
 
-				var fromVal string
-				if method == "git-ssh" {
-					fromVal = fmt.Sprintf("git@github.com:%s/%s.git", owner, shortName)
-				} else {
-					fromVal = fmt.Sprintf("https://github.com/%s/%s.git", owner, shortName)
+			targetDir := filepath.Join(workspaceRoot, kindDir, shortName)
+
+			// Skip if already cloned
+			if _, err := os.Stat(filepath.Join(targetDir, "package.json")); err == nil {
+				msgChan <- logLineMsg(fmt.Sprintf(">>> %s already cloned — skipping", shortName))
+				continue
+			}
+
+			// Resolve git URL
+			owner := "owdproject"
+			if githubUser != "" {
+				owner = githubUser
+			} else {
+				for _, e := range catalogEntries {
+					if e.Name == pkgName && e.Org != "" {
+						owner = e.Org
+						break
+					}
 				}
-				args = append(args, "--from", fromVal)
 			}
 
-			msgChan <- logLineMsg(fmt.Sprintf(">>> Executing: node desktop.js add %s via %s", shortName, method))
-			if err := runtime.runProcessAndStreamLogsSilent(workspaceRoot, "node", args); err != nil {
-				msgChan <- logLineMsg(fmt.Sprintf(">>> Add %s failed: %v", shortName, err))
+			var gitURL string
+			if method == "git-ssh" {
+				gitURL = fmt.Sprintf("git@github.com:%s/%s.git", owner, shortName)
+			} else {
+				gitURL = fmt.Sprintf("https://github.com/%s/%s.git", owner, shortName)
+			}
+
+			msgChan <- logLineMsg(fmt.Sprintf(">>> Cloning %s from %s", shortName, gitURL))
+			if err := runtime.runProcessAndStreamLogsSilent(workspaceRoot, "git", []string{"clone", gitURL, targetDir}); err != nil {
+				msgChan <- logLineMsg(fmt.Sprintf(">>> Clone failed for %s: %v", shortName, err))
 				msgChan <- taskFinishedMsg{Success: false, Err: err}
 				return
 			}
+			msgChan <- logLineMsg(fmt.Sprintf(">>> ✓ %s cloned successfully", shortName))
 		}
 
-		// 2. Run pnpm install for cleanup/removal sync
+		// 2. Sync workspace with pnpm install
 		step++
 		msgChan <- setupProgressMsg{Step: step, Total: totalSteps, Label: "Installing dependencies (pnpm install)…"}
 		msgChan <- logLineMsg(">>> Running pnpm install (syncing workspace)…")