comodor 0.8.4__tar.gz → 0.8.8__tar.gz

{comodor-0.8.4 → comodor-0.8.8}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: comodor
-Version: 0.8.4
+Version: 0.8.8
 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
@@ -22,6 +22,7 @@ Requires-Python: >=3.11
 Requires-Dist: rich>=13.7
 Provides-Extra: dev
 Requires-Dist: pytest>=8.0; extra == 'dev'
+Requires-Dist: ruff>=0.14; extra == 'dev'
 Description-Content-Type: text/markdown
 
 # Comodor
@@ -114,14 +115,14 @@ Then type `comodor`.
 
 ## First run
 
-Four questions, once. Nothing to create beforehand — no config file, no
+Five questions, once. Nothing to create beforehand — no config file, no
 environment variable, no documentation to read first.
 
 ```
  ✓ provider  Ollama (local)
  ✓ api key   not needed
 
- 3/4 Which model?
+ 3/5 Which model?
 ┌─  Models  ──────────────────────────────────────────────┐
 │ ›  qwen2.5-coder:14b  recommended                       │
 │    llama3.3                                             │
@@ -141,6 +142,19 @@ remembered.
 
 You are not asked again. Change your mind later with `comodor setup`.
 
+**Already using another agent?** If OpenClaw or Hermes is installed, the first
+screen offers to bring your API keys, your model and your skills across — the
+tedious part, which you have already done once. Nothing is moved and nothing
+already set here is replaced, so the other tool keeps working exactly as it
+did. What deliberately does not come over is said rather than skipped in
+silence: their memory is prose, and this agent's is lessons with confidence and
+evidence, so importing one as the other would poison recall with entries
+nothing earned.
+
+Afterwards, or if you install one of them later, `comodor import` does the same
+thing — with `--dry-run` to see what it would take, and `--keys-only` to leave
+the rest.
+
 **No API key?** `comodor --demo` runs the whole interface offline — every
 panel, every command, no account required.
 
@@ -457,6 +471,35 @@ comodor --provider groq --model llama-3.3-70b-versatile
 
 ---
 
+## It tells you what it knows
+
+Starting Comodor — the interface, a headless run, or the browser server — puts
+this at the top:
+
+```
+   ______                          __
+  / ____/___  ____ ___  ____  ____/ /___  _____
+ / /   / __ \/ __ `__ \/ __ \/ __  / __ \/ ___/
+/ /___/ /_/ / / / / / / /_/ / /_/ / /_/ / /
+\____/\____/_/ /_/ /_/\____/\__,_/\____/_/
+
+  it learns the way you correct it   0.8.6  ·  claude-sonnet-5
+  412 lessons · 7 skills · 3 rules you set   from 128 finished tasks
+  ~/work/your-project
+```
+
+The second line is the one worth the space. Most banners are a logo and a
+version number nobody reads twice; this one says what *this* installation has
+accumulated, and that number goes up as you use it.
+
+It stays out of the way where a banner would be noise: never in a pipe, so
+`comodor run … | jq` sees the answer and nothing else; never in `--json`;
+never in CI. Below 51 columns the art is dropped rather than reflowed, because
+ASCII art wrapped by a terminal is not a smaller logo. `COMODOR_BANNER=0`
+switches it off for one run and `"banner": false` for good.
+
+---
+
 ## A browser, and the reason it is not screenshots
 
 Comodor drives a real Chrome — one that runs JavaScript, keeps cookies and can
@@ -750,6 +793,23 @@ comodor preview 120x34            # render one frame at a fixed size
 
 ---
 
+## Checking it
+
+```bash
+pip install -e ".[dev]"
+ruff check src tests      # the linter
+pytest -q                 # the suite
+```
+
+Those two commands are the gate, and they are the same two the release runs
+before it builds anything. Nothing reaches PyPI that has not passed them: the
+release workflow's first job is the gate, and building, publishing and the
+GitHub release all depend on it.
+
+That is not a precaution against a hypothetical. Release triggers on a tag and
+CI on a push; before this they were unrelated, and 0.8.0 went to PyPI while CI
+was red.
+
 ## Contributing
 
 Bug reports welcome — please include `comodor doctor`, which prints everything

