fix: telemetry status shows enabled on/off and skips categories when stopped

Author: archandatta

cmd/browsers.go

@@ -799,6 +799,13 @@ func (b BrowsersCmd) TelemetryStatus(ctx context.Context, in BrowsersTelemetrySt
 	if in.Output == "json" {
 		return util.PrintPrettyJSON(browser.Telemetry)
 	}
+	telemetryEnabled := browser.Telemetry.JSON.Browser.Valid()
+	if telemetryEnabled {
+		pterm.Printf("enabled:     on\n")
+	} else {
+		pterm.Printf("enabled:     off\n")
+		return nil
+	}
 	cfg := browser.Telemetry.Browser
 	pterm.Printf("console:     %s\n", categoryOnOff(cfg.Console))
 	pterm.Printf("interaction: %s\n", categoryOnOff(cfg.Interaction))

cmd/browsers_test.go

@@ -1938,6 +1938,7 @@ func TestBrowsersTelemetryStatus_PrintsCategories(t *testing.T) {
 
 	assert.NoError(t, err)
 	out := outBuf.String()
+	assert.Contains(t, out, "enabled:     on")
 	assert.Contains(t, out, "console:     on")
 	assert.Contains(t, out, "interaction: off")
 	assert.Contains(t, out, "network:     on")
@@ -1960,12 +1961,34 @@ func TestBrowsersTelemetryStatus_VMDefaultsAllOn(t *testing.T) {
 
 	assert.NoError(t, err)
 	out := outBuf.String()
+	assert.Contains(t, out, "enabled:     on")
 	assert.Contains(t, out, "console:     on")
 	assert.Contains(t, out, "interaction: on")
 	assert.Contains(t, out, "network:     on")
 	assert.Contains(t, out, "page:        on")
 }
 
+func TestBrowsersTelemetryStatus_StoppedShowsDisabled(t *testing.T) {
+	setupStdoutCapture(t)
+	fake := &FakeBrowsersService{GetFunc: func(ctx context.Context, id string, query kernel.BrowserGetParams, opts ...option.RequestOption) (*kernel.BrowserGetResponse, error) {
+		// API returns {} (no browser key) after telemetry stop
+		var resp kernel.BrowserGetResponse
+		if err := json.Unmarshal([]byte(`{"session_id":"session123","telemetry":{}}`), &resp); err != nil {
+			t.Fatalf("unmarshal: %v", err)
+		}
+		return &resp, nil
+	}}
+	b := BrowsersCmd{browsers: fake}
+
+	err := b.TelemetryStatus(context.Background(), BrowsersTelemetryStatusInput{Identifier: "session123"})
+
+	assert.NoError(t, err)
+	out := outBuf.String()
+	assert.Contains(t, out, "enabled:     off")
+	assert.NotContains(t, out, "console:")
+	assert.NotContains(t, out, "network:")
+}
+
 func TestTelemetryStream_UnknownCategoryErrors(t *testing.T) {
 	b := BrowsersCmd{browsers: &FakeBrowsersService{}}