Drop strategy name from entire status output · Entire

Drop strategy name from entire status output

a6bba29→main

manual-commit is the only strategy, so naming it in the status lines carries no information for the user. Checkpoint metadata, commit trailers, and the explain --json strategy field stay untouched — they record provenance (imports write "import"), not display.

Co-Authored-By: Claude Fable 5 noreply@anthropic.com

Changes

3

208 unmodified lines

t.Error("Expected enabled to be true") }

// Verify status shows manual-commit (the only strategy) // Verify status shows the enabled state stdout = env.RunCLI("status") if !strings.Contains(stdout, "manual-commit") { t.Errorf("Expected status to show 'manual-commit', got: %s", stdout) } if !strings.Contains(stdout, "Enabled") { t.Errorf("Expected status to show 'Enabled', got: %s", stdout) } }

Mcmd/entire/cli/integration_test/setup_cmd_test.go+3/-3

17 unmodified lines

18 19 20 21 21 22 23 135 unmodified lines

159 160 161 163 162 163 165 166 164 165 166 6 unmodified lines

173 174 175 179 180 181 176 177 178 37 unmodified lines

216 217 218 225 219 220 227 228 221 222 223 4 unmodified lines

228 229 230 239 240 241 231 232 233

17 unmodified lines

"github.com/entireio/cli/cmd/entire/cli/paths" "github.com/entireio/cli/cmd/entire/cli/session" "github.com/entireio/cli/cmd/entire/cli/settings" "github.com/entireio/cli/cmd/entire/cli/strategy" "github.com/entireio/cli/cmd/entire/cli/stringutil" "github.com/entireio/cli/cmd/entire/cli/trailers"

135 unmodified lines

}

// formatSettingsStatusShort formats a short settings status line. // Output format: "● Enabled · manual-commit · branch main" or "○ Disabled" // Output format: "● Enabled · branch main" or "○ Disabled" func formatSettingsStatusShort(ctx context.Context, s *EntireSettings, sty statusStyles) string { displayName := strategy.StrategyNameManualCommit

var b strings.Builder

if s.Enabled { b.WriteString(sty.render(sty.bold, "Enabled")) } else { b.WriteString(sty.render(sty.bold, "Disabled")) }

b.WriteString(sty.render(sty.dim, " · ")) b.WriteString(displayName)

// Resolve branch from repo root if repoRoot, err := paths.WorktreeRoot(ctx); err == nil { if branch := resolveWorktreeBranch(ctx, repoRoot); branch != "" { b.WriteString(sty.render(sty.dim, " · branch " + branch)) } }

return b.String() }


Mcmd/entire/cli/status.go+2/-13

323 unmodified lines

func TestRunStatus_BothProjectAndLocal(t *testing.T) { setupTestRepo(t) // Project: enabled=true, strategy=manual-commit // Local: enabled=false, strategy=manual-commit // Detailed mode shows effective status first, then each file separately writeSettings(t, {"enabled": true}) writeLocalSettings(t, {"enabled": false})

output := stdout.String() // Should show effective status first (local overrides project) if !strings.Contains(output, "Disabled") || !strings.Contains(output, "manual-commit") { t.Errorf("Expected output to show effective 'Disabled' with 'manual-commit', got: %s", output) } // Should show both settings separately if !strings.Contains(output, "Project") || !strings.Contains(output, "manual-commit") { t.Errorf("Expected output to show Project with manual-commit, got: %s", output) } }

func TestRunStatus_ShowsManualCommitStrategy(t *testing.T) { setupTestRepo(t) writeSettings(t, {"enabled": false})

var stdout bytes.Buffer if err := runStatus(context.Background(), &stdout, true, false); err != nil { t.Fatalf("runStatus() error = %v", err) }

output := stdout.String() // Should show effective status first if !strings.Contains(output, "Disabled") || !strings.Contains(output, "manual-commit") { t.Errorf("Expected output to show effective 'Disabled' with 'manual-commit', got: %s", output) } }

630 unmodified lines