ralph-code 0.6.2__tar.gz → 0.6.3__tar.gz

{ralph_code-0.6.2/ralph_code.egg-info → ralph_code-0.6.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: ralph-code
-Version: 0.6.2
+Version: 0.6.3
 Summary: Automated task implementation with Claude Code and Codex
 Author: Ralph Coding
 License: MIT
@@ -61,6 +61,12 @@ ralph [OPTIONS] [DIRECTORY]
 
 ## Recent changes
 
+Version `0.6.3` includes:
+- Safer git integration for staged-file commit checks
+- Correct staged diff generation
+- Deduplicated unstaged file reporting
+- Clearer errors when `git` is not available
+
 Version `0.6.2` includes:
 - Bounded non-interactive harness execution with timeout and turn limits
 - Structured `tasks.json` generation for more reliable PRD conversion

{ralph_code-0.6.2 → ralph_code-0.6.3}/README.md

@@ -24,6 +24,12 @@ ralph [OPTIONS] [DIRECTORY]
 
 ## Recent changes
 
+Version `0.6.3` includes:
+- Safer git integration for staged-file commit checks
+- Correct staged diff generation
+- Deduplicated unstaged file reporting
+- Clearer errors when `git` is not available
+
 Version `0.6.2` includes:
 - Bounded non-interactive harness execution with timeout and turn limits
 - Structured `tasks.json` generation for more reliable PRD conversion

{ralph_code-0.6.2 → ralph_code-0.6.3}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "ralph-code"
-version = "0.6.2"
+version = "0.6.3"
 description = "Automated task implementation with Claude Code and Codex"
 readme = "README.md"
 license = {text = "MIT"}

{ralph_code-0.6.2 → ralph_code-0.6.3}/ralph/__init__.py

@@ -1,6 +1,6 @@
 """ralph-code: Automated task implementation with Claude Code and Codex."""
 
-__version__ = "0.6.2"
+__version__ = "0.6.3"
 __author__ = "Ralph Coding"
 
 from .app import RalphApp, main

{ralph_code-0.6.2 → ralph_code-0.6.3}/ralph/git_manager.py

@@ -26,6 +26,8 @@ class GitManager:
                 check=check,
             )
             return result
+        except FileNotFoundError as e:
+            raise GitError("Git executable not found on PATH") from e
         except subprocess.CalledProcessError as e:
             raise GitError(f"Git command failed: git {' '.join(args)}\n{e.stderr}")
 
@@ -241,19 +243,22 @@ class GitManager:
         Returns files that have changes not yet added to the staging area.
         This includes both modified tracked files and new untracked files.
         """
-        files = []
+        files: list[str] = []
+        seen: set[str] = set()
 
         # Get modified but unstaged files
         result = self._run_git("diff", "--name-only")
         for f in result.stdout.strip().split("\n"):
-            if f:
+            if f and f not in seen:
                 files.append(f)
+                seen.add(f)
 
         # Get untracked files
         result = self._run_git("ls-files", "--others", "--exclude-standard")
         for f in result.stdout.strip().split("\n"):
-            if f:
+            if f and f not in seen:
                 files.append(f)
+                seen.add(f)
 
         return files
 
@@ -264,7 +269,7 @@ class GitManager:
 
     def get_diff(self, staged: bool = False) -> str:
         """Get the current diff for stageable files only."""
-        files = self.get_stageable_files()
+        files = self.get_staged_files() if staged else self.get_stageable_files()
         if not files:
             return ""
 

{ralph_code-0.6.2 → ralph_code-0.6.3/ralph_code.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: ralph-code
-Version: 0.6.2
+Version: 0.6.3
 Summary: Automated task implementation with Claude Code and Codex
 Author: Ralph Coding
 License: MIT
@@ -61,6 +61,12 @@ ralph [OPTIONS] [DIRECTORY]
 
 ## Recent changes
 
+Version `0.6.3` includes:
+- Safer git integration for staged-file commit checks
+- Correct staged diff generation
+- Deduplicated unstaged file reporting
+- Clearer errors when `git` is not available
+
 Version `0.6.2` includes:
 - Bounded non-interactive harness execution with timeout and turn limits
 - Structured `tasks.json` generation for more reliable PRD conversion

{ralph_code-0.6.2 → ralph_code-0.6.3}/setup.py

@@ -8,7 +8,7 @@ from setuptools import setup, find_packages  # type: ignore[import-untyped]
 
 setup(
     name="ralph-code",
-    version="0.6.2",
+    version="0.6.3",
     description="Automated task implementation with Claude Code and Codex",
     author="Ralph Coding",
     packages=find_packages(),

{ralph_code-0.6.2 → ralph_code-0.6.3}/tests/test_git_manager.py

@@ -18,6 +18,18 @@ class TestGitManagerInit:
         assert manager.project_dir == tmp_path
 
 
+class TestRunGit:
+    """Tests for low-level git command execution."""
+
+    @patch("ralph.git_manager.subprocess.run", side_effect=FileNotFoundError())
+    def test_run_git_handles_missing_git(self, _mock_run: MagicMock, tmp_path: Path) -> None:
+        """Test a missing git executable raises GitError."""
+        manager = GitManager(tmp_path)
+
+        with pytest.raises(GitError, match="Git executable not found on PATH"):
+            manager._run_git("status")
+
+
 class TestIsGitRepo:
     """Tests for is_git_repo method."""
 
@@ -386,6 +398,49 @@ class TestGetDiff:
 
         assert "modified content here" in diff
 
+    def test_get_diff_staged_only_uses_staged_files(self, tmp_path: Path) -> None:
+        """Test staged diffs ignore separate unstaged changes."""
+        subprocess.run(["git", "init"], cwd=tmp_path, capture_output=True)
+        subprocess.run(["git", "config", "user.email", "test@test.com"], cwd=tmp_path, capture_output=True)
+        subprocess.run(["git", "config", "user.name", "Test"], cwd=tmp_path, capture_output=True)
+
+        staged_file = tmp_path / "staged.py"
+        unstaged_file = tmp_path / "unstaged.py"
+        staged_file.write_text("before staged")
+        unstaged_file.write_text("before unstaged")
+        subprocess.run(["git", "add", "."], cwd=tmp_path, capture_output=True)
+        subprocess.run(["git", "commit", "-m", "Initial"], cwd=tmp_path, capture_output=True)
+
+        staged_file.write_text("after staged")
+        unstaged_file.write_text("after unstaged")
+        subprocess.run(["git", "add", "staged.py"], cwd=tmp_path, capture_output=True)
+
+        manager = GitManager(tmp_path)
+        diff = manager.get_diff(staged=True)
+
+        assert "after staged" in diff
+        assert "after unstaged" not in diff
+
+
+class TestGetUnstagedFiles:
+    """Tests for get_unstaged_files method."""
+
+    def test_get_unstaged_files_deduplicates_results(self, tmp_path: Path) -> None:
+        """Test duplicate file names are collapsed while preserving order."""
+        manager = GitManager(tmp_path)
+
+        with patch.object(
+            manager,
+            "_run_git",
+            side_effect=[
+                MagicMock(stdout="a.py\nb.py\n"),
+                MagicMock(stdout="b.py\nc.py\n"),
+            ],
+        ):
+            files = manager.get_unstaged_files()
+
+        assert files == ["a.py", "b.py", "c.py"]
+
 
 class TestGetLastCommitMessage:
     """Tests for get_last_commit_message method."""