use value constructor, not match pattern, to build return value

looptrace/trace_metadata.py

@@ -165,7 +165,7 @@ def proc1(key: str, value: object) -> Result[TraceGroup, Errors]:
         def combine(state: Result[set[TraceGroup], Errors], new_result: Result[TraceGroup, Errors]) -> Result[set[TraceGroup], Errors]:
             match state, new_result:
                 case result.Result(tag="error", error=old_messages), result.Result(tag="error", error=new_messages):
-                    return result.Result(Seq.of_iterable(concat(old_messages, new_messages)))
+                    return Result(Seq.of_iterable(concat(old_messages, new_messages)))
                 case _, _:
                     return state.map2(new_result, lambda groups, new_group: groups + new_group)