comodor 0.20.2__tar.gz → 0.20.3__tar.gz

{comodor-0.20.2 → comodor-0.20.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: comodor
-Version: 0.20.2
+Version: 0.20.3
 Summary: Comodor — a self-improving terminal coding agent with a Rich TUI
 Project-URL: Homepage, https://comodor.ai
 Project-URL: Repository, https://github.com/ifekri/Comodor
@@ -136,7 +136,7 @@ The whole table, and how to run it against your own model, is in
 | **Runs a model locally** | Pick one, watch it download, use it with the network unplugged. No key, no account. [→](docs/local-models.md) |
 | **Lives in your editor** | Speaks ACP, so VS Code, JetBrains and Zed drive it from their own panels. [→](docs/acp.md) |
 | **Follows your procedures** | Write a skill once; it loads when the work matches. 147 ready to install. [→](docs/skills.md) |
-| **Works with any model** | Nineteen providers, or anything with an OpenAI-compatible URL. [→](docs/models.md) |
+| **Works with any model** | Sixteen hosted providers and three local runtimes, or anything with an OpenAI-compatible URL. [→](docs/models.md) |
 | **Costs less** | 86% of input tokens served from cache, measured — not estimated. [→](docs/cost.md) |
 
 **One dependency.** The HTTP client, the SSE reader, the WebSocket that drives
@@ -215,9 +215,13 @@ The agent works on `./work`; point it somewhere else with
 anything:
 
 ```bash
-docker run --rm -it -p 127.0.0.1:8765:8765   -e ANTHROPIC_API_KEY -v "$PWD:/work" ghcr.io/ifekri/comodor:latest
+docker run --rm -it -p 127.0.0.1:8765:8765 \
+  -e ANTHROPIC_API_KEY -v "$PWD:/work" ifekri/comodor:latest
 ```
 
+Published to both [Docker Hub](https://hub.docker.com/r/ifekri/comodor)
+(`ifekri/comodor`) and GitHub (`ghcr.io/ifekri/comodor`) on every release.
+
 Full details, including how to reach it from another machine safely:
 [Docker](docs/docker.md).
 

{comodor-0.20.2 → comodor-0.20.3}/README.md

@@ -105,7 +105,7 @@ The whole table, and how to run it against your own model, is in
 | **Runs a model locally** | Pick one, watch it download, use it with the network unplugged. No key, no account. [→](docs/local-models.md) |
 | **Lives in your editor** | Speaks ACP, so VS Code, JetBrains and Zed drive it from their own panels. [→](docs/acp.md) |
 | **Follows your procedures** | Write a skill once; it loads when the work matches. 147 ready to install. [→](docs/skills.md) |
-| **Works with any model** | Nineteen providers, or anything with an OpenAI-compatible URL. [→](docs/models.md) |
+| **Works with any model** | Sixteen hosted providers and three local runtimes, or anything with an OpenAI-compatible URL. [→](docs/models.md) |
 | **Costs less** | 86% of input tokens served from cache, measured — not estimated. [→](docs/cost.md) |
 
 **One dependency.** The HTTP client, the SSE reader, the WebSocket that drives
@@ -184,9 +184,13 @@ The agent works on `./work`; point it somewhere else with
 anything:
 
 ```bash
-docker run --rm -it -p 127.0.0.1:8765:8765   -e ANTHROPIC_API_KEY -v "$PWD:/work" ghcr.io/ifekri/comodor:latest
+docker run --rm -it -p 127.0.0.1:8765:8765 \
+  -e ANTHROPIC_API_KEY -v "$PWD:/work" ifekri/comodor:latest
 ```
 
+Published to both [Docker Hub](https://hub.docker.com/r/ifekri/comodor)
+(`ifekri/comodor`) and GitHub (`ghcr.io/ifekri/comodor`) on every release.
+
 Full details, including how to reach it from another machine safely:
 [Docker](docs/docker.md).
 

{comodor-0.20.2 → comodor-0.20.3}/src/comodor/_version.py

@@ -18,7 +18,7 @@ version_tuple: tuple[int | str, ...]
 commit_id: str | None
 __commit_id__: str | None
 
-__version__ = version = '0.20.2'
-__version_tuple__ = version_tuple = (0, 20, 2)
+__version__ = version = '0.20.3'
+__version_tuple__ = version_tuple = (0, 20, 3)
 
 __commit_id__ = commit_id = None

{comodor-0.20.2 → comodor-0.20.3}/src/comodor/catalogue.py

@@ -231,6 +231,21 @@ CATALOGUE: tuple[ProviderSpec, ...] = (
         env_key="XIAOMI_API_KEY",
         rank=15,
     ),
+    ProviderSpec(
+        id="bai",
+        label="B.AI",
+        base_url="https://api.b.ai/v1",
+        keys_url="https://b.ai",
+        blurb="One key, many houses' models — GLM, DeepSeek, MiniMax, GPT.",
+        # Chosen because it is the one verified against the live endpoint: it
+        # returns a well-formed `tool_calls` and a `finish_reason` of
+        # `tool_calls`, which is what makes a provider usable for coding at all.
+        default_model="glm-5.3-flash",
+        models=("glm-5.3-flash", "deepseek-v4-flash", "glm-5.2",
+                "minimax-m3", "gpt-5.6-sol"),
+        env_key="BAI_API_KEY",
+        rank=16,
+    ),
     ProviderSpec(
         id="ollama",
         label="Ollama (local)",

{comodor-0.20.2 → comodor-0.20.3}/tests/test_docker.py

@@ -304,3 +304,53 @@ def test_the_code_really_does_start_without_a_provider():
     assert "return config" in source
     assert "No provider is configured yet" in source, (
         "the message the documentation quotes is not the one the code prints")
+
+
+# --------------------------------------------------------------------------- #
+# where the image is published
+# --------------------------------------------------------------------------- #
+
+
+def test_it_publishes_to_both_registries():
+    """Docker Hub is where somebody looks first — `docker run comodor` with no
+    prefix means Docker Hub — and GHCR is where a fork can publish without any
+    secret at all."""
+    steps = yaml.safe_load(text(WORKFLOW))["jobs"]["build"]["steps"]
+    names = [step.get("name", "") for step in steps]
+
+    assert any("GitHub's registry" in name for name in names)
+    assert any("Docker Hub" in name for name in names)
+
+
+def test_both_registries_come_from_one_build():
+    """Two builds would publish two images from two runs of the same
+    Dockerfile, and only one of them would have been tested."""
+    steps = yaml.safe_load(text(WORKFLOW))["jobs"]["build"]["steps"]
+    builds = [step for step in steps
+              if "build-push-action" in str(step.get("uses", ""))]
+
+    assert len(builds) == 1, f"{len(builds)} builds; the tested one is ambiguous"
+    assert "steps.tags.outputs.list" in str(builds[0]["with"]["tags"])
+
+
+def test_a_missing_docker_hub_secret_skips_rather_than_fails():
+    """A fork has neither secret, and an image that reaches GHCR is still a
+    published image."""
+    body = text(WORKFLOW)
+
+    assert "steps.hub.outputs.ready == 'yes'" in body
+    assert "No Docker Hub credentials" in body
+
+
+def test_the_secrets_are_read_by_the_names_that_were_set():
+    body = text(WORKFLOW)
+
+    assert "secrets.DOCKERHUB_USERNAME" in body
+    assert "secrets.DOCKERHUB_TOKEN" in body
+
+
+def test_the_documentation_names_both():
+    for page in (ROOT / "README.md", ROOT / "docs" / "docker.md"):
+        body = text(page)
+        assert "ifekri/comodor" in body, f"{page.name}: no Docker Hub image"
+        assert "ghcr.io/ifekri/comodor" in body, f"{page.name}: no GHCR image"

{comodor-0.20.2 → comodor-0.20.3}/tests/test_providers.py

@@ -529,3 +529,102 @@ def test_the_user_agent_says_comodor():
     assert DEFAULT_USER_AGENT.startswith("Comodor/")
     assert "comodor.ai" in DEFAULT_USER_AGENT
     assert "requests/" not in DEFAULT_USER_AGENT
+
+
+# --------------------------------------------------------------------------- #
+# the count in the documentation, against the catalogue
+#
+# "Nineteen providers" was in the README, and adding one made it wrong without
+# anything saying so. A number written in prose is a number that goes stale.
+# --------------------------------------------------------------------------- #
+
+
+def test_the_readme_counts_the_providers_that_exist():
+    import re
+    from pathlib import Path
+
+    from comodor import catalogue
+
+    readme = (Path(__file__).resolve().parents[1] / "README.md").read_text(
+        encoding="utf-8")
+
+    words = {2: "Two", 3: "Three", 4: "Four", 5: "Five", 6: "Six", 7: "Seven",
+             8: "Eight", 9: "Nine", 10: "Ten", 11: "Eleven", 12: "Twelve",
+             13: "Thirteen", 14: "Fourteen", 15: "Fifteen", 16: "Sixteen",
+             17: "Seventeen", 18: "Eighteen", 19: "Nineteen", 20: "Twenty"}
+    hosted = words[len(catalogue.hosted())]
+    local = words[len(catalogue.local())].lower()
+
+    claim = re.search(r"\|\s*\*\*Works with any model\*\*\s*\|([^|]+)\|", readme)
+    assert claim, "the README no longer says what it works with"
+
+    line = claim.group(1)
+    assert f"{hosted} hosted" in line, \
+        f"the README says {line.strip()[:60]!r}; there are {hosted.lower()} hosted"
+    assert f"{local} local" in line, \
+        f"the README does not say there are {local} local runtimes"
+
+
+def test_every_hosted_provider_is_listed_in_the_models_page():
+    from pathlib import Path
+
+    from comodor import catalogue
+
+    page = (Path(__file__).resolve().parents[1] / "docs" / "models.md").read_text(
+        encoding="utf-8")
+
+    missing = [spec.label for spec in catalogue.hosted()
+               if spec.label.split(" (")[0] not in page]
+    assert not missing, f"not documented: {missing}"
+
+
+def test_bai_is_reachable_and_makes_tool_calls():
+    """Live, against the real endpoint, because the only thing that matters
+    about a new provider is whether it can drive tools — a provider that
+    cannot is unusable for coding, and finding that out on the sixth turn of a
+    real task is the failure this prevents.
+
+    Skipped without a key, so the suite stays offline for everyone else.
+    """
+    import json
+    import os
+    import urllib.request
+    from pathlib import Path
+
+    env = Path(__file__).resolve().parents[1] / "src" / ".env"
+    key = os.environ.get("BAI_API_KEY", "")
+    if not key and env.is_file():
+        for line in env.read_text(encoding="utf-8").splitlines():
+            if line.startswith("BAI_API_KEY="):
+                key = line.partition("=")[2].strip()
+    if not key:
+        pytest.skip("no BAI_API_KEY")
+
+    from comodor import catalogue
+
+    spec = catalogue.get("bai")
+    request = urllib.request.Request(
+        f"{spec.base_url}/chat/completions",
+        data=json.dumps({
+            "model": spec.default_model,
+            "messages": [{"role": "user", "content": "Weather in Tehran? Use the tool."}],
+            "tools": [{"type": "function", "function": {
+                "name": "get_weather",
+                "description": "Weather for a city.",
+                "parameters": {"type": "object",
+                               "properties": {"city": {"type": "string"}},
+                               "required": ["city"]}}}],
+            "max_tokens": 300,
+        }).encode(),
+        headers={"Authorization": f"Bearer {key}",
+                 "Content-Type": "application/json"})
+
+    try:
+        with urllib.request.urlopen(request, timeout=90) as reply:
+            body = json.loads(reply.read().decode())
+    except Exception as problem:                       # rate limit, or offline
+        pytest.skip(f"B.AI unreachable: {type(problem).__name__}")
+
+    choice = body["choices"][0]
+    assert choice.get("finish_reason") == "tool_calls"
+    assert choice["message"].get("tool_calls"), "no tool call came back"