[leilei3167]

Motivation & Context

On Windows, get_changed_packages() in python/scripts/run_tasks_in_changed_packages.py only strips a python/ prefix. A repository-relative --files path that uses backslashes, such as python\packages\core\agent_framework\_types.py, never matches that prefix. The remainder is joined onto workspace_root, which already ends in python, so the path points at python/python/.... The changed-package set comes back empty and the script skips the tasks.

Forward-slash repository-relative paths already select the right package. Native Windows spellings of those same paths should too.

Description & Review Guide

• What are the major changes? Normalize \ to / before the existing python/ prefix strip.
• What is the impact of these changes? A repository-relative backslash path now selects the same package as the equivalent forward-slash path. Workspace-relative and absolute paths keep their current behavior. Paths outside any package still produce an empty set, and the script still skips that run.
• What do you want reviewers to focus on? The prefix is removed only after separators are normalized, so a native repo-relative path is not joined a second time under python/.

Related Issue

Fixes #9742

No other open pull request changes this script. This is a small separator correction; the expected routing is the behavior described in the issue.

AI Assistance

• No material AI assistance was used.
• This is an AI-assisted contribution. I reviewed, understood, and verified all submitted content and accept responsibility for it.

AI assistance details: Implementation, regression test, and this description.

Contribution Checklist

• The code builds clean without any errors or warnings
• All unit tests pass, and I have added new tests where possible
• The PR follows the Contribution Guidelines
• This PR links to an agreed issue with no competing open PR, or the Related Issue section documents a trivial-change or repository-automation exception.
• This is not a breaking change. If it is a breaking change, add the breaking change label (or add "[BREAKING]" to the title prefix, before or after any language prefix) — a workflow keeps the label and title prefix in sync automatically.

[leilei3167]

The separator normalization is in 8dc4ac0. A repository-relative backslash path now selects the same package as the forward-slash form.

[Copilot]

🟡 Changes recommended

The regression suite needs CI integration, and its global Rich stub can disrupt later imports.

2 open issues

• Avoid globally stubbing an available Rich module · New
• Add regression tests to CI discovery · New

What changed in this PR

This PR addresses #9742 by normalizing Windows path separators before package routing, so native repository-relative paths select the intended Python packages.

Changes:

• Normalize backslashes before stripping the python/ prefix.
• Add regression tests for repository-relative, workspace-relative, absolute, and unrelated paths.

File	Description
python/​scripts/​tests/​test_run_tasks_in_changed_packages.py	Adds path-routing and core-change detection tests.
python/​scripts/​run_tasks_in_changed_packages.py	Normalizes separators before removing the repository prefix.

_{Review effort: Balanced}

[leilei3167]

The script tests now import Rich from the tool-runtime group, and the existing Windows/Linux matrix runs scripts/tests. 7c5452e

[Eduard van Valkenburg (eavanvalkenburg)]

leilei3167, thanks for fixing the Windows changed-package routing gap. We’re going to continue with #9775 because its native-path approach also covers dot-prefixed repository paths and UNC paths while preserving literal backslashes in POSIX filenames. I’m closing this competing implementation so #9742 has one active PR.