Merge pull request #10 from syucream/syucream/schemas

fix required params and schemas

Author: syucream

package.json

@@ -1,6 +1,6 @@
 {
   "name": "lightdash-mcp-server",
-  "version": "0.0.8",
+  "version": "0.0.9",
   "description": "A MCP(Model Context Protocol) server that accesses to lightdash( https://www.lightdash.com/ )",
   "main": "dist/index.js",
   "type": "module",

src/index.ts

@@ -119,8 +119,8 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
 
 server.setRequestHandler(CallToolRequestSchema, async (request) => {
   try {
-    if (!request.params.arguments) {
-      throw new Error('Arguments are required');
+    if (!request.params) {
+      throw new Error('Params are required');
     }
 
     switch (request.params.name) {

src/schemas.ts

@@ -3,48 +3,103 @@ import { z } from 'zod';
 export const ListProjectsSchema = z.object({});
 
 export const GetProjectSchema = z.object({
-  projectUuid: z.string(),
+  projectUuid: z
+    .string()
+    .uuid()
+    .describe(
+      'The UUID of the project. You can obtain it from the project list.'
+    ),
 });
 
 export const ListSpacesSchema = z.object({
-  projectUuid: z.string(),
+  projectUuid: z
+    .string()
+    .uuid()
+    .describe(
+      'The UUID of the project. You can obtain it from the project list.'
+    ),
 });
 
 export const ListChartsSchema = z.object({
-  projectUuid: z.string(),
+  projectUuid: z
+    .string()
+    .uuid()
+    .describe(
+      'The UUID of the project. You can obtain it from the project list.'
+    ),
 });
 
 export const ListDashboardsSchema = z.object({
-  projectUuid: z.string(),
+  projectUuid: z
+    .string()
+    .uuid()
+    .describe(
+      'The UUID of the project. You can obtain it from the project list.'
+    ),
 });
 
 export const GetCustomMetricsSchema = z.object({
-  projectUuid: z.string(),
+  projectUuid: z
+    .string()
+    .uuid()
+    .describe(
+      'The UUID of the project. You can obtain it from the project list.'
+    ),
 });
 
 export const GetCatalogSchema = z.object({
-  projectUuid: z.string(),
+  projectUuid: z
+    .string()
+    .uuid()
+    .describe(
+      'The UUID of the project. You can obtain it from the project list.'
+    ),
 });
 
 export const GetMetricsCatalogSchema = z.object({
-  projectUuid: z.string(),
+  projectUuid: z
+    .string()
+    .uuid()
+    .describe(
+      'The UUID of the project. You can obtain it from the project list.'
+    ),
 });
 
 export const GetChartsAsCodeSchema = z.object({
-  projectUuid: z.string(),
+  projectUuid: z
+    .string()
+    .uuid()
+    .describe(
+      'The UUID of the project. You can obtain it from the project list.'
+    ),
 });
 
 export const GetDashboardsAsCodeSchema = z.object({
-  projectUuid: z.string(),
+  projectUuid: z
+    .string()
+    .uuid()
+    .describe(
+      'The UUID of the project. You can obtain it from the project list.'
+    ),
 });
 
 export const GetMetadataSchema = z.object({
-  projectUuid: z.string(),
-  table: z.string(),
+  projectUuid: z
+    .string()
+    .uuid()
+    .describe(
+      'The UUID of the project. You can obtain it from the project list.'
+    ),
+  table: z.string().min(1, 'Table name cannot be empty'),
 });
 
 export const GetAnalyticsSchema = z.object({
-  projectUuid: z.string(),
+  projectUuid: z
+    .string()
+    .uuid()
+    .describe(
+      'The UUID of the project. You can obtain it from the project list.'
+    ),
   table: z.string(),
 });