@qingflow-tech/qingflow-app-user-mcp 1.0.36 → 1.0.38

package/README.md

@@ -3,13 +3,13 @@
 Install:
 
 ```bash
-npm install @qingflow-tech/qingflow-app-user-mcp@1.0.36
+npm install @qingflow-tech/qingflow-app-user-mcp@1.0.38
 ```
 
 Run:
 
 ```bash
-npx -y -p @qingflow-tech/qingflow-app-user-mcp@1.0.36 qingflow-app-user-mcp
+npx -y -p @qingflow-tech/qingflow-app-user-mcp@1.0.38 qingflow-app-user-mcp
 ```
 
 Environment:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@qingflow-tech/qingflow-app-user-mcp",
-  "version": "1.0.36",
+  "version": "1.0.38",
   "description": "Operational end-user MCP for Qingflow records, tasks, comments, and directory workflows.",
   "license": "MIT",
   "type": "module",

package/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "qingflow-mcp"
-version = "1.0.36"
+version = "1.0.38"
 description = "User-authenticated MCP server for Qingflow"
 readme = "README.md"
 license = "MIT"

package/src/qingflow_mcp/builder_facade/service.py

@@ -9,7 +9,7 @@ import random
 import re
 import string
 import tempfile
-from typing import Any, cast
+from typing import Any, NoReturn, cast
 from urllib.parse import quote_plus, unquote_plus
 from uuid import uuid4
 
@@ -11378,11 +11378,14 @@ class AiBuilderFacade:
 
         if failed_items:
             successful_changes = bool(created_ids or updated_ids or removed_ids or reordered)
+            primary_failure = failed_items[0]
+            primary_error_code = str(primary_failure.get("error_code") or "").strip()
+            primary_message = str(primary_failure.get("message") or "").strip()
             return finalize({
                 "status": "partial_success" if successful_changes else "failed",
-                "error_code": "CHART_APPLY_PARTIAL" if successful_changes else "CHART_APPLY_FAILED",
+                "error_code": "CHART_APPLY_PARTIAL" if successful_changes else primary_error_code or "CHART_APPLY_FAILED",
                 "recoverable": True,
-                "message": "applied some chart operations; at least one chart operation failed" if successful_changes else "one or more chart operations failed",
+                "message": "applied some chart operations; at least one chart operation failed" if successful_changes else primary_message or "one or more chart operations failed",
                 "normalized_args": normalized_args,
                 "missing_fields": [],
                 "allowed_values": {"chart.chart_type": [member.value for member in PublicChartType], "chart.filter.operator": [member.value for member in ViewFilterOperator]},
@@ -14615,7 +14618,18 @@ def _build_qingbi_chart_field_lookup(
     field_lookup: dict[str, dict[str, Any]],
 ) -> dict[str, Any]:
     by_selector: dict[str, list[dict[str, Any]]] = {}
-    form_by_que_id = field_lookup.get("by_que_id") or {}
+    form_by_que_id = dict(field_lookup.get("by_que_id") or {})
+    if not form_by_que_id:
+        for bucket_name in ("by_name", "by_field_id"):
+            bucket = field_lookup.get(bucket_name) or {}
+            if not isinstance(bucket, dict):
+                continue
+            for form_field in bucket.values():
+                if not isinstance(form_field, dict):
+                    continue
+                que_id = _coerce_any_int(form_field.get("que_id"))
+                if que_id is not None:
+                    form_by_que_id.setdefault(que_id, form_field)
 
     def add_selector(key: Any, field: dict[str, Any]) -> None:
         normalized = str(key or "").strip()
@@ -15095,13 +15109,109 @@ def _build_public_chart_filter_matrix(
                 "fieldName": qingbi_field.get("fieldName") or form_field.get("name") or field_id,
                 "fieldType": qingbi_field.get("fieldType") or _qingbi_field_type_from_public_field(str(form_field.get("type") or "")),
                 "judgeType": judge_map.get(operator, "equal"),
-                "judgeValues": values,
+                "judgeValue": _build_qingbi_chart_filter_judge_value(
+                    operator=operator,
+                    values=values,
+                    form_field=form_field,
+                    chart_type=chart_type,
+                ),
+                "judgeValueDetailList": [],
                 "matchType": 1,
             }
         )
     return [group] if group else []
 
 
+def _build_qingbi_chart_filter_judge_value(
+    *,
+    operator: str,
+    values: list[Any],
+    form_field: dict[str, Any],
+    chart_type: str,
+) -> str | None:
+    if operator in {ViewFilterOperator.is_empty.value, ViewFilterOperator.not_empty.value}:
+        return None
+    if operator == ViewFilterOperator.in_.value:
+        return "<&&>".join(
+            _qingbi_chart_filter_value_to_text(value=value, form_field=form_field, chart_type=chart_type) for value in values
+        )
+    if not values:
+        return ""
+    return _qingbi_chart_filter_value_to_text(value=values[0], form_field=form_field, chart_type=chart_type)
+
+
+def _qingbi_chart_filter_value_to_text(*, value: Any, form_field: dict[str, Any], chart_type: str) -> str:
+    option_details = [
+        item
+        for item in (form_field.get("option_details") or [])
+        if isinstance(item, dict) and item.get("id") is not None and item.get("value") is not None
+    ]
+    if not option_details:
+        if isinstance(value, dict):
+            for key in ("value", "label", "name", "title"):
+                raw = str(value.get(key) or "").strip()
+                if raw:
+                    return raw
+            if value.get("id") is not None:
+                return str(value.get("id"))
+        return _stringify_condition_value(value)
+
+    option_by_value = {str(item.get("value") or "").strip(): item for item in option_details if str(item.get("value") or "").strip()}
+    option_by_id = {str(item.get("id")): item for item in option_details if item.get("id") is not None}
+
+    def resolve(raw: Any) -> str | None:
+        text = _stringify_condition_value(raw).strip()
+        if not text:
+            return None
+        matched = option_by_value.get(text) or option_by_id.get(text)
+        if not matched:
+            return None
+        return str(matched.get("value") or "").strip()
+
+    def reject(raw: Any) -> NoReturn:
+        allowed = [
+            {"id": str(item.get("id")), "value": str(item.get("value"))}
+            for item in option_details
+        ]
+        _raise_chart_rule(
+            rule_code="CHART_FILTER_OPTION_VALUE_UNSUPPORTED",
+            chart_type=chart_type,
+            message="chart filter value is not in the field option list",
+            expected="Use an existing option label or option id from the target field schema.",
+            actual={
+                "field_name": form_field.get("name"),
+                "field_id": form_field.get("que_id"),
+                "value": raw,
+                "allowed_values": allowed,
+            },
+            offending_fields=[{"field_name": form_field.get("name"), "field_id": form_field.get("que_id"), "options": allowed}],
+            next_action="Read schema first, then pass one of the allowed option labels or option ids.",
+        )
+
+    if isinstance(value, dict):
+        for key in ("value", "label", "name", "title"):
+            if value.get(key) is not None:
+                resolved = resolve(value.get(key))
+                if resolved is not None:
+                    return resolved
+                reject(value.get(key))
+        if value.get("id") is not None:
+            resolved = resolve(value.get("id"))
+            if resolved is not None:
+                return resolved
+            reject(value.get("id"))
+        reject(value)
+    if isinstance(value, int):
+        resolved = resolve(value)
+        if resolved is not None:
+            return resolved
+        reject(value)
+    resolved = resolve(value)
+    if resolved is not None:
+        return resolved
+    reject(value)
+
+
 def _build_public_chart_config_payload(
     *,
     patch: ChartUpsertPatch,