{comodor-0.8.4 → comodor-0.8.8}/README.md

@@ -88,14 +88,14 @@ Then type `comodor`.
 
 ## First run
 
-Four questions, once. Nothing to create beforehand — no config file, no
+Five questions, once. Nothing to create beforehand — no config file, no
 environment variable, no documentation to read first.
 
 ```
  ✓ provider  Ollama (local)
  ✓ api key   not needed
 
- 3/4 Which model?
+ 3/5 Which model?
 ┌─  Models  ──────────────────────────────────────────────┐
 │ ›  qwen2.5-coder:14b  recommended                       │
 │    llama3.3                                             │
@@ -115,6 +115,19 @@ remembered.
 
 You are not asked again. Change your mind later with `comodor setup`.
 
+**Already using another agent?** If OpenClaw or Hermes is installed, the first
+screen offers to bring your API keys, your model and your skills across — the
+tedious part, which you have already done once. Nothing is moved and nothing
+already set here is replaced, so the other tool keeps working exactly as it
+did. What deliberately does not come over is said rather than skipped in
+silence: their memory is prose, and this agent's is lessons with confidence and
+evidence, so importing one as the other would poison recall with entries
+nothing earned.
+
+Afterwards, or if you install one of them later, `comodor import` does the same
+thing — with `--dry-run` to see what it would take, and `--keys-only` to leave
+the rest.
+
 **No API key?** `comodor --demo` runs the whole interface offline — every
 panel, every command, no account required.
 
@@ -431,6 +444,35 @@ comodor --provider groq --model llama-3.3-70b-versatile
 
 ---
 
+## It tells you what it knows
+
+Starting Comodor — the interface, a headless run, or the browser server — puts
+this at the top:
+
+```
+   ______                          __
+  / ____/___  ____ ___  ____  ____/ /___  _____
+ / /   / __ \/ __ `__ \/ __ \/ __  / __ \/ ___/
+/ /___/ /_/ / / / / / / /_/ / /_/ / /_/ / /
+\____/\____/_/ /_/ /_/\____/\__,_/\____/_/
+
+  it learns the way you correct it   0.8.6  ·  claude-sonnet-5
+  412 lessons · 7 skills · 3 rules you set   from 128 finished tasks
+  ~/work/your-project
+```
+
+The second line is the one worth the space. Most banners are a logo and a
+version number nobody reads twice; this one says what *this* installation has
+accumulated, and that number goes up as you use it.
+
+It stays out of the way where a banner would be noise: never in a pipe, so
+`comodor run … | jq` sees the answer and nothing else; never in `--json`;
+never in CI. Below 51 columns the art is dropped rather than reflowed, because
+ASCII art wrapped by a terminal is not a smaller logo. `COMODOR_BANNER=0`
+switches it off for one run and `"banner": false` for good.
+
+---
+
 ## A browser, and the reason it is not screenshots
 
 Comodor drives a real Chrome — one that runs JavaScript, keeps cookies and can
@@ -724,6 +766,23 @@ comodor preview 120x34            # render one frame at a fixed size
 
 ---
 
+## Checking it
+
+```bash
+pip install -e ".[dev]"
+ruff check src tests      # the linter
+pytest -q                 # the suite
+```
+
+Those two commands are the gate, and they are the same two the release runs
+before it builds anything. Nothing reaches PyPI that has not passed them: the
+release workflow's first job is the gate, and building, publishing and the
+GitHub release all depend on it.
+
+That is not a precaution against a hypothetical. Release triggers on a tag and
+CI on a push; before this they were unrelated, and 0.8.0 went to PyPI while CI
+was red.
+
 ## Contributing
 
 Bug reports welcome — please include `comodor doctor`, which prints everything

