AltimateAI / AltimateAI/altimate-code

mcp: headersCommand missing from V2 ConfigMCP.Remote schema (merge drift risk)

Offen
#996 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
TypeScript
Sterne
811
Forks
134
Ø Merge
3 T. 2 Std.
Gemergte PRs (30 T.)
50

Beschreibung

Found during v0.9.1 release review (Tech-Lead). `packages/core/src/v1/config/mcp.ts` `ConfigMCPV1.Remote` has `headersCommand` (bearer-auth, #793). The newer Effect-Schema `ConfigMCP.Remote` in `packages/core/src/config/mcp.ts` does NOT. Not a live bug today (MCP load goes through V1), but V2 `Config.Service` is already wired into other subsystems. If MCP config loading migrates to V2, `headersCommand` will silently vanish via `onExcessProperty: "ignore"` — dropped auth headers, no error. Add `headersCommand` to V2, or decide V2 `ConfigMCP` is dead and remove it.

Beitragsleitfaden

Beitragsleitfaden öffnen

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.