devops-bot-sdk 1.6.62__tar.gz → 1.6.64__tar.gz

{devops_bot_sdk-1.6.62 → devops_bot_sdk-1.6.64}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: devops-bot-sdk
-Version: 1.6.62
+Version: 1.6.64
 Summary: DevOps Bot Desktop SDK — thin client for the AgentOS Electron desktop app
 Author: noumanaziz2128
 License-Expression: LicenseRef-Proprietary

{devops_bot_sdk-1.6.62 → devops_bot_sdk-1.6.64}/devops_bot_sdk.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: devops-bot-sdk
-Version: 1.6.62
+Version: 1.6.64
 Summary: DevOps Bot Desktop SDK — thin client for the AgentOS Electron desktop app
 Author: noumanaziz2128
 License-Expression: LicenseRef-Proprietary

{devops_bot_sdk-1.6.62 → devops_bot_sdk-1.6.64}/devops_bot_sdk.egg-info/SOURCES.txt

@@ -13,6 +13,7 @@ sdk/cli.py
 sdk/client.py
 sdk/config.py
 sdk/crucial.py
+sdk/custom_agents.py
 sdk/departments.py
 sdk/estimator.py
 sdk/exceptions.py
@@ -147,6 +148,7 @@ tests/test_activity_pipeline.py
 tests/test_agent_activity.py
 tests/test_agent_activity_tracking.py
 tests/test_agent_chat_features.py
+tests/test_agent_crud_cli.py
 tests/test_agent_listing.py
 tests/test_agent_prompt_async.py
 tests/test_agent_prompt_models.py

{devops_bot_sdk-1.6.62 → devops_bot_sdk-1.6.64}/sdk/__init__.py

@@ -1,6 +1,6 @@
 """AgentOS Desktop SDK — thin HTTPS/SSE client for the Electron app.
 
-Version: 1.6.62
+Version: 1.6.64
 
 Public surface:
   BackendClient.from_config()        — create client from ~/.agentos/config.toml
@@ -34,7 +34,7 @@ Rules:
   - All data egress through submit_webhook only
 """
 
-__version__ = "1.6.62"   # SINGLE SOURCE OF TRUTH — bump on every change; pyproject,
+__version__ = "1.6.64"   # SINGLE SOURCE OF TRUTH — bump on every change; pyproject,
 __author__  = "AgentOS"   # sdk.client.SDK_VERSION and the UI server version all read this.
 
 from sdk.client import BackendClient

{devops_bot_sdk-1.6.62 → devops_bot_sdk-1.6.64}/sdk/cli.py

@@ -20,16 +20,28 @@ Command surface:
         runs in createdAt order behind anything already queued.
         See sdk.intake.chat_cli.
 
-    agentos agent new
-        Interactive wizard (input() prompts) — name, description, engine
-        (default "claude_code"), trigger type (schedule/ticket_label/manual/
-        webhook — schedule additionally asks for a cron string, ticket_label
-        for a label), and the standing prompt text. Writes
-        ~/.agentos/agents/<name>.yaml and registers it with the backend.
-    agentos agent add <path>
-        Validate a hand-written manifest YAML, store a copy under
-        ~/.agentos/agents/<name>.yaml (if not already there), and register
-        it with the backend.
+    agentos agent create --name NAME (--prompt TEXT | --prompt-file PATH)
+            [--var NAME[:LABEL[:TYPE[:DESCRIPTION]]]]… [--secret NAME[:DESCRIPTION]]…
+            [--skill ID]…
+    agentos agent create --from-file PATH
+        Create a custom agent through the AI backend (POST /api/v1/agents/custom),
+        which evaluates it — a rejection prints the reasons and score — and
+        forwards it to the platform; the SDK never writes a DB. PATH holds one JSON body
+        ({name, prompt:{system_prompt, variables}, skills, secrets, media}) or
+        {"agents": [...]} to create many; names you already own are skipped.
+        Secrets are NAMES only. Without --var, the {{placeholders}} in the prompt
+        become the variables. See sdk.custom_agents — the Custom Agents page of
+        the control panel uses the same code.
+    agentos agent update NAME_OR_ID [--name NAME] [--prompt[-file] …] [--var …]…
+            [--secret …]… [--skill ID]… [--status active|inactive|disabled] [--from-file PATH]
+        PATCH /api/v1/agents/custom/{name or id} with only the given fields.
+        --var / --secret / --skill each REPLACE that whole list. Your own agents
+        only — the backend answers "not found" for anything else, department
+        agents included.
+    agentos agent delete NAME_OR_ID [--yes]
+        Show the agent and, after a y/N confirmation (skipped by --yes), delete it
+        on the platform (DELETE /api/v1/agents/custom/{name or id}), then forget its local copy
+        and stored keys. Cannot be undone.
     agentos agent list
         Show local (personal) agents (name, engine, trigger type), merged
         with backend status when the backend is reachable, PLUS this user's
