claude-code-generator 0.2.2__tar.gz → 0.2.3__tar.gz

{claude_code_generator-0.2.2/src/claude_code_generator.egg-info → claude_code_generator-0.2.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: claude-code-generator
-Version: 0.2.2
+Version: 0.2.3
 Summary: Orchestrator CLI that drives Claude Code end-to-end to generate whole projects from a requirements.md file.
 Author: Silvio Baratto
 License: MIT

{claude_code_generator-0.2.2 → claude_code_generator-0.2.3}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "claude-code-generator"
-version = "0.2.2"
+version = "0.2.3"
 description = "Orchestrator CLI that drives Claude Code end-to-end to generate whole projects from a requirements.md file."
 readme = "README.md"
 license = { text = "MIT" }

{claude_code_generator-0.2.2 → claude_code_generator-0.2.3/src/claude_code_generator.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: claude-code-generator
-Version: 0.2.2
+Version: 0.2.3
 Summary: Orchestrator CLI that drives Claude Code end-to-end to generate whole projects from a requirements.md file.
 Author: Silvio Baratto
 License: MIT

{claude_code_generator-0.2.2 → claude_code_generator-0.2.3}/src/code_generator/__init__.py

@@ -1,3 +1,3 @@
 """code-generator: orchestrator CLI for end-to-end project generation."""
 
-__version__ = "0.2.2"
+__version__ = "0.2.3"

{claude_code_generator-0.2.2 → claude_code_generator-0.2.3}/src/code_generator/orchestrator/phase2_review.py

@@ -63,6 +63,14 @@ async def run(
         if issue.status != "open":
             continue
 
+        # Skip issues already reviewed in a prior run. Phase 2 never closes
+        # the issue (only Phase 3/4 does), so status alone cannot tell us
+        # whether the review already ran. Without this check, --continue
+        # re-reviews every open issue from scratch on every resume.
+        if issue.reviewed:
+            logger.info("Phase 2: issue #%d already reviewed — skipping.", issue.number)
+            continue
+
         logger.info("Phase 2: reviewing issue #%d (agent=%s).", issue.number, issue.agent)
 
         comments_section = fetch_formatted_comments(issue.number, logger)
@@ -137,6 +145,11 @@ async def run(
         phase_total += issue_usage
 
         if reviewed:
+            # Persist the per-issue reviewed flag immediately so a crash (or
+            # --continue after a rate-limit pause) does not re-review work
+            # that already succeeded.
+            issue.reviewed = True
+            save_state(state_path, state)
             logger.info("Phase 2: issue #%d reviewed.", issue.number)
 
     target = cycle if cycle is not None else state

{claude_code_generator-0.2.2 → claude_code_generator-0.2.3}/src/code_generator/state.py

@@ -27,6 +27,7 @@ class IssueState:
     status: Literal["open", "closed"]
     agent: str
     labels: list[str] = field(default_factory=list)
+    reviewed: bool = False
 
 
 @dataclass
@@ -112,6 +113,7 @@ def _issue_from_dict(d: dict) -> IssueState:  # type: ignore[type-arg]
         status=d["status"],
         agent=d["agent"],
         labels=d.get("labels", []),
+        reviewed=d.get("reviewed", False),
     )
 
 

{claude_code_generator-0.2.2 → claude_code_generator-0.2.3}/tests/test_phase2.py

@@ -504,3 +504,77 @@ class TestPhase2TokenAccumulation:
 
         assert cycle.token_usage.get("phase2") == issue_usage
         assert "phase2" not in st.token_usage
+
+    @pytest.mark.asyncio
+    async def test_skips_already_reviewed_issues(self, tmp_path: Path) -> None:
+        """Issues with reviewed=True from a prior --continue run must be skipped."""
+        state_dir = tmp_path / ".code-generator"
+        state_dir.mkdir(parents=True, exist_ok=True)
+        st = _state.load_state(state_dir / "missing.json")
+        st.issues = [
+            _state.IssueState(number=1, status="open", agent="python-pro", reviewed=True),
+            _state.IssueState(number=2, status="open", agent="python-pro", reviewed=False),
+            _state.IssueState(number=3, status="open", agent="python-pro", reviewed=True),
+        ]
+        _state.save_state(state_dir / "state.json", st)
+
+        reviewed_numbers: list[int] = []
+
+        async def fake_with_backoff(factory, *, breaker, **kw):
+            await factory()
+
+        async def fake_main_loop(runner, prompt, options, *, state_path, logger, **kw):
+            return _make_run_result()
+
+        with (
+            patch.object(phase2_review.rate_limit, "main_loop", side_effect=fake_main_loop),
+            patch.object(phase2_review.retry, "with_backoff", side_effect=fake_with_backoff),
+            patch.object(phase2_review.gh, "view_issue", return_value={"comments": []}),
+            patch.object(phase2_review, "load_prompt", side_effect=lambda *a, **k: "prompt"),
+            patch.object(
+                phase2_review,
+                "fetch_formatted_comments",
+                side_effect=lambda n, logger: reviewed_numbers.append(n) or "",
+            ),
+        ):
+            await phase2_review.run(
+                st,
+                None,
+                tmp_path,
+                runner_module=MagicMock(),
+                logger=logging.getLogger("test"),
+            )
+
+        # Only issue #2 is actually reviewed; #1 and #3 are skipped.
+        assert reviewed_numbers == [2]
+        # Issue #2 is now marked reviewed; #1 and #3 stay reviewed.
+        assert all(i.reviewed for i in st.issues)
+
+    @pytest.mark.asyncio
+    async def test_successful_review_persists_reviewed_flag(self, tmp_path: Path) -> None:
+        """A successful Phase 2 review must persist reviewed=True to state.json."""
+        state = _make_state_with_issues(tmp_path, [42])
+        state_path = tmp_path / ".code-generator" / "state.json"
+
+        async def fake_with_backoff(factory, *, breaker, **kw):
+            await factory()
+
+        async def fake_main_loop(runner, prompt, options, *, state_path, logger, **kw):
+            return _make_run_result()
+
+        with (
+            patch.object(phase2_review.rate_limit, "main_loop", side_effect=fake_main_loop),
+            patch.object(phase2_review.retry, "with_backoff", side_effect=fake_with_backoff),
+            patch.object(phase2_review.gh, "view_issue", return_value={"comments": []}),
+        ):
+            await phase2_review.run(
+                state,
+                None,
+                tmp_path,
+                runner_module=MagicMock(),
+                logger=logging.getLogger("test"),
+            )
+
+        # Reloaded state must show reviewed=True so a subsequent --continue skips it.
+        reloaded = _state.load_state(state_path)
+        assert reloaded.issues[0].reviewed is True