refactor(client): remove two unused OpenLLMetry span helpers

TELEMETRY-CONTRACT.md

@@ -654,19 +654,14 @@ Nothing else in `graph.py` or `native_graph.py` needs work.
 
 ## 11. Superseded, do not carry forward
 
-`set_openllmetry_prompt` and `set_openllmetry_completion` in `utils.py` are the shape this SDK
-emitted before the span work. In TypeScript they still exist and nothing calls them. The content
-layer replaced them, and it handles any number of messages rather than only index 0.
-
-Every Python handler calls them today, verified in all six. That is the clearest single statement
-of the gap: Python is built on the layer TypeScript retired.
-
-Move every call site onto the content helpers. Keep the two functions exported for one release,
-then remove them.
-
-Neither function has a test of its own in the client package today, and neither does
-`set_ld_span_attributes`. The only coverage they have is indirect, through the handler tests. Give
-their replacements direct tests.
+`set_openllmetry_prompt` and `set_openllmetry_completion` were the shape this SDK emitted before
+the span work: one flat span per call, with the prompt and only completion index 0 written as
+OpenLLMetry attributes. The content layer replaced them, and it handles any number of messages.
+Every handler now writes content through `content.py`, and both functions have been removed, from
+this SDK and from TypeScript.
+
+Do not bring them back. Content goes through the content helpers, and the token aliases go through
+`set_usage_span_attributes`, so each attribute has one writer.
 
 ---
 

packages/client/src/launchdarkly_ai_server/__init__.py

@@ -118,8 +118,6 @@
     parse_usage,
     set_ld_span_attributes,
     set_model_identity_attributes,
-    set_openllmetry_completion,
-    set_openllmetry_prompt,
     set_usage_span_attributes,
     to_ld_context,
     to_usage_dict,
@@ -204,8 +202,6 @@
     "parse_template",
     "parse_usage",
     "set_ld_span_attributes",
-    "set_openllmetry_completion",
-    "set_openllmetry_prompt",
     "to_ld_context",
     # history
     "compose_history",

packages/client/src/launchdarkly_ai_server/utils.py

@@ -737,33 +737,6 @@ def set_ld_span_attributes(span: Any, variables: dict[str, Any] | None) -> None:
     span.add_event("feature_flag", feature_flag_attrs)
 
 
-def set_openllmetry_prompt(span: Any, messages: list[dict[str, str]]) -> None:
-    """Set OpenLLMetry-style indexed prompt attributes on a span.
-
-    Gonfalon's LLM Summary tab reads ``gen_ai.prompt.N.role`` / ``.content``
-    (attribute-based, takes precedence over span events).
-    """
-    for i, msg in enumerate(messages):
-        span.set_attribute(f"gen_ai.prompt.{i}.role", msg["role"])
-        span.set_attribute(f"gen_ai.prompt.{i}.content", msg["content"])
-
-
-def set_openllmetry_completion(
-    span: Any,
-    completion: str,
-    usage: dict[str, int],
-) -> None:
-    """Set OpenLLMetry-style indexed completion attributes and token usage aliases.
-
-    Gonfalon reads ``gen_ai.completion.0.role`` / ``.content`` and prefers
-    ``gen_ai.usage.prompt_tokens`` / ``completion_tokens``.
-    """
-    span.set_attribute("gen_ai.completion.0.role", "assistant")
-    span.set_attribute("gen_ai.completion.0.content", completion)
-    span.set_attribute("gen_ai.usage.prompt_tokens", usage.get("input_tokens", 0))
-    span.set_attribute("gen_ai.usage.completion_tokens", usage.get("output_tokens", 0))
-
-
 def parse_json_with_possible_fences(raw_text: str) -> Any | None:
     """
     Parses a JSON string that may be wrapped in markdown code fences

tests/test_cross_handler_parity.py

@@ -386,32 +386,6 @@ def test_the_langchain_provider_name_is_the_configured_name(
     "ld.ai.graph.path",
 }
 
-#: Functions kept exported for one release that nothing calls any more.
-#:
-#: Their bodies are cut out before the scan below, rather than their keys being listed as expected.
-#: Listing the keys does not work: `gen_ai.prompt` is written by the live content writer *and* by
-#: dead `set_openllmetry_prompt`, so naming it as expected lets the dead copy satisfy the lock after
-#: the live one is removed. Cutting the dead code out means only live writes can satisfy anything.
-#:
-#: Delete these names when the functions go. The lock will tell you if you miss one.
-SUPERSEDED_FUNCTIONS = (
-    "set_openllmetry_prompt",
-    "set_openllmetry_completion",
-)
-
-
-def _without_superseded(source: str) -> str:
-    """Drops the body of every superseded function, so a dead write cannot satisfy the lock."""
-    for name in SUPERSEDED_FUNCTIONS:
-        start = source.find(f"def {name}(")
-        if start == -1:
-            continue
-        nxt = source.find("\ndef ", start + 1)
-        end = len(source) if nxt == -1 else nxt
-        source = source[:start] + source[end:]
-    return source
-
-
 _KEY_PATTERN = re.compile(
     r'set_attribute\(\s*f?"([^"{]+)"'
     r'|add_event\(\s*"([^"]+)"'
@@ -431,7 +405,7 @@ def _emitted_vocabulary() -> set[str]:
     """Every attribute key, event name and template found in the package sources."""
     found: set[str] = set()
     for path in (REPO_ROOT / "packages").glob("*/src/*/*.py"):
-        for match in _KEY_PATTERN.finditer(_without_superseded(path.read_text())):
+        for match in _KEY_PATTERN.finditer(path.read_text()):
             key = next((g for g in match.groups() if g), None)
             if key and key.split(".")[0] in (
                 "gen_ai",