feat(dream-cli): --json flag on list/status and document doctor --json (#1000)

* feat(dream-cli): --json flag on list/status and document doctor --json

'dream list --json' and 'dream status --json' previously were silently
dropped as unknown args — scripts piping to jq got parse errors. The
main case dispatch forwarded no arguments to cmd_list / cmd_status,
so any flag passed was ignored. Add explicit --json parsing:

- cmd_list --json emits a JSON array of {id, category, status}.
- cmd_status --json delegates to the existing cmd_status_json.
- Unknown flags on these subcommands now error out cleanly (exit 1).
- cmd_help mentions 'doctor --json' which was previously undocumented,
  and advertises the new 'list --json' / 'status --json' flags.

'dream status-json' remains as a hyphenated alias for one release.

Platform impact: identical on macOS / Linux / Windows (WSL2) — no
platform branching.

* fix(dream-cli): escape list json strings

---------

Co-authored-by: Lightheartdevs <Lightheartdevs@users.noreply.github.com>

Author: yasinBursali

dream-server/dream-cli

@@ -75,6 +75,18 @@ _status_color() {
     esac
 }
 
+_json_escape() {
+    local s="${1-}"
+    s=${s//\\/\\\\}
+    s=${s//\"/\\\"}
+    s=${s//$'\b'/\\b}
+    s=${s//$'\f'/\\f}
+    s=${s//$'\n'/\\n}
+    s=${s//$'\r'/\\r}
+    s=${s//$'\t'/\\t}
+    printf '%s' "$s"
+}
+
 # Print a separator of repeated characters filling the given width.
 # Usage: hr <width> [<char=─>]
 hr() {
@@ -658,6 +670,21 @@ _dream_cli_rebuild_images() {
 #=============================================================================
 
 cmd_status() {
+    local json_mode="false"
+    while [[ $# -gt 0 ]]; do
+        case "$1" in
+            --json) json_mode="true"; shift ;;
+            *) error "Unknown argument to 'dream status': $1" ;;
+        esac
+    done
+
+    if [[ "$json_mode" == "true" ]]; then
+        # Subshell isolates cmd_status_json's RETURN trap so it can't leak
+        # into this caller's frame and re-fire with an unbound $tmp under set -u.
+        ( cmd_status_json )
+        return $?
+    fi
+
     check_install
     cd "$INSTALL_DIR"
     load_env
@@ -1887,10 +1914,45 @@ cmd_purge() {
 }
 
 cmd_list() {
+    local json_mode="false"
+    while [[ $# -gt 0 ]]; do
+        case "$1" in
+            --json) json_mode="true"; shift ;;
+            *) error "Unknown argument to 'dream list': $1" ;;
+        esac
+    done
+
     sr_load
     load_env 2>/dev/null || true
     local active_flags
     active_flags=$(get_compose_flags)
+
+    if [[ "$json_mode" == "true" ]]; then
+        # JSON output: array of {id, category, status}
+        local _first=1
+        printf '['
+        for sid in "${SERVICE_IDS[@]}"; do
+            local cat="${SERVICE_CATEGORIES[$sid]}"
+            local cf="${SERVICE_COMPOSE[$sid]}"
+            local status
+            if [[ "$cat" == "core" ]]; then
+                status="always-on"
+            elif [[ -n "$cf" && -f "$cf" && "$active_flags" == *"${cf#"$INSTALL_DIR/"}"* ]]; then
+                status="enabled"
+            else
+                status="disabled"
+            fi
+            (( _first == 1 )) || printf ','
+            printf '{"id":"%s","category":"%s","status":"%s"}' \
+                "$(_json_escape "$sid")" \
+                "$(_json_escape "$cat")" \
+                "$(_json_escape "$status")"
+            _first=0
+        done
+        printf ']\n'
+        return 0
+    fi
+
     echo -e "${BLUE}━━━ Available Services ━━━${NC}"
     printf "%-20s %-12s %-10s\n" "SERVICE" "CATEGORY" "STATUS"
     printf "%-20s %-12s %-10s\n" "$(hr 20)" "$(hr 12)" "$(hr 10)"
@@ -3561,9 +3623,9 @@ Usage: dream <command> [options]
 ${CYAN}Commands:${NC}
   gpu [status|topology|assignment|validate|reassign]
                       Inspect and manage multi-GPU configuration
-  status              Show service health and GPU status
-  status-json         Machine-readable status (JSON) with mode/tier/model
-  list                List all services and their status
+  status [--json]     Show service health and GPU status (--json = machine-readable)
+  status-json         Alias for 'status --json' (kept for back-compat)
+  list [--json]       List all services and their status (--json = machine-readable)
   enable <service>    Enable an extension service
   disable <service>   Disable an extension service
   purge <service>     Permanently delete service data
@@ -3592,7 +3654,7 @@ ${CYAN}Commands:${NC}
                       View, edit, or validate configuration
   chat "<message>"    Quick chat with the LLM
   benchmark           Run a quick performance test
-  doctor [report]     Run diagnostics and write JSON report
+  doctor [report|--json]   Run diagnostics (--json writes JSON to stdout)
   repair|fix          Run basic repairs (currently redirects to doctor)
   template [action]   Apply pre-built service templates (list|preview|apply)
   audit [extensions]  Audit extension manifests and compose contracts
@@ -3690,9 +3752,9 @@ EOF
 #=============================================================================
 case "${1:-help}" in
     gpu|g)       shift; cmd_gpu "$@" ;;
-    status|s)    cmd_status ;;
+    status|s)    shift; cmd_status "$@" ;;
     status-json) cmd_status_json ;;
-    list|ls)     cmd_list ;;
+    list|ls)     shift; cmd_list "$@" ;;
     enable)      shift; cmd_enable "$@" ;;
     disable)     shift; cmd_disable "$@" ;;
     purge)       shift; cmd_purge "$@" ;;

dream-server/tests/test-dream-list-json-escaping.sh

@@ -0,0 +1,48 @@
+#!/usr/bin/env bash
+# Regression: dream list --json must escape registry strings from user extensions.
+
+set -euo pipefail
+
+SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
+PROJECT_DIR="$(dirname "$SCRIPT_DIR")"
+USER_EXT_DIR="$PROJECT_DIR/data/user-extensions/json-escape-test"
+
+cleanup() {
+    rm -rf "$USER_EXT_DIR"
+}
+trap cleanup EXIT
+
+if ! command -v python3 >/dev/null 2>&1; then
+    echo "[SKIP] python3 not available"
+    exit 0
+fi
+
+if ! python3 -c 'import yaml' >/dev/null 2>&1; then
+    echo "[SKIP] PyYAML not available"
+    exit 0
+fi
+
+mkdir -p "$USER_EXT_DIR"
+cat > "$USER_EXT_DIR/manifest.yaml" <<'YAML'
+schema_version: dream.services.v1
+service:
+  id: json-escape-test
+  name: JSON Escape Test
+  port: 65535
+  health: /health
+  category: 'optional "quoted" \ slash'
+YAML
+
+output=$(DREAM_HOME="$PROJECT_DIR" NO_COLOR=1 "$PROJECT_DIR/dream-cli" list --json)
+
+python3 - "$output" <<'PY'
+import json
+import sys
+
+payload = json.loads(sys.argv[1])
+service = next(item for item in payload if item["id"] == "json-escape-test")
+assert service["category"] == 'optional "quoted" \\ slash'
+assert service["status"] == "disabled"
+PY
+
+echo "[PASS] dream list --json escapes user-extension strings"