@josephyan/qingflow-app-user-mcp 0.2.0-beta.38 → 0.2.0-beta.39

package/README.md

@@ -3,13 +3,13 @@
 Install:
 
 ```bash
-npm install @josephyan/qingflow-app-user-mcp@0.2.0-beta.38
+npm install @josephyan/qingflow-app-user-mcp@0.2.0-beta.39
 ```
 
 Run:
 
 ```bash
-npx -y -p @josephyan/qingflow-app-user-mcp@0.2.0-beta.38 qingflow-app-user-mcp
+npx -y -p @josephyan/qingflow-app-user-mcp@0.2.0-beta.39 qingflow-app-user-mcp
 ```
 
 Environment:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@josephyan/qingflow-app-user-mcp",
-  "version": "0.2.0-beta.38",
+  "version": "0.2.0-beta.39",
   "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 = "0.2.0b38"
+version = "0.2.0b39"
 description = "User-authenticated MCP server for Qingflow"
 readme = "README.md"
 license = "MIT"

package/src/qingflow_mcp/__init__.py

@@ -2,4 +2,4 @@ from __future__ import annotations
 
 __all__ = ["__version__"]
 
-__version__ = "0.2.0b38"
+__version__ = "0.2.0b39"

package/src/qingflow_mcp/builder_facade/models.py

@@ -589,6 +589,12 @@ class ChartUpsertPatch(StrictModel):
             }
             if normalized in aliases:
                 payload["chart_type"] = aliases[normalized]
+        if isinstance(payload.get("chart_id"), int):
+            payload["chart_id"] = str(payload["chart_id"])
+        if isinstance(payload.get("dimension_field_ids"), list):
+            payload["dimension_field_ids"] = [str(item) for item in payload["dimension_field_ids"] if item is not None and str(item).strip()]
+        if isinstance(payload.get("indicator_field_ids"), list):
+            payload["indicator_field_ids"] = [str(item) for item in payload["indicator_field_ids"] if item is not None and str(item).strip()]
         return payload
 
 
@@ -598,6 +604,18 @@ class ChartApplyRequest(StrictModel):
     remove_chart_ids: list[str] = Field(default_factory=list)
     reorder_chart_ids: list[str] = Field(default_factory=list)
 
+    @model_validator(mode="before")
+    @classmethod
+    def normalize_ids(cls, value: Any) -> Any:
+        if not isinstance(value, dict):
+            return value
+        payload = dict(value)
+        for key in ("remove_chart_ids", "reorder_chart_ids"):
+            raw = payload.get(key)
+            if isinstance(raw, list):
+                payload[key] = [str(item) for item in raw if item is not None and str(item).strip()]
+        return payload
+
     @model_validator(mode="after")
     def validate_shape(self) -> "ChartApplyRequest":
         if not self.upsert_charts and not self.remove_chart_ids and not self.reorder_chart_ids:
@@ -615,6 +633,34 @@ class PortalComponentPositionPatch(StrictModel):
     mobile_w: int = Field(default=12, validation_alias=AliasChoices("mobile_w", "mobileW"))
     mobile_h: int = Field(default=8, validation_alias=AliasChoices("mobile_h", "mobileH"))
 
+    @model_validator(mode="before")
+    @classmethod
+    def normalize_nested_layout(cls, value: Any) -> Any:
+        if not isinstance(value, dict):
+            return value
+        payload = dict(value)
+        pc = payload.pop("pc", None)
+        mobile = payload.pop("mobile", None)
+        if isinstance(pc, dict):
+            if "pc_x" not in payload and "x" in pc:
+                payload["pc_x"] = pc.get("x")
+            if "pc_y" not in payload and "y" in pc:
+                payload["pc_y"] = pc.get("y")
+            if "pc_w" not in payload and "cols" in pc:
+                payload["pc_w"] = pc.get("cols")
+            if "pc_h" not in payload and "rows" in pc:
+                payload["pc_h"] = pc.get("rows")
+        if isinstance(mobile, dict):
+            if "mobile_x" not in payload and "x" in mobile:
+                payload["mobile_x"] = mobile.get("x")
+            if "mobile_y" not in payload and "y" in mobile:
+                payload["mobile_y"] = mobile.get("y")
+            if "mobile_w" not in payload and "cols" in mobile:
+                payload["mobile_w"] = mobile.get("cols")
+            if "mobile_h" not in payload and "rows" in mobile:
+                payload["mobile_h"] = mobile.get("rows")
+        return payload
+
 
 class PortalChartRefPatch(StrictModel):
     app_key: str

