Skip to content

docs: document Claude Code local model workflows - #25641

Open
dvdksn wants to merge 2 commits into
docker:mainfrom
dvdksn:codex/document-sandbox-local-models
Open

docs: document Claude Code local model workflows#25641
dvdksn wants to merge 2 commits into
docker:mainfrom
dvdksn:codex/document-sandbox-local-models

Conversation

@dvdksn

@dvdksn dvdksn commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Summary

Document the experimental Claude Code sbx run --model workflows introduced by docker/sandboxes#4216, including bundled llmman, Ollama, and model swapping. Retain Docker Model Runner as an alternative on the Claude Code page.

@netlify /ai/sandboxes/agents/claude-code/

Preview

Generated by Codex

@netlify

netlify Bot commented Jul 24, 2026

Copy link
Copy Markdown

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit 4da1a47
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/6a7312d89e3de100080eaf76
😎 Deploy Preview https://deploy-preview-25641--docsdocker.netlify.app/ai/sandboxes/agents/claude-code/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@dvdksn dvdksn added the status/review Pull requests that are ready for review label Jul 24, 2026
@dvdksn
dvdksn requested a review from a team July 24, 2026 08:27

@docker-agent docker-agent left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Assessment: 🟡 NEEDS ATTENTION

Comment thread content/manuals/ai/sandboxes/agents/claude-code.md

@docker-agent docker-agent left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Assessment: 🟢 APPROVE

The PR adds a well-written "Local models" section to workflows.md and updates the cross-reference in claude-code.md. The new content:

  • Follows Docker documentation style (sentence-case heading, console code blocks with $ prompt, no hedge words, no marketing language)
  • Correctly discloses the experimental feature status and the claude-agent-only restriction
  • Accurately explains the recreate-on-model-change behavior and what persists (workspace, kit-owned volumes)
  • Keeps scope appropriate — the new section is concise and consistent with the rest of the page

No confirmed or likely issues were found.

The sandbox docs only described local models through Docker Model Runner.

Document the experimental --model workflows for llmman and Ollama, including model swaps, and link them from the Claude Code page.
@dvdksn
dvdksn force-pushed the codex/document-sandbox-local-models branch from 1057780 to 0db5506 Compare July 24, 2026 08:36
@dvdksn dvdksn changed the title docs: document sandbox local model workflows docs: document Claude Code local model workflows Jul 24, 2026
@dvdksn
dvdksn marked this pull request as ready for review August 4, 2026 10:25
@dvdksn
dvdksn requested a review from ericcurtin August 4, 2026 10:25

@docker-agent docker-agent left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Assessment: 🟢 APPROVE

The changes document the experimental Claude Code sbx run --model workflows (bundled llmman, Ollama, and model swapping) clearly and accurately. The updated description and keywords match the new content, the replacement Model Runner cross-reference points to the correct sandbox-specific guide, and no AI-isms, broken Markdown, or inaccurate commands were found.

@docker-agent docker-agent left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Assessment: 🟢 APPROVE

The PR adds a new Use a local model section documenting the experimental sbx run --model workflows (bundled llmman, Ollama, and model swapping). The content is clear, accurate, and follows Docker documentation style. No AI-isms, no broken markdown, no vendored files touched, and the scope matches the existing document.

No issues requiring changes were found.

Docker Model Runner, see
The `--model` flag routes Claude Code's Anthropic API requests to a model
served on your host. This feature is experimental and isn't supported on
Windows.

@ericcurtin ericcurtin Aug 5, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could we change this to only "This feature is experimental."? I don't want to set the expectation that Windows is out because it's not, it's an important platform. Could be a follow on PR.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, I heard that we are not shipping llmman + llama in Windows MSI. I took that as Windows is not supported. Can you confirm? cc @ilopezluna

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it's not, i can confirm 👍

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ilopezluna it's not supported for just this release?

It should at least say windows support is pending, not that we are not supporting it.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can confirm its not supported in this release we might say something like "coming soon" that's what we did in DMR. So being clear that is not supported I think its fine and also mentioning that will be here eventually so we set the right expectations

@ericcurtin ericcurtin Aug 5, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Local models IMO isn't supported at all, experimental means, have fun, but don't expect any support yet, but approved regardless, we might be too early even documenting this. But it's done now, we should merge and avoid wasted effort.

@ericcurtin

Copy link
Copy Markdown
Contributor

I'm actually making a breaking change, to use ollama, it will become --provider ollama --model somemodel but that's FYI, can be another PR. Thanks for opening this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/ai status/review Pull requests that are ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants