Skip to content

fix(models): preserve upstream namespaces in capability lookups - #16939

Merged
diegosouzapw merged 1 commit into
release/v3.8.52from
fix/capability-namespace-1017
Oct 17, 2026
Merged

diegosouzapw merged 1 commit into
release/v3.8.52from
fix/capability-namespace-1017

Conversation

@diegosouzapw

Copy link
Copy Markdown
Collaborator

Problem and change

A routed model such as aihorde/deepseek/deepseek-v4-flash inherited DeepSeek tool support and context limits after static-spec lookup recursively removed its upstream namespace. This made AI Horde appear eligible for tool requests despite its provider restriction.

Already-scoped capability lookups now preserve the remaining namespace. Public spec lookups retain their provider-prefix behavior by default, and the existing vision-only leaf fallback remains intact. The indexed lookup is reused; no new cache or scan is introduced.

Validation

  • Existing provider-tools and path-shaped vision regressions failed on release 858d6022270b73489dfa8d89eee6fcc9a6a9e214 before the fix.
  • Focused capability/lookup suites: 17 passed; public-lookup compatibility suite after adding two cases: 5 passed.
  • Adjacent registry, provider-prefix overrides, Mistral vision, command-code/Codex text-only and Claude effort suites: 53 passed.
  • Core TypeScript, changed-file ESLint/Prettier, file-size, changelog and commit hooks passed. No baseline changes.
  • Local tests used Node24.20.0, reused dependencies and isolated DATA_DIR. Full remote CI/Quality admission remains required.

The wider release diagnostic still has unrelated failures; this PR only fixes static capability namespace leakage.

@diegosouzapw
diegosouzapw merged commit 7b3f754 into release/v3.8.52 Oct 17, 2026
44 of 63 checks passed
@github-actions

Copy link
Copy Markdown
Contributor

CI Coverage Report

  • Coverage job: skipped
  • PR test policy: success

Coverage artifact was not available for this run.

@diegosouzapw
diegosouzapw deleted the fix/capability-namespace-1017 branch October 18, 2026 23:50
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