delegate-agent-cli 0.26.0__tar.gz → 0.27.0__tar.gz

{delegate_agent_cli-0.26.0 → delegate_agent_cli-0.27.0}/CHANGELOG.md

@@ -7,6 +7,32 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.27.0] - 2026-08-01
+
+### Changed
+
+- Split the mail module into `mail_core` (mailbox) and `mail_push` (hook
+  protocol) behind a stable `mail` facade; extracted the runner's mail
+  sentinel capture and push finalization; unified the three launch seams'
+  mail preparation in one helper with ownership transfer kept at the seams;
+  `parse_prompt_tail` returns a named, tuple-compatible `PromptTail`.
+- Mail-disabled launches no longer touch the filesystem for mail (lanes keep
+  their identity and storage materializes lazily on first send); workspace
+  resolution runs once per launch; the discovery cache is memoized per
+  process; the directory retry baseline is captured only when a codex
+  fallback profile is configured; watch sleeps are bounded by the remaining
+  deadline; non-once watches dedupe emitted messages and skip unchanged
+  inboxes; mail bodies stream with a bounded read.
+- All mail subcommands render prose in human mode (watch keeps its
+  documented NDJSON contract); JSON output is byte-unchanged.
+
+### Removed
+
+- Dead code from the framer migration and mail rounds: unused safe-prefix
+  helpers, the write-only `prompt_framed` field, and unreferenced mail
+  helpers; development-history comments replaced with durable behavior
+  statements.
+
 ## [0.26.0] - 2026-08-01
 
 ### Added
@@ -862,6 +888,7 @@ Usage-audit fix wave: 82 sessions and 1,241 delegate invocations from one week o
 
 - Releases before 0.1.3 predate this changelog.
 
+[0.27.0]: https://github.com/treygoff24/delegate-agent/compare/v0.26.0...v0.27.0
 [0.26.0]: https://github.com/treygoff24/delegate-agent/compare/v0.25.0...v0.26.0
 [0.25.0]: https://github.com/treygoff24/delegate-agent/compare/v0.24.0...v0.25.0
 [0.24.0]: https://github.com/treygoff24/delegate-agent/compare/v0.23.0...v0.24.0

