Skip to content

[deep-report] Retype Headers any → map[string]string in frontmatter/observability configs #67197

Description

@github-actions

Description

Headers any is used for MCP server header config in pkg/workflow/frontmatter_types.go:252,289 and pkg/parser/import_observability.go:18, even though every consumer treats it as a string map: pkg/workflow/tools_parser.go:717 builds the equivalent headers field as make(map[string]string), and tools_types.go/observability_otlp.go consume HTTP headers as map[string]string elsewhere in the same package. The any typing forces a type assertion at every call site for no actual dynamism.

Expected Impact

Removes a pointless any→concrete-type assertion at every consumer, and makes "HTTP headers" a single consistent shape across the whole config surface (frontmatter, observability imports, tools).

Suggested Agent

General Go-focused coding agent (mechanical retype + call-site fix).

Estimated Effort

Quick (< 1 hour)

Data Source

Typist — Go Type Consistency Analysis, discussion #67180 (2026-10-09), Priority 1 item #2. Live-verified via direct source read of all three declaration sites before filing.

Generated by 🔬 Deep Report · claude · agent · 270.5 AIC · ⌖ 8.95 AIC · ⊞ 7.1K · ◷

  • expires on Oct 11, 2026, 4:47 AM UTC-08:00

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions