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

Remove lat_spec schema workaround after ToolRegistry releases union fix

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

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
65/100
Tipo di issue
Refactoring
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
python
Ambito
api, backend

Direzione di ricerca

La soluzione alternativa è in registry._trim_wire_schema, coinvolgendo _UNTYPED_UNION_SCHEMAS e _type_untyped_unions(). Per prima cosa, controlla l'issue upstream Oaklight/ToolRegistry#269 per vedere se è stato rilasciato un fix. Poi, aggiorna la dipendenza toolregistry, aggiorna uv.lock e testa che run_analysis e plot_dataset servano correttamente lo schema lat_spec senza la riparazione locale. Infine, rimuovi il codice della soluzione alternativa ed esegui i test di regressione per assicurarti che il contratto wire sia mantenuto.

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

Descrizione

Summary

PR #164 added a downstream workaround in registry._trim_wire_schema because ToolRegistry 0.18.1 drops branches from heterogeneous union schemas. Keep that workaround until the upstream defect is fixed and released, then adopt the fixed ToolRegistry release and remove our compatibility code.

Upstream tracking issue: Oaklight/ToolRegistry#269

Current workaround

Commit 14ed993 adds _UNTYPED_UNION_SCHEMAS / _type_untyped_unions() so the served lat_spec schema advertises both accepted wire types:

  • number
  • array<number>

The workaround is deliberately a no-op if ToolRegistry begins emitting type or anyOf itself.

When upstream releases a fix

  • Upgrade the minimum toolregistry version to the first release containing Oaklight/ToolRegistry#269.
  • Refresh uv.lock and package metadata as needed.
  • Confirm run_analysis and plot_dataset serve lat_spec as number | array<number> without local schema repair.
  • Confirm a JSON array such as [-60, 60] passes through execute_tool_calls validation.
  • Remove _UNTYPED_UNION_SCHEMAS, _type_untyped_unions(), and its call from _trim_wire_schema.
  • Retain or adapt the regression tests so the wire contract remains pinned to real registry execution.
  • Re-measure and lower the schema payload budgets if removing the workaround changes their size.

Blocking condition

Do not remove the workaround merely because the upstream issue or PR is merged. Wait until a ToolRegistry release containing the fix is available and the end-to-end tests pass against that released version.

Lingua principale
Python
Stelle
4
Fork
2
Merge medio
3h 1m
PR unite (30g)
21

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 UXARRAY/uxarray-mcp-server

Tutte le issue di UXARRAY/uxarray-mcp-server

Issue simili

Altre issue su Python

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.