Skip to content

docs: update sandbox local model provider syntax - #25787

Open
dvdksn wants to merge 1 commit into
docker:mainfrom
dvdksn:codex/document-sbx-model-provider
Open

docs: update sandbox local model provider syntax#25787
dvdksn wants to merge 1 commit into
docker:mainfrom
dvdksn:codex/document-sbx-model-provider

Conversation

@dvdksn

@dvdksn dvdksn commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Summary

Update the experimental Claude Code local-model instructions for docker/sandboxes#4856. The Ollama example uses the new explicit --provider ollama flag instead of the removed ollama/ model-name prefix.

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

Preview the Claude Code sandbox page

Generated by Codex

The Claude Code sandbox page used the Ollama model prefix removed in sbx v0.39.0. Update the example to select Ollama with the new explicit --provider flag.
@dvdksn dvdksn added this to the sbx/v0.39.0 milestone Aug 11, 2026
@dvdksn
dvdksn requested a review from ericcurtin August 11, 2026 14:29
@dvdksn
dvdksn marked this pull request as ready for review August 11, 2026 14:29
@dvdksn
dvdksn requested a review from a team August 11, 2026 14:30

@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 change correctly updates the Ollama local-model provider syntax from the removed prefix notation (ollama/gemma4) to the new explicit flag (--model gemma4 --provider ollama). The surrounding prose is adjusted to match. No style violations, markdown issues, AI-isms, or content inaccuracies were found in the changed lines.

@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 change is accurate and focused — replacing the deprecated ollama/<model> prefix syntax with the explicit --provider ollama flag, with matching prose updates. No style issues, no incorrect commands, no missing redirects.

@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

No issues found in the changed lines. The update correctly replaces the removed ollama/ model-name prefix with the new --provider ollama explicit flag, updating both the prose and the console code block consistently. The changes are clear, accurate, and match Docker documentation style.

@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 change is minimal and well-scoped: it replaces the deprecated ollama/<model> prefix syntax with the new --provider ollama flag in one prose sentence and one console code block. No style issues, AI-isms, markdown problems, or missing redirects were found.

@ericcurtin

Copy link
Copy Markdown
Contributor

Super approve! :)

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.

4 participants