rhylthyme-cli-runner 0.2.5a0__tar.gz → 0.2.7a0__tar.gz

{rhylthyme_cli_runner-0.2.5a0/src/rhylthyme_cli_runner.egg-info → rhylthyme_cli_runner-0.2.7a0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: rhylthyme-cli-runner
-Version: 0.2.5a0
+Version: 0.2.7a0
 Summary: CLI runner for Rhylthyme real-time program schedules
 Home-page: https://github.com/rhylthyme/rhylthyme-cli-runner
 Author: Rhylthyme Team
@@ -561,8 +561,17 @@ rhylthyme search "pad thai" -i spoonacular                           # then impo
 ```
 
 **Options:** `-i IMPORTER` (default: chosen from the URL), `-o PATH`,
-`--stdout`, `--publish` (and `--open`), `--no-validate`. `-` reads pasted
-source from stdin (with `-i`).
+`--stdout`, `--publish` (and `--open`), `--no-validate`, `--review`. `-` reads
+pasted source from stdin (with `-i`).
+
+`--review` sends the import and its source to a model on the server (GPT-5.6
+Luna, the model that related steps best in the paper's evaluation) and prints
+what looks wrong: a default duration on "simmer until thick", a step the
+parser dropped, steps chained that the source runs in parallel, a total that
+disagrees with the source. It needs `rhylthyme login`, spends one model call
+(capped per day), and changes nothing: the findings are yours to apply. On
+rhylthyme.com the same review runs automatically after every import when you
+are signed in.
 
 ### `rhylthyme render`
 

{rhylthyme_cli_runner-0.2.5a0 → rhylthyme_cli_runner-0.2.7a0}/README.md

@@ -501,8 +501,17 @@ rhylthyme search "pad thai" -i spoonacular                           # then impo
 ```
 
 **Options:** `-i IMPORTER` (default: chosen from the URL), `-o PATH`,
-`--stdout`, `--publish` (and `--open`), `--no-validate`. `-` reads pasted
-source from stdin (with `-i`).
+`--stdout`, `--publish` (and `--open`), `--no-validate`, `--review`. `-` reads
+pasted source from stdin (with `-i`).
+
+`--review` sends the import and its source to a model on the server (GPT-5.6
+Luna, the model that related steps best in the paper's evaluation) and prints
+what looks wrong: a default duration on "simmer until thick", a step the
+parser dropped, steps chained that the source runs in parallel, a total that
+disagrees with the source. It needs `rhylthyme login`, spends one model call
+(capped per day), and changes nothing: the findings are yours to apply. On
+rhylthyme.com the same review runs automatically after every import when you
+are signed in.
 
 ### `rhylthyme render`
 

{rhylthyme_cli_runner-0.2.5a0 → rhylthyme_cli_runner-0.2.7a0}/setup.py

@@ -19,7 +19,7 @@ def read_readme():
 
 setup(
     name="rhylthyme-cli-runner",
-    version="0.2.5a0",
+    version="0.2.7a0",
     description="CLI runner for Rhylthyme real-time program schedules",
     long_description=read_readme(),
     long_description_content_type="text/markdown",

{rhylthyme_cli_runner-0.2.5a0 → rhylthyme_cli_runner-0.2.7a0}/src/rhylthyme_cli_runner/__init__.py

@@ -6,7 +6,7 @@ This package provides the command-line interface for running and validating
 Rhylthyme real-time program schedules.
 """
 
-__version__ = "0.2.5a0"
+__version__ = "0.2.7a0"
 __author__ = "Rhylthyme Team"
 __description__ = "CLI runner for Rhylthyme real-time program schedules"
 

{rhylthyme_cli_runner-0.2.5a0 → rhylthyme_cli_runner-0.2.7a0}/src/rhylthyme_cli_runner/import_cli.py

@@ -74,6 +74,10 @@ def _import_one(importer, source: str, text: Optional[str]):
         if hasattr(importer, "allow_local_files"):
             importer.allow_local_files = True
         return importer.import_from_url(str(path))
+    if not source.startswith(("http://", "https://")) and (
+        path.suffix or "/" in source
+    ):
+        raise click.ClickException(f"No such file: {source}")
     return importer.import_from_url(source)
 
 
@@ -130,8 +134,21 @@ def _validation(program: dict):
 @click.option(
     "--no-validate", is_flag=True, help="Skip validation of the imported program."
 )
+@click.option(
+    "--review",
+    "do_review",
+    is_flag=True,
+    help="Have a model read the import against its source and report what looks wrong (needs `rhylthyme login`).",
+)
 def import_command(
-    source, importer_name, output, to_stdout, do_publish, open_url, no_validate
+    source,
+    importer_name,
+    output,
+    to_stdout,
+    do_publish,
+    open_url,
+    no_validate,
+    do_review,
 ):
     """Import a recipe, protocol or deck as a program.
 
@@ -141,6 +158,7 @@ def import_command(
     .cook or .pptx file. `-` reads the source text from stdin (with -i).
       rhylthyme import https://www.seriouseats.com/the-best-chili-recipe
       rhylthyme import 52772 -i themealdb --publish
+      rhylthyme import recipe.cook -i cooklang --review   # a second opinion on durations, order, gaps
       rhylthyme import https://raw.githubusercontent.com/Opentrons/Protocols/develop/protocols/007992/rna_isolation.ot2.apiv2.py
       rhylthyme importers          # what is installed
     """
@@ -153,8 +171,22 @@ def import_command(
             )
         text = sys.stdin.read()
     importer = _pick(registry, source, importer_name)
+    token = None
+    if do_review:
+        # Before any work: the review needs an account, say so now.
+        from .remote import auth
+
+        try:
+            token = auth.access_token()
+        except auth.AuthError as e:
+            raise click.ClickException(
+                f"{e} (--review sends the import to a model on the server, which needs your account.)"
+            )
     click.echo(f"Importing with {importer.name}…", err=True)
     result = _import_one(importer, source, text)
+    source_text = (
+        text if text is not None else _source_text_for_review(importer, source)
+    )
     if not result.success:
         raise click.ClickException(result.error or "import failed")
     program = result.program
@@ -163,9 +195,10 @@ def import_command(
         v = _validation(program)
         errors = v.get("errors") or v.get("logic_errors") or []
         if not v.get("is_valid", v.get("valid", True)):
-            for e in errors[:8]:
+            for finding in errors[:8]:
                 click.echo(
-                    f"  ✗ {e if isinstance(e, str) else e.get('message', e)}", err=True
+                    f"  ✗ {finding if isinstance(finding, str) else finding.get('message', finding)}",
+                    err=True,
                 )
             raise click.ClickException(
                 "The imported program does not validate; not published. Re-run with --no-validate to keep it anyway."
@@ -187,6 +220,14 @@ def import_command(
         target.write_text(payload + "\n")
         click.echo(f"Saved {target}", err=True)
 
+    if do_review:
+        _review(
+            program,
+            token,
+            source_text=source_text,
+            source_url=source if source.startswith(("http://", "https://")) else None,
+        )
+
     if do_publish:
         from .remote.cli import publish as publish_command
 
@@ -208,6 +249,67 @@ def import_command(
             Path(tmp_path).unlink(missing_ok=True)
 
 
+def _source_text_for_review(importer, source: str) -> Optional[str]:
+    """The text the import came from, when the CLI has it: a local file, or
+    a URL the CLI fetched for a text importer. Recipe-site imports do not
+    keep the page, so the reviewer gets the URL instead."""
+    path = Path(source)
+    if path.is_file() and path.suffix.lower() in (".cook", ".py", ".txt", ".md"):
+        try:
+            return path.read_text()
+        except OSError:
+            return None
+    if (
+        source.startswith(("http://", "https://"))
+        and hasattr(importer, "import_from_source")
+        and not getattr(importer, "supported_domains", None)
+    ):
+        try:
+            from .remote.cli import _fetch_bytes
+
+            return _fetch_bytes(source).decode("utf-8")
+        except Exception:  # noqa: BLE001
+            return None
+    return None
+
+
+def _review(
+    program: dict, token: str, *, source_text: Optional[str], source_url: Optional[str]
+) -> None:
+    from .remote import mcp_client as MC
+
+    click.echo("Reviewing…", err=True)
+    client = MC.McpClient(url=MC.endpoint_for(None))
+    args = {"program": program, "token": token}
+    if source_text:
+        args["source_text"] = source_text
+    elif source_url:
+        args["source_url"] = source_url
+    try:
+        result = client.call_tool("review_program", args, timeout=150)
+    except MC.ToolError as e:
+        raise click.ClickException(f"Review failed: {e.text}")
+    except MC.McpError as e:
+        raise click.ClickException(f"Review failed: {e}")
+    review = result.structured or {}
+    findings = review.get("findings") or []
+    mark = {"error": "✗", "warning": "!", "note": "·"}
+    click.echo("")
+    click.echo(
+        review.get("summary")
+        or ("Review: no problems found." if not findings else "Review:")
+    )
+    for f in findings:
+        where = f" [{f['stepId']}]" if f.get("stepId") else ""
+        click.echo(f"  {mark.get(f.get('severity'), '·')}{where} {f.get('message')}")
+        if f.get("suggestion"):
+            click.echo(f"      → {f['suggestion']}")
+    if findings:
+        click.echo(
+            "\nThe file is unchanged; edit it and `rhylthyme validate` again.", err=True
+        )
+
+
 @click.command("search")
 @click.argument("query")
 @click.option(

{rhylthyme_cli_runner-0.2.5a0 → rhylthyme_cli_runner-0.2.7a0}/src/rhylthyme_cli_runner/remote/checks.py

@@ -252,7 +252,7 @@ def check_initialize(client, endpoint, opts, state) -> str:
     return f"{info['name']} {info.get('version', '?')}, protocol {result['protocolVersion']}"
 
 
-TOOL_DEFINITION_BUDGET_CHARS = 16000
+TOOL_DEFINITION_BUDGET_CHARS = 17000
 
 
 def check_tools(client, endpoint, opts, state) -> str:

{rhylthyme_cli_runner-0.2.5a0 → rhylthyme_cli_runner-0.2.7a0/src/rhylthyme_cli_runner.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: rhylthyme-cli-runner
-Version: 0.2.5a0
+Version: 0.2.7a0
 Summary: CLI runner for Rhylthyme real-time program schedules
 Home-page: https://github.com/rhylthyme/rhylthyme-cli-runner
 Author: Rhylthyme Team
@@ -561,8 +561,17 @@ rhylthyme search "pad thai" -i spoonacular                           # then impo
 ```
 
 **Options:** `-i IMPORTER` (default: chosen from the URL), `-o PATH`,
-`--stdout`, `--publish` (and `--open`), `--no-validate`. `-` reads pasted
-source from stdin (with `-i`).
+`--stdout`, `--publish` (and `--open`), `--no-validate`, `--review`. `-` reads
+pasted source from stdin (with `-i`).
+
+`--review` sends the import and its source to a model on the server (GPT-5.6
+Luna, the model that related steps best in the paper's evaluation) and prints
+what looks wrong: a default duration on "simmer until thick", a step the
+parser dropped, steps chained that the source runs in parallel, a total that
+disagrees with the source. It needs `rhylthyme login`, spends one model call
+(capped per day), and changes nothing: the findings are yours to apply. On
+rhylthyme.com the same review runs automatically after every import when you
+are signed in.
 
 ### `rhylthyme render`
 

{rhylthyme_cli_runner-0.2.5a0 → rhylthyme_cli_runner-0.2.7a0}/tests/test_import_cli.py

@@ -240,3 +240,74 @@ def test_a_text_importer_gets_a_urls_body(fake, monkeypatch):
     )
     assert r.exit_code == 0, r.output
     assert fake.calls[-1] == ("text", "def run(protocol): pass\n", None)
+
+
+def test_a_missing_local_file_says_so(fake, tmp_path):
+    r = CliRunner().invoke(
+        cli, ["import", str(tmp_path / "Neapolitan Pizza.cook"), "-i", "fake"]
+    )
+    assert r.exit_code != 0 and "No such file" in r.output
+    assert fake.calls == []
+
+
+def test_review_needs_a_login_before_any_work(fake, monkeypatch):
+    monkeypatch.delenv("RHYLTHYME_TOKEN", raising=False)
+    from rhylthyme_cli_runner.remote import auth
+
+    monkeypatch.setattr(auth, "load_credentials", lambda path=None: None)
+    r = CliRunner().invoke(
+        cli, ["import", "https://fake.example/dinner", "--review", "--stdout"]
+    )
+    assert r.exit_code != 0 and "rhylthyme login" in r.output
+    assert fake.calls == [], "nothing was imported"
+
+
+def test_review_sends_the_program_and_source_and_prints_findings(
+    fake, monkeypatch, tmp_path
+):
+    monkeypatch.setenv("RHYLTHYME_TOKEN", "tok-1")
+    from rhylthyme_cli_runner.remote import mcp_client as MC
+
+    seen = {}
+
+    class FakeMcp:
+        def __init__(self, url=None, **kw):
+            seen["url"] = url
+
+        def call_tool(self, name, args, timeout=None, **kw):
+            seen["tool"], seen["args"] = name, args
+            return MC.ToolResult(
+                content=[{"type": "text", "text": "x"}],
+                structured={
+                    "summary": "One problem.",
+                    "usable": False,
+                    "findings": [
+                        {
+                            "severity": "error",
+                            "stepId": "s1",
+                            "message": "Too short.",
+                            "suggestion": "90m",
+                        },
+                        {"severity": "note", "message": "Fine."},
+                    ],
+                },
+            )
+
+    monkeypatch.setattr(MC, "McpClient", FakeMcp)
+    src = tmp_path / "dinner.cook"
+    src.write_text("Simmer @sauce for ~{90%minutes}.\n")
+    fake.supported_domains = []
+    r = CliRunner().invoke(
+        cli,
+        ["import", str(src), "-i", "fake", "--review", "-o", str(tmp_path / "d.json")],
+    )
+    assert r.exit_code == 0, r.output
+    assert seen["tool"] == "review_program"
+    assert seen["args"]["token"] == "tok-1"
+    assert seen["args"]["program"]["programId"] == "fake-dinner"
+    assert seen["args"]["source_text"].startswith("Simmer @sauce")
+    assert (
+        "One problem." in r.output
+        and "✗ [s1] Too short." in r.output
+        and "→ 90m" in r.output
+    )