Merge pull request #217 from Gujiassh/fix/sync-version-consistency

fix: align v3 skill version metadata and sync target

scripts/sync.sh

@@ -11,7 +11,7 @@ COMMON_TARGETS=(
   # but local development needs the cache kept in sync with the repo.
   # Do NOT add ~/.claude/skills/last30days - it creates a duplicate
   # /last30days-3 in the slash command menu alongside the plugin version.
-  "$HOME/.claude/plugins/cache/last30days-skill-private/last30days-3/3.0.0-alpha"
+  "$HOME/.claude/plugins/cache/last30days-skill-private/last30days-3/3.0.0"
   "$HOME/.claude/plugins/cache/last30days-skill-private/last30days-3-nogem/3.0.0-nogem"
   "$HOME/.agents/skills/last30days"
   "$HOME/.codex/skills/last30days"

tests/test_version_consistency.py

@@ -0,0 +1,30 @@
+import re
+import unittest
+from pathlib import Path
+
+
+ROOT = Path(__file__).resolve().parents[1]
+
+
+def _skill_version() -> str:
+    text = (ROOT / "SKILL.md").read_text(encoding="utf-8")
+    match = re.search(r'^version:\s*"([^"]+)"\s*$', text, re.MULTILINE)
+    if not match:
+        raise AssertionError("SKILL.md version frontmatter not found")
+    return match.group(1)
+
+
+class TestVersionConsistency(unittest.TestCase):
+    def test_root_skill_header_matches_frontmatter_version(self) -> None:
+        text = (ROOT / "SKILL.md").read_text(encoding="utf-8")
+        version = _skill_version()
+        self.assertIn(f"# last30days v{version}:", text)
+
+    def test_sync_cache_path_uses_skill_version(self) -> None:
+        sync_text = (ROOT / "scripts" / "sync.sh").read_text(encoding="utf-8")
+        version = _skill_version()
+        self.assertIn(f'last30days-3/{version}"', sync_text)
+
+
+if __name__ == "__main__":
+    unittest.main()