alissa-tools-github-devloop 0.4.1__tar.gz → 0.5.1__tar.gz

{alissa_tools_github_devloop-0.4.1/src/main/alissa_tools_github_devloop.egg-info → alissa_tools_github_devloop-0.5.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: alissa-tools-github-devloop
-Version: 0.4.1
+Version: 0.5.1
 Summary: ALISSA-TOOLS-GITHUB-DEVLOOP
 Home-page: https://alissa.app
 Author: Fahera

{alissa_tools_github_devloop-0.4.1 → alissa_tools_github_devloop-0.5.1}/setup.py

@@ -49,6 +49,7 @@ setup(
     entry_points={
         "console_scripts": [
             "alissa-devloop=alissa.tools.github.devloop.__main__:main",
+            "alissa-devloop-ui=alissa.tools.github.devloop.webui.__main__:main",
         ]
     },
     install_requires=requirements,

{alissa_tools_github_devloop-0.4.1 → alissa_tools_github_devloop-0.5.1}/src/main/alissa/tools/github/devloop/__main__.py

@@ -92,6 +92,14 @@ def build_parser() -> argparse.ArgumentParser:
     over.add_argument("--state-path", type=Path, metavar="PATH")
     over.add_argument("--attempt-cap", type=int, metavar="N", help="dev attempts per issue")
     over.add_argument("--stale-minutes", type=int, metavar="MINUTES")
+    over.add_argument(
+        "--max-sessions",
+        type=int,
+        metavar="N",
+        help="max concurrent worker sessions (develop-*/fix-*/maintain-*) "
+        "before actionable items defer to the next poll; 0 = unlimited "
+        "(the default)",
+    )
     over.add_argument(
         "--on-missing-origin-task",
         choices=[ON_MISSING_WARN, ON_MISSING_SPAWN, ON_MISSING_SKIP],
@@ -170,6 +178,7 @@ def overrides_from(args: argparse.Namespace) -> dict:
         "state_path": args.state_path,
         "attempt_cap": args.attempt_cap,
         "stale_minutes": args.stale_minutes,
+        "max_sessions": args.max_sessions,
         "on_missing_origin_task": args.on_missing_origin_task,
         "on_missing_hub": args.on_missing_hub,
         "fix_rounds_enabled": args.fix_rounds_enabled,
@@ -206,9 +215,11 @@ def log_effective_config(config: Config, login: str) -> None:
     log.info("reviewers: %s", ", ".join(config.reviewers) or "none")
     log.info(
         "poll every %ss; dry_run=%s; attempt_cap=%s; stale after %s min; "
-        "fix_rounds=%s; maintain=%s (label %r)",
+        "max_sessions=%s; fix_rounds=%s; maintain=%s (label %r)",
         config.poll_interval, config.dry_run, config.attempt_cap,
-        config.stale_minutes, config.fix_rounds_enabled,
+        config.stale_minutes,
+        config.max_sessions or "unlimited",
+        config.fix_rounds_enabled,
         config.maintain_enabled, config.maintain_label,
     )
     log.debug("hub_template: %s", config.hub_template)

{alissa_tools_github_devloop-0.4.1 → alissa_tools_github_devloop-0.5.1}/src/main/alissa/tools/github/devloop/config.py

@@ -56,6 +56,7 @@ CONFIG_KEYS = (
     "state_path",
     "attempt_cap",
     "stale_minutes",
+    "max_sessions",
     "on_missing_origin_task",
     "on_missing_hub",
     "fix_rounds_enabled",
@@ -133,6 +134,15 @@ class Config:
     attempt_cap: int = 10
     stale_minutes: int = 20
 
+    # Backpressure: the maximum number of concurrent worker sessions
+    # (`develop-*`/`fix-*`/`maintain-*`) the daemon will keep in flight at
+    # once. 0 means UNLIMITED -- today's behavior, preserved as the default so
+    # nothing changes until an operator opts in. At the cap, actionable items
+    # defer cleanly (no self-assign, no ledger row) and retry next poll; see
+    # DevWatcher.poll_once. Counted from the reaper's own `alissa tmux ls`
+    # listing, so enabling it costs no extra GitHub or CLI calls.
+    max_sessions: int = 0
+
     on_missing_origin_task: str = ON_MISSING_WARN
     on_missing_hub: str = HUB_SKIP
 
@@ -269,6 +279,14 @@ class Config:
         if stale < 1:
             raise ValueError(f"stale_minutes must be >= 1, got {stale}")
 
+        # 0 is the sentinel for "unlimited" (the default), so the floor is 0,
+        # not 1 -- a negative cap is the only invalid value.
+        max_sessions = int(raw.get("max_sessions", cls.max_sessions))
+        if max_sessions < 0:
+            raise ValueError(
+                f"max_sessions must be >= 0 (0 = unlimited), got {max_sessions}"
+            )
+
         interval = int(raw.get("poll_interval", cls.poll_interval))
         if interval < MIN_POLL_INTERVAL:
             raise ValueError(
@@ -288,6 +306,7 @@ class Config:
             state_path=Path(state_path).expanduser() if state_path else None,
             attempt_cap=cap,
             stale_minutes=stale,
+            max_sessions=max_sessions,
             on_missing_origin_task=mode,
             on_missing_hub=hub_mode,
             fix_rounds_enabled=bool(

{alissa_tools_github_devloop-0.4.1 → alissa_tools_github_devloop-0.5.1}/src/main/alissa/tools/github/devloop/ghclient.py

@@ -147,6 +147,20 @@ class Review:
         return bool(self.body.strip())
 
 
+@dataclass(frozen=True)
+class IssueComment:
+    """An issue comment on an issue OR a PR (a PR is an issue on the issues
+    endpoints). Distinct from Review: issue comments live on the
+    `/issues/{n}/comments` endpoints and never create review records, so
+    posting or PATCHing one can never disturb the fix edge's round counting
+    or its author-activity liveness probe -- the non-interference the
+    mechanical activity comment leans on."""
+
+    id: int
+    author: str
+    body: str
+
+
 class RateLimited(RuntimeError):
     pass
 
@@ -739,3 +753,50 @@ class GitHub:
             "-f",
             f"body={body}",
         )
+
+    def issue_comments(
+        self, owner: str, repo: str, number: int
+    ) -> list[IssueComment]:
+        """Issue comments on an issue or PR (find-or-create reads these to
+        locate THE activity comment). per_page=100 is a documented single-page
+        window like the searches -- the activity comment is authored by the
+        daemon itself, so it lands on the first page unless a hundred other
+        comments were filed after it, and even then find-or-create degrades to
+        creating a second activity comment, never to touching a foreign one.
+        Malformed entries are tolerated (never fatal): one odd comment must not
+        wedge a spawn's telemetry."""
+        data = (
+            self._api(
+                "-X",
+                "GET",
+                f"repos/{owner}/{repo}/issues/{number}/comments",
+                "-f",
+                "per_page=100",
+            )
+            or []
+        )
+        if not isinstance(data, list):
+            return []
+        return [
+            IssueComment(
+                id=int(c.get("id") or 0),
+                author=(c.get("user") or {}).get("login", ""),
+                body=c.get("body") or "",
+            )
+            for c in data
+            if isinstance(c, dict)
+        ]
+
+    def update_comment(
+        self, owner: str, repo: str, comment_id: int, body: str
+    ) -> None:
+        """PATCH an existing issue comment's body -- the append half of the
+        activity comment's find-or-create. Keyed on the comment id, so it only
+        ever rewrites a comment the caller already located and authored."""
+        self._api(
+            "-X",
+            "PATCH",
+            f"repos/{owner}/{repo}/issues/comments/{comment_id}",
+            "-f",
+            f"body={body}",
+        )