@@ -47,22 +59,25 @@ Command surface:
     agentos agent enable <name> / agentos agent disable <name>
         Flip the agent's backend status to "active" / "disabled".
     agentos agent rm <name>
-        De-register the agent from the backend and delete its local manifest.
+        Remove the agent from THIS machine (local manifest, stored API keys, last
+        inputs) and best-effort de-register it by name on the AI backend. To
+        delete the agent itself on the platform, use `agentos agent delete`.
 
     agentos ticket create --project KEY --summary TEXT [--type T] [--priority P]
             [--assignee EMAIL] [--description[-file] …] [--instructions[-file] …]
             [--parent ID] [--status S] [--points N]
-        File one ticket (POST /tasks-ml) and print its Jira key. Type defaults to
-        Task, priority to Medium, assignee to the token owner. Validated locally
-        first; a gateway rejection prints the gateway's own messages.
+        File one ticket (POST /api/v1/tasks on the AI backend) and print its Jira
+        key. Type defaults to Task, priority to Medium, assignee to the token
+        owner (server-side). Validated locally first; a rejection prints the
+        backend's own messages.
     agentos ticket update KEY_OR_ID [--summary …] [--description[-file] …]
             [--instructions[-file] …] [--priority P] [--status S] [--points N]
             [--assignee EMAIL] [--type T]
-        Change only the given fields (PUT /tasks-ml/{id}). Takes a Jira key or
+        Change only the given fields (PATCH /api/v1/tasks/{id}). Takes a Jira key or
         the platform id. See sdk.tickets — the Tickets page uses the same code.
     agentos ticket delete KEY_OR_ID [--yes]
         Show the ticket and, after a y/N confirmation (skipped by --yes), delete it
-        (DELETE /tasks-ml/{id}). Cannot be undone.
+        (DELETE /api/v1/tasks/{id}). Cannot be undone.
 
     agentos integration add <path>
         Register an integration (MCP server, etc.) from a JSON file.
@@ -226,6 +241,7 @@ def _print_usage() -> None:
     print("  agentos configure [--rotate]")
     print("  agentos chat [--product-id ID] [--pv-id ID] [--no-autorun]   # requirements → ticket")
     print("  agentos agent <list|run NAME|chat NAME MSG|logs NAME|enable NAME|disable NAME|rm NAME>")
+    print("  agentos agent <create [flags]|update NAME_OR_ID [flags]|delete NAME_OR_ID [--yes]|help>   # custom-agent CRUD")
     print("  agentos agent <set-key NAME KEY[=VALUE]…|keys NAME|rm-key NAME KEY…>   # persistent API keys (custom agents)")
     print("  agentos ticket <create|update KEY_OR_ID|delete KEY_OR_ID|help>")
     print("  agentos integration <add PATH|test NAME|list|rm NAME>")
@@ -246,13 +262,18 @@ def _agent_command(args: list[str]) -> None:
     verb = args[0] if args else None
     rest = args[1:]
 
-    # Agent CREATION (`new` / `add`) is not owned by the SDK anymore — agents are
-    # authored in the web app and written to the backend `agents` table. The CLI is
-    # consume-only: list / run / chat / logs / enable / disable / rm / keys.
+    # Create / update / delete go through the AI backend's /api/v1/agents/custom (the SDK
+    # never writes a DB) — see sdk.custom_agents. The old local-manifest wizard
+    # (`new` / `add PATH`) stays gone; point at the replacement.
     if verb in ("new", "add"):
