@pennyfarthing/core 11.3.3 → 11.3.5

package/{pennyfarthing_scripts → pennyfarthing-dist/pf}/tests/test_tiers.py

@@ -28,7 +28,7 @@ class TestTierCLIArgument:
 
     def test_tier_argument_accepts_full(self, tmp_path: Path) -> None:
         """Test --tier FULL is accepted."""
-        from pennyfarthing_scripts.prime.cli import main
+        from pf.prime.cli import main
 
         pf_dir = tmp_path / ".pennyfarthing"
         pf_dir.mkdir()
@@ -36,14 +36,14 @@ class TestTierCLIArgument:
         agents_dir.mkdir()
         (agents_dir / "dev.md").write_text("# Dev Agent")
 
-        with patch("pennyfarthing_scripts.prime.cli.get_project_root", return_value=tmp_path):
+        with patch("pf.prime.cli.get_project_root", return_value=tmp_path):
             result = main(["--agent", "dev", "--tier", "FULL", "--no-workflow", "--no-register"])
 
         assert result == 0
 
     def test_tier_argument_accepts_refresh(self, tmp_path: Path) -> None:
         """Test --tier REFRESH is accepted."""
-        from pennyfarthing_scripts.prime.cli import main
+        from pf.prime.cli import main
 
         pf_dir = tmp_path / ".pennyfarthing"
         pf_dir.mkdir()
@@ -51,14 +51,14 @@ class TestTierCLIArgument:
         agents_dir.mkdir()
         (agents_dir / "dev.md").write_text("# Dev Agent")
 
-        with patch("pennyfarthing_scripts.prime.cli.get_project_root", return_value=tmp_path):
+        with patch("pf.prime.cli.get_project_root", return_value=tmp_path):
             result = main(["--agent", "dev", "--tier", "REFRESH", "--no-workflow", "--no-register"])
 
         assert result == 0
 
     def test_tier_argument_accepts_handoff(self, tmp_path: Path) -> None:
         """Test --tier HANDOFF is accepted."""
-        from pennyfarthing_scripts.prime.cli import main
+        from pf.prime.cli import main
 
         pf_dir = tmp_path / ".pennyfarthing"
         pf_dir.mkdir()
@@ -66,14 +66,14 @@ class TestTierCLIArgument:
         agents_dir.mkdir()
         (agents_dir / "dev.md").write_text("# Dev Agent")
 
-        with patch("pennyfarthing_scripts.prime.cli.get_project_root", return_value=tmp_path):
+        with patch("pf.prime.cli.get_project_root", return_value=tmp_path):
             result = main(["--agent", "dev", "--tier", "HANDOFF", "--no-workflow", "--no-register"])
 
         assert result == 0
 
     def test_tier_argument_accepts_minimal(self, tmp_path: Path) -> None:
         """Test --tier MINIMAL is accepted."""
-        from pennyfarthing_scripts.prime.cli import main
+        from pf.prime.cli import main
 
         pf_dir = tmp_path / ".pennyfarthing"
         pf_dir.mkdir()
@@ -81,14 +81,14 @@ class TestTierCLIArgument:
         agents_dir.mkdir()
         (agents_dir / "dev.md").write_text("# Dev Agent")
 
-        with patch("pennyfarthing_scripts.prime.cli.get_project_root", return_value=tmp_path):
+        with patch("pf.prime.cli.get_project_root", return_value=tmp_path):
             result = main(["--agent", "dev", "--tier", "MINIMAL", "--no-workflow", "--no-register"])
 
         assert result == 0
 
     def test_tier_argument_rejects_invalid_value(self, tmp_path: Path) -> None:
         """Test --tier rejects invalid values."""
-        from pennyfarthing_scripts.prime.cli import main
+        from pf.prime.cli import main
 
         with pytest.raises(SystemExit) as exc_info:
             main(["--tier", "INVALID"])
@@ -98,7 +98,7 @@ class TestTierCLIArgument:
 
     def test_tier_argument_case_insensitive(self, tmp_path: Path) -> None:
         """Test --tier accepts lowercase values."""
