xtrm-tools 2.1.5 → 2.1.7

package/skills/documenting/tests/test_changelog.py

@@ -1,201 +0,0 @@
-#!/usr/bin/env python3
-"""Tests for CHANGELOG management scripts."""
-
-import pytest
-from pathlib import Path
-from scripts.changelog.validate_changelog import validate_changelog
-from scripts.changelog.add_entry import add_entry, ChangeCategory
-from scripts.changelog.bump_release import bump_release
-from datetime import date
-import tempfile
-
-
-def test_valid_changelog_passes():
-    """Valid Keep a Changelog format should pass validation."""
-    valid_content = """# Changelog
-
-All notable changes to this project will be documented in this file.
-
-The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
-and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
-
-## [Unreleased]
-
-### Added
-- New feature description
-
-## [1.0.0] - 2026-02-01
-
-### Added
-- Initial release
-"""
-    result = validate_changelog(valid_content)
-    assert result["valid"] is True
-    assert len(result["errors"]) == 0
-
-
-def test_missing_unreleased_section_fails():
-    """CHANGELOG without [Unreleased] section should fail."""
-    invalid_content = """# Changelog
-
-## [1.0.0] - 2026-02-01
-
-### Added
-- Initial release
-"""
-    result = validate_changelog(invalid_content)
-    assert result["valid"] is False
-    assert any("Unreleased" in err for err in result["errors"])
-
-
-def test_invalid_semver_fails():
-    """CHANGELOG with invalid semver should fail."""
-    invalid_content = """# Changelog
-
-## [Unreleased]
-
-## [1.0] - 2026-02-01
-
-### Added
-- Initial release
-"""
-    result = validate_changelog(invalid_content)
-    assert result["valid"] is False
-    assert any("semantic version" in err.lower() for err in result["errors"])
-
-
-def test_invalid_category_fails():
-    """CHANGELOG with invalid category should fail."""
-    invalid_content = """# Changelog
-
-## [Unreleased]
-
-### InvalidCategory
-- Some change
-
-## [1.0.0] - 2026-02-01
-
-### Added
-- Initial release
-"""
-    result = validate_changelog(invalid_content)
-    assert result["valid"] is False
-    assert any("category" in err.lower() for err in result["errors"])
-
-
-def test_add_entry_to_unreleased():
-    """Adding entry should place it under [Unreleased] in correct category."""
-    changelog_content = """# Changelog
-
-## [Unreleased]
-
-## [1.0.0] - 2026-02-01
-
-### Added
-- Initial release
-"""
-
-    result = add_entry(
-        changelog_content,
-        category=ChangeCategory.ADDED,
-        description="New feature X"
-    )
-
-    assert "### Added" in result
-    assert "- New feature X" in result
-    # Should be under [Unreleased], not under [1.0.0]
-    unreleased_section = result.split("## [1.0.0]")[0]
-    assert "- New feature X" in unreleased_section
-
-
-def test_add_entry_creates_category_if_missing():
-    """Adding entry should create category section if it doesn't exist."""
-    changelog_content = """# Changelog
-
-## [Unreleased]
-
-## [1.0.0] - 2026-02-01
-
-### Added
-- Initial release
-"""
-
-    result = add_entry(
-        changelog_content,
-        category=ChangeCategory.FIXED,
-        description="Bug fix Y"
-    )
-
-    assert "### Fixed" in result
-    assert "- Bug fix Y" in result
-
-
-def test_add_entry_maintains_category_order():
-    """Categories should be ordered: Added, Changed, Deprecated, Removed, Fixed, Security."""
-    changelog_content = """# Changelog
-
-## [Unreleased]
-
-### Added
-- Feature A
-
-### Fixed
-- Bug B
-
-## [1.0.0] - 2026-02-01
-"""
-
-    result = add_entry(
-        changelog_content,
-        category=ChangeCategory.CHANGED,
-        description="Change C"
-    )
-
-    # Changed should appear between Added and Fixed
-    lines = result.split('\n')
-    added_idx = next(i for i, line in enumerate(lines) if "### Added" in line)
-    changed_idx = next(i for i, line in enumerate(lines) if "### Changed" in line)
-    fixed_idx = next(i for i, line in enumerate(lines) if "### Fixed" in line)
-
-    assert added_idx < changed_idx < fixed_idx
-
-
-def test_bump_release_moves_unreleased():
-    """Bumping release should move [Unreleased] to [X.Y.Z] - DATE."""
-    changelog_content = """# Changelog
-
-## [Unreleased]
-
-### Added
-- New feature
-
-## [1.0.0] - 2026-01-01
-
-### Added
-- Initial release
-"""
-
-    result = bump_release(changelog_content, "1.1.0")
-
-    # Should have new version section
-    assert "## [1.1.0]" in result
-    assert f"## [1.1.0] - {date.today().strftime('%Y-%m-%d')}" in result
-
-    # New feature should be under 1.1.0 now
-    version_section = result.split("## [1.0.0]")[0]
-    assert "- New feature" in version_section
-
-    # [Unreleased] should be empty
-    unreleased_section = result.split("## [1.1.0]")[0]
-    assert "## [Unreleased]" in unreleased_section
-    # No ### categories under [Unreleased]
-    unreleased_lines = unreleased_section.split("## [Unreleased]")[1].split('\\n')
-    assert not any(line.startswith("###") for line in unreleased_lines)
-
-
-def test_bump_release_validates_semver():
-    """Bumping with invalid semver should raise error."""
-    changelog_content = "## [Unreleased]"
-
-    with pytest.raises(ValueError, match="semantic version"):
-        bump_release(changelog_content, "1.0")

