Emit the canonical OTel executor.type span attribute instead of executor.implementation.id

[PratikDhanave (PratikDhanave)]

What

The executor.process span in workflow/internal/observability recorded the executor type under the key executor.implementation.id. This PR renames the constant to TagExecutorType with the canonical value executor.type. The emitted value (the Go implementation identity, the correct Go analog of executor type) is unchanged.

Why

Both sibling SDKs emit this exact attribute under executor.type:

• .NET: Tags.ExecutorType = "executor.type", set via SetTag(Tags.ExecutorType, executorType).
• Python: EXECUTOR_TYPE = "executor.type".

Neither defines executor.implementation.id. Because Go diverged on only the key name, cross-language OTel dashboards filtering on executor.type found nothing on Go workflow traces. This aligns Go with the .NET/Python convention.

How tested

Added a black-box test in the package's canonical test file that drives StartExecutorProcess through a fake Tracer/Span capturing attributes, and asserts the span carries executor.type (with the executor-type value) and does NOT carry executor.implementation.id. The test fails before the change and passes after.

go build ./..., go vet ./workflow/internal/observability/..., and go test ./workflow/internal/observability/... all pass.

[Copilot]

Pull request overview

Aligns Go workflow telemetry with the canonical OpenTelemetry span attribute key for executor type so cross-language dashboards can filter consistently across Go/.NET/Python traces.

Changes:

• Renamed the executor-type attribute constant from executor.implementation.id to the canonical executor.type (value emitted unchanged).
• Updated StartExecutorProcess to emit the executor type under executor.type.
• Added a black-box test asserting executor.type is present and executor.implementation.id is not.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
workflow/internal/observability/observability.go	Switches the emitted span attribute key to canonical executor.type.
workflow/internal/observability/observability_test.go	Adds a tracer/span stub test to assert the new attribute key behavior.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[github-actions]

Parity Review — ✅ No Issues

This PR is limited to workflow/internal/observability/ (an internal package). No exported Go API surface changed.

Parity finding: The change improves cross-repo consistency by aligning the OTel span attribute key with the canonical value used in both sibling SDKs:

• .NET: Tags.ExecutorType = "executor.type"
• Python: EXECUTOR_TYPE = "executor.type"

Go previously emitted executor.implementation.id; after this PR it emits executor.type, which is the correct aligned value.

No public-api-change label needed (internal package only). parity-approved label is correctly set.

Generated by Go API Consistency Review Agent · sonnet46 · 12.4 AIC · ⌖ 5.61 AIC · ⊞ 5.7K · ◷