Skip to content

Conversation

@NirajC-Microsoft
Copy link
Contributor

@NirajC-Microsoft NirajC-Microsoft commented Oct 7, 2025

Purpose

This pull request updates import paths throughout the codebase to use fully qualified module paths, reflecting a new project structure. These changes improve code organization and make module dependencies clearer and more maintainable.

Test imports modernization:

  • Updated all test files in src/backend/tests/auth, src/backend/tests/middleware, src/backend/tests/models, and src/backend/tests to use fully qualified imports from src.backend modules, ensuring consistency with the new project structure. [1] [2] [3] [4] [5] [6] [7]

Agent and integration test imports:

  • Changed imports in src/backend/v3/magentic_agents and related integration tests to use the new src.backend.v3.magentic_agents paths for agent classes and configuration models. [1] [2] [3]

MCP server test imports:

  • Updated MCP server test files to import from src.mcp_server for core factory, services, and utility modules, aligning with the reorganized server package structure. [1] [2] [3]
  • ...

Does this introduce a breaking change?

  • Yes
  • No

How to Test

  • Get the code
git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install
  • Test the code

What to Check

Verify that the following are valid

  • ...

Other Information

@NirajC-Microsoft NirajC-Microsoft changed the title fixed for wrong import in test files fix: fixed for wrong import in test files Oct 7, 2025
Removed 'dev-v3' branch from workflow triggers.
Updated import statement to use a relative path.
Updated Field definitions to include Literal=True for data_type attributes in multiple classes. Removed ActionRequest and HumanFeedback classes.
Removed 'dev-v3' branch from push and pull request triggers.
@Roopan-Microsoft Roopan-Microsoft merged commit 1540324 into dev-v3 Oct 7, 2025
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants