Skip to content

fix(compose): resolve overlay-injected design_pattern from entry.spec when entry top-level is absent - #28

Merged
LenGul merged 1 commit into
outshift-open:mainfrom
LenGul:fix/overlay-design-pattern-injection
Aug 5, 2026
Merged

fix(compose): resolve overlay-injected design_pattern from entry.spec when entry top-level is absent#28
LenGul merged 1 commit into
outshift-open:mainfrom
LenGul:fix/overlay-design-pattern-injection

Conversation

@LenGul

@LenGul LenGul commented Aug 4, 2026

Copy link
Copy Markdown
Collaborator

When a MAS overlay patches agents..design_pattern, merge_mas_overlay forwards it to merge_agent_overlay, which writes all per-agent keys into entry["spec"]. The pattern resolver (pattern_for_agent) only checked the top-level entry["design_pattern"], which was always None for overlay-injected patterns, causing unconditional fallback to the default plugin.

Before: entry["design_pattern"] → None → react@v1
After: entry["design_pattern"] or entry["spec"]["design_pattern"] → centralized_moderator@v1

… when entry top-level is absent

Signed-off-by: Lennart Gulikers <lguliker@cisco.com>
@LenGul
LenGul force-pushed the fix/overlay-design-pattern-injection branch from bd5a7d6 to 402c290 Compare August 4, 2026 10:08
@LenGul LenGul self-assigned this Aug 5, 2026
@LenGul LenGul added the bug Something isn't working label Aug 5, 2026
@LenGul
LenGul requested a review from psterpu-c August 5, 2026 12:56

@psterpu-c psterpu-c left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@LenGul
LenGul merged commit 4aed609 into outshift-open:main Aug 5, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants