@agentic-survey/core 0.1.0 → 0.1.1

package/dist/index.d.ts

@@ -5,3 +5,4 @@ export { createSurvey, updateSurvey, publishSurvey, closeSurvey, listSurveys, ge
 export { addQuestion, updateQuestion, removeQuestion, reorderQuestions, } from './questions.js';
 export { listResponses, type ResponseWithAnswers } from './responses.js';
 export { getResults } from './results.js';
+export { validateSurvey, type SurveyLintIssue } from './lint.js';

package/dist/index.js

@@ -5,3 +5,4 @@ export { createSurvey, updateSurvey, publishSurvey, closeSurvey, listSurveys, ge
 export { addQuestion, updateQuestion, removeQuestion, reorderQuestions, } from './questions.js';
 export { listResponses } from './responses.js';
 export { getResults } from './results.js';
+export { validateSurvey } from './lint.js';

package/dist/lint.d.ts

@@ -0,0 +1,15 @@
+import type { Question } from '@agentic-survey/schema';
+/**
+ * Static checks an agent should run before publishing — especially once skip
+ * logic is involved. Pure and DB-free: operates on the survey's questions.
+ *
+ * Errors block a sensible publish (broken/cyclic logic); warnings flag likely
+ * mistakes (e.g. a comparison op on a non-numeric question).
+ */
+export interface SurveyLintIssue {
+    level: 'error' | 'warning';
+    code: string;
+    questionId?: string;
+    message: string;
+}
+export declare function validateSurvey(questions: Question[]): SurveyLintIssue[];

package/dist/lint.js

@@ -0,0 +1,69 @@
+const CHOICE_TYPES = new Set(['single_choice', 'multi_choice']);
+const NUMERIC_TYPES = new Set(['rating', 'number', 'slider']);
+const NUMERIC_OPS = new Set(['gt', 'gte', 'lt', 'lte']);
+const CHOICE_OPS = new Set(['equals', 'not_equals', 'includes']);
+function optionIds(q) {
+    const opts = q.config.options ?? [];
+    return opts.map((o) => o.id);
+}
+export function validateSurvey(questions) {
+    const issues = [];
+    const byId = new Map(questions.map((q) => [q.id, q]));
+    // Structural: choice questions need options; option ids must be unique.
+    for (const q of questions) {
+        if (CHOICE_TYPES.has(q.type)) {
+            const ids = optionIds(q);
+            if (ids.length === 0)
+                issues.push({ level: 'error', code: 'no_options', questionId: q.id, message: `${q.type} question "${q.prompt}" has no options.` });
+            const dupes = ids.filter((id, i) => ids.indexOf(id) !== i);
+            if (dupes.length)
+                issues.push({ level: 'error', code: 'duplicate_option_id', questionId: q.id, message: `Duplicate option id(s): ${[...new Set(dupes)].join(', ')}.` });
+        }
+    }
+    // Logic checks.
+    for (const q of questions) {
+        const logic = q.logic;
+        if (!logic)
+            continue;
+        if (!logic.conditions || logic.conditions.length === 0) {
+            issues.push({ level: 'warning', code: 'empty_logic', questionId: q.id, message: `Question "${q.prompt}" has a logic rule with no conditions (it has no effect).` });
+            continue;
+        }
+        for (const c of logic.conditions) {
+            const target = byId.get(c.questionId);
+            if (!target) {
+                issues.push({ level: 'error', code: 'dangling_reference', questionId: q.id, message: `Condition references unknown question id "${c.questionId}".` });
+                continue;
+            }
+            if (target.id === q.id) {
+                issues.push({ level: 'error', code: 'self_reference', questionId: q.id, message: `Question "${q.prompt}" references itself in its logic.` });
+                continue;
+            }
+            if (target.position >= q.position) {
+                issues.push({ level: 'error', code: 'forward_reference', questionId: q.id, message: `Condition references a later question ("${target.prompt}"). Conditions may only reference earlier questions.` });
+            }
+            checkOpCompatibility(q, target, c, issues);
+        }
+    }
+    return issues;
+}
+function checkOpCompatibility(q, target, c, issues) {
+    if (c.op === 'answered' || c.op === 'not_answered')
+        return;
+    if (NUMERIC_OPS.has(c.op)) {
+        if (!NUMERIC_TYPES.has(target.type))
+            issues.push({ level: 'warning', code: 'op_type_mismatch', questionId: q.id, message: `Op "${c.op}" targets a ${target.type} question; expected rating/number.` });
+        if (typeof c.value !== 'number')
+            issues.push({ level: 'error', code: 'bad_operand', questionId: q.id, message: `Op "${c.op}" needs a numeric value.` });
+        return;
+    }
+    // Choice equality against a choice question: operand must be a real option id.
+    if (CHOICE_OPS.has(c.op) && CHOICE_TYPES.has(target.type)) {
+        const ids = optionIds(target);
+        if (typeof c.value !== 'string' || !ids.includes(c.value))
+            issues.push({ level: 'error', code: 'unknown_option', questionId: q.id, message: `Condition value "${String(c.value)}" is not an option of "${target.prompt}" — this question can never become visible as written.` });
+        return;
+    }
+    if (target.type === 'yes_no' && (c.op === 'equals' || c.op === 'not_equals') && typeof c.value !== 'boolean')
+        issues.push({ level: 'warning', code: 'bad_operand', questionId: q.id, message: `yes_no comparison expects a boolean value.` });
+}

package/dist/questions.d.ts

@@ -1,4 +1,4 @@
-import type { Question, QuestionType, QuestionConfig } from '@agentic-survey/schema';
+import type { Question, QuestionType, QuestionConfig, QuestionLogic } from '@agentic-survey/schema';
 import type { Db } from './client.js';
 import { type Result } from './result.js';
 export declare function addQuestion(db: Db, surveyId: string, input: {
@@ -6,6 +6,7 @@ export declare function addQuestion(db: Db, surveyId: string, input: {
     prompt: string;
     required?: boolean;
     config?: QuestionConfig;
+    logic?: QuestionLogic | null;
     position?: number;
 }): Promise<Result<Question>>;
 export declare function updateQuestion(db: Db, questionId: string, patch: {
@@ -13,6 +14,7 @@ export declare function updateQuestion(db: Db, questionId: string, patch: {
     prompt?: string;
     required?: boolean;
     config?: QuestionConfig;
+    logic?: QuestionLogic | null;
     position?: number;
 }): Promise<Result<Question>>;
 export declare function removeQuestion(db: Db, questionId: string): Promise<Result<void>>;

package/dist/questions.js

@@ -23,6 +23,7 @@ export async function addQuestion(db, surveyId, input) {
             prompt: input.prompt,
             required: input.required ?? false,
             config: input.config ?? {},
+            logic: input.logic ?? null,
             position,
         })
             .select()
@@ -46,6 +47,8 @@ export async function updateQuestion(db, questionId, patch) {
             row.required = patch.required;
         if (patch.config !== undefined)
             row.config = patch.config;
+        if (patch.logic !== undefined)
+            row.logic = patch.logic;
         if (patch.position !== undefined)
             row.position = patch.position;
         const { data, error } = await db

package/dist/results.js

@@ -60,7 +60,7 @@ function yesNoAggregate(values) {
 function numericAggregate(values) {
     const nums = [];
     for (const v of values)
-        if (v.kind === 'rating' || v.kind === 'number')
+        if (v.kind === 'rating' || v.kind === 'number' || v.kind === 'slider')
             nums.push(v.value);
     const totalAnswered = nums.length;
     if (!totalAnswered) {
@@ -88,9 +88,12 @@ function numericAggregate(values) {
 }
 function textAggregate(values) {
     const responses = [];
-    for (const v of values)
+    for (const v of values) {
         if (v.kind === 'short_text' || v.kind === 'long_text')
             responses.push(v.text);
+        else if (v.kind === 'date' || v.kind === 'time')
+            responses.push(v.value);
+    }
     return { kind: 'text', totalAnswered: responses.length, responses };
 }
 function aggregateQuestion(q, values) {
@@ -102,9 +105,12 @@ function aggregateQuestion(q, values) {
             return yesNoAggregate(values);
         case 'rating':
         case 'number':
+        case 'slider':
             return numericAggregate(values);
         case 'short_text':
         case 'long_text':
+        case 'date':
+        case 'time':
             return textAggregate(values);
     }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agentic-survey/core",
-  "version": "0.1.0",
+  "version": "0.1.1",
   "description": "Pure TypeScript service layer for survey logic. No MCP, no HTTP, no framework types.",
   "license": "MIT",
   "author": "Monty Daley <monty@daley.org.nz>",