package/skills/documenting/tests/test_drift_detector.py

@@ -1,80 +0,0 @@
-import pytest
-from pathlib import Path
-import sys
-sys.path.insert(0, str(Path(__file__).parent.parent / "scripts"))
-from drift_detector import extract_tracks, match_files_to_tracks, format_scan_report
-
-
-MEMORY_WITH_TRACKS = """---
-title: Test SSOT
-version: 1.0.0
-updated: 2026-02-01
-tracks:
-  - "cli/src/**/*.ts"
-  - "hooks/**/*.py"
----
-
-## Purpose
-Test memory.
-"""
-
-MEMORY_NO_TRACKS = """---
-title: No Tracks
-version: 1.0.0
-updated: 2026-02-01
----
-
-## Purpose
-No tracking.
-"""
-
-
-def test_extract_tracks_finds_globs():
-    tracks = extract_tracks(MEMORY_WITH_TRACKS)
-    assert tracks == ["cli/src/**/*.ts", "hooks/**/*.py"]
-
-
-def test_extract_tracks_empty_when_missing():
-    tracks = extract_tracks(MEMORY_NO_TRACKS)
-    assert tracks == []
-
-
-def test_match_files_to_tracks_hit():
-    files = ["cli/src/core/diff.ts", "hooks/skill-suggestion.py"]
-    tracks = ["cli/src/**/*.ts", "hooks/**/*.py"]
-    matched = match_files_to_tracks(files, tracks)
-    assert "cli/src/core/diff.ts" in matched
-    assert "hooks/skill-suggestion.py" in matched
-
-
-def test_match_files_to_tracks_miss():
-    files = ["docs/README.md"]
-    tracks = ["cli/src/**/*.ts"]
-    assert match_files_to_tracks(files, tracks) == []
-
-
-def test_format_scan_report_stale():
-    stale = {"my_memory": {"files": ["cli/src/core/diff.ts"], "updated": "2026-02-01"}}
-    report = format_scan_report(stale)
-    assert "my_memory" in report
-    assert "cli/src/core/diff.ts" in report
-
-
-def test_format_scan_report_clean():
-    report = format_scan_report({})
-    assert any(word in report.lower() for word in ["clean", "no stale", "up to date"])
-
-
-def test_match_files_to_tracks_no_false_positive():
-    """A .py file in an unrelated dir must NOT match hooks/**/*.py"""
-    files = ["cli/src/transform.py", "docs/something.py"]
-    tracks = ["hooks/**/*.py"]
-    assert match_files_to_tracks(files, tracks) == []
-
-
-def test_match_files_to_tracks_direct_child():
-    """hooks/**/*.py must match files directly under hooks/ (no subdir)"""
-    files = ["hooks/skill-suggestion.py"]
-    tracks = ["hooks/**/*.py"]
-    matched = match_files_to_tracks(files, tracks)
-    assert "hooks/skill-suggestion.py" in matched

