fix(tools/driver_cli): accept --config globally and improve relative path resolution

- Allow --config before or after subcommand via common parent and top-level
- Resolve config path from CWD first, then repo root, handle ../ paths

Co-authored-by: openhands <openhands@all-hands.dev>

Author: openhands-agent

README.md

@@ -147,6 +147,13 @@ Examples
 - Call a method with kwargs (JSON object):
   python -m benchmesh_service.tools.driver_cli call --id tenmapsu-1 --method set_output true --kwargs '{"ch":1}' --config config.yaml
 
+Other Examples:
+- python -m benchmesh_service.tools.driver_cli call --id tenmapsu-1 --method identify --config config.yaml
+- python -m benchmesh_service.tools.driver_cli call --id tenmapsu-1 --method read_status --config config.yaml
+- python -m benchmesh_service.tools.driver_cli call --id tenmapsu-1 --method set_output --config config.yaml true
+- python -m benchmesh_service.tools.driver_cli call --id spm-1 --method poll_status --config config.yaml
+- ython -m benchmesh_service.tools.driver_cli call --id some-id --method set_voltage 5.0 --kwargs '{"ch":1}' --config config.yaml
+
 Notes
 - Only the targeted device is instantiated to keep testing isolated.
 - Results are printed as JSON when the return type is a collection; otherwise as text.

benchmesh-serial-service/src/benchmesh_service/tools/driver_cli.py

@@ -12,9 +12,18 @@ def _resolve_config_path(config: str | None) -> str:
     if config is None:
         # default to repo-root config.yaml
         return os.path.join(_repo_root(), 'config.yaml')
-    if not os.path.isabs(config):
-        return os.path.join(_repo_root(), config)
-    return config
+    if os.path.isabs(config):
+        return config
+    # Prefer path relative to the current working directory if it exists
+    cwd_candidate = os.path.abspath(config)
+    if os.path.exists(cwd_candidate):
+        return cwd_candidate
+    # Fallback to repo-root relative path
+    repo_candidate = os.path.join(_repo_root(), config)
+    if os.path.exists(repo_candidate):
+        return repo_candidate
+    # As last resort, return the CWD-resolved path
+    return cwd_candidate
 
 
 def _load_devices_from_config(config_path: str) -> List[Dict[str, Any]]:
@@ -163,18 +172,23 @@ def cmd_methods(args: argparse.Namespace) -> int:
 
 
 def build_parser() -> argparse.ArgumentParser:
+    # Define common options so they can appear before or after subcommands
+    common = argparse.ArgumentParser(add_help=False)
+    common.add_argument('--config', help='Path to config.yaml (default: repo root config.yaml)')
+
     p = argparse.ArgumentParser(description='BenchMesh driver manual test tool')
+    # Also allow --config before the subcommand
     p.add_argument('--config', help='Path to config.yaml (default: repo root config.yaml)')
     sub = p.add_subparsers(dest='cmd', required=True)
 
-    p_list = sub.add_parser('list', help='List devices from config')
+    p_list = sub.add_parser('list', parents=[common], help='List devices from config')
     p_list.set_defaults(func=cmd_list)
 
-    p_methods = sub.add_parser('methods', help='List public methods of a device driver')
+    p_methods = sub.add_parser('methods', parents=[common], help='List public methods of a device driver')
     p_methods.add_argument('--id', required=True, help='Device id (from config)')
     p_methods.set_defaults(func=cmd_methods)
 
-    p_call = sub.add_parser('call', help='Call a method on a device driver')
+    p_call = sub.add_parser('call', parents=[common], help='Call a method on a device driver')
     p_call.add_argument('--id', required=True, help='Device id (from config)')
     p_call.add_argument('--method', required=True, help='Method name to call')
     p_call.add_argument('args', nargs='*', help='Positional args (auto-coerced)')