{delegate_agent_cli-0.26.0 → delegate_agent_cli-0.27.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: delegate-agent-cli
-Version: 0.26.0
+Version: 0.27.0
 Summary: A tiny CLI for delegating bounded agent tasks across coding-agent runtimes.
 Author: Trey Goff
 License-Expression: MIT

{delegate_agent_cli-0.26.0 → delegate_agent_cli-0.27.0}/docs/cli-reference.md

@@ -1013,7 +1013,7 @@ delegate runs prune [--older-than DAYS] [--dry-run]
 delegate ps [--harness HARNESS] [--group NAME] [--limit N]
 delegate snapshot [--latest HARNESS] [--no-redact] <handle>
 delegate run-output [--latest HARNESS] <handle> [--completion-report] [--stdout] [--stderr] [--tail N] [--max-chars N] [--raw] [--no-redact]
-delegate resume [--engine ENGINE] [--model MODEL] [--reasoning-effort LEVEL] [--timeout SEC] [--dry-run] <handle> [extra instructions...]
+delegate resume [--engine ENGINE] [--model MODEL] [--reasoning-effort LEVEL] [--fast|--no-fast] [--progress|--no-progress] [--timeout SEC] [--output-schema PATH|--no-output-schema] [--include-dirty] [--persona NAME|--no-persona] [--allow-repo-persona] [--mail-push] [--dry-run] <handle> [extra instructions...]
 delegate wait <handle>... [--latest HARNESS] [--group NAME] [--timeout SEC] [--interval SEC] [--completion-report]
 delegate cancel <handle>...
 ```

{delegate_agent_cli-0.26.0 → delegate_agent_cli-0.27.0}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "delegate-agent-cli"
-version = "0.26.0"
+version = "0.27.0"
 description = "A tiny CLI for delegating bounded agent tasks across coding-agent runtimes."
 readme = "README.md"
 requires-python = ">=3.11"

{delegate_agent_cli-0.26.0 → delegate_agent_cli-0.27.0}/src/delegate_agent/__init__.py

@@ -1,5 +1,5 @@
 """Delegate Agent CLI package."""
 
-VERSION = "0.26.0"
+VERSION = "0.27.0"
 
 __all__ = ["VERSION"]

{delegate_agent_cli-0.26.0 → delegate_agent_cli-0.27.0}/src/delegate_agent/argv_builders.py

@@ -15,7 +15,6 @@ from delegate_agent import reasoning
 from delegate_agent.constants import MODE_CALL, MODE_SAFE, MODE_WORK, validate_mode
 from delegate_agent.errors import DelegateError
 from delegate_agent.json_types import JsonObject
-from delegate_agent.prompt_instructions import SKILL_REVIEW_PREFIX
 from delegate_agent.prompt_transport import (
     CURSOR_PROMPT_REDACTION,
     DEVIN_AGENT_CONFIG_ARG_PLACEHOLDER,
@@ -108,23 +107,6 @@ def prefix_cursor_safe_prompt(prompt: str) -> str:
     return _prefix_safe_prompt(prompt, "cursor")
 
 
-def prefix_kimi_safe_prompt(prompt: str) -> str:
-    return _prefix_safe_prompt(prompt, "kimi")
-
-
-def prefix_droid_safe_prompt(prompt: str) -> str:
-    safe_prefix = SAFE_REVIEW_PREFIX_BY_ENGINE["droid"]
-    if prompt.startswith(safe_prefix):
-        return prompt
-    skill_prefix = SKILL_REVIEW_PREFIX
-    if prompt.startswith(skill_prefix):
-        insert_at = len(skill_prefix)
-        if prompt[insert_at:].startswith(safe_prefix):
-            return prompt
-        return prompt[:insert_at] + safe_prefix + prompt[insert_at:]
-    return f"{safe_prefix}{prompt}"
-
-
 def _harness_bypass_enabled(config: JsonObject, mode: str, engine: str) -> bool:
     """Return true only for explicit harness-scoped bypass policy.
 

{delegate_agent_cli-0.26.0 → delegate_agent_cli-0.27.0}/src/delegate_agent/cli.py

@@ -783,7 +783,7 @@ def _execute_attached_worktree(
         workspace_kind=source_workspace.kind,
     )
     maybe_run_retention_pass(registry_root, config)
-    if request.mode == MODE_WORK:
+    if request.mode == MODE_WORK and delegate_config.mail_enabled(config):
         mail.prepare_mail_storage(registry_root)
     mail.sanitize_inherited_mail_identity(request.env_overrides)
     run_id, alias = run_registry.register_run(
@@ -807,39 +807,27 @@ def _execute_attached_worktree(
     if request.mode == MODE_WORK:
         mail.bind_mail_identity(child_env, run_id, alias)
     request.env_overrides = child_env
-    provision: mail.MailPushProvision | None = None
-    if request.mode == MODE_WORK:
-        wired_argv, wired_display_argv = mail.wire_work_mail_launch(
-            request.engine,
-            execution_request.argv,
-            execution_request.display_argv,
-            registry_root,
-            prompt=request.prompt,
-            prompt_transport=request.prompt_transport,
-            stderr=stderr,
-            isolated_workspace=True,
-        )
-        execution_request = dc_replace(
-            execution_request, argv=wired_argv, display_argv=wired_display_argv
-        )
-        if request.mail_push:
-            provision = mail.provision_mail_push(
-                request.engine,
-                execution_request.argv,
-                execution_request.display_argv,
-                registry_root,
-                run_id,
-                request.env_overrides or {},
-                profiles.codex_fallback_child_env_overrides(
-                    request.profile_resolution, request.env_overrides or {}
-                ),
-            )
-            if provision.warning is not None:
-                request.warnings = (*request.warnings, provision.warning)
-                print(f"delegate mail: WARNING: {provision.warning}", file=stderr)
-            execution_request = dc_replace(
-                execution_request, argv=provision.argv, display_argv=provision.display_argv
-            )
+    mail_launch = mail.prepare_work_mail_launch(
+        enabled=request.mode == MODE_WORK and delegate_config.mail_enabled(config),
+        mail_push=request.mail_push,
+        engine=request.engine,
+        argv=execution_request.argv,
+        display_argv=execution_request.display_argv,
+        registry_root=registry_root,
+        run_id=run_id,
+        env_overrides=request.env_overrides,
+        profile_resolution=request.profile_resolution,
+        prompt=request.prompt,
+        prompt_transport=request.prompt_transport,
+        stderr=stderr,
+        isolated_workspace=True,
+        warnings=request.warnings,
+    )
+    execution_request = dc_replace(
+        execution_request, argv=mail_launch.argv, display_argv=mail_launch.display_argv
+    )
+    request.warnings = mail_launch.request_warnings
+    provision = mail_launch.provision
     ctx_runner = make_run_context(
         registry_root,
         request,
@@ -850,22 +838,10 @@ def _execute_attached_worktree(
     if provision is not None:
         ctx_runner = dc_replace(
             ctx_runner,
-            env_overrides=(
-                dict(provision.env)
-                if provision.warning is not None and provision.env is not None
-                else ctx_runner.env_overrides
-            ),
-            fallback_env_overrides=mail.mail_push_fallback_env_overrides(
-                provision, ctx_runner.fallback_env_overrides, registry_root, run_id
-            ),
-            warnings=tuple(
-                dict.fromkeys(
-                    (*ctx_runner.warnings, *((provision.warning,) if provision.warning else ()))
-                )
+            **mail_launch.context_updates(
+                ctx_runner.env_overrides, ctx_runner.fallback_env_overrides, ctx_runner.warnings
             ),
         )
-        if provision.warning is not None and provision.env is not None:
-            ctx_runner = dc_replace(ctx_runner, env_overrides=dict(provision.env))
     attachment = {
         **(iso.attachment or {}),
         "startHeadOid": start_head_oid,
@@ -1303,7 +1279,7 @@ def execute_request(
             workspace_kind=source_workspace.kind,
         )
         maybe_run_retention_pass(registry_root, config)
-        if isolated_request.mode == MODE_WORK:
+        if isolated_request.mode == MODE_WORK and delegate_config.mail_enabled(config):
             mail.prepare_mail_storage(registry_root)
         mail.sanitize_inherited_mail_identity(isolated_request.env_overrides)
         run_id, alias = run_registry.register_run(
@@ -1330,41 +1306,32 @@ def execute_request(
         provision: mail.MailPushProvision | None = None
         mail_push_cleanup_transferred = False
         try:
-            if isolated_request.mode == MODE_WORK:
-                isolated_request.argv, isolated_request.display_argv = mail.wire_work_mail_launch(
-                    isolated_request.engine,
-                    isolated_request.argv,
-                    isolated_request.display_argv,
-                    registry_root,
-                    prompt=isolated_request.prompt,
-                    prompt_transport=isolated_request.prompt_transport,
-                    stderr=stderr,
-                    isolated_workspace=bool(
-                        isolated_request.isolation_context
-                        and isolated_request.isolation_context.isolation_lifecycle
-                        in ("temporary", "persistent", "attached")
-                    ),
-                )
-                if isolated_request.mail_push:
-                    provision = mail.provision_mail_push(
-                        isolated_request.engine,
-                        isolated_request.argv,
-                        isolated_request.display_argv,
-                        registry_root,
-                        run_id,
-                        isolated_request.env_overrides or {},
-                        profiles.codex_fallback_child_env_overrides(
-                            isolated_request.profile_resolution,
-                            isolated_request.env_overrides or {},
-                        ),
-                    )
-                    if provision.warning is not None:
-                        isolated_request.warnings = (*isolated_request.warnings, provision.warning)
-                        print(f"delegate mail: WARNING: {provision.warning}", file=stderr)
-                    isolated_request.argv, isolated_request.display_argv = (
-                        provision.argv,
-                        provision.display_argv,
-                    )
+            mail_launch = mail.prepare_work_mail_launch(
+                enabled=isolated_request.mode == MODE_WORK and delegate_config.mail_enabled(config),
+                mail_push=isolated_request.mail_push,
+                engine=isolated_request.engine,
+                argv=isolated_request.argv,
+                display_argv=isolated_request.display_argv,
+                registry_root=registry_root,
+                run_id=run_id,
+                env_overrides=isolated_request.env_overrides,
+                profile_resolution=isolated_request.profile_resolution,
+                prompt=isolated_request.prompt,
+                prompt_transport=isolated_request.prompt_transport,
+                stderr=stderr,
+                isolated_workspace=bool(
+                    isolated_request.isolation_context
+                    and isolated_request.isolation_context.isolation_lifecycle
+                    in ("temporary", "persistent", "attached")
+                ),
+                warnings=isolated_request.warnings,
+            )
+            isolated_request.argv, isolated_request.display_argv = (
+                mail_launch.argv,
+                mail_launch.display_argv,
+            )
+            isolated_request.warnings = mail_launch.request_warnings
+            provision = mail_launch.provision
             ctx_runner = make_run_context(
                 registry_root,
                 isolated_request,
@@ -1375,25 +1342,12 @@ def execute_request(
             if provision is not None:
                 ctx_runner = dc_replace(
                     ctx_runner,
-                    env_overrides=(
-                        dict(provision.env)
-                        if provision.warning is not None and provision.env is not None
-                        else ctx_runner.env_overrides
-                    ),
-                    fallback_env_overrides=mail.mail_push_fallback_env_overrides(
-                        provision, ctx_runner.fallback_env_overrides, registry_root, run_id
-                    ),
-                    warnings=tuple(
-                        dict.fromkeys(
-                            (
-                                *ctx_runner.warnings,
-                                *((provision.warning,) if provision.warning else ()),
-                            )
-                        )
+                    **mail_launch.context_updates(
+                        ctx_runner.env_overrides,
+                        ctx_runner.fallback_env_overrides,
+                        ctx_runner.warnings,
                     ),
                 )
-                if provision.warning is not None and provision.env is not None:
-                    ctx_runner = dc_replace(ctx_runner, env_overrides=dict(provision.env))
             # Evaluate every argument expression BEFORE marking ownership
             # transferred: a raise inside an argument (public_argv below) would
             # otherwise skip both the CLI cleanup and the runner's guard.
@@ -1543,6 +1497,7 @@ def main(
     try:
         parsed = parse_cli(argv)
         global_options = parsed.global_options
+        workspace: ResolvedWorkspace | None = None
         json_mode = global_options.json_mode
         profile_guard.enforce_profile_guard(parsed, stderr=stderr)
         if parsed.subcommand == "help":
@@ -1581,16 +1536,18 @@ def main(
                         "call mode does not use --cwd.",
                     )
                 if global_options.group is not None:
-                    config_workspace = workspace_path_for_config(global_options.cwd)
+                    workspace = resolve_workspace(global_options.cwd)
+                    config_workspace = Path(workspace.path)
                 else:
                     config_workspace = None
             else:
-                config_workspace = workspace_path_for_config(global_options.cwd)
+                workspace = resolve_workspace(global_options.cwd)
+                config_workspace = Path(workspace.path)
             config, source = load_config(workspace=config_workspace)
             validate_config(config)
 
         if parsed.subcommand == "personas":
-            workspace = resolve_workspace(global_options.cwd)
+            workspace = workspace or resolve_workspace(global_options.cwd)
             return emit_personas(workspace, json_mode=global_options.json_mode, stdout=stdout)
 
         discovery_profile: profiles.ProfileResolution | None = None
@@ -1641,7 +1598,7 @@ def main(
                 mail_workspace = mail.resolve_mail_workspace(global_options.cwd)
                 workspace = resolve_workspace(str(mail_workspace))
             else:
-                workspace = resolve_workspace(global_options.cwd)
+                workspace = workspace or resolve_workspace(global_options.cwd)
 
         if parsed.subcommand == "capabilities":
             command = parsed.capabilities
@@ -1714,7 +1671,7 @@ def main(
             # through the normal request build and execution path.
             parsed = resume_plan.parsed
             global_options = parsed.global_options
-        request = request_from_parsed(parsed, config, stdin, stderr)
+        request = request_from_parsed(parsed, config, stdin, stderr, workspace=workspace)
         if resume_plan is not None:
             request = resume_command.apply_resume_to_request(request, resume_plan)
         if request.dry_run:

{delegate_agent_cli-0.26.0 → delegate_agent_cli-0.27.0}/src/delegate_agent/cli_parser.py

@@ -43,6 +43,7 @@ from delegate_agent.request_models import (
     InspectionOptions,
     LaunchOptions,
     ParsedCommand,
+    PromptTail,
     ResumeOptions,
     RunJsonOptions,
 )
@@ -985,27 +986,26 @@ def parse_modeless_engine(
     # prompt text (`cursor work explain --help`).
     if len(rest) >= 2 and command_help.is_help_token(rest[1]):
         return help_command(json_mode, f"{topic} call" if mode == "call" else topic)
-    (
-        prompt_file,
-        output_schema,
-        reasoning_effort,
-        fast,
-        progress_intent,
-        forbid_commit,
-        prompt_parts,
-        json_mode,
-        isolation,
-        read_only,
-        pure,
-        timeout,
-        include_dirty,
-        mail_push,
-        model,
-        agent,
-        persona,
-        no_persona,
-        allow_repo_persona,
-    ) = parse_prompt_tail(rest[1:], json_mode, isolation, command_prefix=[engine, mode])
+    tail = parse_prompt_tail(rest[1:], json_mode, isolation, command_prefix=[engine, mode])
+    prompt_file = tail.prompt_file
+    output_schema = tail.output_schema
+    reasoning_effort = tail.reasoning_effort
+    fast = tail.fast
+    progress_intent = tail.progress_intent
+    forbid_commit = tail.forbid_commit
+    prompt_parts = tail.prompt_parts
+    json_mode = tail.json_mode
+    isolation = tail.isolation
+    read_only = tail.read_only
+    pure = tail.pure
+    timeout = tail.timeout
+    include_dirty = tail.include_dirty
+    mail_push = tail.mail_push
+    model = tail.model
+    agent = tail.agent
+    persona = tail.persona
+    no_persona = tail.no_persona
+    allow_repo_persona = tail.allow_repo_persona
     if agent is not None and engine != "opencode":
         raise DelegateError("unsupported_agent", "--agent is only supported by opencode.")
     if fast is not None and engine != "codex":
@@ -1120,27 +1120,26 @@ def parse_droid(
     # Help wins after the mode, before prompt capture: `droid [alias] safe --help`.
     if tail and command_help.is_help_token(tail[0]):
         return help_command(json_mode, f"{topic} call" if mode == "call" else topic)
-    (
-        prompt_file,
-        output_schema,
-        reasoning_effort,
-        fast,
-        progress_intent,
-        forbid_commit,
-        prompt_parts,
-        json_mode,
-        isolation,
-        read_only,
-        pure,
-        timeout,
-        include_dirty,
-        mail_push,
-        model,
-        agent,
-        persona,
-        no_persona,
-        allow_repo_persona,
-    ) = parse_prompt_tail(tail, json_mode, isolation, command_prefix=command_prefix)
+    tail_result = parse_prompt_tail(tail, json_mode, isolation, command_prefix=command_prefix)
+    prompt_file = tail_result.prompt_file
+    output_schema = tail_result.output_schema
+    reasoning_effort = tail_result.reasoning_effort
+    fast = tail_result.fast
+    progress_intent = tail_result.progress_intent
+    forbid_commit = tail_result.forbid_commit
+    prompt_parts = tail_result.prompt_parts
+    json_mode = tail_result.json_mode
+    isolation = tail_result.isolation
+    read_only = tail_result.read_only
+    pure = tail_result.pure
+    timeout = tail_result.timeout
+    include_dirty = tail_result.include_dirty
+    mail_push = tail_result.mail_push
+    model = tail_result.model
+    agent = tail_result.agent
+    persona = tail_result.persona
+    no_persona = tail_result.no_persona
+    allow_repo_persona = tail_result.allow_repo_persona
     if agent is not None:
         raise DelegateError("unsupported_agent", "--agent is only supported by opencode.")
     if fast is not None:
@@ -1467,27 +1466,7 @@ def parse_prompt_tail(
     isolation: str | None,
     *,
     command_prefix: list[str] | None = None,
-) -> tuple[
-    str | None,
-    str | None,
-    str | None,
-    bool | None,
-    str | None,
-    bool,
-    list[str],
-    bool,
-    str | None,
-    bool,
-    bool,
-    int | None,
-    bool,
-    str | None,
-    str | None,
-    str | None,
-    bool,
-    bool,
-    bool,
-]:
+) -> PromptTail:
     prompt_file: str | None = None
     output_schema: str | None = None
     reasoning_effort: str | None = None
@@ -1793,7 +1772,7 @@ def parse_prompt_tail(
         raise_misplaced_global_option(
             "Global options must appear before the subcommand; use --prompt-file for literal flag text.",
         )
-    return (
+    return PromptTail(
         prompt_file,
         output_schema,
         reasoning_effort,

{delegate_agent_cli-0.26.0 → delegate_agent_cli-0.27.0}/src/delegate_agent/harness_discovery.py

@@ -7,6 +7,7 @@ are deliberately layered on later.
 
 from __future__ import annotations
 
+import copy
 import hashlib
 import json
 import os
@@ -43,6 +44,7 @@ CONFIGURED_SELECTOR_MISSING = "configured_selector_missing"
 FUTURE_SCHEMA_CACHE_WARNING = (
     "discovery cache was written by a newer delegate; probing without reading or replacing it"
 )
+_DISCOVERY_CACHE_MEMO: dict[tuple[Path, int, int], JsonObject | None] = {}
 
 
 class FutureCacheSchemaError(ValueError):
@@ -1445,6 +1447,25 @@ def discovery_cache_path(profile_name: str | None, *, home: Path | None = None)
     return root / f"profile-{digest}.json"
 
 
+def _clear_discovery_cache_memo(path: Path) -> None:
+    for key in tuple(_DISCOVERY_CACHE_MEMO):
+        if key[0] == path:
+            del _DISCOVERY_CACHE_MEMO[key]
+
+
+def _discovery_cache_memo_key(path: Path) -> tuple[Path, int, int] | None:
+    try:
+        stat_result = path.stat()
+    except OSError:
+        _clear_discovery_cache_memo(path)
+        return None
+    key = (path, stat_result.st_mtime_ns, stat_result.st_size)
+    for cached_key in tuple(_DISCOVERY_CACHE_MEMO):
+        if cached_key[0] == path and cached_key != key:
+            del _DISCOVERY_CACHE_MEMO[cached_key]
+    return key
+
+
 def cache_schema_is_future(profile_name: str | None, *, home: Path | None = None) -> bool:
     """Report whether the on-disk cache was written by a newer delegate.
 
@@ -1471,9 +1492,15 @@ def load_discovery_cache(
 ) -> JsonObject | None:
     """Load a valid profile snapshot; malformed data is treated as absent."""
     path = discovery_cache_path(profile_name, home=home)
+    memo_key = _discovery_cache_memo_key(path)
+    if memo_key is None:
+        return None
+    if memo_key in _DISCOVERY_CACHE_MEMO:
+        return copy.deepcopy(_DISCOVERY_CACHE_MEMO[memo_key])
     try:
         snapshot = private_io.read_json_object(path)
         if snapshot is None:
+            _DISCOVERY_CACHE_MEMO[memo_key] = None
             return None
         validate_snapshot(
             snapshot,
@@ -1481,16 +1508,19 @@ def load_discovery_cache(
             allow_unknown_harnesses=True,
         )
     except (private_io.RegistryJsonError, ValueError):
+        _DISCOVERY_CACHE_MEMO[memo_key] = None
         return None
 
     harnesses = snapshot["harnesses"]
     if not isinstance(harnesses, dict):  # validate_snapshot already enforces this.
+        _DISCOVERY_CACHE_MEMO[memo_key] = None
         return None
     if any(harness not in KNOWN_ENGINES for harness in harnesses):
         snapshot = dict(snapshot)
         snapshot["harnesses"] = {
             harness: record for harness, record in harnesses.items() if harness in KNOWN_ENGINES
         }
+    _DISCOVERY_CACHE_MEMO[memo_key] = copy.deepcopy(snapshot)
     return snapshot
 
 
@@ -1523,6 +1553,7 @@ def write_discovery_cache(
             raise FutureCacheSchemaError(FUTURE_SCHEMA_CACHE_WARNING)
 
     private_io.write_json_atomic(path, snapshot, before_replace=refuse_to_replace_a_newer_cache)
+    _clear_discovery_cache_memo(path)
     return path