@josephyan/qingflow-cli 1.1.7 → 1.1.9

package/README.md

@@ -3,13 +3,13 @@
 Install:
 
 ```bash
-npm install @josephyan/qingflow-cli@1.1.7
+npm install @josephyan/qingflow-cli@1.1.9
 ```
 
 Run:
 
 ```bash
-npx -y -p @josephyan/qingflow-cli@1.1.7 qingflow
+npx -y -p @josephyan/qingflow-cli@1.1.9 qingflow
 ```
 
 Environment:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@josephyan/qingflow-cli",
-  "version": "1.1.7",
+  "version": "1.1.9",
   "description": "Human-friendly Qingflow command line interface for auth, record operations, import, tasks, and stable builder flows.",
   "license": "MIT",
   "type": "module",

package/pyproject.toml

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

package/src/qingflow_mcp/builder_facade/service.py

@@ -5999,6 +5999,20 @@ class AiBuilderFacade:
             result["message"] = "read app chart config"
         return result
 
+    def _chart_filter_field_names_by_id(
+        self,
+        *,
+        profile: str,
+        app_key: str,
+    ) -> tuple[dict[str, str], dict[str, Any] | None]:
+        try:
+            state = self._load_base_schema_state(profile=profile, app_key=app_key)
+        except (QingflowApiError, RuntimeError) as error:
+            api_error = _coerce_api_error(error)
+            return {}, _transport_error_payload(api_error)
+        fields = cast(list[dict[str, Any]], state["parsed"]["fields"])
+        return _chart_field_names_by_id_from_public_fields(app_key=app_key, fields=fields), None
+
     def app_get_buttons(self, *, profile: str, app_key: str) -> JSONObject:
         self.apps._require_app_key(app_key)
         warnings: list[dict[str, Any]] = []
@@ -6524,6 +6538,8 @@ class AiBuilderFacade:
             )
         charts = _summarize_charts(items)
         chart_visibility_read_errors: list[dict[str, Any]] = []
+        chart_config_read_errors: list[dict[str, Any]] = []
+        field_name_by_id, field_name_read_error = self._chart_filter_field_names_by_id(profile=profile, app_key=resolved_app_key)
         for chart in charts:
             chart_id = str(chart.get("chart_id") or "").strip()
             if not chart_id:
@@ -6546,6 +6562,24 @@ class AiBuilderFacade:
                     base_info.get("visibleAuth") if isinstance(base_info, dict) else None
                 )
             )
+            try:
+                config_response = self.charts.qingbi_report_get_config(profile=profile, chart_id=chart_id)
+                config = config_response.get("result") or {}
+            except (QingflowApiError, RuntimeError) as error:
+                api_error = _coerce_api_error(error)
+                chart_config_read_errors.append(
+                    {
+                        "chart_id": chart_id,
+                        "request_id": api_error.request_id,
+                        "http_status": api_error.http_status,
+                        "backend_code": api_error.backend_code,
+                    }
+                )
+                continue
+            if isinstance(config, dict):
+                chart["group_by"] = _public_chart_group_by_from_qingbi_config(config)
+                chart["metrics"] = _public_chart_metrics_from_qingbi_config(config)
+                chart["filters"] = _public_chart_filter_groups_from_qingbi_config(config, field_name_by_id=field_name_by_id)
         response = AppChartsReadResponse(
             app_key=resolved_app_key,
             charts=charts,
@@ -6559,7 +6593,11 @@ class AiBuilderFacade:
             "normalized_args": {"app_key": resolved_app_key},
             "missing_fields": [],
             "allowed_values": {},
-            "details": {"chart_visibility_read_errors": chart_visibility_read_errors} if chart_visibility_read_errors else {},
+            "details": {
+                **({"chart_visibility_read_errors": chart_visibility_read_errors} if chart_visibility_read_errors else {}),
+                **({"chart_config_read_errors": chart_config_read_errors} if chart_config_read_errors else {}),
+                **({"chart_filter_field_name_read_error": field_name_read_error} if field_name_read_error else {}),
+            },
             "request_id": None,
             "suggested_next_call": None,
             "noop": False,
@@ -6570,14 +6608,26 @@ class AiBuilderFacade:
                     if chart_visibility_read_errors
                     else []
                 )
+                + (
+                    [_warning("CHART_CONFIG_READ_PARTIAL", "some chart configs could not be read back; metrics/group_by/filters are incomplete for those charts")]
+                    if chart_config_read_errors
+                    else []
+                )
+                + (
+                    [_warning("CHART_FILTER_FIELD_NAMES_UNRESOLVED", "chart configs were read, but form fields could not be loaded to resolve filter field names")]
+                    if field_name_read_error
+                    else []
+                )
             ),
             "verification": {
                 "app_exists": True,
                 "chart_order_verified": list_source == "sorted",
                 "chart_list_source": list_source,
                 "chart_visibility_readback_complete": not chart_visibility_read_errors,
+                "chart_config_readback_complete": not chart_config_read_errors,
+                "chart_filter_field_names_resolved": not field_name_read_error,
             },
