Skip to content

fix(otel): preserve operation start status - #597

Open
zhongkechen wants to merge 3 commits into
mainfrom
codex/otel-operation-start-status
Open

fix(otel): preserve operation start status#597
zhongkechen wants to merge 3 commits into
mainfrom
codex/otel-operation-start-status

Conversation

@zhongkechen

Copy link
Copy Markdown
Contributor

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Issue Link, if available

N/A

Description

  • Add nullable status to OperationInfo, matching the Python and TypeScript plugin contracts.
  • Populate operation status from checkpointed operations while preserving the existing constructor for compatibility.
  • Set invocation-view operation spans from OperationInfo.status, falling back to STARTED only when status is unavailable.
  • Add regression coverage for STARTED, supplied statuses, and terminal status replacement.

Demo/Screenshots

N/A

Checklist

  • I have filled out every section of the PR template
  • I have thoroughly tested this change

Testing

Unit Tests

Yes. Added converter and invocation OTel plugin assertions for null, non-null, and terminal operation statuses.

Integration Tests

Existing integration tests pass as part of the full repository suite:

JAVA_HOME=/Library/Java/JavaVirtualMachines/amazon-corretto-17.jdk/Contents/Home mvn test

Examples

N/A; no example behavior or public workflow API changed.

@zhongkechen
zhongkechen requested a review from a team August 6, 2026 00:28
@zhongkechen
zhongkechen requested a deployment to ai-pr-review-runtime August 6, 2026 00:28 — with GitHub Actions Waiting
@zhongkechen
zhongkechen requested a deployment to ai-pr-review-runtime August 6, 2026 00:28 — with GitHub Actions Waiting
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