package/src/qingflow_mcp/builder_facade/service.py

@@ -2947,9 +2947,9 @@ class AiBuilderFacade:
             if isinstance(item, dict) and item.get("fieldId")
         }
         existing_by_id = {
-            str(item.get("chartId") or item.get("chartKey") or ""): deepcopy(item)
+            _extract_chart_identifier(item): deepcopy(item)
             for item in existing_chart_items
-            if isinstance(item, dict) and (item.get("chartId") or item.get("chartKey"))
+            if isinstance(item, dict) and _extract_chart_identifier(item)
         }
         existing_by_name = {
             str(item.get("chartName") or "").strip(): deepcopy(item)
@@ -2970,9 +2970,9 @@ class AiBuilderFacade:
                     existing = existing_by_id.get(str(patch.chart_id))
                 if existing is None:
                     existing = existing_by_name.get(patch.name)
-                chart_id = str((existing or {}).get("chartId") or (existing or {}).get("chartKey") or patch.chart_id or f"mcp_{uuid4().hex[:16]}")
+                chart_id = _extract_chart_identifier(existing or {}) or str(patch.chart_id or f"mcp_{uuid4().hex[:16]}")
                 existing_name = str((existing or {}).get("chartName") or "").strip()
-                existing_type = str((existing or {}).get("chartType") or "").strip()
+                existing_type = _normalize_backend_chart_type((existing or {}).get("chartType"))
                 target_type = _map_public_chart_type_to_backend(patch.chart_type)
                 if existing is None:
                     create_payload = {
@@ -2989,7 +2989,15 @@ class AiBuilderFacade:
                         "editAuthIncludeSubDept": True,
                     }
                     create_result = self.charts.qingbi_report_create(profile=profile, payload=create_payload).get("result") or {}
-                    created_chart_id = str((create_result or {}).get("chartId") or (create_result or {}).get("chartKey") or "").strip()
+                    created_chart_id = _extract_chart_identifier(create_result or {})
+                    if not created_chart_id:
+                        refreshed_items = self.charts.qingbi_report_list(profile=profile, app_key=app_key).get("items") or []
+                        refreshed_existing = _find_chart_by_name(
+                            refreshed_items,
+                            chart_name=patch.name,
+                            chart_type=target_type,
+                        )
+                        created_chart_id = _extract_chart_identifier(refreshed_existing or {})
                     if created_chart_id:
                         chart_id = created_chart_id
                     created_ids.append(chart_id)
@@ -3035,7 +3043,7 @@ class AiBuilderFacade:
             except (QingflowApiError, RuntimeError, ValueError) as error:
                 api_error = _coerce_api_error(error) if not isinstance(error, ValueError) else None
                 failure = {
-                    "chart_id": str(patch.chart_id or ""),
+                    "chart_id": str(locals().get("chart_id") or patch.chart_id or ""),
                     "name": patch.name,
                     "status": "failed",
                     "message": str(error),
@@ -3082,13 +3090,13 @@ class AiBuilderFacade:
                 failed_items.append(failure)
                 chart_results.append(failure)
 
-        noop = not created_ids and not updated_ids and not removed_ids and not reordered
+        noop = not created_ids and not updated_ids and not removed_ids and not reordered and not failed_items
         try:
             readback_items = self.charts.qingbi_report_list(profile=profile, app_key=app_key).get("items") or []
             readback_ids = {
-                str(item.get("chartId") or item.get("chartKey") or "")
+                _extract_chart_identifier(item)
                 for item in readback_items
-                if isinstance(item, dict) and (item.get("chartId") or item.get("chartKey"))
+                if isinstance(item, dict) and _extract_chart_identifier(item)
             }
             verified = (
                 all(chart_id in readback_ids for chart_id in created_ids + updated_ids)
@@ -3096,9 +3104,9 @@ class AiBuilderFacade:
             )
             if request.reorder_chart_ids:
                 ordered_readback = [
-                    str(item.get("chartId") or item.get("chartKey") or "")
+                    _extract_chart_identifier(item)
                     for item in readback_items
-                    if isinstance(item, dict) and (item.get("chartId") or item.get("chartKey"))
+                    if isinstance(item, dict) and _extract_chart_identifier(item)
                 ]
                 requested_existing = [chart_id for chart_id in request.reorder_chart_ids if chart_id in ordered_readback]
                 verified = verified and ordered_readback[: len(requested_existing)] == requested_existing
@@ -3123,10 +3131,10 @@ class AiBuilderFacade:
                 "backend_code": failed_items[0].get("backend_code"),
                 "http_status": failed_items[0].get("http_status"),
                 "noop": noop,
-                "verification": {"charts_verified": verified, "readback_unavailable": readback_unavailable},
+                "verification": {"charts_verified": False if failed_items else verified, "readback_unavailable": readback_unavailable},
                 "app_key": app_key,
                 "chart_results": chart_results,
-                "verified": verified,
+                "verified": False if failed_items else verified,
             }
         return {
             "status": "success" if verified or noop else "partial_success",
@@ -3212,6 +3220,16 @@ class AiBuilderFacade:
             )
             update_payload["components"] = component_payload
             self.portals.portal_update(profile=profile, dash_key=dash_key, payload=update_payload)
+            self.portals.portal_update_base_info(
+                profile=profile,
+                dash_key=dash_key,
+                payload={
+                    "dashName": update_payload.get("dashName"),
+                    "dashIcon": update_payload.get("dashIcon"),
+                    "auth": deepcopy(update_payload.get("auth")),
+                    "tags": deepcopy(update_payload.get("tags") or []),
+                },
+            )
             draft_result = self.portals.portal_get(profile=profile, dash_key=dash_key, being_draft=True).get("result") or {}
         except (QingflowApiError, RuntimeError, ValueError) as error:
             api_error = _coerce_api_error(error) if not isinstance(error, ValueError) else None
@@ -4258,6 +4276,61 @@ def _build_public_portal_base_payload(
     return data
 
 
+def _extract_chart_identifier(chart: Any) -> str:
+    if not isinstance(chart, dict):
+        return ""
+    return str(
+        chart.get("chartId")
+        or chart.get("biChartId")
+        or chart.get("chartKey")
+        or ""
+    ).strip()
+
+
+def _normalize_backend_chart_type(value: Any) -> str:
+    raw = str(value or "").strip()
+    if not raw:
+        return ""
+    by_code = {
+        "1": "detail",
+        "2": "summary",
+        "3": "indicator",
+        "4": "columnar",
+        "5": "line",
+        "6": "pie",
+        "7": "funnel",
+        "8": "radar",
+        "9": "bar",
+        "10": "scatter",
+        "11": "ring",
+        "12": "dualaxes",
+        "13": "map",
+        "14": "timeline",
+    }
+    return by_code.get(raw, raw.lower())
+
+
+def _find_chart_by_name(items: Any, *, chart_name: str, chart_type: str | None = None) -> dict[str, Any] | None:
+    target_name = str(chart_name or "").strip()
+    target_type = _normalize_backend_chart_type(chart_type)
+    candidates: list[dict[str, Any]] = []
+    if not isinstance(items, list) or not target_name:
+        return None
+    for item in items:
+        if not isinstance(item, dict):
+            continue
+        item_name = str(item.get("chartName") or "").strip()
+        if item_name != target_name:
+            continue
+        item_type = _normalize_backend_chart_type(item.get("chartType"))
+        if target_type and item_type and item_type != target_type:
+            continue
+        candidates.append(item)
+    if not candidates:
+        return None
+    return deepcopy(candidates[-1])
+
+
 def _portal_position_payload(position: Any) -> dict[str, Any]:
     return {
         "pc": {
@@ -4332,7 +4405,7 @@ def _resolve_chart_reference(*, charts: QingbiReportTools, profile: str, ref: An
     for item in items:
         if not isinstance(item, dict):
             continue
-        item_id = str(item.get("chartId") or item.get("chartKey") or "").strip()
+        item_id = _extract_chart_identifier(item)
         item_name = str(item.get("chartName") or "").strip()
         if chart_id and item_id == chart_id:
             return {"chart_id": item_id, "chart_name": item_name, "app_key": app_key}

package/src/qingflow_mcp/tools/portal_tools.py

@@ -18,10 +18,18 @@ class PortalTools(ToolBase):
         def portal_get(profile: str = DEFAULT_PROFILE, dash_key: str = "", being_draft: bool = False) -> JSONObject:
             return self.portal_get(profile=profile, dash_key=dash_key, being_draft=being_draft)
 
+        @mcp.tool()
+        def portal_get_base_info(profile: str = DEFAULT_PROFILE, dash_key: str = "") -> JSONObject:
+            return self.portal_get_base_info(profile=profile, dash_key=dash_key)
+
         @mcp.tool()
         def portal_create(profile: str = DEFAULT_PROFILE, payload: JSONObject | None = None) -> JSONObject:
             return self.portal_create(profile=profile, payload=payload or {})
 
+        @mcp.tool(description=self._high_risk_tool_description(operation="update", target="portal base settings"))
+        def portal_update_base_info(profile: str = DEFAULT_PROFILE, dash_key: str = "", payload: JSONObject | None = None) -> JSONObject:
+            return self.portal_update_base_info(profile=profile, dash_key=dash_key, payload=payload or {})
+
         @mcp.tool(description=self._high_risk_tool_description(operation="update", target="portal configuration"))
         def portal_update(profile: str = DEFAULT_PROFILE, dash_key: str = "", payload: JSONObject | None = None) -> JSONObject:
             return self.portal_update(profile=profile, dash_key=dash_key, payload=payload or {})
@@ -59,6 +67,29 @@ class PortalTools(ToolBase):
 
         return self._run(profile, runner)
 
+    def portal_get_base_info(self, *, profile: str, dash_key: str) -> JSONObject:
+        self._require_dash_key(dash_key)
+
+        def runner(session_profile, context):
+            result = self.backend.request("GET", context, f"/dash/{dash_key}/baseInfo")
+            return {"profile": profile, "ws_id": session_profile.selected_ws_id, "dash_key": dash_key, "result": result}
+
+        return self._run(profile, runner)
+
+    def portal_update_base_info(self, *, profile: str, dash_key: str, payload: JSONObject) -> JSONObject:
+        self._require_dash_key(dash_key)
+        body = self._require_dict(payload)
+
+        def runner(session_profile, context):
+            result = self.backend.request("POST", context, f"/dash/{dash_key}/baseInfo", json_body=body)
+            return self._attach_human_review_notice(
+                {"profile": profile, "ws_id": session_profile.selected_ws_id, "dash_key": dash_key, "result": result},
+                operation="update",
+                target="portal base settings",
+            )
+
+        return self._run(profile, runner)
+
     def portal_update(self, *, profile: str, dash_key: str, payload: JSONObject) -> JSONObject:
         self._require_dash_key(dash_key)
         body = self._require_dict(payload)