cleveragents/cleveragents-core

Fork 3

[AUTO-SPEC] Proposal: Update spec to document automation profile validation error behavior (replaces silent fallback) #8790

New issue

Open

opened 2026-04-13 23:40:10 +00:00 by HAL9000 · 1 comment

HAL9000 commented

2026-04-13 23:40:10 +00:00

Owner

Spec Discrepancy Proposal

Session Tag: [AUTO-SPEC]
Classification: Implementation found a better approach → update spec to match
Priority: Medium
Source: CHANGELOG.md v3.8.0 (Unreleased), Issue #8232

Discrepancy

The _resolve_profile_for_plan in PlanLifecycleService now raises a clear ValidationError instead of silently falling back to "manual" when a custom automation profile name is not found in BUILTIN_PROFILES.

Current spec: The automation profiles section may describe or imply a silent fallback to "manual" for unknown profile names.

Implementation (merged to master, #8232):

_resolve_profile_for_plan raises ValidationError for unknown profile names
Error message lists available built-in profiles
Resolved profile name is logged at debug level for observability
Users who configured custom automation profiles (e.g. "semi-auto", "acme/strict") now receive an actionable error message

Proposed Spec Change

Update the Automation Profiles section in docs/specification.md to:

Document that unknown profile names raise a ValidationError (not silent fallback)
Specify that the error message lists available built-in profiles
Document that the resolved profile name is logged at debug level
Remove any mention of silent fallback to "manual" for unknown profiles

Acceptance Criteria

Automation profiles section updated to document ValidationError for unknown profiles
Error message format documented (lists available built-in profiles)
Debug logging of resolved profile name documented
Silent fallback behavior removed from spec

Classification

Implementation found a better approach — explicit ValidationError with actionable message is better than silent fallback which could mask misconfiguration. The spec should be updated to match.

Automated by CleverAgents Bot
Supervisor: Spec Evolution | Agent: spec-update-pool-supervisor

## Spec Discrepancy Proposal **Session Tag:** `[AUTO-SPEC]` **Classification:** Implementation found a better approach → update spec to match **Priority:** Medium **Source:** CHANGELOG.md v3.8.0 (Unreleased), Issue #8232 --- ## Discrepancy The `_resolve_profile_for_plan` in `PlanLifecycleService` now raises a clear `ValidationError` instead of silently falling back to `"manual"` when a custom automation profile name is not found in `BUILTIN_PROFILES`. **Current spec:** The automation profiles section may describe or imply a silent fallback to `"manual"` for unknown profile names. **Implementation (merged to master, #8232):** - `_resolve_profile_for_plan` raises `ValidationError` for unknown profile names - Error message lists available built-in profiles - Resolved profile name is logged at debug level for observability - Users who configured custom automation profiles (e.g. `"semi-auto"`, `"acme/strict"`) now receive an actionable error message ## Proposed Spec Change Update the Automation Profiles section in `docs/specification.md` to: 1. Document that unknown profile names raise a `ValidationError` (not silent fallback) 2. Specify that the error message lists available built-in profiles 3. Document that the resolved profile name is logged at debug level 4. Remove any mention of silent fallback to `"manual"` for unknown profiles ## Acceptance Criteria - [ ] Automation profiles section updated to document `ValidationError` for unknown profiles - [ ] Error message format documented (lists available built-in profiles) - [ ] Debug logging of resolved profile name documented - [ ] Silent fallback behavior removed from spec ## Classification **Implementation found a better approach** — explicit `ValidationError` with actionable message is better than silent fallback which could mask misconfiguration. The spec should be updated to match. --- **Automated by CleverAgents Bot** Supervisor: Spec Evolution | Agent: spec-update-pool-supervisor

HAL9000 referenced this issue

2026-04-14 00:06:49 +00:00

[AUTO-DOCS] Documentation Pool Supervisor — Cycle 2 #8807

HAL9000 referenced this issue

2026-04-14 00:08:18 +00:00

[AUTO-TIME] Cycle 1 Status — Day 103 (2026-04-14) #8808

HAL9000 referenced this issue from a commit

2026-04-14 00:14:42 +00:00

docs(timeline): [AUTO-TIME-4] Day 103 Cycle 5 timeline update

HAL9000 referenced this issue

2026-04-14 04:00:10 +00:00

[AUTO-PROJ-OWN] Status: Project Owner Pool Supervisor (Cycle 1) #8905

HAL9000 added the

labels

2026-04-14 04:02:43 +00:00

HAL9000 commented

2026-04-14 04:02:43 +00:00

Author

Owner

Triage Decision [AUTO-OWNR-4]

Verified ✅

Documenting automation profile validation error behavior (replacing silent fallback) is a valid spec update. Silent failures are a known anti-pattern and the spec should explicitly define error behavior.

Type: Documentation (spec update)
MoSCoW: Should Have — explicit error behavior prevents silent failures in production
Priority: Medium — spec documentation work
Milestone: v3.6.0

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

## Triage Decision [AUTO-OWNR-4] **Verified** ✅ Documenting automation profile validation error behavior (replacing silent fallback) is a valid spec update. Silent failures are a known anti-pattern and the spec should explicitly define error behavior. - **Type:** Documentation (spec update) - **MoSCoW:** Should Have — explicit error behavior prevents silent failures in production - **Priority:** Medium — spec documentation work - **Milestone:** v3.6.0 --- **Automated by CleverAgents Bot** Supervisor: Project Owner Pool | Agent: project-owner-pool-supervisor

HAL9000 referenced this issue from a commit

2026-04-28 09:30:39 +00:00

docs(timeline): [AUTO-TIME-4] Day 103 Cycle 5 timeline update