@nvice-r/pr-commands 0.1.3 → 0.1.5

package/README.md

@@ -47,7 +47,7 @@ Flags:
 | `--agent <id>` | force skill install to this agent (repeatable): `claude-code`\|`antigravity`\|`codex`\|`opencode`. Default: auto-detect |
 | `--all` | install every ready provider binary, not just the detected one |
 | `--agent-dir <dir>` | repo folder for the reviewer roster (default: primary agent's — `.claude`/`.agent`/`.codex`/`.opencode`) |
-| `--yes` | non-interactive: reviewers = existing roster if any, else all; convention = `trueid` |
+| `--yes` | non-interactive: reviewers = existing roster if any, else all; convention = `versioned` |
 | `--reset-reviewers` | ignore an existing roster and rebuild it from the repo's default reviewers |
 | `--dry-run` | print planned actions, write nothing |
 | `--help` | usage |
@@ -78,8 +78,9 @@ identically from every clone and every git worktree, no repo-local symlinks.
 
 | Artifact | Path |
 |----------|------|
-| Shared library | `~/.local/bin/pr-common.sh` |
-| Provider binary | `~/.local/bin/bbpr` (`ghpr`/`glpr` later) |
+| Command (only file on `PATH`) | `~/.local/bin/pr-commands` |
+| Shared library | `~/.local/share/pr-commands/pr-common.sh` |
+| Host provider | `~/.local/share/pr-commands/providers/bitbucket.sh` (`github.sh`/`gitlab.sh` later) |
 | Config | `~/.config/pr/config` |
 | Reviewer roster | `<main-repo-root>/<agent_dir>/pr-reviewers.txt` (primary agent's folder; shared across worktrees) |
 | Canonical skill | `~/.config/pr/skill/pr/SKILL.md` (single source) |
@@ -97,7 +98,7 @@ In Claude Code, from any configured repo:
 
 ```
 /pr                 # infer base, draft PR, confirm, create
-/pr feature/2.3.0/center   # explicit base branch
+/pr develop         # explicit base branch
 /pr draft           # create as draft
 /pr help            # help
 ```
@@ -113,7 +114,7 @@ confirmation, then calls the detected binary. It never creates a PR silently.
 os=macos             # macos | linux
 token_src=keychain   # keychain | secret-tool | env
 email=you@example.com
-convention=trueid    # see presets
+convention=versioned # see presets
 ```
 
 Edit by hand any time. **A missing config is a hard error** — `/pr` and the binaries refuse to
@@ -127,7 +128,7 @@ with different conventions share the same skill:
 
 ```yaml
 title_pattern: "[<ver>] Feature/<ver>/<ticket> <desc>"
-ticket_regex: "SC-(\\d+|NOTICKET)"
+ticket_regex: "PROJ-\\d+"
 template_sections: [What Happened, Insight, Proof of Work]
 body_rules: |
   - Free-text project rules the skill follows literally.
@@ -135,10 +136,10 @@ base: develop            # default base proposal
 draft: ask               # ask | true | false
 push: [origin]           # remotes pushed before create
 fork:                    # optional cross-repo mode (`/pr cross`)
-  remote: social-commerce
-  repo: workspace/fork-repo
+  remote: upstream
+  repo: workspace/repo
 pre_pr: []               # commands run before confirm; non-zero exit stops /pr
-post_pr: [jira-update]   # sibling skills offered after create
+post_pr: []              # sibling skills offered after create
 ```
 
 Without the file, the convention keys below (written by the wizard) are used.
@@ -147,7 +148,7 @@ Without the file, the convention keys below (written by the wizard) are used.
 
 | preset | title pattern | ticket regex | template sections |
 |--------|---------------|--------------|-------------------|
-| `trueid` | `[<ver>] Feature/<ver>/SC-XXXX <desc>` | `SC-\d+` | What Happened / Insight / Proof of Work |
+| `versioned` | `[<ver>] Feature/<ver>/<ticket> <desc>` | `[A-Z]+-\d+` | What Happened / Insight / Proof of Work |
 | `conventional` | `<type>(<scope>): <desc>` | `[A-Z]+-\d+` | Summary / Changes / Testing |
 | `github-default` | `<desc> (#<issue>)` | `#\d+` | Summary / Test plan |
 | `custom` | your `title_pattern=` | your `ticket_regex=` | your `template_sections=` (pipe-separated) |
@@ -175,7 +176,7 @@ roster:
 
 Switching primary agent = re-run the installer; the old file still loads as a fallback.
 
-You (the author) are excluded on install — bbpr also drops self at PR time. One entry per line,
+You (the author) are excluded on install — `pr-commands` also drops self at PR time. One entry per line,
 trailing `# Name` comment optional:
 
 ```
@@ -184,22 +185,21 @@ trailing `# Name` comment optional:
 ```
 
 It's plain data — edit freely, no reinstall. Consider gitignoring the roster unless the team wants
-it committed. Load order: `$BBPR_REVIEWERS` → `<agent_dir>/pr-reviewers.txt` → `.pr/reviewers.txt`
-(0.1.2) → root `.pr-reviewers.txt` (pre-0.1.1) → global `~/.config/pr/reviewers/<ws>-<slug>.txt` →
-legacy. Run-time overrides: `--no-reviewers` skips all, `--reviewer <uuid>` appends.
+it committed. `$PR_REVIEWERS` (a file path) overrides the roster location. Run-time overrides:
+`--no-reviewers` skips all, `--reviewer <uuid>` appends.
 
 ### Auth precedence (email)
 
-`--email` flag → `$BB_EMAIL` → `git config bbpr.email` → config `email` → keychain account.
+`--email` flag → `$BB_EMAIL` → `git config pr.email` → config `email` → keychain account.
 The token is always read from the keychain (or `$PR_TOKEN` when `token_src=env`).
 
 ## Provider support
 
-| provider | binary | status |
-|----------|--------|--------|
-| Bitbucket Cloud | `bbpr` | ✅ ready |
-| GitHub | `ghpr` | 🚧 stub — errors `not-yet-supported` |
-| GitLab | `glpr` | 🚧 stub |
+| provider | file | status |
+|----------|------|--------|
+| Bitbucket Cloud | `providers/bitbucket.sh` | ✅ ready |
+| GitHub | `providers/github.sh` | 🚧 not yet — `pr-commands` errors `github not yet supported` |
+| GitLab | `providers/gitlab.sh` | 🚧 not yet |
 
 | OS | token source | status |
 |----|--------------|--------|
@@ -208,17 +208,21 @@ The token is always read from the keychain (or `$PR_TOKEN` when `token_src=env`)
 
 ## Architecture
 
-- **Split by provider, not by OS.** Each provider gets its own small binary (`bbpr`/`ghpr`/`glpr`)
-  because the REST APIs genuinely differ. OS differences (token retrieval) are a single
-  `get_token()` function switched by `uname`, not a separate binary. Ceiling: 3 binaries, never
-  3 × OS.
+- **One command, one file per host.** `pr-commands` is the only thing on `PATH`. It reads
+  `git remote get-url origin` (override: `$PR_PROVIDER`) and loads `providers/<host>.sh`.
+  Each host keeps its own small file because the REST APIs genuinely differ. OS differences
+  (token retrieval) are a single `get_token()` function, not a separate file.
 - **Shared logic in `pr-common.sh`** — git scan, base save, reviewer load, arg parse, token —
-  sourced by every binary. A provider binary is ~40 lines: payload shape + endpoint + parse.
+  sourced before the provider. A provider file is ~60 lines: payload shape + endpoint + parse.
 - **One generic `/pr` skill.** Convention lives in config, not the skill, so the skill is
-  provider- and team-neutral and installs globally. Adding a provider = one binary + one dispatch
-  line, zero new skill files.
+  host- and team-neutral and installs globally. It always calls `pr-commands create`, so adding
+  a host = one provider file + `ready: true` in `install.js`, zero skill changes.
 
-Full rationale in [`DECISIONS.md`](./DECISIONS.md).
+```bash
+pr-commands create --title "..." --body-file body.md --dest develop [--draft]
+pr-commands check     # which host, and is it supported?
+pr-commands help
+```
 
 ## Troubleshooting
 
@@ -227,39 +231,21 @@ Full rationale in [`DECISIONS.md`](./DECISIONS.md).
 | `no ~/.config/pr/config` | run `npx @nvice-r/pr-commands` |
 | reviewers not fetched | add the keychain token (see Prerequisites), re-run |
 | `WARN ~/.local/bin not on PATH` | add the export line above to your shell rc |
-| `not-yet-supported: <provider>` | GitHub/GitLab aren't implemented yet |
-| worktree runs old behavior | skill/binaries are global, not repo-local; ensure no stale per-repo copy shadows them |
+| `<provider> not yet supported` | GitHub/GitLab aren't implemented yet |
+| worktree runs old behavior | skill/command are global, not repo-local; ensure no stale per-repo copy shadows them |
 | agent doesn't see the skill | re-run and force it: `npx @nvice-r/pr-commands --agent <id>`; check `<agent-skills-dir>/pr` symlink exists |
 
 ## Uninstall
 
 ```bash
-rm ~/.local/bin/bbpr ~/.local/bin/pr-common.sh
+rm ~/.local/bin/pr-commands
+rm -rf ~/.local/share/pr-commands
 rm -rf ~/.config/pr
 rm ~/.claude/commands/pr.md
 # per-agent skill symlinks (remove those you installed):
 rm ~/.claude/skills/pr ~/.gemini/antigravity/skills/pr ~/.codex/skills/pr ~/.config/opencode/skills/pr
 # roster lives in each repo — remove per repo if wanted:
-rm -rf <main-repo-root>/.pr
-```
-
-## Development
-
-```
-install.js             npx entry (wizard)
-assets/bin/*           provider binaries + shared lib (source of truth; copied on install)
-assets/skill/pr.body.md  shared /pr skill body (frontmatter added per-target on install)
-```
-
-`assets/` is the single source of truth — both the installer (`install.js`)
-and the published package (`files` in `package.json`) read from it. No duplicate
-tree to keep in sync.
-
-Publish (owner only):
-
-```bash
-npm login
-npm publish --access public
+rm <main-repo-root>/<agent_dir>/pr-reviewers.txt
 ```
 
 ## License

package/assets/bin/pr-commands

@@ -0,0 +1,71 @@
+#!/usr/bin/env bash
+# pr-commands — create pull requests on any git host (one entry point).
+#
+# Usage:
+#   pr-commands create --title "..." [--body|--body-file f] [--source b] [--dest b]
+#                      [--repo owner/name] [--draft] [--reviewer id]... [--no-reviewers]
+#                      [--email a] [--close-source] [--source-repo owner/fork]
+#   pr-commands check      detect the host from the origin remote; exit 0 if supported
+#   pr-commands help       this help
+#   pr-commands --version
+#
+# The host is detected from `git remote get-url origin` (override: $PR_PROVIDER).
+# Host code lives in <share>/providers/<provider>.sh; shared code in <share>/pr-common.sh,
+# where <share> = $PR_COMMANDS_HOME, else <this dir>/../share/pr-commands.
+# pr-commands-version: 1
+
+set -euo pipefail
+
+PR_CMD="pr-commands"
+_here="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
+SHARE="${PR_COMMANDS_HOME:-$_here/../share/pr-commands}"
+
+usage() { sed -n '2,14p' "${BASH_SOURCE[0]}" | sed 's/^# \{0,1\}//'; }
+fail()  { echo "$PR_CMD: $*" >&2; exit 1; }
+
+# Echo the provider name for the origin remote (or $PR_PROVIDER), or fail.
+detect_provider() {
+  if [ -n "${PR_PROVIDER:-}" ]; then echo "$PR_PROVIDER"; return; fi
+  local url
+  url="$(git remote get-url origin 2>/dev/null)" || fail "no origin remote"
+  case "$url" in
+    *bitbucket.org*) echo bitbucket ;;
+    *github.com*)    echo github ;;
+    *gitlab*)        echo gitlab ;;
+    *) fail "unknown git host for remote: $url (set PR_PROVIDER to override)" ;;
+  esac
+}
+
+# Source the shared lib + the provider file. Fails if the provider is not installed.
+load_provider() {
+  PROVIDER="$(detect_provider)"
+  local pfile="$SHARE/providers/${PROVIDER}.sh"
+  [ -f "$SHARE/pr-common.sh" ] || fail "missing $SHARE/pr-common.sh — run: npx @nvice-r/pr-commands"
+  [ -f "$pfile" ] || fail "${PROVIDER} not yet supported (no $pfile)"
+  # shellcheck source=../share/pr-commands/pr-common.sh
+  . "$SHARE/pr-common.sh"
+  # shellcheck disable=SC1090
+  . "$pfile"
+}
+
+sub="${1:-help}"
+case "$sub" in
+  -h|--help|--version) ;;
+  --*) set -- create "$@"; sub=create ;;   # bare flags = create
+esac
+
+case "$sub" in
+  create)
+    shift
+    load_provider
+    declare -F provider_create >/dev/null || fail "${PROVIDER}: create not implemented"
+    provider_create "$@"
+    ;;
+  check)
+    load_provider
+    echo "provider=${PROVIDER} ok"
+    ;;
+  help|-h|--help) usage ;;
+  --version|version) sed -n 's/^# pr-commands-version: //p' "${BASH_SOURCE[0]}" ;;
+  *) fail "unknown command: $sub (try: $PR_CMD help)" ;;
+esac

package/assets/{bin → share/pr-commands}/pr-common.sh

@@ -1,8 +1,8 @@
 #!/usr/bin/env bash
-# pr-common-version: 2
-# pr-common.sh — shared library for provider PR binaries (bbpr/ghpr/glpr).
-# NOT executable on its own — sourced by a provider binary, which then builds the
-# provider-specific payload + POST. Everything here is provider-AGNOSTIC.
+# pr-common-version: 3
+# pr-common.sh — shared library for the `pr-commands` entry point.
+# NOT executable on its own — sourced by `pr-commands` before providers/<host>.sh, which
+# builds the host-specific payload + POST. Everything here is host-AGNOSTIC.
 #
 # Provides (after `pr_common_init "$@"`): global vars
 #   TITLE BODY SOURCE DEST REPO SOURCE_REPO EMAIL TOKEN DRAFT CLOSE_SOURCE
@@ -82,7 +82,7 @@ pr_parse_args() {
       --reviewer)      EXTRA_REVIEWERS+=("$2"); shift 2 ;;
       --no-reviewers)  NO_REVIEWERS="true"; shift ;;
       --email)         EMAIL_ARG="$2"; shift 2 ;;
-      -h|--help)       sed -n '2,20p' "$0"; exit 0 ;;
+      -h|--help)       if declare -F usage >/dev/null; then usage; else sed -n '2,20p' "$0"; fi; exit 0 ;;
       *) die "unknown flag: $1" ;;
     esac
   done
@@ -101,10 +101,11 @@ pr_git_defaults() {
     [ "$SOURCE" = "HEAD" ] && die "detached HEAD — pass --source"
   fi
   if [ -z "$DEST" ]; then
-    DEST="$(git config --get "branch.${SOURCE}.bbprBase" 2>/dev/null || true)"
-    [ -n "$DEST" ] && echo "${PR_CMD:-pr}: using saved base (branch.${SOURCE}.bbprBase = ${DEST})" >&2
+    pr_migrate_base
+    DEST="$(git config --get "branch.${SOURCE}.base" 2>/dev/null || true)"
+    [ -n "$DEST" ] && echo "${PR_CMD:-pr}: using saved base (branch.${SOURCE}.base = ${DEST})" >&2
   fi
-  [ -n "$DEST" ] || die "--dest required (no saved branch.${SOURCE}.bbprBase)"
+  [ -n "$DEST" ] || die "--dest required (no saved branch.${SOURCE}.base)"
   if [ -z "$REPO" ]; then
     local url
     url="$(git remote get-url origin 2>/dev/null)" || die "no origin remote"
@@ -116,9 +117,11 @@ pr_git_defaults() {
 
 # --- creds: email precedence + token via OS ---
 pr_resolve_creds() {
-  # --email > $BB_EMAIL > git config bbpr.email > config email > keychain acct
+  # --email > $BB_EMAIL > git config pr.email > config email > keychain acct
+  # (legacy git config bbpr.email still read as fallback after pr.email)
   EMAIL="$EMAIL_ARG"
   [ -n "$EMAIL" ] || EMAIL="${BB_EMAIL:-}"
+  [ -n "$EMAIL" ] || EMAIL="$(git config pr.email 2>/dev/null || true)"
   [ -n "$EMAIL" ] || EMAIL="$(git config bbpr.email 2>/dev/null || true)"
   [ -n "$EMAIL" ] || EMAIL="${CFG_EMAIL:-}"
   [ -n "$EMAIL" ] || die "no email resolved (set config email or pass --email)"
@@ -128,7 +131,7 @@ pr_resolve_creds() {
 # --- reviewer roster: work-dir file (shared across worktrees) then global fallbacks ---
 pr_load_reviewers() {
   DEFAULT_REVIEWERS=()
-  local file="${BBPR_REVIEWERS:-}"
+  local file="${PR_REVIEWERS:-${BBPR_REVIEWERS:-}}"   # BBPR_REVIEWERS = legacy name
   if [ -z "$file" ]; then
     local slug; slug="$(echo "$REPO" | tr '/' '-')"      # workspace/slug -> workspace-slug
     # main repo root, shared across worktrees: git-common-dir/.. (empty if git call fails)
@@ -161,12 +164,28 @@ pr_load_reviewers() {
 }
 
 # --- persist resolved base for next run ---
+# branch.<b>.base is the shared, host-neutral key (also read by
+# @nvice-r/git-commands git-restack).
 pr_save_base() {
-  git config "branch.${SOURCE}.bbprBase" "$DEST" 2>/dev/null \
-    && echo "${PR_CMD:-pr}: saved branch.${SOURCE}.bbprBase = ${DEST}" >&2
+  git config "branch.${SOURCE}.base" "$DEST" 2>/dev/null \
+    && echo "${PR_CMD:-pr}: saved branch.${SOURCE}.base = ${DEST}" >&2
   return 0   # base-save is best-effort; never abort the binary on failure
 }
 
+# One-time migration: pr-commands <= 0.1.3 saved the base under the legacy key
+# branch.<b>.bbprBase. Copy it to branch.<b>.base and unset the old key.
+# Remove this function once no branch can still carry the legacy key.
+pr_migrate_base() {
+  local old
+  git config --get "branch.${SOURCE}.base" >/dev/null 2>&1 && return 0
+  old="$(git config --get "branch.${SOURCE}.bbprBase" 2>/dev/null || true)"
+  [ -n "$old" ] || return 0
+  git config "branch.${SOURCE}.base" "$old" 2>/dev/null \
+    && git config --unset "branch.${SOURCE}.bbprBase" 2>/dev/null \
+    && echo "${PR_CMD:-pr}: migrated branch.${SOURCE}.bbprBase -> branch.${SOURCE}.base" >&2
+  return 0   # best-effort; never abort the binary on failure
+}
+
 # --- orchestrate everything a provider needs before its POST ---
 pr_common_init() {
   pr_load_config

package/assets/share/pr-commands/providers/bitbucket.sh

@@ -0,0 +1,77 @@
+# bitbucket-provider-version: 10
+# providers/bitbucket.sh — Bitbucket Cloud (REST API v2.0). Sourced by `pr-commands`
+# after pr-common.sh; never run directly. Only Bitbucket-specific code lives here:
+# payload shape, endpoint, self-as-reviewer rule, response parse.
+
+KEYCHAIN_SERVICE="bitbucket-api-token"
+
+provider_create() {
+  pr_common_init "$@"
+
+  # --- Bitbucket: drop the PR author's own UUID (BB rejects author-as-reviewer) ---
+  if [ "${#FINAL_REVIEWERS[@]}" -gt 0 ]; then
+    local SELF_UUID
+    SELF_UUID="$(curl -sS -u "${EMAIL}:${TOKEN}" "https://api.bitbucket.org/2.0/user" 2>/dev/null \
+      | python3 -c 'import json,sys
+try: print(json.load(sys.stdin).get("uuid",""))
+except Exception: print("")' 2>/dev/null || true)"
+    if [ -n "$SELF_UUID" ]; then
+      local FILTERED=() r
+      for r in "${FINAL_REVIEWERS[@]}"; do
+        [ "$r" = "$SELF_UUID" ] || FILTERED+=("$r")
+      done
+      FINAL_REVIEWERS=("${FILTERED[@]:-}")
+      [ -z "${FINAL_REVIEWERS[0]:-}" ] && FINAL_REVIEWERS=()
+    fi
+  fi
+
+  # --- Bitbucket payload (source/destination keys, draft, reviewers[]) ---
+  local PAYLOAD
+  PAYLOAD="$(
+    TITLE="$TITLE" BODY="$BODY" SOURCE="$SOURCE" DEST="$DEST" SOURCE_REPO="$SOURCE_REPO" \
+    DRAFT="$DRAFT" CLOSE_SOURCE="$CLOSE_SOURCE" \
+    REVIEWERS_JSON="$(printf '%s\n' "${FINAL_REVIEWERS[@]:-}" | python3 -c 'import json,sys; print(json.dumps([{"uuid": l.strip()} for l in sys.stdin if l.strip()]))')" \
+    python3 -c '
+import json, os
+p = {
+  "title": os.environ["TITLE"],
+  "description": os.environ["BODY"],
+  "source": {"branch": {"name": os.environ["SOURCE"]}},
+  "destination": {"branch": {"name": os.environ["DEST"]}},
+  "draft": os.environ["DRAFT"] == "true",
+  "close_source_branch": os.environ["CLOSE_SOURCE"] == "true",
+}
+if os.environ["SOURCE_REPO"]:
+  p["source"]["repository"] = {"full_name": os.environ["SOURCE_REPO"]}
+revs = json.loads(os.environ["REVIEWERS_JSON"])
+if revs: p["reviewers"] = revs
+print(json.dumps(p))
+'
+  )"
+
+  local URL="https://api.bitbucket.org/2.0/repositories/${REPO}/pullrequests"
+  echo "POST $URL"
+  echo "source=${SOURCE_REPO:+$SOURCE_REPO:}$SOURCE  dest=$DEST  draft=$DRAFT  reviewers=${#FINAL_REVIEWERS[@]}"
+
+  local RESP PR_INFO
+  RESP="$(curl -sS -X POST "$URL" -u "${EMAIL}:${TOKEN}" \
+    -H 'Content-Type: application/json' -d "$PAYLOAD")"
+
+  PR_INFO="$(echo "$RESP" | python3 -c '
+import json, sys
+try: d = json.loads(sys.stdin.read())
+except Exception as e: print("non-JSON response", e); sys.exit(2)
+if "error" in d: print("ERROR:", json.dumps(d["error"], indent=2)); sys.exit(1)
+print(str(d.get("id")) + "\t" + str(d.get("links", {}).get("html", {}).get("href")))
+')" || { echo "$PR_INFO" >&2; exit 1; }   # surface the API error; set -e would swallow it
+
+  local NEW_PR_ID NEW_PR_URL ATTACHED
+  NEW_PR_ID="$(echo "$PR_INFO" | cut -f1)"
+  NEW_PR_URL="$(echo "$PR_INFO" | cut -f2)"
+  ATTACHED="$(echo "$RESP" | python3 -c 'import json,sys; print(len(json.load(sys.stdin).get("reviewers", [])))')"
+  echo "created PR #${NEW_PR_ID} with ${ATTACHED} reviewers"
+  echo "$NEW_PR_URL"
+
+  [ -n "$NEW_PR_ID" ] && pr_save_base
+  return 0
+}

package/assets/skill/pr.body.md

@@ -1,5 +1,5 @@
 Create a pull request for the current branch. Provider (Bitbucket / GitHub / GitLab) is
-auto-detected from the git remote; the matching binary (`bbpr` / `ghpr` / `glpr`) is invoked.
+auto-detected from the git remote by the `pr-commands` command, which runs that host's provider.
 Convention (title pattern, ticket regex, PR template, body rules, push remotes, hooks) comes
 from the repo's `.agents/config/pr.yaml` (one file per project, usually copied there by
 `/landing`). Without that file, the convention keys in `~/.config/pr/config` are used.
@@ -36,8 +36,8 @@ UPDATE
                 goals, filler connectors. Keeps every file + technical fact.
 
 CONVENTION    <repo>/.agents/config/pr.yaml (per project; else convention keys in the user config)
-CONFIG        ~/.config/pr/config  (os, token_src, email, agent_dir; legacy convention keys)
-PROVIDER      auto from `git remote get-url origin` -> bbpr | ghpr | glpr
+CONFIG        ~/.config/pr/config  (os, token_src, email, agent_dir; fallback convention keys)
+PROVIDER      auto from `git remote get-url origin` (run by `pr-commands`)
 REVIEWERS     <main-repo-root>/<agent_dir>/pr-reviewers.txt (per-repo, shared across worktrees; agent_dir from config)
 SETUP         npx @nvice-r/pr-commands
 ```
@@ -52,17 +52,11 @@ missing piece). Do NOT improvise config or reviewers.
 CFG="${PR_CONFIG:-$HOME/.config/pr/config}"
 [ -f "$CFG" ] || { echo "MISSING config: $CFG"; }
 
-# b. detect provider from remote
-URL="$(git remote get-url origin 2>/dev/null)" || echo "no origin remote"
-case "$URL" in
-  *bitbucket.org*) PROVIDER=bitbucket; BIN=bbpr ;;
-  *github.com*)    PROVIDER=github;    BIN=ghpr ;;
-  *gitlab.com*)    PROVIDER=gitlab;    BIN=glpr ;;
-  *) echo "unknown provider for remote: $URL" ;;
-esac
+# b. command installed?
+command -v pr-commands >/dev/null 2>&1 || echo "MISSING command: pr-commands"
 
-# c. matching binary installed?
-command -v "$BIN" >/dev/null 2>&1 || echo "MISSING binary: $BIN (run installer, or provider not yet supported)"
+# c. host detected + supported? prints "provider=<name> ok", or the reason it is not
+pr-commands check
 
 # d. project convention file: this checkout first, then the main checkout (worktrees share it,
 #    because .agents/config/ is usually gitignored and only exists in the main checkout)
@@ -75,14 +69,14 @@ done
 echo "convention: ${PR_YAML:-$CFG (no .agents/config/pr.yaml)}"
 ```
 
-- config missing → STOP: "run `npx @nvice-r/pr-commands`".
-- provider unknown → STOP: only bitbucket/github/gitlab supported.
-- `$BIN` = `ghpr`/`glpr` and not installed → STOP: "provider not yet supported (macOS+Bitbucket only for now)". (D6 stub.)
+- config or `pr-commands` missing → STOP: "run `npx @nvice-r/pr-commands`".
+- `pr-commands check` fails → STOP and show its message (unknown host, or
+  "<provider> not yet supported" — only Bitbucket for now). (D6 stub.)
 - all pass → continue.
 
 ## Step 1 — gather context (parallel)
 - `git rev-parse --abbrev-ref HEAD` — source branch (CURR)
-- `git config --get branch.<CURR>.bbprBase` — saved base, if any
+- `git config --get branch.<CURR>.base` — saved base, if any
 - `git log --oneline -20`
 - read the **convention source**: `$PR_YAML` if Step 0 found one, else the convention keys in `$CFG`.
   Every later step that says "convention" means this source.
@@ -103,8 +97,8 @@ echo "convention: ${PR_YAML:-$CFG (no .agents/config/pr.yaml)}"
 | `post_pr` | sibling skills offered after create (Step 10) | none |
 
 ## Step 2 — determine base branch (priority)
-a. `$1` (any arg containing `/`) → use it.
-b. else `branch.<CURR>.bbprBase` from git config → use silently.
+a. an argument containing `/` → use it as the base.
+b. else saved `branch.<CURR>.base` from git config → use silently.
 c. else infer — smallest non-zero `<ref>..HEAD` distance where `<ref>` is an ancestor of HEAD:
    ```bash
    for ref in $(git for-each-ref --format='%(refname:short)' refs/remotes/origin | grep -v "^origin/<CURR>$"); do
@@ -118,7 +112,7 @@ c. else infer — smallest non-zero `<ref>..HEAD` distance where `<ref>` is an a
    stacked PR on another in-flight ticket branch is common. Ties or all >20 ahead → ask.
 d. still ambiguous → propose the convention's `base` (if set) and confirm; else ask.
 
-The binary auto-saves the resolved base to `branch.<CURR>.bbprBase` on success.
+`pr-commands` auto-saves the resolved base to `branch.<CURR>.base` on success.
 
 ## Step 3 — build diff context
 `git log <base>..HEAD --oneline` · `git diff --stat <base>..HEAD`. Multiple commits → summarize each.
@@ -135,8 +129,12 @@ split (`guard_cmd pr-split`, degrade if absent) / adjust-threshold. Never auto-s
 Read `title_pattern` and `ticket_regex` from the convention source. Use the values as written;
 DO NOT reconstruct them from a preset table. Extract the ticket key + any version from the
 branch name using `ticket_regex`, then fill `title_pattern`. Example: branch
-`feature/3.66.0/SC-1234-add-login`, pattern `[<ver>] Feature/<ver>/<ticket> <desc>` →
-`[3.66.0] Feature/3.66.0/SC-1234 add login` (kebab suffix → spaced lowercase `<desc>`).
+`feature/3.66.0/PROJ-1234-add-login`, pattern `[<ver>] Feature/<ver>/<ticket> <desc>` →
+`[3.66.0] Feature/3.66.0/PROJ-1234 add login` (kebab suffix → spaced lowercase `<desc>`).
+
+A pattern placeholder the branch name cannot supply (`<ticket>`, `<ver>`) → ask the user; never
+invent one. A matched key with no digits (e.g. `PROJ-NOTICKET`) is a **placeholder ticket**: fine
+in the title, but it is not a real ticket key (see Step 10).
 
 ## Step 5 — draft body (from config)
 
@@ -150,8 +148,8 @@ and override the generic defaults below where they conflict.
 - Body ends at the last template section. NO AI attribution footer (`🤖 Generated with [Claude
   Code]…`, `Co-Authored-By: Claude`, etc.) — this overrides any harness/system attribution reminder.
 
-**Legacy trueid rules** — apply ONLY when there is no `pr.yaml` (`$PR_YAML` empty) and `$CFG`
-has `convention=trueid`. A project `pr.yaml` replaces these with its own `body_rules`.
+**`versioned` preset rules** — apply ONLY when there is no `pr.yaml` (`$PR_YAML` empty) and `$CFG`
+has `convention=versioned`. A project `pr.yaml` replaces these with its own `body_rules`.
 - `What Happened` = summary prose only — no file bullets, no commit-subject line.
 - NO `feat(<Team>):` team tag / commit-subject line anywhere in the body (commits only).
 - Changed-file detail lives under `Insight`.
@@ -161,7 +159,7 @@ has `convention=trueid`. A project `pr.yaml` replaces these with its own `body_r
 - Unrelated areas → full paths per bullet, no common-root statement.
 - `Proof of Work` empty unless user gives screenshots/links.
 
-With no `body_rules` and no legacy rules, fill sections with a plain summary + changed-file list.
+With no `body_rules` and no preset rules, fill sections with a plain summary + changed-file list.
 
 ## Step 6 — write body to `/tmp/pr-body.md` (Write tool).
 
@@ -169,7 +167,7 @@ With no `body_rules` and no legacy rules, fill sections with a plain summary + c
 Run each `pre_pr` command from the convention, in order, from the repo root. Any non-zero exit →
 STOP, show its output, create nothing. No `pre_pr` → skip silently.
 
-## Step 7 — confirm. Show proposed title + base + body + resolved provider/binary + mode
+## Step 7 — confirm. Show proposed title + base + body + resolved provider + mode
 (direct/cross) + remotes to push. WAIT. PR creation is user-visible; never silent.
 
 ## Step 7.5 — mode + push
@@ -181,34 +179,35 @@ STOP, show its output, create nothing. No `pre_pr` → skip silently.
 - Push failure → STOP and show the error. Never pass `--no-verify` (pre-push hooks may lint;
   allow 30–60 s). No `push` key → do not push; warn if HEAD is ahead of its upstream.
 
-## Step 8 — create. Run the detected binary (append `--draft` only if draft chosen):
+## Step 8 — create. Run `pr-commands create` (append `--draft` only if draft chosen):
 ```bash
-<BIN> --title "<title>" --body-file /tmp/pr-body.md --dest "<base>" [--draft] \
-      [--source-repo "<fork.repo>"]        # cross mode only
+pr-commands create --title "<title>" --body-file /tmp/pr-body.md --dest "<base>" [--draft] \
+      [--source-repo "<fork.repo>"]                     # cross mode only
 ```
 Draft: `draft` arg → draft. Else convention `draft: true|false` → use it. Else (`ask` or unset)
 ask open-vs-draft before creating. After a cross-mode success, run
-`git config branch.<CURR>.prMode cross` so the next `/pr` reuses the mode. The binary attaches the
+`git config branch.<CURR>.prMode cross` so the next `/pr` reuses the mode. `pr-commands` attaches the
 per-repo reviewer roster automatically; `--no-reviewers` skips, `--reviewer <id>` appends.
 
-## Step 9 — print final PR URL + reviewer count from binary output. On error, surface raw response.
+## Step 9 — print final PR URL + reviewer count from `pr-commands` output. On error, surface raw response.
 
 ## Step 10 — post-PR skills (guarded, optional).
-For each skill in the convention's `post_pr` (legacy config without `pr.yaml`: `jira-update`):
+For each skill in the convention's `post_pr`:
 `guard_cmd <skill>`; absent → one-line note, skip. Present → `AskUserQuestion`: run now
-(`/<skill> <ticket>` with the Step 4 ticket key) or skip. No ticket key → skip ticket-based skills.
+(`/<skill> <ticket>` with the Step 4 ticket key) or skip. No ticket key, or only a placeholder
+ticket (no digits, e.g. `PROJ-NOTICKET`) → skip ticket-based skills with a one-line note.
 
 ## Update mode (`/pr update [PR#] [short]`)
 
 Rewrites an existing open PR's **description** in place — no new branch, no new PR. Same Step 0
 install check runs first (config + provider + creds). Provider PATCH is currently Bitbucket-only,
-matching the create path; `ghpr`/`glpr` providers → STOP with the same "not yet supported" note.
+matching the create path; any other host → STOP with the same "not yet supported" note.
 
 **U1 — parse args.** Tokens after `update`: a bare integer = `PR#`; the word `short` (or
 `caveman`) = compress-hard flag. Missing `PR#` → resolve from the current branch: query the
 provider for the open PR whose source branch is `HEAD` (Bitbucket: `GET /pullrequests?q=source.branch.name="<CURR>"&state=OPEN`). Zero or multiple matches → ask.
 
-**U2 — fetch current PR.** Resolve creds the same way the binary does — email from `$CFG`
+**U2 — fetch current PR.** Resolve creds the same way `pr-commands` does — email from `$CFG`
 (`email=`), token from the OS keychain (macOS service `bitbucket-api-token`, account = email).
 `REPO` = workspace/slug from `git remote get-url origin`. Fetch title + description:
 ```bash
@@ -216,8 +215,8 @@ curl -sS -u "$EMAIL:$TOKEN" \
   "https://api.bitbucket.org/2.0/repositories/$REPO/pullrequests/$PRNUM"
 ```
 
-**U3 — re-draft the body.** Rebuild the description with the **Step 5** rules (trueid template
-when `convention=trueid`): `What Happened` prose, `Insight` file detail, `Proof of Work`. Prefer
+**U3 — re-draft the body.** Rebuild the description with the **Step 5** rules (`versioned` template
+when `convention=versioned`): `What Happened` prose, `Insight` file detail, `Proof of Work`. Prefer
 the diff/commits as the source of truth over the old text; carry forward Proof-of-Work links the
 old body already had. Default output = terse, anti-slop normal English:
 - Cut throat-clearing openers ("The X feature needs to…", "This adds…"), restated ticket goals,
@@ -258,7 +257,7 @@ absent, degrade gracefully (one-line note). Reusable check (scans Claude command
 agent skills dir):
 ```bash
 guard_cmd() {  # usage: guard_cmd pr-split → prints path or "no-<cmd>"
-  local c="$1"
+  local c="${@:1:1}"   # first arg; avoids the positional form the skill loader rewrites
   ls "$PWD/.claude/commands/$c.md" "$HOME/.claude/commands/$c.md" \
      "${CLAUDE_CONFIG_DIR:-$HOME/.claude}/skills/$c/SKILL.md" \
      "$HOME/.gemini/antigravity/skills/$c/SKILL.md" \

package/install.js

@@ -1,7 +1,8 @@
 #!/usr/bin/env node
 // @nvice-r/pr-commands installer. Zero external deps (Node >=16 builtins only).
 // Wizard: detect OS + provider, pick reviewers + convention, write config/roster,
-// install binaries -> ~/.local/bin, install the portable `pr` SKILL.md into every
+// install the `pr-commands` entry point -> ~/.local/bin (+ shared lib and host providers ->
+// ~/.local/share/pr-commands), install the portable `pr` SKILL.md into every
 // detected agent (symlinked to one canonical copy) + Claude /pr slash-command.
 //
 // Flags: --dir <path>  --provider <p>  --agent <id>  --all  --yes  --dry-run  --help
@@ -21,15 +22,16 @@ const ASSETS = path.join(PKG, 'assets');
 const BACK = Symbol('back');   // returned by a prompt when the user wants the previous step
 
 const PROVIDERS = {
-  bitbucket: { bin: 'bbpr', host: 'bitbucket.org', keychainSvc: 'bitbucket-api-token', ready: true },
-  github:    { bin: 'ghpr', host: 'github.com',    keychainSvc: 'github-api-token',    ready: false },
-  gitlab:    { bin: 'glpr', host: 'gitlab.com',     keychainSvc: 'gitlab-api-token',    ready: false },
+  // provider code = assets/share/pr-commands/providers/<name>.sh
+  bitbucket: { host: 'bitbucket.org', keychainSvc: 'bitbucket-api-token', ready: true },
+  github:    { host: 'github.com',    keychainSvc: 'github-api-token',    ready: false },
+  gitlab:    { host: 'gitlab.com',     keychainSvc: 'gitlab-api-token',    ready: false },
 };
-const CONVENTIONS = ['trueid', 'conventional', 'github-default', 'custom'];
+const CONVENTIONS = ['versioned', 'conventional', 'github-default', 'custom'];
 // Concrete, editable defaults per preset. Whatever the user confirms is written to config,
 // so `custom` is just "a preset with blank seeds" — same edit flow for every choice.
 const PRESET_DEFS = {
-  trueid:           { title_pattern: '[<ver>] Feature/<ver>/SC-XXXX <desc>', ticket_regex: 'SC-\\d+',    template_sections: 'What Happened|Insight|Proof of Work' },
+  versioned:        { title_pattern: '[<ver>] Feature/<ver>/<ticket> <desc>', ticket_regex: '[A-Z]+-\\d+', template_sections: 'What Happened|Insight|Proof of Work' },
   conventional:     { title_pattern: '<type>(<scope>): <desc>',              ticket_regex: '[A-Z]+-\\d+', template_sections: 'Summary|Changes|Testing' },
   'github-default': { title_pattern: '<desc> (#<issue>)',                    ticket_regex: '#\\d+',       template_sections: 'Summary|Test plan' },
   custom:           { title_pattern: '',                                     ticket_regex: '',            template_sections: '' },
@@ -96,8 +98,8 @@ function printHelp() {
                      folder — .claude / .agent / .codex / .opencode)
   --agent <id>       Force skill install to this agent (repeatable). Ids: claude-code,
                      antigravity, codex, opencode. Default: auto-detect installed agents.
-  --all              Install all provider binaries (default: only the detected one)
-  --yes              Accept defaults, no prompts (reviewers=existing roster or all, convention=trueid)
+  --all              Install every ready host provider (default: only the detected one)
+  --yes              Accept defaults, no prompts (reviewers=existing roster or all, convention=versioned)
   --reset-reviewers  Ignore an existing roster and rebuild it from the repo's default reviewers
   --dry-run          Print planned actions, write nothing
   --help             This help`);
@@ -241,7 +243,7 @@ function printLegend() {
   console.log('  Placeholders (used in title_pattern; filled by /pr from the branch):');
   console.log('    \x1b[2m<ver>     branch version    e.g. 2.3.0\x1b[0m');
   console.log('    \x1b[2m<desc>    description       from branch suffix / commits\x1b[0m');
-  console.log('    \x1b[2m<ticket>  ticket key        matched by ticket_regex, e.g. SC-5424\x1b[0m');
+  console.log('    \x1b[2m<ticket>  ticket key        matched by ticket_regex, e.g. PROJ-123\x1b[0m');
   console.log('    \x1b[2m<type>    conventional type feat | fix | chore | ...\x1b[0m');
   console.log('    \x1b[2m<scope>   conventional scope e.g. auth\x1b[0m');
   console.log('    \x1b[2m<issue>   issue number      e.g. 42\x1b[0m');
@@ -345,6 +347,20 @@ function scanGitRepos(root, maxDepth = 3, cap = 50) {
   return found;
 }
 
+// Remove binaries from <= 0.1.3 (bbpr + pr-common.sh in ~/.local/bin). Only files that carry
+// our own version marker are deleted, so an unrelated file with the same name is left alone.
+function removeLegacyBins(binDir) {
+  for (const [name, marker] of [['bbpr', 'bbpr-version'], ['pr-common.sh', 'pr-common-version']]) {
+    const p = path.join(binDir, name);
+    let txt;
+    try { txt = fs.readFileSync(p, 'utf8'); } catch { continue; }
+    if (!txt.includes(marker)) { log(`  kept ${p} (not ours)`); continue; }
+    if (flags.dryRun) { log(`  [dry-run] remove legacy ${p}`); continue; }
+    fs.unlinkSync(p);
+    log(`  removed legacy ${p}`);
+  }
+}
+
 // ---- provider detection ----
 function detectProvider(remoteUrl) {
   if (flags.provider) return flags.provider;
@@ -426,7 +442,7 @@ async function fetchBitbucketReviewers(slug, email, token, selfUuid) {
     const L = [];
     if (stage > 0 && st.workDir) {
       L.push(`  \x1b[32m✓\x1b[0m work dir    ${st.workDir}`);
-      L.push(`  \x1b[32m✓\x1b[0m provider    ${st.provider} (${st.slug}) → ${st.pcfg.bin}`);
+      L.push(`  \x1b[32m✓\x1b[0m provider    ${st.provider} (${st.slug})`);
       L.push(`  \x1b[32m✓\x1b[0m auth        ${st.email || '(none)'}${st.token ? ' · token ok' : ' · no token'}`);
     }
     if (stage > 1 && st.chosen) L.push(`  \x1b[32m✓\x1b[0m reviewers   ${st.chosen.length} selected`);
@@ -481,7 +497,7 @@ async function fetchBitbucketReviewers(slug, email, token, selfUuid) {
     if (!provider) { console.error(`  ERROR: could not detect provider from ${remoteUrl}`); process.exit(1); }
     const pcfg = PROVIDERS[provider];
     const slug = parseSlug(remoteUrl);
-    log(`  remote: ${remoteUrl}\n  provider=${provider}  slug=${slug}  binary=${pcfg.bin}`);
+    log(`  remote: ${remoteUrl}\n  provider=${provider}  slug=${slug}`);
     if (!pcfg.ready) { console.error(`  ERROR: ${provider} not yet supported (macOS+Bitbucket only). Aborting.`); process.exit(1); }
     // identity + token (derived from provider)
     let email = '';
@@ -534,11 +550,11 @@ async function fetchBitbucketReviewers(slug, email, token, selfUuid) {
   const sPreset = async () => {
     screen(2, 'Convention — preset');
     if (flags.yes) {
-      st.convention = st.convention || 'trueid';
+      st.convention = PRESET_DEFS[st.convention] ? st.convention : 'versioned';
       st.conv = { ...PRESET_DEFS[st.convention] };
       return;
     }
-    const idx = Math.max(0, CONVENTIONS.indexOf(st.convention || 'trueid'));
+    const idx = Math.max(0, CONVENTIONS.indexOf(st.convention || 'versioned'));
     const pick = await multiSelectOne('Convention preset:', CONVENTIONS, idx);
     if (pick === BACK) return BACK;
     const chosenConv = CONVENTIONS[pick];
@@ -592,7 +608,7 @@ async function fetchBitbucketReviewers(slug, email, token, selfUuid) {
 
   // --- install ---
   screen(4, 'Install');
-  const { slug, pcfg, email, convention, conv, chosen } = st;
+  const { slug, email, convention, conv, chosen } = st;
   // Roster folder = --agent-dir, else the first chosen agent's repo folder, else .claude.
   const agentIds = st.agents || detectedAgentIds();
   const primary = agentTargets().find((t) => t.id === agentIds[0]);
@@ -615,16 +631,21 @@ async function fetchBitbucketReviewers(slug, email, token, selfUuid) {
     const header = `# pr reviewers for ${slug} — {uuid}  # Name per line; blanks/# ignored. Edit freely.\n`;
     writeFile(rosterFile, header + (roster || '# (none selected)\n'));
   }
+  // One entry point on PATH; shared lib + host providers under ~/.local/share/pr-commands.
   const binDir = path.join(HOME, '.local/bin');
-  const binsToInstall = flags.all
-    ? Object.values(PROVIDERS).filter((p) => p.ready).map((p) => p.bin)
-    : [pcfg.bin];
-  writeFile(path.join(binDir, 'pr-common.sh'), fs.readFileSync(path.join(ASSETS, 'bin/pr-common.sh')), 0o755);
-  for (const b of binsToInstall) {
-    const src = path.join(ASSETS, 'bin', b);
-    if (!fs.existsSync(src)) { log(`  skip ${b} (asset not present yet)`); continue; }
-    writeFile(path.join(binDir, b), fs.readFileSync(src), 0o755);
+  const shareDir = path.join(HOME, '.local/share/pr-commands');
+  const shareSrc = path.join(ASSETS, 'share/pr-commands');
+  writeFile(path.join(binDir, 'pr-commands'), fs.readFileSync(path.join(ASSETS, 'bin/pr-commands')), 0o755);
+  writeFile(path.join(shareDir, 'pr-common.sh'), fs.readFileSync(path.join(shareSrc, 'pr-common.sh')), 0o644);
+  const provNames = flags.all
+    ? Object.keys(PROVIDERS).filter((n) => PROVIDERS[n].ready)
+    : [st.provider];
+  for (const n of provNames) {
+    const src = path.join(shareSrc, 'providers', `${n}.sh`);
+    if (!fs.existsSync(src)) { log(`  skip ${n} (provider not present yet)`); continue; }
+    writeFile(path.join(shareDir, 'providers', `${n}.sh`), fs.readFileSync(src), 0o644);
   }
+  removeLegacyBins(binDir);
   installSkill(agentIds);
   if (!(process.env.PATH || '').split(':').includes(binDir)) {
     log(`\n  WARN: ${binDir} not on PATH. Add to your shell rc:`);

package/package.json

@@ -1,9 +1,9 @@
 {
   "name": "@nvice-r/pr-commands",
-  "version": "0.1.3",
-  "description": "Provider-agnostic PR helper for Claude Code — npx installer + bbpr/ghpr/glpr binaries + /pr skill",
+  "version": "0.1.5",
+  "description": "Provider-agnostic PR helper for Claude Code — npx installer + one `pr-commands` command (per-host providers) + /pr skill",
   "bin": {
-    "pr-commands": "install.js"
+    "pr-commands-setup": "install.js"
   },
   "files": [
     "install.js",

package/assets/bin/bbpr

@@ -1,81 +0,0 @@
-#!/usr/bin/env bash
-# bbpr-version: 9
-# bbpr — create a Bitbucket Cloud pull request (REST API v2.0).
-# Provider-specific layer; all git/config/creds/reviewer logic lives in pr-common.sh.
-# Usage: bbpr --title "..." [--body|--body-file] [--source b] [--dest b] [--repo w/s]
-#             [--draft] [--reviewer uuid]... [--no-reviewers] [--email a] [--close-source]
-#             [--source-repo ws/fork]   (cross-repo: PR opens in --repo/origin, from the fork)
-
-set -euo pipefail
-
-PR_CMD="bbpr"
-KEYCHAIN_SERVICE="bitbucket-api-token"
-
-# locate + source the shared lib (same dir as this binary)
-_here="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
-# shellcheck source=pr-common.sh
-. "$_here/pr-common.sh"
-
-pr_common_init "$@"
-
-# --- Bitbucket: drop the PR author's own UUID (BB rejects author-as-reviewer) ---
-if [ "${#FINAL_REVIEWERS[@]}" -gt 0 ]; then
-  SELF_UUID="$(curl -sS -u "${EMAIL}:${TOKEN}" "https://api.bitbucket.org/2.0/user" 2>/dev/null \
-    | python3 -c 'import json,sys
-try: print(json.load(sys.stdin).get("uuid",""))
-except Exception: print("")' 2>/dev/null || true)"
-  if [ -n "$SELF_UUID" ]; then
-    FILTERED=()
-    for r in "${FINAL_REVIEWERS[@]}"; do
-      [ "$r" = "$SELF_UUID" ] || FILTERED+=("$r")
-    done
-    FINAL_REVIEWERS=("${FILTERED[@]:-}")
-    [ -z "${FINAL_REVIEWERS[0]:-}" ] && FINAL_REVIEWERS=()
-  fi
-fi
-
-# --- Bitbucket payload (source/destination keys, draft, reviewers[]) ---
-PAYLOAD="$(
-  TITLE="$TITLE" BODY="$BODY" SOURCE="$SOURCE" DEST="$DEST" SOURCE_REPO="$SOURCE_REPO" \
-  DRAFT="$DRAFT" CLOSE_SOURCE="$CLOSE_SOURCE" \
-  REVIEWERS_JSON="$(printf '%s\n' "${FINAL_REVIEWERS[@]:-}" | python3 -c 'import json,sys; print(json.dumps([{"uuid": l.strip()} for l in sys.stdin if l.strip()]))')" \
-  python3 -c '
-import json, os
-p = {
-  "title": os.environ["TITLE"],
-  "description": os.environ["BODY"],
-  "source": {"branch": {"name": os.environ["SOURCE"]}},
-  "destination": {"branch": {"name": os.environ["DEST"]}},
-  "draft": os.environ["DRAFT"] == "true",
-  "close_source_branch": os.environ["CLOSE_SOURCE"] == "true",
-}
-if os.environ["SOURCE_REPO"]:
-  p["source"]["repository"] = {"full_name": os.environ["SOURCE_REPO"]}
-revs = json.loads(os.environ["REVIEWERS_JSON"])
-if revs: p["reviewers"] = revs
-print(json.dumps(p))
-'
-)"
-
-URL="https://api.bitbucket.org/2.0/repositories/${REPO}/pullrequests"
-echo "POST $URL"
-echo "source=${SOURCE_REPO:+$SOURCE_REPO:}$SOURCE  dest=$DEST  draft=$DRAFT  reviewers=${#FINAL_REVIEWERS[@]}"
-
-RESP="$(curl -sS -X POST "$URL" -u "${EMAIL}:${TOKEN}" \
-  -H 'Content-Type: application/json' -d "$PAYLOAD")"
-
-PR_INFO="$(echo "$RESP" | python3 -c '
-import json, sys
-try: d = json.loads(sys.stdin.read())
-except Exception as e: print("non-JSON response", e); sys.exit(2)
-if "error" in d: print("ERROR:", json.dumps(d["error"], indent=2)); sys.exit(1)
-print(str(d.get("id")) + "\t" + str(d.get("links", {}).get("html", {}).get("href")))
-')" || { echo "$PR_INFO" >&2; exit 1; }   # surface the API error; set -e would swallow it
-
-NEW_PR_ID="$(echo "$PR_INFO" | cut -f1)"
-NEW_PR_URL="$(echo "$PR_INFO" | cut -f2)"
-ATTACHED="$(echo "$RESP" | python3 -c 'import json,sys; print(len(json.load(sys.stdin).get("reviewers", [])))')"
-echo "created PR #${NEW_PR_ID} with ${ATTACHED} reviewers"
-echo "$NEW_PR_URL"
-
-[ -n "$NEW_PR_ID" ] && pr_save_base