fix: MCP resource name should be the display name, not the description sentence

The manifest set each skill resource's `name` to `skill.name`, which is the
full description sentence. Clients that mint one tool per MCP resource derive
the tool name from `name`; the sentence-length names overflow Anthropic's
128-char tool-name limit, and one invalid name 400s the entire request — so an
agent connected in CLI mode makes zero tool calls (a no-op run).

Use the short `displayName` for the resource `name` instead. The agent-facing
skill-menu still carries the descriptive `skill.name` for skill selection, so
this only shortens the MCP resource name — no menu or selection change.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

Author: gewenyu99

scripts/lib/build-phases.js

@@ -123,8 +123,13 @@ function generateManifest({ resources, uriSchema, version, docContents = {} }) {
                 ? `${scheme}${docPattern.replace('{id}', skill.id)}`
                 : `${scheme}${skillPattern.replace('{group}', skill.group).replace('{id}', skill.shortId)}`;
             const base = {
+                // The MCP resource name is a short, human-readable label (per the
+                // MCP spec) — use the display name, not the description sentence.
+                // Clients that mint a tool per resource derive the tool name from
+                // this; a sentence here overflows the 128-char tool-name limit and
+                // 400s the whole request. Docs carry their short name in `name`.
                 id: skill.id,
-                name: skill.name,
+                name: skill.displayName ?? skill.name,
                 description: skill.description,
                 tags: skill.tags,
                 uri,