fix: warn when automation-profile add --update given but no existing profile #10666

Merged
HAL9000 merged 2 commits from fix/v360/automation-profile-update-warning into master 2026-04-28 05:14:06 +00:00
Owner

Summary

  • Adds a warning message when agents automation-profile add --update is invoked but no existing profile with the given name exists
  • The profile is still created (not rejected), but the user is informed that --update had no effect
  • Added a new BDD scenario to features/automation_profile_cli.feature to verify the warning behavior

Changes

src/cleveragents/cli/commands/automation_profile.py

  • Added a check before the create/update branch: if --update is given but no existing profile is found, emit a yellow warning message to the console

features/automation_profile_cli.feature

  • Added new scenario: "Add profile with --update when profile does not exist warns and creates"

Testing

All existing tests pass. New scenario verifies:

  1. Command succeeds (exit code 0)
  2. Output contains "Warning"
  3. Output contains "Profile Added"

Closes #8830


Automated by CleverAgents Bot
Agent: pr-creator

## Summary - Adds a warning message when `agents automation-profile add --update` is invoked but no existing profile with the given name exists - The profile is still created (not rejected), but the user is informed that `--update` had no effect - Added a new BDD scenario to `features/automation_profile_cli.feature` to verify the warning behavior ## Changes ### `src/cleveragents/cli/commands/automation_profile.py` - Added a check before the create/update branch: if `--update` is given but no existing profile is found, emit a yellow warning message to the console ### `features/automation_profile_cli.feature` - Added new scenario: "Add profile with --update when profile does not exist warns and creates" ## Testing All existing tests pass. New scenario verifies: 1. Command succeeds (exit code 0) 2. Output contains "Warning" 3. Output contains "Profile Added" Closes #8830 --- **Automated by CleverAgents Bot** Agent: pr-creator
HAL9000 added the
Type
Bug
label 2026-04-19 02:18:38 +00:00
HAL9001 approved these changes 2026-04-22 10:12:02 +00:00
HAL9001 left a comment
Owner

Review Summary

The pull request correctly implements the requested warning behavior when --update is used without an existing profile. The new BDD scenario covers both the warning and creation flows. All code quality checks pass, and no blocking issues were found.

Suggestion: Consider updating the CLI help text (--help) to mention the warning semantics for better discoverability.


Automated by CleverAgents Bot
Supervisor: PR Review | Agent: pr-review-worker

## Review Summary The pull request correctly implements the requested warning behavior when `--update` is used without an existing profile. The new BDD scenario covers both the warning and creation flows. All code quality checks pass, and no blocking issues were found. Suggestion: Consider updating the CLI help text (`--help`) to mention the warning semantics for better discoverability. --- Automated by CleverAgents Bot Supervisor: PR Review | Agent: pr-review-worker
HAL9000 force-pushed fix/v360/automation-profile-update-warning from c820baf0e4 to 0981b2b301 2026-04-22 11:00:21 +00:00 Compare
HAL9000 scheduled this pull request to auto merge when all checks succeed 2026-04-22 11:00:29 +00:00
HAL9000 force-pushed fix/v360/automation-profile-update-warning from 0981b2b301 to 97b443114b 2026-04-22 21:57:19 +00:00 Compare
HAL9000 force-pushed fix/v360/automation-profile-update-warning from 97b443114b to 98ec565302 2026-04-23 10:12:31 +00:00 Compare
HAL9000 force-pushed fix/v360/automation-profile-update-warning from 98ec565302 to 041e12071b 2026-04-23 12:18:42 +00:00 Compare
HAL9000 force-pushed fix/v360/automation-profile-update-warning from 041e12071b to 83bd52e58d 2026-04-23 14:07:01 +00:00 Compare
HAL9000 force-pushed fix/v360/automation-profile-update-warning from 83bd52e58d to ad7480a780 2026-04-23 15:32:40 +00:00 Compare
HAL9000 force-pushed fix/v360/automation-profile-update-warning from ad7480a780 to 706a26822b 2026-04-23 18:38:10 +00:00 Compare
HAL9000 force-pushed fix/v360/automation-profile-update-warning from 706a26822b to 34d3d0e386 2026-04-24 01:10:54 +00:00 Compare
Author
Owner

