feat(skill): add safari reference card for AI-SRE Customize self-management#60
Merged
Conversation
…nagement The `fduty safari` group (mcp-server-*, skill-*, a2a-agent-*, session-*) had no reference card, so the flashduty skill could not route install/configure intents to it — agents fell back to a tool search (wrong tool) and --help-dance. Seed `reference/safari.md` (skilldoc fence auto-filled from the cobra tree) + a SKILL.md domain-index row. Hot flows cover stdio and remote MCP install; key concepts document transport/scope/auth_mode and the per_user_oauth model: empty oauth_metadata is normal — the runtime auto-discovers + DCRs on first authorization; no OAuth client config is collected up front. `skilldoc check` passes (fence in sync; examples valid).
…t safari surface) Merge feat/ai-sre and regenerate the safari command card against the current cli: /safari/skill/download was reclassified jwt (console-only) and dropped from the public surface, while session-delete was added. The GENERATED fence syncs automatically; also update the hand-written intent table (remove the skill-download routing line, add session-delete to the session line).
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
The
fduty safaricommand group (mcp-server-*,skill-*,a2a-agent-*,session-*) had no reference card, soskilldocskipped it and the flashduty skill's domain index never pointed there. Asked to "install an MCP server", an agent had nothing to route to — it reached for a tool search (which only discovers tools on already-connected servers) and--help-danced.Change
skills/flashduty/reference/safari.md— modeled on the other cards. Prose covers routing keywords, intent→verb table, two hot-flow installs (stdio + remote streamable-http withper_user_oauth), key concepts (transport ⇒ fields,team_idscope,auth_mode), and gotchas. The<!-- GENERATED:safari -->fence is auto-filled byskilldocfrom the cobra tree (stays in sync; no hand-maintained flag tables).SKILL.mddomain-index row pointing intent (install/configure MCP·skill·A2A, customize, session export) →reference/safari.md.Headline correctness note baked into the card:
per_user_oauthservers are created with an emptyoauth_metadata— that is the normal, complete state. The runtime auto-discovers the OAuth server and does Dynamic Client Registration on first authorization; you do not collectauthorization_url/client_id/client_secret/scopes.Why public
Everything in the card is OpenAPI-derived — the
fduty safariverbs are generated bycligenfrom the safari OpenAPI ops (/safari/mcp/server/*,/safari/skill/*,/safari/a2a/*). No internals exposed.Verification
go run ./internal/cmd/skilldoc check→skilldoc: cards OK(fence fresh, examples valid).Companion
fc-safari PR #300 vendors this card byte-identical into the bundled skill and adds the agent-runtime-only OAuth finish note (
tool_search(server=...)→ authorize card), which can't live in the shared cli card.