agent-notes 2.0.4__py3-none-any.whl

tests/integration/test_build_output.py

@@ -0,0 +1,129 @@
+"""Integration tests that verify real build output in dist/."""
+import pytest
+from pathlib import Path
+
+
+def _parse_frontmatter(text: str) -> dict:
+    """Parse YAML frontmatter from markdown text. Returns dict of key/value strings."""
+    lines = text.split("\n")
+    if not lines or lines[0].strip() != "---":
+        return {}
+    end = None
+    for i in range(1, len(lines)):
+        if lines[i].strip() == "---":
+            end = i
+            break
+    if end is None:
+        return {}
+    result = {}
+    for line in lines[1:end]:
+        if ":" in line:
+            key, _, value = line.partition(":")
+            result[key.strip()] = value.strip()
+    return result
+
+
+# --- Directory structure ---
+
+def test_claude_agents_dir_exists(built_dist):
+    assert (built_dist / "claude" / "agents").is_dir()
+
+
+def test_claude_agents_has_enough_files(built_dist):
+    files = list((built_dist / "claude" / "agents").glob("*.md"))
+    assert len(files) >= 15
+
+
+def test_opencode_agents_dir_exists(built_dist):
+    assert (built_dist / "opencode" / "agents").is_dir()
+
+
+def test_opencode_agents_has_files(built_dist):
+    files = list((built_dist / "opencode" / "agents").glob("*.md"))
+    assert len(files) > 0
+
+
+def test_skills_dir_exists(built_dist):
+    assert (built_dist / "skills").is_dir()
+
+
+def test_skills_has_enough_subdirs(built_dist):
+    subdirs = [p for p in (built_dist / "skills").iterdir() if p.is_dir()]
+    assert len(subdirs) >= 30
+
+
+def test_rules_dir_exists(built_dist):
+    assert (built_dist / "rules").is_dir()
+
+
+def test_rules_has_markdown_files(built_dist):
+    files = list((built_dist / "rules").glob("*.md"))
+    assert len(files) >= 1
+
+
+# --- Scripts ---
+
+def test_cost_report_script_exists(built_dist):
+    assert (built_dist / "scripts" / "cost-report").exists()
+
+
+def test_cost_report_is_executable(built_dist):
+    import stat
+    script = built_dist / "scripts" / "cost-report"
+    mode = script.stat().st_mode
+    assert mode & stat.S_IXUSR
+
+
+def test_cost_report_contains_pricing_json(built_dist):
+    content = (built_dist / "scripts" / "cost-report").read_text()
+    assert '"providers"' in content
+
+
+def test_cost_report_has_no_placeholder(built_dist):
+    content = (built_dist / "scripts" / "cost-report").read_text()
+    assert "{{PRICING}}" not in content
+
+
+# --- Specific agent files ---
+
+def test_coder_md_exists(built_dist):
+    assert (built_dist / "claude" / "agents" / "coder.md").exists()
+
+
+def test_reviewer_md_exists(built_dist):
+    assert (built_dist / "claude" / "agents" / "reviewer.md").exists()
+
+
+def test_coder_frontmatter_name(built_dist):
+    text = (built_dist / "claude" / "agents" / "coder.md").read_text()
+    fm = _parse_frontmatter(text)
+    assert fm.get("name") == "coder"
+
+
+def test_coder_frontmatter_description(built_dist):
+    text = (built_dist / "claude" / "agents" / "coder.md").read_text()
+    fm = _parse_frontmatter(text)
+    assert fm.get("description", "").strip()
+
+
+def test_coder_frontmatter_model(built_dist):
+    text = (built_dist / "claude" / "agents" / "coder.md").read_text()
+    fm = _parse_frontmatter(text)
+    assert fm.get("model", "").strip()
+
+
+def test_coder_frontmatter_tools(built_dist):
+    text = (built_dist / "claude" / "agents" / "coder.md").read_text()
+    fm = _parse_frontmatter(text)
+    assert fm.get("tools", "").strip()
+
+
+# --- Global config file ---
+
+def test_claude_md_exists(built_dist):
+    assert (built_dist / "claude" / "CLAUDE.md").exists()
+
+
+def test_claude_md_is_non_empty(built_dist):
+    content = (built_dist / "claude" / "CLAUDE.md").read_text()
+    assert len(content.strip()) > 0

