@withpica/mcp-server-directory 1.1.0 → 1.3.0

package/src/server.ts

@@ -16,13 +16,20 @@ import { ToolRegistry } from "./tools/index.js";
 import { PromptRegistry } from "./prompts/index.js";
 import { logError } from "./utils/errors.js";
 import { DIRECTORY_PRIMER } from "./resources/llms-primer.js";
+import { SkillsRegistry } from "./skills/index.js";
 
 export class DirectoryMcpServer {
   private server: Server;
   private client: DirectoryClient;
   private toolRegistry: ToolRegistry;
   private promptRegistry: PromptRegistry;
+  private skillsRegistry: SkillsRegistry;
   private config: ServerConfig;
+  // Logged once after the MCP `initialize` handshake completes so each
+  // session prints a single client-attribution breadcrumb to stderr
+  // (Railway logs / Sentry breadcrumbs). Directory MCP has no DB audit
+  // pipeline today; this is provenance-only.
+  private clientAttributionLogged = false;
 
   constructor(config: ServerConfig) {
     this.config = config;
@@ -48,6 +55,7 @@ export class DirectoryMcpServer {
 
     this.toolRegistry = new ToolRegistry(this.client);
     this.promptRegistry = new PromptRegistry();
+    this.skillsRegistry = new SkillsRegistry();
     this.setupHandlers();
   }
 
@@ -60,17 +68,36 @@ export class DirectoryMcpServer {
     });
 
     this.server.setRequestHandler(ListResourcesRequestSchema, async () => {
-      return {
-        resources: [
-          {
-            uri: "llms://primer",
-            name: "PICA Domain Primer",
-            description:
-              "Plain-language description of PICA's domain model, entity relationships, and recommended agent workflows. Read this first.",
-            mimeType: "text/markdown",
-          },
-        ],
-      };
+      const resources: Array<{
+        uri: string;
+        name: string;
+        description: string;
+        mimeType: string;
+      }> = [
+        {
+          uri: "llms://primer",
+          name: "PICA Domain Primer",
+          description:
+            "Plain-language description of PICA's domain model, entity relationships, and recommended agent workflows. Read this first.",
+          mimeType: "text/markdown",
+        },
+        {
+          uri: "skill://withpica/index.json",
+          name: "PICA Directory Skills Index",
+          description:
+            "Enumeration of downloadable directory skill methodologies (SEP-2640 bridge). Each entry maps a kebab-case name to its skill:// URI, description, and trigger phrases.",
+          mimeType: "application/json",
+        },
+      ];
+      for (const summary of this.skillsRegistry.listSkills()) {
+        resources.push({
+          uri: `skill://withpica/${summary.name}/SKILL.md`,
+          name: `Skill — ${summary.name}`,
+          description: summary.description,
+          mimeType: "text/markdown",
+        });
+      }
+      return { resources };
     });
 
     this.server.setRequestHandler(
@@ -88,6 +115,9 @@ export class DirectoryMcpServer {
             ],
           };
         }
+        if (uri.startsWith("skill://withpica/")) {
+          return this.readSkillResource(uri);
+        }
         throw new Error(`Resource not found: ${uri}`);
       },
     );
