Feature: agents actor add NAME positional argument
  As a user of the CleverAgents CLI
  I want to pass the actor name as a positional argument to `agents actor add`
  So that the CLI matches the spec synopsis: agents actor add --config <FILE> [<NAME>]

  @tdd_issue @tdd_issue_4230 @tdd_expected_fail
  Scenario: actor add accepts NAME as positional argument
    Given an actor CLI runner
    And I have an actor JSON config file without a name field
    When I run actor add with NAME positional argument and config
    Then the actor add should succeed with the positional name

  @tdd_issue @tdd_issue_4230 @tdd_expected_fail
  Scenario: actor add NAME positional argument takes precedence over config name
    Given an actor CLI runner
    And I have an actor JSON config file with a different name
    When I run actor add with NAME positional argument overriding config name
    Then the actor add should use the positional NAME not the config name

  @tdd_issue @tdd_issue_4186
  Scenario: actor add without NAME positional argument uses config name
    Given an actor CLI runner
    And I have an actor JSON config file with name "local/config-derived-actor"
    When I run actor add with config but no NAME positional argument
    Then the actor add should succeed using the config name

  @tdd_issue @tdd_issue_4186
  Scenario: actor add without NAME and without config name field raises BadParameter
    Given an actor CLI runner
    And I have an actor JSON config file without a name field
    When I run actor add with config but no NAME positional argument
    Then the actor add should fail with a BadParameter error about missing actor name