booking-agent-cli 0.11.0__tar.gz → 0.11.2__tar.gz

{booking_agent_cli-0.11.0 → booking_agent_cli-0.11.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: booking-agent-cli
-Version: 0.11.0
+Version: 0.11.2
 Summary: The Booking Agent CLI: a stateful USP booking flow with hard gates, plus its USP protocol library.
 Project-URL: Homepage, https://github.com/wix/booking-agent
 Project-URL: Documentation, https://github.com/wix/booking-agent/blob/master/booking-agent/README.md
@@ -82,7 +82,7 @@ Each release also publishes that skill at an immutable `skill-v<version>` tag an
 the release workflow, so a harness can pin and verify the exact text it follows:
 
 ```bash
-curl -fsSLO https://raw.githubusercontent.com/wix/booking-agent/skill-v0.58.0/booking-agent/skills/booking-agent/SKILL.md
+curl -fsSLO https://raw.githubusercontent.com/wix/booking-agent/skill-v0.58.2/booking-agent/skills/booking-agent/SKILL.md
 gh attestation verify SKILL.md --repo wix/booking-agent \
   --signer-workflow wix/booking-agent/.github/workflows/release.yml
 ```

{booking_agent_cli-0.11.0 → booking_agent_cli-0.11.2}/README.md

@@ -40,7 +40,7 @@ Each release also publishes that skill at an immutable `skill-v<version>` tag an
 the release workflow, so a harness can pin and verify the exact text it follows:
 
 ```bash
-curl -fsSLO https://raw.githubusercontent.com/wix/booking-agent/skill-v0.58.0/booking-agent/skills/booking-agent/SKILL.md
+curl -fsSLO https://raw.githubusercontent.com/wix/booking-agent/skill-v0.58.2/booking-agent/skills/booking-agent/SKILL.md
 gh attestation verify SKILL.md --repo wix/booking-agent \
   --signer-workflow wix/booking-agent/.github/workflows/release.yml
 ```

{booking_agent_cli-0.11.0 → booking_agent_cli-0.11.2}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "booking-agent-cli"
-version = "0.11.0"
+version = "0.11.2"
 description = "The Booking Agent CLI: a stateful USP booking flow with hard gates, plus its USP protocol library."
 readme = "README.md"
 license = "Apache-2.0"

{booking_agent_cli-0.11.0 → booking_agent_cli-0.11.2}/src/booking_agent/__init__.py

@@ -3,4 +3,4 @@
 Ships in the booking-agent-cli wheel and never imports ``booking_agent_cli``.
 """
 
-__version__ = "0.11.0"
+__version__ = "0.11.2"

{booking_agent_cli-0.11.0 → booking_agent_cli-0.11.2}/src/booking_agent_cli/__init__.py

@@ -1,3 +1,3 @@
 """booking_agent_cli: typer-based stateful CLI for USP booking flows."""
 
-__version__ = "0.11.0"
+__version__ = "0.11.2"

{booking_agent_cli-0.11.0 → booking_agent_cli-0.11.2}/src/booking_agent_cli/bundled_skill.md

@@ -1,5 +1,5 @@
 ---
-version: 0.58.0
+version: 0.58.2
 name: booking-agent
 description: |
   Books time-based services via USP from a Link wallet. Use when the user wants to book, schedule, reserve, or pay for a service from a business (haircut, lesson, repair, class, appointment, consult). The `booking-agent` CLI owns the booking state machine and refuses to proceed when human gates are skipped. Sign the user in to Link on your own, right after install and at the start of every booking (`booking-agent flow link-auth connect`, then start `booking-agent flow link-auth poll` in the background and post the sign-in URL in the same turn, exactly like `pay poll`); never wait for the user to ask to sign in or to say they approved. After `booking-agent flow business use`, follow `next_command` through `use-ucp-checkout-mode`; businesses without native checkout fail closed. Payment uses `booking-agent flow pay request` (default returns immediately with the approval URL), then start `booking-agent flow pay poll` in the background and post the approval URL to the user in the same turn; the poll finishes checkout once they approve. Confirm through `booking-agent flow book poll --wait`; never start an inbound webhook. Surface the approval URL exactly once. Personal identification (name, phone, email, address) must always come from the user's Link account via `link-cli`, never from chat or placeholders. Before any location-based search, the CLI proposes the Link shipping address and the user confirms it or gives another; online services skip this. Talk to the user in plain everyday language only; never expose CLI, UCP, auth, session, or other implementation details unless they explicitly ask.
@@ -19,7 +19,7 @@ metadata:
         - link-cli
     install:
       - kind: uv
-        package: "booking-agent-cli==0.11.0"
+        package: "booking-agent-cli==0.11.2"
         bins: [booking-agent]
 user-invocable: true
 ---
@@ -220,7 +220,7 @@ To take a fixed, published version instead of the latest, fetch it at the releas
 workflow signs it under, and check the signature before you follow it:
 
 ```bash
-curl -fsSLO https://raw.githubusercontent.com/wix/booking-agent/skill-v0.58.0/booking-agent/skills/booking-agent/SKILL.md
+curl -fsSLO https://raw.githubusercontent.com/wix/booking-agent/skill-v0.58.2/booking-agent/skills/booking-agent/SKILL.md
 gh attestation verify SKILL.md \
   --repo wix/booking-agent \
   --signer-workflow wix/booking-agent/.github/workflows/release.yml
@@ -245,17 +245,17 @@ before installing:
 
 ```bash
 dir="$HOME/.booking-agent/verified-wheels"
-uvx pip download "booking-agent-cli==0.11.0" --no-deps --only-binary :all: -d "$dir"
+uvx pip download "booking-agent-cli==0.11.2" --no-deps --only-binary :all: -d "$dir"
 uvx pypi-attestations==0.0.30 verify pypi --repository https://github.com/wix/booking-agent \
-  "$dir/booking_agent_cli-0.11.0-py3-none-any.whl"
-uv tool install --force "$dir/booking_agent_cli-0.11.0-py3-none-any.whl"
+  "$dir/booking_agent_cli-0.11.2-py3-none-any.whl"
+uv tool install --force "$dir/booking_agent_cli-0.11.2-py3-none-any.whl"
 
 booking-agent --help
 ```
 
 Install only a wheel whose verify step passed. If `uvx` is not
-available, install the same pinned release with `uv tool install booking-agent-cli==0.11.0` or
-`pipx install booking-agent-cli==0.11.0`; the CLI checks its own files against the signed release
+available, install the same pinned release with `uv tool install booking-agent-cli==0.11.2` or
+`pipx install booking-agent-cli==0.11.2`; the CLI checks its own files against the signed release
 on the first `booking-agent flow init` and refuses to book or pay if they do not match.
 
 Harnesses that read this skill's `install` block do this for you. Do not go on
@@ -468,7 +468,7 @@ confirmation email or on the business's page (`booking_url`). Do not invent a lo
 **Step 1b — Create a new session:**
 
 ```bash
-booking-agent flow init --user-request "<user's exact request>" --skill-version 0.58.0 \
+booking-agent flow init --user-request "<user's exact request>" --skill-version 0.58.2 \
   --skill-path "<absolute path of the SKILL.md you loaded>"
 ```
 

{booking_agent_cli-0.11.0 → booking_agent_cli-0.11.2}/src/booking_agent_cli/commands/flow.py

@@ -793,6 +793,23 @@ def _unfinished_similar_sessions(store: Any, user_request: str) -> List[Dict[str
     return matches
 
 
+def _inactive_booking_status_message(session: Session) -> str:
+    """What status tells the user once the appointment itself is over."""
+    from ..booking_summary import selected_slot, service_and_business, when_text
+
+    label = {
+        "canceled": "cancelled",
+        "completed": "completed",
+        "no_show": "a no-show",
+    }.get(session.booking.status, (session.booking.status or "inactive").replace("_", " "))
+    booking = f"Your booking, {service_and_business(session)},"
+    slot = selected_slot(session)
+    when = when_text(session, slot, dual=False) if slot is not None else None
+    if when:
+        booking = f"{booking} {when},"
+    return f"{booking} is {label}."
+
+
 @app.command("status")
 def status() -> None:
     session = load_session()
@@ -873,6 +890,18 @@ def status() -> None:
         )
         extra_data["services"] = session.discover_services
         extra_data["discover_query"] = session.discover_query
+    elif (
+        session.flow_phase == "CONFIRMED"
+        and session.booking.status in INACTIVE_BOOKING_STATUSES
+    ):
+        # Phase stays CONFIRMED: that is where a finished booking lives. The appointment
+        # itself is booking.status, and status must not describe a canceled one as booked.
+        status_message = _inactive_booking_status_message(session)
+        prompt = (
+            "This booking is no longer active. If the user asked how it is going, post "
+            "user_message as written. Do not say they are booked, and do not offer to "
+            "add or change a calendar event."
+        )
     elif session.flow_phase == "CONFIRMED" and not session.reschedule_slot_id \
             and not session.reschedule_original_slot_id:
         from ..booking_summary import booked_message

{booking_agent_cli-0.11.0 → booking_agent_cli-0.11.2}/src/booking_agent_cli/updates.py

@@ -220,27 +220,28 @@ def local_pair_reason(reported: Optional[str]) -> Optional[str]:
 
 
 def select_target(installed: str, releases: Sequence[Release]) -> Tuple[Optional[Release], List[Release]]:
-    """The newest release that is safe to move to, and every release that move brings in.
+    """The newest release that is safe to move to, and the ready releases that move brings in.
 
-    Installing a version brings every release between the installed one and it, so a
-    deferred release in that range holds back everything after it.
+    A deferred release is skipped, not a stopping point: it holds back only itself, so
+    a later recommended or urgent release is still offered. ``releases`` carries either
+    the general rollout or one replay's effective status; both are read the same way.
+    CLIs before 0.11.2 stop at the first deferred release, which is why the release
+    notes recommend every release up to 0.11.2 for each verified replay.
     """
     current = parse_version(installed)
     if current is None:
         return None, []
     target: Optional[Release] = None
-    passed: List[Release] = []
+    brought: List[Release] = []
     for release in releases:
-        if release.key <= current:
+        if release.key <= current or release.update_status == "deferred":
             continue
-        if release.update_status == "deferred":
-            break
-        passed.append(release)
+        brought.append(release)
         if release.update_status != "withdrawn":
             target = release
     if target is None:
         return None, []
-    return target, [r for r in passed if r.key <= target.key]
+    return target, [r for r in brought if r.key <= target.key]
 
 
 def decide(

{booking_agent_cli-0.11.0 → booking_agent_cli-0.11.2}/tests/integration/test_ucp_cancel_result.py

@@ -398,6 +398,38 @@ def test_successful_cancel_persists_status_and_stops_being_actionable(
     assert not is_actionable(row)
 
 
+def test_status_after_cancel_does_not_say_the_booking_is_still_booked(
+    runner: CliRunner, isolated_session, monkeypatch,
+):
+    """flow status keeps phase CONFIRMED, but must not tell the user they are booked."""
+    store = load_store()
+    session = _completed_checkout_409_session()
+    session.business.name = "HVAC-Koby"
+    session.live_service = {**session.live_service, "name": "AC Repair"}
+    session.calendar_event_html_link = "https://www.google.com/calendar/event?eid=abc"
+    add_session(store, session)
+    save_store(store)
+    _install_completed_checkout_409(
+        monkeypatch,
+        checkout={},
+        booking_result=_canceled_booking(),
+    )
+    _invoke_cancel(runner, exit_code=0)
+
+    result = runner.invoke(app, ["flow", "status"])
+    text = result.stdout or result.output or ""
+    assert result.exit_code == 0, text
+    body, _ = json.JSONDecoder().raw_decode(text[text.index("{"):])
+    message = body["user_message"]
+
+    assert body["phase"] == "CONFIRMED"
+    assert "cancelled" in message
+    assert "AC Repair with HVAC-Koby" in message
+    assert not message.startswith("You're booked")
+    assert "calendar/event" not in message
+    assert "confirmed" not in body["prompt"].lower()
+
+
 def test_completed_checkout_409_still_cancels_when_the_checkout_cannot_be_reread(
     runner: CliRunner, isolated_session, monkeypatch,
 ):

{booking_agent_cli-0.11.0 → booking_agent_cli-0.11.2}/tests/unit/test_updates.py

@@ -53,12 +53,64 @@ def test_replay_identity_is_generic_runtime_configuration(monkeypatch):
     assert updates.configured_replay() is None
 
 
-def test_the_committed_index_parses():
+def _committed_releases():
     index = Path(__file__).resolve().parents[4] / "release-notes" / "index.json"
-    releases = updates.parse_index(json.loads(index.read_text()))
+    return updates.parse_index(json.loads(index.read_text()))
+
+
+def test_the_committed_index_parses():
+    releases = _committed_releases()
     assert releases and all(updates.parse_version(r.cli_version) for r in releases)
 
 
+def _verified_harnesses():
+    return [h for h in release_notes.load_registry()["harnesses"].values() if h["status"] == "verified"]
+
+
+def test_every_verified_harness_is_offered_its_approved_release():
+    """From each release since the replayed one, the decision targets the registry's pin."""
+    releases = _committed_releases()
+    for harness in _verified_harnesses():
+        replayed = updates.parse_version(harness["replayed"]["cli"])
+        approved = harness["recommended"]["cli"]
+        for start in [r.cli_version for r in releases if replayed <= r.key < updates.parse_version(approved)]:
+            d = updates.decide(
+                reported_skill=SKILL_VERSION, releases=releases, installed=start, replay_id=harness["replay"],
+            )
+            assert d.target_version == approved, (harness["name"], start, d.target_version)
+        at_pin = updates.decide(
+            reported_skill=SKILL_VERSION, releases=releases, installed=approved, replay_id=harness["replay"],
+        )
+        assert at_pin.target_version is None, (harness["name"], at_pin.target_version)
+
+
+def _pre_skip_target(installed, releases):
+    """The walk inside CLIs before 0.11.2: it stops at the first deferred release."""
+    current, target = updates.parse_version(installed), None
+    for release in releases:
+        if release.key <= current:
+            continue
+        if release.update_status == "deferred":
+            break
+        if release.update_status != "withdrawn":
+            target = release
+    return target.cli_version if target else None
+
+
+@pytest.mark.parametrize("installed,replay", [
+    ("0.7.0", None),
+    ("0.10.0", "grok-bot"),
+    ("0.10.2", "base44-superagent"),
+    ("0.11.1", "muse"),
+])
+def test_installed_harness_clis_can_walk_to_the_first_cli_that_skips_gaps(installed, replay):
+    releases = _committed_releases()
+    if replay:
+        releases = updates.releases_for_replay(releases, replay)
+    target = _pre_skip_target(installed, releases)
+    assert target is not None and updates.parse_version(target) >= updates.parse_version("0.11.2"), target
+
+
 @pytest.mark.parametrize("text,ok", [
     ("0.2.0", True), ("1.10.3", True), ("0.2.0.dev1", True),
     ("v0.2.0", False), ("0.2", False), ("latest", False), (None, False), (3, False),
@@ -132,7 +184,7 @@ def test_harnesses_that_share_a_replay_promote_together():
     assert first.replay == second.replay == "shared"
 
 
-def test_every_deferred_release_in_a_verified_upgrade_needs_that_replay():
+def test_a_gap_does_not_hold_back_the_newest_release_a_replay_recommends():
     old = _entry(
         "0.2.0", status="deferred", ready=False,
         replay_rollouts={"grok-bot": _rollout()},
@@ -142,15 +194,15 @@ def test_every_deferred_release_in_a_verified_upgrade_needs_that_replay():
         "0.4.0", status="deferred", ready=False,
         replay_rollouts={"grok-bot": _rollout()},
     )
-    blocked = _decide(old, missing, newest, replay_id="grok-bot")
-    assert blocked.target_version == "0.2.0"
+    assert _decide(old, missing, newest, replay_id="grok-bot").target_version == "0.4.0"
 
-    middle = _entry(
+    held = _entry(
         "0.3.0", status="deferred", ready=False,
-        replay_rollouts={"grok-bot": _rollout()},
+        replay_rollouts={"grok-bot": _rollout("deferred", ready=False)},
     )
-    promoted = _decide(old, middle, newest, replay_id="grok-bot")
-    assert promoted.target_version == "0.4.0"
+    d = _decide(old, held, newest, replay_id="grok-bot")
+    assert d.target_version == "0.4.0"
+    assert "0.3.0" not in d.user_summary
 
 
 def test_a_withdrawn_release_stays_withdrawn_for_a_verified_harness():
@@ -161,12 +213,22 @@ def test_a_withdrawn_release_stays_withdrawn_for_a_verified_harness():
     ).reason == "installed_version_withdrawn"
 
 
-def test_a_deferred_release_holds_back_every_later_one():
+def test_a_deferred_release_holds_back_only_itself():
+    d = _decide(
+        _entry("0.2.0", "breaking", status="deferred", ready=False),
+        _entry("0.3.0", "fix", "urgent"),
+    )
+    assert (d.action, d.target_version, d.change_type) == ("urgent", "0.3.0", "fix")
+
+
+def test_a_withdrawn_release_in_a_gap_is_never_the_target():
     d = _decide(
         _entry("0.2.0", status="deferred", ready=False),
-        _entry("0.3.0", status="urgent"),
+        _entry("0.3.0", status="withdrawn", replacement_version="0.4.0"),
+        _entry("0.4.0", status="recommended"),
+        _entry("0.5.0", status="withdrawn", replacement_version="0.5.1"),
     )
-    assert d.action == "none" and d.target_version is None
+    assert d.target_version == "0.4.0"
 
 
 def test_the_target_is_the_newest_safe_release_and_carries_the_strongest_status():