-        print("`agentos agent new/add` has been removed — agents are created in the "
-              "web app now. Use `agentos agent list` to see yours and `agentos agent "
-              "run <name>` to run one.")
+        print(f"`agentos agent {verb}` has been replaced by `agentos agent create` "
+              "(flags or --from-file PATH). See `agentos agent help`.")
+        sys.exit(1)
+    if verb in ("help", "-h", "--help"):
+        _agent_crud_usage()
+        sys.exit(0)
+    if verb in ("create", "update", "delete"):
+        _agent_crud_command(verb, rest)
         return
     if verb == "list":
         _agent_list()
@@ -288,7 +309,7 @@ def _agent_command(args: list[str]) -> None:
         _agent_rm_key(rest[0], rest[1:])
         return
 
-    print("usage: agentos agent <new|add PATH|list|run NAME|run-ticket NAME [--ticket KEY] "
+    print("usage: agentos agent <create|update NAME_OR_ID|delete NAME_OR_ID|help|list|run NAME|run-ticket NAME [--ticket KEY] "
           "[--var k=v]… [--secret K=V]… [--scope department --department-id UUID]|chat NAME MESSAGE|"
           "template [NAME]|logs NAME|enable NAME|disable NAME|rm NAME|set-key NAME KEY[=VALUE]…|"
           "keys NAME|rm-key NAME KEY…>")
@@ -506,21 +527,311 @@ def _agent_enable_disable(name: str, enable: bool) -> None:
 
 
 def _agent_rm(name: str) -> None:
+    from sdk.client import BackendClient
+
+    asyncio.run(BackendClient.from_config().delete_custom_agent(name))
+    if _agent_forget_local(name):
+        print(f"removed '{name}' (and its stored API keys)")
+    else:
+        print(f"removed '{name}'")
+
+
+def _agent_forget_local(name: str) -> bool:
+    """Drop this machine's copy of an agent: manifest, last-used variables and
+    stored API keys. True when stored keys were removed."""
     from sdk.agents import secrets as agent_secrets
     from sdk.agents import varstore as agent_varstore
     from sdk.agents.manifest import AGENTS_DIR
-    from sdk.client import BackendClient
 
-    asyncio.run(BackendClient.from_config().delete_custom_agent(name))
     path = AGENTS_DIR / f"{name}.yaml"
     if path.exists():
         path.unlink()
-    # Remove the agent's persisted API keys + last-used variables too.
     agent_varstore.delete_vars(name)
-    if agent_secrets.delete_all(name):
-        print(f"removed '{name}' (and its stored API keys)")
+    return bool(agent_secrets.delete_all(name))
+
+
+# ── agent create / update / delete (AI backend /api/v1/agents/custom) ────
+
+# Flags that take one value. Repeatable list flags are collected separately.
+_AGENT_VALUE_FLAGS = {"--name", "--prompt", "--prompt-file", "--status", "--from-file"}
+_AGENT_LIST_FLAGS = {"--var", "--secret", "--skill"}
+_AGENT_UPDATE_ONLY = {"--status"}
+
+
+def _agent_crud_usage() -> None:
+    print("usage: agentos agent <create|update NAME_OR_ID|delete NAME_OR_ID|help>")
+    print()
+    print("  agentos agent create --name NAME (--prompt TEXT | --prompt-file PATH)")
+    print("      [--var NAME[:LABEL[:TYPE[:DESCRIPTION]]]]… [--secret NAME[:DESCRIPTION]]… [--skill ID]…")
+    print("  agentos agent create --from-file PATH      # one JSON body, or {\"agents\": [...]} (existing names skipped)")
+    print("  agentos agent update NAME_OR_ID [--name NAME] [--prompt TEXT | --prompt-file PATH]")
+    print("      [--var …]… [--secret …]… [--skill ID]… [--status active|inactive|disabled] [--from-file PATH]")
+    print("  agentos agent delete NAME_OR_ID [--yes]    # asks to confirm unless --yes")
+    print()
+    print("  Use {{name}} in the prompt for inputs; without --var they are derived from the prompt.")
+    print("  Secrets are NAMES only (e.g. --secret GITHUB_TOKEN:\"repo access\"); values stay on")
+    print("  your machine (`agentos agent set-key`). On update, --var/--secret/--skill replace the list.")
+    print("  Only your own agents can be changed; department agents are read-only (\"not found\").")
+    print("  A create the backend's evaluation rejects prints `rejected: <reasons> (score N)`.")
+    print("  `agentos agent rm NAME` only removes this machine's copy — `delete` removes the agent.")
+
+
+def _parse_var_flag(value: str) -> dict:
+    parts = value.split(":", 3)
+    parts += [""] * (4 - len(parts))
+    name, label, vtype, desc = (p.strip() for p in parts)
+    out = {"name": name, "label": label or name, "description": desc}
+    out["type"] = vtype or "text"
+    return out
+
+
+def _parse_secret_flag(value: str) -> dict:
+    name, _, desc = value.partition(":")
+    return {"name": name.strip(), "description": desc.strip()}
+
+
+def _read_text_file(flag: str, value: str) -> str:
+    path = Path(value).expanduser()
+    try:
+        return path.read_text(encoding="utf-8")
+    except OSError as exc:
+        print(f"error: {flag}: cannot read {path}: {exc.strerror or exc}")
+        sys.exit(1)
+
+
+def _parse_agent_flags(args: list[str], *, creating: bool) -> tuple[dict, str | None]:
+    """Flags → (wire payload, --from-file path). Exits 1 on an unknown flag, a
+    missing value, a value flag given twice, or an unreadable file."""
+    verb = "create" if creating else "update"
+    single: dict[str, str] = {}
+    lists: dict[str, list[str]] = {}
+    i = 0
+    while i < len(args):
+        flag = args[i]
+        known = flag in _AGENT_VALUE_FLAGS or flag in _AGENT_LIST_FLAGS
+        if not known or (creating and flag in _AGENT_UPDATE_ONLY):
+            print(f"error: unknown option for `agentos agent {verb}`: {flag}")
+            sys.exit(1)
+        if i + 1 >= len(args):
+            print(f"error: expected a value after {flag}")
+            sys.exit(1)
+        value = args[i + 1]
+        if flag in _AGENT_LIST_FLAGS:
+            lists.setdefault(flag, []).append(value)
+        else:
+            if flag in single:
+                print(f"error: pass {flag} once")
+                sys.exit(1)
+            single[flag] = value
+        i += 2
+    if "--prompt" in single and "--prompt-file" in single:
+        print("error: pass --prompt or --prompt-file, not both")
+        sys.exit(1)
+
+    payload: dict = {}
+    if "--name" in single:
+        payload["name"] = single["--name"]
+    prompt_text = single.get("--prompt")
+    if "--prompt-file" in single:
+        prompt_text = _read_text_file("--prompt-file", single["--prompt-file"])
+    prompt: dict = {}
+    if prompt_text is not None:
+        prompt["system_prompt"] = prompt_text
+    if "--var" in lists:
+        prompt["variables"] = [_parse_var_flag(v) for v in lists["--var"]]
+    if prompt:
+        payload["prompt"] = prompt
+    for v in lists.get("--secret", []):
+        if "=" in v:
+            print(f"error: --secret {v.split('=', 1)[0]}=…: secrets are NAMES only — never "
+                  "pass a value. Store it on this machine with `agentos agent set-key`.")
+            sys.exit(1)
+    if "--secret" in lists:
+        payload["secrets"] = [_parse_secret_flag(v) for v in lists["--secret"]]
+    if "--skill" in lists:
+        payload["skills"] = list(lists["--skill"])
+    if "--status" in single:
+        payload["status"] = single["--status"]
+    return payload, single.get("--from-file")
+
+
+def _load_agent_file(path_str: str) -> list[dict] | dict:
+    """--from-file → one body (dict) or many (list, from {"agents": [...]})."""
+    import json
+
+    raw = _read_text_file("--from-file", path_str)
+    try:
+        data = json.loads(raw)
+    except ValueError as exc:
+        print(f"error: --from-file: {path_str} is not valid JSON: {exc}")
+        sys.exit(1)
+    if isinstance(data, dict) and "agents" in data:
+        agents = data["agents"]
+        if not isinstance(agents, list) or not all(isinstance(a, dict) for a in agents):
+            print('error: --from-file: "agents" must be a list of agent objects')
+            sys.exit(1)
+        return agents
+    if isinstance(data, list):
+        if not all(isinstance(a, dict) for a in data):
+            print("error: --from-file: every entry must be an agent object")
+            sys.exit(1)
+        return data
+    if not isinstance(data, dict):
+        print("error: --from-file: expected a JSON object")
+        sys.exit(1)
+    return data
+
+
+def _merge_payload(base: dict, over: dict) -> dict:
+    """Flags win over the file; a partial flag prompt merges into the file's prompt."""
+    out = {**base, **{k: v for k, v in over.items() if k != "prompt"}}
+    if "prompt" in over:
+        file_prompt = base.get("prompt") if isinstance(base.get("prompt"), dict) else {}
+        if isinstance(base.get("prompt"), str):
+            file_prompt = {"system_prompt": base["prompt"]}
+        out["prompt"] = {**file_prompt, **over["prompt"]}
+    return out
+
+
+def _print_warnings(res: dict) -> None:
+    for w in res.get("warnings") or []:
+        print(f"warning: {w}")
+
+
+def _agent_crud_command(verb: str, args: list[str]) -> None:
+    """`agentos agent create|update|delete` — see sdk.custom_agents for the rules."""
+    if verb in {"update", "delete"} and (not args or args[0].startswith("--")):
+        _agent_crud_usage()
+        sys.exit(1)
+    if verb == "delete":
+        _agent_crud_delete(args[0], args[1:])
+        return
+
+    from sdk import custom_agents
+    from sdk.client import BackendClient
+
+    creating = verb == "create"
+    flag_args = args if creating else args[1:]
+    payload, from_file = _parse_agent_flags(flag_args, creating=creating)
+    many: list[dict] | None = None
+    if from_file:
+        loaded = _load_agent_file(from_file)
+        if isinstance(loaded, list):
+            if not creating:
+                print("error: --from-file with several agents is for `create` only")
+                sys.exit(1)
+            if payload:
+                print("error: a bulk --from-file cannot be combined with other flags")
+                sys.exit(1)
+            many = loaded
+        else:
+            loaded.pop("id", None)
+            payload = _merge_payload(loaded, payload)
+
+    if many is not None:
+        # Validate every entry before building a client, so a bad file never needs a token.
+        for n, item in enumerate(many, 1):
+            _, err, _ = custom_agents.build_body(item, creating=True)
+            if err:
+                print(f"error: agent #{n} ({item.get('name') or 'unnamed'}): {err}")
+                sys.exit(1)
+        client = BackendClient.from_config()
+        results = asyncio.run(custom_agents.create_many(client, many))
+        created = skipped = failed = 0
+        for res in results:
+            if res.get("skipped"):
+                skipped += 1
+                print(f"skipped {res['name']}  (already exists)")
+            elif res["ok"]:
+                created += 1
+                print(f"created {res['name']}  (id {res['id']})")
+                _print_warnings(res)
+            else:
+                failed += 1
+                print(f"error: {res.get('name') or ''}: {res['error']}".replace("error: : ", "error: "))
+        print(f"{created} created, {skipped} skipped, {failed} failed")
+        sys.exit(1 if failed else 0)
+
+    if creating:
+        if not payload.get("name") or not (
+                isinstance(payload.get("prompt"), (dict, str)) and payload.get("prompt")):
+            print("error: --name and --prompt (or --prompt-file / --from-file) are required")
+            sys.exit(1)
+    elif not payload:
+        print("error: nothing to update: pass at least one field")
+        _agent_crud_usage()
+        sys.exit(1)
+    # Validate before building a client, so a bad flag never needs a token.
+    if creating:
+        _, err, _ = custom_agents.build_body(payload, creating=True)
     else:
-        print(f"removed '{name}'")
+        err = custom_agents.precheck_update(payload)
+    if err:
+        print(f"error: {err}")
+        sys.exit(1)
+
+    client = BackendClient.from_config()
+    if creating:
+        res = asyncio.run(custom_agents.create_agent(client, payload))
+    else:
+        res = asyncio.run(custom_agents.update_agent(client, args[0], payload))
+    _print_warnings(res)
+    if not res["ok"]:
+        print(f"error: {res['error']}")
+        sys.exit(1)
+    if creating:
+        print(f"created {res['name']}  (id {res['id'] or 'unknown'})")
+        ev = res.get("evaluation") or {}
+        if isinstance(ev, dict) and ev.get("score") is not None:
+            print(f"evaluation: accepted, score {ev['score']}")
+    else:
+        print(f"updated {res['name']}: {', '.join(res.get('fields') or sorted(payload))}")
+    sys.exit(0)
+
+
+def _agent_crud_delete(target: str, flags: list[str]) -> None:
+    """`agentos agent delete NAME_OR_ID [--yes]` — shows the agent, then asks."""
+    from sdk import custom_agents
+    from sdk.client import BackendClient
+
+    unknown = [f for f in flags if f not in {"--yes", "-y"}]
+    if unknown:
+        print(f"error: unknown option for `agentos agent delete`: {unknown[0]}")
+        sys.exit(1)
+    client = BackendClient.from_config()
+    rec: dict | None = None
+    name = target
+    if not flags:
+        # Show what is about to go. The server is the authority on "is it yours",
+        # so only a definite "not found" stops here.
+        rec, err = asyncio.run(custom_agents.get_agent(client, target))
+        if rec is None and err.startswith("agent not found"):
+            print(f"error: {err}")
+            sys.exit(1)
+        if rec is not None:
+            name = rec["name"] or rec["id"]
+            first = (rec["prompt"]["system_prompt"].strip().splitlines() or [""])[0][:100]
+            print(f"{name}  (id {rec['id']})  [{rec.get('status') or '?'}]  {first}")
+        try:
+            answer = input(f"Delete agent {name}? This cannot be undone. [y/N] ")
+        except EOFError:
+            answer = ""
+        if answer.strip().lower() not in {"y", "yes"}:
+            print("not deleted")
+            sys.exit(1)
+    res = asyncio.run(custom_agents.delete_agent(client, rec["id"] if rec else target))
+    if not res["ok"]:
+        print(f"error: {res['error']}")
+        sys.exit(1)
+    local_name = (rec or {}).get("name") or target
+    if local_name:
+        try:
+            _agent_forget_local(local_name)
+        except Exception as exc:  # noqa: BLE001 — the platform delete already happened
+            print(f"warning: deleted on the platform, but this machine's copy was not "
+                  f"cleaned up ({exc}); run `agentos agent rm {local_name}`")
+    print(f"deleted {name}")
+    sys.exit(0)
 
 
 def _agent_exists(name: str) -> bool:
@@ -542,7 +853,7 @@ def _agent_set_key(name: str, pairs: list[str]) -> None:
 
     if not _agent_exists(name):
         print(f"error: no local agent named '{name}'. Install it first "
-              f"(agentos agent add / default-agents activate).")
+              f"(`agentos agent list` shows what this machine has; create one with `agentos agent create`).")
         sys.exit(1)
 
     updates: dict[str, str] = {}
