cleveragents/cleveragents-core

Fork 3

Unexpected behavior: For `agents actor add`, the name parameter in the command line overrides the name parameter in the YAML file. #6752

New issue

Open

opened 2026-04-10 01:49:45 +00:00 by brent.edwards · 2 comments

brent.edwards commented

2026-04-10 01:49:45 +00:00

Member

Setup

Run the following script:

cat << EOF > simple_llm.yaml
# Simple LLM Actor Example
# Demonstrates the most basic actor configuration with just an LLM and system prompt

name: local/code_reviewer
type: llm
description: Reviews Python code for best practices, style, and potential bugs
version: "1.0"

# LLM configuration
provider: openai
model: gpt-4
system_prompt: |
  You are an expert Python code reviewer. Review code for:
  - PEP 8 style compliance
  - Best practices and design patterns
  - Potential bugs and edge cases
  - Performance considerations
  - Security vulnerabilities

  Provide constructive feedback with specific suggestions for improvement.

# Context and memory settings
context_view: reviewer
memory:
  enabled: true
  max_messages: 20
  max_tokens: 4000

context:
  include_files:
    - "README.md"
    - "pyproject.toml"
  include_dirs:
    - "src/"
  exclude_patterns:
    - "**/__pycache__/**"
    - "*.pyc"
    - "**/.pytest_cache/**"
  max_context_tokens: 8000
EOF

uv venv -c
source .venv/bin/activate
uv pip install /app
agents init --force

When

Notice that the name in the file is local/code-reviewer.

agents actor add local/different_name --config /home/devuser/tests/simple_llm.yaml

Expected

Some information that the name in the file and the name in the command line are different.

Actual

The actor is added exactly as in the command line.

# Setup Run the following script: ``` cat << EOF > simple_llm.yaml # Simple LLM Actor Example # Demonstrates the most basic actor configuration with just an LLM and system prompt name: local/code_reviewer type: llm description: Reviews Python code for best practices, style, and potential bugs version: "1.0" # LLM configuration provider: openai model: gpt-4 system_prompt: | You are an expert Python code reviewer. Review code for: - PEP 8 style compliance - Best practices and design patterns - Potential bugs and edge cases - Performance considerations - Security vulnerabilities Provide constructive feedback with specific suggestions for improvement. # Context and memory settings context_view: reviewer memory: enabled: true max_messages: 20 max_tokens: 4000 context: include_files: - "README.md" - "pyproject.toml" include_dirs: - "src/" exclude_patterns: - "**/__pycache__/**" - "*.pyc" - "**/.pytest_cache/**" max_context_tokens: 8000 EOF uv venv -c source .venv/bin/activate uv pip install /app agents init --force ``` # When Notice that the name in the file is `local/code-reviewer`. ``` agents actor add local/different_name --config /home/devuser/tests/simple_llm.yaml ``` # Expected Some information that the name in the file and the name in the command line are different. # Actual The actor is added exactly as in the command line.

brent.edwards added this to the v3.2.0 milestone

2026-04-10 01:49:45 +00:00

brent.edwards added the

labels

2026-04-10 01:49:45 +00:00

HAL9000 referenced this issue

2026-04-10 02:05:40 +00:00

UAT: agents actor add requires spurious positional NAME argument — spec defines config-only registration #6781

HAL9000 referenced this issue

2026-04-10 04:27:33 +00:00

AUTO-PROD-BLDR: Product Builder Session — v3.7.0 Production Ready #6870

HAL9000 commented

2026-04-10 04:28:11 +00:00

Owner

Thank you for filing this issue, @brent.edwards. I am reviewing it now and will provide triage feedback shortly.

Automated by CleverAgents Bot
Supervisor: Human Liaison | Agent: human-liaison

Thank you for filing this issue, @brent.edwards. I am reviewing it now and will provide triage feedback shortly. --- **Automated by CleverAgents Bot** Supervisor: Human Liaison | Agent: human-liaison

HAL9000 added

and removed

labels

2026-04-10 04:31:05 +00:00

HAL9000 commented

2026-04-10 04:31:46 +00:00

Owner

Issue verified and triaged:

Priority: Low — this is a UX improvement (warning when CLI name differs from YAML name), not a data loss or functional breakage issue. The current behavior is technically consistent (CLI argument takes precedence), but the lack of feedback is confusing.
Milestone: v3.2.0 (already assigned)
Story Points: 1 (XS) — the fix is straightforward: detect when the CLI-provided name differs from the YAML name field and emit a warning or informational message.
Type: Bug — the behavior is unexpected and undocumented.
Next step: This issue is now State/Verified and ready for implementation.

Implementation note: The fix should add a check in the agents actor add command handler that compares the CLI-provided name against the name field in the loaded YAML config. If they differ, emit a clear message (e.g., Warning: CLI name 'local/different_name' overrides YAML name 'local/code_reviewer'). The behavior of using the CLI name should be preserved — only the silent override needs to be addressed.

Automated by CleverAgents Bot
Supervisor: Human Liaison | Agent: human-liaison

Issue verified and triaged: - **Priority**: Low — this is a UX improvement (warning when CLI name differs from YAML name), not a data loss or functional breakage issue. The current behavior is technically consistent (CLI argument takes precedence), but the lack of feedback is confusing. - **Milestone**: v3.2.0 (already assigned) - **Story Points**: 1 (XS) — the fix is straightforward: detect when the CLI-provided name differs from the YAML `name` field and emit a warning or informational message. - **Type**: Bug — the behavior is unexpected and undocumented. - **Next step**: This issue is now `State/Verified` and ready for implementation. **Implementation note:** The fix should add a check in the `agents actor add` command handler that compares the CLI-provided name against the `name` field in the loaded YAML config. If they differ, emit a clear message (e.g., `Warning: CLI name 'local/different_name' overrides YAML name 'local/code_reviewer'`). The behavior of using the CLI name should be preserved — only the silent override needs to be addressed. --- **Automated by CleverAgents Bot** Supervisor: Human Liaison | Agent: human-liaison

HAL9000 referenced this issue

2026-04-10 04:35:42 +00:00

AUTO-PROD-BLDR: Product Builder Session — v3.7.0 Production Ready #6870

HAL9000 referenced this issue

2026-04-10 04:54:03 +00:00

AUTO-PROD-BLDR: Product Builder Session — v3.7.0 Production Ready #6870

HAL9000 referenced this issue

2026-04-10 05:04:55 +00:00

AUTO-PROD-BLDR: Product Builder Session — v3.7.0 Production Ready #6870