Skip to content

codex-local 3/9: /model lists Ollama models (H2) - #7

Closed
ehrvs wants to merge 2 commits into
pr/02-h8from
pr/03-models
Closed

ehrvs wants to merge 2 commits into
pr/02-h8from
pr/03-models

Conversation

@ehrvs

@ehrvs ehrvs commented Oct 14, 2026

Copy link
Copy Markdown
Collaborator

codex-local-models implements ModelsEndpointClient over Ollama /api/tags (merged across nodes behind PAIR) and /api/show (context window). It uses .with_provider_catalog(), so only Ollama models are listed.

Hook H2 (model-provider, OnceLock factory): consulted only for provider codex-local. When unregistered, stock is unchanged.

Tests:

  • Mock /api/tags with 3 models: exactly those 3, no gpt-*.
  • The factory is consulted only for codex-local.
  • No auth headers on any request (verified with a logging mock).

Stack (merge bottom-up): 01-cli → 02-h8 → 03-models → 04-resume → 05-surface → 06-install → 07-ci → 08-sync → 09-docs. Plan: local/docs/plan-deltas.md; spike: local/docs/spike-responses.md.

🤖 Generated with Claude Code

https://claude.ai/code/session_01RPCzgwU247gMtjybR5v4iR

ehrvs and others added 2 commits October 14, 2026 14:58
…rovider

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Qp4Gg5UQcjWVPZz3y58bWs
…t model lookup

Silences the 'Model metadata not found' fallback warning on exec.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Qp4Gg5UQcjWVPZz3y58bWs
@ehrvs

ehrvs commented Oct 16, 2026

Copy link
Copy Markdown
Collaborator Author

Superseded: merged into codex-local/main via the integration PR #21 (3908b3d).

@ehrvs ehrvs closed this Oct 16, 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