@@ -645,7 +956,7 @@ def _pop_department_id(args: list[str]) -> tuple[str | None, list[str]]:
     return dept_id, out
 
 
-# `agentos ticket` flag → POST/PUT /tasks-ml wire name. Text flags have a
+# `agentos ticket` flag → POST/PATCH /api/v1/tasks wire name. Text flags have a
 # `-file` twin that reads the value from a file instead.
 _TICKET_FLAGS = {
     "--project": "projectKey", "--type": "type", "--summary": "summary",
@@ -655,7 +966,7 @@ _TICKET_FLAGS = {
 }
 _TICKET_FILE_FLAGS = {"--description-file": "--description",
                       "--instructions-file": "--instructions"}
-# Create-only: neither is editable through PUT /tasks-ml/{id}.
+# Create-only: neither is editable through PATCH /api/v1/tasks/{id}.
 _TICKET_CREATE_ONLY = {"--project", "--parent"}
 
 
@@ -830,7 +1141,7 @@ def _require_department_manager(verb: str) -> None:
               f"activate department agents, but only department managers (team_lead+) "
               f"can '{verb}' them.")
         print("note: your own custom agents are unrestricted — use `agentos agent "
-              "<new|add|enable|disable|rm> …`.")
+              "<create|update|delete|enable|disable|rm> …`.")
         sys.exit(1)