Skip to content

fix(policy): update HuggingFace preset to use router.huggingface.co - #1473

Closed
ac12644 wants to merge 2 commits into
NVIDIA:mainfrom
ac12644:fix/huggingface-preset-endpoint
Closed

fix(policy): update HuggingFace preset to use router.huggingface.co#1473
ac12644 wants to merge 2 commits into
NVIDIA:mainfrom
ac12644:fix/huggingface-preset-endpoint

Conversation

@ac12644

@ac12644 ac12644 commented Apr 4, 2026

Copy link
Copy Markdown

Summary

Update the HuggingFace policy preset to replace the deprecated api-inference.huggingface.co endpoint with router.huggingface.co.

The old endpoint now returns HTTP 410 Gone:

$ curl -sI https://api-inference.huggingface.co | head -1
HTTP/1.1 410 Gone

$ curl -sI https://router.huggingface.co/v1/models | head -1
HTTP/1.1 200 OK

Changes

  • nemoclaw-blueprint/policies/presets/huggingface.yaml: api-inference.huggingface.corouter.huggingface.co

Fixes #1453

Summary by CodeRabbit

  • Chores
    • Updated Hugging Face inference network configuration to use an optimized routing endpoint for improved connectivity and reliability.

The api-inference.huggingface.co endpoint now returns HTTP 410 Gone.
HuggingFace migrated their Inference Providers API to
router.huggingface.co.

Fixes NVIDIA#1453

Signed-off-by: Abhishek Chauhan <ac12644@gmail.com>
@coderabbitai

coderabbitai Bot commented Apr 4, 2026

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: d9aadaa5-6b9d-4fd6-8040-9a88ed7b71ad

📥 Commits

Reviewing files that changed from the base of the PR and between 3328359 and 68ae5da.

📒 Files selected for processing (1)
  • nemoclaw-blueprint/policies/presets/huggingface.yaml

📝 Walkthrough

Walkthrough

This change updates the Hugging Face network policy preset to replace the deprecated api-inference.huggingface.co endpoint with router.huggingface.co. The previous host was returning HTTP 410 Gone, indicating it is no longer available.

Changes

Cohort / File(s) Summary
HuggingFace Network Policy
nemoclaw-blueprint/policies/presets/huggingface.yaml
Updated HTTPS endpoint host from deprecated api-inference.huggingface.co to active router.huggingface.co for inference requests. Protocol, enforcement, TLS termination, and allowed routes remain unchanged.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A hop, a skip, through code I go,
Old endpoints dead, but now we know—
To router bright, the traffic flows,
No more 410s, that's how it goes!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: updating the HuggingFace preset to use the new router.huggingface.co endpoint instead of the deprecated api-inference.huggingface.co.
Linked Issues check ✅ Passed The PR successfully addresses issue #1453 by replacing the deprecated api-inference.huggingface.co endpoint with router.huggingface.co on port 443, exactly as required.
Out of Scope Changes check ✅ Passed The change is narrowly focused on updating the HuggingFace policy preset host endpoint, with no extraneous modifications to other files or unrelated code.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands and usage tips.

@latenighthackathon

Copy link
Copy Markdown
Collaborator

Great find! We have an open PR at #1454 addressing the same deprecated endpoint. Linking here for maintainers so we know to close any related duplicate PRs once one merges.

@wscurran

wscurran commented Apr 4, 2026

Copy link
Copy Markdown
Contributor

✨ Thanks for submitting this fix, which proposes a way to update the HuggingFace policy preset to use the current router.huggingface.co endpoint instead of the deprecated api-inference.huggingface.co. This restores connectivity for HuggingFace integrations.

@wscurran

Copy link
Copy Markdown
Contributor

Thanks for the HuggingFace endpoint fix. This was addressed by 5da9f44c (fix(policy): update HuggingFace endpoint and restrict Discord DELETE method, #1540), which is on main. Closing as superseded.

@wscurran wscurran closed this Apr 14, 2026
@ac12644
ac12644 deleted the fix/huggingface-preset-endpoint branch April 15, 2026 03:45
@wscurran wscurran added area: cli Command line interface, flags, terminal UX, or output area: providers Inference provider integrations and provider behavior bug-fix PR fixes a bug or regression feature PR adds or expands user-visible functionality and removed NemoClaw CLI feature PR adds or expands user-visible functionality labels Jun 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: cli Command line interface, flags, terminal UX, or output area: providers Inference provider integrations and provider behavior bug-fix PR fixes a bug or regression

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(policy): HuggingFace preset references deprecated api-inference.huggingface.co endpoint

3 participants