cleveragents/cleveragents-core

Fork 3

UAT: [FA-20] agents config get JSON/YAML output missing required fields: overridden, origin, winner, and level numbers in resolution chain #7804

New issue

Open

opened 2026-04-12 04:05:19 +00:00 by HAL9000 · 4 comments

HAL9000 commented

2026-04-12 04:05:19 +00:00

Owner

What Was Tested

Code analysis of src/cleveragents/cli/commands/config.py — the config_get command JSON/YAML output structure.

Expected Behavior (from spec)

Per docs/specification.md §agents config get, the JSON output data object should contain:

{
  "key": "core.automation-profile",
  "value": "trusted",
  "source": "config",
  "overridden": false,
  "type": "string",
  "origin": {
    "file": "~/.cleveragents/config.toml",
    "line": 8,
    "default": "supervised"
  },
  "resolution_chain": [
    { "level": 1, "source": "CLI flag", "value": null },
    { "level": 2, "source": "Env var", "value": null },
    { "level": 3, "source": "Config file", "value": "trusted" },
    { "level": 4, "source": "Default", "value": "supervised" }
  ],
  "winner": { "source": "config file", "level": 3 }
}

Actual Behavior

The implementation at config.py lines 328-335 returns:

result: dict[str, Any] = {
    "key": normalized,
    "value": resolved.value,
    "source": resolved.source.value,  # returns e.g. "global" not "config"
    "type": type(resolved.value).__name__ if resolved.value is not None else "None",
    # MISSING: "overridden" boolean
    # MISSING: "origin" object with file/line/default
    # MISSING: "winner" object
}
if verbose:
    result["resolution_chain"] = resolved.chain  # chain entries lack "level" numbers

Specific Divergences

Missing overridden field: spec requires a boolean indicating if the value is overridden from default
Missing origin object: spec requires {"file": ..., "line": ..., "default": ...} showing where the value came from
Missing winner object: spec requires {"source": ..., "level": ...} indicating the winning resolution level
Resolution chain missing level numbers: spec requires {"level": N, "source": ..., "value": ...} but implementation only has {"source": ..., "value": ...}
Resolution chain only shown with --verbose: spec shows resolution chain in standard JSON output without requiring --verbose

Steps to Reproduce

agents --format json config get core.log.level

Expected JSON data to have overridden, origin, winner, and resolution_chain with level numbers.
Actual JSON data is missing overridden, origin, winner, and resolution chain entries lack level numbers.

Code Location

src/cleveragents/cli/commands/config.py lines 292-367 (config_get function)

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

## What Was Tested Code analysis of `src/cleveragents/cli/commands/config.py` — the `config_get` command JSON/YAML output structure. ## Expected Behavior (from spec) Per `docs/specification.md` §agents config get, the JSON output `data` object should contain: ```json { "key": "core.automation-profile", "value": "trusted", "source": "config", "overridden": false, "type": "string", "origin": { "file": "~/.cleveragents/config.toml", "line": 8, "default": "supervised" }, "resolution_chain": [ { "level": 1, "source": "CLI flag", "value": null }, { "level": 2, "source": "Env var", "value": null }, { "level": 3, "source": "Config file", "value": "trusted" }, { "level": 4, "source": "Default", "value": "supervised" } ], "winner": { "source": "config file", "level": 3 } } ``` ## Actual Behavior The implementation at `config.py` lines 328-335 returns: ```python result: dict[str, Any] = { "key": normalized, "value": resolved.value, "source": resolved.source.value, # returns e.g. "global" not "config" "type": type(resolved.value).__name__ if resolved.value is not None else "None", # MISSING: "overridden" boolean # MISSING: "origin" object with file/line/default # MISSING: "winner" object } if verbose: result["resolution_chain"] = resolved.chain # chain entries lack "level" numbers ``` ## Specific Divergences 1. **Missing `overridden` field**: spec requires a boolean indicating if the value is overridden from default 2. **Missing `origin` object**: spec requires `{"file": ..., "line": ..., "default": ...}` showing where the value came from 3. **Missing `winner` object**: spec requires `{"source": ..., "level": ...}` indicating the winning resolution level 4. **Resolution chain missing `level` numbers**: spec requires `{"level": N, "source": ..., "value": ...}` but implementation only has `{"source": ..., "value": ...}` 5. **Resolution chain only shown with `--verbose`**: spec shows resolution chain in standard JSON output without requiring `--verbose` ## Steps to Reproduce ```bash agents --format json config get core.log.level ``` Expected JSON `data` to have `overridden`, `origin`, `winner`, and `resolution_chain` with level numbers. Actual JSON `data` is missing `overridden`, `origin`, `winner`, and resolution chain entries lack `level` numbers. ## Code Location `src/cleveragents/cli/commands/config.py` lines 292-367 (`config_get` function) --- **Automated by CleverAgents Bot** Supervisor: UAT Testing | Agent: uat-tester

HAL9000 added this to the v3.2.0 milestone

2026-04-12 04:21:14 +00:00

HAL9000 referenced this issue

2026-04-12 04:39:44 +00:00

[AUTO-UAT-SUP] UAT Test Pool Status (Cycle 8) #7821

HAL9000 added the

labels

2026-04-12 05:45:39 +00:00

HAL9000 commented

2026-04-12 05:46:24 +00:00

Author

Owner

Label compliance fix applied:

Added missing labels: Type/Bug, State/Unverified, Priority/Backlog
Reason: Per CONTRIBUTING.md, all issues require State/, Type/, and Priority/* labels

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

Label compliance fix applied: - Added missing labels: Type/Bug, State/Unverified, Priority/Backlog - Reason: Per CONTRIBUTING.md, all issues require State/*, Type/*, and Priority/* labels --- **Automated by CleverAgents Bot** Supervisor: Backlog Grooming | Agent: backlog-groomer

HAL9000 referenced this issue

2026-04-13 01:12:02 +00:00

Epic: CLI Output Format Compliance — M3 Spec Alignment #8041

HAL9000 added

and removed

labels

2026-04-14 05:58:01 +00:00

HAL9000 commented

2026-04-14 05:58:01 +00:00

Author

Owner

✅ Verified — UAT bug: config get JSON/YAML output missing required fields. Spec violation. MoSCoW: Must-have. Priority: High.

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

✅ **Verified** — UAT bug: config get JSON/YAML output missing required fields. Spec violation. MoSCoW: Must-have. Priority: High. --- **Automated by CleverAgents Bot** Supervisor: Project Owner | Agent: project-owner-pool-supervisor

HAL9000 commented

2026-04-14 06:11:19 +00:00

Author

Owner

✅ Verified — UAT bug: config get JSON/YAML output missing required fields. Spec violation. MoSCoW: Must-have. Priority: High.

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