tests/plugins/test_agents.py

@@ -0,0 +1,93 @@
+"""Parametrized tests for every agent file in dist/claude/agents/."""
+import pytest
+from pathlib import Path
+
+from agent_notes.config import DIST_DIR
+
+_VALID_MODEL_KEYWORDS = {"haiku", "sonnet", "opus", "claude", "gpt", "gemini"}
+
+_CLAUDE_AGENTS_DIR = DIST_DIR / "claude" / "agents"
+
+
+def _agent_files():
+    if not _CLAUDE_AGENTS_DIR.is_dir():
+        return []
+    return sorted(_CLAUDE_AGENTS_DIR.glob("*.md"))
+
+
+def _parse_frontmatter(text: str) -> dict:
+    lines = text.split("\n")
+    if not lines or lines[0].strip() != "---":
+        return {}
+    end = None
+    for i in range(1, len(lines)):
+        if lines[i].strip() == "---":
+            end = i
+            break
+    if end is None:
+        return {}
+    result = {}
+    for line in lines[1:end]:
+        if ":" in line:
+            key, _, value = line.partition(":")
+            result[key.strip()] = value.strip().strip('"\'')
+    return result
+
+
+AGENT_FILES = _agent_files()
+
+
+@pytest.fixture(scope="module", autouse=True)
+def require_built_dist(built_dist):
+    """Ensure the build has run before collecting agent files."""
+    pass
+
+
+@pytest.mark.parametrize("agent_file", AGENT_FILES, ids=[f.name for f in AGENT_FILES])
+def test_agent_file_non_empty(agent_file):
+    assert agent_file.stat().st_size > 0, f"{agent_file.name} is empty"
+
+
+@pytest.mark.parametrize("agent_file", AGENT_FILES, ids=[f.name for f in AGENT_FILES])
+def test_agent_has_frontmatter_markers(agent_file):
+    text = agent_file.read_text()
+    lines = text.split("\n")
+    assert lines[0].strip() == "---", f"{agent_file.name} does not start with ---"
+    has_close = any(line.strip() == "---" for line in lines[1:])
+    assert has_close, f"{agent_file.name} has no closing ---"
+
+
+@pytest.mark.parametrize("agent_file", AGENT_FILES, ids=[f.name for f in AGENT_FILES])
+def test_agent_frontmatter_has_name(agent_file):
+    text = agent_file.read_text()
+    fm = _parse_frontmatter(text)
+    assert fm.get("name", "").strip(), f"{agent_file.name} missing 'name' in frontmatter"
+
+
+@pytest.mark.parametrize("agent_file", AGENT_FILES, ids=[f.name for f in AGENT_FILES])
+def test_agent_frontmatter_has_description(agent_file):
+    text = agent_file.read_text()
+    fm = _parse_frontmatter(text)
+    assert fm.get("description", "").strip(), f"{agent_file.name} missing 'description' in frontmatter"
+
+
+@pytest.mark.parametrize("agent_file", AGENT_FILES, ids=[f.name for f in AGENT_FILES])
+def test_agent_frontmatter_model_is_valid(agent_file):
+    text = agent_file.read_text()
+    fm = _parse_frontmatter(text)
+    model = fm.get("model", "").strip().lower()
+    assert model, f"{agent_file.name} missing 'model' in frontmatter"
+    assert any(kw in model for kw in _VALID_MODEL_KEYWORDS), (
+        f"{agent_file.name}: model '{model}' does not match any known keyword"
+    )
+
+
+@pytest.mark.parametrize("agent_file", AGENT_FILES, ids=[f.name for f in AGENT_FILES])
+def test_agent_frontmatter_has_tools(agent_file):
+    text = agent_file.read_text()
+    fm = _parse_frontmatter(text)
+    has_tools = fm.get("tools", "").strip()
+    has_disallowed = fm.get("disallowedTools", "").strip()
+    assert has_tools or has_disallowed, (
+        f"{agent_file.name} missing both 'tools' and 'disallowedTools' in frontmatter"
+    )

