[dotnet-port-api] Response.ModelID and ConversationID fields

Aperta
  • Qualcosa non va?
#1,140 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

@PratikDhanave ci sta già lavorando.

Dal 22/9/2026.

  • #1156 di @PratikDhanave — aperta

Valutazione

Difficoltà
5/5
Tempo stimato
Più di una settimana
Idoneità per principianti
35/100
Tipo di issue
Funzionalità
Chiarezza
Da chiarire
Stato di attività
Attiva
Stack tecnologico
go
Ambito
api

Direzione di ricerca

Start with agent.Response, agent.ResponseUpdate, and Response.Update, then read CONTRIBUTING and the referenced M.E.AI response fields. Done means the maintainer resolves whether to add ModelID, ConversationID, or both, including provider population and AdditionalProperties migration scope.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

Parity gap surfaced comparing agent.Response.Update against M.E.AI ChatResponseExtensions.ProcessUpdate.

Gap

ProcessUpdate folds two update fields onto the response that Go has no home for:

  • update.ModelId → response.ModelId
  • update.ConversationId → response.ConversationId

Go's agent.Response has neither ModelID nor ConversationID. Today the model id is only surfaced ad hoc via provider AdditionalProperties (e.g. the recent Copilot usage model and OpenAI paths), and conversation identity lives on Session.ServiceID. So there is no canonical, provider-agnostic way to read "which model produced this response" or "which server-side conversation it belongs to."

Reference

  • M.E.AI ChatResponse.ModelId / ChatResponse.ConversationId, folded in ChatResponseExtensions.ProcessUpdate.
  • ChatResponseUpdate.ModelId / .ConversationId.

Proposed shape (needs maintainer input)

Add ModelID string (and possibly ConversationID string) to agent.Response and agent.ResponseUpdate, and fold them in Response.Update the same way ResponseID/FinishReason are folded (prefer later non-empty values). Providers would populate ResponseUpdate.ModelID from the model on the response.

Open questions:

  1. Scope: ModelID alone (smaller, immediately useful) vs also ConversationID (overlaps Session.ServiceID — may be redundant in Go).
  2. Whether providers should stop stashing model in AdditionalProperties once a first-class field exists (migration/back-compat).

Filing issue-first per CONTRIBUTING (new public API). Note #1133 fixes the related AdditionalProperties scoping bug in the same Response.Update, independent of this proposal.

Lingua principale
Go
Stelle
671
Fork
68
Merge medio
18h 11m
PR unite (30g)
82

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di microsoft/agent-framework-go

Tutte le issue di microsoft/agent-framework-go

Issue simili

Altre issue su Go

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.