Fix TDD test: actor list validation rejects multi-slash model names #4176

Closed
opened 2026-04-06 16:14:16 +00:00 by freemo · 0 comments
Owner

Bug Description

The test in features/tdd_actor_list_validation.feature is failing.

When a provider has a model name containing / separators (e.g. anthropic/claude-sonnet-4-20250514), the actor list command raises a validation error because the resulting name has 2+ slashes.

Root Cause

ActorRegistry._actor_name() builds actor names via f"{provider}/{model}". When a provider's default model already contains /, the resulting name has 2+ slashes and ActorService._normalize_name() rejects it with ValidationError("Actor names must include exactly one '/' separator").

Test File

features/tdd_actor_list_validation.feature

Original Issue

TDD Issue #592

Parent Epic

Part of #13 - Epic: Restore All Quality Tests to Passing State

Acceptance Criteria

  • Actor list command handles multi-slash model names correctly
  • Validation error is not raised for valid provider/model combinations
  • Test passes without @tdd_expected_fail tag
## Bug Description The test in `features/tdd_actor_list_validation.feature` is failing. When a provider has a model name containing `/` separators (e.g. `anthropic/claude-sonnet-4-20250514`), the actor list command raises a validation error because the resulting name has 2+ slashes. ## Root Cause `ActorRegistry._actor_name()` builds actor names via `f"{provider}/{model}"`. When a provider's default model already contains `/`, the resulting name has 2+ slashes and `ActorService._normalize_name()` rejects it with `ValidationError("Actor names must include exactly one '/' separator")`. ## Test File `features/tdd_actor_list_validation.feature` ## Original Issue TDD Issue #592 ## Parent Epic Part of #13 - Epic: Restore All Quality Tests to Passing State ## Acceptance Criteria - [ ] Actor list command handles multi-slash model names correctly - [ ] Validation error is not raised for valid provider/model combinations - [ ] Test passes without `@tdd_expected_fail` tag
freemo added this to the v3.2.0 milestone 2026-04-06 17:48:35 +00:00
freemo removed this from the v3.2.0 milestone 2026-04-06 20:48:02 +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#4176
No description provided.