agex-cli 0.23.0__tar.gz → 0.24.0__tar.gz

{agex_cli-0.23.0 → agex_cli-0.24.0}/CHANGELOG.md

@@ -7,6 +7,17 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.24.0] - 2026-05-28
+
+### Added
+
+- `agex pr review [<PR>]` verb — posts the Qodo agentic-review trigger (`/agentic_review`) on demand; re-triggers a review on an existing PR or triggers a draft once ready.
+- `agex pr open` now auto-posts `/agentic_review` out of the box for a freshly created, non-draft PR (skipped for drafts and already-open PRs) so the review trigger is never hand-typed (#60).
+
+### Changed
+
+- Centralized the Qodo review-trigger command string in one place (`commands/pr/scripts/review.QODO_REVIEW_TRIGGER`). agex emits the current `/agentic_review`; the legacy `/improve` command is deprecated by Qodo and is never emitted by agex (#60).
+
 ## [0.23.0] - 2026-05-28
 
 ### Added

{agex_cli-0.23.0 → agex_cli-0.24.0}/CLAUDE.md

@@ -19,7 +19,7 @@ Read the spec before any non-trivial change — the design invariants below are
 1. **Zero LLM calls inside agex.** All output is deterministic markdown from Jinja templates + Python.
 2. **Markdown is the only output format.** No `--json` flag.
 3. **`--agent <backend>` is required** on backend-sensitive commands. The CLI never auto-detects.
-4. **Side effects only in** `gamify`, `gamify --uninstall`, `hook write`, `pr open`, `pr reply`, `pr read` (journal writes), and first-run `.agex/` init. Everything else is read-only. The `agex pr` namespace allows scoped network I/O (via `gh`) and bounded `--wait` sleep — a deliberate carve-out from the no-network/no-sleep invariants.
+4. **Side effects only in** `gamify`, `gamify --uninstall`, `hook write`, `pr open`, `pr reply`, `pr review`, `pr read` (journal writes), and first-run `.agex/` init. Everything else is read-only. The `agex pr` namespace allows scoped network I/O (via `gh`) and bounded `--wait` sleep — a deliberate carve-out from the no-network/no-sleep invariants.
 5. **"Unsupported" is success** — exit 0 with a markdown notice that links to the issue tracker, not a non-zero exit.
 6. **Skills are authored by the agent, not shipped by agex.** `agex learn <topic>` teaches; `agex explain <topic>` describes; agex never writes a user skill file on the agent's behalf in v0.1.
 
@@ -43,7 +43,7 @@ cli.py ──► commands/<name>/scripts/<name>.py ──► core/render.py
 
 ## `agex pr` namespace (v0.17.0+)
 
-`lint`, `open`, `read`, `reply`, `delta`. Each command ends with a deterministic "Next step:" footer. The `pr` namespace allows scoped network I/O (via `gh`) and bounded `--wait` sleep — a deliberate carve-out from the no-network/no-sleep invariants. Key modules:
+`lint`, `open`, `read`, `reply`, `review`, `await`, `delta`. Each command ends with a deterministic "Next step:" footer. The `pr` namespace allows scoped network I/O (via `gh`) and bounded `--wait` sleep — a deliberate carve-out from the no-network/no-sleep invariants. `pr open` (non-draft, new PR) and `pr review` post the Qodo `/agentic_review` trigger comment; the legacy `/improve` is deprecated and never emitted. The trigger string lives in one place: `commands/pr/scripts/review.QODO_REVIEW_TRIGGER`. Key modules:
 
 - `core/github.py` — thin `gh` shellout wrapper; future zero-trust httpx swap touches only this file.
 - `core/journal.py` — nested-stream JSONL append/load for `.agex/data/<dir>/<stream>.jsonl`.

{agex_cli-0.23.0 → agex_cli-0.24.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: agex-cli
-Version: 0.23.0
+Version: 0.24.0
 Summary: Agent-operated developer-experience CLI — deterministic per-backend markdown briefings for autonomous agents.
 Project-URL: Homepage, https://culture.dev/agex/
 Project-URL: Repository, https://github.com/agentculture/devex

{agex_cli-0.23.0 → agex_cli-0.24.0}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "agex-cli"
-version = "0.23.0"
+version = "0.24.0"
 description = "Agent-operated developer-experience CLI — deterministic per-backend markdown briefings for autonomous agents."
 authors = [{name = "Ori Nachum"}]
 license = {text = "MIT"}

{agex_cli-0.23.0 → agex_cli-0.24.0}/src/agent_experience/cli.py

@@ -27,6 +27,7 @@ from agent_experience.commands.pr.scripts import lint as pr_lint_script
 from agent_experience.commands.pr.scripts import open_ as pr_open_script
 from agent_experience.commands.pr.scripts import read as pr_read_script
 from agent_experience.commands.pr.scripts import reply as pr_reply_script
+from agent_experience.commands.pr.scripts import review as pr_review_script
 from agent_experience.core.backend import parse_backend
 from agent_experience.core.prog import prog_name
 
@@ -237,6 +238,22 @@ def _cmd_pr_await(args: argparse.Namespace) -> int:
     return exit_code
 
 
+def _cmd_pr_review(args: argparse.Namespace) -> int:
+    try:
+        stdout, exit_code, stderr = pr_review_script.run(
+            agent=args.agent, project_dir=Path.cwd(), pr=args.pr
+        )
+    except ValueError as exc:
+        print(f"{prog_name()}: {exc}", file=sys.stderr)
+        return 2
+    except RuntimeError as exc:
+        print(str(exc), file=sys.stderr)
+        print(_gh_rerun_hint(), file=sys.stderr)
+        return 1
+    _emit(stdout, stderr)
+    return exit_code
+
+
 def _cmd_pr_delta(args: argparse.Namespace) -> int:
     try:
         stdout, exit_code, stderr = pr_delta_script.run(agent=args.agent, project_dir=Path.cwd())
@@ -403,6 +420,14 @@ def _register_pr(sub: argparse._SubParsersAction) -> None:
     _add_agent_option(p_await, required=False, help_text=_AGENT_HELP)
     p_await.set_defaults(func=_cmd_pr_await)
 
+    p_review = pr_sub.add_parser(
+        "review",
+        help="Post the Qodo agentic-review trigger (/agentic_review) on a PR.",
+    )
+    p_review.add_argument("pr", type=int, nargs="?", default=None)
+    _add_agent_option(p_review, required=False, help_text=_AGENT_HELP)
+    p_review.set_defaults(func=_cmd_pr_review)
+
     p_delta = pr_sub.add_parser("delta", help="Show the delta since the last PR read.")
     _add_agent_option(p_delta, required=False, help_text=_AGENT_HELP)
     p_delta.set_defaults(func=_cmd_pr_delta)

{agex_cli-0.23.0 → agex_cli-0.24.0}/src/agent_experience/commands/pr/SKILL.md

@@ -1,12 +1,12 @@
 ---
 name: pr
-description: GitHub PR lifecycle commands for agents — lint, open, read, reply, await, delta. Each command ends with a deterministic "Next step:" footer so the agent never has to guess what to chain.
+description: GitHub PR lifecycle commands for agents — lint, open, read, reply, review, await, delta. Each command ends with a deterministic "Next step:" footer so the agent never has to guess what to chain.
 type: command
 ---
 
 # `agex pr` — PR lifecycle for agents
 
-Six verbs, in roughly the order an agent uses them on a PR:
+Seven verbs, in roughly the order an agent uses them on a PR:
 
 | Verb | Purpose |
 |---|---|
@@ -14,6 +14,7 @@ Six verbs, in roughly the order an agent uses them on a PR:
 | `agex pr open --title T [--body-file F] [--draft] [--delayed-read]` | `gh pr create` with auto-signed body; with `--delayed-read` chains to `read --wait 180`. |
 | `agex pr read [<PR>] [--wait SECS]` | Unified briefing: CI checks, SonarCloud quality gate, all comments, reviewer readiness. With `--wait`, polls until required reviewers are ready or timeout. Always exits 0. |
 | `agex pr reply <PR>` | Read JSONL replies on stdin, post each, resolve threads. |
+| `agex pr review [<PR>]` | Post the Qodo agentic-review trigger (`/agentic_review`) on a PR. Re-triggers on an already-open PR; `pr open` posts it automatically for a new non-draft PR. |
 | `agex pr await [<PR>] [--max-wait SECS]` | "Wake me when this is triage-able" — polls readiness, runs CI + Sonar gate, dumps briefing. **Exits 1 on quality-gate `ERROR`, unresolved threads, or failing CI checks**, 0 on clean state or timeout. Default `--max-wait 1800` (30 min). |
 | `agex pr delta` | Dump sibling-project `CLAUDE.md` heads + `culture.yaml` for alignment review. |
 
@@ -40,6 +41,24 @@ PR health (e.g., in scripts that should fail if Sonar errors).
   gates on top of the same readiness loop and **exits non-zero** when the PR
   isn't clean.
 
+## Qodo agentic review
+
+When a PR is opened, Qodo runs an agentic code review on demand when it sees a
+trigger comment. The current command is **`/agentic_review`**. The legacy
+`/improve` command is **deprecated** (Qodo replies with a deprecation banner and
+no committed removal date), so agex never posts `/improve`.
+
+- `agex pr open` posts `/agentic_review` **automatically** for a freshly created,
+  non-draft PR — out of the box, no flag. Drafts are skipped (not review-ready)
+  and already-open PRs are skipped (idempotent re-opens shouldn't spam the
+  thread).
+- `agex pr review [<PR>]` posts the same trigger on demand — use it to re-trigger
+  a review on an existing PR (e.g. after pushing fixes) or to trigger a draft
+  once it's ready.
+
+The command string lives in one place (`scripts/review.QODO_REVIEW_TRIGGER`), so
+a future Qodo rename is a one-line change for every consumer of `agex pr`.
+
 ## SonarCloud project key
 
 `pr read` and `pr await` query the SonarCloud quality gate for the current
@@ -57,7 +76,9 @@ Every command ends with a `**Next step:**` footer — chase the chain without gu
 ## Side effects
 
 Network: every command except `lint` and `delta` talks to GitHub via `gh`.
-Disk: `pr open`, `pr read`, and `pr reply` append events to
+`pr open` (non-draft, new PR) and `pr review` post the `/agentic_review` trigger
+comment.
+Disk: `pr open`, `pr read`, `pr reply`, and `pr review` append events to
 `.agex/data/pr/events.jsonl`.
 
 ## Prerequisites

{agex_cli-0.23.0 → agex_cli-0.24.0}/src/agent_experience/commands/pr/assets/backends/acp.yaml

@@ -12,6 +12,7 @@ hints:
   reply_with_failures: "Resubmit the failed lines (see table above) to `{{ prog }} pr reply {{ pr }}`."
   reply_clean: "Push fixes (if any), then `{{ prog }} pr read {{ pr }} --wait 180`."
   delta_done: "Triage each sibling — open follow-up PRs where alignment drifted; mention them in your reply."
+  review_posted: "Posted `/agentic_review` — `{{ prog }} pr read {{ pr }} --wait 180` once Qodo responds."
   await_clean: "PR #{{ pr }} is ready — wait for human merge."
   await_gate_error: "SonarCloud quality gate is ERROR — fix the new issues above, push, then `{{ prog }} pr await {{ pr }}` again."
   await_gate_unknown: "SonarCloud quality gate is UNKNOWN — analysis is pending or the project isn't analyzed yet; wait and re-run `{{ prog }} pr await {{ pr }}`."

{agex_cli-0.23.0 → agex_cli-0.24.0}/src/agent_experience/commands/pr/assets/backends/claude-code.yaml

@@ -12,6 +12,7 @@ hints:
   reply_with_failures: "Resubmit the failed lines (see table above) to `{{ prog }} pr reply {{ pr }}`."
   reply_clean: "Push fixes (if any), then `{{ prog }} pr read {{ pr }} --wait 180`."
   delta_done: "Triage each sibling — open follow-up PRs where alignment drifted; mention them in your reply."
+  review_posted: "Posted `/agentic_review` — `{{ prog }} pr read {{ pr }} --wait 180` once Qodo responds."
   await_clean: "PR #{{ pr }} is ready — wait for human merge."
   await_gate_error: "SonarCloud quality gate is ERROR — fix the new issues above, push, then `{{ prog }} pr await {{ pr }}` again."
   await_gate_unknown: "SonarCloud quality gate is UNKNOWN — analysis is pending or the project isn't analyzed yet; wait and re-run `{{ prog }} pr await {{ pr }}`."

{agex_cli-0.23.0 → agex_cli-0.24.0}/src/agent_experience/commands/pr/assets/backends/codex.yaml

@@ -12,6 +12,7 @@ hints:
   reply_with_failures: "Resubmit the failed lines (see table above) to `{{ prog }} pr reply {{ pr }}`."
   reply_clean: "Push fixes (if any), then `{{ prog }} pr read {{ pr }} --wait 180`."
   delta_done: "Triage each sibling — open follow-up PRs where alignment drifted; mention them in your reply."
+  review_posted: "Posted `/agentic_review` — `{{ prog }} pr read {{ pr }} --wait 180` once Qodo responds."
   await_clean: "PR #{{ pr }} is ready — wait for human merge."
   await_gate_error: "SonarCloud quality gate is ERROR — fix the new issues above, push, then `{{ prog }} pr await {{ pr }}` again."
   await_gate_unknown: "SonarCloud quality gate is UNKNOWN — analysis is pending or the project isn't analyzed yet; wait and re-run `{{ prog }} pr await {{ pr }}`."

{agex_cli-0.23.0 → agex_cli-0.24.0}/src/agent_experience/commands/pr/assets/backends/copilot.yaml

@@ -12,6 +12,7 @@ hints:
   reply_with_failures: "Resubmit the failed lines (see table above) to `{{ prog }} pr reply {{ pr }}`."
   reply_clean: "Push fixes (if any), then `{{ prog }} pr read {{ pr }} --wait 180`."
   delta_done: "Triage each sibling — open follow-up PRs where alignment drifted; mention them in your reply."
+  review_posted: "Posted `/agentic_review` — `{{ prog }} pr read {{ pr }} --wait 180` once Qodo responds."
   await_clean: "PR #{{ pr }} is ready — wait for human merge."
   await_gate_error: "SonarCloud quality gate is ERROR — fix the new issues above, push, then `{{ prog }} pr await {{ pr }}` again."
   await_gate_unknown: "SonarCloud quality gate is UNKNOWN — analysis is pending or the project isn't analyzed yet; wait and re-run `{{ prog }} pr await {{ pr }}`."

{agex_cli-0.23.0 → agex_cli-0.24.0}/src/agent_experience/commands/pr/assets/rules/next_step_rules.py

@@ -70,5 +70,9 @@ def reply_next_step(pr: int, failure_count: int) -> tuple[str, dict[str, Any]]:
     return "reply_clean", {"pr": pr}
 
 
+def review_next_step(pr: int) -> tuple[str, dict[str, Any]]:
+    return "review_posted", {"pr": pr}
+
+
 def delta_next_step() -> tuple[str, dict[str, Any]]:
     return "delta_done", {}

{agex_cli-0.23.0 → agex_cli-0.24.0}/src/agent_experience/commands/pr/assets/templates/pr_open_result.md.j2

@@ -9,5 +9,9 @@ PR opened: **#{{ pr }}**{% if url %} ({{ url }}){% endif %}.
 - Title: `{{ title }}`
 - Signed: {{ "yes" if signed else "no" }}
 - Draft: {{ "yes" if draft else "no" }}
-
+{% if review_posted %}- Posted `{{ review_command }}` to start a Qodo agentic review.
+{% endif %}
+{%- if review_failed %}
+> ⚠️ PR created, but posting `{{ review_command }}` failed (network/`gh`). The PR is open — run `{{ prog }} pr review {{ pr }}` to post the Qodo trigger once reachable.
+{% endif %}
 {{ footer }}

agex_cli-0.24.0/src/agent_experience/commands/pr/assets/templates/pr_review_result.md.j2

@@ -0,0 +1,5 @@
+# `{{ prog }} pr review`
+
+Posted `{{ command }}` on PR **#{{ pr }}** to start a Qodo agentic review.
+
+{{ footer }}

{agex_cli-0.23.0 → agex_cli-0.24.0}/src/agent_experience/commands/pr/scripts/await_.py

@@ -29,15 +29,6 @@ from agent_experience.core.render import render_string
 _TEMPLATES_PKG = "agent_experience.commands.pr.assets.templates"
 
 
-def _resolve_pr(pr: int | None) -> int:
-    if pr is not None:
-        return pr
-    view = github.pr_view(None)
-    if view is None:
-        raise ValueError("no PR found for current branch; pass <PR> explicitly")
-    return int(view["number"])
-
-
 def _gate_status(gate: Mapping[str, Any] | None) -> str | None:
     """Normalized SonarCloud quality-gate status, or None when there's no gate.
 
@@ -57,7 +48,7 @@ def run(
     max_wait: int,
 ) -> tuple[str, int, str]:
     backend = resolve_backend(agent, project_dir)
-    pr_number = _resolve_pr(pr)
+    pr_number = github.resolve_pr_number(pr)
 
     waited_secs = 0
     waiting_for: list[str] = []

{agex_cli-0.23.0 → agex_cli-0.24.0}/src/agent_experience/commands/pr/scripts/open_.py

@@ -7,7 +7,7 @@ from importlib.resources import files
 from pathlib import Path
 
 from agent_experience.commands.pr.assets.rules.next_step_rules import open_next_step
-from agent_experience.commands.pr.scripts import _journal
+from agent_experience.commands.pr.scripts import _journal, review
 from agent_experience.commands.pr.scripts._footer import render_footer
 from agent_experience.core import github
 from agent_experience.core.backend import resolve_backend
@@ -55,6 +55,25 @@ def run(
         was_already_open = False
         _journal.append({"type": "pr_opened", "pr": pr, "title": title})
 
+    # Auto-post the Qodo agentic-review trigger out of the box for a freshly
+    # created, non-draft PR.  Skipped for drafts (not review-ready yet — use
+    # `pr review` to trigger later) and for already-open PRs (idempotent
+    # re-opens shouldn't spam the thread).
+    #
+    # The trigger post is best-effort: PR creation is the primary side effect
+    # and has already succeeded, so a transient `gh` failure here must NOT abort
+    # the command (which would tell the user to rerun `pr open`, only to skip
+    # the trigger forever as an already-open PR).  On failure we keep exit 0 and
+    # point the user at `pr review` to retry just the trigger.
+    review_posted = False
+    review_failed = False
+    if not was_already_open and not draft:
+        try:
+            review.post_trigger(pr)
+            review_posted = True
+        except RuntimeError:
+            review_failed = True
+
     footer_key, footer_ctx = open_next_step(pr, was_already_open)
     footer = render_footer(footer_key, backend, footer_ctx)
 
@@ -68,6 +87,9 @@ def run(
             "signed": signed,
             "draft": draft,
             "was_already_open": was_already_open,
+            "review_posted": review_posted,
+            "review_failed": review_failed,
+            "review_command": review.QODO_REVIEW_TRIGGER,
             "footer": footer,
         },
     )

{agex_cli-0.23.0 → agex_cli-0.24.0}/src/agent_experience/commands/pr/scripts/read.py

@@ -25,15 +25,6 @@ from agent_experience.core.render import render_string
 _TEMPLATES_PKG = "agent_experience.commands.pr.assets.templates"
 
 
-def _resolve_pr(pr: int | None) -> int:
-    if pr is not None:
-        return pr
-    view = github.pr_view(None)
-    if view is None:
-        raise ValueError("no PR found for current branch; pass <PR> explicitly")
-    return int(view["number"])
-
-
 def _has_recent_local_commits(journal_events: list[dict[str, Any]], pr: int) -> bool:
     """True if `git log` shows commits authored after the most recent
     `pr_read` event for this PR.  No event yet → False (first read)."""
@@ -60,7 +51,7 @@ def run(
     wait: int | None,
 ) -> tuple[str, int, str]:
     backend = resolve_backend(agent, project_dir)
-    pr_number = _resolve_pr(pr)
+    pr_number = github.resolve_pr_number(pr)
 
     waited_secs = 0
     waiting_for: list[str] = []

agex_cli-0.24.0/src/agent_experience/commands/pr/scripts/review.py

@@ -0,0 +1,59 @@
+"""`agex pr review` — post the Qodo agentic-review trigger comment.
+
+Qodo's current command to start an agentic PR review is ``/agentic_review``.
+The legacy ``/improve`` command is deprecated (Qodo emits a deprecation banner
+when it is used), so agex never posts ``/improve``.  ``QODO_REVIEW_TRIGGER`` is
+the single source of truth for the command string — any future Qodo rename is a
+one-line change here, picked up by both this verb and the auto-post on
+``pr open``.
+"""
+
+from __future__ import annotations
+
+from importlib.resources import files
+from pathlib import Path
+
+from agent_experience.commands.pr.assets.rules.next_step_rules import review_next_step
+from agent_experience.commands.pr.scripts import _journal
+from agent_experience.commands.pr.scripts._footer import render_footer
+from agent_experience.core import github
+from agent_experience.core.backend import resolve_backend
+from agent_experience.core.render import render_string
+
+_TEMPLATES_PKG = "agent_experience.commands.pr.assets.templates"
+
+# The non-deprecated Qodo command to start an agentic code review. Single
+# source of truth — referenced here and by the `pr open` auto-post.
+QODO_REVIEW_TRIGGER = "/agentic_review"
+
+
+def post_trigger(pr: int) -> int:
+    """Post the Qodo review-trigger comment on ``pr``; return the comment ID.
+
+    Also appends a ``pr_review_triggered`` journal event.  Shared by the
+    ``pr review`` verb and the ``pr open`` auto-post.
+    """
+    comment_id = github.pr_post_comment(pr, QODO_REVIEW_TRIGGER, None)
+    _journal.append({"type": "pr_review_triggered", "pr": pr, "command": QODO_REVIEW_TRIGGER})
+    return comment_id
+
+
+def run(agent: str | None, project_dir: Path, pr: int | None) -> tuple[str, int, str]:
+    backend = resolve_backend(agent, project_dir)
+    pr_number = github.resolve_pr_number(pr)
+
+    post_trigger(pr_number)
+
+    footer_key, footer_ctx = review_next_step(pr_number)
+    footer = render_footer(footer_key, backend, footer_ctx)
+
+    template = files(_TEMPLATES_PKG).joinpath("pr_review_result.md.j2").read_text(encoding="utf-8")
+    stdout = render_string(
+        template,
+        {
+            "pr": pr_number,
+            "command": QODO_REVIEW_TRIGGER,
+            "footer": footer,
+        },
+    )
+    return stdout, 0, ""

{agex_cli-0.23.0 → agex_cli-0.24.0}/src/agent_experience/core/github.py

@@ -93,6 +93,21 @@ def pr_view(pr_or_branch: str | None) -> dict[str, Any] | None:
     return json.loads(stdout)
 
 
+def resolve_pr_number(pr: int | None) -> int:
+    """Return ``pr`` if given, else the open PR number for the current branch.
+
+    Raises ``ValueError`` when no number is given and the branch has no PR —
+    the shared resolution used by every `agex pr` verb that takes an optional
+    ``<PR>`` positional (``read``, ``await``, ``review``).
+    """
+    if pr is not None:
+        return pr
+    view = pr_view(None)
+    if view is None:
+        raise ValueError("no PR found for current branch; pass <PR> explicitly")
+    return int(view["number"])
+
+
 def _repo_slug() -> str:
     """Return 'owner/repo' from `gh repo view --json owner,name`."""
     out = _run_gh(["repo", "view", "--json", "owner,name"])

{agex_cli-0.23.0 → agex_cli-0.24.0}/tests/commands/pr/test_open.py

@@ -6,6 +6,7 @@ from agent_experience.core import github
 def _patch(monkeypatch, *, view_returns, create_returns_pr=42, captured=None):
     if captured is None:
         captured = {}
+    captured.setdefault("posts", [])
     monkeypatch.setattr(github, "pr_view", lambda branch=None: view_returns)
     monkeypatch.setattr(github, "resolve_nick", lambda d: "agex-cli")
 
@@ -15,7 +16,12 @@ def _patch(monkeypatch, *, view_returns, create_returns_pr=42, captured=None):
         captured["draft"] = draft
         return create_returns_pr
 
+    def fake_post(pr, body, in_reply_to):
+        captured["posts"].append({"pr": pr, "body": body, "in_reply_to": in_reply_to})
+        return 1000 + len(captured["posts"])
+
     monkeypatch.setattr(github, "pr_create", fake_create)
+    monkeypatch.setattr(github, "pr_post_comment", fake_post)
     return captured
 
 
@@ -123,6 +129,90 @@ def test_pr_open_draft_flag(monkeypatch, tmp_path, capsys):
     assert captured["draft"] is True
 
 
+def test_pr_open_auto_posts_agentic_review(monkeypatch, tmp_path, capsys):
+    monkeypatch.chdir(tmp_path)
+    captured = _patch(monkeypatch, view_returns=None)
+    body_file = tmp_path / "body.md"
+    body_file.write_text("b\n", encoding="utf-8")
+    code = cli.main(
+        ["pr", "open", "--agent", "claude-code", "--title", "t", "--body-file", str(body_file)]
+    )
+    out = capsys.readouterr()
+    assert code == 0
+    # Exactly one auto-posted trigger, top-level, with the non-deprecated command.
+    assert captured["posts"] == [{"pr": 42, "body": "/agentic_review", "in_reply_to": None}]
+    assert "/agentic_review" in out.out
+    assert "/improve" not in out.out
+    # Journal records the trigger alongside pr_opened.
+    events = _journal.load()
+    assert "pr_review_triggered" in [e["type"] for e in events]
+
+
+def test_pr_open_draft_does_not_post_trigger(monkeypatch, tmp_path, capsys):
+    monkeypatch.chdir(tmp_path)
+    captured = _patch(monkeypatch, view_returns=None)
+    body_file = tmp_path / "body.md"
+    body_file.write_text("b\n", encoding="utf-8")
+    cli.main(
+        [
+            "pr",
+            "open",
+            "--agent",
+            "claude-code",
+            "--title",
+            "t",
+            "--body-file",
+            str(body_file),
+            "--draft",
+        ]
+    )
+    capsys.readouterr()
+    assert captured["posts"] == []
+
+
+def test_pr_open_already_open_does_not_post_trigger(monkeypatch, tmp_path, capsys):
+    monkeypatch.chdir(tmp_path)
+    posts: list = []
+    monkeypatch.setattr(github, "pr_view", lambda branch=None: {"number": 7, "state": "OPEN"})
+    monkeypatch.setattr(github, "resolve_nick", lambda d: "agex-cli")
+    monkeypatch.setattr(github, "pr_create", lambda **kw: 999)
+    monkeypatch.setattr(
+        github, "pr_post_comment", lambda pr, body, in_reply_to: posts.append(body) or 1
+    )
+    body_file = tmp_path / "body.md"
+    body_file.write_text("b\n", encoding="utf-8")
+    cli.main(
+        ["pr", "open", "--agent", "claude-code", "--title", "t", "--body-file", str(body_file)]
+    )
+    capsys.readouterr()
+    assert posts == []
+
+
+def test_pr_open_survives_trigger_post_failure(monkeypatch, tmp_path, capsys):
+    # PR creation is the primary side effect; a failed trigger post must not
+    # abort the command (which would tell the user to rerun `pr open` and skip
+    # the trigger forever). Exit 0, PR reported open, and point at `pr review`.
+    monkeypatch.chdir(tmp_path)
+    _patch(monkeypatch, view_returns=None)
+
+    def boom(pr, body, in_reply_to):
+        raise RuntimeError("gh failed: network unreachable")
+
+    monkeypatch.setattr(github, "pr_post_comment", boom)
+    body_file = tmp_path / "body.md"
+    body_file.write_text("b\n", encoding="utf-8")
+    code = cli.main(
+        ["pr", "open", "--agent", "claude-code", "--title", "t", "--body-file", str(body_file)]
+    )
+    out = capsys.readouterr()
+    assert code == 0
+    assert "PR opened" in out.out
+    assert "#42" in out.out
+    # Surfaces the recovery path without claiming the trigger was posted.
+    assert "pr review 42" in out.out
+    assert "Posted `/agentic_review`" not in out.out
+
+
 def test_pr_open_with_delayed_read_chains(monkeypatch, tmp_path, capsys):
     monkeypatch.chdir(tmp_path)
     _patch(monkeypatch, view_returns=None)

agex_cli-0.24.0/tests/commands/pr/test_review.py

@@ -0,0 +1,97 @@
+from pathlib import Path
+
+import agent_experience
+import agent_experience.cli as cli
+from agent_experience.commands.pr.scripts import _journal, review
+from agent_experience.core import github
+
+
+def test_package_never_emits_deprecated_improve_command():
+    """Acceptance criterion: no code path posts the deprecated `/improve`.
+
+    Scans shipped Python and template/asset files for `/improve` as a *posted*
+    command — i.e. a quoted string literal or a Jinja/YAML value — guarding
+    against a regression that reintroduces it. Prose that *names* `/improve`
+    only to say it's deprecated (e.g. SKILL.md, the review.py docstring) is
+    allowed; what's forbidden is emitting it as a value.
+    """
+    pkg_root = Path(agent_experience.__file__).parent
+    forbidden = ('"/improve"', "'/improve'")
+    offenders = []
+    for p in pkg_root.rglob("*"):
+        if not (p.is_file() and p.suffix in {".py", ".j2", ".yaml", ".yml"}):
+            continue
+        text = p.read_text(encoding="utf-8", errors="ignore")
+        if any(token in text for token in forbidden):
+            offenders.append(p.relative_to(pkg_root))
+    assert offenders == [], f"deprecated Qodo command `/improve` emitted in: {offenders}"
+
+
+def _patch_post(monkeypatch, captured):
+    def fake_post(pr, body, in_reply_to):
+        captured.append({"pr": pr, "body": body, "in_reply_to": in_reply_to})
+        return 1234
+
+    monkeypatch.setattr(github, "pr_post_comment", fake_post)
+
+
+def test_trigger_constant_is_agentic_review():
+    # Single source of truth — the non-deprecated Qodo command.
+    assert review.QODO_REVIEW_TRIGGER == "/agentic_review"
+
+
+def test_pr_review_posts_agentic_review_on_explicit_pr(monkeypatch, tmp_path, capsys):
+    monkeypatch.chdir(tmp_path)
+    posts: list = []
+    _patch_post(monkeypatch, posts)
+
+    code = cli.main(["pr", "review", "42", "--agent", "claude-code"])
+    out = capsys.readouterr()
+
+    assert code == 0
+    assert posts == [{"pr": 42, "body": "/agentic_review", "in_reply_to": None}]
+    assert "/agentic_review" in out.out
+    assert "/improve" not in out.out
+    # Next-step footer present.
+    assert "Next step:" in out.out
+    assert "pr read 42" in out.out
+
+
+def test_pr_review_resolves_current_branch_pr(monkeypatch, tmp_path, capsys):
+    monkeypatch.chdir(tmp_path)
+    posts: list = []
+    _patch_post(monkeypatch, posts)
+    monkeypatch.setattr(github, "pr_view", lambda branch=None: {"number": 7, "state": "OPEN"})
+
+    code = cli.main(["pr", "review", "--agent", "claude-code"])
+    capsys.readouterr()
+
+    assert code == 0
+    assert posts == [{"pr": 7, "body": "/agentic_review", "in_reply_to": None}]
+
+
+def test_pr_review_errors_when_no_pr_for_branch(monkeypatch, tmp_path, capsys):
+    monkeypatch.chdir(tmp_path)
+    posts: list = []
+    _patch_post(monkeypatch, posts)
+    monkeypatch.setattr(github, "pr_view", lambda branch=None: None)
+
+    code = cli.main(["pr", "review", "--agent", "claude-code"])
+    err = capsys.readouterr().err
+
+    assert code == 2
+    assert "no PR found for current branch" in err
+    assert posts == []
+
+
+def test_pr_review_writes_journal_event(monkeypatch, tmp_path, capsys):
+    monkeypatch.chdir(tmp_path)
+    _patch_post(monkeypatch, [])
+
+    cli.main(["pr", "review", "99", "--agent", "claude-code"])
+    capsys.readouterr()
+
+    events = _journal.load()
+    triggered = next(e for e in events if e["type"] == "pr_review_triggered")
+    assert triggered["pr"] == 99
+    assert triggered["command"] == "/agentic_review"

{agex_cli-0.23.0 → agex_cli-0.24.0}/uv.lock

@@ -4,7 +4,7 @@ requires-python = ">=3.10"
 
 [[package]]
 name = "agex-cli"
-version = "0.23.0"
+version = "0.24.0"
 source = { editable = "." }
 dependencies = [
     { name = "jinja2" },