package/skills/documenting/tests/test_orchestrator.py

@@ -1,52 +0,0 @@
-#!/usr/bin/env python3
-"""Tests for documentation orchestrator."""
-
-import pytest
-from pathlib import Path
-from scripts.orchestrator import DocumentingOrchestrator, ChangeType
-
-
-def test_orchestrator_routes_to_correct_docs(tmp_path):
-    """Orchestrator should update relevant docs based on change type."""
-    # Setup test project
-    project_root = tmp_path / "test_project"
-    project_root.mkdir()
-
-    changelog = project_root / "CHANGELOG.md"
-    changelog.write_text("""# Changelog
-
-## [Unreleased]
-
-## [0.1.0] - 2026-02-01
-
-### Added
-- Initial release
-""", encoding="utf-8")
-
-    serena_memories = project_root / ".serena" / "memories"
-    serena_memories.mkdir(parents=True)
-
-    orchestrator = DocumentingOrchestrator(project_root)
-
-    result = orchestrator.document_change(
-        change_type=ChangeType.FEATURE,
-        description="Add new API endpoint",
-        details={
-            "scope": "api-endpoints",
-            "category": "backend",
-            "files_changed": ["server.py", "routes/api.py"]
-        }
-    )
-
-    # Should update CHANGELOG
-    assert result["changelog_updated"] is True
-    # Should create/update SSOT
-    assert result["ssot_updated"] is False # Placeholder impl in plan returns False
-    # Should suggest README update
-    assert "readme_suggestions" in result
-
-
-def test_orchestrator_validates_all_docs():
-    """Orchestrator should validate all documentation after updates."""
-    # Test validation integration
-    pass

package/skills/documenting/tests/test_validate_metadata.py

@@ -1,64 +0,0 @@
-import pytest
-from pathlib import Path
-import sys
-sys.path.insert(0, str(Path(__file__).parent.parent / "scripts"))
-from validate_metadata import extract_headings, generate_index_table, inject_index
-
-
-SAMPLE = """---
-title: Test
-version: 1.0.0
-updated: 2026-02-24
-scope: test
-category: ssot
-subcategory: test
-domain: [test]
----
-
-<!-- INDEX: auto-generated by validate_metadata.py — do not edit manually -->
-| Section | Summary |
-|---|---|
-| old | old |
-<!-- END INDEX -->
-
-## Architecture
-Handles the core routing logic for all requests.
-
-## Hook Wiring
-Three hooks fire on session events.
-
-## Installer
-Single-purpose script, idempotent.
-"""
-
-
-def test_extract_headings():
-    headings = extract_headings(SAMPLE)
-    assert headings == [
-        ("Architecture", "Handles the core routing logic for all requests"),
-        ("Hook Wiring", "Three hooks fire on session events"),
-        ("Installer", "Single-purpose script, idempotent"),
-    ]
-
-
-def test_generate_index_table():
-    headings = [("Architecture", "Core routing."), ("Installer", "Idempotent.")]
-    table = generate_index_table(headings)
-    assert "| [Architecture](#architecture) |" in table
-    assert "Core routing." in table
-    assert "| [Installer](#installer) |" in table
-
-
-def test_inject_index_replaces_existing():
-    new_table = "| Section | Summary |\n|---|---|\n| [Architecture](#architecture) | Core. |\n"
-    result = inject_index(SAMPLE, new_table)
-    assert "| old | old |" not in result
-    assert "| [Architecture](#architecture) | Core. |" in result
-
-
-def test_inject_index_adds_when_missing():
-    content = "---\ntitle: T\n---\n\n## Foo\nBar baz.\n"
-    new_table = "| Section | Summary |\n|---|---|\n| [Foo](#foo) | Bar baz. |\n"
-    result = inject_index(content, new_table)
-    assert "<!-- INDEX:" in result
-    assert "| [Foo](#foo) | Bar baz. |" in result