-            "verified": True,
+            "verified": not chart_config_read_errors and not field_name_read_error,
             **response.model_dump(mode="json"),
         }
 
@@ -7775,11 +7825,24 @@ class AiBuilderFacade:
                     suggested_next_call={"tool_name": "chart_get", "arguments": {"profile": profile, "chart_id": chart_id}},
                 )
 
+        field_name_by_id: dict[str, str] = {}
+        data_source = config.get("dataSource") if isinstance(config.get("dataSource"), dict) else {}
+        data_source_app_key = str(data_source.get("dataSourceId") or config.get("dataSourceId") or "").strip()
+        if data_source_app_key:
+            field_name_by_id, field_name_error = self._chart_filter_field_names_by_id(profile=profile, app_key=data_source_app_key)
+            if field_name_error:
+                warnings.append(
+                    _warning(
+                        "CHART_FILTER_FIELD_NAMES_UNRESOLVED",
+                        "chart config was read, but form fields could not be loaded to resolve filter field names",
+                        **field_name_error,
+                    )
+                )
         response = ChartGetResponse(
             chart_id=chart_id,
             base=deepcopy(base) if isinstance(base, dict) else {},
             visibility=_public_visibility_from_chart_visible_auth(base.get("visibleAuth")),
-            filters=_public_chart_filter_groups_from_qingbi_config(config) if isinstance(config, dict) else [],
+            filters=_public_chart_filter_groups_from_qingbi_config(config, field_name_by_id=field_name_by_id) if isinstance(config, dict) else [],
             group_by=_public_chart_group_by_from_qingbi_config(config) if isinstance(config, dict) else [],
             metrics=_public_chart_metrics_from_qingbi_config(config) if isinstance(config, dict) else [],
             config=deepcopy(config) if isinstance(config, dict) else {},
@@ -16288,11 +16351,38 @@ def _qingbi_chart_filter_value_to_text(*, value: Any, form_field: dict[str, Any]
     reject(value)
 
 
-def _public_chart_filter_groups_from_qingbi_config(config: dict[str, Any]) -> list[list[dict[str, Any]]]:
+def _chart_field_names_by_id_from_public_fields(*, app_key: str, fields: list[dict[str, Any]]) -> dict[str, str]:
+    field_name_by_id: dict[str, str] = {}
+    for field in fields:
+        if not isinstance(field, dict):
+            continue
+        name = str(field.get("name") or "").strip()
+        if not name:
+            continue
+        que_id = field.get("que_id")
+        field_id = str(field.get("field_id") or "").strip()
+        for raw_key in (
+            que_id,
+            field_id,
+            f"{app_key}:{que_id}" if que_id is not None else None,
+            f"{app_key}:{field_id}" if field_id else None,
+        ):
+            key = str(raw_key or "").strip()
+            if key:
+                field_name_by_id.setdefault(key, name)
+    return field_name_by_id
+
+
+def _public_chart_filter_groups_from_qingbi_config(
+    config: dict[str, Any],
+    *,
+    field_name_by_id: dict[str, str] | None = None,
+) -> list[list[dict[str, Any]]]:
     groups: list[list[dict[str, Any]]] = []
     raw_groups = config.get("beforeAggregationFilterMatrix")
     if not isinstance(raw_groups, list):
         return groups
+    resolved_field_name_by_id = field_name_by_id or {}
     for raw_group in raw_groups:
         if not isinstance(raw_group, list):
             continue
@@ -16302,19 +16392,26 @@ def _public_chart_filter_groups_from_qingbi_config(config: dict[str, Any]) -> li
                 continue
             operator = _public_chart_filter_operator_from_judge_type(raw_rule.get("judgeType"))
             field_id = raw_rule.get("fieldId") or raw_rule.get("field_id")
-            field_name = (
+            field_id_text = _stringify_condition_value(field_id).strip() if field_id is not None else ""
+            raw_field_name = (
                 raw_rule.get("fieldName")
                 or raw_rule.get("field_name")
                 or raw_rule.get("queTitle")
                 or raw_rule.get("title")
-                or field_id
+            )
+            raw_field_name_text = _stringify_condition_value(raw_field_name).strip()
+            field_name = (
+                resolved_field_name_by_id.get(raw_field_name_text)
+                or resolved_field_name_by_id.get(field_id_text)
+                or raw_field_name_text
+                or field_id_text
             )
             public_rule: dict[str, Any] = {
-                "field_name": _stringify_condition_value(field_name).strip(),
+                "field_name": field_name,
                 "operator": operator,
             }
             if field_id is not None:
-                public_rule["field_id"] = _stringify_condition_value(field_id).strip()
+                public_rule["field_id"] = field_id_text
             values = _public_chart_filter_values_from_rule(raw_rule, operator=operator)
             if values:
                 public_rule["values"] = values
@@ -23330,6 +23427,24 @@ def _extract_view_question_entries(questions: Any) -> list[dict[str, Any]]:
                 "visible": visible,
                 "display_order": display_order if display_order is not None else fallback_order,
             }
+            option_details: list[dict[str, Any]] = []
+            option_values: list[str] = []
+            raw_options = item.get("options")
+            if isinstance(raw_options, list):
+                for raw_option in raw_options:
+                    if not isinstance(raw_option, dict):
+                        continue
+                    option_id = raw_option.get("optId")
+                    option_value = str(raw_option.get("optValue") or "").strip()
+                    if not option_value and option_id is None:
+                        continue
+                    if option_value:
+                        option_values.append(option_value)
+                    option_details.append({"id": option_id, "value": option_value or str(option_id)})
+            if option_values:
+                entry["options"] = option_values
+            if option_details:
+                entry["option_details"] = option_details
             width = _coerce_positive_int(item.get("width"))
             if width is not None:
                 entry["width"] = width
@@ -23348,7 +23463,14 @@ def _view_field_lookup_from_question_entries(entries: list[dict[str, Any]]) -> d
         que_id = _coerce_positive_int(entry.get("field_id") or entry.get("que_id") or entry.get("queId"))
         if not name or que_id is None:
             continue
-        fields_by_name.setdefault(name, {"name": name, "que_id": que_id})
+        field_entry: dict[str, Any] = {"name": name, "que_id": que_id}
+        options = entry.get("options")
+        if isinstance(options, list) and options:
+            field_entry["options"] = [str(value) for value in options if str(value or "").strip()]
+        option_details = entry.get("option_details")
+        if isinstance(option_details, list) and option_details:
+            field_entry["option_details"] = [deepcopy(value) for value in option_details if isinstance(value, dict)]
+        fields_by_name.setdefault(name, field_entry)
     return fields_by_name
 
 
@@ -26464,6 +26586,38 @@ def _view_filter_rule_values_for_signature(rule: dict[str, Any]) -> list[str]:
     return fallback_values
 
 
+def _view_filter_option_value_by_id(field: dict[str, Any]) -> dict[str, str]:
+    value_by_id: dict[str, str] = {}
+    for detail in field.get("option_details") or []:
+        if not isinstance(detail, dict):
+            continue
+        option_id = detail.get("id")
+        option_value = str(detail.get("value") or "").strip()
+        if option_id is None or not option_value:
+            continue
+        value_by_id[str(option_id)] = option_value
+    return value_by_id
+
+
+def _public_view_filter_rule_values(rule: dict[str, Any], *, field: dict[str, Any]) -> list[str]:
+    value_by_id = _view_filter_option_value_by_id(field)
+    detail_value_by_id: dict[str, str] = {}
+    ordered_detail_values: list[str] = []
+    for detail in rule.get("judgeValueDetails") or []:
+        if not isinstance(detail, dict):
+            continue
+        detail_value = str(detail.get("value") or "").strip()
+        detail_id = detail.get("id")
+        if detail_value:
+            ordered_detail_values.append(detail_value)
+            if detail_id is not None:
+                detail_value_by_id[str(detail_id)] = detail_value
+    values = [str(value) for value in (rule.get("judgeValues") or []) if str(value or "").strip()]
+    if values:
+        return [value_by_id.get(value) or detail_value_by_id.get(value) or value for value in values]
+    return ordered_detail_values
+
+
 def _view_filter_groups_signature(groups: Any) -> list[list[dict[str, Any]]]:
     signature: list[list[dict[str, Any]]] = []
     for group in _normalize_view_filter_groups_for_compare(groups):
@@ -26545,7 +26699,7 @@ def _public_view_filter_groups_from_match_rules(
         for rule in group:
             que_id = _coerce_positive_int(rule.get("queId")) or 0
             field = fields_by_que_id.get(que_id) or {}
-            values = _view_filter_rule_values_for_signature(rule)
+            values = _public_view_filter_rule_values(rule, field=field)
             public_rule: dict[str, Any] = {
                 "field_name": str(field.get("name") or rule.get("queTitle") or que_id),
                 "operator": _public_view_filter_operator_from_judge_type(rule.get("judgeType")),