Skip to content

feat(skills): add Docker image update check to wren-mcp-setup#1433

Merged
douenergy merged 1 commit intoCanner:mainfrom
goldmedal:chore/enhance-install-skill
Mar 9, 2026
Merged

feat(skills): add Docker image update check to wren-mcp-setup#1433
douenergy merged 1 commit intoCanner:mainfrom
goldmedal:chore/enhance-install-skill

Conversation

@goldmedal
Copy link
Copy Markdown
Contributor

@goldmedal goldmedal commented Mar 9, 2026

Summary

  • Adds a "Check for a newer image" sub-section to Step 2 of the wren-mcp-setup skill
  • Before docker run, the skill now guides users to compare the local wren-engine-ibis image digest against the remote registry and pull only when a newer version is available
  • Includes a note to stop and remove the existing container if a new image was pulled
  • Bumps wren-mcp-setup to v1.2 in SKILL.md, versions.json, and index.json

Test plan

  • Follow the skill — verify the image check section is clear and the bash snippet runs correctly on a machine with an existing image
  • Verify the skill version check at startup reports 1.2 as installed

🤖 Generated with Claude Code

Summary by CodeRabbit

  • New Features

    • Added automatic Docker image freshness check that compares local versus remote image digests to determine if an update is needed.
  • Chores

    • Bumped wren-mcp-setup version from 1.1 to 1.2.

Before starting the container in Step 2, the skill now guides the user
to compare the local image digest against the remote registry and pull
only when a newer version is available.

Bumps wren-mcp-setup to v1.2 in SKILL.md, versions.json, and index.json.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Mar 9, 2026
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 9, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 7db4fdcb-e9b4-476b-be31-39ad8ea6e467

📥 Commits

Reviewing files that changed from the base of the PR and between 0f1a176 and 9ad5e99.

📒 Files selected for processing (3)
  • skills/index.json
  • skills/versions.json
  • skills/wren-mcp-setup/SKILL.md

📝 Walkthrough

Walkthrough

Version bump of wren-mcp-setup skill from 1.1 to 1.2 across metadata files. Documentation updated to include Docker image digest comparison logic for detecting and pulling newer wren-engine-ibis images before container startup.

Changes

Cohort / File(s) Summary
Version Metadata
skills/index.json, skills/versions.json
Bumped wren-mcp-setup version from 1.1 to 1.2 in both skill registry and versions mapping.
Skill Documentation
skills/wren-mcp-setup/SKILL.md
Updated version references from 1.1 to 1.2 and added new pre-start check section for Docker image digest comparison to detect and pull newer wren-engine-ibis images.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested labels

documentation

Suggested reviewers

  • douenergy

Poem

🐰 A version hop, from one-point-one to two,
With Docker digests dancing in the queue,
New images checked before the start,
Fresh versions crafted with care and heart! 📦✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The pull request title accurately summarizes the main change: adding a Docker image update check to the wren-mcp-setup skill, which is the primary focus across all three modified files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@goldmedal goldmedal requested a review from douenergy March 9, 2026 08:03
@douenergy douenergy merged commit edb444b into Canner:main Mar 9, 2026
8 checks passed
nhaluc1005 pushed a commit to nhaluc1005/text2sql-practice that referenced this pull request Apr 3, 2026
…#1433)

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants