Commit aa75ff323 for llama.cpp
commit aa75ff3239f9cc675b081697deda9e1b112345c6
Author: Ramzi <146260554+rmz-24@users.noreply.github.com>
Date: Tue Nov 10 23:30:14 2026 +0100
chat : fix Muse Spark ignoring response_format json_schema (#32017)
diff --git a/common/parsers/muse-spark.cpp b/common/parsers/muse-spark.cpp
index 577090aa5..945ee2aaf 100644
--- a/common/parsers/muse-spark.cpp
+++ b/common/parsers/muse-spark.cpp
@@ -46,9 +46,10 @@ common_chat_params common_chat_params_init_muse_spark(const common_chat_template
auto extract_reasoning = inputs.reasoning_format != COMMON_REASONING_FORMAT_NONE;
- auto has_tools = inputs.tools.is_array() && !inputs.tools.empty();
- // Constrained grammar whenever tools are offered.
- auto include_grammar = has_tools && inputs.tool_choice != COMMON_CHAT_TOOL_CHOICE_NONE;
+ auto has_tools = inputs.tools.is_array() && !inputs.tools.empty();
+ auto has_response_format = !inputs.json_schema.is_null() && inputs.json_schema.is_object();
+ // Constrained grammar whenever tools are offered or a response format is requested.
+ auto include_grammar = has_response_format || (has_tools && inputs.tool_choice != COMMON_CHAT_TOOL_CHOICE_NONE);
data.parser = build_chat_peg_parser([&](common_chat_peg_builder & p) {
auto start = p.rule("start", p.literal("<|start|>assistant"));
@@ -75,6 +76,15 @@ common_chat_params common_chat_params_init_muse_spark(const common_chat_template
auto final_msg = p.rule("final", recipient + p.literal("<|message|>") +
p.content(p.until_one_of({ "<|eot|>", "<|eom|>" })));
+ if (has_response_format) {
+ auto response_json = p.content(p.schema(p.json(), "response-format-schema", inputs.json_schema));
+ auto response_format = p.rule("response-format",
+ recipient + p.literal("<|message|>") +
+ ((p.literal("```json") + p.space() + response_json + p.space() + p.literal("```")) | response_json));
+
+ return p.zero_or_more(start + analysis) + start + response_format;
+ }
+
if (has_tools && inputs.tool_choice != COMMON_CHAT_TOOL_CHOICE_NONE) {
auto string_value = p.ac(
p.tool_arg_string_value(p.until("</atem:parameter>")) + p.tool_arg_close(p.literal("</atem:parameter>")),
@@ -132,7 +142,7 @@ common_chat_params common_chat_params_init_muse_spark(const common_chat_template
});
if (include_grammar) {
- data.grammar_lazy = inputs.tool_choice != COMMON_CHAT_TOOL_CHOICE_REQUIRED;
+ data.grammar_lazy = !(has_response_format || (has_tools && inputs.tool_choice == COMMON_CHAT_TOOL_CHOICE_REQUIRED));
data.grammar = build_grammar([&](const common_grammar_builder & builder) {
data.parser.build_grammar(builder, data.grammar_lazy);
});
diff --git a/tests/test-chat.cpp b/tests/test-chat.cpp
index a583604a7..b565775bf 100644
--- a/tests/test-chat.cpp
+++ b/tests/test-chat.cpp
@@ -7018,6 +7018,22 @@ static void test_template_output_peg_parsers(bool detailed_debug) {
.expect_reasoning("Thinking.")
.expect_content("Working on it.Hello!")
.run();
+
+ // Structured output, straight to the final answer
+ tst.test(" to=user<|message|>" R"({"amount": 123.45, "date": "2025-12-03"})")
+ .reasoning_format(COMMON_REASONING_FORMAT_AUTO)
+ .json_schema(invoice_schema)
+ .expect_content(R"({"amount": 123.45, "date": "2025-12-03"})")
+ .run();
+
+ // Structured output after a reasoning message: reasoning stays free-form
+ tst.test(" to=self<|message|>I need to output the invoice details in JSON<|eom|>"
+ "<|start|>assistant to=user<|message|>" R"({"amount": 123.45, "date": "2025-12-03"})")
+ .reasoning_format(COMMON_REASONING_FORMAT_AUTO)
+ .json_schema(invoice_schema)
+ .expect_reasoning("I need to output the invoice details in JSON")
+ .expect_content(R"({"amount": 123.45, "date": "2025-12-03"})")
+ .run();
}
// GPT-OSS format tests