[dotnet-port-fixes] Add observability serialization parity tests

[Michelle Clayton (michelle-clayton-work)]

Summary

Adds regression coverage for the existing Go telemetry fallback when sensitive-data serialization fails. The new tests lock in that unserializable payloads do not fail workflow execution and that observability spans record the [Unserializable: ...] marker on message and executor attributes, matching the upstream .NET fix intent selected from PR #7612.

Ported .NET PRs

• .NET: Prevent telemetry serialization failures from failing workflows agent-framework#7612 - Prevent telemetry serialization failures from failing workflows (8a0731ad92560d7eb5c181682d9501a82a6b4a70, microsoft/agent-framework@8a0731a)

Breaking Changes

No. This PR only adds regression tests around existing fallback behavior and does not change the public Go API.

Tests and Examples

• go test ./workflow/internal/observability ./workflow/inproc
• Added internal observability coverage for executor input/output fallback on serialization errors
• Added end-to-end in-process workflow coverage ensuring unserializable sensitive-data telemetry does not break runs

Notes

The Go runtime already returned the [Unserializable: %T] fallback from workflow/internal/observability/observability.go; this port is test-parity to keep that behavior aligned with the upstream .NET regression suite.

Generated by .NET to Go Fixes and Test Porting Agent · gpt54 · 270.4 AIC · ⌖ 17 AIC · ⊞ 24.2K · ◷

Closes #826

[Copilot]

Pull request overview

Adds regression tests to ensure observability telemetry safely falls back to a [Unserializable: %T] marker when sensitive-data serialization fails, and that workflow execution continues successfully (parity with the referenced upstream .NET fix intent).

Changes:

• Added unit coverage for SerializedAttribute(...) fallback behavior on json.Marshal errors.
• Added internal observability coverage ensuring executor input/output attributes use the unserializable fallback when sensitive-data serialization fails.
• Added an in-process end-to-end workflow test verifying unserializable sensitive telemetry does not break runs and records the fallback marker on relevant spans.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
workflow/internal/observability/observability_test.go	Adds targeted tests for serialization-fallback behavior in internal observability attributes.
workflow/inproc/observability_test.go	Adds an end-to-end inproc workflow regression test validating fallback telemetry markers without failing execution.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[github-actions]

Parity Review: No parity issues

This PR adds regression tests only (workflow/inproc/observability_test.go and workflow/internal/observability/observability_test.go). No exported Go APIs are added, changed, or removed.

The test coverage locks in the existing [Unserializable: ...] fallback behavior, which matches the intent of the upstream .NET fix in microsoft/agent-framework#7612. No cross-repo consistency issues found.

public-api-change label: Not applicable — no exported API surface changed.

Generated by Go API Consistency Review Agent for #832 · sonnet46 · 11.1 AIC · ⌖ 4.66 AIC · ⊞ 5.7K · ◷