treat more spans as llm spans

[dinmukhamedm]

openllmetry seems not to set gen_ai.system on OpenAI assistant spans, so our checks don't treat the span as an llm span. While we/traceloop fixes openllmetry loosen our checks a bit.

---

Important

Loosen span type checks in span_type() in spans.rs to classify spans as LLM if attributes start with gen_ai. or llm..

• Behavior:
  • Loosen span type checks in span_type() in spans.rs to classify spans as LLM if attributes start with gen_ai. or llm..
  • Addresses issue where gen_ai.system is not set on OpenAI assistant spans.

^{This description was created by for f6537ab. It will automatically update as commits are pushed.}

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to f6537ab in 11 seconds

More details

• Looked at 18 lines of code in 1 files
• Skipped 0 files when reviewing.
• Skipped posting 1 drafted comments based on config settings.

1. app-server/src/traces/spans.rs:144

• Draft comment:
  The logic for determining if a span is an LLM span has been updated to check for keys starting with 'gen_ai.' or 'llm.'. Ensure this change aligns with the intended behavior across the codebase.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The PR modifies the logic to determine if a span is an LLM span by checking for keys starting with 'gen_ai.' or 'llm.'. This change is in line with the PR description, which mentions loosening the checks to accommodate missing 'gen_ai.system' attributes.

Workflow ID: wflow_oWKJJyWhD2ZvzIBc

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.