@event4u/agent-config 5.10.0 → 6.0.0

package/scripts/release.py

@@ -1,1049 +0,0 @@
-#!/usr/bin/env python3
-"""End-to-end release automation for `event4u/agent-config`.
-
-Invoked via `task release`. The bump level (major/minor/patch) is
-auto-detected from Conventional Commits since the last tag; pass
-`--as {major,minor,patch}` to force, or `--version X.Y.Z` to pin.
-
-Pipeline:
-    1. Preflight         — on main, clean tree, origin in sync, gh available,
-                           target tag doesn't exist yet.
-    2. Plan              — compute new version, parse Conventional Commits
-                           since the last tag, render CHANGELOG section.
-    3. Confirm           — show preview, ask once (skippable with --yes).
-    4. Branch + bump     — create `release/X.Y.Z`, update package.json,
-                           .claude-plugin/marketplace.json, CHANGELOG.md,
-                           then run `task release-prepare` so pack
-                           manifests and tool projections pick up the
-                           new version (otherwise the PR's own consistency
-                           check fails — see PR #226 post-mortem).
-    5. Commit + push     — `release: X.Y.Z`, push branch, open PR.
-    6. Wait for CI       — `gh pr checks --watch` (skippable with --no-wait).
-    7. Merge             — `gh pr merge --merge --delete-branch`.
-    8. Tag main          — fast-forward main, tag the merge commit,
-                           push the tag (this triggers publish-npm.yml).
-    9. GitHub Release    — `gh release create X.Y.Z --notes <changelog>`.
-
-Idempotency: pass `--resume` to recover from a partial failure. Each
-step then probes existing state (branch, commit, PR, tag, GitHub
-Release) and skips work that is already done, instead of erroring out.
-Without `--resume` the pipeline still mutates git/network state, so
-re-running on a dirty tree needs `--resume` (or a manual cleanup).
-Each step prints what it's about to do before doing it, so a crash
-leaves a recoverable trail.
-
-Stdlib-only (Python 3.10+). No third-party runtime dependencies.
-
-See also:
-    - docs/contracts/release-pr-gating.md — release-PR shape, cut surface,
-      kept surface, fail-closed contract.
-    - docs/contracts/branch-protection-policy.md — per-PR-shape
-      required-check matrix; `task ci:required-checks` previews it.
-    - docs/contracts/ci-cost-budget.md — measured baselines + quarterly
-      review cadence.
-    - .github/workflows/release-validation.yml — the tight release-PR
-      validation jobs (release-shape, changelog-entry, version-consistency).
-"""
-
-from __future__ import annotations
-
-import argparse
-import json
-import re
-import subprocess
-import sys
-import time
-from dataclasses import dataclass
-from datetime import date as _date
-from pathlib import Path
-
-from _lib.changelog_eras import (
-    CURRENT_ERA_BODY_CAP,
-    SplitPlan,
-    current_era_body_size,
-    current_era_insertion_point,
-    perform_split,
-    plan_split,
-    read_changelog_lines,
-)
-
-REPO_ROOT = Path(__file__).resolve().parent.parent
-PACKAGE_JSON = REPO_ROOT / "package.json"
-MARKETPLACE_JSON = REPO_ROOT / ".claude-plugin" / "marketplace.json"
-CHANGELOG = REPO_ROOT / "CHANGELOG.md"
-MAIN_BRANCH = "main"
-REMOTE = "origin"
-REPO_SLUG = "event4u-app/agent-config"
-
-# Conventional Commit types and how they map into CHANGELOG sections.
-# Order in this tuple determines order in the rendered entry.
-SECTIONS: tuple[tuple[str, str, tuple[str, ...]], ...] = (
-    ("Features", "minor", ("feat",)),
-    ("Bug Fixes", "patch", ("fix",)),
-    ("Performance", "patch", ("perf",)),
-    ("Reverts", "patch", ("revert",)),
-    ("Documentation", None, ("docs",)),
-    ("Refactoring", None, ("refactor",)),
-    ("Tests", None, ("test",)),
-    ("Build", None, ("build",)),
-    ("CI", None, ("ci",)),
-    ("Chores", None, ("chore",)),
-)
-BREAKING_RE = re.compile(r"^([a-z]+)(\([^)]+\))?!:")
-CONVENTIONAL_RE = re.compile(
-    r"^(?P<type>[a-z]+)(?:\((?P<scope>[^)]+)\))?(?P<bang>!)?: (?P<subject>.+)$"
-)
-
-
-# ─── dataclasses ──────────────────────────────────────────────────────────────
-
-
-@dataclass(frozen=True)
-class Commit:
-    sha: str
-    type: str
-    scope: str | None
-    subject: str
-    breaking: bool
-
-
-@dataclass(frozen=True)
-class Plan:
-    current: str
-    target: str
-    bump: str  # "major" | "minor" | "patch"
-    commits: list[Commit]
-    last_tag: str | None
-    changelog_body: str  # rendered body (without the heading)
-    changelog_entry: str  # full entry including heading, for CHANGELOG.md
-    # Populated only when the release crosses an era boundary AND the
-    # current era body has grown past CURRENT_ERA_BODY_CAP. None for
-    # patch releases and for minor/major bumps where the era still fits.
-    split_plan: SplitPlan | None = None
-
-
-# ─── utilities ────────────────────────────────────────────────────────────────
-
-
-def die(msg: str, code: int = 2) -> None:
-    print(f"error: {msg}", file=sys.stderr)
-    sys.exit(code)
-
-
-def run(
-    *args: str,
-    check: bool = True,
-    capture: bool = False,
-    cwd: Path | None = None,
-) -> subprocess.CompletedProcess[str]:
-    """Thin subprocess wrapper with sane defaults.
-
-    When ``check`` and ``capture`` are both True and the command fails,
-    Python's default behaviour swallows stderr — callers only see a
-    ``CalledProcessError`` with no hint of what went wrong. We catch that
-    path and die with the actual stderr so release preflight failures are
-    diagnosable without re-running with a debugger.
-    """
-    try:
-        return subprocess.run(
-            list(args),
-            check=check,
-            cwd=cwd or REPO_ROOT,
-            text=True,
-            capture_output=capture,
-        )
-    except subprocess.CalledProcessError as err:
-        if capture:
-            cmd = " ".join(args)
-            out = (err.stderr or err.stdout or "").strip()
-            die(f"command failed ({err.returncode}): {cmd}\n{out}")
-        raise
-
-
-def git(*args: str, capture: bool = False) -> str:
-    r = run("git", *args, capture=capture)
-    return r.stdout.strip() if capture else ""
-
-
-def gh(*args: str, capture: bool = False, check: bool = True) -> str:
-    r = run("gh", *args, capture=capture, check=check)
-    return r.stdout.strip() if capture else ""
-
-
-def watch_pr_checks() -> None:
-    """Watch PR checks and tolerate the 'no checks' case.
-
-    ``gh pr checks --watch`` exits 1 both on real failures and when no
-    checks are reported at all (no workflow triggered, no required
-    checks configured in branch protection). The latter must not block
-    the release — we warn and continue. Real failures still die.
-
-    A short grace period gives GitHub time to register workflow runs
-    on a freshly-pushed branch.
-    """
-    time.sleep(5)
-    proc = subprocess.run(
-        ["gh", "pr", "checks", "--watch"],
-        cwd=REPO_ROOT,
-        text=True,
-        capture_output=True,
-    )
-    output = ((proc.stdout or "") + (proc.stderr or "")).strip()
-    if proc.returncode == 0:
-        if output:
-            print(output)
-        return
-    if "no checks reported" in output.lower():
-        print(f"⚠️  {output}")
-        print(
-            "   Continuing without check validation — configure required "
-            "checks in branch protection to enforce this gate."
-        )
-        return
-    if output:
-        print(output, file=sys.stderr)
-    die(f"PR checks failed (exit {proc.returncode})")
-
-
-def have(bin: str) -> bool:
-    return (
-        subprocess.run(
-            ["which", bin], capture_output=True, text=True
-        ).returncode
-        == 0
-    )
-
-
-# ─── resume-mode state probes ────────────────────────────────────────────────
-
-
-def _branch_exists_local(branch: str) -> bool:
-    r = run(
-        "git", "rev-parse", "--verify", "--quiet", f"refs/heads/{branch}",
-        check=False, capture=True,
-    )
-    return r.returncode == 0
-
-
-def _branch_exists_remote(branch: str) -> bool:
-    r = run(
-        "git", "ls-remote", "--exit-code", "--heads", REMOTE, branch,
-        check=False, capture=True,
-    )
-    return r.returncode == 0
-
-
-def _tag_exists_local(tag: str) -> bool:
-    return tag in git("tag", "-l", tag, capture=True).splitlines()
-
-
-def _tag_exists_remote(tag: str) -> bool:
-    r = run(
-        "git", "ls-remote", "--exit-code", "--tags", REMOTE, tag,
-        check=False, capture=True,
-    )
-    return r.returncode == 0
-
-
-def _pr_for_branch(branch: str) -> dict | None:
-    """Most recent PR (any state) with `release/X.Y.Z` as head, or None."""
-    r = run(
-        "gh", "pr", "list",
-        "--head", branch,
-        "--state", "all",
-        "--json", "number,state,url",
-        "--limit", "1",
-        check=False, capture=True,
-    )
-    if r.returncode != 0:
-        return None
-    try:
-        items = json.loads(r.stdout or "[]")
-    except json.JSONDecodeError:
-        return None
-    return items[0] if items else None
-
-
-def _release_exists(tag: str) -> bool:
-    r = run("gh", "release", "view", tag, check=False, capture=True)
-    return r.returncode == 0
-
-
-# ─── version math ─────────────────────────────────────────────────────────────
-
-
-SEMVER_RE = re.compile(r"^(\d+)\.(\d+)\.(\d+)$")
-
-
-def parse_version(s: str) -> tuple[int, int, int]:
-    m = SEMVER_RE.match(s.strip())
-    if not m:
-        die(f"not a bare semver (X.Y.Z): {s!r}")
-    return int(m.group(1)), int(m.group(2)), int(m.group(3))
-
-
-def bump_version(current: str, kind: str) -> str:
-    major, minor, patch = parse_version(current)
-    if kind == "major":
-        return f"{major + 1}.0.0"
-    if kind == "minor":
-        return f"{major}.{minor + 1}.0"
-    if kind == "patch":
-        return f"{major}.{minor}.{patch + 1}"
-    die(f"unknown bump kind: {kind}")
-    return ""  # unreachable
-
-
-# ─── commit parsing + changelog rendering ────────────────────────────────────
-
-
-def commits_since(tag: str | None) -> list[Commit]:
-    """Return non-merge commits after `tag` (or all of history if tag is None)."""
-    rev = f"{tag}..HEAD" if tag else "HEAD"
-    raw = git("log", rev, "--no-merges", "--format=%H%x1f%s", capture=True)
-    out: list[Commit] = []
-    for line in raw.splitlines():
-        if "\x1f" not in line:
-            continue
-        sha, subject = line.split("\x1f", 1)
-        m = CONVENTIONAL_RE.match(subject)
-        if not m:
-            out.append(Commit(sha, "other", None, subject, False))
-            continue
-        breaking = bool(m.group("bang")) or "BREAKING CHANGE" in subject
-        out.append(
-            Commit(
-                sha=sha,
-                type=m.group("type"),
-                scope=m.group("scope"),
-                subject=m.group("subject"),
-                breaking=breaking,
-            )
-        )
-    return out
-
-
-def infer_bump(commits: list[Commit]) -> str:
-    """Derive the semver bump from commit types (for preview only)."""
-    if any(c.breaking for c in commits):
-        return "major"
-    for _label, level, types in SECTIONS:
-        if level == "minor" and any(c.type in types for c in commits):
-            return "minor"
-    return "patch"
-
-
-def latest_tag() -> str | None:
-    r = run(
-        "git", "describe", "--tags", "--abbrev=0", "--match", "[0-9]*.[0-9]*.[0-9]*",
-        check=False, capture=True,
-    )
-    tag = r.stdout.strip()
-    return tag or None
-
-
-def render_changelog_entry(
-    version: str,
-    prev: str | None,
-    commits: list[Commit],
-    today: str,
-    *,
-    test_trend_line: str | None = None,
-) -> tuple[str, str]:
-    """Return (heading-aware full entry, body-only for GitHub Release notes).
-
-    ``test_trend_line`` — optional pre-computed ``Tests: N (+M …)`` footer
-    (road-to-feedback-followups P3.2). Computed by the caller so tests
-    don't trigger a recursive pytest collection.
-    """
-    if prev:
-        heading = (
-            f"## [{version}](https://github.com/{REPO_SLUG}/compare/"
-            f"{prev}...{version}) ({today})"
-        )
-    else:
-        heading = f"## {version} ({today})"
-
-    # Group by section; commits of unknown type drop into "Other".
-    grouped: dict[str, list[Commit]] = {label: [] for label, _, _ in SECTIONS}
-    grouped["BREAKING CHANGES"] = []
-    other: list[Commit] = []
-    for c in commits:
-        if c.breaking:
-            grouped["BREAKING CHANGES"].append(c)
-            continue
-        placed = False
-        for label, _level, types in SECTIONS:
-            if c.type in types:
-                grouped[label].append(c)
-                placed = True
-                break
-        if not placed:
-            other.append(c)
-
-    body_lines: list[str] = []
-    ordered_labels = ["BREAKING CHANGES"] + [label for label, _, _ in SECTIONS]
-    for label in ordered_labels:
-        bucket = grouped.get(label) or []
-        if not bucket:
-            continue
-        body_lines.append("")
-        body_lines.append(f"### {label}")
-        body_lines.append("")
-        for c in bucket:
-            body_lines.append(_changelog_line(c))
-    if other:
-        body_lines.append("")
-        body_lines.append("### Other")
-        body_lines.append("")
-        for c in other:
-            body_lines.append(_changelog_line(c))
-
-    # Test-count trend footer (road-to-feedback-followups P3.2). Silent
-    # on errors — never a release blocker.
-    if test_trend_line:
-        body_lines.append("")
-        body_lines.append(test_trend_line)
-
-    body = "\n".join(body_lines).lstrip("\n")
-    full = heading + "\n\n" + body + "\n"
-    return full, body
-
-
-def _changelog_line(c: Commit) -> str:
-    scope = f"**{c.scope}:** " if c.scope else ""
-    short = c.sha[:7]
-    link = f"https://github.com/{REPO_SLUG}/commit/{c.sha}"
-    return f"* {scope}{c.subject} ([{short}]({link}))"
-
-
-# ─── test-count trend (road-to-feedback-followups P3.2) ───────────────────────
-
-_TEST_COUNT_LINE_RE = re.compile(r"^Tests:\s+(\d+)", re.MULTILINE)
-_PYTEST_COLLECTED_RE = re.compile(r"^(\d+)\s+tests?\s+collected", re.MULTILINE)
-
-
-def _count_tests_current() -> int | None:
-    """Return the count from `pytest --collect-only -q` on the current
-    tree. Returns None when pytest isn't available or collection fails —
-    the trend line is informational, never a release blocker.
-    """
-    try:
-        result = subprocess.run(
-            ["python3", "-m", "pytest", "--collect-only", "-q"],
-            cwd=str(REPO_ROOT),
-            capture_output=True,
-            text=True,
-            timeout=120,
-        )
-    except (FileNotFoundError, subprocess.TimeoutExpired):
-        return None
-    if result.returncode != 0:
-        return None
-    match = _PYTEST_COLLECTED_RE.search(result.stdout)
-    return int(match.group(1)) if match else None
-
-
-def _previous_test_count_from_changelog(prev_tag: str | None) -> int | None:
-    """Read CHANGELOG.md and return the most recent ``Tests: N`` footer
-    under the ``prev_tag`` heading, or None when not found.
-    """
-    if not prev_tag or not CHANGELOG.exists():
-        return None
-    text = CHANGELOG.read_text(encoding="utf-8")
-    heading_re = re.compile(rf"^##\s+\[?{re.escape(prev_tag)}\b", re.MULTILINE)
-    m = heading_re.search(text)
-    if not m:
-        return None
-    next_heading = re.search(r"^##\s+\[?\d+\.\d+\.\d+", text[m.end():], re.MULTILINE)
-    section = text[m.end(): m.end() + (next_heading.start() if next_heading else len(text))]
-    count_match = _TEST_COUNT_LINE_RE.search(section)
-    return int(count_match.group(1)) if count_match else None
-
-
-def _render_test_trend_line(prev_tag: str | None) -> str | None:
-    """Return the ``Tests: N (+M since X.Y.Z)`` footer line, or None when
-    the current count cannot be determined. Silent on collection errors.
-    """
-    current = _count_tests_current()
-    if current is None:
-        return None
-    previous = _previous_test_count_from_changelog(prev_tag)
-    if previous is None or not prev_tag:
-        return f"Tests: {current}"
-    delta = current - previous
-    sign = "+" if delta >= 0 else ""
-    return f"Tests: {current} ({sign}{delta} since {prev_tag})"
-
-
-def prepend_changelog(path: Path, entry: str) -> None:
-    """Insert ``entry`` inside the current era block.
-
-    Strategy delegates to ``current_era_insertion_point`` so a fresh
-    era (no version headings yet, just the intro blockquote) places the
-    new entry after the intro instead of appended at end-of-file. When
-    no current era header exists, falls back to the legacy "above the
-    most recent ## [" heuristic for safety.
-    """
-    text = path.read_text(encoding="utf-8")
-    lines = text.splitlines()
-    insert_at = current_era_insertion_point(lines)
-    if insert_at is not None:
-        before = "\n".join(lines[:insert_at])
-        after = "\n".join(lines[insert_at:])
-        head = before + ("\n" if before else "")
-        path.write_text(head + entry + "\n" + after + "\n", encoding="utf-8")
-        return
-
-    # Legacy fallback — no era header present at all.
-    marker_re = re.compile(r"^## \[?\d+\.\d+\.\d+", re.MULTILINE)
-    m = marker_re.search(text)
-    if not m:
-        path.write_text(text.rstrip() + "\n\n" + entry, encoding="utf-8")
-        return
-    before = text[: m.start()]
-    after = text[m.start():]
-    path.write_text(before + entry + "\n" + after, encoding="utf-8")
-
-
-
-# ─── file mutations ───────────────────────────────────────────────────────────
-
-
-def set_package_version(path: Path, version: str) -> None:
-    """Update the top-level `version` field; preserve 4-space indentation."""
-    data = json.loads(path.read_text(encoding="utf-8"))
-    data["version"] = version
-    path.write_text(json.dumps(data, indent=4) + "\n", encoding="utf-8")
-
-
-def set_marketplace_version(path: Path, version: str) -> None:
-    """Update `metadata.version`; preserve 2-space indentation + UTF-8."""
-    data = json.loads(path.read_text(encoding="utf-8"))
-    data.setdefault("metadata", {})["version"] = version
-    path.write_text(json.dumps(data, indent=2) + "\n", encoding="utf-8")
-
-
-# ─── preflight ────────────────────────────────────────────────────────────────
-
-
-def preflight(target: str, *, resume: bool = False) -> None:
-    """Fail fast on conditions that would break the release mid-flight.
-
-    In ``--resume`` mode two invariants are relaxed:
-
-    * The starting branch may be ``release/{target}`` in addition to
-      ``main`` — both are valid resume positions (mid-pipeline crash
-      after step 1 leaves you on the release branch).
-    * The target-tag-exists check is dropped — execute() probes for
-      existing tags/releases and skips them.
-
-    Tree cleanliness, gh auth, and ``main`` in-sync with origin are
-    still enforced, so resuming has the same starting posture as a
-    fresh run; only step-level outcomes differ.
-    """
-    for b in ("git", "gh"):
-        if not have(b):
-            die(f"{b!r} not found on PATH")
-
-    # Probe the active token directly via an authenticated API call. `gh auth
-    # status` returns non-zero if *any* account in the keyring is broken, even
-    # when the active one is fine — so we'd rather ask "does the token the
-    # release will actually use work?" than parse multi-account status output.
-    r = run("gh", "api", "user", "--jq", ".login", check=False, capture=True)
-    if r.returncode != 0:
-        die("gh is not authenticated; run `gh auth login` first")
-
-    branch = git("rev-parse", "--abbrev-ref", "HEAD", capture=True)
-    release_branch = f"release/{target}"
-    allowed = {MAIN_BRANCH, release_branch} if resume else {MAIN_BRANCH}
-    if branch not in allowed:
-        if resume:
-            die(
-                f"resume must run from {MAIN_BRANCH!r} or {release_branch!r}, "
-                f"currently on {branch!r}"
-            )
-        die(f"release must run from {MAIN_BRANCH!r}, currently on {branch!r}")
-
-    porcelain = git("status", "--porcelain", capture=True)
-    if porcelain:
-        die("working tree is not clean; commit or stash first")
-
-    # --force lets the remote's tag positions win over stale local tags.
-    # The release consumes the remote view as source of truth, and we're
-    # about to create a new tag anyway — local drift (e.g. from renamed
-    # release-please tags) should not block the fetch.
-    run("git", "fetch", REMOTE, "--tags", "--prune", "--force", capture=True)
-
-    # The local-in-sync-with-origin check only applies to main; if we're
-    # already on the release branch in resume mode, the relevant invariant
-    # is "main hasn't moved beyond what release/X.Y.Z branched off", which
-    # `git pull --ff-only` enforces in step 8 anyway.
-    if branch == MAIN_BRANCH:
-        local = git("rev-parse", "HEAD", capture=True)
-        remote = git("rev-parse", f"{REMOTE}/{MAIN_BRANCH}", capture=True)
-        if local != remote:
-            die(
-                f"local {MAIN_BRANCH} is not in sync with "
-                f"{REMOTE}/{MAIN_BRANCH}; pull or push first"
-            )
-
-    if not resume:
-        tags = git("tag", "-l", target, capture=True).splitlines()
-        if target in tags:
-            die(f"tag {target!r} already exists; nothing to release")
-
-
-# ─── plan ─────────────────────────────────────────────────────────────────────
-
-
-def print_preview(plan: Plan) -> None:
-    print()
-    print("═" * 72)
-    print(f"  Release preview — {plan.current} → {plan.target} ({plan.bump})")
-    print("═" * 72)
-    print()
-    print(f"Previous tag:   {plan.last_tag or '(none)'}")
-    print(f"New tag:        {plan.target}")
-    print(f"Commits:        {len(plan.commits)} since {plan.last_tag or 'start'}")
-    detected = infer_bump(plan.commits) if plan.commits else "patch"
-    if detected != plan.bump:
-        print(
-            f"NOTE:           commits suggest a {detected!r} bump, "
-            f"you picked {plan.bump!r}"
-        )
-    print()
-    print("Files to change:")
-    print(f"  · {PACKAGE_JSON.relative_to(REPO_ROOT)}")
-    print(f"  · {MARKETPLACE_JSON.relative_to(REPO_ROOT)}")
-    print(f"  · {CHANGELOG.relative_to(REPO_ROOT)}")
-    print("  · regenerated derived files via `task release-prepare`")
-    print("    (packages/*/pack.yaml + README.md, .agent-src/, tool projections)")
-    if plan.split_plan is not None:
-        sp = plan.split_plan
-        print()
-        print("Era split (separate commit, before release commit):")
-        print(f"  · archive   → {sp.archive_path.relative_to(REPO_ROOT)}")
-        print(f"  · old era   → pre-{sp.boundary} (archived pointer)")
-        print(f"  · new era   → {sp.new_era_label} — current (empty body)")
-        print(f"  · subject   → {sp.commit_subject}")
-    print()
-    print("Changelog section:")
-    print("─" * 72)
-    print(plan.changelog_entry.rstrip())
-    print("─" * 72)
-    print()
-    print("Release-PR CI shape (docs/contracts/release-pr-gating.md):")
-    print(
-        "  will run: Consistency · Smoke Contracts · Migration Dry-Run · "
-        "Release Validation · Release Guard (post-tag, ~30 s)"
-    )
-    print(
-        "  will skip: Tests (install / aux / python / node / windows-lockfile-export) · "
-        "Public Install Smoke — heavy install matrices cannot be regressed by a release-shape diff"
-    )
-    print()
-
-
-def confirm(prompt: str) -> bool:
-    ans = input(f"{prompt} [y/N] ").strip().lower()
-    return ans in {"y", "yes"}
-
-
-# ─── orchestration ────────────────────────────────────────────────────────────
-
-
-def _step(n: int, total: int, msg: str) -> None:
-    print(f"[{n}/{total}] {msg}")
-
-
-def execute(
-    plan: Plan,
-    *,
-    wait_for_checks: bool,
-    dry_run: bool,
-    resume: bool = False,
-) -> None:
-    branch = f"release/{plan.target}"
-    total = 10
-
-    if dry_run:
-        print("(dry-run) no git/gh mutations will be performed.")
-        return
-
-    # Probe the world once at the top so each step skip-decision is cheap.
-    pr_info = _pr_for_branch(branch) if resume else None
-    pr_state = (pr_info or {}).get("state")
-    pr_merged = pr_state == "MERGED"
-
-    # ─── 1. branch ──────────────────────────────────────────────────────────
-    if pr_merged:
-        _step(1, total, f"PR for {branch} already merged — staying on {MAIN_BRANCH}")
-        if git("rev-parse", "--abbrev-ref", "HEAD", capture=True) != MAIN_BRANCH:
-            run("git", "checkout", MAIN_BRANCH)
-        run("git", "pull", "--ff-only", REMOTE, MAIN_BRANCH)
-    elif resume and _branch_exists_local(branch):
-        _step(1, total, f"Branch {branch} exists locally — checkout")
-        run("git", "checkout", branch)
-    elif resume and _branch_exists_remote(branch):
-        _step(1, total, f"Branch {branch} exists on {REMOTE} — fetch + checkout")
-        run("git", "fetch", REMOTE, branch)
-        run("git", "checkout", "-b", branch, f"{REMOTE}/{branch}")
-    else:
-        _step(1, total, f"Create branch {branch}")
-        run("git", "checkout", "-b", branch)
-
-    # ─── 1b. era split (optional, separate commit) ─────────────────────────
-    # Lands as `chore(changelog): split era ...` BEFORE the release commit
-    # so the split is reviewable on its own and the release commit only
-    # touches the bump + new entry. Idempotent: archive already on disk
-    # OR a prior split commit on the branch is treated as already done.
-    if plan.split_plan is not None and not pr_merged:
-        sp = plan.split_plan
-        split_already_committed = (
-            sp.commit_subject
-            in git("log", f"{MAIN_BRANCH}..HEAD", "--format=%s", capture=True)
-            .splitlines()
-        )
-        if sp.archive_path.exists() and split_already_committed:
-            _step(
-                1, total,
-                f"Era split for pre-{sp.boundary} already committed — skip",
-            )
-        elif sp.archive_path.exists() and not split_already_committed:
-            die(
-                f"era archive {sp.archive_path.relative_to(REPO_ROOT)} exists "
-                "but no matching split commit found on this branch — inspect "
-                "manually before resuming"
-            )
-        else:
-            _step(
-                1, total,
-                f"Split era {sp.old_era_label} → pre-{sp.boundary} "
-                f"(new era {sp.new_era_label})",
-            )
-            perform_split(sp)
-            run("git", "add", "-A")
-            run("git", "commit", "-m", sp.commit_subject)
-
-    # ─── 2. file mutations ──────────────────────────────────────────────────
-    if pr_merged:
-        _step(2, total, "PR already merged — skip file bumps")
-    else:
-        current_pkg = json.loads(PACKAGE_JSON.read_text(encoding="utf-8")).get("version")
-        if resume and current_pkg == plan.target:
-            _step(2, total, f"Files already at {plan.target} — skip bump")
-        else:
-            _step(2, total, "Bump package.json + marketplace.json, prepend CHANGELOG")
-            set_package_version(PACKAGE_JSON, plan.target)
-            set_marketplace_version(MARKETPLACE_JSON, plan.target)
-            prepend_changelog(CHANGELOG, plan.changelog_entry)
-
-        # Regenerate derived files (pack manifests, .agent-src/, tool
-        # projections) so the PR's own consistency check passes. Without
-        # this the bump only lands in package.json + marketplace.json and
-        # the Sync + Generate Tools Consistency gate fails on the release
-        # PR itself — exactly the failure mode PR #226 hit. `task
-        # release-prepare` is idempotent, so resume runs are safe.
-        _step(2, total, "Regenerate derived files (`task release-prepare`)")
-        run("task", "release-prepare")
-
-    # ─── 3. commit ──────────────────────────────────────────────────────────
-    if pr_merged:
-        _step(3, total, "PR already merged — skip commit")
-    else:
-        last_msg = git("log", "-1", "--format=%s", capture=True)
-        porcelain = git("status", "--porcelain", capture=True)
-        if resume and last_msg == f"release: {plan.target}" and not porcelain:
-            _step(3, total, f"Last commit already `release: {plan.target}` and tree clean — skip")
-        else:
-            _step(3, total, f"Commit `release: {plan.target}`")
-            # `git add -A` stages the three primary bump files AND every
-            # regenerated derived file (packages/*/pack.yaml + README.md,
-            # .agent-src/, .augment/, tool projections). Listing them
-            # explicitly would silently drift the moment a new generated
-            # tree is added.
-            run("git", "add", "-A")
-            run("git", "commit", "-m", f"release: {plan.target}")
-
-    # ─── 4. push ────────────────────────────────────────────────────────────
-    if pr_merged:
-        _step(4, total, "PR already merged — skip push")
-    else:
-        # `git push -u` is naturally idempotent — it prints "Everything
-        # up-to-date" when remote already matches. No probe needed.
-        _step(4, total, f"Push {branch} to {REMOTE}")
-        run("git", "push", "-u", REMOTE, branch)
-
-    # ─── 5. PR ──────────────────────────────────────────────────────────────
-    if pr_merged:
-        _step(5, total, f"PR #{pr_info.get('number')} already merged — skip")
-    elif resume and pr_state == "OPEN":
-        _step(5, total, f"PR already open: {pr_info.get('url')}")
-    else:
-        _step(5, total, "Open pull request")
-        pr_body = (
-            f"Release {plan.target}.\n\n"
-            f"{plan.changelog_body}\n\n"
-            "Created by `scripts/release.py`."
-        )
-        run(
-            "gh", "pr", "create",
-            "--base", MAIN_BRANCH,
-            "--head", branch,
-            "--title", f"release: {plan.target}",
-            "--body", pr_body,
-        )
-
-    # ─── 6. wait for checks ─────────────────────────────────────────────────
-    if pr_merged:
-        _step(6, total, "PR already merged — skip checks wait")
-    elif wait_for_checks:
-        _step(6, total, "Wait for PR checks")
-        watch_pr_checks()
-    else:
-        _step(6, total, "Skip waiting for checks (--no-wait)")
-
-    # ─── 7. merge ───────────────────────────────────────────────────────────
-    if pr_merged:
-        _step(7, total, f"PR #{pr_info.get('number')} already merged — skip")
-    else:
-        _step(7, total, "Merge pull request (merge commit) and delete branch")
-        run("gh", "pr", "merge", "--merge", "--delete-branch")
-
-    # ─── 8. tag main + push tag ─────────────────────────────────────────────
-    # Always idempotent — even outside resume mode this prevents a mid-flight
-    # crash on step 9 from leaving a half-tagged release that subsequent
-    # `task release` invocations can't recover from without `--resume`.
-    if git("rev-parse", "--abbrev-ref", "HEAD", capture=True) != MAIN_BRANCH:
-        run("git", "checkout", MAIN_BRANCH)
-    run("git", "pull", "--ff-only", REMOTE, MAIN_BRANCH)
-
-    if _tag_exists_local(plan.target):
-        if _tag_exists_remote(plan.target):
-            _step(8, total, f"Tag {plan.target} already on {REMOTE} — skip")
-        else:
-            _step(8, total, f"Tag {plan.target} exists locally — push only")
-            run("git", "push", REMOTE, plan.target)
-    else:
-        _step(8, total, f"Tag merge commit and push {plan.target}")
-        run("git", "tag", plan.target)
-        run("git", "push", REMOTE, plan.target)
-
-    # ─── 9. GitHub Release ──────────────────────────────────────────────────
-    if _release_exists(plan.target):
-        _step(9, total, f"GitHub Release {plan.target} already exists — skip")
-    else:
-        _step(9, total, "Create GitHub Release (triggers publish-npm on the tag)")
-        notes = plan.changelog_body or f"Release {plan.target}"
-        run(
-            "gh", "release", "create", plan.target,
-            "--title", plan.target,
-            "--notes", notes,
-        )
-
-    # ─── 10. delete the merged release branch (local + remote) ───────────────
-    # Branch hygiene: a merged-but-undeleted release/X.Y.Z is what made
-    # `--resume` mis-detect an old version. Delete it now so it can never
-    # accumulate. Idempotent — skips whatever is already gone. Never touches
-    # `main` or any tag.
-    if dry_run:
-        _step(10, total, f"Would delete merged branch {branch} (local + remote)")
-    else:
-        deleted = []
-        if _branch_exists_local(branch) and \
-                git("rev-parse", "--abbrev-ref", "HEAD", capture=True) != branch:
-            run("git", "branch", "-D", branch, check=False)
-            deleted.append("local")
-        if _branch_exists_remote(branch):
-            run("git", "push", REMOTE, "--delete", branch, check=False)
-            deleted.append("remote")
-        where = " + ".join(deleted) if deleted else "already gone"
-        _step(10, total, f"Delete merged branch {branch} ({where})")
-
-    print()
-    print(f"✅  Released {plan.target}")
-    print(f"   https://github.com/{REPO_SLUG}/releases/tag/{plan.target}")
-    print("   npm publish runs asynchronously via publish-npm.yml on the tag.")
-
-
-# ─── entrypoint ───────────────────────────────────────────────────────────────
-
-
-def _parse_args(argv: list[str]) -> argparse.Namespace:
-    p = argparse.ArgumentParser(description=__doc__.splitlines()[0])
-    p.add_argument(
-        "--as",
-        dest="bump_override",
-        choices=("major", "minor", "patch"),
-        default=None,
-        help=(
-            "Force a specific bump level. Default is auto-detect from "
-            "Conventional Commits since the last tag."
-        ),
-    )
-    p.add_argument(
-        "--version",
-        dest="explicit",
-        default=None,
-        help="Use an explicit X.Y.Z instead of the auto-detected bump.",
-    )
-    p.add_argument(
-        "--yes", "-y", action="store_true",
-        help="Skip the confirmation prompt.",
-    )
-    p.add_argument(
-        "--dry-run", action="store_true",
-        help="Print the plan and exit without touching git/gh.",
-    )
-    p.add_argument(
-        "--no-wait", action="store_true",
-        help="Merge immediately without waiting for PR checks to pass.",
-    )
-    p.add_argument(
-        "--resume", action="store_true",
-        help=(
-            "Recover from a partial run. Each step probes existing state "
-            "(branch, commit, PR, tag, GitHub Release) and skips work that "
-            "is already done. Use this when an earlier `task release` "
-            "crashed mid-pipeline."
-        ),
-    )
-    return p.parse_args(argv)
-
-
-def resolve_bump(override: str | None, commits: list[Commit]) -> str:
-    """Override wins; otherwise auto-detect from commits (or 'patch' if empty)."""
-    if override:
-        return override
-    return infer_bump(commits)
-
-
-_RELEASE_BRANCH_RE = re.compile(r"^release/(\d+\.\d+\.\d+)$")
-
-
-def _detect_in_flight_target() -> str | None:
-    """Find the in-flight release target — the SOURCE OF TRUTH is package.json.
-
-    An "in-flight" release is one whose version was already bumped into
-    ``main``'s ``package.json`` (and possibly merged) but whose tag has not
-    yet been pushed — i.e. the publish step never completed. The canonical
-    anchor is therefore ``package.json`` version `V` **with no matching tag
-    `V`**, NOT the set of ``release/X.Y.Z`` branches.
-
-    Why not the branch set: merged release branches are frequently left
-    undeleted on the remote, so "highest existing release/* branch" can
-    resolve to an OLD, already-published version (e.g. picking 5.4.0 while
-    5.8.0 is the real in-flight target) and tag a downgrade. The package.json
-    version cannot lie that way — it is the version main currently claims to
-    be, and an untagged claim is exactly an incomplete release.
-
-    Resolution order:
-      1. If HEAD is on a ``release/X.Y.Z`` branch, that explicit checkout wins.
-      2. Else: read ``package.json`` version `V`. If tag `V` does not exist
-         (local or remote), `V` is the in-flight target. If it is already
-         tagged, the release is complete → return None (regular bump path).
-
-    Stale ``release/*`` branches are never used for version detection.
-    """
-    head = git("rev-parse", "--abbrev-ref", "HEAD", capture=True)
-    m = _RELEASE_BRANCH_RE.match(head)
-    if m:
-        return m.group(1)
-
-    try:
-        version = json.loads(PACKAGE_JSON.read_text(encoding="utf-8"))["version"]
-    except (OSError, KeyError, json.JSONDecodeError):
-        return None
-    try:
-        parse_version(version)
-    except Exception:
-        return None
-
-    # An already-tagged version is a completed release, not in-flight.
-    if _tag_exists_local(version) or _tag_exists_remote(version):
-        return None
-    return version
-
-
-def main(argv: list[str] | None = None) -> int:
-    args = _parse_args(list(sys.argv[1:] if argv is None else argv))
-
-    current = json.loads(PACKAGE_JSON.read_text(encoding="utf-8"))["version"]
-    parse_version(current)
-
-    prev = latest_tag()
-    commits = commits_since(prev)
-    bump = resolve_bump(args.bump_override, commits)
-
-    # Resume mode: prefer an existing `release/X.Y.Z` over computed bump,
-    # so we don't accidentally start a 1.16.0 release while 1.15.0 is
-    # still in flight. Explicit --version still wins.
-    in_flight = _detect_in_flight_target() if args.resume else None
-    if args.explicit:
-        target = args.explicit
-    elif in_flight:
-        target = in_flight
-        print(f"(resume) in-flight target {in_flight} (package.json version with no tag yet)")
-    else:
-        target = bump_version(current, bump)
-    parse_version(target)
-
-    if not args.dry_run:
-        preflight(target, resume=args.resume)
-
-    today = _date.today().isoformat()
-    test_trend_line = _render_test_trend_line(prev)
-    full, body = render_changelog_entry(
-        target, prev, commits, today, test_trend_line=test_trend_line
-    )
-
-    # Era-split planning: only crosses the gate when the current era body
-    # has grown past the drift cap AND the release crosses a minor/major
-    # boundary. Patch overflow is caught by the drift test (red CI), not
-    # by an auto-split into a nonsensical "pre-X.Y.Z" archive.
-    split: SplitPlan | None = None
-    body_size = current_era_body_size()
-    if body_size > CURRENT_ERA_BODY_CAP:
-        candidate = plan_split(target)
-        if candidate is None:
-            die(
-                f"current era body is {body_size} lines (cap "
-                f"{CURRENT_ERA_BODY_CAP}) but release {target} is a patch "
-                f"within the same era — split needs a minor/major bump. "
-                "Cut a minor release or split CHANGELOG.md manually first."
-            )
-        split = candidate
-
-    plan = Plan(
-        current=current,
-        target=target,
-        bump=bump,
-        commits=commits,
-        last_tag=prev,
-        changelog_body=body,
-        changelog_entry=full,
-        split_plan=split,
-    )
-    print_preview(plan)
-    if args.resume:
-        print("(resume) probing existing state — completed steps will be skipped.")
-
-    if args.dry_run:
-        return 0
-
-    if not args.yes and not confirm(f"Proceed with release {plan.target}?"):
-        print("aborted.")
-        return 1
-
-    execute(
-        plan,
-        wait_for_checks=not args.no_wait,
-        dry_run=False,
-        resume=args.resume,
-    )
-    return 0
-
-
-if __name__ == "__main__":
-    raise SystemExit(main())