fix(cli): bound incident summary detail

Author: ysyneu

internal/cli/fieldproject_test.go

@@ -297,7 +297,8 @@ func TestIncidentDetailFieldsProjection(t *testing.T) {
 	row["images"] = []map[string]any{{"src": strings.Repeat("https://example.test/image/", 100)}}
 	stub.data = row
 
-	out, err := execCommand("incident", "detail", "inc-1", "--fields", "incident_id,title,root_cause", "--output-format", "json")
+	fields := []string{"incident_id", "title", "incident_severity", "progress", "ai_summary", "root_cause", "resolution", "alert_cnt", "start_time", "channel_id"}
+	out, err := execCommand("incident", "detail", "inc-1", "--fields", strings.Join(fields, ","), "--output-format", "json")
 	if err != nil {
 		t.Fatalf("execCommand: %v", err)
 	}
@@ -308,8 +309,13 @@ func TestIncidentDetailFieldsProjection(t *testing.T) {
 	if err := json.Unmarshal([]byte(strings.TrimSpace(out)), &detail); err != nil {
 		t.Fatalf("parse projected detail json: %v\n%s", err, out)
 	}
-	if len(detail) != 3 || detail["incident_id"] == nil || detail["title"] == nil || detail["root_cause"] == nil {
-		t.Fatalf("projected detail = %v, want exactly incident_id,title,root_cause", detail)
+	if len(detail) != len(fields) {
+		t.Fatalf("projected detail keys = %v, want exactly %v", detail, fields)
+	}
+	for _, field := range fields {
+		if detail[field] == nil {
+			t.Errorf("projected detail missing %q: %v", field, detail)
+		}
 	}
 	if _, ok := detail["description"]; ok {
 		t.Errorf("projected detail includes description: %v", detail)

internal/cli/incident_summary_script_test.go

@@ -39,7 +39,11 @@ func TestIncidentSummaryScriptCompactOutput(t *testing.T) {
 	if len(lines) != 6 {
 		t.Fatalf("fduty calls = %d, want 6:\n%s", len(lines), invocations)
 	}
-	if strings.Contains(string(invocations), "--output-format toon") {
-		t.Fatalf("summary forces toon instead of each command's compact default:\n%s", invocations)
+	wantDetail := "incident detail inc-1 --fields incident_id,title,incident_severity,progress,ai_summary,root_cause,resolution,alert_cnt,start_time,channel_id --output-format toon"
+	if lines[0] != wantDetail {
+		t.Fatalf("detail call = %q, want compact projection %q", lines[0], wantDetail)
+	}
+	if strings.Contains(strings.Join(lines[1:], "\n"), "--output-format toon") {
+		t.Fatalf("non-detail reads force raw toon instead of their compact defaults:\n%s", invocations)
 	}
 }

internal/skilldoc/incident_projection_guidance_test.go

@@ -16,8 +16,8 @@ func TestIncidentCardAvoidsUnboundedStructuredHotFlows(t *testing.T) {
 	body := string(card)
 	for description, command := range map[string]string{
 		"triage list":    "incident list --severity Critical --progress Triggered --since 4h --fields incident_id,title,incident_severity,progress,start_time,channel_id --output-format toon",
-		"triage detail":  "incident detail <incident-id> --fields incident_id,title,incident_severity,progress,ai_summary,root_cause,resolution,alert_cnt,start_time --output-format toon",
-		"summary detail": `incident detail   "$ID" --fields incident_id,title,incident_severity,progress,ai_summary,root_cause,resolution,alert_cnt,start_time --output-format toon`,
+		"triage detail":  "incident detail <incident-id> --fields incident_id,title,incident_severity,progress,ai_summary,root_cause,resolution,alert_cnt,start_time,channel_id --output-format toon",
+		"summary detail": `incident detail   "$ID" --fields incident_id,title,incident_severity,progress,ai_summary,root_cause,resolution,alert_cnt,start_time,channel_id --output-format toon`,
 	} {
 		if !strings.Contains(body, command) {
 			t.Errorf("incident %s must project structured output to the fields needed by the workflow", description)

skills/flashduty/reference/incident.md

@@ -45,7 +45,7 @@ Prereq: `SKILL.md` read. Read verbs are free. **Mutating verbs notify responders
 fduty incident list --severity Critical --progress Triggered --since 4h --fields incident_id,title,incident_severity,progress,start_time,channel_id --output-format toon
 
 # 2. Get AI summary + full detail (use the 24-char incident_id from step 1)
-fduty incident detail <incident-id> --fields incident_id,title,incident_severity,progress,ai_summary,root_cause,resolution,alert_cnt,start_time --output-format toon
+fduty incident detail <incident-id> --fields incident_id,title,incident_severity,progress,ai_summary,root_cause,resolution,alert_cnt,start_time,channel_id --output-format toon
 
 # 3. See contributing alerts
 fduty incident alerts <incident-id>
@@ -77,7 +77,7 @@ If you fetch the pieces by hand instead, run **all six** — they are cheap read
 
 ```bash
 ID=<incident-id>                                          # 24-char id from `incident list`
-fduty incident detail   "$ID" --fields incident_id,title,incident_severity,progress,ai_summary,root_cause,resolution,alert_cnt,start_time --output-format toon  # ① 详情 + AI summary + alert counts + channel
+fduty incident detail   "$ID" --fields incident_id,title,incident_severity,progress,ai_summary,root_cause,resolution,alert_cnt,start_time,channel_id --output-format toon  # ① 详情 + AI summary + alert counts + channel
 fduty incident alerts   "$ID"                             # ② contributing alerts (detail's embedded alerts are empty here)
 fduty incident timeline "$ID"                             # ④ timeline  (or `incident feed "$ID"` for the paginated view)
 fduty incident similar  "$ID" --limit 5 --output-format toon          # ⑤ similar past incidents (channel-backed; see Gotchas; compact by default)

skills/flashduty/scripts/incident-summary.sh

@@ -8,8 +8,8 @@
 #   usage: bash incident-summary.sh <incident-id>
 #
 # Section ⑥ lists recent post-mortems account-wide. To scope them to THIS incident's
-# channel, read its channel_id (fduty incident info --incident-id <id> --output-format
-# toon | grep '^channel_id:') and re-run: fduty incident post-mortem-list --channel-ids <id>
+# channel, read channel_id from the projected detail section and re-run:
+# fduty incident post-mortem-list --channel-ids <id>
 #
 # Note: errexit (-e) is intentionally NOT set — every section must run even if one
 # command fails, so the summary stays as complete as possible. Each command's own
@@ -22,14 +22,12 @@ if [ -z "$ID" ]; then
   exit 2
 fi
 
-# Print each command's DEFAULT renderer (a curated table/summary that projects the
-# summary-relevant fields), NOT --output-format toon: toon dumps the full raw objects
-# — every empty field plus heavy blobs like a change's labels.steps — which overflowed
-# the output cap and forced repeated paging. For these read verbs the lean default IS
-# the field projection a fault summary needs (id/severity/status/title/channel/times/…).
+# Project detail explicitly because its default table includes unbounded narrative
+# fields. The other read verbs use their compact default renderers; raw toon dumps
+# every empty field plus heavy blobs like a change's labels.steps.
 run() { echo "===== fduty $* ====="; fduty "$@" 2>&1; echo; }
 
-run incident detail        "$ID"              # ① 详情 + AI summary + alert counts + channel
+run incident detail        "$ID" --fields incident_id,title,incident_severity,progress,ai_summary,root_cause,resolution,alert_cnt,start_time,channel_id --output-format toon # ① 详情 + AI summary + alert counts + channel
 run incident alerts        "$ID"              # ② contributing alerts
 run incident timeline      "$ID"              # ④ timeline
 run incident similar       "$ID" --limit 5    # ⑤ similar past incidents (channel-backed)