[PR #360] [MERGED] Register StrategyArchitectAgent in orchestrator and include in Today workflow committee #326

Closed
opened 2026-03-02 23:35:09 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/AJaySi/ALwrity/pull/360
Author: @AJaySi
Created: 3/2/2026
Status: Merged
Merged: 3/2/2026
Merged by: @AJaySi

Base: mainHead: codex/add-strategyarchitectagent-instantiation-and-tests


📝 Commits (1)

  • 83f1edc Add strategy architect agent to daily planning committee

📊 Changes

3 files changed (+79 additions, -7 deletions)

View changed files

📝 backend/services/intelligence/agents/agent_orchestrator.py (+12 -1)
📝 backend/services/today_workflow_service.py (+1 -1)
📝 backend/tests/test_today_workflow_pillar_coverage.py (+66 -5)

📄 Description

Motivation

  • Ensure the StrategyArchitectAgent is part of the orchestrator’s specialized-agent registry so it can contribute PLAN-pillar proposals to the daily committee.
  • Allow strategy-origin proposals to participate in the existing dedupe + pillar-coverage guardrails so higher-priority strategy suggestions can win duplicates and still preserve full-pillar coverage.

Description

  • Import and instantiate StrategyArchitectAgent in ALwrityAgentOrchestrator._create_specialized_agents() and register it under the stable key self.agents['strategy'] using the existing enable/disable profile lookup.
  • Add orchestrator.agents.get('strategy') to the agents_to_poll list in generate_agent_enhanced_plan so the strategy agent is polled alongside other specialists.
  • Extend backend/tests/test_today_workflow_pillar_coverage.py to mock a strategy slot in the orchestrator, add a DummyMemoryService, and include a focused test that asserts a PLAN task originating from StrategyArchitectAgent wins dedupe (higher priority) and remains after pillar-coverage backfill.

Testing

  • Ran pytest backend/tests/test_today_workflow_pillar_coverage.py -q which initially failed collection due to module path configuration (missing PYTHONPATH=backend).
  • Ran PYTHONPATH=backend pytest backend/tests/test_today_workflow_pillar_coverage.py -q and all tests in that file passed (4 passed).

Codex Task


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/AJaySi/ALwrity/pull/360 **Author:** [@AJaySi](https://github.com/AJaySi) **Created:** 3/2/2026 **Status:** ✅ Merged **Merged:** 3/2/2026 **Merged by:** [@AJaySi](https://github.com/AJaySi) **Base:** `main` ← **Head:** `codex/add-strategyarchitectagent-instantiation-and-tests` --- ### 📝 Commits (1) - [`83f1edc`](https://github.com/AJaySi/ALwrity/commit/83f1edcd12ff305d912e77003b50908f6d0d7e1c) Add strategy architect agent to daily planning committee ### 📊 Changes **3 files changed** (+79 additions, -7 deletions) <details> <summary>View changed files</summary> 📝 `backend/services/intelligence/agents/agent_orchestrator.py` (+12 -1) 📝 `backend/services/today_workflow_service.py` (+1 -1) 📝 `backend/tests/test_today_workflow_pillar_coverage.py` (+66 -5) </details> ### 📄 Description ### Motivation - Ensure the StrategyArchitectAgent is part of the orchestrator’s specialized-agent registry so it can contribute PLAN-pillar proposals to the daily committee. - Allow strategy-origin proposals to participate in the existing dedupe + pillar-coverage guardrails so higher-priority strategy suggestions can win duplicates and still preserve full-pillar coverage. ### Description - Import and instantiate `StrategyArchitectAgent` in `ALwrityAgentOrchestrator._create_specialized_agents()` and register it under the stable key `self.agents['strategy']` using the existing enable/disable profile lookup. - Add `orchestrator.agents.get('strategy')` to the `agents_to_poll` list in `generate_agent_enhanced_plan` so the strategy agent is polled alongside other specialists. - Extend `backend/tests/test_today_workflow_pillar_coverage.py` to mock a `strategy` slot in the orchestrator, add a `DummyMemoryService`, and include a focused test that asserts a PLAN task originating from `StrategyArchitectAgent` wins dedupe (higher priority) and remains after pillar-coverage backfill. ### Testing - Ran `pytest backend/tests/test_today_workflow_pillar_coverage.py -q` which initially failed collection due to module path configuration (missing `PYTHONPATH=backend`). - Ran `PYTHONPATH=backend pytest backend/tests/test_today_workflow_pillar_coverage.py -q` and all tests in that file passed (4 passed). ------ [Codex Task](https://chatgpt.com/codex/tasks/task_e_69a47afe5cbc83289888f5f533720a3e) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-02 23:35:09 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/ALwrity#326
No description provided.