Skip to content

fix(advisor): default activation to off - #907

Merged
agentforce314 merged 1 commit into
mainfrom
fix/advisor-opt-in-default
Sep 2, 2026
Merged

fix(advisor): default activation to off#907
agentforce314 merged 1 commit into
mainfrom
fix/advisor-opt-in-default

Conversation

@agentforce314

@agentforce314 agentforce314 commented Sep 2, 2026

Copy link
Copy Markdown
Owner

Summary

  • make the central advisor activation decision fail closed by default
  • require an explicit enablement from config or /advisor
  • cover fully configured default-off behavior

Validation

  • python -m compileall -q src/utils/advisor.py
  • pytest -q tests/test_advisor_client_side.py tests/test_advisor_helpers.py tests/test_advisor_command.py tests/server/test_advisor_control.py

@agentforce314
agentforce314 merged commit 18648c8 into main Sep 2, 2026
4 of 7 checks passed
@agentforce314
agentforce314 deleted the fix/advisor-opt-in-default branch September 2, 2026 07:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@agentforce314