@pennyfarthing/core 11.3.3 → 11.3.5

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

@@ -1,6 +1,6 @@
 """Tests for common/ shared utilities package.
 
-Story 63-9: Reorganize pennyfarthing_scripts into fan-out CLI pattern.
+Story 63-9: Reorganize pf into fan-out CLI pattern.
 
 These tests verify the common/ package provides shared utilities
 that work correctly when imported from the new location.
@@ -20,7 +20,7 @@ class TestOutputModule:
 
     def test_success_prints_green_prefix(self) -> None:
         """success() should print with [OK] prefix."""
-        from pennyfarthing_scripts.common import output
+        from pf.common import output
 
         buffer = io.StringIO()
         # Force color support for testing
@@ -33,7 +33,7 @@ class TestOutputModule:
 
     def test_info_prints_blue_prefix(self) -> None:
         """info() should print with [INFO] prefix."""
-        from pennyfarthing_scripts.common import output
+        from pf.common import output
 
         buffer = io.StringIO()
         output.info("Info message", file=buffer)
@@ -43,7 +43,7 @@ class TestOutputModule:
 
     def test_warn_prints_yellow_prefix(self) -> None:
         """warn() should print with [WARN] prefix."""
-        from pennyfarthing_scripts.common import output
+        from pf.common import output
 
         buffer = io.StringIO()
         output.warn("Warning message", file=buffer)
@@ -53,7 +53,7 @@ class TestOutputModule:
 
     def test_error_prints_red_prefix(self) -> None:
         """error() should print with [ERROR] prefix."""
-        from pennyfarthing_scripts.common import output
+        from pf.common import output
 
         buffer = io.StringIO()
         output.error("Error message", file=buffer)
@@ -63,7 +63,7 @@ class TestOutputModule:
 
     def test_no_color_env_disables_colors(self) -> None:
         """NO_COLOR environment variable should disable colors."""
-        from pennyfarthing_scripts.common import output
+        from pf.common import output
 
         with patch.dict(os.environ, {"NO_COLOR": "1"}):
             buffer = io.StringIO()
@@ -74,7 +74,7 @@ class TestOutputModule:
 
     def test_force_color_env_enables_colors(self) -> None:
         """FORCE_COLOR environment variable should enable colors."""
-        from pennyfarthing_scripts.common import output
+        from pf.common import output
 
         with patch.dict(os.environ, {"FORCE_COLOR": "1"}, clear=False):
             # Remove NO_COLOR if present
@@ -83,7 +83,7 @@ class TestOutputModule:
 
     def test_header_prints_decorated_line(self) -> None:
         """header() should print decorated header."""
-        from pennyfarthing_scripts.common import output
+        from pf.common import output
 
         buffer = io.StringIO()
         output.header("Test Header", char="=", width=40, file=buffer)
@@ -94,7 +94,7 @@ class TestOutputModule:
 
     def test_divider_prints_line(self) -> None:
         """divider() should print a line."""
-        from pennyfarthing_scripts.common import output
+        from pf.common import output
 
         buffer = io.StringIO()
         output.divider(char="-", width=20, file=buffer)
@@ -108,7 +108,7 @@ class TestConfigModule:
 
     def test_get_project_root_finds_pennyfarthing_dir(self) -> None:
         """get_project_root() should find .pennyfarthing directory."""
-        from pennyfarthing_scripts.common import config
+        from pf.common import config
 
         # This test assumes we're running from within the pennyfarthing project
         root = config.get_project_root()
@@ -117,7 +117,7 @@ class TestConfigModule:
 
     def test_get_project_root_raises_if_not_found(self) -> None:
         """get_project_root() should raise if no .pennyfarthing found."""
-        from pennyfarthing_scripts.common import config
+        from pf.common import config
 
         # Clear env vars that bypass the directory walk
         env_overrides = {"PROJECT_ROOT": "", "CLAUDE_PROJECT_DIR": ""}
@@ -131,7 +131,7 @@ class TestConfigModule:
 
     def test_load_yaml_config_returns_dict(self) -> None:
         """load_yaml_config() should return parsed YAML as dict."""
-        from pennyfarthing_scripts.common import config
+        from pf.common import config
 
         # Test with existing sprint file
         root = config.get_project_root()
@@ -143,20 +143,20 @@ class TestConfigModule:
 
     def test_load_yaml_config_returns_none_if_missing(self) -> None:
         """load_yaml_config() should return None for missing files."""
-        from pennyfarthing_scripts.common import config
+        from pf.common import config
 
         result = config.load_yaml_config(Path("/nonexistent/file.yaml"))
         assert result is None
 
     def test_find_project_root_alias(self) -> None:
         """find_project_root should be an alias for get_project_root."""
-        from pennyfarthing_scripts.common import config
+        from pf.common import config
 
         assert config.find_project_root == config.get_project_root
 
     def test_load_pennyfarthing_config_returns_dict(self) -> None:
         """load_pennyfarthing_config() should return config or empty dict."""
-        from pennyfarthing_scripts.common import config
+        from pf.common import config
 
         result = config.load_pennyfarthing_config()
         assert isinstance(result, dict)
@@ -167,7 +167,7 @@ class TestColorsClass:
 
     def test_colors_has_expected_constants(self) -> None:
         """Colors class should have standard ANSI color codes."""
-        from pennyfarthing_scripts.common.output import Colors
+        from pf.common.output import Colors
 
         assert hasattr(Colors, "RED")
         assert hasattr(Colors, "GREEN")
@@ -179,7 +179,7 @@ class TestColorsClass:
 
     def test_colors_are_ansi_escape_codes(self) -> None:
         """Color constants should be ANSI escape sequences."""
-        from pennyfarthing_scripts.common.output import Colors
+        from pf.common.output import Colors
 
         assert Colors.RED.startswith("\x1b[")
         assert Colors.RESET == "\x1b[0m"

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

@@ -28,7 +28,7 @@ import pytest
 # ---------------------------------------------------------------------------
 
 _THIS_DIR = Path(__file__).resolve().parent
-_SCRIPTS_DIR = _THIS_DIR.parent  # pennyfarthing_scripts/
+_SCRIPTS_DIR = _THIS_DIR.parent  # pf/
 _FRAMEWORK_ROOT = _SCRIPTS_DIR.parent  # pennyfarthing/
 _EVAL_FILE = (
     _FRAMEWORK_ROOT

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

@@ -25,8 +25,8 @@ from pathlib import Path
 
 import pytest
 
-from pennyfarthing_scripts.handoff.gate_file import resolve_gate_file
-from pennyfarthing_scripts.handoff.gate_runner import parse_gate_file
+from pf.handoff.gate_file import resolve_gate_file
+from pf.handoff.gate_runner import parse_gate_file
 
 # ---------------------------------------------------------------------------
 # Fixtures
@@ -38,7 +38,7 @@ GATE_NAME = "confidence"
 # In the dogfooding context, the project root is the orchestrator, so we need
 # to resolve paths relative to this test file.
 _THIS_DIR = Path(__file__).resolve().parent
-_SCRIPTS_DIR = _THIS_DIR.parent  # pennyfarthing_scripts/
+_SCRIPTS_DIR = _THIS_DIR.parent  # pf/
 _FRAMEWORK_ROOT = _SCRIPTS_DIR.parent  # pennyfarthing/
 _GATE_FILE = _FRAMEWORK_ROOT / "pennyfarthing-dist" / "gates" / f"{GATE_NAME}.md"
 

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

@@ -4,7 +4,7 @@ RED state tests for the Python dialogue file persistence layer.
 Ported from packages/core/src/consultation/dialogue-manager.test.ts.
 
 Acceptance Criteria:
-  AC1: pennyfarthing_scripts/consultation/ package with dialogue_manager.py
+  AC1: pf/consultation/ package with dialogue_manager.py
        implementing: create, format, parse, summary, append, update_outcome,
        refresh_summary, archive
   AC2: pf consultation Click CLI group with subcommands: init, append,
@@ -12,7 +12,7 @@ Acceptance Criteria:
   AC3: All 6 ACs from 86-3 still pass — same file format, behavior, output
   AC4: Tests ported to pytest in this file
 
-Run with: pytest pennyfarthing_scripts/tests/test_dialogue_manager.py -v
+Run with: pytest pf/tests/test_dialogue_manager.py -v
 """
 
 from __future__ import annotations
