Skip to content

fix: support current LM Studio model metadata - #1

Closed
themizzi wants to merge 6 commits into
mainfrom
fix/lmstudio-native-model-discovery
Closed

themizzi wants to merge 6 commits into
mainfrom
fix/lmstudio-native-model-discovery

Conversation

@themizzi

@themizzi themizzi commented Sep 9, 2026

Copy link
Copy Markdown
Owner

Summary

  • accept current LM Studio native /api/v1/models records, including xhigh reasoning and newer fields
  • validate LLM records separately from embedding/non-LLM records
  • preserve unloaded and loaded context discovery
  • include detailed fetch and validation reasons in discovery failures
  • add regression coverage using current MLX/GGUF, loaded-instance, reasoning, and embedding payloads

Root cause

The release candidate schema validated every record as an LLM and restricted reasoning options to off, on, low, medium, and high. The live response included an embedding record without LLM fields and a model using xhigh, causing whole-response validation to fail and auto-detection to silently fall back.

Verification

  • npm run typecheck
  • npm run lint
  • npm run test:run (30 tests passed)
  • npm run build
  • live smoke test against http://127.0.0.1:1234: 3 LLMs discovered, 2 embeddings skipped

@themizzi

themizzi commented Sep 9, 2026

Copy link
Copy Markdown
Owner Author

Superseded by upstream PR agustif#58

@themizzi themizzi closed this Sep 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant