Skip to content

chore: add structured span elements for AI observability - #1208

Draft
nboyse wants to merge 8 commits into
devfrom
chore/add-proper-span-for-ai-observability
Draft

chore: add structured span elements for AI observability#1208
nboyse wants to merge 8 commits into
devfrom
chore/add-proper-span-for-ai-observability

Conversation

@nboyse

@nboyse nboyse commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Signed-off-by: DBT pre-commit check

Context

What

Have you written unit tests?

  • Yes
  • No (add why you have not)

Are there any specific instructions on how to test this change?

  • Yes (if so provide more detail)
  • No

Relevant links

@github-actions

Copy link
Copy Markdown
Contributor

Dependency Review

The following issues were found:
  • ✅ 0 vulnerable package(s)
See the Details below.

OpenSSF Scorecard

PackageVersionScoreDetails
pip/ddtrace 4.11.0 UnknownUnknown
pip/ddtrace 4.11.0 UnknownUnknown
pip/ddtrace 4.11.0 UnknownUnknown
pip/ddtrace ^ 4.11.0 UnknownUnknown

Scanned Files

  • django_app/poetry.lock
  • poetry.lock
  • redbox/poetry.lock
  • redbox/pyproject.toml

@codecov-commenter

codecov-commenter commented Jul 16, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.76%. Comparing base (b1e559d) to head (6254282).

Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #1208      +/-   ##
==========================================
- Coverage   85.01%   83.76%   -1.26%     
==========================================
  Files         226      179      -47     
  Lines       10059     4878    -5181     
==========================================
- Hits         8552     4086    -4466     
+ Misses       1507      792     -715     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@datadog-uktrade

datadog-uktrade Bot commented Jul 16, 2026

Copy link
Copy Markdown

Code Quality  Code Vulnerabilities  IaC  Library Vulnerabilities  Secrets

⚠️ Warnings

🛠️ 1 Code quality issue detected

High: python-best-practices/nested-blocks Do not have too many nested blocks View rule
redbox/redbox/transform.py:200-201

ℹ️ Info

No other issues found (see more)

🛡️ No new code vulnerabilities
📚 No new vulnerable libraries detected
🧱 No new Infrastructure as Code (IaC) issues
🔑 No new secrets detected

Useful? React with 👍 / 👎

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 6254282 | Docs | Give us feedback!

Comment thread redbox/redbox/transform.py Outdated
Comment thread redbox/redbox/transform.py Outdated
nboyse added 3 commits July 21, 2026 09:03
Signed-off-by: DBT pre-commit check
Signed-off-by: DBT pre-commit check
@nboyse
nboyse force-pushed the chore/add-proper-span-for-ai-observability branch from b5cbfea to a6423b1 Compare July 21, 2026 08:04
nboyse added 2 commits July 21, 2026 10:04
Signed-off-by: DBT pre-commit check
Signed-off-by: DBT pre-commit check
Comment on lines +155 to +156
if isinstance(message, dict):
return _extract_text_from_content_blocks(message.get("content"))

@datadog-uktrade datadog-uktrade Bot Jul 21, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔴 High: Code Quality Violation

too many nesting levels (...read more)

Avoid to nest too many loops together. Having too many loops make your code harder to understand.
Prefer to organize your code in functions and unit of code you can clearly understand.

Learn More

Helpful? 👍/👎View in Datadog  Leave us feedback  Documentation
🚩 Report as false positive. Flags the rule for review to improve detection accuracy.

nboyse added 3 commits July 21, 2026 11:47
Signed-off-by: DBT pre-commit check
Signed-off-by: DBT pre-commit check
Signed-off-by: DBT pre-commit check
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants