fix: normalize extension skill authors

Assisted-by: OpenAI Codex (model: GPT-5, autonomous)

Author: philo-y

src/specify_cli/agents.py

@@ -402,7 +402,7 @@ def render_skill_command(
         source_file: str,
         project_root: Path,
         extension_id: Optional[str] = None,
-        author: str = "github-spec-kit",
+        author: object = "github-spec-kit",
     ) -> str:
         """Render a command override as a SKILL.md file.
 
@@ -443,15 +443,18 @@ def build_skill_frontmatter(
         skill_name: str,
         description: str,
         source: str,
-        author: str = "github-spec-kit",
+        author: object = "github-spec-kit",
     ) -> dict:
         """Build consistent SKILL.md frontmatter across all skill generators."""
+        normalized_author = (
+            "github-spec-kit" if author is None or author == "" else str(author)
+        )
         skill_frontmatter = {
             "name": skill_name,
             "description": description,
             "compatibility": "Requires spec-kit project structure with .specify/ directory",
             "metadata": {
-                "author": author,
+                "author": normalized_author,
                 "source": source,
             },
         }
@@ -621,7 +624,7 @@ def register_commands(
         _resolved_dir: Optional[Path] = None,
         link_outputs: bool = False,
         extension_id: Optional[str] = None,
-        author: str = "github-spec-kit",
+        author: object = "github-spec-kit",
     ) -> List[str]:
         """Register commands for a specific agent.
 
@@ -1068,7 +1071,7 @@ def register_commands_for_all_agents(
         create_missing_active_skills_dir: bool = False,
         extension_id: Optional[str] = None,
         only_agent: Optional[str] = None,
-        author: str = "github-spec-kit",
+        author: object = "github-spec-kit",
     ) -> Dict[str, List[str]]:
         """Register commands for all detected agents in the project.
 

src/specify_cli/extensions/__init__.py

@@ -1687,7 +1687,7 @@ def _replacement(match: re.Match[str]) -> str:
                 skill_name,
                 description,
                 f"extension:{manifest.id}",
-                author=manifest.data["extension"].get("author") or "github-spec-kit",
+                author=manifest.data["extension"].get("author"),
             )
             # Preserve the command's argument-hint in the generated skill,
             # mirroring the core template path (ClaudeIntegration.setup injects
@@ -3616,7 +3616,7 @@ def register_commands_for_agent(
             context_note=context_note,
             link_outputs=link_outputs,
             extension_id=manifest.id,
-            author=manifest.data["extension"].get("author") or "github-spec-kit",
+            author=manifest.data["extension"].get("author"),
         )
 
     def register_commands_for_all_agents(
@@ -3640,7 +3640,7 @@ def register_commands_for_all_agents(
             create_missing_active_skills_dir=create_missing_active_skills_dir,
             only_agent=only_agent,
             extension_id=manifest.id,
-            author=manifest.data["extension"].get("author") or "github-spec-kit",
+            author=manifest.data["extension"].get("author"),
         )
 
     def unregister_commands(

src/specify_cli/presets/__init__.py

@@ -2894,7 +2894,7 @@ def _build_extension_skill_restore_index(self) -> Dict[str, Dict[str, Any]]:
                     "command_name": cmd_name,
                     "source_file": source_file,
                     "source": f"extension:{manifest.id}",
-                    "author": manifest.data["extension"].get("author") or "github-spec-kit",
+                    "author": manifest.data["extension"].get("author"),
                     "extension_id": manifest.id,
                     "extension_dir": ext_root,
                 }

tests/test_extension_skills.py

@@ -447,9 +447,26 @@ def test_skill_md_content_correct(self, skills_project, extension_dir):
 
     @pytest.mark.parametrize("register_commands", [False, True])
     @pytest.mark.parametrize("link_commands", [False, True])
-    @pytest.mark.parametrize("author", ["acme-corp", 'Acme: "Platform"\nTeam', None, ""])
+    @pytest.mark.parametrize(
+        ("author", "expected_author"),
+        [
+            ("acme-corp", "acme-corp"),
+            ('Acme: "Platform"\nTeam', 'Acme: "Platform"\nTeam'),
+            (None, "github-spec-kit"),
+            ("", "github-spec-kit"),
+            (123, "123"),
+            (0, "0"),
+            (False, "False"),
+        ],
+    )
     def test_extension_author_preserved(
-        self, skills_project, extension_dir, register_commands, link_commands, author
+        self,
+        skills_project,
+        extension_dir,
+        register_commands,
+        link_commands,
+        author,
+        expected_author,
     ):
         """Both skill generators retain attribution, including dev output and aliases."""
         project_dir, skills_dir = skills_project
@@ -471,7 +488,7 @@ def test_extension_author_preserved(
         for name in names:
             content = (skills_dir / f"speckit-test-ext-{name}" / "SKILL.md").read_text()
             frontmatter = yaml.safe_load(content.split("---", 2)[1])
-            assert frontmatter["metadata"]["author"] == (author or "github-spec-kit")
+            assert frontmatter["metadata"]["author"] == expected_author
             assert "test-ext" in frontmatter["metadata"]["source"]
 
     def test_skill_md_has_parseable_yaml(self, skills_project, extension_dir):