Skip to content

feat: record telemetry span outcomes - #402

Merged
codeforester merged 8 commits into
mainfrom
enhancement/392-20260930-enhancement-record-span-status-and-exceptions-on-the-lifecyc
Oct 4, 2026
Merged

codeforester merged 8 commits into
mainfrom
enhancement/392-20260930-enhancement-record-span-status-and-exceptions-on-the-lifecyc

Conversation

@codeforester

@codeforester codeforester commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

Fixes #392

Summary

  • set OpenTelemetry span status from the normalized lifecycle outcome
  • record the original exception on failed invocations when available
  • cover successful and failed telemetry paths

Branch maintenance

Refs #426. Targets main.

The branch already contained main at a576cc279739eae5e4cfc33ffab2a7fb56de24de. No new commit was needed.

Current-head validation

At f32811439ab15bf72b8ee1d77e467852c78ea8dd: uv lock freshness and baseline, runtime, strict typing, style, and contracts passed locally with all declared extras. Runtime result: 609 passed, 1 warning, 262 subtests passed in 7.65s.

Hosted checks: 7/7 required checks passed; 0 checks pending; 0 unsuccessful checks at 2026-10-04T14:19:40.760978+00:00. See the PR Checks tab and #426 for subsequent results.

Comment thread lib/python/base_cli/integrations.py Outdated
Comment thread lib/python/base_cli/_attach.py Outdated
Comment thread tests/test_integrations.py
@codeforester

Copy link
Copy Markdown
Contributor Author

Resolved and revalidated.

  • The original failures were benchmark-contract outliers (Unix persistence-enabled p95 and WSL cold-import p95); the full WSL test suite itself reported 606 passed.
  • Reran the failed hosted jobs: Unix benchmark and WSL runtime/benchmark both passed.
  • Synced this branch with current main after security: guard delimited spreadsheet formulas #404 landed, then reran the complete Tests workflow: all 23 jobs passed.
  • Local validation: 606 passed; Ruff, formatting, and strict mypy all pass.

The PR is now clean/mergeable and remains open for review.

@codeforester
codeforester merged commit dec305b into main Oct 4, 2026
117 checks passed
@codeforester
codeforester deleted the enhancement/392-20260930-enhancement-record-span-status-and-exceptions-on-the-lifecyc branch October 4, 2026 16:55
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.

enhancement: record span status and exceptions on the lifecycle telemetry span

1 participant