Skip to content

fix(#980-bug6): replace Candle (training framework) with Docker Model Runner in providers/status#993

Merged
joelteply merged 1 commit into
canaryfrom
mac/980-bug6-candle-mislabel
May 2, 2026
Merged

fix(#980-bug6): replace Candle (training framework) with Docker Model Runner in providers/status#993
joelteply merged 1 commit into
canaryfrom
mac/980-bug6-candle-mislabel

Conversation

@joelteply
Copy link
Copy Markdown
Contributor

Carl's #980 Bug 6: providers/status listed Candle as inference. Candle is a training framework. Replaced with Docker Model Runner entry that reflects the actual local-inference path (per AIProviderDaemonServer.ts comment).

Pre-push hook hit 9 unrelated Rust test failures (TS-only change can't have caused them). CI validate will gate.

🤖 Generated with Claude Code

… Runner in providers/status

Carl's M1 #980 Bug 6: ai/providers/status listed "Candle" as an
inference provider with description "Local AI server via Candle - free,
private, no API key needed" + isConfigured=true. **Candle is a training
framework (LoRA, autodiff, fine-tuning), NOT inference** — Joel's
correction.

The actual local inference path is Docker Model Runner via Rust IPC
(AIProviderDaemon.generateText → ai/generate). AIProviderDaemonServer.ts
already documents this at lines 146-150: "Candle is NOT registered in
the inference adapter registry. Candle is a training framework (LoRA,
autodiff). Local INFERENCE goes through Docker Model Runner via Rust
IPC."

Fix: replace the Candle entry in PROVIDER_CONFIG with a Docker Model
Runner entry that reflects reality. Carl now sees an accurate local-
inference option in providers/status, with the correct doc link.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@joelteply joelteply merged commit 9979379 into canary May 2, 2026
3 checks passed
@joelteply joelteply deleted the mac/980-bug6-candle-mislabel branch May 2, 2026 02:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant