modelcontextprotocol / modelcontextprotocol/python-sdk

Audit MCPServer constructor parameters

Aperta
#2,052 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

enhancement v2
Lingua principale
Python
Stelle
24.3k
Fork
4k
Merge medio
1g 1h
PR unite (30g)
31

Descrizione

From the V2 checklist in #1253:

We need to recheck the constructor parameters of the MCPServer class. I did remove and refactor a bit, but we need to audit it a bit better.

PR #1732 renamed FastMCP to MCPServer and moved transport-specific options (host, port, stateless_http, etc.) to run(). However, the constructor still has parameters that may need further review:

  • Are all remaining constructor parameters appropriate for the constructor vs. run()?
  • Are there parameters that should be removed, renamed, or have their defaults reconsidered?
  • Is the constructor signature clean and minimal for the common case?

This is a follow-up audit to ensure the MCPServer API surface is well-designed before V2 stabilizes.

Ref: Kludex's V2 checklist in #1253, follow-up to #1732

Guida per i contributori

Apri la guida per i contributori

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.

Direzione di ricerca

Inizia esaminando il costruttore di MCPServer e il relativo punto di ingresso run(), insieme alle modifiche di PR #1732 e alla checklist V2 in #1253. Verifica quali parametri appartengono a ciascuna API e documenta o testa le decisioni risultanti, in modo che il costruttore sia minimale e adatto a V2.

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

Valutazione

Stack tecnologico
python
Ambito
backend-api-design
Tipo di issue
Refactoring
Difficoltà
5/5
Tempo stimato
Più di una settimana
Stato di attività
Tranquilla
Chiarezza
Da chiarire
Idoneità per principianti
35/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.