orbi-cli 0.6.1__tar.gz → 0.6.2__tar.gz

{orbi_cli-0.6.1/src/orbi_cli.egg-info → orbi_cli-0.6.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: orbi-cli
-Version: 0.6.1
+Version: 0.6.2
 Summary: Orbi — local AI development worker: claims GitHub Issues, runs Pi in isolated worktrees, and delivers PRs (GitHub Issues + labels are the only state store).
 Author: xqliu
 License-Expression: AGPL-3.0-only OR LicenseRef-Sustainable-Use-1.0

{orbi_cli-0.6.1 → orbi_cli-0.6.2}/pyproject.toml

@@ -44,7 +44,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "orbi-cli"
-version = "0.6.1"
+version = "0.6.2"
 description = "Orbi — local AI development worker: claims GitHub Issues, runs Pi in isolated worktrees, and delivers PRs (GitHub Issues + labels are the only state store)."
 readme = "README.md"
 requires-python = ">=3.14"

{orbi_cli-0.6.1 → orbi_cli-0.6.2}/src/orbi/__init__.py

@@ -18,4 +18,4 @@ the package would shadow the installed package for every process with
 the checkout root on sys.path. The direct-execution compatibility entry
 is `python3 -m orbi.cli` (development path only).
 """
-__version__ = "0.6.1"
+__version__ = "0.6.2"

{orbi_cli-0.6.1 → orbi_cli-0.6.2}/src/orbi/pi_session.py

@@ -857,7 +857,7 @@ def run_ticket_agent(
             config, ROLE_TICKET, IMPLEMENT_EXCLUDED_SKILLS, session_dir,
             system_prompt, context,
             context_placeholder="<issue-context-redacted>",
-            tools=False, extensions=False,
+            tools=False, extensions=True,
         )
         # Startup phase: the provider config is loaded and resolved
         # before the spawn; a configured provider file is materialized

{orbi_cli-0.6.1 → orbi_cli-0.6.2/src/orbi_cli.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: orbi-cli
-Version: 0.6.1
+Version: 0.6.2
 Summary: Orbi — local AI development worker: claims GitHub Issues, runs Pi in isolated worktrees, and delivers PRs (GitHub Issues + labels are the only state store).
 Author: xqliu
 License-Expression: AGPL-3.0-only OR LicenseRef-Sustainable-Use-1.0

{orbi_cli-0.6.1 → orbi_cli-0.6.2}/tests/test_pi_command.py

@@ -63,17 +63,18 @@ def test_ticket_command_and_log_with_everything_configured():
         pi_command.IMPLEMENT_EXCLUDED_SKILLS, SESSION_DIR,
         SYSTEM_PROMPT, CONTEXT,
         context_placeholder="<issue-context-redacted>",
-        tools=False, extensions=False,
+        tools=False, extensions=True,
     )
     assert command == [
-        "pi", "--no-tools", *IMPLEMENT_SKILL_ARGS,
+        "pi", "--no-tools", *EXTENSION_ARGS, *IMPLEMENT_SKILL_ARGS,
         "--provider", "openai", "--model", "gpt-5.6-sol",
         "--thinking", "medium",
         "--print", "--session-dir", str(SESSION_DIR),
         "--system-prompt", SYSTEM_PROMPT, CONTEXT,
     ]
     assert log_command == [
-        "pi", "--provider", "openai", "--model", "gpt-5.6-sol",
+        "pi", *EXTENSION_ARGS,
+        "--provider", "openai", "--model", "gpt-5.6-sol",
         "--thinking", "medium",
         "--print", "--session-dir", str(SESSION_DIR),
         "--system-prompt", "<redacted>", "<issue-context-redacted>",
@@ -131,7 +132,7 @@ def test_review_command_and_log_with_everything_configured():
      "placeholder", "expect_no_tools"),
     [
         (pi_command.ROLE_TICKET, pi_command.IMPLEMENT_EXCLUDED_SKILLS,
-         False, False, "<issue-context-redacted>", True),
+         False, True, "<issue-context-redacted>", True),
         (ROLE_IMPLEMENT, pi_command.IMPLEMENT_EXCLUDED_SKILLS,
          True, True, "<issue-context-redacted>", False),
         (pi_command.ROLE_REVIEW, pi_command.REVIEW_EXCLUDED_SKILLS,

{orbi_cli-0.6.1 → orbi_cli-0.6.2}/tests/test_pi_extensions.py

@@ -76,16 +76,35 @@ def test_pi_extension_args_and_env_isolate_disabled_and_secrets():
     assert "secret" not in " ".join(args)
 
 
-def test_run_pi_and_review_share_extension_contract(monkeypatch, tmp_path):
+def test_run_pi_review_and_ticket_share_extension_contract(monkeypatch, tmp_path):
     (tmp_path / "prompt.md").write_text("system", encoding="utf-8")
     (tmp_path / "prompt_review.md").write_text("review", encoding="utf-8")
+    disabled = tmp_path / "disabled.mjs"
+    disabled.write_text("export default () => {}", encoding="utf-8")
     calls = []
     monkeypatch.setattr(pi_session, "stream_pi", lambda command, **kw: calls.append((command, kw)) or "ok")
-    config = config_domain.RunnerConfig(prompt=tmp_path / "prompt.md", prompt_review=tmp_path / "prompt_review.md", repo_dir=tmp_path, source_repos=("owner/repo",), workspace_root=tmp_path, context_files=(), skills=(), base_branch="main", base_sha="abc", run_id="deadbeef", pi_extensions=({"source": "npm:fixture@1.2.3", "enabled": True, "env": {"FIXTURE_TOKEN": "secret"}},))
+    config = config_domain.RunnerConfig(prompt=tmp_path / "prompt.md", prompt_review=tmp_path / "prompt_review.md", repo_dir=tmp_path, source_repos=("owner/repo",), workspace_root=tmp_path, context_files=(), skills=(), base_branch="main", base_sha="abc", run_id="deadbeef", pi_extensions=({"source": "npm:fixture@1.2.3", "enabled": True, "env": {"FIXTURE_TOKEN": "secret"}}, {"source": str(disabled), "enabled": False, "env": {}}))
+    extension_args = pi_command._pi_extension_args(config)
+    assert extension_args == ["--no-extensions", "--extension", "npm:fixture@1.2.3"]
     pi_session.run_pi({"number": 1, "title": "t", "body": ""}, RunContext(run_id=config.run_id, issue={"number": 1, "title": "t", "body": ""}["number"], branch="b", worktree=tmp_path, source_repo="owner/repo"), config)
     pi_session.run_review(RunContext(run_id=config.run_id, issue=1, branch="b", worktree=tmp_path, source_repo="owner/repo"), {"number": 1, "url": "u", "base_oid": "b", "head_oid": "h", "head_ref": "r"}, config, 1)
     for command, kwargs in calls:
-        assert command[1:4] == ["--no-extensions", "--extension", "npm:fixture@1.2.3"]
+        assert command[1:4] == extension_args
         assert kwargs["pi_env"] == {"FIXTURE_TOKEN": "secret"}
         assert "secret" not in " ".join(kwargs["log_command"])
         assert kwargs["log_command"][1:4] == command[1:4]
+    assert len(calls) == 2
+    # The ticket role (#1558) keeps the --no-tools boundary and passes the
+    # SAME isolated extension flags implement/review use.
+    pi_session.run_ticket_agent(
+        {"number": 1558, "title": "t", "body": ""}, config, "owner/repo",
+    )
+    command, kwargs = calls[2]
+    assert command[:2 + len(extension_args)] == [
+        "pi", "--no-tools", *extension_args,
+    ]
+    assert kwargs["log_command"][:1 + len(extension_args)] == [
+        "pi", *extension_args,
+    ]
+    assert kwargs["pi_env"] == {"FIXTURE_TOKEN": "secret"}
+    assert "secret" not in " ".join(kwargs["log_command"])