Skip to content

chore: rename phat image to stdio-wrapper#6478

Merged
thedadams merged 1 commit intoobot-platform:mainfrom
thedadams:push-nzonoxkzvmwz
May 1, 2026
Merged

chore: rename phat image to stdio-wrapper#6478
thedadams merged 1 commit intoobot-platform:mainfrom
thedadams:push-nzonoxkzvmwz

Conversation

@thedadams
Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings April 30, 2026 21:58
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Renames the default MCP container base image from phat to stdio-wrapper and updates related references across runtime defaults, tests, and documentation to keep deployments aligned with the new image naming/versioning.

Changes:

  • Update MCP runtime defaults to use ghcr.io/obot-platform/mcp-images/stdio-wrapper:v0.20.4 and bump HTTP webhook converter default to v0.20.4.
  • Update Kubernetes backend tests to reference stdio-wrapper:main instead of phat:main.
  • Update current and v0.20.0 versioned docs to reflect the new base image default (but see doc-version mismatches noted in comments).

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
pkg/mcp/loader.go Switches default MCP base image to stdio-wrapper and bumps HTTP webhook base image tag.
pkg/mcp/kubernetes_test.go Updates test fixtures to use stdio-wrapper:main.
docs/versioned_docs/version-v0.20.0/configuration/server-configuration.md Updates documented default MCP base image (webhook default version still stale per comment).
docs/versioned_docs/version-v0.20.0/configuration/mcp-deployments-in-kubernetes.md Updates documented default MCP image in Kubernetes deployments.
docs/docs/configuration/server-configuration.md Updates documented default MCP base image (version mismatch + webhook default still stale per comment).
docs/docs/configuration/mcp-deployments-in-kubernetes.md Updates documented default MCP image in Kubernetes deployments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/versioned_docs/version-v0.20.0/configuration/server-configuration.md Outdated
Comment thread docs/docs/configuration/server-configuration.md Outdated
@entelligence-ai-pr-reviews
Copy link
Copy Markdown

EntelligenceAI PR Summary

Renames the MCP base container image from phat to stdio-wrapper and updates version tags across the codebase.

  • pkg/mcp/loader.go: Bumps MCPBaseImage to stdio-wrapper:v0.20.4 and MCPHTTPWebhookBaseImage to v0.20.4
  • pkg/mcp/kubernetes_test.go: Replaces phat:main with stdio-wrapper:main in test fixtures and helper
  • Documentation (current and versioned): Reflects the image rename and version bump for OBOT_SERVER_MCPBASE_IMAGE and Kubernetes deployment references

Confidence Score: 5/5 - Safe to Merge

Safe to merge — this is a straightforward rename of the MCP base container image from phat to stdio-wrapper with an accompanying version bump to v0.20.4 across pkg/mcp/loader.go, pkg/mcp/kubernetes_test.go, and documentation files. The change is consistent and mechanical: MCPBaseImage and MCPHTTPWebhookBaseImage constants are updated in tandem, test fixtures referencing phat:main are replaced with stdio-wrapper:main, and documentation for OBOT_SERVER_MCPBASE_IMAGE reflects the new image name. No logic, validation, or runtime behavior is altered — this is purely an image reference rename with no functional side effects.

Key Findings:

  • MCPBaseImage constant in pkg/mcp/loader.go is updated from phat to stdio-wrapper:v0.20.4 atomically alongside MCPHTTPWebhookBaseImage, ensuring both constants remain consistent and preventing a partial-rename state.
  • Test fixtures in pkg/mcp/kubernetes_test.go correctly replace phat:main with stdio-wrapper:main, maintaining test validity and ensuring tests will still exercise the correct image reference behavior.
  • Documentation updates in both current and versioned docs mirror the code changes, so operator-facing guidance for OBOT_SERVER_MCPBASE_IMAGE stays accurate — no documentation drift introduced.
  • No review comments were generated and heuristic analysis found zero issues of any severity, making this a clean, low-risk housekeeping change.
Files requiring special attention
  • pkg/mcp/loader.go
  • pkg/mcp/kubernetes_test.go

Signed-off-by: Donnie Adams <donnie@obot.ai>
@thedadams thedadams merged commit 4ae13cb into obot-platform:main May 1, 2026
4 of 5 checks passed
@thedadams thedadams deleted the push-nzonoxkzvmwz branch May 1, 2026 00:33
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.

4 participants