fix(skills): fix stale docstring, add invalid content test

Davide Gallitelli · claude · Davide Gallitelli · commit 8db72a7cd630 · 2026-04-14T04:21:28.000Z
- Fix stale "auto-resolved to raw content" in AgentSkills docstring
- Add test for non-SKILL.md content (HTML page → ValueError)
- Add from_url() example to Skill class docstring

Co-Authored-By: Claude Opus 4.6 (1M context) &lt;noreply@anthropic.com&gt;
diff --git a/src/strands/vended_plugins/skills/agent_skills.py b/src/strands/vended_plugins/skills/agent_skills.py
@@ -86,8 +86,7 @@ def __init__(
                 - A ``str`` or ``Path`` to a skill directory (containing SKILL.md)
                 - A ``str`` or ``Path`` to a parent directory (containing skill subdirectories)
                 - A ``Skill`` dataclass instance
-                - An ``https://`` URL pointing to a SKILL.md file or a GitHub
-                  repository/directory URL (auto-resolved to raw content)
+                - An ``https://`` URL pointing directly to raw SKILL.md content
             state_key: Key used to store plugin state in ``agent.state``.
             max_resource_files: Maximum number of resource files to list in skill responses.
             strict: If True, raise on skill validation issues. If False (default), warn and load anyway.
diff --git a/src/strands/vended_plugins/skills/skill.py b/src/strands/vended_plugins/skills/skill.py
@@ -224,6 +224,9 @@ class Skill:
         # Load all skills from a parent directory
         skills = Skill.from_directory("./skills/")
 
+        # From an HTTPS URL
+        skill = Skill.from_url("https://example.com/SKILL.md")
+
     Attributes:
         name: Unique identifier for the skill (1-64 chars, lowercase alphanumeric + hyphens).
         description: Human-readable description of what the skill does.
diff --git a/tests/strands/vended_plugins/skills/test_skill.py b/tests/strands/vended_plugins/skills/test_skill.py
@@ -627,6 +627,16 @@ def test_from_url_strict_mode(self):
             with pytest.raises(ValueError):
                 Skill.from_url("https://example.com/SKILL.md", strict=True)
 
+    def test_from_url_invalid_content_raises(self):
+        """Test that non-SKILL.md content (e.g. HTML page) raises ValueError."""
+        from unittest.mock import patch
+
+        html_content = "<html><body>Not a SKILL.md</body></html>"
+
+        with patch(f"{self._SKILL_MODULE}.urllib.request.urlopen", return_value=self._mock_urlopen(html_content)):
+            with pytest.raises(ValueError, match="frontmatter"):
+                Skill.from_url("https://example.com/SKILL.md")
+
 
 class TestSkillClassmethods:
     """Tests for Skill classmethod existence."""