tests/plugins/test_skills.py

@@ -0,0 +1,77 @@
+"""Parametrized tests for every skill directory in data/skills/."""
+import pytest
+from pathlib import Path
+
+from agent_notes.config import SKILLS_DIR
+
+
+def _skill_dirs():
+    if not SKILLS_DIR.is_dir():
+        return []
+    return sorted(d for d in SKILLS_DIR.iterdir() if d.is_dir())
+
+
+def _parse_frontmatter(text: str) -> dict:
+    lines = text.split("\n")
+    if not lines or lines[0].strip() != "---":
+        return {}
+    end = None
+    for i in range(1, len(lines)):
+        if lines[i].strip() == "---":
+            end = i
+            break
+    if end is None:
+        return {}
+    result = {}
+    for line in lines[1:end]:
+        if ":" in line:
+            key, _, value = line.partition(":")
+            result[key.strip()] = value.strip().strip('"\'')
+    return result
+
+
+SKILL_DIRS = _skill_dirs()
+
+
+@pytest.mark.parametrize("skill_dir", SKILL_DIRS, ids=[d.name for d in SKILL_DIRS])
+def test_skill_md_exists(skill_dir):
+    assert (skill_dir / "SKILL.md").exists(), f"SKILL.md missing in {skill_dir.name}"
+
+
+@pytest.mark.parametrize("skill_dir", SKILL_DIRS, ids=[d.name for d in SKILL_DIRS])
+def test_skill_has_frontmatter(skill_dir):
+    text = (skill_dir / "SKILL.md").read_text()
+    lines = text.split("\n")
+    assert lines[0].strip() == "---", f"{skill_dir.name}/SKILL.md does not start with ---"
+    has_close = any(line.strip() == "---" for line in lines[1:])
+    assert has_close, f"{skill_dir.name}/SKILL.md has no closing ---"
+
+
+@pytest.mark.parametrize("skill_dir", SKILL_DIRS, ids=[d.name for d in SKILL_DIRS])
+def test_skill_frontmatter_has_name(skill_dir):
+    text = (skill_dir / "SKILL.md").read_text()
+    fm = _parse_frontmatter(text)
+    assert fm.get("name", "").strip(), f"{skill_dir.name}/SKILL.md missing non-empty 'name'"
+
+
+@pytest.mark.parametrize("skill_dir", SKILL_DIRS, ids=[d.name for d in SKILL_DIRS])
+def test_skill_frontmatter_has_description(skill_dir):
+    text = (skill_dir / "SKILL.md").read_text()
+    fm = _parse_frontmatter(text)
+    assert fm.get("description", "").strip(), f"{skill_dir.name}/SKILL.md missing non-empty 'description'"
+
+
+@pytest.mark.parametrize("skill_dir", SKILL_DIRS, ids=[d.name for d in SKILL_DIRS])
+def test_skill_frontmatter_has_group(skill_dir):
+    text = (skill_dir / "SKILL.md").read_text()
+    fm = _parse_frontmatter(text)
+    assert fm.get("group", "").strip(), f"{skill_dir.name}/SKILL.md missing non-empty 'group'"
+
+
+@pytest.mark.parametrize("skill_dir", SKILL_DIRS, ids=[d.name for d in SKILL_DIRS])
+def test_skill_name_matches_dir(skill_dir):
+    text = (skill_dir / "SKILL.md").read_text()
+    fm = _parse_frontmatter(text)
+    assert fm.get("name") == skill_dir.name, (
+        f"{skill_dir.name}/SKILL.md: 'name' field is '{fm.get('name')}', expected '{skill_dir.name}'"
+    )