@@ -104,6 +134,8 @@ export class DirectoryMcpServer {
     this.server.setRequestHandler(CallToolRequestSchema, async (request) => {
       const { name, arguments: args } = request.params;
 
+      this.logClientAttributionOnce();
+
       if (this.config.debug) {
         console.error(`[Directory MCP] Executing tool: ${name}`, args);
       }
@@ -117,6 +149,86 @@ export class DirectoryMcpServer {
     });
   }
 
+  /**
+   * Emit a single stderr line per session naming the client that handshook
+   * with us — sourced from `clientInfo` on the MCP `initialize` request.
+   * Self-declared, NOT authenticated; treat as provenance metadata only.
+   *
+   * Directory MCP is unauthenticated and writes no audit rows, so this
+   * stderr breadcrumb is the only attribution surface today. When/if a
+   * directory audit pipeline lands, fold these values into McpAuditEntry
+   * (the wire shape already carries optional client_name + client_version).
+   */
+  private logClientAttributionOnce(): void {
+    if (this.clientAttributionLogged) return;
+    const info = this.server.getClientVersion?.();
+    if (!info) return;
+    this.clientAttributionLogged = true;
+    console.error(
+      `[Directory MCP] client: ${info.name ?? "unknown"} v${info.version ?? "unknown"}`,
+    );
+  }
+
+  /**
+   * Resolve a skill:// URI. ADR-140 Phase 2b — SEP-2640 bridge pattern.
+   * Two URI shapes:
+   *   - skill://withpica/index.json
+   *   - skill://withpica/<name>/SKILL.md
+   */
+  private readSkillResource(uri: string): {
+    contents: Array<{ uri: string; mimeType: string; text: string }>;
+  } {
+    if (uri === "skill://withpica/index.json") {
+      const index = this.skillsRegistry.listSkills().map((s) => ({
+        name: s.name,
+        uri: `skill://withpica/${s.name}/SKILL.md`,
+        description: s.description,
+        triggers: s.triggers,
+        audience: s.audience,
+      }));
+      return {
+        contents: [
+          {
+            uri,
+            mimeType: "application/json",
+            text: JSON.stringify({ skills: index }, null, 2),
+          },
+        ],
+      };
+    }
+    const match = uri.match(/^skill:\/\/withpica\/([a-z0-9-]+)\/SKILL\.md$/);
+    if (!match) {
+      throw new Error(`Malformed skill URI: ${uri}`);
+    }
+    const name = match[1];
+    if (!this.skillsRegistry.has(name)) {
+      throw new Error(`Skill not found: ${name}`);
+    }
+    const skill = this.skillsRegistry.getSkill(name);
+    const frontmatter = [
+      "---",
+      `name: ${skill.name}`,
+      `description: ${skill.description}`,
+      "triggers:",
+      ...skill.triggers.map((t) => `  - ${t}`),
+      `audience: ${skill.audience}`,
+      "tools_required:",
+      ...skill.tools_required.map((t) => `  - ${t}`),
+      `output: ${skill.output}`,
+      "---",
+      "",
+    ].join("\n");
+    return {
+      contents: [
+        {
+          uri,
+          mimeType: "text/markdown",
+          text: frontmatter + skill.body,
+        },
+      ],
+    };
+  }
+
   async start(): Promise<void> {
     const transport = new StdioServerTransport();
     await this.server.connect(transport);

package/src/skills/find-music-for-sync-brief/SKILL.md

@@ -0,0 +1,89 @@
+<!-- Copyright (c) 2024-2026 Withpica Ltd. All rights reserved. -->
+---
+name: find-music-for-sync-brief
+description: Find music in the PICA public directory for a sync brief — translate a mood/vibe/scene description into audio search parameters, run the search, and return a rights-aware shortlist with credit details for licensing.
+triggers:
+  - find music for a sync brief
+  - find me a track for a film scene
+  - music for this brief
+  - find an uplifting indie folk track
+  - I need music that sounds like
+audience: sync supervisor / music library / ad agency / film & TV music team
+tools_required:
+  - directory_search_recordings
+  - directory_lookup_work
+  - directory_lookup_person
+output: A rights-aware shortlist — each result with title, artist, audio characteristics (BPM, key, energy), credits summary for licensing contact, and any flags (unattested credits, low registration score, missing ISRC).
+---
+
+# find-music-for-sync-brief
+
+Help the user find music in PICA's public directory for a sync brief. Translate their description (mood, vibe, scene, era, vocal style) into search parameters, run the search, and return a shortlist that includes the information they need to license — not just the track.
+
+This is a public skill — any AI agent connected to the directory MCP can use it. The directory only contains works from organisations that have opted in.
+
+## Step 1 — Translate the brief
+
+Most sync briefs are described in natural language. Translate before searching:
+
+| User says | Search parameters |
+|---|---|
+| upbeat / uplifting / driving | `min_energy: 0.6`, `min_danceability: 0.5` |
+| dark / moody / brooding | `max_energy: 0.4`, `key_mode: minor` |
+| chill / ambient / background | `max_energy: 0.4`, `max_bpm: 100` |
+| epic / cinematic | `min_energy: 0.7`, often longer durations |
+| specific BPM ("around 120bpm") | `min_bpm: 115`, `max_bpm: 125` |
+| acoustic / sparse | typically lower `danceability` and `energy` |
+| vocal / instrumental | filter on `has_vocals` if available |
+
+If the user gives an artist or song reference ("something like X"), look up that reference first via `directory_lookup_work` or `directory_search_recordings` by title — then mimic its audio profile in the new search.
+
+## Step 2 — Search
+
+Call `directory_search_recordings` with the parameters from step 1. Default limit is fine for an initial scan.
+
+If the brief is vague, do two searches with slightly different parameters and compare. If too many results, narrow on BPM or key. If too few, widen the BPM range first, then drop key_mode.
+
+## Step 3 — Lookup full details
+
+For the top 5-10 promising results, call `directory_lookup_work` to get:
+
+- **Who wrote it** — credits with IPI numbers (essential for the licensing contact path)
+- **Whether credits are attested** — an unattested credit means the ownership claim hasn't been verified; flag this
+- **DSP links** (Spotify, Apple Music) — the user wants to listen before shortlisting
+- **Registration score** — higher score = cleaner rights chain = easier to license
+
+For high-value matches where the user wants to know more about the creator, call `directory_lookup_person` with the IPI to see the writer's broader catalog and collaborator network.
+
+## Step 4 — Present the shortlist
+
+Present each shortlisted track with:
+
+- **Title, artist** — what the user listens for
+- **Audio profile** — BPM, key, energy (one line, the supervisor's quick filter)
+- **Credits summary** — writers with IPI, publisher if known (this is the licensing path)
+- **DSP links** — so the user can listen
+- **Flags** — unattested credits, low registration score, missing ISRC, multiple writers (= multi-party clearance needed)
+
+If a track has all green flags (attested credits, high registration score, ISRC present, single or low-count writers), call it out as "clean to clear."
+
+## Step 5 — If no results
+
+If the search returns nothing, walk the user back through the brief:
+- Widen the BPM range first (most supervisors don't actually mind ±10 BPM)
+- Drop the key filter (key matching matters less than mood for most placements)
+- Drop the energy bounds and search by genre/mood text only
+- Suggest the user expand to adjacent moods ("uplifting" → also try "warm" or "hopeful")
+
+Be honest if the directory is small for a specific style — PICA's directory grows with org opt-ins; some niches are sparse.
+
+## What not to do
+
+- Don't claim to license tracks — this skill returns a shortlist, not a license. The user contacts the rights holder via the credits summary.
+- Don't speculate about pricing — sync fees are negotiated per-placement
+- Don't return results from outside the directory (no general web search, no Spotify metadata directly) — the directory is the source of truth for this skill
+- Don't return tracks with no attested credits without flagging them as "rights chain unverified"
+
+## Follow-on skills
+
+- `due-diligence-on-creator` (Stage 2 candidate) — for deeper research on a specific writer before reaching out

package/src/skills/index.ts

@@ -0,0 +1,63 @@
+// Copyright (c) 2024-2026 Withpica Ltd. All rights reserved.
+
+/**
+ * Skills Registry — directory MCP, ADR-140 Phase 2b
+ *
+ * Sister of mcp-server/src/skills/index.ts. Same shape, scoped to the public
+ * directory MCP. Body bytes are generated at build time by
+ * `scripts/build-skills.ts` and inlined into `skills.generated.ts`.
+ *
+ * Public/anonymous surface — every skill body must clear the IP-protection
+ * lint (scripts/lint-mcp-tools.ts Rule 14) before publishing. Directory MCP
+ * skills MUST NOT quote from trade-secret directories listed in
+ * `.claude/rules/ip-protection.md`.
+ */
+
+import { SKILLS, SKILL_NAMES, type Skill } from "./skills.generated.js";
+
+export interface SkillSummary {
+  name: string;
+  description: string;
+  triggers: string[];
+  audience: string;
+}
+
+export class SkillsRegistry {
+  listNames(): ReadonlyArray<string> {
+    return SKILL_NAMES;
+  }
+
+  listSkills(): SkillSummary[] {
+    return SKILL_NAMES.map((name) => {
+      const skill = SKILLS[name];
+      return {
+        name: skill.name,
+        description: skill.description,
+        triggers: skill.triggers,
+        audience: skill.audience,
+      };
+    });
+  }
+
+  getSkill(name: string): Skill {
+    const skill = SKILLS[name];
+    if (!skill) {
+      throw new SkillNotFoundError(name);
+    }
+    return skill;
+  }
+
+  has(name: string): boolean {
+    return name in SKILLS;
+  }
+}
+
+export class SkillNotFoundError extends Error {
+  constructor(public readonly skillName: string) {
+    super(`Skill not found: ${skillName}`);
+    this.name = "SkillNotFoundError";
+  }
+}
+
+export { SKILLS, SKILL_NAMES };
+export type { Skill };

package/src/skills/skills.generated.ts

@@ -0,0 +1,33 @@
+// Copyright (c) 2024-2026 Withpica Ltd. All rights reserved.
+
+/**
+ * GENERATED FILE — DO NOT EDIT BY HAND.
+ *
+ * Regenerated by `scripts/build-skills.ts` from `src/skills/<name>/SKILL.md`.
+ * Runs in `prebuild`. See ADR-140 Phase 2b.
+ */
+
+export interface Skill {
+  name: string;
+  description: string;
+  triggers: string[];
+  audience: string;
+  tools_required: string[];
+  output: string;
+  body: string;
+}
+
+
+export const SKILLS: Record<string, Skill> = {
+  "find-music-for-sync-brief": {
+    name: "find-music-for-sync-brief",
+    description: "Find music in the PICA public directory for a sync brief — translate a mood/vibe/scene description into audio search parameters, run the search, and return a rights-aware shortlist with credit details for licensing.",
+    triggers: ["find music for a sync brief","find me a track for a film scene","music for this brief","find an uplifting indie folk track","I need music that sounds like"],
+    audience: "sync supervisor / music library / ad agency / film & TV music team",
+    tools_required: ["directory_search_recordings","directory_lookup_work","directory_lookup_person"],
+    output: "A rights-aware shortlist — each result with title, artist, audio characteristics (BPM, key, energy), credits summary for licensing contact, and any flags (unattested credits, low registration score, missing ISRC).",
+    body: "# find-music-for-sync-brief\n\nHelp the user find music in PICA's public directory for a sync brief. Translate their description (mood, vibe, scene, era, vocal style) into search parameters, run the search, and return a shortlist that includes the information they need to license — not just the track.\n\nThis is a public skill — any AI agent connected to the directory MCP can use it. The directory only contains works from organisations that have opted in.\n\n## Step 1 — Translate the brief\n\nMost sync briefs are described in natural language. Translate before searching:\n\n| User says | Search parameters |\n|---|---|\n| upbeat / uplifting / driving | `min_energy: 0.6`, `min_danceability: 0.5` |\n| dark / moody / brooding | `max_energy: 0.4`, `key_mode: minor` |\n| chill / ambient / background | `max_energy: 0.4`, `max_bpm: 100` |\n| epic / cinematic | `min_energy: 0.7`, often longer durations |\n| specific BPM (\"around 120bpm\") | `min_bpm: 115`, `max_bpm: 125` |\n| acoustic / sparse | typically lower `danceability` and `energy` |\n| vocal / instrumental | filter on `has_vocals` if available |\n\nIf the user gives an artist or song reference (\"something like X\"), look up that reference first via `directory_lookup_work` or `directory_search_recordings` by title — then mimic its audio profile in the new search.\n\n## Step 2 — Search\n\nCall `directory_search_recordings` with the parameters from step 1. Default limit is fine for an initial scan.\n\nIf the brief is vague, do two searches with slightly different parameters and compare. If too many results, narrow on BPM or key. If too few, widen the BPM range first, then drop key_mode.\n\n## Step 3 — Lookup full details\n\nFor the top 5-10 promising results, call `directory_lookup_work` to get:\n\n- **Who wrote it** — credits with IPI numbers (essential for the licensing contact path)\n- **Whether credits are attested** — an unattested credit means the ownership claim hasn't been verified; flag this\n- **DSP links** (Spotify, Apple Music) — the user wants to listen before shortlisting\n- **Registration score** — higher score = cleaner rights chain = easier to license\n\nFor high-value matches where the user wants to know more about the creator, call `directory_lookup_person` with the IPI to see the writer's broader catalog and collaborator network.\n\n## Step 4 — Present the shortlist\n\nPresent each shortlisted track with:\n\n- **Title, artist** — what the user listens for\n- **Audio profile** — BPM, key, energy (one line, the supervisor's quick filter)\n- **Credits summary** — writers with IPI, publisher if known (this is the licensing path)\n- **DSP links** — so the user can listen\n- **Flags** — unattested credits, low registration score, missing ISRC, multiple writers (= multi-party clearance needed)\n\nIf a track has all green flags (attested credits, high registration score, ISRC present, single or low-count writers), call it out as \"clean to clear.\"\n\n## Step 5 — If no results\n\nIf the search returns nothing, walk the user back through the brief:\n- Widen the BPM range first (most supervisors don't actually mind ±10 BPM)\n- Drop the key filter (key matching matters less than mood for most placements)\n- Drop the energy bounds and search by genre/mood text only\n- Suggest the user expand to adjacent moods (\"uplifting\" → also try \"warm\" or \"hopeful\")\n\nBe honest if the directory is small for a specific style — PICA's directory grows with org opt-ins; some niches are sparse.\n\n## What not to do\n\n- Don't claim to license tracks — this skill returns a shortlist, not a license. The user contacts the rights holder via the credits summary.\n- Don't speculate about pricing — sync fees are negotiated per-placement\n- Don't return results from outside the directory (no general web search, no Spotify metadata directly) — the directory is the source of truth for this skill\n- Don't return tracks with no attested credits without flagging them as \"rights chain unverified\"\n\n## Follow-on skills\n\n- `due-diligence-on-creator` (Stage 2 candidate) — for deeper research on a specific writer before reaching out\n",
+  },
+};
+
+export const SKILL_NAMES: ReadonlyArray<string> = ["find-music-for-sync-brief"];

package/src/tools/chain.ts

@@ -0,0 +1,118 @@
+// Copyright (c) 2024-2026 Withpica Ltd. All rights reserved.
+
+import { DirectoryClient } from "../client.js";
+import { ToolDefinition, ToolExecutor } from "./index.js";
+
+export class DirectoryChainTools {
+  private client: DirectoryClient;
+
+  constructor(client: DirectoryClient) {
+    this.client = client;
+  }
+
+  getTools(): Array<{ definition: ToolDefinition; executor: ToolExecutor }> {
+    return [
+      {
+        definition: {
+          name: "directory_chain",
+          tier: "read",
+          description:
+            "Use when the user asks: 'show me the rights chain for X', " +
+            "'give me everything on this song', 'what's the full picture for X?', " +
+            "'trace the chain from this writer to that work'. " +
+            "Graph lookup — resolves a query, ISWC, or ISRC to its full rights chain in one call. " +
+            "Each result bundles the work, its writers and publishers (names + roles, no IPIs or splits), " +
+            "the linked recording (ISRC, DSP presence, duration), and audio characteristics (BPM, key, energy). " +
+            "Prefer over stitching list_works + lookup_work + lookup_person when you want the whole graph " +
+            "around a track in a single round-trip. " +
+            "Supports audio-only queries (min_bpm, key, mood, etc.) when you don't have a text query. " +
+            "→ then: directory_lookup_work (deeper work detail), directory_lookup_person (research a named writer)",
+          inputSchema: {
+            type: "object",
+            properties: {
+              q: {
+                type: "string",
+                description:
+                  "Free-text query (work title, creator name, etc.). Minimum 2 characters.",
+              },
+              identifier: {
+                type: "string",
+                description: "ISWC or ISRC — direct identifier lookup",
+              },
+              min_bpm: { type: "number", description: "Minimum BPM" },
+              max_bpm: { type: "number", description: "Maximum BPM" },
+              key: {
+                type: "string",
+                description: "Musical key (e.g. 'C', 'F#', 'Bb')",
+              },
+              key_mode: {
+                type: "string",
+                enum: ["major", "minor"],
+                description: "Key mode",
+              },
+              min_energy: {
+                type: "number",
+                description: "Minimum energy (0–1)",
+              },
+              max_energy: {
+                type: "number",
+                description: "Maximum energy (0–1)",
+              },
+              mood: {
+                type: "string",
+                description: "Mood filter (exact match)",
+              },
+              limit: {
+                type: "number",
+                description: "Max results (default 10, max 20)",
+              },
+            },
+          },
+        },
+        executor: this.chainLookup.bind(this),
+      },
+    ];
+  }
+
+  private async chainLookup(args: Record<string, any>): Promise<any> {
+    const params: Record<string, string> = {};
+
+    if (args.q) params.q = String(args.q);
+    if (args.identifier) params.identifier = String(args.identifier);
+
+    const numericKeys = [
+      "min_bpm",
+      "max_bpm",
+      "min_energy",
+      "max_energy",
+      "limit",
+    ];
+    for (const k of numericKeys) {
+      if (args[k] != null) params[k] = String(args[k]);
+    }
+
+    if (args.key) params.key = args.key;
+    if (args.key_mode) params.key_mode = args.key_mode;
+    if (args.mood) params.mood = args.mood;
+
+    const response: any = await this.client.request("/chain", params);
+
+    const results: any[] = response.results || [];
+
+    return {
+      content: [
+        {
+          type: "text",
+          text:
+            results.length === 0
+              ? "No chain results."
+              : `Found ${results.length} chain result(s). See structuredContent for the full graph.`,
+        },
+      ],
+      structuredContent: {
+        total: response.total ?? results.length,
+        results,
+      },
+    };
+  }
+}

package/src/tools/index.ts

@@ -5,15 +5,28 @@ import { DirectoryWorksTools } from "./works.js";
 import { DirectoryPeopleTools } from "./people.js";
 import { DirectorySearchTools } from "./search.js";
 import { DirectoryRecordingsTools } from "./recordings.js";
+import { DirectoryChainTools } from "./chain.js";
+import { DirectorySkillsTools } from "./skills.js";
 import { formatError, logError, ToolExecutionError } from "../utils/errors.js";
 
+/**
+ * ADR-230 — authority tier on every directory_* tool. Sister of the
+ * creator + team MCP's `Tier` declaration. Every directory tool is
+ * `read` (public catalogue lookup with no mutation surface), but the
+ * field is required for cross-surface uniformity and lint enforcement.
+ */
+export type Tier = "read" | "draft" | "write" | "destructive";
+
 export interface ToolDefinition {
   name: string;
+  /** ADR-230 — see {@link Tier}. Required on every directory tool. */
+  tier: Tier;
   description: string;
   inputSchema: {
     type: string;
     properties: Record<string, any>;
     required?: string[];
+    additionalProperties?: boolean;
   };
 }
 
@@ -39,6 +52,8 @@ export class ToolRegistry {
       new DirectoryPeopleTools(client),
       new DirectorySearchTools(client),
       new DirectoryRecordingsTools(client),
+      new DirectoryChainTools(client),
+      new DirectorySkillsTools(),
     ];
 
     for (const group of groups) {

package/src/tools/people.ts

@@ -2,7 +2,7 @@
 
 import { DirectoryClient } from "../client.js";
 import { ToolDefinition, ToolExecutor } from "./index.js";
-import { formatAsText, formatStructuredList } from "../utils/formatting.js";
+import { formatStructuredList } from "../utils/formatting.js";
 
 export class DirectoryPeopleTools {
   private client: DirectoryClient;
@@ -16,10 +16,14 @@ export class DirectoryPeopleTools {
       {
         definition: {
           name: "directory_list_people",
+          tier: "read",
           description:
+            "Use when the user asks: 'list creators in PICA', 'people starting with B', " +
+            "'browse the people directory'. " +
             "Browse and filter creators in the PICA public directory. " +
-            "Search by name, ISNI, or IPI number. Returns paginated results. " +
-            "→ then: directory_lookup_person_full (full profile with works and identifiers)",
+            "Filter by free-text (q), ISNI, IPI, or starting letter (A–Z, or '#' for numeric). " +
+            "Returns paginated results. " +
+            "→ then: directory_lookup_person (full profile with works and identifiers)",
           inputSchema: {
             type: "object",
             properties: {
@@ -35,6 +39,11 @@ export class DirectoryPeopleTools {
                 type: "string",
                 description: "Filter by IPI number",
               },
+              letter: {
+                type: "string",
+                description:
+                  "Filter by last name starting letter (A–Z), or '#' for numeric",
+              },
               page: {
                 type: "number",
                 description: "Page number (default: 1, 20 results per page)",
@@ -47,10 +56,16 @@ export class DirectoryPeopleTools {
       {
         definition: {
           name: "directory_lookup_person",
+          tier: "read",
           description:
-            "Get full details of a single creator by their global creator ID, ISNI, IPI, or MusicBrainz ID. " +
-            "Returns roles, works, identifiers, and profile information. " +
-            "→ then: directory_lookup_person_full (agent-optimised markdown), directory_lookup_work_full (inspect a credited work)",
+            "Use when the user asks: 'lookup IPI 12345', 'who is creator X?', " +
+            "'tell me about creator X', 'lookup ISNI X'. " +
+            "Also call this BEFORE directory_list_works when the input person name is ambiguous " +
+            "(e.g. 'Bowie', 'Sade', 'Madonna') to surface candidate persons for disambiguation. " +
+            "Get full details of a creator by global creator ID (UUID), ISNI, IPI number, or MusicBrainz ID. " +
+            "Returns identifiers, credited works with roles, collaborator network, and verification score " +
+            "as a structured markdown summary. " +
+            "→ then: directory_lookup_work (inspect a credited work), directory_search_recordings (find their recordings by audio)",
           inputSchema: {
             type: "object",
             properties: {
@@ -65,28 +80,6 @@ export class DirectoryPeopleTools {
         },
         executor: this.lookupPerson.bind(this),
       },
-      {
-        definition: {
-          name: "directory_lookup_person_full",
-          description:
-            "Get complete details about a person in the public directory — identifiers (IPI, ISNI), " +
-            "credited works with roles, collaborator network, and verification score. " +
-            "Returns a structured markdown summary. " +
-            "→ then: directory_lookup_work_full (inspect a specific work), directory_search_recordings (find their recordings by audio)",
-          inputSchema: {
-            type: "object",
-            properties: {
-              id: {
-                type: "string",
-                description:
-                  "Global creator ID (UUID), ISNI, IPI number, or MusicBrainz ID",
-              },
-            },
-            required: ["id"],
-          },
-        },
-        executor: this.lookupPersonFull.bind(this),
-      },
     ];
   }
 
@@ -103,6 +96,7 @@ export class DirectoryPeopleTools {
     if (args.q) params.q = args.q;
     if (args.isni) params.isni = args.isni;
     if (args.ipi) params.ipi = args.ipi;
+    if (args.letter) params.letter = args.letter;
 
     const response: any = await this.client.request("/people", params);
 
@@ -118,14 +112,6 @@ export class DirectoryPeopleTools {
       `/people/${encodeURIComponent(args.id)}`,
     );
 
-    return formatAsText(response.data);
-  }
-
-  private async lookupPersonFull(args: Record<string, any>): Promise<any> {
-    const response: any = await this.client.request(
-      `/people/${encodeURIComponent(args.id)}`,
-    );
-
     const data = response.data;
     if (!data) {
       return {
@@ -136,12 +122,10 @@ export class DirectoryPeopleTools {
 
     const lines: string[] = [];
 
-    // Name heading
     const name = data.name || data.full_name || data.display_name || "Unknown";
     lines.push(`## ${name}`);
     lines.push("");
 
-    // Score / verification
     if (data.score !== undefined && data.score !== null) {
       const tier =
         data.score >= 80
@@ -153,7 +137,6 @@ export class DirectoryPeopleTools {
       lines.push("");
     }
 
-    // Identifiers
     const identifiers: Array<{ type: string; value: string }> = [];
     if (data.ipi || data.ipi_number)
       identifiers.push({ type: "IPI", value: data.ipi || data.ipi_number });
@@ -174,7 +157,6 @@ export class DirectoryPeopleTools {
     }
     lines.push("");
 
-    // Works
     const works: any[] = data.works || data.credited_works || [];
     lines.push(`### Works (${works.length})`);
     if (works.length > 0) {
@@ -190,7 +172,6 @@ export class DirectoryPeopleTools {
     }
     lines.push("");
 
-    // Collaborators
     const collaborators: any[] = data.collaborators || [];
     lines.push(`### Collaborators (${collaborators.length})`);
     if (collaborators.length > 0) {