{comodor-0.8.4 → comodor-0.8.8}/pyproject.toml

@@ -32,7 +32,7 @@ Issues = "https://github.com/ifekri/Comodor/issues"
 Changelog = "https://github.com/ifekri/Comodor/blob/main/CHANGELOG.md"
 
 [project.optional-dependencies]
-dev = ["pytest>=8.0"]
+dev = ["pytest>=8.0", "ruff>=0.14"]
 
 [project.scripts]
 comodor = "comodor.cli:main"
@@ -41,6 +41,29 @@ comodor = "comodor.cli:main"
 # with it, so nothing can fall out of step with it: `git tag v0.3.0` is the
 # whole of a release. `_version.py` is generated at build time - including by an
 # editable install - and is not committed.
+[tool.ruff]
+line-length = 100
+target-version = "py311"
+extend-exclude = ["src/comodor/_version.py"]
+
+[tool.ruff.lint]
+# Chosen by counting what each family actually found in this tree, not by
+# taking a preset. The reasoning is in the commit that added this.
+select = ["F", "E4", "E7", "E9", "E501", "W", "I", "B", "C4", "PIE"]
+
+[tool.ruff.lint.per-file-ignores]
+# A table of models, one per line, whose columns line up across rows.
+# That alignment is why it can be read at a glance, and wrapping every
+# row onto two lines to satisfy a column count would cost more than the
+# rule is worth here.
+"src/comodor/providers/registry.py" = ["E501"]
+# A test may import a name only to prove it is importable, and its fixtures
+# shadow each other by design.
+"tests/*" = ["B008"]
+
+[tool.ruff.lint.isort]
+known-first-party = ["comodor"]
+
 [tool.hatch.version]
 source = "vcs"
 

{comodor-0.8.4 → comodor-0.8.8}/src/comodor/_version.py

@@ -18,7 +18,7 @@ version_tuple: tuple[int | str, ...]
 commit_id: str | None
 __commit_id__: str | None
 
-__version__ = version = '0.8.4'
-__version_tuple__ = version_tuple = (0, 8, 4)
+__version__ = version = '0.8.8'
+__version_tuple__ = version_tuple = (0, 8, 8)
 
 __commit_id__ = commit_id = None

{comodor-0.8.4 → comodor-0.8.8}/src/comodor/agent/context.py

@@ -139,7 +139,7 @@ class Conversation:
     # -- introspection ---------------------------------------------------- #
 
     def summary_line(self) -> str:
-        roles = {role: 0 for role in ("user", "assistant", "tool")}
+        roles = dict.fromkeys(("user", "assistant", "tool"), 0)
         for message in self.messages:
             key = message.role.value
             if key in roles:

{comodor-0.8.4 → comodor-0.8.8}/src/comodor/agent/loop.py

@@ -256,7 +256,7 @@ class AgentLoop:
         else:
             results = [self._run_one(call, context) for call in calls]
 
