UAT: actor context CLI mismatch — robot test uses delete command and positional output path but source only has remove command and --output option #2775

Closed
opened 2026-04-04 19:23:36 +00:00 by freemo · 5 comments
Owner

Transitioning State from State/Verified to State/In Progress per Ticket Lifecycle.

Transitioning State from State/Verified to State/In Progress per Ticket Lifecycle.
freemo added this to the v3.2.0 milestone 2026-04-04 19:23:41 +00:00
Author
Owner

Issue triaged by project owner:

  • State: Verified
  • Priority: High — Robot test calls CLI commands that don't exist in the source (delete instead of remove, positional args instead of options). Integration tests will fail.
  • Milestone: v3.2.0 (already set)
  • MoSCoW: Should Have — The test/source mismatch means integration tests for actor context export/import are broken. The fix direction should align with the spec — check whether the spec says delete or remove and whether paths should be positional or options.
  • Parent Epic: #396 (ACMS Context Pipeline)

This is a PR #2629 follow-up issue. The source code change was omitted from the PR.


Automated by CleverAgents Bot
Supervisor: Project Owner | Agent: ca-project-owner

Issue triaged by project owner: - **State**: Verified - **Priority**: High — Robot test calls CLI commands that don't exist in the source (`delete` instead of `remove`, positional args instead of options). Integration tests will fail. - **Milestone**: v3.2.0 (already set) - **MoSCoW**: Should Have — The test/source mismatch means integration tests for actor context export/import are broken. The fix direction should align with the spec — check whether the spec says `delete` or `remove` and whether paths should be positional or options. - **Parent Epic**: #396 (ACMS Context Pipeline) This is a PR #2629 follow-up issue. The source code change was omitted from the PR. --- **Automated by CleverAgents Bot** Supervisor: Project Owner | Agent: ca-project-owner
Author
Owner

Starting implementation on branch fix/actor-context-cli-delete-positional-args.

Analysis complete. After reviewing the spec, source code, and robot test:

  • Spec (docs/specification.md) defines: actor context remove, --output FILE, --input FILE (named options, not positional)
  • Source (actor_context.py) correctly implements the spec
  • Robot test (actor_context_export_import.robot) is incorrect — uses delete instead of remove, and positional path args instead of --output/--input

Fix direction: Update the robot test to match the spec and source (Option 2 from the issue).

Changes needed in robot/actor_context_export_import.robot:

  1. Replace actor context deleteactor context remove
  2. Replace positional ${export_path}--output ${export_path} in export calls
  3. Replace positional ${import_path}--input ${import_path} in import calls
  4. Add --update flag to the import-into-existing-context test case

Automated by CleverAgents Bot
Supervisor: Implementation | Agent: ca-issue-worker

Starting implementation on branch `fix/actor-context-cli-delete-positional-args`. **Analysis complete.** After reviewing the spec, source code, and robot test: - **Spec** (`docs/specification.md`) defines: `actor context remove`, `--output FILE`, `--input FILE` (named options, not positional) - **Source** (`actor_context.py`) correctly implements the spec - **Robot test** (`actor_context_export_import.robot`) is incorrect — uses `delete` instead of `remove`, and positional path args instead of `--output`/`--input` **Fix direction: Update the robot test to match the spec and source** (Option 2 from the issue). Changes needed in `robot/actor_context_export_import.robot`: 1. Replace `actor context delete` → `actor context remove` 2. Replace positional `${export_path}` → `--output ${export_path}` in export calls 3. Replace positional `${import_path}` → `--input ${import_path}` in import calls 4. Add `--update` flag to the import-into-existing-context test case --- **Automated by CleverAgents Bot** Supervisor: Implementation | Agent: ca-issue-worker
Author
Owner

All subtasks complete. Quality gates passed. PR #3179 created on branch fix/actor-context-cli-delete-positional-args. PR review and merge handled by continuous review stream.

Implementation summary:

  • Fixed robot/actor_context_export_import.robot to align with spec and source
  • actor context deleteactor context remove (spec uses remove)
  • Positional export path → --output PATH named option (per spec)
  • Positional import path → --input PATH named option (per spec)
  • Added --update flag to import-into-existing-context test case
  • All nox quality gates pass (lint , typecheck , security_scan )

Automated by CleverAgents Bot
Supervisor: Implementation | Agent: ca-issue-worker

All subtasks complete. Quality gates passed. PR #3179 created on branch `fix/actor-context-cli-delete-positional-args`. PR review and merge handled by continuous review stream. **Implementation summary:** - Fixed `robot/actor_context_export_import.robot` to align with spec and source - `actor context delete` → `actor context remove` (spec uses `remove`) - Positional export path → `--output PATH` named option (per spec) - Positional import path → `--input PATH` named option (per spec) - Added `--update` flag to import-into-existing-context test case - All nox quality gates pass (lint ✅, typecheck ✅, security_scan ✅) --- **Automated by CleverAgents Bot** Supervisor: Implementation | Agent: ca-issue-worker
Author
Owner

Issue Body Compliance Problem: This issue's body contains only a state transition log ("Transitioning State from State/Verified to State/In Progress per Ticket Lifecycle.") instead of the actual issue description.

Per CONTRIBUTING.md, every issue body must contain:

  • ## Metadata section (with Branch and Commit Message)
  • ## Subtasks section (with at least one checkbox)
  • ## Definition of Done section

The original issue description for actor context CLI mismatch appears to have been overwritten. Please restore the original issue description.


Automated by CleverAgents Bot
Supervisor: Backlog Grooming | Agent: ca-backlog-groomer

**Issue Body Compliance Problem:** This issue's body contains only a state transition log ("Transitioning State from State/Verified to State/In Progress per Ticket Lifecycle.") instead of the actual issue description. Per CONTRIBUTING.md, every issue body must contain: - `## Metadata` section (with Branch and Commit Message) - `## Subtasks` section (with at least one checkbox) - `## Definition of Done` section The original issue description for `actor context` CLI mismatch appears to have been overwritten. Please restore the original issue description. --- **Automated by CleverAgents Bot** Supervisor: Backlog Grooming | Agent: ca-backlog-groomer
Owner

State label reconciliation:

  • Corrected to: State/Completed
  • Reason: Issue is closed but had a non-terminal state label (State/In Review, State/Verified, or State/In Progress). CONTRIBUTING.md requires closed issues to have State/Completed or State/Wont Do.

Automated by CleverAgents Bot
Supervisor: Backlog Grooming | Agent: backlog-groomer

State label reconciliation: - Corrected to: `State/Completed` - Reason: Issue is closed but had a non-terminal state label (`State/In Review`, `State/Verified`, or `State/In Progress`). CONTRIBUTING.md requires closed issues to have `State/Completed` or `State/Wont Do`. --- **Automated by CleverAgents Bot** Supervisor: Backlog Grooming | Agent: backlog-groomer
Sign in to join this conversation.
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

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