dirigent-cli 0.16.3__tar.gz → 0.16.5__tar.gz

{dirigent_cli-0.16.3 → dirigent_cli-0.16.5}/PKG-INFO

@@ -1,16 +1,18 @@
 Metadata-Version: 2.4
 Name: dirigent-cli
-Version: 0.16.3
+Version: 0.16.5
 Summary: The dirigent command line interface (dirigent / dg).
 License-Expression: LicenseRef-Proprietary
 License-File: LICENSE
-Requires-Dist: dirigent-blocks==0.16.3
-Requires-Dist: dirigent-client==0.16.3
-Requires-Dist: dirigent-common==0.16.3
-Requires-Dist: dirigent-core==0.16.3
-Requires-Dist: dirigent-examples==0.16.3
-Requires-Dist: dirigent-plugin==0.16.3
-Requires-Dist: dirigent-server==0.16.3
+Classifier: Programming Language :: Python :: 3
+Classifier: Programming Language :: Python :: 3.13
+Requires-Dist: dirigent-blocks==0.16.5
+Requires-Dist: dirigent-client==0.16.5
+Requires-Dist: dirigent-common==0.16.5
+Requires-Dist: dirigent-core==0.16.5
+Requires-Dist: dirigent-examples==0.16.5
+Requires-Dist: dirigent-plugin==0.16.5
+Requires-Dist: dirigent-server==0.16.5
 Requires-Dist: httpx2>=2.12.0
 Requires-Dist: python-dotenv>=1.1.0
 Requires-Dist: pyyaml>=6.0.3

{dirigent_cli-0.16.3 → dirigent_cli-0.16.5}/pyproject.toml

@@ -1,19 +1,23 @@
 [project]
 name = "dirigent-cli"
-version = "0.16.3"
+version = "0.16.5"
 description = "The dirigent command line interface (dirigent / dg)."
 readme = "README.md"
 requires-python = ">=3.13"
 license = "LicenseRef-Proprietary"
 license-files = ["LICENSE"]
