Skip to content

fix(plan): fetch exact Hugging Face repo IDs - #164

Open
hannibal-lee wants to merge 1 commit into
Andyyyy64:mainfrom
hannibal-lee:fix/plan-repo-id
Open

fix(plan): fetch exact Hugging Face repo IDs#164
hannibal-lee wants to merge 1 commit into
Andyyyy64:mainfrom
hannibal-lee:fix/plan-repo-id

Conversation

@hannibal-lee

Copy link
Copy Markdown
Contributor

What

  • Fetch an exact Hugging Face repository when whichllm plan owner/repo cannot find it in the cached model catalog.
  • Skip the bulk model catalog fetch for exact Repo ID queries.
  • Reuse the existing model metadata parser and respect HF_ENDPOINT.
  • Add clear errors for missing, private or gated repositories, network failures, and insufficient model metadata.
  • Keep directly fetched repositories out of the shared ranking cache.

Why

whichllm plan documents support for Hugging Face Repo IDs, but it previously searched only the limited cached catalog. Valid repositories outside that catalog returned No model found.

This particularly affected ilsp/Llama-Krikri-8B-Instruct, which exists on Hugging Face but does not appear in the default model listing.

Fixes #163

Testing

  • Tests pass (pytest) — 493 passed
  • New tests added
  • Tested on real hardware (not hardware-related)

Notes

Verified against the real Hugging Face API with:

whichllm plan ilsp/Llama-Krikri-8B-Instruct --json

Fall back to the Hugging Face model detail API when a plan query uses an exact repository ID that is absent from the cached catalog. Add repository-specific errors and regression coverage.\n\nFixes Andyyyy64#163
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.

Add a specific repo ID with a command

1 participant