@@ -21,7 +21,7 @@ from pathlib import Path
 
 from click.testing import CliRunner
 
-from pennyfarthing_scripts.consultation.dialogue_manager import (
+from pf.consultation.dialogue_manager import (
     DialogueExchange,
     DialogueHeader,
     DialogueResult,
@@ -746,45 +746,45 @@ class TestConsultationCLI:
         self.runner = CliRunner()
 
     def test_consultation_group_exists(self):
-        from pennyfarthing_scripts.cli import cli
+        from pf.cli import cli
 
         result = self.runner.invoke(cli, ["consultation", "--help"])
         assert result.exit_code == 0
         assert "consultation" in result.output.lower() or "dialogue" in result.output.lower()
 
     def test_init_subcommand_exists(self):
-        from pennyfarthing_scripts.cli import cli
+        from pf.cli import cli
 
         result = self.runner.invoke(cli, ["consultation", "init", "--help"])
         assert result.exit_code == 0
 
     def test_append_subcommand_exists(self):
-        from pennyfarthing_scripts.cli import cli
+        from pf.cli import cli
 
         result = self.runner.invoke(cli, ["consultation", "append", "--help"])
         assert result.exit_code == 0
 
     def test_outcome_subcommand_exists(self):
-        from pennyfarthing_scripts.cli import cli
+        from pf.cli import cli
 
         result = self.runner.invoke(cli, ["consultation", "outcome", "--help"])
         assert result.exit_code == 0
 
     def test_summarize_subcommand_exists(self):
-        from pennyfarthing_scripts.cli import cli
+        from pf.cli import cli
 
         result = self.runner.invoke(cli, ["consultation", "summarize", "--help"])
         assert result.exit_code == 0
 
     def test_archive_subcommand_exists(self):
-        from pennyfarthing_scripts.cli import cli
+        from pf.cli import cli
 
         result = self.runner.invoke(cli, ["consultation", "archive", "--help"])
         assert result.exit_code == 0
 
     def test_init_creates_dialogue_file(self, tmp_path: Path):
         """CLI init should create a dialogue file in .session/."""
-        from pennyfarthing_scripts.cli import cli
+        from pf.cli import cli
 
         with self.runner.isolated_filesystem(temp_dir=tmp_path):
             session_dir = Path(".session")
@@ -799,7 +799,7 @@ class TestConsultationCLI:
             assert dialogue_file.exists()
 
     def test_init_output_confirms_creation(self, tmp_path: Path):
-        from pennyfarthing_scripts.cli import cli
+        from pf.cli import cli
 
         with self.runner.isolated_filesystem(temp_dir=tmp_path):
             Path(".session").mkdir()

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

@@ -19,14 +19,14 @@ from typing import Any
 import pytest
 import yaml
 
-from pennyfarthing_scripts.sprint.validator import (
+from pf.sprint.validator import (
     REQUIRED_EPIC_SHARD_FIELDS,
     REQUIRED_SHARD_STORY_FIELDS,
     ValidationResult,
     validate_epic_shard,
     validate_sprint_file,
 )
-from pennyfarthing_scripts.sprint.yaml_io import _get_epic_ref
+from pf.sprint.yaml_io import _get_epic_ref
 
 # =============================================================================
 # Fixtures
@@ -359,7 +359,7 @@ class TestWritePathValidatorIntegration:
 
     def test_epic_add_validates_before_write(self, tmp_path: Path) -> None:
         """epic_add.add_epic() should reject invalid epics before writing."""
-        from pennyfarthing_scripts.sprint.epic_add import add_epic
+        from pf.sprint.epic_add import add_epic
 
         # Create a minimal sprint index
         index = tmp_path / "current-sprint.yaml"
@@ -377,7 +377,7 @@ class TestWritePathValidatorIntegration:
 
     def test_epic_add_valid_epic_succeeds(self, tmp_path: Path) -> None:
         """epic_add.add_epic() should accept valid epics."""
-        from pennyfarthing_scripts.sprint.epic_add import add_epic
+        from pf.sprint.epic_add import add_epic
 
         index = tmp_path / "current-sprint.yaml"
         index.write_text("sprint:\n  name: Test\nepics: []\n")
@@ -392,7 +392,7 @@ class TestWritePathValidatorIntegration:
 
     def test_import_epic_validates_before_write(self, tmp_path: Path) -> None:
         """import_epic() validates generated YAML before writing to future.yaml."""
-        from pennyfarthing_scripts.sprint.import_epic import import_epic
+        from pf.sprint.import_epic import import_epic
 
         # Create a markdown file with an epic
         md_file = tmp_path / "epics.md"
@@ -432,7 +432,7 @@ class TestWritePathValidatorIntegration:
         """
         import inspect
 
-        from pennyfarthing_scripts.sprint import cli
+        from pf.sprint import cli
 
         # epic_promote is a Click Command; inspect the underlying callback
         source = inspect.getsource(cli.epic_promote.callback)
@@ -445,7 +445,7 @@ class TestWritePathValidatorIntegration:
         """
         import inspect
 
-        from pennyfarthing_scripts.jira.create import create_epic_in_jira
+        from pf.jira.create import create_epic_in_jira
 
         source = inspect.getsource(create_epic_in_jira)
         assert "validate_epic_shard" in source
@@ -465,7 +465,7 @@ class TestLoaderWarnings:
 
     def test_missing_shard_emits_warning(self, tmp_path: Path) -> None:
         """Unresolvable shard ref should emit a warning, not silently skip."""
-        from pennyfarthing_scripts.sprint.loader import _merge_epic_shards
+        from pf.sprint.loader import _merge_epic_shards
 
         data = {
             "epics": ["MSSCI-99999"],  # Doesn't exist
@@ -480,7 +480,7 @@ class TestLoaderWarnings:
 
     def test_missing_shard_excluded_from_result(self, tmp_path: Path) -> None:
         """Missing shard refs should not appear in the merged epics list."""
-        from pennyfarthing_scripts.sprint.loader import _merge_epic_shards
+        from pf.sprint.loader import _merge_epic_shards
 
         data = {"epics": ["MSSCI-99999"]}
 
@@ -492,7 +492,7 @@ class TestLoaderWarnings:
 
     def test_valid_shard_loaded_missing_shard_warned(self, tmp_path: Path) -> None:
         """Mix of valid and missing shards: load valid, warn on missing."""
-        from pennyfarthing_scripts.sprint.loader import _merge_epic_shards
+        from pf.sprint.loader import _merge_epic_shards
 
         # Create one valid shard
         shard = tmp_path / "epic-MSSCI-14298.yaml"
@@ -514,7 +514,7 @@ class TestLoaderWarnings:
 
     def test_non_sharded_data_unchanged(self, tmp_path: Path) -> None:
         """Non-sharded data (epics are dicts) should pass through unchanged."""
-        from pennyfarthing_scripts.sprint.loader import _merge_epic_shards
+        from pf.sprint.loader import _merge_epic_shards
 
         data = {
             "epics": [
@@ -582,7 +582,7 @@ class TestJiraIdempotencyGuard:
         """Source code should contain duplicate title search logic."""
         import inspect
 
-        from pennyfarthing_scripts.jira.create import create_epic_in_jira
+        from pf.jira.create import create_epic_in_jira
 
         source = inspect.getsource(create_epic_in_jira)
         # Should search for existing epic with same title
@@ -592,7 +592,7 @@ class TestJiraIdempotencyGuard:
         """create_epic_in_jira should accept a force parameter."""
         import inspect
 
-        from pennyfarthing_scripts.jira.create import create_epic_in_jira
+        from pf.jira.create import create_epic_in_jira
 
         sig = inspect.signature(create_epic_in_jira)
         assert "force" in sig.parameters
@@ -601,7 +601,7 @@ class TestJiraIdempotencyGuard:
         """force parameter should default to False (safe by default)."""
         import inspect
 
-        from pennyfarthing_scripts.jira.create import create_epic_in_jira
+        from pf.jira.create import create_epic_in_jira
 
         sig = inspect.signature(create_epic_in_jira)
         assert sig.parameters["force"].default is False
@@ -619,7 +619,7 @@ class TestValidationIntegration:
         self, tmp_path: Path, valid_epic_shard_with_jira: dict[str, Any]
     ) -> None:
         """Valid shard passes validation, writes correctly, reads back."""
-        from pennyfarthing_scripts.sprint.yaml_io import _get_epic_ref
+        from pf.sprint.yaml_io import _get_epic_ref
 
         # Validate
         result = validate_epic_shard(valid_epic_shard_with_jira)

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

@@ -19,7 +19,7 @@ from pathlib import Path
 
 import pytest
 
-from pennyfarthing_scripts.handoff.gate_file import resolve_gate_file
+from pf.handoff.gate_file import resolve_gate_file
 
 # ---------------------------------------------------------------------------
 # Fixtures: Project structure

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

@@ -24,7 +24,7 @@ from pathlib import Path
 
 import pytest
 
-from pennyfarthing_scripts.handoff.gate_runner import (
+from pf.handoff.gate_runner import (
     extract_gate_result,
     parse_gate_file,
 )

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

@@ -21,7 +21,7 @@ from unittest.mock import AsyncMock, patch
 
 import pytest
 
-from pennyfarthing_scripts.git.create_branches import (
+from pf.git.create_branches import (
     BranchAction,
     BranchResult,
     create_feature_branches,
@@ -30,7 +30,7 @@ from pennyfarthing_scripts.git.create_branches import (
     filter_repos,
     format_results,
 )
-from pennyfarthing_scripts.git.status_all import (
+from pf.git.status_all import (
     RepoStatus,
     format_status_brief,
     format_status_full,
@@ -240,7 +240,7 @@ class TestGetAllRepoStatus:
     ) -> None:
         """get_all_repo_status should use asyncio.gather for parallelism."""
         with patch(
-            "pennyfarthing_scripts.git.status_all.get_repo_status",
+            "pf.git.status_all.get_repo_status",
             new_callable=AsyncMock,
         ) as mock_get_status:
             mock_get_status.return_value = RepoStatus(
@@ -534,7 +534,7 @@ class TestCreateFeatureBranches:
     ) -> None:
         """create_feature_branches should use asyncio.gather for parallelism."""
         with patch(
-            "pennyfarthing_scripts.git.create_branches.create_or_checkout_branch",
+            "pf.git.create_branches.create_or_checkout_branch",
             new_callable=AsyncMock,
         ) as mock_create:
             mock_create.return_value = BranchResult(
@@ -790,8 +790,8 @@ class TestCrossPlatformCompatibility:
     def test_no_shell_specific_commands(self) -> None:
         """Implementation should not use shell-specific commands."""
         # This is more of a code review check, but we can verify the modules exist
-        import pennyfarthing_scripts.git.create_branches as branch_mod
-        import pennyfarthing_scripts.git.status_all as status_mod
+        import pf.git.create_branches as branch_mod
+        import pf.git.status_all as status_mod
 
         # Modules should exist and be importable
         assert status_mod is not None
@@ -824,7 +824,7 @@ class TestAsyncPerformance:
         repos = [(f"repo-{i}", Path(f"/repo-{i}")) for i in range(3)]
 
         with patch(
-            "pennyfarthing_scripts.git.status_all.get_repo_status",
+            "pf.git.status_all.get_repo_status",
             side_effect=mock_get_status,
         ):
             await get_all_repo_status(repos)
@@ -853,7 +853,7 @@ class TestAsyncPerformance:
         repos = [(f"repo-{i}", Path(f"/repo-{i}")) for i in range(3)]
 
         with patch(
-            "pennyfarthing_scripts.git.create_branches.create_or_checkout_branch",
+            "pf.git.create_branches.create_or_checkout_branch",
             side_effect=mock_create_branch,
         ):
             await create_feature_branches(repos, "feature/test")

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

@@ -12,7 +12,7 @@ Acceptance Criteria:
         checks assessment, returns structured RESOLVE_RESULT
 - [AC2] complete-phase atomically updates session file (temp+mv) with
         phase transition, timestamps, and history tables
-- [AC3] Python module in pennyfarthing_scripts.handoff
+- [AC3] Python module in pf.handoff
 - [AC4] stdout is the only communication channel — no side-channel files
 - [AC5] Exit codes: 0 = ready/skip, 1 = blocked
 - [AC6] YAML parsing via PyYAML
@@ -30,9 +30,9 @@ import pytest
 import yaml
 from click.testing import CliRunner
 
-from pennyfarthing_scripts.cli import cli
-from pennyfarthing_scripts.handoff.complete_phase import complete_phase
-from pennyfarthing_scripts.handoff.resolve_gate import resolve_gate
+from pf.cli import cli
+from pf.handoff.complete_phase import complete_phase
+from pf.handoff.resolve_gate import resolve_gate
 
 # ---------------------------------------------------------------------------
 # Fixtures: Workflow YAML data
@@ -277,7 +277,7 @@ class TestResolveGateReady:
     ) -> None:
         """AC5: Exit code 0 when gate resolves to ready."""
         with patch(
-            "pennyfarthing_scripts.handoff.resolve_gate.resolve_gate",
+            "pf.handoff.resolve_gate.resolve_gate",
             return_value={
                 "status": "ready",
                 "gate_type": "tests_pass",
@@ -319,7 +319,7 @@ class TestResolveGateBlocked:
     def test_blocked_exit_code_one(self, runner: CliRunner) -> None:
         """AC5: Exit code 1 when gate resolves to blocked."""
         with patch(
-            "pennyfarthing_scripts.handoff.resolve_gate.resolve_gate",
+            "pf.handoff.resolve_gate.resolve_gate",
             return_value={
                 "status": "blocked",
                 "gate_type": "tests_pass",
@@ -356,7 +356,7 @@ class TestResolveGateSkip:
     def test_skip_exit_code_zero(self, runner: CliRunner) -> None:
         """AC5: Exit code 0 for skip status."""
         with patch(
-            "pennyfarthing_scripts.handoff.resolve_gate.resolve_gate",
+            "pf.handoff.resolve_gate.resolve_gate",
             return_value={
                 "status": "skip",
                 "gate_type": "manual",
@@ -459,7 +459,7 @@ class TestResolveGateCLIOutput:
     def test_cli_output_is_valid_yaml(self, runner: CliRunner) -> None:
         """AC6: CLI output should be parseable YAML."""
         with patch(
-            "pennyfarthing_scripts.handoff.resolve_gate.resolve_gate",
+            "pf.handoff.resolve_gate.resolve_gate",
             return_value={
                 "status": "ready",
                 "gate_type": "tests_pass",
@@ -479,7 +479,7 @@ class TestResolveGateCLIOutput:
     def test_cli_output_wraps_in_resolve_result_key(self, runner: CliRunner) -> None:
         """AC4: Output should be wrapped in RESOLVE_RESULT key."""
         with patch(
-            "pennyfarthing_scripts.handoff.resolve_gate.resolve_gate",
+            "pf.handoff.resolve_gate.resolve_gate",
             return_value={
                 "status": "ready",
                 "gate_type": "tests_pass",
@@ -742,7 +742,7 @@ class TestCompletePhaseOutputContract:
     def test_cli_output_is_valid_yaml(self, runner: CliRunner) -> None:
         """AC6: CLI output should be parseable YAML."""
         with patch(
-            "pennyfarthing_scripts.handoff.complete_phase.complete_phase",
+            "pf.handoff.complete_phase.complete_phase",
             return_value={
                 "status": "success",
                 "session_file": ".session/105-1-session.md",
@@ -768,7 +768,7 @@ class TestCompletePhaseOutputContract:
     def test_cli_exit_code_zero_on_success(self, runner: CliRunner) -> None:
         """AC5: Exit code 0 on successful phase completion."""
         with patch(
-            "pennyfarthing_scripts.handoff.complete_phase.complete_phase",
+            "pf.handoff.complete_phase.complete_phase",
             return_value={
                 "status": "success",
                 "session_file": ".session/105-1-session.md",
@@ -804,7 +804,7 @@ class TestCompletePhaseErrors:
     def test_error_exit_code_one(self, runner: CliRunner) -> None:
         """AC5: Exit code 1 on error."""
         with patch(
-            "pennyfarthing_scripts.handoff.complete_phase.complete_phase",
+            "pf.handoff.complete_phase.complete_phase",
             return_value={
                 "status": "error",
                 "session_file": None,

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

@@ -25,8 +25,8 @@ from pathlib import Path
 import pytest
 import yaml
 
-from pennyfarthing_scripts.handoff.complete_phase import complete_phase
-from pennyfarthing_scripts.handoff.resolve_gate import resolve_gate
+from pf.handoff.complete_phase import complete_phase
+from pf.handoff.resolve_gate import resolve_gate
 
 # ---------------------------------------------------------------------------
 # Workflow definitions (matching real workflow YAMLs)