+classifiers = [
+    "Programming Language :: Python :: 3",
+    "Programming Language :: Python :: 3.13",
+]
 dependencies = [
-    "dirigent-blocks==0.16.3",
-    "dirigent-client==0.16.3",
-    "dirigent-common==0.16.3",
-    "dirigent-core==0.16.3",
-    "dirigent-examples==0.16.3",
-    "dirigent-plugin==0.16.3",
-    "dirigent-server==0.16.3",
+    "dirigent-blocks==0.16.5",
+    "dirigent-client==0.16.5",
+    "dirigent-common==0.16.5",
+    "dirigent-core==0.16.5",
+    "dirigent-examples==0.16.5",
+    "dirigent-plugin==0.16.5",
+    "dirigent-server==0.16.5",
     "httpx2>=2.12.0",
     "python-dotenv>=1.1.0",
     "pyyaml>=6.0.3",

{dirigent_cli-0.16.3 → dirigent_cli-0.16.5}/pyproject.toml.orig

@@ -1,19 +1,23 @@
 [project]
 name = "dirigent-cli"
-version = "0.16.3"
+version = "0.16.5"
 description = "The dirigent command line interface (dirigent / dg)."
 readme = "README.md"
 requires-python = ">=3.13"
 license = "LicenseRef-Proprietary"
 license-files = ["LICENSE"]
+classifiers = [
+    "Programming Language :: Python :: 3",
+    "Programming Language :: Python :: 3.13",
+]
 dependencies = [
-    "dirigent-blocks==0.16.3",
-    "dirigent-client==0.16.3",
-    "dirigent-common==0.16.3",
-    "dirigent-core==0.16.3",
-    "dirigent-examples==0.16.3",
-    "dirigent-plugin==0.16.3",
-    "dirigent-server==0.16.3",
+    "dirigent-blocks==0.16.5",
+    "dirigent-client==0.16.5",
+    "dirigent-common==0.16.5",
+    "dirigent-core==0.16.5",
+    "dirigent-examples==0.16.5",
+    "dirigent-plugin==0.16.5",
+    "dirigent-server==0.16.5",
     "httpx2>=2.12.0",
     "python-dotenv>=1.1.0",
     "pyyaml>=6.0.3",

{dirigent_cli-0.16.3 → dirigent_cli-0.16.5}/src/dirigent_cli/commands.py

@@ -170,6 +170,11 @@ def fail(message: str) -> NoReturn:
     raise typer.Exit(code=1)
 
 
+def at_a_terminal() -> bool:
+    """Report whether somebody is there to answer a prompt."""
+    return sys.stdin.isatty()
+
+
 def ask(label: str, *, hide: bool = False) -> str:
     """Read one value from the terminal, refusing under --json where nothing can answer.
 
@@ -885,11 +890,21 @@ def parse_params(
     schema: dict[str, Any] | None = None,
     files: list[Path] | None = None,
 ) -> dict[str, Any]:
-    """Build a run's parameters from files and flags, coerced against the pipeline's schema."""
+    """Build a run's parameters from files and flags, coerced against the pipeline's schema.
+
+    An empty value for a secret field means unset: ``--set api_token=`` leaves the field out
+    altogether, so nothing stores a credential that every read afterwards reports as set.
+    """
     try:
-        return build_params(schema or {}, pairs=pairs or [], files=files or [])
+        built = build_params(schema or {}, pairs=pairs or [], files=files or [])
     except ParamError as error:
         fail(str(error))
+    properties = cast("dict[str, Any]", (schema or {}).get("properties") or {})
+    return {
+        name: value
+        for name, value in built.items()
+        if value != "" or not _is_secret_field(cast("dict[str, Any]", properties.get(name) or {}))
+    }
 
 
 #: What separates the two ends of a ``--window`` value.
@@ -2224,12 +2239,19 @@ def connection_create(
     """Create a connection, prompting for the secret fields without echoing them.
 
     Prompting happens only on a terminal; in a script every value must arrive via ``--set``.
+    An invocation carrying ``--set`` is prompted only for what it cannot do without: a
+    required field it left out. A bare invocation is the interactive form and is offered
+    every secret the kind declares.
     """
     with client_for(state_of(ctx)) as dg:
         schema = _connection_schema(dg.call(dg.blocks.catalog()), kind_id)
         config = parse_params(set_value, schema=schema)
         required = set(cast("list[str]", schema.get("required") or []))
-        interactive = sys.stdin.isatty() and not json_mode()
+        interactive = at_a_terminal() and not json_mode()
+        # An invocation that carried --set said what it wanted, so only a required secret is
+        # still asked for there: an optional one it left out is left out. With no --set at
+        # all the command is the form, and every secret is offered.
+        offered = not set_value
         for field, body in cast("dict[str, Any]", schema.get("properties") or {}).items():
             if field in config:
                 continue
@@ -2241,6 +2263,8 @@ def connection_create(
             if not interactive:
                 continue
             if _is_secret_field(body):
+                if not offered and field not in required:
+                    continue
                 value = typer.prompt(f"{field}", hide_input=True, default="", show_default=False)
                 if value:
                     config[field] = value

{dirigent_cli-0.16.3 → dirigent_cli-0.16.5}/src/dirigent_cli/main.py

@@ -3,7 +3,7 @@
 import os
 import shutil
 import sys
-from collections.abc import Sequence
+from collections.abc import Callable, Sequence
 from datetime import UTC, datetime, timedelta
 from importlib.metadata import PackageNotFoundError, version
 from pathlib import Path
@@ -1076,8 +1076,15 @@ async def _dev(
     install_signal_handlers(worker)
     server_config = uvicorn.Config(build_app(), host=host, port=port, log_config=None)
     api = uvicorn.Server(server_config)
+
+    def shut_down() -> None:
+        """Ask for the shutdown SIGTERM asks for: the worker drains, and serve returns."""
+        worker.request_stop()
+        api.should_exit = True
+
     worker_task = asyncio.create_task(worker.run())
     ready = asyncio.create_task(_announce_ready(api))
+    orphaned = asyncio.create_task(watch_parent(os.getppid, shut_down))
     seeding = (
         asyncio.create_task(_seed(api, local_url(host, port), bearer, seed, installed)) if seed or installed else None
     )
@@ -1085,6 +1092,7 @@ async def _dev(
         await api.serve()
     finally:
         ready.cancel()
+        orphaned.cancel()
         if seeding is not None:
             seeding.cancel()
         worker.request_stop()
@@ -1107,6 +1115,30 @@ async def _announce_ready(api: "uvicorn.Server") -> None:
     emit(make("process", at=datetime.now(UTC), message="ready", process="dev"))
 
 
+#: How often the instance looks at whether the process that started it is still there.
+PARENT_POLL_SECONDS = 1.0
+
+
+async def watch_parent(
+    parent: Callable[[], int], stop: Callable[[], None], *, every: float = PARENT_POLL_SECONDS
+) -> None:
+    """Shut the instance down once the process that started it is gone.
+
+    ``uv run dg dev`` forwards SIGTERM to the instance, but a wrapper that was killed outright
+    cannot, and the instance it leaves behind goes on claiming runs against the same state.
+    macOS has no PR_SET_PDEATHSIG, so the parent pid is polled: a pid that changed means this
+    process was reparented to init. A dev instance therefore does not outlive the process that
+    started it, backgrounding it past the shell that launched it included.
+    """
+    import asyncio
+
+    started = parent()
+    while parent() == started:
+        await asyncio.sleep(every)
+    emit(make("process", at=datetime.now(UTC), message="parent gone", process="dev", parent=started))
+    stop()
+
+
 async def _seed(
     api: "uvicorn.Server",
     url: str,

{dirigent_cli-0.16.3 → dirigent_cli-0.16.5}/src/dirigent_cli/output.py

@@ -16,6 +16,7 @@ from pydantic import BaseModel
 from rich.console import Console, Group, RenderableType
 from rich.markup import escape
 from rich.table import Table
+from rich.text import Text
 
 from dirigent_client.schemas.common import Problem
 from dirigent_core.protocol import Format, Record, as_json, make
@@ -500,16 +501,26 @@ def elapsed(value: object) -> str:
     return f"{int(hours)}h{int(minutes)}m"
 
 
+def titled(title: str, built: Table) -> RenderableType:
+    """Put a title above a table, on a line of its own.
+
+    Rich folds a table's own title to the width of the table, which the widest row sets, so a
+    title carrying a 36-character run id breaks in half wherever the rows are narrower than
+    it. Above the table the title is folded only by the console.
+    """
+    return Group(Text.from_markup(title, style="table.title"), built)
+
+
 def build_table(title: str, columns: Sequence[str], rows: Iterable[Sequence[str]]) -> RenderableType:
     """Build one rich table, or the quiet line that stands in for an empty one."""
-    built = Table(title=title, show_header=True, header_style="bold", title_justify="left")
+    built = Table(show_header=True, header_style="bold")
     for column in columns:
         built.add_column(column)
     count = 0
     for row in rows:
         built.add_row(*row)
         count += 1
-    return built if count else f"[dim]{title}: nothing to show.[/]"
+    return titled(title, built) if count else f"[dim]{title}: nothing to show.[/]"
 
 
 def table(title: str, columns: Sequence[str], rows: Iterable[Sequence[str]]) -> None:
@@ -518,13 +529,13 @@ def table(title: str, columns: Sequence[str], rows: Iterable[Sequence[str]]) ->
 
 
 def build_fields(title: str, values: Mapping[str, Any]) -> RenderableType:
-    """Build one record as a two-column table."""
-    built = Table(title=title, show_header=False, box=None, title_justify="left", padding=(0, 2, 0, 0))
+    """Build one record as a two-column table under its title."""
+    built = Table(show_header=False, box=None, padding=(0, 2, 0, 0))
     built.add_column(style="bold")
     built.add_column()
     for name, value in values.items():
         built.add_row(name, styled(value))
-    return built
+    return titled(title, built)
 
 
 def fields(title: str, values: Mapping[str, Any]) -> None: