Updated version in skill.md when there is a conflict

Author: bhanurp

skills/commands/publish/publish.go

@@ -114,6 +114,13 @@ func (pc *PublishCommand) Run() error {
 		return err
 	}
 
+	if meta.Version != "" && meta.Version != version {
+		if updateErr := UpdateSkillMetaVersion(pc.skillDir, version); updateErr != nil {
+			return fmt.Errorf("failed to update SKILL.md version: %w", updateErr)
+		}
+		log.Info(fmt.Sprintf("Updated SKILL.md version from '%s' to '%s'", meta.Version, version))
+	}
+
 	log.Info(fmt.Sprintf("Publishing skill '%s' version '%s'", slug, version))
 
 	zipPath, err := pc.resolveZip(slug, version)

skills/commands/publish/publish_test.go

@@ -105,6 +105,114 @@ version: '1.5.0'
 	assert.Equal(t, "1.5.0", meta.Version)
 }
 
+func TestUpdateSkillMetaVersion_ReplacesExisting(t *testing.T) {
+	dir := t.TempDir()
+	skillMD := `---
+name: my-skill
+description: A great skill
+version: 1.0.0
+---
+
+# My Skill
+
+Body content here.
+`
+	require.NoError(t, os.WriteFile(filepath.Join(dir, "SKILL.md"), []byte(skillMD), 0644))
+
+	err := UpdateSkillMetaVersion(dir, "2.0.0")
+	require.NoError(t, err)
+
+	meta, err := ParseSkillMeta(dir)
+	require.NoError(t, err)
+	assert.Equal(t, "2.0.0", meta.Version)
+	assert.Equal(t, "my-skill", meta.Name)
+	assert.Equal(t, "A great skill", meta.Description)
+
+	data, err := os.ReadFile(filepath.Join(dir, "SKILL.md"))
+	require.NoError(t, err)
+	assert.Contains(t, string(data), "Body content here.")
+}
+
+func TestUpdateSkillMetaVersion_QuotedVersion(t *testing.T) {
+	dir := t.TempDir()
+	skillMD := `---
+name: quoted-skill
+version: "1.0.0"
+---
+
+# Quoted
+`
+	require.NoError(t, os.WriteFile(filepath.Join(dir, "SKILL.md"), []byte(skillMD), 0644))
+
+	err := UpdateSkillMetaVersion(dir, "3.0.0")
+	require.NoError(t, err)
+
+	meta, err := ParseSkillMeta(dir)
+	require.NoError(t, err)
+	assert.Equal(t, "3.0.0", meta.Version)
+}
+
+func TestUpdateSkillMetaVersion_SingleQuotedVersion(t *testing.T) {
+	dir := t.TempDir()
+	skillMD := `---
+name: sq-skill
+version: '1.5.0'
+---
+`
+	require.NoError(t, os.WriteFile(filepath.Join(dir, "SKILL.md"), []byte(skillMD), 0644))
+
+	err := UpdateSkillMetaVersion(dir, "1.6.0")
+	require.NoError(t, err)
+
+	meta, err := ParseSkillMeta(dir)
+	require.NoError(t, err)
+	assert.Equal(t, "1.6.0", meta.Version)
+}
+
+func TestUpdateSkillMetaVersion_NoVersionField(t *testing.T) {
+	dir := t.TempDir()
+	skillMD := `---
+name: no-version-skill
+description: No version here
+---
+
+# Content
+`
+	require.NoError(t, os.WriteFile(filepath.Join(dir, "SKILL.md"), []byte(skillMD), 0644))
+
+	err := UpdateSkillMetaVersion(dir, "1.0.0")
+	require.NoError(t, err)
+
+	data, err := os.ReadFile(filepath.Join(dir, "SKILL.md"))
+	require.NoError(t, err)
+	assert.Equal(t, skillMD, string(data))
+}
+
+func TestUpdateSkillMetaVersion_PreservesBody(t *testing.T) {
+	dir := t.TempDir()
+	skillMD := `---
+name: body-skill
+version: 0.1.0
+---
+
+# My Skill
+
+Some **markdown** content with [links](https://example.com).
+
+` + "```python\nprint('hello')\n```\n"
+	require.NoError(t, os.WriteFile(filepath.Join(dir, "SKILL.md"), []byte(skillMD), 0644))
+
+	err := UpdateSkillMetaVersion(dir, "0.2.0")
+	require.NoError(t, err)
+
+	data, err := os.ReadFile(filepath.Join(dir, "SKILL.md"))
+	require.NoError(t, err)
+	content := string(data)
+	assert.Contains(t, content, "version: 0.2.0")
+	assert.Contains(t, content, "Some **markdown** content with [links](https://example.com).")
+	assert.Contains(t, content, "print('hello')")
+}
+
 func TestValidateSlug(t *testing.T) {
 	assert.NoError(t, ValidateSlug("my-skill"))
 	assert.NoError(t, ValidateSlug("skill123"))

skills/commands/publish/skillmeta.go

@@ -87,6 +87,54 @@ func stripQuotes(s string) string {
 	return s
 }
 
+// versionLineRegex matches a YAML version field (with optional quoting) inside front matter.
+var versionLineRegex = regexp.MustCompile(`(?m)^(version:\s*).*$`)
+
+// UpdateSkillMetaVersion replaces the version value in the SKILL.md YAML front matter.
+// It only acts when a version: line already exists; it never inserts a new one.
+func UpdateSkillMetaVersion(skillDir, newVersion string) error {
+	skillMDPath := filepath.Join(skillDir, "SKILL.md")
+	// #nosec G304 -- path is constructed from user-provided skill directory argument
+	data, err := os.ReadFile(skillMDPath)
+	if err != nil {
+		return fmt.Errorf("failed to read SKILL.md at %s: %w", skillMDPath, err)
+	}
+
+	content := string(data)
+	trimmed := strings.TrimSpace(content)
+	if !strings.HasPrefix(trimmed, "---") {
+		return fmt.Errorf("SKILL.md does not start with YAML frontmatter delimiter '---'")
+	}
+
+	rest := trimmed[3:]
+	endIdx := strings.Index(rest, "---")
+	if endIdx < 0 {
+		return fmt.Errorf("SKILL.md missing closing YAML frontmatter delimiter '---'")
+	}
+
+	// Locate the front matter boundaries within the original (untrimmed) content
+	fmStart := strings.Index(content, "---")
+	fmEnd := strings.Index(content[fmStart+3:], "---")
+	if fmEnd < 0 {
+		return fmt.Errorf("SKILL.md missing closing YAML frontmatter delimiter '---'")
+	}
+	fmEnd += fmStart + 3
+
+	frontmatter := content[fmStart+3 : fmEnd]
+	if !versionLineRegex.MatchString(frontmatter) {
+		return nil
+	}
+
+	updatedFM := versionLineRegex.ReplaceAllString(frontmatter, "${1}"+newVersion)
+	updated := content[:fmStart+3] + updatedFM + content[fmEnd:]
+
+	// #nosec G306 G703 -- SKILL.md is a user-owned source file; path constructed from user-provided skill directory
+	if err := os.WriteFile(skillMDPath, []byte(updated), 0644); err != nil {
+		return fmt.Errorf("failed to write updated SKILL.md: %w", err)
+	}
+	return nil
+}
+
 // ValidateSlug checks that a skill slug matches the required pattern.
 func ValidateSlug(slug string) error {
 	if !slugRegex.MatchString(slug) {