fix(cli): correct forwarded flag name to --dangerously-skip-permissions
Claude CLI uses --dangerously-skip-permissions, not --dangerously-allow-browser-control. Update option name, help text, and all test references.
This commit is contained in:
+5
-5
@@ -44,16 +44,16 @@ def _resolve_chat_cwd() -> Path:
|
|||||||
help="Use specified path as working directory (default: workspace/).",
|
help="Use specified path as working directory (default: workspace/).",
|
||||||
)
|
)
|
||||||
@click.option(
|
@click.option(
|
||||||
"--dangerously-allow-browser-control",
|
"--dangerously-skip-permissions",
|
||||||
is_flag=True,
|
is_flag=True,
|
||||||
default=False,
|
default=False,
|
||||||
help="Forward to claude: allow browser automation without confirmation.",
|
help="Forward to claude: bypass all permission checks.",
|
||||||
)
|
)
|
||||||
@click.pass_context
|
@click.pass_context
|
||||||
def cli(
|
def cli(
|
||||||
ctx: click.Context,
|
ctx: click.Context,
|
||||||
cwd: str | None,
|
cwd: str | None,
|
||||||
dangerously_allow_browser_control: bool,
|
dangerously_skip_permissions: bool,
|
||||||
) -> None:
|
) -> None:
|
||||||
"""Myclaude: CLI toolkit.
|
"""Myclaude: CLI toolkit.
|
||||||
|
|
||||||
@@ -80,8 +80,8 @@ def cli(
|
|||||||
raise SystemExit(1)
|
raise SystemExit(1)
|
||||||
|
|
||||||
cmd: list[str] = [binary]
|
cmd: list[str] = [binary]
|
||||||
if dangerously_allow_browser_control:
|
if dangerously_skip_permissions:
|
||||||
cmd.append("--dangerously-allow-browser-control")
|
cmd.append("--dangerously-skip-permissions")
|
||||||
cmd.extend(ctx.args)
|
cmd.extend(ctx.args)
|
||||||
|
|
||||||
proc = subprocess.run(cmd, cwd=str(chat_cwd), check=False)
|
proc = subprocess.run(cmd, cwd=str(chat_cwd), check=False)
|
||||||
|
|||||||
+6
-6
@@ -12,12 +12,12 @@ class TestCliHelp:
|
|||||||
"""Tests for CLI help and basic invocation."""
|
"""Tests for CLI help and basic invocation."""
|
||||||
|
|
||||||
def test_help_shows_options(self) -> None:
|
def test_help_shows_options(self) -> None:
|
||||||
"""--help should show cwd and browser control options."""
|
"""--help should show cwd and skip permissions options."""
|
||||||
runner = CliRunner()
|
runner = CliRunner()
|
||||||
result = runner.invoke(cli, ["--help"])
|
result = runner.invoke(cli, ["--help"])
|
||||||
assert result.exit_code == 0
|
assert result.exit_code == 0
|
||||||
assert "--cwd" in result.output
|
assert "--cwd" in result.output
|
||||||
assert "--dangerously-allow-browser-control" in result.output
|
assert "--dangerously-skip-permissions" in result.output
|
||||||
|
|
||||||
def test_version_shows_version(self) -> None:
|
def test_version_shows_version(self) -> None:
|
||||||
"""--version should show package version."""
|
"""--version should show package version."""
|
||||||
@@ -83,8 +83,8 @@ class TestCliDefaultBehavior:
|
|||||||
call_kwargs = mock_run.call_args.kwargs
|
call_kwargs = mock_run.call_args.kwargs
|
||||||
assert call_kwargs.get("cwd") == str(test_dir.resolve())
|
assert call_kwargs.get("cwd") == str(test_dir.resolve())
|
||||||
|
|
||||||
def test_dangerously_allow_browser_control_flag(self) -> None:
|
def test_dangerously_skip_permissions_flag(self) -> None:
|
||||||
"""--dangerously-allow-browser-control should be forwarded."""
|
"""--dangerously-skip-permissions should be forwarded."""
|
||||||
runner = CliRunner()
|
runner = CliRunner()
|
||||||
|
|
||||||
with (
|
with (
|
||||||
@@ -92,8 +92,8 @@ class TestCliDefaultBehavior:
|
|||||||
patch("myclaude.cli.subprocess.run") as mock_run,
|
patch("myclaude.cli.subprocess.run") as mock_run,
|
||||||
):
|
):
|
||||||
mock_run.return_value = MagicMock(returncode=0)
|
mock_run.return_value = MagicMock(returncode=0)
|
||||||
result = runner.invoke(cli, ["--dangerously-allow-browser-control"])
|
result = runner.invoke(cli, ["--dangerously-skip-permissions"])
|
||||||
assert result.exit_code == 0
|
assert result.exit_code == 0
|
||||||
call_args = mock_run.call_args[0][0]
|
call_args = mock_run.call_args[0][0]
|
||||||
assert "--dangerously-allow-browser-control" in call_args
|
assert "--dangerously-skip-permissions" in call_args
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user