fix: presentation.go with mcp-table.go: the message for each MCP server comes back
I maintainer di solito rispondono entro 1 giorno
Una pull request collegata è già stata integrata.
- #243 di @mge1512 — integrata
Valutazione
- Difficoltà
- 1/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 78/100
Direzione di ricerca
Leggi examples/extensions/presentation.go e pkg/extensions/test/presentation_yaegi_test.go, iniziando da come OnSessionStart imposta i flag di visibilità dell’interfaccia utente. Aggiungi l’impostazione mancante HideMCPStartupMessages e aggiorna o esegui il test della presentazione per coprire entrambe le estensioni insieme. Il lavoro è completato quando il messaggio di avvio MCP rimane nascosto se presentation.go viene caricato insieme a mcp-table.go, mentre i messaggi di errore restano visibili.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Bug Description
With both example extensions loaded, Kit prints the message for each MCP server that has loaded, and then the table of mcp-table.go.
Steps to Reproduce
To reproduce with v0.133.0 and at least one MCP server in the config:
1. Copy examples/extensions/mcp-table.go to ~/.config/kit/extensions/.
2. Run `kit -e examples/extensions/presentation.go`.
Kit shows "MCP server 'demo' loaded with 1 tools" and then the table. With mcp-table.go alone, Kit shows only the table.
Cause: both examples call `SetUIVisibility` in `OnSessionStart`, and each call replaces all flags. presentation.go loads after mcp-table.go, in a directory (by file name) and with `-e` (explicit paths load last). Its call does not set `HideMCPStartupMessages`, so it clears the flag that mcp-table.go set.
Fix: presentation.go also sets `HideMCPStartupMessages`. This suits a presentation: without mcp-table.go, the message for each server is hidden too, and failure messages still appear.
presentation.go came with #222, my PR, and I missed this interaction. A PR with the fix follows.
Relevant Code / Configuration
Affected Component
examples/extensions/presentation.go pkg/extensions/test/presentation_yaegi_test.go
Kit Version
master
Additional Context
I apologize for this combination not having been tested before.
Checklist
- I've searched existing issues and this hasn't been reported yet
- I've tested with the latest version of Kit
- Lingua principale
- Go
- Stelle
- 142
- Fork
- 29
- Merge medio
- 4h 32m
- PR unite (30g)
- 61
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Issue simili
-
native
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
prime-radiant-inc/evener#4839 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 85/100
521xueweihan/HelloGitHub#4073 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 66/100
goharbor/terraform-provider-harbor#655 · 1 commento ·