chore: Dev merge to Main by Prachig-Microsoft · Pull Request #273 · microsoft/Container-Migration-Solution-Accelerator

Prachig-Microsoft · 2026-06-04T13:26:18Z

Purpose

This pull request includes several updates across the codebase, focusing on improving naming consistency for Docker build steps, updating and adding dependencies, and making minor code and test improvements. The most significant changes are grouped below:

Docker Build & Naming Consistency:

Updated the names of Docker build steps in .github/workflows/docker-build-and-push.yml to use more generic and consistent names: "Backend API", "Processor", and "Frontend" instead of the previous "ContentProcessorAPI", "ContentProcessor", and "ContentProcessorWeb". This improves clarity and maintainability in CI workflows. [1] [2] [3]

Dependency Updates:

Upgraded werkzeug from version 3.1.4 to 3.1.6 and added idna==3.15 to the override-dependencies in src/backend-api/pyproject.toml to address potential security or compatibility issues.
Added idna==3.15 to the override-dependencies in src/processor/pyproject.toml for consistency and dependency management.

Testing and Code Quality Improvements:

Simplified test service registration in test_application_context_extras.py by passing class references directly instead of lambda factories, making the tests more concise and idiomatic. [1] [2]
Improved clarity in test exception handling comments in test_queue_service_internals.py.
Removed an unused import in test_sk_logic_base.py and cleaned up React Redux imports in batchHistoryPanel.tsx for minor code hygiene. [1] [2]

Does this introduce a breaking change?

Yes
No

Golden Path Validation

I have tested the primary workflows (the "golden path") to ensure they function correctly without errors.

Deployment Validation

I have validated the deployment process successfully and all services are running as expected with this change.

What to Check

Verify that the following are valid

...

Other Information

…mportError

…proved telemetry and error reporting

…and OrchestratorBase

…eter support and backward compatibility

fix: update werkzeug to 3.1.6 and add idna 3.15 to dependencies

When callers pass credential=None explicitly, the key exists in kwargs but the ad_token_provider mapping was skipped. Use kwargs.get() is None to correctly handle this case. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Keep agent-framework==1.3.0 upgrade from PR (per user story 43673). Accept newer general dependency versions from dev branch. Keep upgraded azure_openai_response_retry.py from PR. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

…de clarity - Fix create_agents return type annotation to dict[str, Agent] - Narrow participants param to Mapping only (Sequence was unused) - Normalize self.agents with dict() and correct value type - Replace redundant pass with continue and clarifying comment - Add teardown_module to test files to restore patched Message class Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

- Update InputObserverMiddleware to use Message(contents=) instead of Message(text=) since agent-framework 1.3.0 renamed the parameter - Update corresponding tests to verify contents field - Fix teardown_module signature to accept optional module parameter Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

The teardown_module from one test file was restoring the real Message before another test file's tests ran. Adding setup_module ensures the stub is re-applied before each module's tests execute. Also fix test assertion to check contents instead of text since the middleware now uses Message(contents=). Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

The middleware now uses Message(contents=) so the test must verify the contents field, not text. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

- Set both text= and contents= when constructing Message in InputObserverMiddleware for compatibility with downstream code - Restore missing copyright header in azure_openai_response_retry.py Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

…lity

Copilot

Pull request overview

Copilot reviewed 34 out of 36 changed files in this pull request and generated 4 comments.

Comments suppressed due to low confidence (1)

src/processor/src/libs/agent_framework/shared_memory_context_provider.py:311

SharedMemoryContextProvider._get_text() doesn’t consider the contents attribute, but agent-framework 1.3.0 Messages often carry the actual text in contents (and your updated middleware/tests reflect that). This can cause memory retrieval queries to become empty/low-signal when .text is unset, reducing recall quality.

        if hasattr(message, "text") and message.text:
            return message.text
        if hasattr(message, "content"):
            return str(message.content) if message.content else ""
        return str(message) if message else ""

…strators"

Revert "fix: enhance message handling and context management in orchestrators"

…content' retry check"

…nd-remove-unused-import Revert "fix: remove unused WorkflowEvent import and restore 'invalid content' retry check"

…ntainability"

Revert "fix: Refactor code structure for improved readability and maintainability"

Copilot

Pull request overview

Copilot reviewed 7 out of 9 changed files in this pull request and generated 1 comment.

Copilot

Pull request overview

Copilot reviewed 7 out of 9 changed files in this pull request and generated 2 comments.

github-actions · 2026-06-10T15:05:23Z

🎉 This PR is included in version 2.1.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Dhanushree-Microsoft and others added 22 commits May 21, 2026 11:44

Refactor code structure for improved readability and maintainability

46b4e82

refactor: remove ImportError handling for agent_framework imports

4e0a59a

refactor: update Message stubs for consistency and clarity in tests

b6001e8

refactor: simplify client import handling in AgentFrameworkHelper

64c78f4

