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

#2714 Phase 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

vercel Bot commented Jul 9, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
spec Ready Ready Preview, Comment Jul 9, 2026 6:59am

Request Review

@github-actions

github-actions Bot commented Jul 9, 2026

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.

@github-actions github-actions Bot added the size/s label Jul 9, 2026
@os-zhuang
os-zhuang merged commit e097576 into main Jul 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 free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation size/s tests tooling

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant