[Repo Assist] refactor(tracing): unify semconv import to v1.34.0 in span_helpers

[github-actions]

🤖 This is a draft PR from Repo Assist, an automated AI assistant.

Addresses recommendation #1 from the Go Fan review in #6841.

Summary

span_helpers.go and span_helpers_test.go imported semconv/v1.27.0 while every other file in internal/tracing/ used semconv/v1.34.0. This creates a mixed schema URL inconsistency in exported traces — spans from StartDIFCPipelineSpan and StartProxyForwardSpan would carry the v1.27.0 schema URL while all other spans use v1.34.0.

Root Cause

The span_helpers.go file was created when the project was on semconv v1.27.0, and wasn't updated when the rest of the tracing package was upgraded to v1.34.0.

Fix

Change the import alias in both files:

// Before:
semconv "go.opentelemetry.io/otel/semconv/v1.27.0"
// After:
semconv "go.opentelemetry.io/otel/semconv/v1.34.0"

URLPathKey (url.path) is defined identically in both versions — this is a zero-risk change.

No go.mod changes are needed: semconv/v1.27.0 and v1.34.0 are both sub-packages within the existing go.opentelemetry.io/otel v1.43.0 module.

Trade-offs

• Pure import path change; no API differences at the call sites
• Eliminates mixed schema URL in OTel traces (all spans now report SchemaURL: semconv.SchemaURL consistently)
• v1.27.0 sub-package becomes unused in the module and can be removed with go mod tidy in a future pass

Related: #6841

Test Status

⚠️ Infrastructure limitation: Network access to proxy.golang.org is blocked in this CI environment, preventing go build and go test from running. However:

• gofmt parses both files without error
• URLPathKey has been defined as url.path since semconv v1.20.0 and the definition is unchanged in v1.34.0
• Both versions are sub-packages of the same Go module at the same version, so no module resolution is needed
• All other internal/tracing/ source files already use v1.34.0 successfully

Generated by Repo Assist

Warning

Firewall blocked 1 domain

The following domain was blocked by the firewall during workflow execution:

• proxy.golang.org

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "proxy.golang.org"

See Network Configuration for more information.

Generated by Repo Assist · sonnet46 3.8M · ◷

Add this agentic workflows to your repo

To install this agentic workflow, run

gh aw add githubnext/agentics/workflows/repo-assist.md@851905c06e905bf362a9f6cc54f912e3df747d55

[Copilot]

Pull request overview

Unifies OpenTelemetry semantic convention (semconv) imports in internal/tracing/ by updating span_helpers.go and its test to use go.opentelemetry.io/otel/semconv/v1.34.0, matching the rest of the tracing package and avoiding mixed SchemaURL values in exported spans.

Changes:

• Switch semconv import in internal/tracing/span_helpers.go from v1.27.0 to v1.34.0.
• Switch semconv import in internal/tracing/span_helpers_test.go from v1.27.0 to v1.34.0.

Show a summary per file

File	Description
internal/tracing/span_helpers.go	Updates semconv import to v1.34.0 to keep span attributes/schema consistent with the rest of tracing.
internal/tracing/span_helpers_test.go	Aligns test semconv import with v1.34.0 to match production tracing code and avoid mixed versions.

Copilot's findings

Tip

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

• Files reviewed: 2/2 changed files
• Comments generated: 0