-        from pennyfarthing_scripts.prime.cli import main
+        from pf.prime.cli import main
 
         pf_dir = tmp_path / ".pennyfarthing"
         pf_dir.mkdir()
@@ -106,7 +106,7 @@ class TestTierCLIArgument:
         agents_dir.mkdir()
         (agents_dir / "dev.md").write_text("# Dev Agent")
 
-        with patch("pennyfarthing_scripts.prime.cli.get_project_root", return_value=tmp_path):
+        with patch("pf.prime.cli.get_project_root", return_value=tmp_path):
             # Should accept lowercase
             result = main(["--agent", "dev", "--tier", "full", "--no-workflow", "--no-register"])
 
@@ -123,13 +123,13 @@ class TestTierComponentLoading:
 
     def test_full_tier_loads_all_components(self, tmp_path: Path, capsys) -> None:
         """Test FULL tier loads all 10 components."""
-        from pennyfarthing_scripts.prime.cli import prime
+        from pf.prime.cli import prime
 
         # Setup complete project structure
         self._setup_full_project(tmp_path)
 
-        with patch("pennyfarthing_scripts.prime.cli.get_project_root", return_value=tmp_path):
-            with patch("pennyfarthing_scripts.prime.loader.get_project_root", return_value=tmp_path):
+        with patch("pf.prime.cli.get_project_root", return_value=tmp_path):
+            with patch("pf.prime.loader.get_project_root", return_value=tmp_path):
                 result = prime(
                     agent_name="dev",
                     tier="FULL",
@@ -148,13 +148,13 @@ class TestTierComponentLoading:
 
     def test_refresh_tier_loads_dynamic_only(self, tmp_path: Path, capsys) -> None:
         """Test REFRESH tier loads only dynamic state (~600 tokens)."""
-        from pennyfarthing_scripts.prime.cli import prime
+        from pf.prime.cli import prime
 
         # Setup
         self._setup_full_project(tmp_path)
 
-        with patch("pennyfarthing_scripts.prime.cli.get_project_root", return_value=tmp_path):
-            with patch("pennyfarthing_scripts.prime.loader.get_project_root", return_value=tmp_path):
+        with patch("pf.prime.cli.get_project_root", return_value=tmp_path):
+            with patch("pf.prime.loader.get_project_root", return_value=tmp_path):
                 result = prime(
                     agent_name="dev",
                     tier="REFRESH",
@@ -182,13 +182,13 @@ class TestTierComponentLoading:
 
     def test_handoff_tier_loads_agent_essentials(self, tmp_path: Path, capsys) -> None:
         """Test HANDOFF tier loads agent definition + compressed persona (~700 tokens)."""
-        from pennyfarthing_scripts.prime.cli import prime
+        from pf.prime.cli import prime
 
         # Setup
         self._setup_full_project(tmp_path)
 
-        with patch("pennyfarthing_scripts.prime.cli.get_project_root", return_value=tmp_path):
-            with patch("pennyfarthing_scripts.prime.loader.get_project_root", return_value=tmp_path):
+        with patch("pf.prime.cli.get_project_root", return_value=tmp_path):
+            with patch("pf.prime.loader.get_project_root", return_value=tmp_path):
                 result = prime(
                     agent_name="dev",
                     tier="HANDOFF",
@@ -219,13 +219,13 @@ class TestTierComponentLoading:
 
     def test_minimal_tier_loads_workflow_only(self, tmp_path: Path, capsys) -> None:
         """Test MINIMAL tier loads only workflow state (~200 tokens)."""
-        from pennyfarthing_scripts.prime.cli import prime
+        from pf.prime.cli import prime
 
         # Setup
         self._setup_full_project(tmp_path)
 
-        with patch("pennyfarthing_scripts.prime.cli.get_project_root", return_value=tmp_path):
-            with patch("pennyfarthing_scripts.prime.loader.get_project_root", return_value=tmp_path):
+        with patch("pf.prime.cli.get_project_root", return_value=tmp_path):
+            with patch("pf.prime.loader.get_project_root", return_value=tmp_path):
                 result = prime(
                     agent_name="dev",
                     tier="MINIMAL",
@@ -315,8 +315,8 @@ class TestCompressedPersonaFormat:
     def test_compressed_persona_has_xml_structure(self, tmp_path: Path) -> None:
         """Test compressed persona uses XML format."""
         # Import will fail until function is implemented
-        from pennyfarthing_scripts.prime.models import Persona
-        from pennyfarthing_scripts.prime.persona import format_persona_compressed
+        from pf.prime.models import Persona
+        from pf.prime.persona import format_persona_compressed
 
         persona = Persona(
             character="Rosie the Riveter",
@@ -333,8 +333,8 @@ class TestCompressedPersonaFormat:
 
     def test_compressed_persona_includes_voice(self, tmp_path: Path) -> None:
         """Test compressed persona includes voice element."""
-        from pennyfarthing_scripts.prime.models import Persona
-        from pennyfarthing_scripts.prime.persona import format_persona_compressed
+        from pf.prime.models import Persona
+        from pf.prime.persona import format_persona_compressed
 
         persona = Persona(
             character="Rosie the Riveter",
@@ -349,8 +349,8 @@ class TestCompressedPersonaFormat:
 
     def test_compressed_persona_includes_style(self, tmp_path: Path) -> None:
         """Test compressed persona includes style element."""
-        from pennyfarthing_scripts.prime.models import Persona
-        from pennyfarthing_scripts.prime.persona import format_persona_compressed
+        from pf.prime.models import Persona
+        from pf.prime.persona import format_persona_compressed
 
         persona = Persona(
             character="Rosie the Riveter",
@@ -364,8 +364,8 @@ class TestCompressedPersonaFormat:
 
     def test_compressed_persona_includes_catchphrase_if_present(self, tmp_path: Path) -> None:
         """Test compressed persona includes catchphrase from quote."""
-        from pennyfarthing_scripts.prime.models import Persona
-        from pennyfarthing_scripts.prime.persona import format_persona_compressed
+        from pf.prime.models import Persona
+        from pf.prime.persona import format_persona_compressed
 
         persona = Persona(
             character="Rosie the Riveter",
@@ -381,8 +381,8 @@ class TestCompressedPersonaFormat:
 
     def test_compressed_persona_token_count(self, tmp_path: Path) -> None:
         """Test compressed persona is approximately 100 tokens (~400 chars)."""
-        from pennyfarthing_scripts.prime.models import Persona
-        from pennyfarthing_scripts.prime.persona import format_persona_compressed
+        from pf.prime.models import Persona
+        from pf.prime.persona import format_persona_compressed
 
         persona = Persona(
             character="Rosie the Riveter",
@@ -401,8 +401,8 @@ class TestCompressedPersonaFormat:
 
     def test_full_persona_token_count(self, tmp_path: Path) -> None:
         """Test full persona is approximately 300 tokens (~1200 chars)."""
-        from pennyfarthing_scripts.prime.models import CrewMember, Persona
-        from pennyfarthing_scripts.prime.persona import format_persona_output
+        from pf.prime.models import CrewMember, Persona
+        from pf.prime.persona import format_persona_output
 
         persona = Persona(
             character="Rosie the Riveter",
@@ -434,7 +434,7 @@ class TestDefaultBehavior:
 
     def test_no_tier_argument_defaults_to_full(self, tmp_path: Path, capsys) -> None:
         """Test that omitting --tier defaults to FULL behavior."""
-        from pennyfarthing_scripts.prime.cli import prime
+        from pf.prime.cli import prime
 
         # Setup
         pf_dir = tmp_path / ".pennyfarthing"
@@ -451,8 +451,8 @@ class TestDefaultBehavior:
         sidecar_dir.mkdir(parents=True)
         (sidecar_dir / "patterns.md").write_text("# Patterns")
 
-        with patch("pennyfarthing_scripts.prime.cli.get_project_root", return_value=tmp_path):
-            with patch("pennyfarthing_scripts.prime.loader.get_project_root", return_value=tmp_path):
+        with patch("pf.prime.cli.get_project_root", return_value=tmp_path):
+            with patch("pf.prime.loader.get_project_root", return_value=tmp_path):
                 # No --tier argument
                 result = prime(
                     agent_name="dev",
@@ -471,7 +471,7 @@ class TestDefaultBehavior:
 
     def test_explicit_full_tier_matches_default(self, tmp_path: Path, capsys) -> None:
         """Test that --tier FULL produces same output as default."""
-        from pennyfarthing_scripts.prime.cli import prime
+        from pf.prime.cli import prime
 
         # Setup
         pf_dir = tmp_path / ".pennyfarthing"
@@ -480,8 +480,8 @@ class TestDefaultBehavior:
         agents_dir.mkdir()
         (agents_dir / "dev.md").write_text("# Dev Agent")
 
-        with patch("pennyfarthing_scripts.prime.cli.get_project_root", return_value=tmp_path):
-            with patch("pennyfarthing_scripts.prime.loader.get_project_root", return_value=tmp_path):
+        with patch("pf.prime.cli.get_project_root", return_value=tmp_path):
+            with patch("pf.prime.loader.get_project_root", return_value=tmp_path):
                 # Run without tier
                 result1 = prime(
                     agent_name="dev",
@@ -510,7 +510,7 @@ class TestDefaultBehavior:
         """Test existing flags (--quiet, --json, etc.) work with --tier."""
         import json
 
-        from pennyfarthing_scripts.prime.cli import prime
+        from pf.prime.cli import prime
 
         # Setup
         pf_dir = tmp_path / ".pennyfarthing"
@@ -519,7 +519,7 @@ class TestDefaultBehavior:
         agents_dir.mkdir()
         (agents_dir / "dev.md").write_text("# Dev Agent")
 
-        with patch("pennyfarthing_scripts.prime.cli.get_project_root", return_value=tmp_path):
+        with patch("pf.prime.cli.get_project_root", return_value=tmp_path):
             result = prime(
                 agent_name="dev",
                 tier="REFRESH",
@@ -548,7 +548,7 @@ class TestTierLoadingPaths:
 
     def test_full_tier_with_all_options(self, tmp_path: Path) -> None:
         """Test FULL tier with all context sources available."""
-        from pennyfarthing_scripts.prime.tiers import ContextTier, load_tier_components
+        from pf.prime.tiers import ContextTier, load_tier_components
 
         # Setup complete project
         self._setup_complete_project(tmp_path)
@@ -569,7 +569,7 @@ class TestTierLoadingPaths:
 
     def test_full_tier_with_missing_optional(self, tmp_path: Path) -> None:
         """Test FULL tier gracefully handles missing optional components."""
-        from pennyfarthing_scripts.prime.tiers import ContextTier, load_tier_components
+        from pf.prime.tiers import ContextTier, load_tier_components
 
         # Minimal setup - only agent definition
         pf_dir = tmp_path / ".pennyfarthing"
@@ -590,7 +590,7 @@ class TestTierLoadingPaths:
 
     def test_refresh_tier_components(self, tmp_path: Path) -> None:
         """Test REFRESH tier returns correct component set."""
-        from pennyfarthing_scripts.prime.tiers import ContextTier, load_tier_components
+        from pf.prime.tiers import ContextTier, load_tier_components
 
         self._setup_complete_project(tmp_path)
 
@@ -612,7 +612,7 @@ class TestTierLoadingPaths:
 
     def test_handoff_tier_components(self, tmp_path: Path) -> None:
         """Test HANDOFF tier returns correct component set."""
-        from pennyfarthing_scripts.prime.tiers import ContextTier, load_tier_components
+        from pf.prime.tiers import ContextTier, load_tier_components
 
         self._setup_complete_project(tmp_path)
 
@@ -633,7 +633,7 @@ class TestTierLoadingPaths:
 
     def test_minimal_tier_components(self, tmp_path: Path) -> None:
         """Test MINIMAL tier returns minimal component set."""
-        from pennyfarthing_scripts.prime.tiers import ContextTier, load_tier_components
+        from pf.prime.tiers import ContextTier, load_tier_components
 
         self._setup_complete_project(tmp_path)
 
@@ -654,7 +654,7 @@ class TestTierLoadingPaths:
 
     def test_tier_enum_values(self) -> None:
         """Test ContextTier enum has all required values."""
-        from pennyfarthing_scripts.prime.tiers import ContextTier
+        from pf.prime.tiers import ContextTier
 
         assert hasattr(ContextTier, "FULL")
         assert hasattr(ContextTier, "REFRESH")
@@ -663,7 +663,7 @@ class TestTierLoadingPaths:
 
     def test_tier_from_string(self) -> None:
         """Test converting string to ContextTier."""
-        from pennyfarthing_scripts.prime.tiers import ContextTier, tier_from_string
+        from pf.prime.tiers import ContextTier, tier_from_string
 
         assert tier_from_string("FULL") == ContextTier.FULL
         assert tier_from_string("REFRESH") == ContextTier.REFRESH
@@ -726,13 +726,13 @@ class TestTokenReduction:
 
     def test_refresh_tier_under_800_tokens(self, tmp_path: Path, capsys) -> None:
         """Test REFRESH tier output is under 800 tokens (~3200 chars)."""
-        from pennyfarthing_scripts.prime.cli import prime
+        from pf.prime.cli import prime
 
         # Setup with realistic content
         self._setup_realistic_project(tmp_path)
 
-        with patch("pennyfarthing_scripts.prime.cli.get_project_root", return_value=tmp_path):
-            with patch("pennyfarthing_scripts.prime.loader.get_project_root", return_value=tmp_path):
+        with patch("pf.prime.cli.get_project_root", return_value=tmp_path):
+            with patch("pf.prime.loader.get_project_root", return_value=tmp_path):
                 result = prime(
                     agent_name="dev",
                     tier="REFRESH",
@@ -748,13 +748,13 @@ class TestTokenReduction:
 
     def test_handoff_tier_under_900_tokens(self, tmp_path: Path, capsys) -> None:
         """Test HANDOFF tier output is under 900 tokens (~3600 chars)."""
-        from pennyfarthing_scripts.prime.cli import prime
+        from pf.prime.cli import prime
 
         # Setup with realistic content
         self._setup_realistic_project(tmp_path)
 
-        with patch("pennyfarthing_scripts.prime.cli.get_project_root", return_value=tmp_path):
-            with patch("pennyfarthing_scripts.prime.loader.get_project_root", return_value=tmp_path):
+        with patch("pf.prime.cli.get_project_root", return_value=tmp_path):
+            with patch("pf.prime.loader.get_project_root", return_value=tmp_path):
                 result = prime(
                     agent_name="dev",
                     tier="HANDOFF",
@@ -770,13 +770,13 @@ class TestTokenReduction:
 
     def test_minimal_tier_under_300_tokens(self, tmp_path: Path, capsys) -> None:
         """Test MINIMAL tier output is under 300 tokens (~1200 chars)."""
-        from pennyfarthing_scripts.prime.cli import prime
+        from pf.prime.cli import prime
 
         # Setup with realistic content
         self._setup_realistic_project(tmp_path)
 
-        with patch("pennyfarthing_scripts.prime.cli.get_project_root", return_value=tmp_path):
-            with patch("pennyfarthing_scripts.prime.loader.get_project_root", return_value=tmp_path):
+        with patch("pf.prime.cli.get_project_root", return_value=tmp_path):
+            with patch("pf.prime.loader.get_project_root", return_value=tmp_path):
                 result = prime(
                     agent_name="dev",
                     tier="MINIMAL",
@@ -792,13 +792,13 @@ class TestTokenReduction:
 
     def test_full_tier_approximately_4000_tokens(self, tmp_path: Path, capsys) -> None:
         """Test FULL tier output is approximately 4000 tokens (~16000 chars)."""
-        from pennyfarthing_scripts.prime.cli import prime
+        from pf.prime.cli import prime
 
         # Setup with realistic content
         self._setup_realistic_project(tmp_path)
 
-        with patch("pennyfarthing_scripts.prime.cli.get_project_root", return_value=tmp_path):
-            with patch("pennyfarthing_scripts.prime.loader.get_project_root", return_value=tmp_path):
+        with patch("pf.prime.cli.get_project_root", return_value=tmp_path):
+            with patch("pf.prime.loader.get_project_root", return_value=tmp_path):
                 result = prime(
                     agent_name="dev",
                     tier="FULL",
@@ -815,14 +815,14 @@ class TestTokenReduction:
 
     def test_tier_reduction_ratio(self, tmp_path: Path, capsys) -> None:
         """Test that reduced tiers are significantly smaller than FULL."""
-        from pennyfarthing_scripts.prime.cli import prime
+        from pf.prime.cli import prime
 
         self._setup_realistic_project(tmp_path)
 
         sizes = {}
         for tier in ["FULL", "REFRESH", "HANDOFF", "MINIMAL"]:
-            with patch("pennyfarthing_scripts.prime.cli.get_project_root", return_value=tmp_path):
-                with patch("pennyfarthing_scripts.prime.loader.get_project_root", return_value=tmp_path):
+            with patch("pf.prime.cli.get_project_root", return_value=tmp_path):
+                with patch("pf.prime.loader.get_project_root", return_value=tmp_path):
                     prime(
                         agent_name="dev",
                         tier=tier,
@@ -992,7 +992,7 @@ class TestTierIntegration:
         """Test JSON output includes current tier."""
         import json
 
-        from pennyfarthing_scripts.prime.cli import prime
+        from pf.prime.cli import prime
 
         pf_dir = tmp_path / ".pennyfarthing"
         pf_dir.mkdir()
@@ -1000,7 +1000,7 @@ class TestTierIntegration:
         agents_dir.mkdir()
         (agents_dir / "dev.md").write_text("# Dev Agent")
 
-        with patch("pennyfarthing_scripts.prime.cli.get_project_root", return_value=tmp_path):
+        with patch("pf.prime.cli.get_project_root", return_value=tmp_path):
             result = prime(
                 agent_name="dev",
                 tier="REFRESH",
@@ -1019,7 +1019,7 @@ class TestTierIntegration:
 
     def test_tier_with_workflow_detection(self, tmp_path: Path, capsys) -> None:
         """Test tier works correctly with workflow detection enabled."""
-        from pennyfarthing_scripts.prime.cli import prime
+        from pf.prime.cli import prime
 
         pf_dir = tmp_path / ".pennyfarthing"
         pf_dir.mkdir()
@@ -1034,8 +1034,8 @@ class TestTierIntegration:
             "epics": []
         }))
 
-        with patch("pennyfarthing_scripts.prime.cli.get_project_root", return_value=tmp_path):
-            with patch("pennyfarthing_scripts.prime.loader.get_project_root", return_value=tmp_path):
+        with patch("pf.prime.cli.get_project_root", return_value=tmp_path):
+            with patch("pf.prime.loader.get_project_root", return_value=tmp_path):
                 result = prime(
                     agent_name="dev",
                     tier="MINIMAL",
@@ -1051,7 +1051,7 @@ class TestTierIntegration:
 
     def test_tier_with_redirect_detection(self, tmp_path: Path, capsys) -> None:
         """Test tier works correctly with redirect detection."""
-        from pennyfarthing_scripts.prime.cli import prime
+        from pf.prime.cli import prime
 
         pf_dir = tmp_path / ".pennyfarthing"
         pf_dir.mkdir()
@@ -1075,7 +1075,7 @@ class TestTierIntegration:
 - **Current Phase:** green
 """)
 
-        with patch("pennyfarthing_scripts.prime.cli.get_project_root", return_value=tmp_path):
+        with patch("pf.prime.cli.get_project_root", return_value=tmp_path):
             result = prime(
                 agent_name="tea",
                 tier="HANDOFF",