Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

CLI error messages are parsed as Rich markup, so `skillspector mcp` drops `[mcp]` from its install hint

Aperta Adatta ai principianti
  • Qualcosa non va?
#885 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 4 giorni

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
84/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
python
Ambito
cli

Direzione di ricerca

Inizia dai punti in cli.py in cui vengono stampate le eccezioni (inclusi gli esempi vicino alle righe 396, 699, 743, 908 e 3413), poi leggi il messaggio correlato in mcp_server.py:268 e le correzioni in #725 e #734. Aggiungi test CLI per i tre comandi di riproduzione dell’issue; il lavoro è concluso quando il testo tra parentesi quadre resta intatto e il percorso malformato produce il consueto errore con codice di uscita 2.

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

Descrizione

On main at 95a52d89fa16, the CLI prints exception text with err_console.print(f"[red]Error:[/red] {e}") in cli.py (for example lines 396, 699, 743, 908 and 3413), so any brackets in the message are read as Rich markup.

Repro in a fresh venv with pip install git+https://github.com/NVIDIA/SkillSpector:

  • skillspector mcp prints Install it with: pip install 'skillspector'. The message in mcp_server.py:268 says skillspector[mcp], and following the printed advice doesn't install the extra.
  • skillspector scan "skills[old]/demo" reports Cannot determine input type for: skills/demo.
  • skillspector scan "nope[/x]" ends in a MarkupError traceback instead of the usual exit-2 error.

#725 fixed the same class for terminal reports (terminal_text.visible_terminal_text) and #734 for batch output; the CLI error lines weren't covered.

A fix would escape the interpolated text with rich.markup.escape at each of those call sites, with a CLI test per case above. Would a PR for that be welcome?

Lingua principale
Python
Stelle
20.2k
Fork
1.8k
Merge medio
4g 14h
PR unite (30g)
120

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 NVIDIA/SkillSpector

Tutte le issue di NVIDIA/SkillSpector

Issue simili

Altre issue su Python

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.