From 6415d0f03f45f2d9438bdb6702668c89e9bbadd9 Mon Sep 17 00:00:00 2001
From: Piotr Wilkin <piotr.wilkin@syndatis.com>
Date: Fri, 13 Feb 2026 14:42:26 +0100
Subject: [PATCH] Add TODO

---
 common/chat-auto-parser-helpers.cpp | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/common/chat-auto-parser-helpers.cpp b/common/chat-auto-parser-helpers.cpp
index d03845d861..0f40d9e813 100644
--- a/common/chat-auto-parser-helpers.cpp
+++ b/common/chat-auto-parser-helpers.cpp
@@ -244,6 +244,9 @@ std::string after_common_suffix(const std::string & full, const std::string & le
     return full.substr(pos + common_suffix_len);
 }
 
+// TODO: segmentize will treat a JSON array inside tags as a tag: <calls>[{ "fun": { ... } }]</calls> will be three markers
+// not too worried about that because it hasn't turned out as a problem anywhere, but noting here in case it will
+// Might have to put some restrictions on tag contents as well (like "no { }")
 std::vector<segment> segmentize_markers(const std::string & text) {
     std::vector<segment> retval;
     bool in_marker = false;