Skip to content

SMOODEV-4188: th smoo booking config publish/unpublish - #853

Merged
brentrager merged 1 commit into
mainfrom
SMOODEV-4188-booking-publish
Oct 26, 2026
Merged

brentrager merged 1 commit into
mainfrom
SMOODEV-4188-booking-publish

Conversation

@brentrager

@brentrager brentrager commented Oct 26, 2026 •

Copy link
Copy Markdown
Contributor

Problem

SMOODEV-4188. Since SMOODEV-3766 (smooai ADR-147 §5), a booking page starts unpublished, and nobody can book it until it is published. The dashboard, the hosted MCP and the API can publish a page; th couldn't. Also, th smoo booking link and types create handed out links that answer "not available" without saying so.

Solution

  • th smoo booking config publish and config unpublish call PUT /booking/config/:org/published on the caller's own page. The server still refuses a legacy org's shared page without bookings.write, and a page that was never set up.
  • booking link warns when the config says published: false.
  • booking types create warns on stderr when the response says hostBookable: false. The field arrives with SmooAI/smooai SMOODEV-4188. An older server omits it and gets no warning.

Verification

  • cargo test -p smooai-smooth-cli --bin th booking: 17 passed. That includes new tests that the publish and unpublish subcommands parse, and that the warning fires only on an explicit false.
  • cargo clippy -p smooai-smooth-cli --bin th reports no new warnings in booking.rs. rustfmt --check is clean.

🤖 Generated with Claude Code

…d-page warnings

A booking page now starts unpublished (SMOODEV-3766, ADR-147 §5), but the CLI
had no way to publish it and handed out links that answered "not available".
Add `config publish` / `config unpublish` over PUT /booking/config/:org/published,
and warn from `link` and `types create` when the page isn't published.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@changeset-bot

changeset-bot Bot commented Oct 26, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 8a10fb7

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@smooai/smooth Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@brentrager
brentrager enabled auto-merge (squash) October 26, 2026 02:04
@brentrager
brentrager merged commit cfd88b6 into main Oct 26, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant