Skip to content

Replace default claude code docker image#37

Merged
nezhar merged 1 commit into
mainfrom
claude-images
Mar 12, 2026
Merged

Replace default claude code docker image#37
nezhar merged 1 commit into
mainfrom
claude-images

Conversation

@nezhar
Copy link
Copy Markdown
Collaborator

@nezhar nezhar commented Mar 12, 2026

This pull request updates the default Docker image namespace for the Claude agent from nezhar to vibepod across documentation, configuration, code, and tests. This change ensures consistency and reflects the new official image location for Claude. Other agents remain unchanged.

Summary by CodeRabbit

  • Chores
    • Updated the Claude agent's default container image namespace to vibepod and added new default image mappings for additional agents (auggie, copilot, codex, datasette, proxy).
  • Documentation
    • Updated configuration and agent docs to show the new image references and example usage.
  • Tests
    • Adjusted tests to reflect the updated default image mappings.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Mar 12, 2026

📝 Walkthrough

Walkthrough

Updates default Docker image namespace and image names for Claude and several agents from nezhar/* to vibepod/* across docs, configuration, source constants, and tests; adds example environment variable overrides for the new images.

Changes

Cohort / File(s) Summary
Documentation
README.md, docs/agents/index.md, docs/configuration.md
Replaced nezhar/claude-container:latest with vibepod/claude:latest in examples and tables; added/updated example env vars for auggie, copilot, codex, datasette, and proxy images (new namespaces/names).
Source Constants
src/vibepod/constants.py
Changed DEFAULT_IMAGES["claude"] to use f"{os.environ.get('VP_IMAGE_NAMESPACE', 'vibepod')}/claude:latest" (namespace fallback updated from nezhar to vibepod).
Tests
tests/test_run.py
Updated test expected image string for Claude to vibepod/claude:latest.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 I hopped through configs, docs, and tests to play,
I nudged the namespace from nezhar to vibepod today,
Claude now sports a fresh, vibey tag, hooray!
Env vars lined up, images all in a row,
A happy little rabbit, off I go! 🥕✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and accurately describes the main change: updating the default Claude Docker image from nezhar to vibepod namespace.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch claude-images

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@docs/configuration.md`:
- Line 40: Update the VP_IMAGE_NAMESPACE example value to match the renamed
Claude repo: replace the stale example string "myorg/claude-container:latest"
with "myorg/claude:latest" in the configuration example so it aligns with the
new default image name (vibepod/claude:latest) and remains consistent across
docs; locate the example referencing VP_IMAGE_NAMESPACE and change the namespace
portion only.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 9ea9ecd1-33be-4f43-b7b9-8a45755c8717

📥 Commits

Reviewing files that changed from the base of the PR and between 43f1f9f and 2fd11d8.

📒 Files selected for processing (5)
  • README.md
  • docs/agents/index.md
  • docs/configuration.md
  • src/vibepod/constants.py
  • tests/test_run.py

Comment thread docs/configuration.md
Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
README.md (1)

35-48: ⚠️ Potential issue | 🟡 Minor

Clarify the mixed default namespaces in the intro sentence.

Line 35 now implies every non-Claude default stays under nezhar, but the list right below also includes vibepod/datasette:latest and vibepod/proxy:latest. Tightening that sentence would avoid sending readers to the wrong namespace.

Suggested doc tweak
-By default, agent images use the `vibepod` namespace for Claude (for example `vibepod/claude:latest`) and `nezhar` for other agents.
+By default, Claude uses the `vibepod` namespace (for example `vibepod/claude:latest`), most other agents use `nezhar`, and VibePod support services such as Datasette and the proxy also use `vibepod`.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@README.md` around lines 35 - 48, The intro sentence "By default, agent images
use the `vibepod` namespace for Claude ... and `nezhar` for other agents." is
misleading because some non-Claude images (e.g., `datasette` and `proxy`) use
`vibepod`; update that sentence to clearly state that Claude and certain
infrastructure images use `vibepod` while most other agents use `nezhar` (or
explicitly list exceptions), so readers know that entries like
`vibepod/datasette:latest` and `vibepod/proxy:latest` are intentional exceptions
to the `nezhar` defaults.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Outside diff comments:
In `@README.md`:
- Around line 35-48: The intro sentence "By default, agent images use the
`vibepod` namespace for Claude ... and `nezhar` for other agents." is misleading
because some non-Claude images (e.g., `datasette` and `proxy`) use `vibepod`;
update that sentence to clearly state that Claude and certain infrastructure
images use `vibepod` while most other agents use `nezhar` (or explicitly list
exceptions), so readers know that entries like `vibepod/datasette:latest` and
`vibepod/proxy:latest` are intentional exceptions to the `nezhar` defaults.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 3a4f3a02-d31e-4280-98ee-4a4e1b8643a1

📥 Commits

Reviewing files that changed from the base of the PR and between 2fd11d8 and 987fc10.

📒 Files selected for processing (5)
  • README.md
  • docs/agents/index.md
  • docs/configuration.md
  • src/vibepod/constants.py
  • tests/test_run.py
🚧 Files skipped from review as they are similar to previous changes (3)
  • docs/agents/index.md
  • src/vibepod/constants.py
  • tests/test_run.py

@nezhar nezhar merged commit a6da98a into main Mar 12, 2026
17 checks passed
@nezhar nezhar deleted the claude-images branch March 20, 2026 13:55
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