run-dmcp 0.3.0 → 0.5.0

package/dist/register/core.js

@@ -1,9 +1,24 @@
 import { z } from "zod";
 import * as gameTools from "../tools/game.js";
 import * as rulesTools from "../tools/rules.js";
-import { LIMITS } from "../utils/validation.js";
+import { LIMITS, validatedSchemas } from "../utils/validation.js";
 import { getGameUrl, getWebUiBaseUrl } from "../utils/webui.js";
 import { ANNOTATIONS } from "../utils/tool-annotations.js";
+/**
+ * One answer from the preferences interview: what the player said, or a
+ * delegation to the DM. `save_game_preferences` takes eighteen of these and
+ * they were eighteen copies of the same three fields, which is how half of
+ * them ended up unbounded.
+ *
+ * A FUNCTION, not a shared constant. Reusing one instance eighteen times
+ * inside a single tool publishes seventeen `$ref` pointers where a consumer
+ * expects a declaration -- see the comment on `validatedSchemas`.
+ */
+const preferenceAnswer = () => z.object({
+    value: validatedSchemas.token.nullable(),
+    delegatedToDM: z.boolean(),
+    notes: validatedSchemas.description.optional(),
+});
 export function registerCoreTools(server) {
     // ============================================================================
     // GAME TOOLS
@@ -109,7 +124,7 @@ export function registerCoreTools(server) {
     server.registerTool("get_game_state", {
         description: "Get full current state overview for a game",
         inputSchema: {
-            gameId: z.string().describe("The game ID"),
+            gameId: validatedSchemas.id.describe("The game ID"),
         },
         annotations: ANNOTATIONS.READ_ONLY,
     }, async ({ gameId }) => {
@@ -140,7 +155,7 @@ export function registerCoreTools(server) {
     server.registerTool("delete_game", {
         description: "Delete a game and all its data. This is IRREVERSIBLE and removes all characters, locations, quests, and history.",
         inputSchema: {
-            gameId: z.string().describe("The game ID to delete"),
+            gameId: validatedSchemas.id.describe("The game ID to delete"),
         },
         annotations: ANNOTATIONS.DESTRUCTIVE,
     }, async ({ gameId }) => {
@@ -153,7 +168,7 @@ export function registerCoreTools(server) {
     server.registerTool("update_game", {
         description: "Update a game's name, setting, or style",
         inputSchema: {
-            gameId: z.string().describe("The game ID to update"),
+            gameId: validatedSchemas.id.describe("The game ID to update"),
             name: z.string().min(1).max(LIMITS.NAME_MAX).optional().describe("New name for the game"),
             setting: z.string().min(1).max(LIMITS.DESCRIPTION_MAX).optional().describe("New setting description"),
             style: z.string().min(1).max(LIMITS.NAME_MAX).optional().describe("New narrative style"),
@@ -190,8 +205,8 @@ export function registerCoreTools(server) {
     server.registerTool("set_game_title_image", {
         description: "Set or remove the title image for a game",
         inputSchema: {
-            gameId: z.string().describe("The game ID"),
-            imageId: z.string().nullable().describe("The image ID to set as title image, or null to remove"),
+            gameId: validatedSchemas.id.describe("The game ID"),
+            imageId: validatedSchemas.id.nullable().describe("The image ID to set as title image, or null to remove"),
         },
         annotations: ANNOTATIONS.SET,
     }, async ({ gameId, imageId }) => {
@@ -219,8 +234,8 @@ export function registerCoreTools(server) {
     server.registerTool("set_game_favicon", {
         description: "Set or remove the favicon image for a game. This image will be displayed as the browser favicon when viewing this game in the web UI. Should be a square image (ideally 32x32 or larger).",
         inputSchema: {
-            gameId: z.string().describe("The game ID"),
-            imageId: z.string().nullable().describe("The image ID to set as favicon, or null to remove"),
+            gameId: validatedSchemas.id.describe("The game ID"),
+            imageId: validatedSchemas.id.nullable().describe("The image ID to set as favicon, or null to remove"),
         },
         annotations: ANNOTATIONS.SET,
     }, async ({ gameId, imageId }) => {
@@ -596,30 +611,30 @@ export function registerCoreTools(server) {
     server.registerTool("save_game_preferences", {
         description: "Save the player's game preferences after the interview",
         inputSchema: {
-            gameId: z.string().describe("The game ID"),
+            gameId: validatedSchemas.id.describe("The game ID"),
             preferences: z.object({
-                genre: z.object({ value: z.string().nullable(), delegatedToDM: z.boolean(), notes: z.string().optional() }),
-                tone: z.object({ value: z.string().nullable(), delegatedToDM: z.boolean(), notes: z.string().optional() }),
-                setting: z.object({ value: z.string().nullable(), delegatedToDM: z.boolean(), notes: z.string().optional() }),
-                complexity: z.object({ value: z.string().nullable(), delegatedToDM: z.boolean(), notes: z.string().optional() }),
-                combatFrequency: z.object({ value: z.string().nullable(), delegatedToDM: z.boolean(), notes: z.string().optional() }),
-                combatStyle: z.object({ value: z.string().nullable(), delegatedToDM: z.boolean(), notes: z.string().optional() }),
-                lethality: z.object({ value: z.string().nullable(), delegatedToDM: z.boolean(), notes: z.string().optional() }),
-                narrativeStyle: z.object({ value: z.string().nullable(), delegatedToDM: z.boolean(), notes: z.string().optional() }),
-                playerAgency: z.object({ value: z.string().nullable(), delegatedToDM: z.boolean(), notes: z.string().optional() }),
-                npcDepth: z.object({ value: z.string().nullable(), delegatedToDM: z.boolean(), notes: z.string().optional() }),
-                romanceContent: z.object({ value: z.string().nullable(), delegatedToDM: z.boolean(), notes: z.string().optional() }),
-                worldFamiliarity: z.object({ value: z.string().nullable(), delegatedToDM: z.boolean(), notes: z.string().optional() }),
-                magicOrTechLevel: z.object({ value: z.string().nullable(), delegatedToDM: z.boolean(), notes: z.string().optional() }),
-                politicalComplexity: z.object({ value: z.string().nullable(), delegatedToDM: z.boolean(), notes: z.string().optional() }),
-                sessionLength: z.object({ value: z.string().nullable(), delegatedToDM: z.boolean(), notes: z.string().optional() }),
-                pacingPreference: z.object({ value: z.string().nullable(), delegatedToDM: z.boolean(), notes: z.string().optional() }),
-                characterCreation: z.object({ value: z.string().nullable(), delegatedToDM: z.boolean(), notes: z.string().optional() }),
-                startingPowerLevel: z.object({ value: z.string().nullable(), delegatedToDM: z.boolean(), notes: z.string().optional() }),
-                contentToAvoid: z.array(z.string()),
-                contentToInclude: z.array(z.string()),
-                inspirations: z.array(z.string()),
-                additionalNotes: z.string(),
+                genre: preferenceAnswer(),
+                tone: preferenceAnswer(),
+                setting: preferenceAnswer(),
+                complexity: preferenceAnswer(),
+                combatFrequency: preferenceAnswer(),
+                combatStyle: preferenceAnswer(),
+                lethality: preferenceAnswer(),
+                narrativeStyle: preferenceAnswer(),
+                playerAgency: preferenceAnswer(),
+                npcDepth: preferenceAnswer(),
+                romanceContent: preferenceAnswer(),
+                worldFamiliarity: preferenceAnswer(),
+                magicOrTechLevel: preferenceAnswer(),
+                politicalComplexity: preferenceAnswer(),
+                sessionLength: preferenceAnswer(),
+                pacingPreference: preferenceAnswer(),
+                characterCreation: preferenceAnswer(),
+                startingPowerLevel: preferenceAnswer(),
+                contentToAvoid: validatedSchemas.stringArray,
+                contentToInclude: validatedSchemas.stringArray,
+                inspirations: validatedSchemas.stringArray,
+                additionalNotes: validatedSchemas.description,
             }).describe("The collected preferences"),
         },
         annotations: ANNOTATIONS.CREATE,
@@ -654,7 +669,7 @@ export function registerCoreTools(server) {
     server.registerTool("get_game_preferences", {
         description: "Get the saved game preferences for a game",
         inputSchema: {
-            gameId: z.string().describe("The game ID"),
+            gameId: validatedSchemas.id.describe("The game ID"),
         },
         annotations: ANNOTATIONS.READ_ONLY,
     }, async ({ gameId }) => {
@@ -676,96 +691,98 @@ export function registerCoreTools(server) {
         defaultTool: z.enum(["dalle", "sdxl", "midjourney", "comfyui", "flux", "other"]).optional()
             .describe("Default image generation tool/service"),
         defaultStyle: z.object({
-            artisticStyle: z.string().optional().describe("E.g., 'digital painting', 'oil painting', 'anime'"),
-            mood: z.string().optional().describe("E.g., 'dark', 'epic', 'whimsical'"),
-            colorScheme: z.string().optional().describe("E.g., 'warm', 'cold', 'muted', 'vibrant'"),
-            qualityTags: z.array(z.string()).optional().describe("E.g., ['highly detailed', '8k', 'masterpiece']"),
-            negativePrompts: z.array(z.string()).optional().describe("Things to avoid globally"),
-            influences: z.array(z.string()).optional().describe("Artist/game/movie style references"),
+            artisticStyle: validatedSchemas.token.optional().describe("E.g., 'digital painting', 'oil painting', 'anime'"),
+            mood: validatedSchemas.token.optional().describe("E.g., 'dark', 'epic', 'whimsical'"),
+            colorScheme: validatedSchemas.token.optional().describe("E.g., 'warm', 'cold', 'muted', 'vibrant'"),
+            qualityTags: validatedSchemas.stringArray.optional().describe("E.g., ['highly detailed', '8k', 'masterpiece']"),
+            negativePrompts: validatedSchemas.stringArray.optional().describe("Things to avoid globally"),
+            influences: validatedSchemas.stringArray.optional().describe("Artist/game/movie style references"),
         }).optional().describe("Default style settings applied to all images"),
         comfyui: z.object({
-            endpoint: z.string().optional().describe("ComfyUI server URL"),
-            checkpoint: z.string().optional().describe("Default checkpoint model"),
+            endpoint: validatedSchemas.token.optional().describe("ComfyUI server URL"),
+            checkpoint: validatedSchemas.token.optional().describe("Default checkpoint model"),
             loras: z.array(z.object({
-                name: z.string(),
+                name: validatedSchemas.token,
                 weight: z.number(),
             })).optional().describe("LoRA models to apply"),
             samplerSettings: z.object({
-                sampler: z.string().optional(),
-                scheduler: z.string().optional(),
+                sampler: validatedSchemas.token.optional(),
+                scheduler: validatedSchemas.token.optional(),
                 steps: z.number().optional(),
                 cfg: z.number().optional(),
             }).optional(),
             workflows: z.record(z.string(), z.object({
-                name: z.string().describe("Human-readable workflow name"),
-                description: z.string().optional().describe("What this workflow does"),
+                name: validatedSchemas.token.describe("Human-readable workflow name"),
+                description: validatedSchemas.description.optional().describe("What this workflow does"),
                 workflow: z.record(z.string(), z.unknown()).describe("Full ComfyUI workflow JSON (API format)"),
                 inputNodes: z.object({
-                    positivePrompt: z.string().optional().describe("Node ID for positive prompt"),
-                    negativePrompt: z.string().optional().describe("Node ID for negative prompt"),
-                    checkpoint: z.string().optional().describe("Node ID for checkpoint loader"),
-                    seed: z.string().optional().describe("Node ID for seed/noise"),
-                    width: z.string().optional().describe("Node ID for width"),
-                    height: z.string().optional().describe("Node ID for height"),
-                    steps: z.string().optional().describe("Node ID for steps"),
-                    cfg: z.string().optional().describe("Node ID for CFG scale"),
-                    sampler: z.string().optional().describe("Node ID for sampler"),
-                    scheduler: z.string().optional().describe("Node ID for scheduler"),
+                    positivePrompt: validatedSchemas.token.optional().describe("Node ID for positive prompt"),
+                    negativePrompt: validatedSchemas.token.optional().describe("Node ID for negative prompt"),
+                    checkpoint: validatedSchemas.token.optional().describe("Node ID for checkpoint loader"),
+                    seed: validatedSchemas.token.optional().describe("Node ID for seed/noise"),
+                    width: validatedSchemas.token.optional().describe("Node ID for width"),
+                    height: validatedSchemas.token.optional().describe("Node ID for height"),
+                    steps: validatedSchemas.token.optional().describe("Node ID for steps"),
+                    cfg: validatedSchemas.token.optional().describe("Node ID for CFG scale"),
+                    sampler: validatedSchemas.token.optional().describe("Node ID for sampler"),
+                    scheduler: validatedSchemas.token.optional().describe("Node ID for scheduler"),
                 }).optional().describe("Node IDs for dynamic value injection"),
             })).optional().describe("Named workflow templates - full ComfyUI workflows"),
-            defaultWorkflowId: z.string().optional().describe("ID of the default workflow to use"),
-            defaultWorkflow: z.string().optional().describe("Legacy: Workflow name or ID"),
+            defaultWorkflowId: validatedSchemas.id.optional().describe("ID of the default workflow to use"),
+            defaultWorkflow: validatedSchemas.token.optional().describe("Legacy: Workflow name or ID"),
             workflowOverrides: z.record(z.string(), z.unknown()).optional(),
         }).optional().describe("ComfyUI-specific settings"),
         dalle: z.object({
-            model: z.string().optional().describe("'dall-e-3' or 'dall-e-2'"),
+            model: validatedSchemas.token.optional().describe("'dall-e-3' or 'dall-e-2'"),
             quality: z.enum(["standard", "hd"]).optional(),
             style: z.enum(["vivid", "natural"]).optional(),
             size: z.enum(["1024x1024", "1792x1024", "1024x1792"]).optional(),
         }).optional().describe("DALL-E specific settings"),
         midjourney: z.object({
-            version: z.string().optional().describe("'v5', 'v6', 'niji'"),
+            version: validatedSchemas.token.optional().describe("'v5', 'v6', 'niji'"),
             stylize: z.number().optional().describe("0-1000"),
             chaos: z.number().optional().describe("0-100"),
             quality: z.number().optional().describe("0.25, 0.5, 1, 2"),
-            aspectRatio: z.string().optional().describe("E.g., '1:1', '16:9', '2:3'"),
+            aspectRatio: validatedSchemas.token.optional().describe("E.g., '1:1', '16:9', '2:3'"),
         }).optional().describe("Midjourney specific settings"),
         sdxl: z.object({
-            model: z.string().optional().describe("Model ID or path"),
-            samplerName: z.string().optional(),
+            model: validatedSchemas.token.optional().describe("Model ID or path"),
+            samplerName: validatedSchemas.token.optional(),
             steps: z.number().optional(),
             cfg: z.number().optional(),
             width: z.number().optional(),
             height: z.number().optional(),
-            negativePrompt: z.string().optional(),
+            // Free text, not a node ID -- a negative prompt is a comma-separated
+            // list that routinely runs past a token's 200.
+            negativePrompt: validatedSchemas.description.optional(),
         }).optional().describe("Stable Diffusion / SDXL settings"),
         flux: z.object({
-            model: z.string().optional().describe("'schnell', 'dev', 'pro'"),
+            model: validatedSchemas.token.optional().describe("'schnell', 'dev', 'pro'"),
             steps: z.number().optional(),
             guidance: z.number().optional(),
         }).optional().describe("Flux settings"),
         defaults: z.object({
-            aspectRatio: z.string().optional().describe("Default aspect ratio for images"),
+            aspectRatio: validatedSchemas.token.optional().describe("Default aspect ratio for images"),
             generateOnCreate: z.boolean().optional().describe("Auto-generate images when entities are created"),
             savePrompts: z.boolean().optional().describe("Store prompts with entities"),
             framing: z.object({
-                character: z.string().optional().describe("Default framing for character portraits"),
-                location: z.string().optional().describe("Default framing for location scenes"),
-                item: z.string().optional().describe("Default framing for item images"),
+                character: validatedSchemas.token.optional().describe("Default framing for character portraits"),
+                location: validatedSchemas.token.optional().describe("Default framing for location scenes"),
+                item: validatedSchemas.token.optional().describe("Default framing for item images"),
             }).optional(),
         }).optional().describe("Generation defaults"),
         consistency: z.object({
             maintainColorPalette: z.boolean().optional(),
-            characterSeedImages: z.record(z.string(), z.string()).optional().describe("characterId -> seed image"),
-            styleReferenceImage: z.string().optional().describe("Game-wide style reference"),
+            characterSeedImages: z.record(z.string(), validatedSchemas.description).optional().describe("characterId -> seed image"),
+            styleReferenceImage: validatedSchemas.description.optional().describe("Game-wide style reference"),
             useCharacterRefs: z.boolean().optional().describe("Use existing character images as reference"),
         }).optional().describe("Consistency settings"),
-        notes: z.string().optional().describe("Custom notes for the DM about image generation"),
+        notes: validatedSchemas.description.optional().describe("Custom notes for the DM about image generation"),
     });
     server.registerTool("list_image_generation_presets", {
         description: "List all image generation presets for a game. Presets allow different configurations for different use cases (character portraits, location art, items with text, etc.)",
         inputSchema: {
-            gameId: z.string().describe("The game ID"),
+            gameId: validatedSchemas.id.describe("The game ID"),
         },
         annotations: ANNOTATIONS.READ_ONLY,
     }, async ({ gameId }) => {
@@ -792,8 +809,8 @@ export function registerCoreTools(server) {
     server.registerTool("get_image_generation_preset", {
         description: "Get a specific image generation preset by ID, including full configuration details",
         inputSchema: {
-            gameId: z.string().describe("The game ID"),
-            presetId: z.string().describe("The preset ID"),
+            gameId: validatedSchemas.id.describe("The game ID"),
+            presetId: validatedSchemas.id.describe("The preset ID"),
         },
         annotations: ANNOTATIONS.READ_ONLY,
     }, async ({ gameId, presetId }) => {
@@ -811,7 +828,7 @@ export function registerCoreTools(server) {
     server.registerTool("get_default_image_preset", {
         description: "Get the default image generation preset for a game",
         inputSchema: {
-            gameId: z.string().describe("The game ID"),
+            gameId: validatedSchemas.id.describe("The game ID"),
         },
         annotations: ANNOTATIONS.READ_ONLY,
     }, async ({ gameId }) => {
@@ -834,9 +851,9 @@ export function registerCoreTools(server) {
     server.registerTool("create_image_generation_preset", {
         description: "Create a new image generation preset. Use different presets for different purposes: character portraits, location art, items with text, etc. Each preset can have its own tool, model, style, and workflow configuration.",
         inputSchema: {
-            gameId: z.string().describe("The game ID"),
-            name: z.string().describe("Human-readable preset name (e.g., 'Character Portraits', 'Location Art', 'Items with Text')"),
-            description: z.string().optional().describe("What this preset is for"),
+            gameId: validatedSchemas.id.describe("The game ID"),
+            name: validatedSchemas.token.describe("Human-readable preset name (e.g., 'Character Portraits', 'Location Art', 'Items with Text')"),
+            description: validatedSchemas.description.optional().describe("What this preset is for"),
             entityTypes: z.array(z.string().max(50)).optional()
                 .describe("Which entity types this preset is best suited for (e.g., character, location, item, scene, faction)"),
             isDefault: z.boolean().optional().describe("Make this the default preset"),
@@ -871,10 +888,10 @@ export function registerCoreTools(server) {
     server.registerTool("update_image_generation_preset", {
         description: "Update an existing image generation preset. Only specified fields are updated; others are preserved.",
         inputSchema: {
-            gameId: z.string().describe("The game ID"),
-            presetId: z.string().describe("The preset ID to update"),
-            name: z.string().optional().describe("New preset name"),
-            description: z.string().optional().describe("New description"),
+            gameId: validatedSchemas.id.describe("The game ID"),
+            presetId: validatedSchemas.id.describe("The preset ID to update"),
+            name: validatedSchemas.token.optional().describe("New preset name"),
+            description: validatedSchemas.description.optional().describe("New description"),
             entityTypes: z.array(z.string().max(50)).optional()
                 .describe("Update which entity types this preset is for (e.g., character, location, item, scene, faction)"),
             isDefault: z.boolean().optional().describe("Make this the default preset"),
@@ -902,8 +919,8 @@ export function registerCoreTools(server) {
     server.registerTool("delete_image_generation_preset", {
         description: "Delete an image generation preset. This is IRREVERSIBLE.",
         inputSchema: {
-            gameId: z.string().describe("The game ID"),
-            presetId: z.string().describe("The preset ID to delete"),
+            gameId: validatedSchemas.id.describe("The game ID"),
+            presetId: validatedSchemas.id.describe("The preset ID to delete"),
         },
         annotations: ANNOTATIONS.DELETE,
     }, async ({ gameId, presetId }) => {
@@ -921,8 +938,8 @@ export function registerCoreTools(server) {
     server.registerTool("set_default_image_preset", {
         description: "Set which image generation preset should be used by default",
         inputSchema: {
-            gameId: z.string().describe("The game ID"),
-            presetId: z.string().describe("The preset ID to make default"),
+            gameId: validatedSchemas.id.describe("The game ID"),
+            presetId: validatedSchemas.id.describe("The preset ID to make default"),
         },
         annotations: ANNOTATIONS.SET,
     }, async ({ gameId, presetId }) => {
@@ -943,50 +960,50 @@ export function registerCoreTools(server) {
     server.registerTool("set_rules", {
         description: "Store a rule system for the game",
         inputSchema: {
-            gameId: z.string().describe("The game ID"),
+            gameId: validatedSchemas.id.describe("The game ID"),
             rules: z.object({
-                name: z.string(),
-                description: z.string(),
+                name: validatedSchemas.token,
+                description: validatedSchemas.description,
                 attributes: z.array(z.object({
-                    name: z.string(),
-                    abbreviation: z.string(),
-                    description: z.string(),
+                    name: validatedSchemas.token,
+                    abbreviation: validatedSchemas.token,
+                    description: validatedSchemas.description,
                     defaultValue: z.number(),
                     minValue: z.number(),
                     maxValue: z.number(),
                 })),
                 skills: z.array(z.object({
-                    name: z.string(),
-                    governingAttribute: z.string(),
-                    description: z.string(),
+                    name: validatedSchemas.token,
+                    governingAttribute: validatedSchemas.token,
+                    description: validatedSchemas.description,
                 })),
                 derivedStats: z.array(z.object({
-                    name: z.string(),
-                    abbreviation: z.string(),
-                    description: z.string(),
-                    formula: z.string(),
+                    name: validatedSchemas.token,
+                    abbreviation: validatedSchemas.token,
+                    description: validatedSchemas.description,
+                    formula: validatedSchemas.token,
                 })),
                 combatRules: z.object({
-                    initiativeFormula: z.string(),
+                    initiativeFormula: validatedSchemas.token,
                     actionsPerTurn: z.number(),
-                    attackFormula: z.string(),
-                    defenseFormula: z.string(),
-                    damageFormula: z.string(),
+                    attackFormula: validatedSchemas.token,
+                    defenseFormula: validatedSchemas.token,
+                    damageFormula: validatedSchemas.token,
                     conditions: z.array(z.object({
-                        name: z.string(),
-                        description: z.string(),
-                        effects: z.string(),
+                        name: validatedSchemas.token,
+                        description: validatedSchemas.description,
+                        effects: validatedSchemas.description,
                     })),
                 }),
                 checkMechanics: z.object({
-                    baseDice: z.string(),
-                    modifierCalculation: z.string(),
+                    baseDice: validatedSchemas.token,
+                    modifierCalculation: validatedSchemas.token,
                     difficultyScale: z.record(z.string(), z.number()),
                     criticalSuccess: z.number().optional(),
                     criticalFailure: z.number().optional(),
                 }),
                 progression: z.object({
-                    experienceFormula: z.string(),
+                    experienceFormula: validatedSchemas.token,
                     levelUpThresholds: z.array(z.number()),
                     attributePointsPerLevel: z.number(),
                     skillPointsPerLevel: z.number(),
@@ -1004,7 +1021,7 @@ export function registerCoreTools(server) {
     server.registerTool("get_rules", {
         description: "Get the current rule system for a game",
         inputSchema: {
-            gameId: z.string().describe("The game ID"),
+            gameId: validatedSchemas.id.describe("The game ID"),
         },
         annotations: ANNOTATIONS.READ_ONLY,
     }, async ({ gameId }) => {
@@ -1021,7 +1038,7 @@ export function registerCoreTools(server) {
     server.registerTool("update_rules", {
         description: "Partially update the rule system",
         inputSchema: {
-            gameId: z.string().describe("The game ID"),
+            gameId: validatedSchemas.id.describe("The game ID"),
             updates: z.record(z.string(), z.unknown()).describe("Partial rule updates"),
         },
         annotations: ANNOTATIONS.UPDATE,

package/dist/register/display.js

@@ -1,6 +1,7 @@
 import { z } from "zod";
 import { getDisplayConfig, setDisplayConfig, resetDisplayConfig, applyThemePreset, listThemePresets, getGameDisplayConfig, setGameDisplayConfig, applyGameThemePreset, resetGameTheme, inferAndApplyTheme, } from "../tools/display.js";
 import { ANNOTATIONS } from "../utils/tool-annotations.js";
+import { validatedSchemas } from "../utils/validation.js";
 // All available theme presets
 const themePresetNames = [
     "high-fantasy",
@@ -37,19 +38,19 @@ export function registerDisplayTools(server) {
     server.registerTool("set_display_config", {
         description: "Set display configuration for the web viewer. Can set any subset of options.",
         inputSchema: {
-            bgColor: z.string().optional().describe("Background color (e.g., '#1a1a2e')"),
-            bgSecondary: z.string().optional().describe("Secondary background color"),
-            bgElevated: z.string().optional().describe("Elevated surface background"),
-            textColor: z.string().optional().describe("Main text color"),
-            textMuted: z.string().optional().describe("Muted text color"),
-            accentColor: z.string().optional().describe("Accent color for links and highlights"),
-            accentHover: z.string().optional().describe("Accent color on hover"),
-            borderColor: z.string().optional().describe("Border color"),
-            successColor: z.string().optional().describe("Success/positive color"),
-            warningColor: z.string().optional().describe("Warning/caution color"),
-            dangerColor: z.string().optional().describe("Danger/error color"),
-            codeBackground: z.string().optional().describe("Code/monospace block background"),
-            codeText: z.string().optional().describe("Code/monospace text color"),
+            bgColor: validatedSchemas.token.optional().describe("Background color (e.g., '#1a1a2e')"),
+            bgSecondary: validatedSchemas.token.optional().describe("Secondary background color"),
+            bgElevated: validatedSchemas.token.optional().describe("Elevated surface background"),
+            textColor: validatedSchemas.token.optional().describe("Main text color"),
+            textMuted: validatedSchemas.token.optional().describe("Muted text color"),
+            accentColor: validatedSchemas.token.optional().describe("Accent color for links and highlights"),
+            accentHover: validatedSchemas.token.optional().describe("Accent color on hover"),
+            borderColor: validatedSchemas.token.optional().describe("Border color"),
+            successColor: validatedSchemas.token.optional().describe("Success/positive color"),
+            warningColor: validatedSchemas.token.optional().describe("Warning/caution color"),
+            dangerColor: validatedSchemas.token.optional().describe("Danger/error color"),
+            codeBackground: validatedSchemas.token.optional().describe("Code/monospace block background"),
+            codeText: validatedSchemas.token.optional().describe("Code/monospace text color"),
             borderRadius: z
                 .enum(["sharp", "rounded", "soft"])
                 .optional()
@@ -58,13 +59,13 @@ export function registerDisplayTools(server) {
                 .enum(["clean", "grungy", "tech", "parchment", "metallic", "wooden"])
                 .optional()
                 .describe("Card visual style"),
-            fontDisplay: z.string().optional().describe("Display/heading font (Google Font name)"),
-            fontBody: z.string().optional().describe("Body text font (Google Font name)"),
-            fontMono: z.string().optional().describe("Monospace font (Google Font name)"),
+            fontDisplay: validatedSchemas.token.optional().describe("Display/heading font (Google Font name)"),
+            fontBody: validatedSchemas.token.optional().describe("Body text font (Google Font name)"),
+            fontMono: validatedSchemas.token.optional().describe("Monospace font (Google Font name)"),
             showHealthBars: z.boolean().optional().describe("Show health bars on character cards"),
             showConditionTags: z.boolean().optional().describe("Show condition tags"),
             showImages: z.boolean().optional().describe("Show images in the viewer"),
-            appTitle: z.string().optional().describe("Custom title for the web viewer"),
+            appTitle: validatedSchemas.token.optional().describe("Custom title for the web viewer"),
         },
         annotations: ANNOTATIONS.SET,
     }, async (args) => {
@@ -143,7 +144,7 @@ export function registerDisplayTools(server) {
     server.registerTool("get_game_theme", {
         description: "Get the display configuration for a specific game",
         inputSchema: {
-            gameId: z.string().describe("The game ID"),
+            gameId: validatedSchemas.id.describe("The game ID"),
         },
         annotations: ANNOTATIONS.READ_ONLY,
     }, async ({ gameId }) => {
@@ -161,29 +162,29 @@ export function registerDisplayTools(server) {
     server.registerTool("set_game_theme", {
         description: "Set display configuration for a specific game. Each game can have its own visual theme.",
         inputSchema: {
-            gameId: z.string().describe("The game ID"),
-            bgColor: z.string().optional().describe("Background color"),
-            bgSecondary: z.string().optional().describe("Secondary background"),
-            bgElevated: z.string().optional().describe("Elevated surface background"),
-            textColor: z.string().optional().describe("Text color"),
-            textMuted: z.string().optional().describe("Muted text"),
-            accentColor: z.string().optional().describe("Accent color"),
-            accentHover: z.string().optional().describe("Accent hover color"),
-            borderColor: z.string().optional().describe("Border color"),
-            successColor: z.string().optional().describe("Success color"),
-            warningColor: z.string().optional().describe("Warning color"),
-            dangerColor: z.string().optional().describe("Danger color"),
-            codeBackground: z.string().optional().describe("Code block background"),
-            codeText: z.string().optional().describe("Code text color"),
+            gameId: validatedSchemas.id.describe("The game ID"),
+            bgColor: validatedSchemas.token.optional().describe("Background color"),
+            bgSecondary: validatedSchemas.token.optional().describe("Secondary background"),
+            bgElevated: validatedSchemas.token.optional().describe("Elevated surface background"),
+            textColor: validatedSchemas.token.optional().describe("Text color"),
+            textMuted: validatedSchemas.token.optional().describe("Muted text"),
+            accentColor: validatedSchemas.token.optional().describe("Accent color"),
+            accentHover: validatedSchemas.token.optional().describe("Accent hover color"),
+            borderColor: validatedSchemas.token.optional().describe("Border color"),
+            successColor: validatedSchemas.token.optional().describe("Success color"),
+            warningColor: validatedSchemas.token.optional().describe("Warning color"),
+            dangerColor: validatedSchemas.token.optional().describe("Danger color"),
+            codeBackground: validatedSchemas.token.optional().describe("Code block background"),
+            codeText: validatedSchemas.token.optional().describe("Code text color"),
             borderRadius: z.enum(["sharp", "rounded", "soft"]).optional().describe("Border radius style"),
             cardStyle: z.enum(["clean", "grungy", "tech", "parchment", "metallic", "wooden"]).optional().describe("Card style"),
-            fontDisplay: z.string().optional().describe("Display font"),
-            fontBody: z.string().optional().describe("Body font"),
-            fontMono: z.string().optional().describe("Mono font"),
+            fontDisplay: validatedSchemas.token.optional().describe("Display font"),
+            fontBody: validatedSchemas.token.optional().describe("Body font"),
+            fontMono: validatedSchemas.token.optional().describe("Mono font"),
             showHealthBars: z.boolean().optional().describe("Show health bars"),
             showConditionTags: z.boolean().optional().describe("Show condition tags"),
             showImages: z.boolean().optional().describe("Show images"),
-            appTitle: z.string().optional().describe("App title"),
+            appTitle: validatedSchemas.token.optional().describe("App title"),
         },
         annotations: ANNOTATIONS.SET,
     }, async ({ gameId, ...config }) => {
@@ -201,7 +202,7 @@ export function registerDisplayTools(server) {
     server.registerTool("apply_game_theme_preset", {
         description: "Apply a predefined theme preset to a specific game. This allows different games to have completely different visual themes.",
         inputSchema: {
-            gameId: z.string().describe("The game ID"),
+            gameId: validatedSchemas.id.describe("The game ID"),
             preset: z.enum(themePresetNames).describe("Theme preset name"),
         },
         annotations: ANNOTATIONS.SET,
@@ -226,7 +227,7 @@ export function registerDisplayTools(server) {
     server.registerTool("reset_game_theme", {
         description: "Remove a game's custom theme, reverting to the global theme",
         inputSchema: {
-            gameId: z.string().describe("The game ID"),
+            gameId: validatedSchemas.id.describe("The game ID"),
         },
         annotations: ANNOTATIONS.SET,
     }, async ({ gameId }) => {
@@ -244,9 +245,9 @@ export function registerDisplayTools(server) {
     server.registerTool("auto_theme_game", {
         description: "Automatically apply an appropriate theme to a game based on its genre and setting. Call this when creating a new game to set up the visual style.",
         inputSchema: {
-            gameId: z.string().describe("The game ID"),
-            genre: z.string().describe("Game genre (e.g., 'fantasy', 'sci-fi', 'western', 'noir')"),
-            setting: z.string().optional().describe("Optional setting description for more accurate theme matching"),
+            gameId: validatedSchemas.id.describe("The game ID"),
+            genre: validatedSchemas.token.describe("Game genre (e.g., 'fantasy', 'sci-fi', 'western', 'noir')"),
+            setting: validatedSchemas.description.optional().describe("Optional setting description for more accurate theme matching"),
         },
         annotations: ANNOTATIONS.SET,
     }, async ({ gameId, genre, setting }) => {

package/dist/register/resolve.js

@@ -1,5 +1,6 @@
 import { z } from "zod";
 import { ANNOTATIONS } from "../utils/tool-annotations.js";
+import { validatedSchemas } from "../utils/validation.js";
 import { createLogger } from "../utils/logger.js";
 import { errors, formatErrorResponse } from "../utils/errors.js";
 import { ConstraintViolationError } from "../timeline/registry.js";
@@ -37,7 +38,9 @@ export function registerResolveTools(server, resolver) {
                 .array(z.object({
                 entityId: z.string().max(100).describe("The entity this expectation is about"),
                 key: z.string().max(200).describe("The fact key this expectation is about"),
-                value: z.union([z.string(), z.number()]).describe("The value this proposal declares it depends on"),
+                value: z
+                    .union([validatedSchemas.token, z.number()])
+                    .describe("The value this proposal declares it depends on"),
             }))
                 .optional()
                 .describe("Facts this proposal declares it depends on, verified BEFORE the mechanic is dispatched. A caller's " +