-        for call, result in zip(calls, results):
+        for call, result in zip(calls, results, strict=True):
             message = Message.tool(
                 call_id=call.id, name=call.name,
                 content=result.content, is_error=not result.ok,

{comodor-0.8.4 → comodor-0.8.8}/src/comodor/catalogue.py

@@ -65,8 +65,12 @@ CATALOGUE: tuple[ProviderSpec, ...] = (
         base_url="https://api.anthropic.com/v1",
         keys_url="https://console.anthropic.com/settings/keys",
         blurb="Claude, direct from the source. Strongest at long agentic work.",
-        default_model="claude-sonnet-4-5",
-        models=("claude-opus-4-1", "claude-sonnet-4-5", "claude-haiku-4-5"),
+        # These have to be models the pricing registry knows. It did not know
+        # any of the three that were here, so the wizard's own default landed
+        # on an unpriced model: the context gauge read a fallback 128k instead
+        # of a million, and the spend limit could not be enforced at all.
+        default_model="claude-sonnet-5",
+        models=("claude-opus-5", "claude-sonnet-5", "claude-haiku-4-5"),
         env_key="ANTHROPIC_API_KEY",
         rank=2,
     ),

{comodor-0.8.4 → comodor-0.8.8}/src/comodor/cli.py

@@ -27,7 +27,8 @@ import sys
 from typing import Any
 
 from . import __version__
-from .config import Config, load as load_config
+from .config import Config
+from .config import load as load_config
 
 
 def build_parser() -> argparse.ArgumentParser:
@@ -62,6 +63,12 @@ def build_parser() -> argparse.ArgumentParser:
     run.add_argument("--max-steps", type=int, help="override the step limit")
 
     sub.add_parser("setup", help="choose a provider and model, and save the answers")
+    brought = sub.add_parser(
+        "import", help="bring keys, model and skills over from another agent")
+    brought.add_argument("--dry-run", action="store_true",
+                         help="say what would come over, and change nothing")
+    brought.add_argument("--keys-only", action="store_true",
+                         help="leave the skills and the model where they are")
     doctor = sub.add_parser(
         "doctor", help="check everything, and repair what can be repaired")
     doctor.add_argument("--fix", action="store_true",
@@ -122,15 +129,42 @@ def apply_overrides(config: Config, args: argparse.Namespace) -> Config:
 # --------------------------------------------------------------------------- #
 
 
+def _greet_on_stderr(config: Config, memory: Any, skills: Any) -> None:
+    """The wordmark, on the error stream with the progress.
+
+    Same reason the progress is there: `comodor run … | jq` has to see the
+    answer and nothing else, and a logo in the middle of a JSON document is
+    somebody else's afternoon.
+    """
+    from rich.console import Console
+
+    from .ui import banner
+    from .ui import console as console_module
+
+    theme = console_module.prepare_theme(config.ui.theme, config.ui.ascii_borders,
+                                         no_color=False)
+    errors = Console(stderr=True, theme=theme.rich_theme())
+    banner.show(errors, theme, config,
+                version=_release(), model=config.active_model(),
+                project=str(config.paths.project),
+                standing=banner.gather(memory, skills))
+
+
+def _release() -> str:
+    from . import __version__
+
+    return __version__
+
+
 def run_headless(config: Config, args: argparse.Namespace) -> int:
     """One task, no TUI. Used by scripts, hooks and CI."""
     from .agent import AgentLoop, Conversation
     from .agent.spawn import spawner
     from .events import EventBus, Kind
     from .learning import LearningEngine
+    from .mcp import MCPManager
     from .providers.gateway import Gateway
     from .safety import CheckpointStore, PermissionEngine, Redactor
-    from .mcp import MCPManager
     from .skills import load_for as load_skills
     from .tools import ToolRegistry
 
@@ -161,6 +195,14 @@ def run_headless(config: Config, args: argparse.Namespace) -> int:
                       permissions, Conversation(), memory, skills=skills)
 
     if not args.json:
+        _greet_on_stderr(config, memory, skills)
+        for note in config.complaints:
+            print(f"config: {note}", file=sys.stderr)
+        if config.project_refused:
+            print(f"config: this project cannot set "
+                  f"{', '.join(sorted(config.project_refused))} — only yours can",
+                  file=sys.stderr)
+
         # Progress on stderr keeps stdout clean for the answer itself.
         def echo(event: Any) -> None:
             if event.kind is Kind.TOOL_START:
@@ -330,8 +372,8 @@ def run_update(config: Config, check_only: bool = False) -> int:
     on: what is out there, whether it is newer than this, how this copy would
     be upgraded, and what version answers afterwards.
     """
-    from .ui import console as console_module
     from . import update as updater
+    from .ui import console as console_module
 
     theme = console_module.prepare_theme(config.ui.theme, config.ui.ascii_borders,
                                          no_color=False)
@@ -563,6 +605,91 @@ def run_preview(config: Config, args: argparse.Namespace) -> int:
 # --------------------------------------------------------------------------- #
 
 
+def _say_what_the_copy_refused(console, found: list, already: dict) -> None:
+    """Reasons the copy added after the listing was printed."""
+    for entry in found:
+        for note in entry.passed_over[already[id(entry)]:]:
+            console.print(f"[dim]  not imported: {note}[/dim]")
+
+
+def run_import(config: Config, args: argparse.Namespace) -> int:
+    """Bring another agent's settings over, outside the first run.
+
+    Same rules as the wizard's step: nothing already set here is replaced,
+    nothing is moved, and what was deliberately left behind is said rather than
+    skipped silently.
+    """
+    from . import migrate
+    from .ui import console as console_module
+
+    theme = console_module.prepare_theme(config.ui.theme, config.ui.ascii_borders,
+                                         no_color=False)
+    console = console_module.build(theme)
+
+    try:
+        found = migrate.discover()
+    except OSError as error:
+        console.print(f"[bad]could not look: {error}[/bad]")
+        return 1
+
+    if not found:
+        console.print("Nothing to import — no OpenClaw or Hermes installation "
+                      "was found in your home directory.")
+        return 0
+
+    for entry in found:
+        console.print(f"[accent]{entry.tool}[/accent]  {entry.summary()}")
+        console.print(f"[dim]{entry.root}[/dim]")
+        for note in entry.passed_over:
+            console.print(f"[dim]  not imported: {note}[/dim]")
+
+    # Anything the copy itself decides against - a skill folder holding a link
+    # out of itself, one far larger than a skill - is added to this list while
+    # applying, which is after the lines above. Remembering where each list
+    # ended is what lets those reasons reach the screen, rather than leaving
+    # the reader to wonder why a skill they can see did not arrive.
+    already = {id(entry): len(entry.passed_over) for entry in found}
+
+    if args.dry_run:
+        console.print("\n[dim]--dry-run: nothing was changed.[/dim]")
+        return 0
+
+    took = migrate.Outcome()
+    for entry in found:
+        try:
+            got = migrate.apply(entry, config, take_keys=True,
+                                take_skills=not args.keys_only,
+                                take_model=not args.keys_only)
+        except OSError as error:
+            console.print(f"[bad]could not read {entry.tool}: {error}[/bad]")
+            continue
+        took.keys += got.keys
+        took.skills += got.skills
+        took.skipped += got.skipped
+        took.model = took.model or got.model
+
+    if not took.anything:
+        console.print("\nNothing new — everything it found is already set here.")
+        for note in took.skipped:
+            console.print(f"[dim]  {note}[/dim]")
+        _say_what_the_copy_refused(console, found, already)
+        return 0
+
+    config.save()
+    console.print()
+    _say_what_the_copy_refused(console, found, already)
+    if took.keys:
+        console.print(f"[good]keys[/good]    {', '.join(took.keys)}")
+    if took.model:
+        console.print(f"[good]model[/good]   {took.model}")
+    if took.skills:
+        console.print(f"[good]skills[/good]  {', '.join(took.skills)}")
+    for note in took.skipped:
+        console.print(f"[dim]  kept as it was — {note}[/dim]")
+    console.print(f"\n[dim]saved to {config.paths.config_file}[/dim]")
+    return 0
+
+
 def main(argv: list[str] | None = None) -> int:
     args = build_parser().parse_args(argv)
     config = apply_overrides(load_config(args.cwd), args)
@@ -589,6 +716,8 @@ def main(argv: list[str] | None = None) -> int:
         return run_preview(config, args)
     if args.command == "setup":
         return run_setup_command(config)
+    if args.command == "import":
+        return run_import(config, args)
     if args.command == "run":
         return run_headless(config, args)