From 4cc60303f29bbb7a427a3cfa8754e5222f1d2a02 Mon Sep 17 00:00:00 2001
From: Massimiliano Galli <massimiliano.galli@cern.ch>
Date: Mon, 31 Mar 2025 15:07:00 +0200
Subject: [PATCH] [RF] Fix implementation of HS3 importers

Fix by @guitargeek.
The RooWSFactoryTool expression handler was not correctly matching branckets, leading to failures when importing
valid JSON filed with RooProdPdf objects.
---
 roofit/hs3/src/RooJSONFactoryWSTool.cxx | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/roofit/hs3/src/RooJSONFactoryWSTool.cxx b/roofit/hs3/src/RooJSONFactoryWSTool.cxx
index 168c4fc741e86..ac70193e240f8 100644
--- a/roofit/hs3/src/RooJSONFactoryWSTool.cxx
+++ b/roofit/hs3/src/RooJSONFactoryWSTool.cxx
@@ -337,8 +337,9 @@ std::string generate(const RooFit::JSONIO::ImportExpression &ex, const JSONNode
          RooJSONFactoryWSTool::error(errMsg.str());
       } else if (p[k].is_seq()) {
          bool firstInner = true;
+         expression << "{";
          for (RooAbsArg *arg : tool->requestArgList<RooAbsReal>(p, k)) {
-            expression << (firstInner ? "{" : ",") << arg->GetName();
+            expression << (firstInner ? "" : ",") << arg->GetName();
             firstInner = false;
          }
          expression << "}";