Skip to content

docs(ai-sre): drop deprecated session/cancel from the public API#101

Merged
ysyneu merged 1 commit into
mainfrom
fix/docs-drop-session-cancel
Jun 24, 2026
Merged

docs(ai-sre): drop deprecated session/cancel from the public API#101
ysyneu merged 1 commit into
mainfrom
fix/docs-drop-session-cancel

Conversation

@ysyneu

@ysyneu ysyneu commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Why

/safari/session/cancel (pgy registry ID 7018) has had its auth changed to service (internal). It is a deprecated alias that delegates to /safari/session/interrupt (also service), kept registered only so privately-deployed ai-assistant frontends that still POST to /cancel keep working. It is no longer an app_key-callable public endpoint, so it must not appear in the public OpenAPI.

Change

  • Remove the /safari/session/cancel operation from all four specs: safari.openapi.{en,zh}.json + consolidated openapi.{en,zh}.json.
  • Drop the now-orphaned SessionCancelRequest + SessionInterruptAck schemas (each referenced only by the removed op).
  • Remove the "POST /safari/session/cancel" nav entry from docs.json (en + zh) — it was just added in docs(ai-sre): add Sessions subgroup to the API reference nav #100.

Public session surface is now list / get / export / delete.

Verification

  • All 5 touched files are valid JSON.
  • No remaining reference to /safari/session/cancel, SessionCancelRequest, or SessionInterruptAck anywhere; zero dangling $ref.
  • en/zh path-key and schema-key parity holds.
  • Nav check: every safari spec path is still reachable in docs.json; cancel is absent.
  • Diff is a clean removal (no reserialization noise — matched the api-review serializer ensure_ascii=False, indent=2).

go-flashduty and flashduty-cli never wrapped cancel, so no SDK/CLI change is needed.

`/safari/session/cancel` (pgy ID 7018) had its auth changed to `service`
(internal) — it is a deprecated alias that delegates to
`/safari/session/interrupt` (also `service`) and is kept registered only so
privately-deployed ai-assistant frontends keep working. It is no longer an
app_key-callable public endpoint, so it must not appear in the public OpenAPI.

Remove the operation from all four specs (safari.openapi.{en,zh}.json +
consolidated openapi.{en,zh}.json), drop the now-orphaned SessionCancelRequest
and SessionInterruptAck schemas, and remove the nav entry from docs.json
(en + zh). Session public surface is now list / get / export / delete.
@ysyneu
ysyneu merged commit ee4fc11 into main Jun 24, 2026
@ysyneu
ysyneu deleted the fix/docs-drop-session-cancel branch June 24, 2026 12:34
@mintlify

mintlify Bot commented Jun 24, 2026

Copy link
Copy Markdown

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
flashduty 🔴 Failed Jun 24, 2026, 12:36 PM

💡 Tip: Enable Workflows to automatically generate PRs for you.

ysyneu added a commit that referenced this pull request Jun 24, 2026
The API catalog (the hand-maintained master index) listed the AI SRE skill /
MCP / A2A operations but had no Sessions section — the session endpoints were
never added when they were adopted into the spec, the same gap that left them
out of the API-reference nav (#100).

Add a Sessions section (en + zh) listing the four public session operations —
list / get / export / delete — and bump the AI SRE count 22 → 26 and the total
236 → 240. (cancel was dropped in #101: it is now auth=service / internal.)
ysyneu added a commit that referenced this pull request Jun 24, 2026
…ly) (#106)

/safari/skill/download is reclassified auth=all -> jwt in the registry
(fc-pgy #570): it is a console-only binary (application/zip) download used
by the Customize UI via the session cookie, with no app_key consumer. Remove
it from the public OpenAPI specs (per-module + consolidated, en+zh), the
docs.json navigation, and the api-catalog overview (AI SRE 26->25, total
240->239). Mirrors the session/cancel removal (#101).
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