@josephyan/qingflow-cli 0.2.0-beta.70 → 0.2.0-beta.71

package/README.md

@@ -3,13 +3,13 @@
 Install:
 
 ```bash
-npm install @josephyan/qingflow-cli@0.2.0-beta.70
+npm install @josephyan/qingflow-cli@0.2.0-beta.71
 ```
 
 Run:
 
 ```bash
-npx -y -p @josephyan/qingflow-cli@0.2.0-beta.70 qingflow
+npx -y -p @josephyan/qingflow-cli@0.2.0-beta.71 qingflow
 ```
 
 Environment:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@josephyan/qingflow-cli",
-  "version": "0.2.0-beta.70",
+  "version": "0.2.0-beta.71",
   "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 = "0.2.0b70"
+version = "0.2.0b71"
 description = "User-authenticated MCP server for Qingflow"
 readme = "README.md"
 license = "MIT"

package/src/qingflow_mcp/builder_facade/service.py

@@ -2505,9 +2505,14 @@ class AiBuilderFacade:
         page_size_y: int | None = None,
     ) -> JSONObject:
         normalized_payload = deepcopy(data_payload) if isinstance(data_payload, dict) else {}
+        warnings: list[dict[str, Any]] = []
+        verification = {
+            "chart_exists": True,
+            "chart_data_loaded": True,
+            "chart_config_loaded": True,
+        }
         try:
             base = self.charts.qingbi_report_get_base(profile=profile, chart_id=chart_id).get("result") or {}
-            config = self.charts.qingbi_report_get_config(profile=profile, chart_id=chart_id).get("result") or {}
             data = self.charts.qingbi_report_get_data(
                 profile=profile,
                 chart_id=chart_id,
@@ -2534,6 +2539,36 @@ class AiBuilderFacade:
                 suggested_next_call={"tool_name": "chart_get", "arguments": {"profile": profile, "chart_id": chart_id}},
             )
 
+        try:
+            config = self.charts.qingbi_report_get_config(profile=profile, chart_id=chart_id).get("result") or {}
+        except (QingflowApiError, RuntimeError) as error:
+            config_from_data = data.get("config") if isinstance(data, dict) else None
+            if isinstance(config_from_data, dict):
+                config = deepcopy(config_from_data)
+                verification["chart_config_loaded"] = True
+                warnings.append(
+                    _warning(
+                        "CHART_CONFIG_FALLBACK_FROM_DATA",
+                        "chart config endpoint is unavailable for this chart id; using config embedded in chart data instead",
+                    )
+                )
+            else:
+                api_error = _coerce_api_error(error)
+                return _failed_from_api_error(
+                    "CHART_GET_FAILED",
+                    api_error,
+                    normalized_args={
+                        "chart_id": chart_id,
+                        "data_payload": normalized_payload,
+                        "page_num": page_num,
+                        "page_size": page_size,
+                        "page_num_y": page_num_y,
+                        "page_size_y": page_size_y,
+                    },
+                    details={"chart_id": chart_id},
+                    suggested_next_call={"tool_name": "chart_get", "arguments": {"profile": profile, "chart_id": chart_id}},
+                )
+
         response = ChartGetResponse(
             chart_id=chart_id,
             base=deepcopy(base) if isinstance(base, dict) else {},
@@ -2559,8 +2594,8 @@ class AiBuilderFacade:
             "request_id": None,
             "suggested_next_call": None,
             "noop": False,
-            "warnings": [],
-            "verification": {"chart_exists": True, "chart_data_loaded": True},
+            "warnings": warnings,
+            "verification": verification,
             "verified": True,
             **response.model_dump(mode="json"),
         }

package/src/qingflow_mcp/tools/qingbi_report_tools.py

@@ -1,5 +1,6 @@
 from __future__ import annotations
 
+import json
 from uuid import uuid4
 
 from mcp.server.fastmcp import FastMCP
@@ -18,6 +19,37 @@ def _qingbi_base_url(base_url: str) -> str:
     return normalized[:-4] if normalized.endswith("/api") else normalized
 
 
+def _should_retry_qflow_base(error: QingflowApiError) -> bool:
+    return int(getattr(error, "backend_code", 0) or 0) == 81007
+
+
+def _should_retry_asos_data(error: QingflowApiError) -> bool:
+    backend_code = int(getattr(error, "backend_code", 0) or 0)
+    http_status = getattr(error, "http_status", None)
+    return backend_code in {44011, 81007} or http_status == 404
+
+
+def _coerce_tool_error(error: RuntimeError | QingflowApiError) -> QingflowApiError | None:
+    if isinstance(error, QingflowApiError):
+        return error
+    if not isinstance(error, RuntimeError):
+        return None
+    try:
+        payload = json.loads(str(error))
+    except Exception:
+        return None
+    if not isinstance(payload, dict):
+        return None
+    return QingflowApiError(
+        category=str(payload.get("category") or "runtime"),
+        message=str(payload.get("message") or str(error)),
+        backend_code=payload.get("backend_code"),
+        request_id=payload.get("request_id"),
+        http_status=payload.get("http_status"),
+        details=payload.get("details") if isinstance(payload.get("details"), dict) else None,
+    )
+
+
 class QingbiReportTools(ToolBase):
     def register(self, mcp: FastMCP) -> None:
         @mcp.tool()
@@ -127,7 +159,13 @@ class QingbiReportTools(ToolBase):
 
     def qingbi_report_get_base(self, *, profile: str, chart_id: str) -> JSONObject:
         self._require_chart_id(chart_id)
-        return self._request(profile, "GET", f"/qingbi/charts/baseinfo/{chart_id}", chart_id=chart_id)
+        try:
+            return self._request(profile, "GET", f"/qingbi/charts/baseinfo/{chart_id}", chart_id=chart_id)
+        except (QingflowApiError, RuntimeError) as raw_error:
+            error = _coerce_tool_error(raw_error)
+            if error is None or not _should_retry_qflow_base(error):
+                raise
+        return self._request(profile, "GET", f"/qingbi/charts/qflow/baseinfo/{chart_id}", chart_id=chart_id)
 
     def qingbi_report_update_base(self, *, profile: str, chart_id: str, payload: JSONObject) -> JSONObject:
         self._require_chart_id(chart_id)
@@ -166,21 +204,34 @@ class QingbiReportTools(ToolBase):
             params["pageNumY"] = page_num_y
         if page_size_y is not None:
             params["pageSizeY"] = page_size_y
-        if payload:
+        try:
+            if payload:
+                return self._request(
+                    profile,
+                    "POST",
+                    f"/qingbi/charts/data/qflow/{chart_id}/detail",
+                    chart_id=chart_id,
+                    params=params,
+                    json_body=payload,
+                )
             return self._request(
                 profile,
-                "POST",
-                f"/qingbi/charts/data/qflow/{chart_id}/detail",
+                "GET",
+                f"/qingbi/charts/data/qflow/{chart_id}",
                 chart_id=chart_id,
                 params=params,
-                json_body=payload,
             )
+        except (QingflowApiError, RuntimeError) as raw_error:
+            error = _coerce_tool_error(raw_error)
+            if error is None or not _should_retry_asos_data(error):
+                raise
         return self._request(
             profile,
-            "GET",
-            f"/qingbi/charts/data/qflow/{chart_id}",
+            "POST",
+            f"/qingbi/charts/data/qflow/{chart_id}/asos",
             chart_id=chart_id,
             params=params,
+            json_body=payload or {},
         )
 
     def qingbi_report_delete(self, *, profile: str, chart_id: str) -> JSONObject: