UAT: `agents actor context import` uses `--input` option instead of spec-required positional `<FILE>` argument #6315

New issue

Open

opened 2026-04-09 20:09:32 +00:00 by HAL9000 · 0 comments

HAL9000 commented

2026-04-09 20:09:32 +00:00

Owner

Summary

The spec defines agents actor context import <ACTOR> <FILE> with both <ACTOR> and <FILE> as positional arguments. The implementation takes an optional positional [NAME] and a required --input <FILE> option. The <FILE> argument is not positional as required by the spec, and --input is mandatory, meaning spec-compliant invocations will fail.

Spec Reference

docs/specification.md — Actor Context Commands:

agents actor context import <ACTOR> <FILE>

Imports context for actor <ACTOR> from file <FILE>
Both are positional arguments

Code Location

src/cleveragents/cli/commands/actor_context.py, lines 333–446

@app.command("import")
def context_import(
    name: Annotated[
        str | None,
        typer.Argument(
            help="Name for the imported context (inferred from file if omitted)"
        ),
    ] = None,                                  # line 337 — optional positional (not actor name)
    input_file: Annotated[
        Path,
        typer.Option(
            "--input",
            "-i",
            help="Input file path (JSON or YAML)",
            exists=True,
            ...
        ),
    ] = ...,                                   # line 342 — REQUIRED OPTION (not positional)

Issues:

name is optional (defaults to None, inferred from the file) — spec requires <ACTOR> as mandatory positional
input_file is a named option (--input) not a positional argument
The = ... at line 353 makes --input mandatory

Steps to Reproduce

# Spec-compliant invocation
agents actor context import local/my-actor ./backup.json
# Result: "local/my-actor" becomes NAME, "./backup.json" is an unrecognised argument
# Error: Got unexpected extra argument (./backup.json)

# Required invocation (implementation)
agents actor context import local/my-actor --input ./backup.json

Expected vs Actual

	Expected	Actual
`<ACTOR>`	Mandatory positional	Optional positional (used as context name, not actor reference)
`<FILE>`	Mandatory positional	Required `--input` option

Impact

Users following the spec signature receive errors. The semantic model differs — the spec ties import to a specific actor, while the implementation treats it as a standalone context import by name.

Automated by CleverAgents Bot
Supervisor: UAT Testing | Agent: uat-tester

## Summary The spec defines `agents actor context import <ACTOR> <FILE>` with both `<ACTOR>` and `<FILE>` as **positional arguments**. The implementation takes an optional positional `[NAME]` and a required `--input <FILE>` option. The `<FILE>` argument is not positional as required by the spec, and `--input` is mandatory, meaning spec-compliant invocations will fail. ## Spec Reference `docs/specification.md` — Actor Context Commands: ``` agents actor context import <ACTOR> <FILE> ``` - Imports context for actor `<ACTOR>` from file `<FILE>` - Both are positional arguments ## Code Location `src/cleveragents/cli/commands/actor_context.py`, lines 333–446 ```python @app.command("import") def context_import( name: Annotated[ str | None, typer.Argument( help="Name for the imported context (inferred from file if omitted)" ), ] = None, # line 337 — optional positional (not actor name) input_file: Annotated[ Path, typer.Option( "--input", "-i", help="Input file path (JSON or YAML)", exists=True, ... ), ] = ..., # line 342 — REQUIRED OPTION (not positional) ``` Issues: 1. `name` is **optional** (defaults to `None`, inferred from the file) — spec requires `<ACTOR>` as mandatory positional 2. `input_file` is a **named option** (`--input`) not a positional argument 3. The `= ...` at line 353 makes `--input` mandatory ## Steps to Reproduce ```bash # Spec-compliant invocation agents actor context import local/my-actor ./backup.json # Result: "local/my-actor" becomes NAME, "./backup.json" is an unrecognised argument # Error: Got unexpected extra argument (./backup.json) # Required invocation (implementation) agents actor context import local/my-actor --input ./backup.json ``` ## Expected vs Actual | | Expected | Actual | |---|---|---| | `<ACTOR>` | Mandatory positional | Optional positional (used as context name, not actor reference) | | `<FILE>` | Mandatory positional | Required `--input` option | ## Impact Users following the spec signature receive errors. The semantic model differs — the spec ties import to a specific actor, while the implementation treats it as a standalone context import by name. --- **Automated by CleverAgents Bot** Supervisor: UAT Testing | Agent: uat-tester

Rows
Columns

UAT: agents actor context import uses --input option instead of spec-required positional <FILE> argument #6315