refactor: update exception handling in client creation tests to use I…

7e514c5

…mportError

refactor: normalize executor_id handling in MigrationProcessor for im…

aab94b9

…proved telemetry and error reporting

refactor: streamline WorkflowEvent handling in GroupChatOrchestrator …

913bffa

…and OrchestratorBase

refactor: enhance AzureOpenAIResponseClientWithRetry for legacy param…

f64adcd

…eter support and backward compatibility

code quality issue

0ad41f1

chore: update werkzeug to 3.1.6 and add idna 3.15 to dependencies

26f584c

Merge pull request #267 from microsoft/cm-dm0206

05cc25d

fix: update werkzeug to 3.1.6 and add idna 3.15 to dependencies

fix: assert contents instead of text in input observer test

271fc27

The middleware now uses Message(contents=) so the test must verify the contents field, not text. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

fix: remove unused modified_count variable (F841)

ed3d29c

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

fix: remove unused Agent import (F401)

d7055f2

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

fix: Refactor code structure for improved readability and maintainabi…

3968aed

…lity

Copilot AI review requested due to automatic review settings June 4, 2026 13:26

Prachig-Microsoft requested review from Avijit-Microsoft, Dongbumlee, Roopan-Microsoft, aniaroramsft, nchandhi, sethsteenken and toherman-msft as code owners June 4, 2026 13:26

Roopan-Microsoft had a problem deploying to production June 10, 2026 07:14 — with GitHub Actions Failure

Copilot AI reviewed Jun 10, 2026

View reviewed changes

Roopan-Microsoft added 2 commits June 10, 2026 17:46

Revert "fix: enhance message handling and context management in orche…

7423487

…strators"

Merge pull request #278 from microsoft/revert-277-psl-agentframework

6a3a9fc

Revert "fix: enhance message handling and context management in orchestrators"

Roopan-Microsoft temporarily deployed to production June 10, 2026 12:26 — with GitHub Actions Inactive

Roopan-Microsoft added 2 commits June 10, 2026 17:56

Revert "fix: remove unused WorkflowEvent import and restore 'invalid …

30f360c

…content' retry check"

Merge pull request #279 from microsoft/revert-275-fix/restore-retry-a…

be9d690

…nd-remove-unused-import Revert "fix: remove unused WorkflowEvent import and restore 'invalid content' retry check"

Copilot AI review requested due to automatic review settings June 10, 2026 12:27

Roopan-Microsoft temporarily deployed to production June 10, 2026 12:27 — with GitHub Actions Inactive

Copilot started reviewing on behalf of Roopan-Microsoft June 10, 2026 12:27 View session

Roopan-Microsoft added 2 commits June 10, 2026 17:57

Revert "fix: Refactor code structure for improved readability and mai…

01df24f

…ntainability"

Merge pull request #280 from microsoft/revert-272-psl-cmupgrade

268f922

Revert "fix: Refactor code structure for improved readability and maintainability"

Roopan-Microsoft temporarily deployed to production June 10, 2026 12:28 — with GitHub Actions Inactive

Roopan-Microsoft had a problem deploying to production June 10, 2026 12:30 — with GitHub Actions Failure

Copilot AI reviewed Jun 10, 2026

View reviewed changes

Comment thread src/processor/pyproject.toml

Roopan-Microsoft temporarily deployed to production June 10, 2026 14:06 — with GitHub Actions Inactive

Avijit-Microsoft approved these changes Jun 10, 2026

View reviewed changes

Roopan-Microsoft temporarily deployed to production June 10, 2026 14:08 — with GitHub Actions Inactive

Roopan-Microsoft requested a review from Copilot June 10, 2026 14:10

Copilot started reviewing on behalf of Roopan-Microsoft June 10, 2026 14:10 View session

Copilot AI reviewed Jun 10, 2026

View reviewed changes

Comment thread src/processor/src/tests/unit/libs/application/test_application_context_extras.py

Comment thread src/processor/src/tests/unit/libs/application/test_application_context_extras.py

Roopan-Microsoft temporarily deployed to production June 10, 2026 14:17 — with GitHub Actions Inactive

Roopan-Microsoft merged commit 868bd5d into main Jun 10, 2026
26 of 27 checks passed

github-actions Bot added the released label Jun 10, 2026

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

chore: Dev merge to Main#273

chore: Dev merge to Main#273
Roopan-Microsoft merged 39 commits into
mainfrom
dev

Prachig-Microsoft commented Jun 4, 2026 •

edited by Roopan-Microsoft

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

github-actions Bot commented Jun 10, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

Conversation

Prachig-Microsoft commented Jun 4, 2026 • edited by Roopan-Microsoft Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Purpose

Does this introduce a breaking change?

Golden Path Validation

Deployment Validation

What to Check

Other Information

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

github-actions Bot commented Jun 10, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

Prachig-Microsoft commented Jun 4, 2026 •

edited by Roopan-Microsoft

Loading