From ab67a38a4609728b1d828f6e0c42339c7bc33f99 Mon Sep 17 00:00:00 2001
From: breken-ai <312387581+breken-ai@users.noreply.github.com>
Date: Fri, 25 Sep 2026 21:18:23 -0700
Subject: [PATCH] utils: keep docstring continuation lines that contain a colon

_parse_docstring treated every line in the Args section that contains a
colon or a parenthesis as a new argument. An indented continuation line
such as a URL or "YYYY-MM-DD (for example: 2024-01-31)" therefore ended
the real argument's description early and created a phantom argument,
so the tool schema sent to the model lost that part of the description.

Treat lines indented deeper than the current argument as continuation
lines, which is how Google-style docstrings wrap descriptions.
---
 ollama/_utils.py    | 15 ++++++++++++---
 tests/test_utils.py | 17 +++++++++++++++++
 2 files changed, 29 insertions(+), 3 deletions(-)

diff --git a/ollama/_utils.py b/ollama/_utils.py
index 15f1cc0c..09ac59f0 100644
--- a/ollama/_utils.py
+++ b/ollama/_utils.py
@@ -16,6 +16,7 @@ def _parse_docstring(doc_string: Union[str, None]) -> dict[str, str]:
     return parsed_docstring
 
   key = str(hash(doc_string))
+  args_lines = []
   for line in doc_string.splitlines():
     lowered_line = line.lower().strip()
     if lowered_line.startswith('args:'):
@@ -26,11 +27,18 @@ def _parse_docstring(doc_string: Union[str, None]) -> dict[str, str]:
     else:
       # maybe change to a list and join later
       parsed_docstring[key] += f'{line.strip()}\n'
+      if key == 'args':
+        args_lines.append(line)
 
   last_key = None
-  for line in parsed_docstring['args'].splitlines():
-    line = line.strip()
-    if ':' in line:
+  last_indent = 0
+  for raw_line in args_lines:
+    line = raw_line.strip()
+    indent = len(raw_line) - len(raw_line.lstrip())
+    # A line indented deeper than the current argument continues its description,
+    # even if it contains a colon (a URL, "e.g.: ...", a time).
+    is_continuation = last_key is not None and indent > last_indent
+    if ':' in line and not is_continuation:
       # Split the line on either:
       # 1. A parenthetical expression like (integer) - captured in group 1
       # 2. A colon :
@@ -39,6 +47,7 @@ def _parse_docstring(doc_string: Union[str, None]) -> dict[str, str]:
 
       arg_name = parts[0].strip()
       last_key = arg_name
+      last_indent = indent
 
       # Get the description - will be in parts[1] if parenthetical or parts[-1] if after colon
       arg_description = parts[-1].strip()
diff --git a/tests/test_utils.py b/tests/test_utils.py
index cb9e0d4f..52dda6a4 100644
--- a/tests/test_utils.py
+++ b/tests/test_utils.py
@@ -256,3 +256,20 @@ def func_with_parentheses_and_args(a: int, b: int):
   tool = convert_function_to_tool(func_with_parentheses_and_args).model_dump()
   assert tool['function']['parameters']['properties']['a']['description'] == 'First (:thing) number to add'
   assert tool['function']['parameters']['properties']['b']['description'] == 'Second number to add'
+
+
+def test_docstring_continuation_lines_with_colons():
+  def get_page(url: str, date: str) -> str:
+    """
+    Fetch a page.
+
+    Args:
+        url: Page to fetch, for example
+            https://example.com/docs
+        date: Day to fetch, in the format
+            YYYY-MM-DD (for example: 2024-01-31)
+    """
+
+  tool = convert_function_to_tool(get_page).model_dump()
+  assert tool['function']['parameters']['properties']['url']['description'] == 'Page to fetch, for example https://example.com/docs'
+  assert tool['function']['parameters']['properties']['date']['description'] == 'Day to fetch, in the format YYYY-MM-DD (for example: 2024-01-31)'