fix(extensions): drop alias pattern enforcement from _validate()

Aliases are intentionally free-form to preserve community extension
compatibility (e.g. 'speckit.verify' short aliases used by spec-kit-verify
and other existing extensions). This aligns _validate() with the intent of
upstream commit 4deb90f (fix: restore alias compatibility, #2110/#2125).

Only type and None-normalization checks remain for aliases. Pattern
enforcement continues for primary command names only.

Updated tests to verify free-form aliases pass through unchanged with
no warnings instead of being auto-corrected.

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

Author: iamaeroplane

src/specify_cli/extensions.py

@@ -239,34 +239,22 @@ def _validate(self):
                         "must follow pattern 'speckit.{extension}.{command}'"
                     )
 
-            # Validate and auto-correct alias name formats
+            # Validate alias types; no pattern enforcement on aliases — they are
+            # intentionally free-form to preserve community extension compatibility
+            # (e.g. 'speckit.verify' short aliases used by existing extensions).
             aliases = cmd.get("aliases")
             if aliases is None:
+                cmd["aliases"] = []
                 aliases = []
             if not isinstance(aliases, list):
                 raise ValidationError(
                     f"Aliases for command '{cmd['name']}' must be a list"
                 )
-            for i, alias in enumerate(aliases):
+            for alias in aliases:
                 if not isinstance(alias, str):
                     raise ValidationError(
                         f"Aliases for command '{cmd['name']}' must be strings"
                     )
-                if not EXTENSION_COMMAND_NAME_PATTERN.match(alias):
-                    corrected = self._try_correct_command_name(alias, ext["id"])
-                    if corrected:
-                        self.warnings.append(
-                            f"Alias '{alias}' does not follow the required pattern "
-                            f"'speckit.{{extension}}.{{command}}'. Registering as '{corrected}'. "
-                            f"The extension author should update the manifest to use this name."
-                        )
-                        rename_map[alias] = corrected
-                        aliases[i] = corrected
-                    else:
-                        raise ValidationError(
-                            f"Invalid alias '{alias}': "
-                            "must follow pattern 'speckit.{extension}.{command}'"
-                        )
 
         # Rewrite any hook command references that pointed at a renamed command or
         # an alias-form ref (ext.cmd → speckit.ext.cmd).  Always emit a warning when

tests/test_extensions.py

@@ -305,8 +305,8 @@ def test_command_name_mismatched_namespace_not_corrected(self, temp_dir, valid_m
         with pytest.raises(ValidationError, match="Invalid command name"):
             ExtensionManifest(manifest_path)
 
-    def test_alias_autocorrect_speckit_prefix(self, temp_dir, valid_manifest_data):
-        """Test that a legacy 'speckit.command' alias is auto-corrected."""
+    def test_alias_free_form_accepted(self, temp_dir, valid_manifest_data):
+        """Aliases are free-form — a 'speckit.command' alias must be accepted unchanged."""
         import yaml
 
         valid_manifest_data["provides"]["commands"][0]["aliases"] = ["speckit.hello"]
@@ -317,10 +317,8 @@ def test_alias_autocorrect_speckit_prefix(self, temp_dir, valid_manifest_data):
 
         manifest = ExtensionManifest(manifest_path)
 
-        assert manifest.commands[0]["aliases"] == ["speckit.test-ext.hello"]
-        assert len(manifest.warnings) == 1
-        assert "speckit.hello" in manifest.warnings[0]
-        assert "speckit.test-ext.hello" in manifest.warnings[0]
+        assert manifest.commands[0]["aliases"] == ["speckit.hello"]
+        assert manifest.warnings == []
 
     def test_valid_command_name_has_no_warnings(self, temp_dir, valid_manifest_data):
         """Test that a correctly-named command produces no warnings."""
@@ -778,8 +776,8 @@ def test_install_rejects_extension_id_in_core_namespace(self, temp_dir, project_
         with pytest.raises(ValidationError, match="conflicts with core command namespace"):
             manager.install_from_directory(ext_dir, "0.1.0", register_commands=False)
 
-    def test_install_autocorrects_alias_without_extension_namespace(self, temp_dir, project_dir):
-        """Legacy short aliases are auto-corrected to 'speckit.{ext_id}.{cmd}' with a warning."""
+    def test_install_accepts_free_form_alias(self, temp_dir, project_dir):
+        """Aliases are free-form — a short 'speckit.shortcut' alias must be preserved unchanged."""
         import yaml
 
         ext_dir = temp_dir / "alias-shortcut"
@@ -812,10 +810,8 @@ def test_install_autocorrects_alias_without_extension_namespace(self, temp_dir,
         manager = ExtensionManager(project_dir)
         manifest = manager.install_from_directory(ext_dir, "0.1.0", register_commands=False)
 
-        assert manifest.commands[0]["aliases"] == ["speckit.alias-shortcut.shortcut"]
-        assert len(manifest.warnings) == 1
-        assert "speckit.shortcut" in manifest.warnings[0]
-        assert "speckit.alias-shortcut.shortcut" in manifest.warnings[0]
+        assert manifest.commands[0]["aliases"] == ["speckit.shortcut"]
+        assert manifest.warnings == []
 
     def test_install_rejects_namespace_squatting(self, temp_dir, project_dir):
         """Install should reject commands and aliases outside the extension namespace."""