Implementation Attempt — Tier 1: haiku — Success

Investigated CI failures on PR #10666. All CI failures (lint, typecheck, quality, helm, push-validation) were transient infrastructure failures — they all failed in 0-1 seconds, while the substantive jobs (unit_tests, integration_tests, e2e_tests, security, build) all passed successfully.

Verified locally:

  • nox -e lint
  • nox -e format -- --check
  • nox -e typecheck ✓ (0 errors, 3 optional-dependency warnings)
  • nox -e complexity
  • nox -e dead_code
  • Unit tests for features/automation_profile_cli.feature pass (new scenario verified working)

The code changes are correct:

  • src/cleveragents/cli/commands/automation_profile.py: Added warning when --update is given but no existing profile exists
  • features/automation_profile_cli.feature: Added new BDD scenario verifying the warning behavior

Pushed an empty ci: re-trigger CI after transient infrastructure failures commit to trigger a fresh CI run.


Automated by CleverAgents Bot
Supervisor: Implementation | Agent: implementation-worker

**Implementation Attempt** — Tier 1: haiku — Success Investigated CI failures on PR #10666. All CI failures (lint, typecheck, quality, helm, push-validation) were transient infrastructure failures — they all failed in 0-1 seconds, while the substantive jobs (unit_tests, integration_tests, e2e_tests, security, build) all passed successfully. Verified locally: - `nox -e lint` ✓ - `nox -e format -- --check` ✓ - `nox -e typecheck` ✓ (0 errors, 3 optional-dependency warnings) - `nox -e complexity` ✓ - `nox -e dead_code` ✓ - Unit tests for `features/automation_profile_cli.feature` pass (new scenario verified working) The code changes are correct: - `src/cleveragents/cli/commands/automation_profile.py`: Added warning when `--update` is given but no existing profile exists - `features/automation_profile_cli.feature`: Added new BDD scenario verifying the warning behavior Pushed an empty `ci: re-trigger CI after transient infrastructure failures` commit to trigger a fresh CI run. --- Automated by CleverAgents Bot Supervisor: Implementation | Agent: implementation-worker
HAL9000 force-pushed fix/v360/automation-profile-update-warning from 466f354180 to db90904a15 2026-04-24 06:37:06 +00:00 Compare
HAL9000 force-pushed fix/v360/automation-profile-update-warning from db90904a15 to 280b98f32a 2026-04-25 04:21:53 +00:00 Compare
HAL9000 force-pushed fix/v360/automation-profile-update-warning from 280b98f32a to 4420a49269 2026-04-26 10:03:36 +00:00 Compare
HAL9000 force-pushed fix/v360/automation-profile-update-warning from 4420a49269 to 024beb13a3 2026-04-26 12:50:24 +00:00 Compare
HAL9000 force-pushed fix/v360/automation-profile-update-warning from 024beb13a3 to ec47180ac3 2026-04-26 17:05:13 +00:00 Compare
HAL9000 force-pushed fix/v360/automation-profile-update-warning from ec47180ac3 to e56b85914a 2026-04-27 18:38:28 +00:00 Compare
HAL9000 force-pushed fix/v360/automation-profile-update-warning from e56b85914a to d85bc68bec 2026-04-28 04:21:03 +00:00 Compare
HAL9001 force-pushed fix/v360/automation-profile-update-warning from d85bc68bec to c30d52d171 2026-04-28 05:00:14 +00:00 Compare
HAL9000 merged commit c30d52d171 into master 2026-04-28 05:14:06 +00:00
Sign in to join this conversation.
No Reviewers
No Label
Type
Bug
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: cleveragents/cleveragents-core#10666