From fbcab3064ce824589e055de35b9e7e6319416ce9 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=E6=80=BB=E5=B7=A5?= Date: Wed, 16 Sep 2026 12:00:55 +0800 Subject: [PATCH] fix(HEL-560): verify tools provide a non-mutating --help path Co-authored-by: multica-agent --- tools/verify_datahub_console.py | 3 +++ tools/verify_datahub_console_ui.py | 7 +++++-- 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/tools/verify_datahub_console.py b/tools/verify_datahub_console.py index 6602848..4a079e1 100644 --- a/tools/verify_datahub_console.py +++ b/tools/verify_datahub_console.py @@ -150,6 +150,9 @@ def start_hub(workdir: Path, port: int, review_port: int) -> None: def main() -> int: + if any(argument in {"-h", "--help"} for argument in sys.argv[1:]): + print(__doc__.strip()) + return 0 workdir = Path(tempfile.mkdtemp(prefix="hel560-smoke-")) review_port, hub_port = free_port(), free_port() review = f"http://127.0.0.1:{review_port}" diff --git a/tools/verify_datahub_console_ui.py b/tools/verify_datahub_console_ui.py index ba7d754..82e9621 100644 --- a/tools/verify_datahub_console_ui.py +++ b/tools/verify_datahub_console_ui.py @@ -11,8 +11,6 @@ import os import sys from pathlib import Path -from playwright.sync_api import sync_playwright - import verify_datahub_console as backend ROOT = Path(__file__).resolve().parents[1] @@ -42,6 +40,11 @@ def check(label: str, ok: bool, detail: str = "") -> None: def main() -> int: + if any(argument in {"-h", "--help"} for argument in sys.argv[1:]): + print(__doc__.strip()) + return 0 + from playwright.sync_api import sync_playwright + shots = Path(sys.argv[sys.argv.index("--shots") + 1]) if "--shots" in sys.argv else None if shots: shots.mkdir(parents=True, exist_ok=True)