agex-cli 0.16.0__tar.gz → 0.18.0__tar.gz

{agex_cli-0.16.0 → agex_cli-0.18.0}/CHANGELOG.md

@@ -7,6 +7,50 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.18.0] - 2026-05-12
+
+### Added
+
+- `agex pr await [<PR>] [--max-wait SECS]` — combo verb that polls
+  reviewer readiness, runs CI + SonarCloud quality gate, and dumps the
+  unified briefing. **Exits 1 on quality-gate `ERROR`, unresolved
+  review threads, or failing CI checks**, 0 on clean state or timeout.
+  Use this when a script should fail if the PR isn't triage-able.
+  Closes [#41](https://github.com/agentculture/agex-cli/issues/41).
+- `SONAR_PROJECT_KEY` env var (and `[pr].sonar_project_key` config key)
+  override the default `<owner>_<repo>` SonarCloud project-key
+  derivation, for repos with non-standard project naming.
+- `agex learn cicd` lesson now documents the 5-minute Anthropic prompt
+  cache TTL and recommends running long waits inside a background
+  subagent so the parent session's cache stays warm.
+
+### Changed
+
+- Polling helpers (`is_ready`, `threads_unresolved`, `required_reviewers`,
+  `POLL_INTERVAL_SEC`) moved from `commands/pr/scripts/read.py` to a
+  shared `commands/pr/scripts/_readiness.py` so `pr read --wait` and
+  `pr await` share one implementation. No behavior change for
+  `pr read`.
+
+## [0.17.0] - 2026-05-10
+
+### Added
+
+- `agex pr` command namespace: `lint`, `open`, `read`, `reply`, `delta`. Supersedes the bash `cicd` skill.
+- `agex pr open --delayed-read` chains to `agex pr read --wait 180` after creating the PR.
+- `agex pr read --wait SECS` polls for required reviewer readiness (default `qodo`; configurable via `[pr].required_reviewers`) before rendering the briefing.
+- Per-command "Next step:" footers driven by `commands/pr/assets/rules/next_step_rules.py` and per-backend phrasing under `commands/pr/assets/backends/<backend>.yaml`.
+- `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`.
+- `core/backend.resolve_backend()` — `--agent` resolution with `culture.yaml` fallback.
+- `agex learn cicd` lesson teaching the new workflow.
+- `agex hook read` discovers nested `data/<dir>/*.jsonl` streams.
+
+### Changed
+
+- Invariant carve-out: the `agex pr` namespace is allowed scoped network I/O and bounded `--wait` sleep. No silent retries anywhere.
+- Side-effect inventory extended to include `pr open`, `pr reply`, and `pr read` (journal writes).
+
 ## [0.16.0] — 2026-05-10
 
 ### Added

{agex_cli-0.16.0 → agex_cli-0.18.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`, and first-run `.agex/` init. Everything else is read-only.
+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.
 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.
 
@@ -41,6 +41,17 @@ cli.py ──► commands/<name>/scripts/<name>.py ──► core/render.py
 - `core/` is shared plumbing — backend enum, `.agex/` paths, Jinja renderer (`StrictUndefined`), TOML config, SKILL.md frontmatter parser, capability matrix loader, hook JSON I/O. Command- and content-agnostic.
 - A backend lives in three places: `core/backend.Backend` (enum entry), `backends/<name>/probe.py` (optional Python probe), and one YAML per relevant command under `commands/*/assets/backends/<name>.yaml`. Adding a backend touches only those locations.
 
+## `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:
+
+- `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`.
+- `core/backend.resolve_backend()` — `--agent` resolution with `culture.yaml` fallback.
+- `commands/pr/assets/rules/next_step_rules.py` — "Next step:" footer decision logic, per-backend phrasing under `commands/pr/assets/backends/<backend>.yaml`.
+
+Spec: `docs/superpowers/specs/2026-05-10-agex-pr-design.md`. Plan: `docs/superpowers/plans/2026-05-10-agex-pr.md`.
+
 ## Conventions worth following
 
 - **Resource loading: use `importlib.resources.files(...)` and treat the result as a `Traversable`.** Call `.joinpath()` / `.is_file()` / `.read_text(encoding="utf-8")` directly. Wrap with `importlib.resources.as_file()` only when a third-party API needs a real `pathlib.Path`. Do NOT do `Path(str(files(...)))` — it's not zipapp/PEX safe.
@@ -90,6 +101,6 @@ uv run pytest --cov=src/agent_experience --cov-report=xml --cov-report=term
 - Push, open a PR, let CI + SonarCloud + Qodo + Copilot run. Address inline comments + resolve threads before merge.
 - Merging to `main` publishes to PyPI automatically (via `autotag` → `publish-pypi` → `github-release` in `publish.yml`). No manual tagging.
 - When posting on GitHub on the user's behalf (PR descriptions, issue replies, review-thread replies), sign so it's clear the message came from an AI. Three conventions, in priority order:
-  - **Inside `cicd` workflow scripts** (`pr-reply.sh`, `pr-batch.sh`) — the script auto-appends `- agex-cli (Claude)` from the repo's `culture.yaml`. Don't sign manually.
+  - **`agex pr open` / `agex pr reply`** — auto-append `- agex-cli (Claude)` (the nick is resolved from `culture.yaml` via `core.github.resolve_nick`, falling back to the repo basename). Don't sign manually.
   - **Inside `communicate` workflow scripts** (`post-issue.sh`, `post-comment.sh`) — `agtag` resolves the same nick from `culture.yaml`. Don't sign manually.
   - **Manual posts the scripts didn't author** (a hand-typed `gh pr create --body …`, a one-off review reply) — sign explicitly as `- agex-cli (Claude)`.

{agex_cli-0.16.0 → agex_cli-0.18.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: agex-cli
-Version: 0.16.0
+Version: 0.18.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/agex-cli