Restore defaults after rig manifest rebase

Author: easel

internal/config/types.go

@@ -597,6 +597,11 @@ func defaultProcessNames(provider, command string) []string {
 	if provider == "claude" {
 		return []string{"node"}
 	}
+	if provider == "opencode" {
+		// OpenCode runs as Node.js process, need both for IsAgentRunning detection.
+		// tmux pane_current_command may show "node" or "opencode" depending on how invoked.
+		return []string{"opencode", "node"}
+	}
 	if command != "" {
 		return []string{filepath.Base(command)}
 	}
@@ -605,7 +610,8 @@ func defaultProcessNames(provider, command string) []string {
 
 func defaultReadyPromptPrefix(provider string) string {
 	if provider == "claude" {
-		return "> "
+		// Claude Code uses ❯ (U+276F) as the prompt character
+		return "❯ "
 	}
 	return ""
 }
@@ -617,6 +623,12 @@ func defaultReadyDelayMs(provider string) int {
 	if provider == "codex" {
 		return 3000
 	}
+	if provider == "opencode" {
+		// OpenCode requires delay-based detection because its TUI uses
+		// box-drawing characters (┃) that break prompt prefix matching.
+		// 8000ms provides reliable startup detection across models.
+		return 8000
+	}
 	return 0
 }
 
@@ -632,9 +644,15 @@ func defaultInstructionsFile(provider string) string {
 
 // quoteForShell quotes a string for safe shell usage.
 func quoteForShell(s string) string {
-	// Simple quoting: wrap in double quotes, escape internal quotes
+	// Wrap in double quotes, escaping characters that are special in double-quoted strings:
+	// - backslash (escape character)
+	// - double quote (string delimiter)
+	// - backtick (command substitution)
+	// - dollar sign (variable expansion)
 	escaped := strings.ReplaceAll(s, `\`, `\\`)
 	escaped = strings.ReplaceAll(escaped, `"`, `\"`)
+	escaped = strings.ReplaceAll(escaped, "`", "\\`")
+	escaped = strings.ReplaceAll(escaped, "$", `\$`)
 	return `"` + escaped + `"`
 }
 

internal/crew/manager.go

@@ -378,16 +378,13 @@ func (m *Manager) loadState(name string) (*CrewWorker, error) {
 		return nil, fmt.Errorf("parsing state: %w", err)
 	}
 
-	// Backfill essential fields if missing (handles empty or incomplete state.json)
-	if crew.Name == "" {
-		crew.Name = name
-	}
+	// Directory name is source of truth for Name and ClonePath.
+	// state.json can become stale after directory rename, copy, or corruption.
+	crew.Name = name
 	if crew.Rig == "" {
 		crew.Rig = m.rig.Name
 	}
-	if crew.ClonePath == "" {
-		crew.ClonePath = m.crewDir(name)
-	}
+	crew.ClonePath = m.crewDir(name)
 
 	return &crew, nil
 }

internal/git/git.go

@@ -482,11 +482,6 @@ func (g *Git) RemoteDefaultBranch() string {
 	return "main" // final fallback
 }
 
-// ConfigGet gets a git config value.
-func (g *Git) ConfigGet(key string) (string, error) {
-	return g.run("config", "--get", key)
-}
-
 // HasUncommittedChanges returns true if there are uncommitted changes.
 func (g *Git) HasUncommittedChanges() (bool, error) {
 	status, err := g.Status()
@@ -513,6 +508,17 @@ func (g *Git) Remotes() ([]string, error) {
 	return strings.Split(out, "\n"), nil
 }
 
+// ConfigGet returns the value of a git config key.
+// Returns empty string if the key is not set.
+func (g *Git) ConfigGet(key string) (string, error) {
+	out, err := g.run("config", "--get", key)
+	if err != nil {
+		// git config --get returns exit code 1 if key not found
+		return "", nil
+	}
+	return out, nil
+}
+
 // SetRemoteURL sets an existing remote URL or adds it if missing.
 func (g *Git) SetRemoteURL(remote, url string) error {
 	if url == "" {