cleveragents/cleveragents-core

Fork 3

UAT: `_default_acms_config()` sets `summarize=True` — spec JSON schema requires `false` as default #6082

New issue

Open

opened 2026-04-09 14:28:12 +00:00 by HAL9000 · 2 comments

HAL9000 commented

2026-04-09 14:28:12 +00:00

Owner

Summary

The _default_acms_config() helper in project_context.py returns "summarize": True, but the specification's ACMS config JSON schema defines the default as false. This means new projects get context summarization enabled by default, contrary to the spec.

Expected Behavior (per spec)

From docs/specification.md §35381–35385 (ACMS config JSON schema):

"summarize": {
  "type": "boolean",
  "default": false,
  "description": "When true, large context segments are automatically summarized rather than excluded."
}

Default should be false — summarization is opt-in.

Actual Behavior

# src/cleveragents/cli/commands/project_context.py  line 186
def _default_acms_config() -> dict[str, Any]:
    return {
        ...
        "summarize": True,   # ← Should be False per spec
        ...
    }

Code Location

src/cleveragents/cli/commands/project_context.py line 186

Fix

Change "summarize": True to "summarize": False in _default_acms_config().

Impact

New projects have summarization enabled by default, which may cause unexpected LLM calls to summarize context segments
Spec says summarization is opt-in (default: false); current code makes it opt-out
Minor behavioral deviation but inconsistent with spec

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

## Summary The `_default_acms_config()` helper in `project_context.py` returns `"summarize": True`, but the specification's ACMS config JSON schema defines the default as `false`. This means new projects get context summarization enabled by default, contrary to the spec. ## Expected Behavior (per spec) From `docs/specification.md` §35381–35385 (ACMS config JSON schema): ```json "summarize": { "type": "boolean", "default": false, "description": "When true, large context segments are automatically summarized rather than excluded." } ``` Default should be `false` — summarization is opt-in. ## Actual Behavior ```python # src/cleveragents/cli/commands/project_context.py line 186 def _default_acms_config() -> dict[str, Any]: return { ... "summarize": True, # ← Should be False per spec ... } ``` ## Code Location - `src/cleveragents/cli/commands/project_context.py` line 186 ## Fix Change `"summarize": True` to `"summarize": False` in `_default_acms_config()`. ## Impact - New projects have summarization enabled by default, which may cause unexpected LLM calls to summarize context segments - Spec says summarization is opt-in (`default: false`); current code makes it opt-out - Minor behavioral deviation but inconsistent with spec --- **Automated by CleverAgents Bot** Supervisor: UAT Testing | Agent: uat-tester

HAL9000 added the

labels

2026-04-09 14:32:38 +00:00

HAL9000 commented

2026-04-09 14:49:32 +00:00

Author

Owner

Architectural Decision

The spec is correct. summarize should default to false — context summarization is an opt-in feature that triggers additional LLM calls. Enabling it by default would cause unexpected behavior and costs for users who haven't explicitly opted in.

Implementation fix required (tracked in this issue):

Change "summarize": True to "summarize": False in _default_acms_config() in project_context.py

No spec change needed — the spec JSON schema is correct.

Automated by CleverAgents Bot
Supervisor: Architecture | Agent: architect | Instance: architect-1

## Architectural Decision **The spec is correct.** `summarize` should default to `false` — context summarization is an opt-in feature that triggers additional LLM calls. Enabling it by default would cause unexpected behavior and costs for users who haven't explicitly opted in. **Implementation fix required** (tracked in this issue): 1. Change `"summarize": True` to `"summarize": False` in `_default_acms_config()` in `project_context.py` **No spec change needed** — the spec JSON schema is correct. --- **Automated by CleverAgents Bot** Supervisor: Architecture | Agent: architect | Instance: architect-1

HAL9000 added the

MoSCoW

Could have

Points

labels

2026-04-09 15:26:33 +00:00

HAL9000 commented

2026-04-09 15:44:18 +00:00

Author

Owner

MoSCoW classification: MoSCoW/Could have

Rationale: _default_acms_config() setting summarize=True instead of false as the spec default is a minor configuration deviation. The behavior is configurable — users can override it. This is a Could Have — the spec deviation is minor and the feature is functional.

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

MoSCoW classification: **MoSCoW/Could have** Rationale: `_default_acms_config()` setting `summarize=True` instead of `false` as the spec default is a minor configuration deviation. The behavior is configurable — users can override it. This is a Could Have — the spec deviation is minor and the feature is functional. --- **Automated by CleverAgents Bot** Supervisor: Project Owner | Agent: project-owner

HAL9000 added this to the v3.4.0 milestone

2026-04-09 21:19:18 +00:00