From cd8ab8e55ae148fe0b57d81da981602ab6bc30c6 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E6=9D=8E=E9=80=9A=E6=B4=B2?= <zhangsongcui3371@sina.com>
Date: Mon, 17 Nov 2025 11:01:32 +0800
Subject: [PATCH] Command: Adds splitLines option

---
 doc/json_schema.json          |  5 ++++
 src/modules/command/command.c | 54 ++++++++++++++++++++++++++++++-----
 src/modules/command/option.h  |  1 +
 3 files changed, 53 insertions(+), 7 deletions(-)

diff --git a/doc/json_schema.json b/doc/json_schema.json
index 7e64a7cf5..0bdf56bab 100644
--- a/doc/json_schema.json
+++ b/doc/json_schema.json
@@ -2111,6 +2111,11 @@
                                         "type": "boolean",
                                         "default": true
                                     },
+                                    "splitLines": {
+                                        "description": "Set if the command output should be split into multiple lines",
+                                        "type": "boolean",
+                                        "default": false
+                                    },
                                     "key": {
                                         "$ref": "#/$defs/key"
                                     },
diff --git a/src/modules/command/command.c b/src/modules/command/command.c
index 59f6b4951..84ecba557 100644
--- a/src/modules/command/command.c
+++ b/src/modules/command/command.c
@@ -20,16 +20,39 @@ bool ffPrintCommand(FFCommandOptions* options)
         return false;
     }
 
-    if (options->moduleArgs.outputFormat.length == 0)
+    if (options->splitLines)
     {
-        ffPrintLogoAndKey(FF_COMMAND_MODULE_NAME, 0, &options->moduleArgs, FF_PRINT_TYPE_DEFAULT);
-        ffStrbufPutTo(&result, stdout);
+        uint8_t index = 0;
+        char* line = NULL;
+        size_t len = 0;
+        while (ffStrbufGetline(&line, &len, &result))
+        {
+            if (options->moduleArgs.outputFormat.length == 0)
+            {
+                ffPrintLogoAndKey(FF_COMMAND_MODULE_NAME, ++index, &options->moduleArgs, FF_PRINT_TYPE_DEFAULT);
+                puts(line);
+            }
+            else
+            {
+                FF_PRINT_FORMAT_CHECKED(FF_COMMAND_MODULE_NAME, ++index, &options->moduleArgs, FF_PRINT_TYPE_DEFAULT, ((FFformatarg[]){
+                    FF_FORMAT_ARG(line, "result")
+                }));
+            }
+        }
     }
     else
     {
-        FF_PRINT_FORMAT_CHECKED(FF_COMMAND_MODULE_NAME, 0, &options->moduleArgs, FF_PRINT_TYPE_DEFAULT, ((FFformatarg[]){
-            FF_FORMAT_ARG(result, "result")
-        }));
+        if (options->moduleArgs.outputFormat.length == 0)
+        {
+            ffPrintLogoAndKey(FF_COMMAND_MODULE_NAME, 0, &options->moduleArgs, FF_PRINT_TYPE_DEFAULT);
+            ffStrbufPutTo(&result, stdout);
+        }
+        else
+        {
+            FF_PRINT_FORMAT_CHECKED(FF_COMMAND_MODULE_NAME, 0, &options->moduleArgs, FF_PRINT_TYPE_DEFAULT, ((FFformatarg[]){
+                FF_FORMAT_ARG(result, "result")
+            }));
+        }
     }
 
     return true;
@@ -74,6 +97,12 @@ void ffParseCommandJsonObject(FFCommandOptions* options, yyjson_val* module)
             continue;
         }
 
+        if (unsafe_yyjson_equals_str(key, "splitLines"))
+        {
+            options->splitLines = yyjson_get_bool(val);
+            continue;
+        }
+
         ffPrintError(FF_COMMAND_MODULE_NAME, 0, &options->moduleArgs, FF_PRINT_TYPE_DEFAULT, "Unknown JSON key %s", unsafe_yyjson_get_str(key));
     }
 }
@@ -87,6 +116,7 @@ void ffGenerateCommandJsonConfig(FFCommandOptions* options, yyjson_mut_doc* doc,
     yyjson_mut_obj_add_strbuf(doc, module, "text", &options->text);
     yyjson_mut_obj_add_bool(doc, module, "useStdErr", options->useStdErr);
     yyjson_mut_obj_add_bool(doc, module, "parallel", options->parallel);
+    yyjson_mut_obj_add_bool(doc, module, "splitLines", options->splitLines);
 }
 
 bool ffGenerateCommandJsonResult(FF_MAYBE_UNUSED FFCommandOptions* options, yyjson_mut_doc* doc, yyjson_mut_val* module)
@@ -106,7 +136,16 @@ bool ffGenerateCommandJsonResult(FF_MAYBE_UNUSED FFCommandOptions* options, yyjs
         return false;
     }
 
-    yyjson_mut_obj_add_strbuf(doc, module, "result", &result);
+    if (options->splitLines)
+    {
+        yyjson_mut_val* jsonArray = yyjson_mut_obj_add_arr(doc, module, "result");
+        char* line = NULL;
+        size_t len = 0;
+        while (ffStrbufGetline(&line, &len, &result))
+            yyjson_mut_arr_add_strncpy(doc, jsonArray, line, len);
+    }
+    else
+        yyjson_mut_obj_add_strbuf(doc, module, "result", &result);
 
     return true;
 }
@@ -132,6 +171,7 @@ void ffInitCommandOptions(FFCommandOptions* options)
     ffStrbufInit(&options->text);
     options->useStdErr = false;
     options->parallel = true;
+    options->splitLines = false;
 }
 
 void ffDestroyCommandOptions(FFCommandOptions* options)
diff --git a/src/modules/command/option.h b/src/modules/command/option.h
index e85a65f15..06f3b05b3 100644
--- a/src/modules/command/option.h
+++ b/src/modules/command/option.h
@@ -11,6 +11,7 @@ typedef struct FFCommandOptions
     FFstrbuf text;
     bool useStdErr;
     bool parallel;
+    bool splitLines;
 } FFCommandOptions;
 
 static_assert(sizeof(FFCommandOptions) <= FF_OPTION_MAX_SIZE, "FFCommandOptions size exceeds maximum allowed size");