@agentic-survey/mcp-server 0.1.1 → 0.1.3

package/README.md

@@ -47,6 +47,10 @@ The agent chains the tools for you: `create_survey` → `add_question` → `publ
 
 > Tip: `npm i -g @agentic-survey/mcp-server` gives you the shorter `agentic-survey init` command.
 
+### Do I need to enable RLS / anything in the Supabase dashboard?
+
+No. The `init --print-sql` migration enables Row Level Security and the access policies for you, on all four tables. **Don't disable RLS:** it's what fences the publishable key (which travels in the share link) so it can *only* read **published** surveys and **insert** responses. It cannot read anyone's responses or your drafts. Your secret key, stored locally on your machine, bypasses RLS for authoring and analysis.
+
 ## How it works
 
 ```
@@ -59,9 +63,13 @@ Publishing a survey returns a share link carrying your project ref and publishab
 
 ## Tools
 
-`setup_connection`, `create_survey`, `add_question`, `update_question`, `remove_question`, `reorder_questions`, `publish_survey`, `get_share_link`, `list_surveys`, `get_survey`, `list_responses`, `get_results`.
+`setup_connection`, `create_survey`, `add_question`, `update_question`, `remove_question`, `reorder_questions`, `set_question_logic`, `validate_survey`, `publish_survey`, `get_share_link`, `list_surveys`, `get_survey`, `list_responses`, `get_results`.
+
+Question types: `single_choice`, `multi_choice`, `rating`, `yes_no`, `number`, `short_text`, `long_text`, `date`, `time` (24-hour), `slider` (0–100 percentage by default).
+
+### Branching / skip logic
 
