refactor(tui): modularize monolithic tui.go, secure concurrency, and clean style extraction

Author: dxlliv

bin/cli.js

@@ -468,7 +468,11 @@ export async function runCli(name, argv, options = {}) {
       })
       process.exit(0)
     }
-    runNpmInstall(kind, pkgName, workspaceRoot)
+    try {
+      await runNpmInstall(kind, pkgName, workspaceRoot)
+    } catch (error) {
+      fail(error.message ?? String(error))
+    }
   } else {
     try {
       await runWorkspaceInstall({

bin/lib/cpFormat.js

@@ -160,7 +160,7 @@ export function formatCatalogRowPlain(item, ctx = {}) {
 
   // 3.5 DIR Column (4 chars)
   let dirTag = ''
-  const isMissing = item.installed && !item.localSource
+  const isMissing = item.installed && !item.localSource && !item.inPackageJson
   if (isMissing) {
     dirTag = `{red-fg}MISS{/}`
   } else if (item.localSource) {
@@ -321,7 +321,7 @@ export function formatDetailPanel(item, targetDir, colors = DEFAULT_COLORS, widt
     lines.push(`  {${c.muted}-fg}${item.description.slice(0, maxDescLen)}{/}`)
   }
 
-  const isMissing = item.installed && !item.localSource
+  const isMissing = item.installed && !item.localSource && !item.inPackageJson
   if (isMissing) {
     lines.push(`  {red-fg}{bold}⚠ WARNING: Local workspace folder is missing!{/}{/}`)
   }

bin/lib/install.js

@@ -463,16 +463,22 @@ export async function runWorkspaceInstall({
 }
 
 export function runNpmInstall(kind, pkgName, workspaceRoot, stdio = 'inherit') {
-  const nxTarget = KINDS[kind].nx
-  const child = spawn('pnpm', ['nx', 'run', nxTarget, `--name=${pkgName}`], {
-    stdio,
-    shell: true,
-    cwd: workspaceRoot,
-  })
-  child.on('exit', (code) => {
-    if (stdio === 'inherit') process.exit(code ?? 1)
+  return new Promise((resolve, reject) => {
+    const nxTarget = KINDS[kind].nx
+    const child = spawn('pnpm', ['nx', 'run', nxTarget, `--name=${pkgName}`], {
+      stdio,
+      shell: true,
+      cwd: workspaceRoot,
+    })
+    child.on('error', reject)
+    child.on('exit', (code) => {
+      if (code === 0) {
+        resolve()
+      } else {
+        reject(new Error(`Installation failed with exit code ${code}`))
+      }
+    })
   })
-  return child
 }
 
 /** Preview label for TUI list rows */

bin/lib/installWizardFlow.js

@@ -440,7 +440,7 @@ export async function runStartupInstallFlow(ctx, { isStartup = false } = {}) {
 
   const merged = mergeInstalled(catalog, config, deps, workspaceRoot)
   const missingAppsModules = merged
-    .filter((item) => item.installed && !item.localSource && item.kind !== 'theme')
+    .filter((item) => item.installed && !item.localSource && !item.inPackageJson && item.kind !== 'theme')
     .map((item) => item.name)
 
   const desiredTheme = ctx.getPendingTheme() ?? config.theme

go/bridge/bridge.go

@@ -127,6 +127,7 @@ func getBridgePath(root string) string {
 // ReadWorkspaceContext runs the node bridge with --read
 func ReadWorkspaceContext(root string) (*WorkspaceContext, error) {
 	cmd := exec.Command("node", getBridgePath(root), "--read")
+	cmd.Dir = root
 	var stdout, stderr bytes.Buffer
 	cmd.Stdout = &stdout
 	cmd.Stderr = &stderr
@@ -150,6 +151,7 @@ func ReadCatalog(root string, force bool) (*CatalogResponse, error) {
 		args = append(args, "--force")
 	}
 	cmd := exec.Command("node", args...)
+	cmd.Dir = root
 	var stdout, stderr bytes.Buffer
 	cmd.Stdout = &stdout
 	cmd.Stderr = &stderr
@@ -169,6 +171,7 @@ func ReadCatalog(root string, force bool) (*CatalogResponse, error) {
 // WriteChanges feeds the write payload to the node bridge via stdin
 func WriteChanges(root string, payload *WritePayload) error {
 	cmd := exec.Command("node", getBridgePath(root), "--write")
+	cmd.Dir = root
 	var stdout, stderr bytes.Buffer
 	cmd.Stdout = &stdout
 	cmd.Stderr = &stderr

go/main.go

@@ -20,16 +20,15 @@ func main() {
 	}
 
 	model := tui.NewModel(root)
-	p = tea.NewProgram(model, tea.WithAltScreen())
-	tui.Program = p
+	p = tea.NewProgram(&model, tea.WithAltScreen())
 
 	finalModel, err := p.Run()
 	if err != nil {
 		fmt.Printf("Alas, there's been an error: %v", err)
 		os.Exit(1)
 	}
 
-	if m, ok := finalModel.(tui.TuiModel); ok && m.ExitCode != 0 {
+	if m, ok := finalModel.(*tui.TuiModel); ok && m.ExitCode != 0 {
 		os.Exit(m.ExitCode)
 	}
 }