Skip to content

MAF-19235: feat(skills): add bump-dependency skill#80

Merged
hhk7734 merged 9 commits intomainfrom
MAF-19235
Mar 5, 2026
Merged

MAF-19235: feat(skills): add bump-dependency skill#80
hhk7734 merged 9 commits intomainfrom
MAF-19235

Conversation

@hhk7734
Copy link
Copy Markdown
Member

@hhk7734 hhk7734 commented Mar 5, 2026

Summary

  • Add skills/bump-dependency/ skill with agent guidelines for updating MIF dependency versions
  • Covers Helm sub-charts (Odin, Odin-CRD, LWS, Heimdall), heimdall-proxy image tags, moreh-vLLM presets, MIF chart releases, and website documentation updates for spec changes
  • Register the new skill in skills/README.md and AGENTS.md

Test plan

  • Verify the skill is discoverable via Claude Code plugin (claude --plugin-dir ./skills)
  • Test a dependency bump workflow using the skill as a reference
  • Confirm make helm-lint passes with no changes to chart files

🤖 Generated with Claude Code

Agent guideline for updating MIF dependency versions. Covers Helm
sub-charts (Odin, Odin-CRD, LWS, Heimdall), heimdall-proxy image tags,
moreh-vLLM presets, MIF chart releases, and website documentation
updates for spec changes.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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

Adds a new agent skill (bump-dependency) to standardize how MoAI Inference Framework dependency/version bumps are performed across Helm sub-charts, image tags, presets, and website docs, and registers the skill in the repository’s skill index and agent entrypoints.

Changes:

  • Introduce skills/bump-dependency/SKILL.md with step-by-step dependency bump procedures and verification/commit checklists.
  • Register the new skill in skills/README.md.
  • Link the new skill from the root AGENTS.md “Agent Skills” section.

Reviewed changes

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

File Description
skills/bump-dependency/SKILL.md New skill content describing dependency bump workflows, affected file locations, and verification steps.
skills/README.md Adds bump-dependency to the Available Skills table.
AGENTS.md Adds a pointer to the new skill under Agent Skills.

You can also share your feedback on Copilot code review. Take the survey.

Comment thread skills/bump-dependency/SKILL.md Outdated
Comment thread skills/bump-dependency/SKILL.md
hhk7734 and others added 2 commits March 5, 2026 10:51
…ependency

Heimdall is deployed as a separate Helm chart, not as a sub-chart of
moai-inference-framework. Update the component table and add a dedicated
procedure section that asks the user for chart repo and deployment details.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The prefix is automatically added by the pre-commit hook based on the
branch name. Documenting it in examples was misleading.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings March 5, 2026 01:53
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

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


You can also share your feedback on Copilot code review. Take the survey.

Comment thread skills/bump-dependency/SKILL.md
Comment thread skills/bump-dependency/SKILL.md Outdated
Comment thread skills/README.md
hhk7734 and others added 2 commits March 5, 2026 11:05
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…o and doc references

Heimdall is installed from the public Moreh Helm repo (moreh/heimdall).
Document concrete version locations (quickstart, reference docs) and add
steps to update reference docs when config or API changes occur.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings March 5, 2026 02:08
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

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


You can also share your feedback on Copilot code review. Take the survey.

Comment thread skills/bump-dependency/SKILL.md Outdated
Comment thread skills/bump-dependency/SKILL.md Outdated
hhk7734 and others added 2 commits March 5, 2026 11:51
…ll-creator reference

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings March 5, 2026 02:53
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

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.


You can also share your feedback on Copilot code review. Take the survey.

Comment thread skills/README.md
… changes

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.


You can also share your feedback on Copilot code review. Take the survey.

Comment thread skills/README.md
- Fix "Both skills" → "All skills" in README
- Remove reference/heimdall/ from component table (not a version location)
- Move InferencePool from Odin to Heimdall search targets, add
  inferencePoolRefs to Odin

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@hhk7734 hhk7734 merged commit c8cf22a into main Mar 5, 2026
3 checks passed
@hhk7734 hhk7734 deleted the MAF-19235 branch March 5, 2026 04:56
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.

2 participants