From c23e77ce90bd14c1a24eba6bbdd7cdbdaac933d9 Mon Sep 17 00:00:00 2001 From: eavanvalkenburg Date: Mon, 26 Oct 2026 13:54:29 +0100 Subject: [PATCH] Python: Consolidate tooling helper fixes --- python/scripts/check_md_code_blocks.py | 2 +- python/scripts/dependencies/add_dependency_to_project.py | 3 ++- python/scripts/integration_test_report/aggregate.py | 2 +- 3 files changed, 4 insertions(+), 3 deletions(-) diff --git a/python/scripts/check_md_code_blocks.py b/python/scripts/check_md_code_blocks.py index 43d8328376e..cd4714bfa57 100644 --- a/python/scripts/check_md_code_blocks.py +++ b/python/scripts/check_md_code_blocks.py @@ -44,7 +44,7 @@ def expand_file_patterns(patterns: list[str], skip_glob: bool = False) -> list[s # When skip_glob is True, treat patterns as literal file paths # Only include if it's a markdown file if pattern.endswith(".md"): - matches = glob.glob(pattern, recursive=False) + matches = glob.glob(glob.escape(pattern), recursive=False) all_files.extend(matches) else: # Handle both relative and absolute paths with glob expansion diff --git a/python/scripts/dependencies/add_dependency_to_project.py b/python/scripts/dependencies/add_dependency_to_project.py index cb389a89ba9..21516fbec41 100644 --- a/python/scripts/dependencies/add_dependency_to_project.py +++ b/python/scripts/dependencies/add_dependency_to_project.py @@ -90,7 +90,6 @@ def main() -> None: "--package", dest="project", metavar="PACKAGE", - required=True, help="Workspace package selector, such as `core`.", ) # Keep the old long flag as a silent alias while downstream automation @@ -98,6 +97,8 @@ def main() -> None: parser.add_argument("--project", dest="project", help=argparse.SUPPRESS) parser.add_argument("-D", "--dependency", required=True, help="Dependency specifier to add.") args = parser.parse_args() + if args.project is None: + parser.error("the following arguments are required: -P/--package") workspace_root = Path(__file__).resolve().parents[2] package = _resolve_workspace_package(workspace_root, args.project) diff --git a/python/scripts/integration_test_report/aggregate.py b/python/scripts/integration_test_report/aggregate.py index 708f47fcf08..6a6e470b8b6 100644 --- a/python/scripts/integration_test_report/aggregate.py +++ b/python/scripts/integration_test_report/aggregate.py @@ -478,7 +478,7 @@ def _generate_per_test_table( # Sort by module then test name for nodeid in sorted(all_tests, key=lambda n: (all_modules.get(n, ""), n)): module = all_modules.get(nodeid, "") - short = _short_name(nodeid) + short = _short_name(nodeid).replace("|", r"\|") if provider_filter: row = f"| `{short}` | `{module}` |" else: