qingflow-mcp 0.2.3 → 0.2.5

package/README.md

@@ -105,6 +105,12 @@ MCP client config example:
 
 ## List Query Tips
 
+Strict mode (`qf_records_list`):
+
+1. `select_columns` is required.
+2. `include_answers=false` is not allowed.
+3. Output `items[].answers` contains only selected columns, not full answers.
+
 1. For `qf_records_list.sort[].que_id`, use a real field `que_id` (numeric) or exact field title from `qf_form_get`.
 2. Avoid aliases like `create_time`; Qingflow often rejects them.
 3. Use `max_rows` (or `max_items`) to cap returned rows.
@@ -126,6 +132,16 @@ Example:
 }
 ```
 
+For single record details (`qf_record_get`), the same column controls are supported:
+
+```json
+{
+  "apply_id": "497600278750478338",
+  "max_columns": 5,
+  "select_columns": [1, "客户名称"]
+}
+```
+
 Optional env vars:
 
 ```bash

package/dist/server.js

@@ -147,7 +147,8 @@ const formOutputSchema = z.object({
     }),
     meta: apiMetaSchema
 });
-const listInputSchema = z.object({
+const listInputSchema = z
+    .object({
     app_key: z.string().min(1),
     user_id: z.string().min(1).optional(),
     page_num: z.number().int().positive().optional(),
@@ -193,12 +194,12 @@ const listInputSchema = z.object({
     max_rows: z.number().int().positive().max(200).optional(),
     max_items: z.number().int().positive().max(200).optional(),
     max_columns: z.number().int().positive().max(200).optional(),
-    select_columns: z
-        .array(z.union([z.string().min(1), z.number().int()]))
-        .min(1)
-        .max(200)
-        .optional(),
+    // Strict mode: callers must explicitly choose columns.
+    select_columns: z.array(z.union([z.string().min(1), z.number().int()])).min(1).max(200),
     include_answers: z.boolean().optional()
+})
+    .refine((value) => value.include_answers !== false, {
+    message: "include_answers=false is not allowed in strict column mode"
 });
 const listOutputSchema = z.object({
     ok: z.literal(true),
@@ -216,21 +217,33 @@ const listOutputSchema = z.object({
             include_answers: z.boolean(),
             row_cap: z.number().int().nonnegative(),
             column_cap: z.number().int().positive().nullable(),
-            selected_columns: z.array(z.string()).nullable()
+            selected_columns: z.array(z.string())
         })
             .optional()
     }),
     meta: apiMetaSchema
 });
 const recordGetInputSchema = z.object({
-    apply_id: z.union([z.string().min(1), z.number().int()])
+    apply_id: z.union([z.string().min(1), z.number().int()]),
+    max_columns: z.number().int().positive().max(200).optional(),
+    select_columns: z
+        .array(z.union([z.string().min(1), z.number().int()]))
+        .min(1)
+        .max(200)
+        .optional()
 });
 const recordGetOutputSchema = z.object({
     ok: z.literal(true),
     data: z.object({
         apply_id: z.union([z.string(), z.number(), z.null()]),
         answer_count: z.number().int().nonnegative(),
-        record: z.unknown()
+        record: z.unknown(),
+        applied_limits: z
+            .object({
+            column_cap: z.number().int().positive().nullable(),
+            selected_columns: z.array(z.string()).nullable()
+        })
+            .optional()
     }),
     meta: apiMetaSchema
 });
@@ -380,7 +393,7 @@ server.registerTool("qf_records_list", {
         const pageNum = args.page_num ?? 1;
         const pageSize = args.page_size ?? DEFAULT_PAGE_SIZE;
         const normalizedSort = await normalizeListSort(args.sort, args.app_key, args.user_id);
-        const includeAnswers = Boolean(args.include_answers || (args.select_columns && args.select_columns.length > 0));
+        const includeAnswers = true;
         const payload = buildListPayload({
             pageNum,
             pageSize,
@@ -410,6 +423,11 @@ server.registerTool("qf_records_list", {
             maxColumns: args.max_columns,
             selectColumns: args.select_columns
         });
+        if (items.length > 0 && columnProjection.matchedAnswersCount === 0) {
+            throw new Error(`No answers matched select_columns (${args.select_columns
+                .map((item) => String(item))
+                .join(", ")}). Check que_id/title from qf_form_get.`);
+        }
         const fitted = fitListItemsWithinSize({
             items: columnProjection.items,
             limitBytes: MAX_LIST_ITEMS_BYTES
@@ -457,13 +475,26 @@ server.registerTool("qf_record_get", {
     try {
         const response = await client.getRecord(String(args.apply_id));
         const record = asObject(response.result) ?? {};
-        const answerCount = asArray(record.answers).length;
+        const projection = projectAnswersForOutput({
+            answers: asArray(record.answers),
+            maxColumns: args.max_columns,
+            selectColumns: args.select_columns
+        });
+        const projectedRecord = {
+            ...record,
+            answers: projection.answers
+        };
+        const answerCount = projection.answers.length;
         return okResult({
             ok: true,
             data: {
                 apply_id: record.applyId ?? null,
                 answer_count: answerCount,
-                record: response.result
+                record: projectedRecord,
+                applied_limits: {
+                    column_cap: args.max_columns ?? null,
+                    selected_columns: projection.selectedColumns
+                }
             },
             meta: buildMeta(response)
         }, `Fetched record ${String(args.apply_id)}`);
@@ -901,12 +932,17 @@ function projectRecordItemsColumns(params) {
         return {
             items: params.items,
             reason: null,
-            selectedColumns: null
+            selectedColumns: [],
+            matchedAnswersCount: 0
         };
     }
     const normalizedSelectors = normalizeColumnSelectors(params.selectColumns);
+    if (normalizedSelectors.length === 0) {
+        throw new Error("select_columns must contain at least one non-empty column identifier");
+    }
     const selectorSet = new Set(normalizedSelectors.map((item) => normalizeColumnSelector(item)));
     let columnCapped = false;
+    let matchedAnswersCount = 0;
     const projectedItems = params.items.map((item) => {
         const answers = asArray(item.answers);
         let projected = answers;
@@ -917,6 +953,7 @@ function projectRecordItemsColumns(params) {
             projected = projected.slice(0, params.maxColumns);
             columnCapped = true;
         }
+        matchedAnswersCount += projected.length;
         return {
             ...item,
             answers: projected
@@ -928,6 +965,22 @@ function projectRecordItemsColumns(params) {
     return {
         items: projectedItems,
         reason,
+        selectedColumns: normalizedSelectors,
+        matchedAnswersCount
+    };
+}
+function projectAnswersForOutput(params) {
+    const normalizedSelectors = normalizeColumnSelectors(params.selectColumns);
+    const selectorSet = new Set(normalizedSelectors.map((item) => normalizeColumnSelector(item)));
+    let projected = params.answers;
+    if (selectorSet.size > 0) {
+        projected = projected.filter((answer) => answerMatchesAnySelector(answer, selectorSet));
+    }
+    if (params.maxColumns !== undefined && projected.length > params.maxColumns) {
+        projected = projected.slice(0, params.maxColumns);
+    }
+    return {
+        answers: projected,
         selectedColumns: normalizedSelectors.length > 0 ? normalizedSelectors : null
     };
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "qingflow-mcp",
-  "version": "0.2.3",
+  "version": "0.2.5",
   "private": false,
   "license": "MIT",
   "type": "module",