Skip to content

fix(instrumentation-py): repair OpenLIT, OpenRouter, and PGVector OTel 2.x spans - #387

Merged
fran3cc merged 1 commit into
respanai:mainfrom
Nightingalelyy:fix/otel2-py-openlit-openrouter-pgvector
Aug 21, 2026
Merged

fix(instrumentation-py): repair OpenLIT, OpenRouter, and PGVector OTel 2.x spans#387
fran3cc merged 1 commit into
respanai:mainfrom
Nightingalelyy:fix/otel2-py-openlit-openrouter-pgvector

Conversation

@Nightingalelyy

@Nightingalelyy Nightingalelyy commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Repair OpenLIT, OpenRouter, and PGVector OTel 2.x instrumentation across current SDK shapes, lifecycle ownership, streaming, tools, errors, and bounded privacy-safe serialization.
  • Add current real-SDK/export regressions and a patch release intent for all three packages.
  • Preserve canonical span-contract fields without compatibility aliases.

Validation

  • Focused package tests: OpenLIT 30/30, OpenRouter 49/49, PGVector 22/22 plus 2 credential-gated live-database skips.

  • Root-scoped Ruff check and format check pass for all three package directories.

  • All three wheels build and pass isolated import/install checks; applicable pip check and editable direct_url.json checks pass.

  • Release intent and release inventory validators pass.

  • All 17 paired example scripts completed under otel2-fix-py-group-21-20260817T214109Z; the PGVector live-database script reported its intended missing-DSN skip.

  • Respan MCP: all 16 trace trees and all 48 full details inspected; exact roots, parents, IDs, markers, content, usage, streams, tools, errors, and privacy checked with no duplicates or credential leakage.

  • Real OpenRouter provider validation passed with model openai/gpt-4o-mini, exact output, and non-estimated 15/4/19 usage.

  • Full source and paired-example diffs were audited against every applicable document under contribution/; canonical span fields, OTel attribute shapes, release-intent/version ownership, and package boundaries have no remaining package-owned violation.

External follow-ups

  • Current ingestion still projects OpenRouter provider, status, and tool convenience fields inconsistently and estimates usage on selected no-terminal-usage error or early-close records.
  • Shared @tool input normalization, PGVector local-error classification and entity-path projection, upstream OpenAI/httpcore2 [DONE] teardown, and credential-gated PostgreSQL/pgvector runtime coverage remain separately tracked.

Companion examples

@Nightingalelyy Nightingalelyy changed the title fix(instrumentation): repair OpenLIT, OpenRouter, and PGVector OTel 2.x spans fix(instrumentation-py): repair OpenLIT, OpenRouter, and PGVector OTel 2.x spans Aug 18, 2026
@Nightingalelyy
Nightingalelyy marked this pull request as ready for review August 19, 2026 15:59

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Your trial has ended. Reactivate Greptile to resume code reviews.

@fran3cc
fran3cc merged commit f543157 into respanai:main Aug 21, 2026
5 checks passed
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