@centrali-io/centrali-mcp 4.2.7 → 4.2.9

package/dist/tools/compute.js

@@ -155,11 +155,22 @@ function registerComputeTools(server, sdk) {
     }));
     server.tool("create_function", "Create a new compute function. Compute functions are JavaScript code blocks that run server-side.", {
         name: zod_1.z.string().describe("Display name for the function"),
-        code: zod_1.z.string().describe("JavaScript source code. Must export an async function: module.exports = async (ctx) => { ... }"),
+        code: zod_1.z.string().describe("JavaScript source code. Must define 'async function run() { ... }'. Globals: api, triggerParams, executionParams. Do NOT use module.exports."),
         description: zod_1.z.string().optional().describe("Optional description of what the function does"),
-        timeoutMs: zod_1.z.number().optional().describe("Execution timeout in milliseconds (default: 30000, min: 1000, max: 300000)"),
+        timeoutMs: zod_1.z.number().optional().describe("Execution timeout in milliseconds (default: 30000, max: 300000 / 5 minutes)"),
     }, (_a) => __awaiter(this, [_a], void 0, function* ({ name, code, description, timeoutMs }) {
         try {
+            if (/module\.exports\s*=/.test(code)) {
+                return {
+                    content: [
+                        {
+                            type: "text",
+                            text: "Error: Do not use module.exports. Functions must define 'async function run() { ... }'. Globals available: api, triggerParams, executionParams.",
+                        },
+                    ],
+                    isError: true,
+                };
+            }
             const input = { name, code };
             if (description !== undefined)
                 input.description = description;
@@ -188,10 +199,21 @@ function registerComputeTools(server, sdk) {
         functionId: zod_1.z.string().describe("The compute function ID (UUID) to update"),
         name: zod_1.z.string().optional().describe("Updated display name"),
         description: zod_1.z.string().optional().describe("Updated description"),
-        code: zod_1.z.string().optional().describe("Updated JavaScript source code"),
-        timeoutMs: zod_1.z.number().optional().describe("Updated execution timeout in milliseconds (min: 1000, max: 300000)"),
+        code: zod_1.z.string().optional().describe("Updated JavaScript source code. Must define 'async function run() { ... }'. Do NOT use module.exports."),
+        timeoutMs: zod_1.z.number().optional().describe("Updated execution timeout in milliseconds (max: 300000 / 5 minutes)"),
     }, (_a) => __awaiter(this, [_a], void 0, function* ({ functionId, name, description, code, timeoutMs }) {
         try {
+            if (code && /module\.exports\s*=/.test(code)) {
+                return {
+                    content: [
+                        {
+                            type: "text",
+                            text: "Error: Do not use module.exports. Functions must define 'async function run() { ... }'. Globals available: api, triggerParams, executionParams.",
+                        },
+                    ],
+                    isError: true,
+                };
+            }
             const input = {};
             if (name !== undefined)
                 input.name = name;
@@ -247,14 +269,25 @@ function registerComputeTools(server, sdk) {
         }
     }));
     server.tool("test_function", "Test execute code without saving it as a function. Useful for validating function code before creating or updating.", {
-        code: zod_1.z.string().describe("JavaScript code to test. Must export an async function: module.exports = async (ctx) => { ... }"),
+        code: zod_1.z.string().describe("JavaScript code to test. Must define 'async function run() { ... }'. Globals: api, triggerParams, executionParams. Do NOT use module.exports."),
         params: zod_1.z
             .record(zod_1.z.string(), zod_1.z.any())
             .optional()
             .describe("Optional parameters passed to the function as executionParams"),
-        timeoutMs: zod_1.z.number().optional().describe("Execution timeout in milliseconds (default: 30000, min: 1000, max: 300000)"),
+        timeoutMs: zod_1.z.number().optional().describe("Execution timeout in milliseconds (default: 30000, max: 300000 / 5 minutes)"),
     }, (_a) => __awaiter(this, [_a], void 0, function* ({ code, params, timeoutMs }) {
         try {
+            if (/module\.exports\s*=/.test(code)) {
+                return {
+                    content: [
+                        {
+                            type: "text",
+                            text: "Error: Do not use module.exports. Functions must define 'async function run() { ... }'. Globals available: api, triggerParams, executionParams.",
+                        },
+                    ],
+                    isError: true,
+                };
+            }
             const testInput = { code };
             if (params !== undefined)
                 testInput.params = params;

package/dist/tools/describe.js

@@ -432,7 +432,7 @@ function registerDescribeTools(server) {
                                 description: "Create a new compute function",
                                 required_params: ["name", "code"],
                                 optional_params: ["description", "timeoutMs"],
-                                code_format: "module.exports = async (ctx) => { /* your code */ return result; }",
+                                code_format: "async function run() { /* api, triggerParams, executionParams are globals */ return result; }",
                             },
                             update_function: {
                                 description: "Update an existing function. Partial updates supported.",
@@ -893,12 +893,27 @@ function registerDescribeTools(server) {
                                 inherit: "boolean — if true, inherits the workspace theme. Usually set to true.",
                             },
                         },
+                        draft_vs_publish: {
+                            description: "Drafts are lenient — most fields are optional and will default. Publishing validates strictly. Always provide the fields below to avoid 400 errors.",
+                            always_required_even_in_drafts: [
+                                "sections[*].id — MUST be a valid UUID",
+                                "sections[*].blocks — array (can be empty)",
+                                "blocks[*].id — MUST be a valid UUID",
+                            ],
+                            required_for_publish: [
+                                "sections[*].kind — enum (see section_kinds)",
+                                "sections[*].title — non-empty string",
+                                "sections[*].layout — enum (see section_layouts)",
+                                "blocks[*].blockType — non-empty string",
+                                "theme.inherit — boolean",
+                            ],
+                        },
                         section_shape: {
-                            id: "string — unique ID for the section (use a UUID or descriptive slug like 'metrics-strip')",
-                            kind: "'content' | 'metrics' | 'exceptions' | 'actions' | 'hero' | 'supporting'",
-                            title: "string — section heading displayed to users",
-                            layout: "'single-column' | 'two-column' | 'metric-strip' | 'stack'",
-                            blocks: "PageBlock[] — array of blocks in this section",
+                            id: "string — MUST be a valid UUID (e.g., crypto.randomUUID()). Non-UUID strings will cause 400 errors.",
+                            kind: "'content' | 'metrics' | 'exceptions' | 'actions' | 'hero' | 'supporting' — optional in drafts, required for publish",
+                            title: "string — section heading displayed to users. Optional in drafts, required (non-empty) for publish.",
+                            layout: "'single-column' | 'two-column' | 'metric-strip' | 'stack' — optional in drafts, required for publish",
+                            blocks: "PageBlock[] — array of blocks in this section (required, can be empty)",
                         },
                         section_kinds: {
                             hero: "Top of page — typically a title/summary or key metric. Use for detail pages.",
@@ -915,8 +930,8 @@ function registerDescribeTools(server) {
                             stack: "Compact vertical stack",
                         },
                         block_shape: {
-                            id: "string — unique ID for the block (use a UUID)",
-                            blockType: "string — the type of UI component (see describe_page_blocks for all types)",
+                            id: "string — MUST be a valid UUID. Non-UUID strings will cause 400 errors.",
+                            blockType: "string — the type of UI component (see describe_page_blocks for all types). Optional in drafts, required (non-empty) for publish.",
                             dataSource: "DataSource | null — where this block gets its data (see below)",
                             presentation: "BlockPresentation | null — display options (density, columns, chart type, emphasis rules)",
                             actions: "BlockAction[] | null — interactive actions (buttons, row clicks, form submissions). See describe_page_actions.",
@@ -954,16 +969,17 @@ function registerDescribeTools(server) {
                             comparison: "{ type: 'previous-period' | 'target' | 'baseline', value: number } — for metrics comparison",
                             nextActionLabel: "string — label for a suggested next action (e.g., 'View all orders')",
                         },
+                        IMPORTANT_all_ids_must_be_uuids: "Section IDs and block IDs MUST be valid UUIDs (e.g., 'a1b2c3d4-e5f6-7890-abcd-ef1234567890'). Using slugs like 'main-content' will cause a 400 error.",
                         example_list_page_definition: {
                             sections: [
                                 {
-                                    id: "main-content",
+                                    id: "f47ac10b-58cc-4372-a567-0e02b2c3d479",
                                     kind: "content",
                                     title: "All Customers",
                                     layout: "single-column",
                                     blocks: [
                                         {
-                                            id: "customer-table",
+                                            id: "d290f1ee-6c54-4b01-90e6-d701748f0851",
                                             blockType: "data-table",
                                             dataSource: {
                                                 type: "structure",
@@ -991,13 +1007,13 @@ function registerDescribeTools(server) {
                         example_dashboard_definition: {
                             sections: [
                                 {
-                                    id: "kpi-strip",
+                                    id: "c73bcdcc-2669-4bf6-81d3-e4ae73fb11fd",
                                     kind: "metrics",
                                     title: "Key Metrics",
                                     layout: "metric-strip",
                                     blocks: [
                                         {
-                                            id: "total-orders",
+                                            id: "9b1deb4d-3b7d-4bad-9bdd-2b0d7b3dcb6d",
                                             blockType: "metric-card",
                                             dataSource: {
                                                 type: "structure",
@@ -1013,7 +1029,7 @@ function registerDescribeTools(server) {
                                             },
                                         },
                                         {
-                                            id: "total-revenue",
+                                            id: "1b9d6bcd-bbfd-4b2d-9b5d-ab8dfbbd4bed",
                                             blockType: "metric-card",
                                             dataSource: {
                                                 type: "structure",
@@ -1033,13 +1049,13 @@ function registerDescribeTools(server) {
                                     ],
                                 },
                                 {
-                                    id: "charts",
+                                    id: "6ec0bd7f-11c0-43da-975e-2a8ad9ebae0b",
                                     kind: "content",
                                     title: "Trends",
                                     layout: "two-column",
                                     blocks: [
                                         {
-                                            id: "orders-by-status",
+                                            id: "3fa85f64-5717-4562-b3fc-2c963f66afa6",
                                             blockType: "chart",
                                             dataSource: {
                                                 type: "structure",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@centrali-io/centrali-mcp",
-  "version": "4.2.7",
+  "version": "4.2.9",
   "description": "Centrali MCP Server - AI assistant integration for Centrali workspaces",
   "main": "dist/index.js",
   "type": "commonjs",

package/src/tools/compute.ts

@@ -174,12 +174,24 @@ export function registerComputeTools(server: McpServer, sdk: CentraliSDK) {
     "Create a new compute function. Compute functions are JavaScript code blocks that run server-side.",
     {
       name: z.string().describe("Display name for the function"),
-      code: z.string().describe("JavaScript source code. Must export an async function: module.exports = async (ctx) => { ... }"),
+      code: z.string().describe("JavaScript source code. Must define 'async function run() { ... }'. Globals: api, triggerParams, executionParams. Do NOT use module.exports."),
       description: z.string().optional().describe("Optional description of what the function does"),
-      timeoutMs: z.number().optional().describe("Execution timeout in milliseconds (default: 30000, min: 1000, max: 300000)"),
+      timeoutMs: z.number().optional().describe("Execution timeout in milliseconds (default: 30000, max: 300000 / 5 minutes)"),
     },
     async ({ name, code, description, timeoutMs }) => {
       try {
+        if (/module\.exports\s*=/.test(code)) {
+          return {
+            content: [
+              {
+                type: "text",
+                text: "Error: Do not use module.exports. Functions must define 'async function run() { ... }'. Globals available: api, triggerParams, executionParams.",
+              },
+            ],
+            isError: true,
+          };
+        }
+
         const input: Record<string, any> = { name, code };
         if (description !== undefined) input.description = description;
         if (timeoutMs !== undefined) input.timeoutMs = timeoutMs;
@@ -211,11 +223,23 @@ export function registerComputeTools(server: McpServer, sdk: CentraliSDK) {
       functionId: z.string().describe("The compute function ID (UUID) to update"),
       name: z.string().optional().describe("Updated display name"),
       description: z.string().optional().describe("Updated description"),
-      code: z.string().optional().describe("Updated JavaScript source code"),
-      timeoutMs: z.number().optional().describe("Updated execution timeout in milliseconds (min: 1000, max: 300000)"),
+      code: z.string().optional().describe("Updated JavaScript source code. Must define 'async function run() { ... }'. Do NOT use module.exports."),
+      timeoutMs: z.number().optional().describe("Updated execution timeout in milliseconds (max: 300000 / 5 minutes)"),
     },
     async ({ functionId, name, description, code, timeoutMs }) => {
       try {
+        if (code && /module\.exports\s*=/.test(code)) {
+          return {
+            content: [
+              {
+                type: "text",
+                text: "Error: Do not use module.exports. Functions must define 'async function run() { ... }'. Globals available: api, triggerParams, executionParams.",
+              },
+            ],
+            isError: true,
+          };
+        }
+
         const input: Record<string, any> = {};
         if (name !== undefined) input.name = name;
         if (description !== undefined) input.description = description;
@@ -277,15 +301,27 @@ export function registerComputeTools(server: McpServer, sdk: CentraliSDK) {
     "test_function",
     "Test execute code without saving it as a function. Useful for validating function code before creating or updating.",
     {
-      code: z.string().describe("JavaScript code to test. Must export an async function: module.exports = async (ctx) => { ... }"),
+      code: z.string().describe("JavaScript code to test. Must define 'async function run() { ... }'. Globals: api, triggerParams, executionParams. Do NOT use module.exports."),
       params: z
         .record(z.string(), z.any())
         .optional()
         .describe("Optional parameters passed to the function as executionParams"),
-      timeoutMs: z.number().optional().describe("Execution timeout in milliseconds (default: 30000, min: 1000, max: 300000)"),
+      timeoutMs: z.number().optional().describe("Execution timeout in milliseconds (default: 30000, max: 300000 / 5 minutes)"),
     },
     async ({ code, params, timeoutMs }) => {
       try {
+        if (/module\.exports\s*=/.test(code)) {
+          return {
+            content: [
+              {
+                type: "text",
+                text: "Error: Do not use module.exports. Functions must define 'async function run() { ... }'. Globals available: api, triggerParams, executionParams.",
+              },
+            ],
+            isError: true,
+          };
+        }
+
         const testInput: Record<string, any> = { code };
         if (params !== undefined) testInput.params = params;
         if (timeoutMs !== undefined) testInput.timeoutMs = timeoutMs;

package/src/tools/describe.ts

@@ -502,7 +502,7 @@ export function registerDescribeTools(server: McpServer) {
                   description: "Create a new compute function",
                   required_params: ["name", "code"],
                   optional_params: ["description", "timeoutMs"],
-                  code_format: "module.exports = async (ctx) => { /* your code */ return result; }",
+                  code_format: "async function run() { /* api, triggerParams, executionParams are globals */ return result; }",
                 },
                 update_function: {
                   description: "Update an existing function. Partial updates supported.",
@@ -1055,13 +1055,28 @@ export function registerDescribeTools(server: McpServer) {
                     "boolean — if true, inherits the workspace theme. Usually set to true.",
                 },
               },
+              draft_vs_publish: {
+                description: "Drafts are lenient — most fields are optional and will default. Publishing validates strictly. Always provide the fields below to avoid 400 errors.",
+                always_required_even_in_drafts: [
+                  "sections[*].id — MUST be a valid UUID",
+                  "sections[*].blocks — array (can be empty)",
+                  "blocks[*].id — MUST be a valid UUID",
+                ],
+                required_for_publish: [
+                  "sections[*].kind — enum (see section_kinds)",
+                  "sections[*].title — non-empty string",
+                  "sections[*].layout — enum (see section_layouts)",
+                  "blocks[*].blockType — non-empty string",
+                  "theme.inherit — boolean",
+                ],
+              },
               section_shape: {
-                id: "string — unique ID for the section (use a UUID or descriptive slug like 'metrics-strip')",
-                kind: "'content' | 'metrics' | 'exceptions' | 'actions' | 'hero' | 'supporting'",
-                title: "string — section heading displayed to users",
+                id: "string — MUST be a valid UUID (e.g., crypto.randomUUID()). Non-UUID strings will cause 400 errors.",
+                kind: "'content' | 'metrics' | 'exceptions' | 'actions' | 'hero' | 'supporting' — optional in drafts, required for publish",
+                title: "string — section heading displayed to users. Optional in drafts, required (non-empty) for publish.",
                 layout:
-                  "'single-column' | 'two-column' | 'metric-strip' | 'stack'",
-                blocks: "PageBlock[] — array of blocks in this section",
+                  "'single-column' | 'two-column' | 'metric-strip' | 'stack' — optional in drafts, required for publish",
+                blocks: "PageBlock[] — array of blocks in this section (required, can be empty)",
               },
               section_kinds: {
                 hero: "Top of page — typically a title/summary or key metric. Use for detail pages.",
@@ -1084,9 +1099,9 @@ export function registerDescribeTools(server: McpServer) {
                 stack: "Compact vertical stack",
               },
               block_shape: {
-                id: "string — unique ID for the block (use a UUID)",
+                id: "string — MUST be a valid UUID. Non-UUID strings will cause 400 errors.",
                 blockType:
-                  "string — the type of UI component (see describe_page_blocks for all types)",
+                  "string — the type of UI component (see describe_page_blocks for all types). Optional in drafts, required (non-empty) for publish.",
                 dataSource:
                   "DataSource | null — where this block gets its data (see below)",
                 presentation:
@@ -1140,16 +1155,17 @@ export function registerDescribeTools(server: McpServer) {
                 nextActionLabel:
                   "string — label for a suggested next action (e.g., 'View all orders')",
               },
+              IMPORTANT_all_ids_must_be_uuids: "Section IDs and block IDs MUST be valid UUIDs (e.g., 'a1b2c3d4-e5f6-7890-abcd-ef1234567890'). Using slugs like 'main-content' will cause a 400 error.",
               example_list_page_definition: {
                 sections: [
                   {
-                    id: "main-content",
+                    id: "f47ac10b-58cc-4372-a567-0e02b2c3d479",
                     kind: "content",
                     title: "All Customers",
                     layout: "single-column",
                     blocks: [
                       {
-                        id: "customer-table",
+                        id: "d290f1ee-6c54-4b01-90e6-d701748f0851",
                         blockType: "data-table",
                         dataSource: {
                           type: "structure",
@@ -1177,13 +1193,13 @@ export function registerDescribeTools(server: McpServer) {
               example_dashboard_definition: {
                 sections: [
                   {
-                    id: "kpi-strip",
+                    id: "c73bcdcc-2669-4bf6-81d3-e4ae73fb11fd",
                     kind: "metrics",
                     title: "Key Metrics",
                     layout: "metric-strip",
                     blocks: [
                       {
-                        id: "total-orders",
+                        id: "9b1deb4d-3b7d-4bad-9bdd-2b0d7b3dcb6d",
                         blockType: "metric-card",
                         dataSource: {
                           type: "structure",
@@ -1199,7 +1215,7 @@ export function registerDescribeTools(server: McpServer) {
                         },
                       },
                       {
-                        id: "total-revenue",
+                        id: "1b9d6bcd-bbfd-4b2d-9b5d-ab8dfbbd4bed",
                         blockType: "metric-card",
                         dataSource: {
                           type: "structure",
@@ -1219,13 +1235,13 @@ export function registerDescribeTools(server: McpServer) {
                     ],
                   },
                   {
-                    id: "charts",
+                    id: "6ec0bd7f-11c0-43da-975e-2a8ad9ebae0b",
                     kind: "content",
                     title: "Trends",
                     layout: "two-column",
                     blocks: [
                       {
-                        id: "orders-by-status",
+                        id: "3fa85f64-5717-4562-b3fc-2c963f66afa6",
                         blockType: "chart",
                         dataSource: {
                           type: "structure",