Fix default skill version and reject leading zeros in semver

- Change default skill version from 0.1.0 to 0.0.1 (skills should
  start at 0.0.1, distinct from the CLI's own v0.1.0 release version)
- Reject leading zeros in ParseVersion per strict semver spec
  (e.g. "01.2.3" is now invalid)
- Update all tests and docs accordingly

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: devrimcavusoglu

AGENTS.md

@@ -176,7 +176,7 @@ metadata:
     name: author-name
     type: human           # human | agent
     platform: claude-code  # only when type=agent
-  version: "0.1.0"
+  version: "0.0.1"
   modified-by:            # append-only provenance list
     - name: codex-cli
       type: agent

internal/cli/skill_create.go

@@ -144,7 +144,7 @@ func newSkillCreateCmd() *cobra.Command {
 	cmd.Flags().BoolVar(&force, "force", false, "bypass overlap detection block")
 	cmd.Flags().StringVar(&fromTemplate, "from-template", "", "path to a template file for the skill body")
 	cmd.Flags().StringSliceVar(&tags, "tags", nil, "comma-separated tags for the skill")
-	cmd.Flags().StringVar(&version, "version", "", "initial version (default: 0.1.0)")
+	cmd.Flags().StringVar(&version, "version", "", "initial version (default: 0.0.1)")
 
 	return cmd
 }

internal/cli/skill_version_test.go

@@ -23,7 +23,7 @@ func TestSkillVersion_Show(t *testing.T) {
 	var result output.SkillVersionResult
 	require.NoError(t, json.Unmarshal([]byte(out), &result))
 	assert.Equal(t, "ver-skill", result.Name)
-	assert.Equal(t, "0.1.0", result.Version)
+	assert.Equal(t, "0.0.1", result.Version)
 	assert.Equal(t, "user", result.Scope)
 	assert.False(t, result.Bumped)
 }
@@ -36,7 +36,7 @@ func TestSkillVersion_Show_Text(t *testing.T) {
 
 	out, err := runCmd(t, cc, "skill", "version", "ver-text")
 	require.NoError(t, err)
-	assert.Contains(t, out, "0.1.0")
+	assert.Contains(t, out, "0.0.1")
 }
 
 func TestSkillVersion_NotFound(t *testing.T) {
@@ -60,8 +60,8 @@ func TestSkillVersion_BumpPatch(t *testing.T) {
 	var result output.SkillVersionResult
 	require.NoError(t, json.Unmarshal([]byte(out), &result))
 	assert.Equal(t, "bump-patch", result.Name)
-	assert.Equal(t, "0.1.1", result.Version)
-	assert.Equal(t, "0.1.0", result.PreviousVersion)
+	assert.Equal(t, "0.0.2", result.Version)
+	assert.Equal(t, "0.0.1", result.PreviousVersion)
 	assert.True(t, result.Bumped)
 
 	// Verify the change persisted
@@ -70,7 +70,7 @@ func TestSkillVersion_BumpPatch(t *testing.T) {
 
 	var showResult output.SkillResult
 	require.NoError(t, json.Unmarshal([]byte(showOut), &showResult))
-	assert.Equal(t, "0.1.1", showResult.Version)
+	assert.Equal(t, "0.0.2", showResult.Version)
 }
 
 func TestSkillVersion_BumpMinor(t *testing.T) {
@@ -84,8 +84,8 @@ func TestSkillVersion_BumpMinor(t *testing.T) {
 
 	var result output.SkillVersionResult
 	require.NoError(t, json.Unmarshal([]byte(out), &result))
-	assert.Equal(t, "0.2.0", result.Version)
-	assert.Equal(t, "0.1.0", result.PreviousVersion)
+	assert.Equal(t, "0.1.0", result.Version)
+	assert.Equal(t, "0.0.1", result.PreviousVersion)
 	assert.True(t, result.Bumped)
 }
 
@@ -101,7 +101,7 @@ func TestSkillVersion_BumpMajor(t *testing.T) {
 	var result output.SkillVersionResult
 	require.NoError(t, json.Unmarshal([]byte(out), &result))
 	assert.Equal(t, "1.0.0", result.Version)
-	assert.Equal(t, "0.1.0", result.PreviousVersion)
+	assert.Equal(t, "0.0.1", result.PreviousVersion)
 	assert.True(t, result.Bumped)
 }
 
@@ -124,8 +124,8 @@ func TestSkillVersion_BumpText(t *testing.T) {
 	out, err := runCmd(t, cc, "skill", "version", "bump-text", "--bump", "patch")
 	require.NoError(t, err)
 	assert.Contains(t, out, "Bumped")
-	assert.Contains(t, out, "0.1.0")
-	assert.Contains(t, out, "0.1.1")
+	assert.Contains(t, out, "0.0.1")
+	assert.Contains(t, out, "0.0.2")
 }
 
 func TestSkillVersion_MultipleBumps(t *testing.T) {
@@ -145,7 +145,7 @@ func TestSkillVersion_MultipleBumps(t *testing.T) {
 
 	var result output.SkillVersionResult
 	require.NoError(t, json.Unmarshal([]byte(out), &result))
-	assert.Equal(t, "0.1.2", result.Version)
+	assert.Equal(t, "0.0.3", result.Version)
 }
 
 func TestSkillVersion_Scoped(t *testing.T) {
@@ -160,7 +160,7 @@ func TestSkillVersion_Scoped(t *testing.T) {
 	var result output.SkillVersionResult
 	require.NoError(t, json.Unmarshal([]byte(out), &result))
 	assert.Equal(t, "project", result.Scope)
-	assert.Equal(t, "0.1.0", result.Version)
+	assert.Equal(t, "0.0.1", result.Version)
 }
 
 // --- skill create --version ---
@@ -220,5 +220,5 @@ func TestSkillCreate_DefaultVersion(t *testing.T) {
 
 	var result output.SkillResult
 	require.NoError(t, json.Unmarshal([]byte(out), &result))
-	assert.Equal(t, "0.1.0", result.Version)
+	assert.Equal(t, "0.0.1", result.Version)
 }

internal/skill/scaffold.go

@@ -31,7 +31,7 @@ func NewSkillWithBody(name, description, authorName, authorType, authorPlatform,
 		Description: description,
 		Metadata: Metadata{
 			Author:  author,
-			Version: "0.1.0",
+			Version: "0.0.1",
 		},
 		Body: body,
 	}

internal/skill/scaffold_test.go

@@ -11,7 +11,7 @@ func TestNewSkill_NameOnly(t *testing.T) {
 
 	assert.Equal(t, "my-skill", s.Name)
 	assert.Contains(t, s.Description, "TODO")
-	assert.Equal(t, "0.1.0", s.Metadata.Version)
+	assert.Equal(t, "0.0.1", s.Metadata.Version)
 	assert.Contains(t, s.Body, "## Instructions")
 }
 

internal/skill/version.go

@@ -6,6 +6,18 @@ import (
 	"strings"
 )
 
+// parseVersionPart parses a single version component, rejecting leading zeros per semver spec.
+func parseVersionPart(s string) (int, error) {
+	if len(s) > 1 && s[0] == '0' {
+		return 0, fmt.Errorf("leading zeros not allowed: %q", s)
+	}
+	n, err := strconv.Atoi(s)
+	if err != nil || n < 0 {
+		return 0, fmt.Errorf("invalid part: %q", s)
+	}
+	return n, nil
+}
+
 // Version represents a semantic version (MAJOR.MINOR.PATCH).
 type Version struct {
 	Major int
@@ -20,19 +32,19 @@ func ParseVersion(s string) (Version, error) {
 		return Version{}, fmt.Errorf("invalid version %q: must be MAJOR.MINOR.PATCH", s)
 	}
 
-	major, err := strconv.Atoi(parts[0])
-	if err != nil || major < 0 {
-		return Version{}, fmt.Errorf("invalid version %q: major must be a non-negative integer", s)
+	major, err := parseVersionPart(parts[0])
+	if err != nil {
+		return Version{}, fmt.Errorf("invalid version %q: major must be a non-negative integer without leading zeros", s)
 	}
 
-	minor, err := strconv.Atoi(parts[1])
-	if err != nil || minor < 0 {
-		return Version{}, fmt.Errorf("invalid version %q: minor must be a non-negative integer", s)
+	minor, err := parseVersionPart(parts[1])
+	if err != nil {
+		return Version{}, fmt.Errorf("invalid version %q: minor must be a non-negative integer without leading zeros", s)
 	}
 
-	patch, err := strconv.Atoi(parts[2])
-	if err != nil || patch < 0 {
-		return Version{}, fmt.Errorf("invalid version %q: patch must be a non-negative integer", s)
+	patch, err := parseVersionPart(parts[2])
+	if err != nil {
+		return Version{}, fmt.Errorf("invalid version %q: patch must be a non-negative integer without leading zeros", s)
 	}
 
 	return Version{Major: major, Minor: minor, Patch: patch}, nil

internal/skill/version_test.go

@@ -26,8 +26,8 @@ func TestParseVersion(t *testing.T) {
 		},
 		{
 			name:  "default version",
-			input: "0.1.0",
-			want:  Version{Major: 0, Minor: 1, Patch: 0},
+			input: "0.0.1",
+			want:  Version{Major: 0, Minor: 0, Patch: 1},
 		},
 		{
 			name:  "large numbers",
@@ -79,6 +79,21 @@ func TestParseVersion(t *testing.T) {
 			input:   "1.0.-1",
 			wantErr: true,
 		},
+		{
+			name:    "leading zero major",
+			input:   "01.0.0",
+			wantErr: true,
+		},
+		{
+			name:    "leading zero minor",
+			input:   "1.01.0",
+			wantErr: true,
+		},
+		{
+			name:    "leading zero patch",
+			input:   "1.0.01",
+			wantErr: true,
+		},
 	}
 
 	for _, tt := range tests {
@@ -165,19 +180,19 @@ func TestBumpVersion(t *testing.T) {
 		},
 		{
 			name:    "bump default version patch",
-			version: "0.1.0",
+			version: "0.0.1",
 			level:   "patch",
-			want:    "0.1.1",
+			want:    "0.0.2",
 		},
 		{
 			name:    "bump default version minor",
-			version: "0.1.0",
+			version: "0.0.1",
 			level:   "minor",
-			want:    "0.2.0",
+			want:    "0.1.0",
 		},
 		{
 			name:    "bump default version major",
-			version: "0.1.0",
+			version: "0.0.1",
 			level:   "major",
 			want:    "1.0.0",
 		},