UAT: agents actor context remove takes PATHS... (file paths) instead of spec-required [--yes|-y] (--all|-a|<NAME>) (named context removal); missing --yes/-y and --all/-a flags #6545

Open
opened 2026-04-09 21:17:07 +00:00 by HAL9000 · 0 comments
Owner

Summary

The spec defines agents actor context remove [--yes|-y] (--all|-a|<NAME>) to remove a named context (or all contexts). The current implementation (from context.py) takes PATHS... — a list of file paths to remove from the current plan's context. The semantics are completely different: the spec removes named contexts, the implementation removes files from a project context.

Spec Reference

docs/specification.md line 283:

agents actor context remove [--yes|-y] (--all|-a|<NAME>)

docs/specification.md lines 5772–5783:

agents actor context remove [--yes|-y] (--all|-a|<NAME>)

Arguments:
- <NAME>: Context name (positional argument). Use --all/-a to target all contexts.
- --all, -a: Target all contexts instead of a named one.
- --yes, -y: Skip confirmation prompt.

Code Location

src/cleveragents/cli/commands/context.py, lines 305–376 (context_remove):

@app.command("remove")
def context_remove(
    paths: Annotated[
        list[str],
        typer.Argument(help="Paths to remove from context"),
    ],
) -> None:
    """Remove files or directories from the current plan's context."""
    # Takes file PATHS, not a context NAME
    # No --yes/-y, no --all/-a

Steps to Reproduce

agents actor context remove --help
# Shows: PATHS... (required file paths)
# Missing: --yes/-y, --all/-a flags; NAME argument

# Spec-compliant invocation
agents actor context remove docs
# Actual: Tries to remove a file/dir named "docs" from current plan context
# Expected: Removes the named context "docs"

agents actor context remove --all --yes
# Error: Got unexpected extra argument (--all)

Expected vs Actual

Expected Actual
Signature remove [--yes|-y] (--all|-a|\<NAME\>) remove PATHS...
--yes/-y Skip confirmation Missing
--all/-a Remove all contexts Missing
<NAME> Named context to remove File path(s) to remove from plan context
Operation Remove named context directory Remove files from current plan's context

Impact

Users cannot remove named contexts via agents actor context remove. The spec-required --all/-a and --yes/-y flags are missing. The command has completely different semantics from the spec.


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

## Summary The spec defines `agents actor context remove [--yes|-y] (--all|-a|<NAME>)` to remove a named context (or all contexts). The current implementation (from `context.py`) takes `PATHS...` — a list of file paths to remove from the current plan's context. The semantics are completely different: the spec removes named contexts, the implementation removes files from a project context. ## Spec Reference `docs/specification.md` line 283: ``` agents actor context remove [--yes|-y] (--all|-a|<NAME>) ``` `docs/specification.md` lines 5772–5783: ``` agents actor context remove [--yes|-y] (--all|-a|<NAME>) Arguments: - <NAME>: Context name (positional argument). Use --all/-a to target all contexts. - --all, -a: Target all contexts instead of a named one. - --yes, -y: Skip confirmation prompt. ``` ## Code Location `src/cleveragents/cli/commands/context.py`, lines 305–376 (`context_remove`): ```python @app.command("remove") def context_remove( paths: Annotated[ list[str], typer.Argument(help="Paths to remove from context"), ], ) -> None: """Remove files or directories from the current plan's context.""" # Takes file PATHS, not a context NAME # No --yes/-y, no --all/-a ``` ## Steps to Reproduce ```bash agents actor context remove --help # Shows: PATHS... (required file paths) # Missing: --yes/-y, --all/-a flags; NAME argument # Spec-compliant invocation agents actor context remove docs # Actual: Tries to remove a file/dir named "docs" from current plan context # Expected: Removes the named context "docs" agents actor context remove --all --yes # Error: Got unexpected extra argument (--all) ``` ## Expected vs Actual | | Expected | Actual | |---|---|---| | Signature | `remove [--yes\|-y] (--all\|-a\|\<NAME\>)` | `remove PATHS...` | | `--yes/-y` | Skip confirmation | Missing | | `--all/-a` | Remove all contexts | Missing | | `<NAME>` | Named context to remove | File path(s) to remove from plan context | | Operation | Remove named context directory | Remove files from current plan's context | ## Impact Users cannot remove named contexts via `agents actor context remove`. The spec-required `--all/-a` and `--yes/-y` flags are missing. The command has completely different semantics from the spec. --- **Automated by CleverAgents Bot** Supervisor: UAT Testing | Agent: uat-tester
HAL9000 added this to the v3.2.0 milestone 2026-04-09 21:28:38 +00:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
cleveragents/cleveragents-core#6545
No description provided.