remote-coder 0.4.4__tar.gz → 0.5.0__tar.gz

{remote_coder-0.4.4/remote_coder.egg-info → remote_coder-0.5.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: remote-coder
-Version: 0.4.4
+Version: 0.5.0
 Summary: Telegram-based remote AI coding automation server
 Author: Remote AI Coder contributors
 License:                                  Apache License
@@ -234,8 +234,9 @@ Run Claude Code, Codex, or Gemini on your local development machine by sending a
 - One Telegram bot per registered project, with project-scoped allowlists and settings.
 - Request-specific Git worktrees, branch creation, commit, push, and result notifications.
 - Claude, Codex, and Gemini runners behind the same job flow.
+- Reply-linked jobs continue the same AI CLI session, so follow-ups build on prior context.
 - Local admin UI for project setup, advanced settings, logs, and conversation memory.
-- Read-only `plan:` and `ask:` modes when you want analysis without commits.
+- Read-only `plan:` and `ask:` modes when you want analysis without commits. PLAN mode asks open decisions through inline buttons first, then finalizes the plan from your answers.
 
 ## Quick Start
 
@@ -268,6 +269,7 @@ Open `http://127.0.0.1:8000/`, add your first project, and message the project b
 - Allowed Telegram Chat IDs, and optionally User IDs
 - At least one local AI CLI: `claude`, `codex`, or `gemini`
 - A local Git repository to automate
+- GitHub CLI (`gh`) authenticated with `gh auth login` when using `/pr`
 
 ## How It Works
 
@@ -294,13 +296,15 @@ Each project uses its own bot. The webhook path contains the first 16 hex charac
 | `/branch [name]` | Show or switch the bound project's local branch |
 | `/pull` | Fetch remotes and pull the current branch |
 | `/rebase [branch]` | Rebase and fast-forward a completed branch into `main` or `master` |
-| `/pr [branch]` | Create a GitHub PR with `gh` |
+| `/pr [branch]` | Create a GitHub PR from a succeeded Job branch with `gh` |
 | `/fix ...` | Rework a previous job's commit message or source |
 | `/monitor ...` | Inspect model, memory, branch, worktree, code, or project status |
 | `/clear ...` | Clean managed branches, worktrees, or conversation memory |
 | `/stop [job_id]` | Cancel a queued or running job |
 | `/init` | Reset chat-local model and pending confirmations |
 
+Calling `/pr` without a branch lists only remote branches produced by succeeded Jobs in the current project and Telegram chat. Direct `/pr <branch>` calls enforce the same ownership check and verify that the branch still exists on the configured Git remote. Install the [GitHub CLI](https://cli.github.com/) and run `gh auth login` before using this command.
+
 Natural-language examples:
 
 ```text

{remote_coder-0.4.4 → remote_coder-0.5.0}/README.md

@@ -13,8 +13,9 @@ Run Claude Code, Codex, or Gemini on your local development machine by sending a
 - One Telegram bot per registered project, with project-scoped allowlists and settings.
 - Request-specific Git worktrees, branch creation, commit, push, and result notifications.
 - Claude, Codex, and Gemini runners behind the same job flow.
+- Reply-linked jobs continue the same AI CLI session, so follow-ups build on prior context.
 - Local admin UI for project setup, advanced settings, logs, and conversation memory.
-- Read-only `plan:` and `ask:` modes when you want analysis without commits.
+- Read-only `plan:` and `ask:` modes when you want analysis without commits. PLAN mode asks open decisions through inline buttons first, then finalizes the plan from your answers.
 
 ## Quick Start
 
@@ -47,6 +48,7 @@ Open `http://127.0.0.1:8000/`, add your first project, and message the project b
 - Allowed Telegram Chat IDs, and optionally User IDs
 - At least one local AI CLI: `claude`, `codex`, or `gemini`
 - A local Git repository to automate
+- GitHub CLI (`gh`) authenticated with `gh auth login` when using `/pr`
 
 ## How It Works
 
@@ -73,13 +75,15 @@ Each project uses its own bot. The webhook path contains the first 16 hex charac
 | `/branch [name]` | Show or switch the bound project's local branch |
 | `/pull` | Fetch remotes and pull the current branch |
 | `/rebase [branch]` | Rebase and fast-forward a completed branch into `main` or `master` |
-| `/pr [branch]` | Create a GitHub PR with `gh` |
+| `/pr [branch]` | Create a GitHub PR from a succeeded Job branch with `gh` |
 | `/fix ...` | Rework a previous job's commit message or source |
 | `/monitor ...` | Inspect model, memory, branch, worktree, code, or project status |
 | `/clear ...` | Clean managed branches, worktrees, or conversation memory |
 | `/stop [job_id]` | Cancel a queued or running job |
 | `/init` | Reset chat-local model and pending confirmations |
 
+Calling `/pr` without a branch lists only remote branches produced by succeeded Jobs in the current project and Telegram chat. Direct `/pr <branch>` calls enforce the same ownership check and verify that the branch still exists on the configured Git remote. Install the [GitHub CLI](https://cli.github.com/) and run `gh auth login` before using this command.
+
 Natural-language examples:
 
 ```text

{remote_coder-0.4.4 → remote_coder-0.5.0}/app/__init__.py

@@ -1,3 +1,3 @@
 """Remote AI Coder application package."""
 
-__version__ = "0.4.4"
+__version__ = "0.5.0"

{remote_coder-0.4.4 → remote_coder-0.5.0}/app/admin/advanced_settings.py

@@ -14,26 +14,31 @@ CONVERSATION_REPLY_SNIPPET_MAX_CHARS_DEFAULT = 3000
 CONVERSATION_REPLY_SNIPPET_MAX_CHARS_MIN = 200
 CONVERSATION_REPLY_SNIPPET_MAX_CHARS_MAX = 20000
 
+# Settings retired for usability; popped on load so old config files still validate.
+_REMOVED_SETTING_KEYS = (
+    "auto_pull_on_project_switch",
+    "server_lifecycle_notify_enabled",
+    "natural_job_confirmation_buttons_enabled",
+    "status_recent_job_limit",
+    "conversation_recent_limit",
+    "conversation_reply_snippet_max_chars",
+)
+
 
 class AdvancedSettings(BaseModel):
     model_config = {"extra": "forbid"}
 
     ui_language: UiLanguage = UiLanguage.ENGLISH
-    server_lifecycle_notify_enabled: bool = True
     pull_projects_on_server_startup_enabled: bool = False
     auto_merge_to_main_enabled: bool = False
     delete_rebased_branch_enabled: bool = True
-    natural_job_confirmation_buttons_enabled: bool = False
     conversation_memory_limit_enabled: bool = False
     conversation_memory_max_rows: int | None = None
     conversation_memory_max_bytes: int | None = None
-    status_recent_job_limit: int = 10
     job_timeout_seconds: int = 1800
     git_remote_name: str = "origin"
     keep_worktree_on_success: bool = True
     codex_sandbox: CodexSandboxMode = CodexSandboxMode.WORKSPACE_WRITE
-    conversation_recent_limit: int = 10
-    conversation_reply_snippet_max_chars: int = CONVERSATION_REPLY_SNIPPET_MAX_CHARS_DEFAULT
 
     @model_validator(mode="after")
     def _validate_memory_limits(self) -> Self:
@@ -51,22 +56,8 @@ class AdvancedSettings(BaseModel):
             raise ValueError("conversation_memory_max_rows must be positive or blank.")
         if self.conversation_memory_max_bytes is not None and self.conversation_memory_max_bytes <= 0:
             raise ValueError("conversation_memory_max_bytes must be positive or blank.")
-        if self.status_recent_job_limit < 1:
-            raise ValueError("status_recent_job_limit must be at least 1.")
         if self.job_timeout_seconds <= 0:
             raise ValueError("job_timeout_seconds must be positive.")
-        if self.conversation_recent_limit < 1:
-            raise ValueError("conversation_recent_limit must be at least 1.")
-        if not (
-            CONVERSATION_REPLY_SNIPPET_MAX_CHARS_MIN
-            <= self.conversation_reply_snippet_max_chars
-            <= CONVERSATION_REPLY_SNIPPET_MAX_CHARS_MAX
-        ):
-            raise ValueError(
-                "conversation_reply_snippet_max_chars must be between "
-                f"{CONVERSATION_REPLY_SNIPPET_MAX_CHARS_MIN} and "
-                f"{CONVERSATION_REPLY_SNIPPET_MAX_CHARS_MAX}.",
-            )
         return self
 
 
@@ -92,7 +83,8 @@ class FileAdvancedSettingsStore:
             raw = self._path.read_text(encoding="utf-8")
             data = json.loads(raw) if raw.strip() else {}
             if isinstance(data, dict):
-                data.pop("auto_pull_on_project_switch", None)
+                for removed_key in _REMOVED_SETTING_KEYS:
+                    data.pop(removed_key, None)
             self._cached = AdvancedSettings.model_validate(data)
             return self._cached.model_copy(deep=True)
 

{remote_coder-0.4.4 → remote_coder-0.5.0}/app/admin/router.py

@@ -1,9 +1,11 @@
 from __future__ import annotations
 
+import secrets
 from functools import lru_cache
 from pathlib import Path
 from typing import Annotated, Callable
 
+import httpx
 from fastapi import APIRouter, Depends, HTTPException, Query, Request
 from fastapi.responses import FileResponse, HTMLResponse, JSONResponse, StreamingResponse
 from pydantic import BaseModel, Field, SecretStr
@@ -43,7 +45,7 @@ def require_localhost(request: Request) -> None:
 
 LocalhostOnly = Annotated[None, Depends(require_localhost)]
 
-_DEFAULT_NEW_PROJECT_WEBHOOK_SECRET = "optional-secret"
+_WEBHOOK_SECRET_BYTES = 32
 
 
 class ProjectUpsertBody(BaseModel):
@@ -61,6 +63,25 @@ class DefaultProjectBody(BaseModel):
     name: str = Field(min_length=1)
 
 
+class SetupTokenBody(BaseModel):
+    bot_token: str = Field(min_length=1)
+
+
+def _telegram_get(token: str, method: str, params: dict | None = None) -> dict:
+    url = f"https://api.telegram.org/bot{token}/{method}"
+    try:
+        response = httpx.get(url, params=params, timeout=httpx.Timeout(10.0, connect=5.0))
+    except httpx.HTTPError as exc:
+        raise HTTPException(status_code=502, detail="Could not reach Telegram.") from exc
+    try:
+        data = response.json()
+    except ValueError:
+        data = {}
+    if not isinstance(data, dict):
+        data = {}
+    return data
+
+
 _ADMIN_ICON_NAMES = frozenset(
     {"home.svg", "projects.svg", "advanced.svg", "logs.svg", "database.svg", "download.svg"}
 )
@@ -336,7 +357,6 @@ def create_admin_router(
             "job_timeout_seconds": adv.job_timeout_seconds,
             "git_remote_name": adv.git_remote_name,
             "codex_sandbox": adv.codex_sandbox.value,
-            "conversation_recent_limit": adv.conversation_recent_limit,
             "keep_worktree_on_success": adv.keep_worktree_on_success,
             "webhook_token_hash_prefix_length": WEBHOOK_TOKEN_HASH_PREFIX_LENGTH,
             "webhook_route_template": "/telegram/webhook/{token_hash_prefix}",
@@ -364,6 +384,50 @@ def create_admin_router(
         )
         return report.model_dump()
 
+    @router.post("/api/setup/validate-token")
+    def api_setup_validate_token(body: SetupTokenBody, _: LocalhostOnly) -> dict:
+        token = body.bot_token.strip()
+        if not token:
+            raise HTTPException(status_code=400, detail="bot_token is required")
+        data = _telegram_get(token, "getMe")
+        if not data.get("ok"):
+            _adminlog.warning("setup token validation rejected by Telegram")
+            raise HTTPException(status_code=400, detail="Telegram rejected the bot token.")
+        result = data.get("result") or {}
+        _adminlog.info("setup token validated bot=%s", result.get("username") or "-")
+        return {
+            "ok": True,
+            "bot_username": result.get("username"),
+            "bot_name": result.get("first_name"),
+        }
+
+    @router.post("/api/setup/detect-chat")
+    def api_setup_detect_chat(body: SetupTokenBody, _: LocalhostOnly) -> dict:
+        token = body.bot_token.strip()
+        if not token:
+            raise HTTPException(status_code=400, detail="bot_token is required")
+        data = _telegram_get(token, "getUpdates", {"timeout": 0, "limit": 100})
+        if not data.get("ok"):
+            _adminlog.warning("setup chat detection failed (getUpdates not ok)")
+            raise HTTPException(
+                status_code=409,
+                detail="Could not read updates for this bot (a webhook may already be set). "
+                "Enter the chat ID manually.",
+            )
+        for update in reversed(data.get("result") or []):
+            message = update.get("message") if isinstance(update, dict) else None
+            chat = (message or {}).get("chat") if isinstance(message, dict) else None
+            if isinstance(chat, dict) and chat.get("id") is not None:
+                sender = message.get("from") or {}
+                _adminlog.info("setup chat detected chat_id=%s", chat.get("id"))
+                return {
+                    "found": True,
+                    "chat_id": chat.get("id"),
+                    "chat_name": chat.get("title") or chat.get("username") or chat.get("first_name"),
+                    "user_id": sender.get("id"),
+                }
+        return {"found": False}
+
     @router.get("/api/projects")
     def api_projects_get(_: LocalhostOnly) -> JSONResponse:
         _adminlog.info("projects queried count=%d", len(registry.list_projects()))
@@ -377,7 +441,7 @@ def create_admin_router(
             raise HTTPException(status_code=400, detail="allowed_chat_ids must have at least one entry")
         wh_stripped = (body.webhook_secret or "").strip()
         webhook_secret = SecretStr(
-            wh_stripped if wh_stripped else _DEFAULT_NEW_PROJECT_WEBHOOK_SECRET
+            wh_stripped if wh_stripped else secrets.token_urlsafe(_WEBHOOK_SECRET_BYTES)
         )
         record = ProjectRecord(
             name=body.name,
@@ -492,19 +556,23 @@ def create_admin_router(
             for project in registry.list_projects():
                 webhook_registrar.sync_project_commands(project)
         _adminlog.info(
-            "advanced settings updated ui_language=%s lifecycle_notify=%s pull_startup=%s auto_merge=%s delete_rebased_branch=%s natural_confirm_buttons=%s status_limit=%d job_timeout=%s memory_limit=%s",
+            "advanced settings updated ui_language=%s pull_startup=%s auto_merge=%s delete_rebased_branch=%s job_timeout=%s memory_limit=%s",
             saved.ui_language.value,
-            saved.server_lifecycle_notify_enabled,
             saved.pull_projects_on_server_startup_enabled,
             saved.auto_merge_to_main_enabled,
             saved.delete_rebased_branch_enabled,
-            saved.natural_job_confirmation_buttons_enabled,
-            saved.status_recent_job_limit,
             saved.job_timeout_seconds or "-",
             saved.conversation_memory_limit_enabled,
         )
         return saved.model_dump(mode="json")
 
+    @router.get("/admin-static/admin.css")
+    def admin_css(_: LocalhostOnly) -> FileResponse:
+        path = Path(__file__).parent / "static" / "admin.css"
+        if not path.is_file():
+            raise HTTPException(status_code=404, detail="not found")
+        return FileResponse(path, media_type="text/css; charset=utf-8")
+
     @router.get("/admin-static/i18n.js")
     def admin_i18n_js(_: LocalhostOnly) -> FileResponse:
         path = Path(__file__).parent / "static" / "i18n.js"