-Question types: `single_choice`, `multi_choice`, `rating`, `yes_no`, `number`, `short_text`, `long_text`.
+`set_question_logic` makes a question conditional: it shows (or hides) based on the answers to **earlier** questions — e.g. only ask the follow-up if someone rated you ≤ 2. The page-service evaluates this live and submission validation respects it (a hidden required question isn't required; answers to hidden questions are rejected). Run `validate_survey` after wiring logic and before `publish_survey` — it catches forward/dangling references, conditions citing options that don't exist (so a question could never appear), and choice questions with no options.
 
 ## License
 

package/dist/cli.js

@@ -72,13 +72,30 @@ async function runInit(flags) {
         existing.secretKey ||
         '';
     const publishableKey = flags['publishable-key'] ||
-        (await ask(`Supabase publishable key (sb_publishable_…, optional${existing.publishableKey ? ', set' : ''}): `)) ||
+        (await ask(`Supabase publishable key (sb_publishable_…, needed for share links${existing.publishableKey ? ', already set' : ''}): `)) ||
         existing.publishableKey ||
         '';
-    const pageEndpoint = flags['page-endpoint'] ||
-        (await ask(`Page-service endpoint [${existing.pageEndpoint ?? DEFAULT_PAGE_ENDPOINT}]: `)) ||
-        existing.pageEndpoint ||
-        DEFAULT_PAGE_ENDPOINT;
+    // Page-service: the domain published survey links point at. Default to the
+    // hosted instance; let people opt into their own self-hosted/custom domain.
+    let pageEndpoint = flags['page-endpoint'] || '';
+    if (!pageEndpoint) {
+        const current = existing.pageEndpoint ?? DEFAULT_PAGE_ENDPOINT;
+        console.log('\nWhich domain do you want your surveys to appear on?');
+        console.log(`  1. ${DEFAULT_PAGE_ENDPOINT}/s/your-survey   (default, quick start)`);
+        console.log('  2. A custom domain (your own self-hosted page-service)');
+        const choice = (await ask(`Choose 1 or 2 [keep ${current}]: `)).trim();
+        if (choice === '1') {
+            pageEndpoint = DEFAULT_PAGE_ENDPOINT;
+        }
+        else if (choice === '2') {
+            pageEndpoint =
+                (await ask('Your page-service base URL (e.g. https://surveys.example.com): ')).trim() ||
+                    current;
+        }
+        else {
+            pageEndpoint = current; // Enter keeps the shown default
+        }
+    }
     if (!supabaseUrl || !secretKey) {
         console.error('\nA project URL and secret key are required. Aborting.');
         process.exitCode = 1;
@@ -107,6 +124,9 @@ async function runInit(flags) {
     else {
         console.log(`\n✓ Connected. Schema present (${probe.data.length} survey(s)).`);
     }
+    if (!publishableKey) {
+        console.log("\n⚠  No publishable key set. Share links and response collection won't work until you add one (re-run init).");
+    }
     printAgentSnippet();
 }
 async function main() {

package/dist/server.js

@@ -1,6 +1,6 @@
 import { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
 import { z } from 'zod';
-import { createDb, createSurvey, updateSurvey, publishSurvey, closeSurvey, listSurveys, getSurvey, addQuestion, updateQuestion, removeQuestion, reorderQuestions, listResponses, getResults, buildShareUrl, isErr, } from '@agentic-survey/core';
+import { createDb, createSurvey, updateSurvey, publishSurvey, closeSurvey, listSurveys, getSurvey, addQuestion, updateQuestion, removeQuestion, reorderQuestions, listResponses, getResults, validateSurvey, buildShareUrl, isErr, } from '@agentic-survey/core';
 import { readInitialMigrationSql } from '@agentic-survey/schema';
 import { loadConfig, isConnectable, projectRefFromUrl, configPath, DEFAULT_PAGE_ENDPOINT, } from './config.js';
 const QUESTION_TYPE = z.enum([
@@ -11,8 +11,36 @@ const QUESTION_TYPE = z.enum([
     'rating',
     'yes_no',
     'number',
+    'date',
+    'time',
+    'slider',
 ]);
 const SURVEY_STATUS = z.enum(['draft', 'published', 'closed']);
+const LOGIC_CONDITION = z.object({
+    questionId: z.string().describe('An EARLIER question (lower position) whose answer is tested.'),
+    op: z.enum([
+        'equals',
+        'not_equals',
+        'includes',
+        'gt',
+        'gte',
+        'lt',
+        'lte',
+        'answered',
+        'not_answered',
+    ]),
+    value: z
+        .union([z.string(), z.number(), z.boolean()])
+        .optional()
+        .describe('Choice ops → option id (string); rating/number → number; yes_no equals → boolean. Omit for answered/not_answered.'),
+});
+const LOGIC = z
+    .object({
+    action: z.enum(['show', 'hide']).describe('`show`: visible only when conditions match. `hide`: hidden when they match.'),
+    match: z.enum(['all', 'any']).describe('Combine conditions with AND (`all`) or OR (`any`).'),
+    conditions: z.array(LOGIC_CONDITION).min(1),
+})
+    .describe('Skip-logic rule. Conditions may reference earlier questions only. Validate with validate_survey before publishing.');
 const ok = (summary, data) => ({
     content: [{ type: 'text', text: summary }],
     structuredContent: data,
@@ -99,16 +127,20 @@ export function buildServer() {
         description: 'Append (or insert at `position`) a question. `config` shape depends on `type`: ' +
             'single_choice/multi_choice → { options: [{ id, label }] }; rating → { min, max }; ' +
             'number → { min?, max?, step?, unit? }; short_text/long_text → { placeholder?, maxLength? }; ' +
-            'yes_no → {}. Give each choice option a stable `id`.',
+            'date → { min?, max? } (ISO YYYY-MM-DD); time → { min?, max? } (24-hour HH:MM); ' +
+            'slider → { min?, max?, step?, unit? } (defaults 0–100, unit "%"); ' +
+            'yes_no → {}. Give each choice option a stable `id`. Optional `logic` makes the question ' +
+            'conditional — see set_question_logic.',
         inputSchema: {
             surveyId: z.string(),
             type: QUESTION_TYPE,
             prompt: z.string().min(1),
             required: z.boolean().optional(),
             config: z.record(z.string(), z.unknown()).optional().describe('Per-type config (see description)'),
+            logic: LOGIC.optional(),
             position: z.number().int().min(0).optional(),
         },
-    }, async ({ surveyId, type, prompt, required, config, position }) => {
+    }, async ({ surveyId, type, prompt, required, config, logic, position }) => {
         const c = getConn();
         if (!c.ok)
             return c.res;
@@ -117,6 +149,7 @@ export function buildServer() {
             prompt,
             required,
             config: config,
+            logic,
             position,
         });
         if (isErr(r))
@@ -125,13 +158,15 @@ export function buildServer() {
     });
     server.registerTool('update_question', {
         title: 'Update a question',
-        description: 'Patch a question (prompt, type, required, config, position). Only provided fields change.',
+        description: 'Patch a question (prompt, type, required, config, logic, position). Only provided fields change. ' +
+            'Pass `logic: null` to clear an existing conditional rule.',
         inputSchema: {
             questionId: z.string(),
             prompt: z.string().min(1).optional(),
             type: QUESTION_TYPE.optional(),
             required: z.boolean().optional(),
             config: z.record(z.string(), z.unknown()).optional(),
+            logic: LOGIC.nullable().optional(),
             position: z.number().int().min(0).optional(),
         },
     }, async ({ questionId, ...patch }) => {
@@ -170,6 +205,49 @@ export function buildServer() {
             return fail(r.error.code, r.error.message);
         return ok('Questions reordered.', { reordered: true });
     });
+    server.registerTool('set_question_logic', {
+        title: 'Set a question’s branching / skip logic',
+        description: 'Make a question conditional. The question appears (action "show") or is hidden (action "hide") ' +
+            'when its conditions match. Conditions test the answers to EARLIER questions: ' +
+            '{ questionId, op, value }. Ops: equals/not_equals (single_choice option id, yes_no boolean), ' +
+            'includes (multi_choice contains an option id), gt/gte/lt/lte (rating/number), answered/not_answered. ' +
+            'Combine with match "all" (AND) or "any" (OR). Pass logic null to clear. ' +
+            'Always run validate_survey after wiring logic and before publishing.',
+        inputSchema: {
+            questionId: z.string(),
+            logic: LOGIC.nullable().describe('The rule to set, or null to clear.'),
+        },
+    }, async ({ questionId, logic }) => {
+        const c = getConn();
+        if (!c.ok)
+            return c.res;
+        const r = await updateQuestion(c.db, questionId, { logic });
+        if (isErr(r))
+            return fail(r.error.code, r.error.message);
+        return ok(logic ? 'Question logic set.' : 'Question logic cleared.', { question: r.data });
+    });
+    server.registerTool('validate_survey', {
+        title: 'Validate a survey’s structure and branching logic',
+        description: 'Static pre-publish check. Reports errors (broken/forward/cyclic logic references, conditions citing ' +
+            'option ids that don’t exist so a question can never show, choice questions with no options) and ' +
+            'warnings (comparison op on a non-numeric question, logic with no conditions). Returns { ok, issues }; ' +
+            'ok is false when there is at least one error. Call this after building/editing logic, before publish_survey.',
+        inputSchema: { surveyId: z.string() },
+        annotations: { readOnlyHint: true },
+    }, async ({ surveyId }) => {
+        const c = getConn();
+        if (!c.ok)
+            return c.res;
+        const r = await getSurvey(c.db, surveyId);
+        if (isErr(r))
+            return fail(r.error.code, r.error.message);
+        const issues = validateSurvey(r.data.questions);
+        const errorCount = issues.filter((i) => i.level === 'error').length;
+        const summary = issues.length
+            ? `${errorCount} error(s), ${issues.length - errorCount} warning(s).`
+            : 'No issues — ready to publish.';
+        return ok(summary, { ok: errorCount === 0, issues });
+    });
     server.registerTool('publish_survey', {
         title: 'Publish a survey',
         description: 'Publish a draft survey and return its public share link (if a publishable key is configured). ' +

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agentic-survey/mcp-server",
-  "version": "0.1.1",
+  "version": "0.1.3",
   "description": "MCP server (stdio) exposing survey tools over core, plus the agentic-survey CLI.",
   "license": "MIT",
   "author": "Monty Daley <monty@daley.org.nz>",