Skip to content

fix(mcp): SKILL.md documents the business-action tools (#2714 Phase 0) - #2715

Merged
os-zhuang merged 1 commit into
mainfrom
fix/skill-document-action-tools
Jul 9, 2026
Merged

fix(mcp): SKILL.md documents the business-action tools (#2714 Phase 0)#2715
os-zhuang merged 1 commit into
mainfrom
fix/skill-document-action-tools

Conversation

@os-zhuang

Copy link
Copy Markdown
Contributor

#2714Phase 0 — prerequisite for every distribution shell (they all copy the rendered skill).

renderSkillMarkdown() documented only the 7 object-CRUD tools; the MCP surface exposes 9. The business-action pair from #2307 (list_actions / run_action) was absent, so an agent installing the skill never learned it can run approvals, conversions, or flow triggers directly — it would hand-edit the underlying records instead.

Changes (skill content only, no runtime behavior)

  • Tools section: adds list_actions() and run_action({ actionName, objectName?, recordId?, params? }) with their real semantics (record-scoped actions, destructive flag, param declaration, objectName as disambiguator only)
  • Intro / When-to-use / Discover / Workflow: business actions woven in as first-class — discovery step 3 is list_actions, and the workflow branches to run_action when a matching action exists
  • New convention: prefer actions over hand-edits — actions carry the app's validation and side effects; confirm destructive or confirmation-flagged actions with the user first
  • Tests: full 9-tool surface asserted + a dedicated action-preference assertion (63 pass)

Refs #2714, #2698.

🤖 Generated with Claude Code

…2714 Phase 0)
The skill's tool list covered 7 of the 9 native tools — the business-action
pair from #2307 was missing, so skill-installed agents never learned they can
run app-registered actions directly. Adds the two tools with their real
signatures, weaves actions into the intro / when-to-use / discover / workflow
sections, and teaches action preference over hand-editing records.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@vercel

vercelBot commented Jul 9, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

ProjectDeploymentActionsUpdated (UTC)
specReadyReadyPreview, CommentJul 9, 2026 6:59am

Request Review

@github-actions

Copy link
Copy Markdown
Contributor

📓 Docs Drift Check

This PR changes 1 package(s): @objectstack/mcp.

9 hand-written doc(s) reference the affected code and may need an implementation-accuracy re-verification:

  • content/docs/ai/actions-as-tools.mdx(via @objectstack/mcp)
  • content/docs/ai/agents.mdx(via @objectstack/mcp)
  • content/docs/ai/index.mdx(via @objectstack/mcp)
  • content/docs/ai/natural-language-queries.mdx(via @objectstack/mcp)
  • content/docs/api/index.mdx(via @objectstack/mcp)
  • content/docs/deployment/environment-variables.mdx(via @objectstack/mcp)
  • content/docs/plugins/packages.mdx(via @objectstack/mcp)
  • content/docs/protocol/knowledge.mdx(via @objectstack/mcp)
  • content/docs/releases/implementation-status.mdx(via @objectstack/mcp)

Advisory only. To re-verify, run the docs-accuracy-audit workflow scoped to these files:
node scripts/docs-audit/affected-docs.mjs origin/main → pass the list as args.docs.

@os-zhuang
os-zhuang merged commit e097576 into mainJul 9, 2026
16 checks passed
@os-zhuang
os-zhuang deleted the fix/skill-document-action-tools branch July 9, 2026 07:07
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentationImprovements or additions to documentationsize/steststooling

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@os-zhuang