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

package/src/qingflow_mcp/builder_facade/service.py

@@ -1079,7 +1079,7 @@ class AiBuilderFacade:
                 api_error,
                 normalized_args=normalized_args,
                 details={"app_key": app_key},
-                suggested_next_call={"tool_name": "app_read_summary", "arguments": {"profile": profile, "app_key": app_key}},
+                suggested_next_call={"tool_name": "app_get", "arguments": {"profile": profile, "app_key": app_key}},
             )
         edit_version_no = _coerce_positive_int(version_result.get("editVersionNo") or version_result.get("versionNo")) or 1
         try:
@@ -1114,7 +1114,7 @@ class AiBuilderFacade:
                 "edit_version_no": edit_version_no,
             },
             "request_id": None,
-            "suggested_next_call": {"tool_name": "app_read_summary", "arguments": {"profile": profile, "app_key": app_key}},
+            "suggested_next_call": {"tool_name": "app_get", "arguments": {"profile": profile, "app_key": app_key}},
             "noop": False,
             "verification": {"released": True},
             "app_key": app_key,
@@ -1154,7 +1154,7 @@ class AiBuilderFacade:
                             "permission_check_skipped": True,
                         },
                         "request_id": api_error.request_id,
-                        "suggested_next_call": {"tool_name": "app_read_summary", "arguments": {"profile": profile, "app_key": app_key}},
+                        "suggested_next_call": {"tool_name": "app_get", "arguments": {"profile": profile, "app_key": app_key}},
                         "noop": False,
                         "warnings": [
                             _warning(
@@ -1174,7 +1174,7 @@ class AiBuilderFacade:
                     "APP_NOT_FOUND" if api_error.http_status == 404 else "APP_RESOLVE_FAILED",
                     api_error,
                     details={"app_key": app_key},
-                    suggested_next_call={"tool_name": "app_read_summary", "arguments": {"profile": profile, "app_key": app_key}},
+                    suggested_next_call={"tool_name": "app_get", "arguments": {"profile": profile, "app_key": app_key}},
                 )
             result = base.get("result") if isinstance(base.get("result"), dict) else {}
             return {
@@ -1852,7 +1852,7 @@ class AiBuilderFacade:
                             "category": api_error.category,
                         },
                     },
-                    suggested_next_call={"tool_name": "app_read_summary", "arguments": {"profile": profile, "app_key": app_key}},
+                    suggested_next_call={"tool_name": "app_get", "arguments": {"profile": profile, "app_key": app_key}},
                     request_id=api_error.request_id,
                     backend_code=api_error.backend_code,
                     http_status=None if api_error.http_status == 404 else api_error.http_status,
@@ -1890,7 +1890,7 @@ class AiBuilderFacade:
                     "required_permission": required_permission,
                     "permission_summary": permission_summary,
                 },
-                suggested_next_call={"tool_name": "app_read_summary", "arguments": {"profile": profile, "app_key": app_key}},
+                suggested_next_call={"tool_name": "app_get", "arguments": {"profile": profile, "app_key": app_key}},
             )
         )
 
@@ -2003,7 +2003,7 @@ class AiBuilderFacade:
                 "current user does not have builder edit permission on this portal",
                 normalized_args=normalized_args,
                 details={"dash_key": dash_key, "permission_summary": permission_summary},
-                suggested_next_call={"tool_name": "portal_read_summary", "arguments": {"profile": profile, "dash_key": dash_key}},
+                suggested_next_call={"tool_name": "portal_get", "arguments": {"profile": profile, "dash_key": dash_key}},
             )
         )
 
@@ -2066,6 +2066,52 @@ class AiBuilderFacade:
             **response.model_dump(mode="json"),
         }
 
+    def app_get(self, *, profile: str, app_key: str) -> JSONObject:
+        result = self.app_read_summary(profile=profile, app_key=app_key)
+        if result.get("status") != "success":
+            if not result.get("suggested_next_call"):
+                result["suggested_next_call"] = {"tool_name": "app_get", "arguments": {"profile": profile, "app_key": app_key}}
+            return result
+        permission_summary = self._read_app_permission_summary(profile=profile, app_key=app_key)
+        result["message"] = "read app config summary"
+        result["editability"] = {
+            "can_edit_form": permission_summary.get("can_edit_app"),
+            "can_edit_flow": permission_summary.get("can_edit_app"),
+            "can_edit_views": permission_summary.get("can_manage_data"),
+            "can_edit_charts": permission_summary.get("can_manage_data"),
+        }
+        return result
+
+    def app_get_fields(self, *, profile: str, app_key: str) -> JSONObject:
+        result = self.app_read_fields(profile=profile, app_key=app_key)
+        if result.get("status") == "success":
+            result["message"] = "read app field config"
+        return result
+
+    def app_get_layout(self, *, profile: str, app_key: str) -> JSONObject:
+        result = self.app_read_layout_summary(profile=profile, app_key=app_key)
+        if result.get("status") == "success":
+            result["message"] = "read app layout config"
+        return result
+
+    def app_get_views(self, *, profile: str, app_key: str) -> JSONObject:
+        result = self.app_read_views_summary(profile=profile, app_key=app_key)
+        if result.get("status") == "success":
+            result["message"] = "read app view config"
+        return result
+
+    def app_get_flow(self, *, profile: str, app_key: str) -> JSONObject:
+        result = self.app_read_flow_summary(profile=profile, app_key=app_key)
+        if result.get("status") == "success":
+            result["message"] = "read app flow config"
+        return result
+
+    def app_get_charts(self, *, profile: str, app_key: str) -> JSONObject:
+        result = self.app_read_charts_summary(profile=profile, app_key=app_key)
+        if result.get("status") == "success":
+            result["message"] = "read app chart config"
+        return result
+
     def app_read_fields(self, *, profile: str, app_key: str) -> JSONObject:
         try:
             state = self._load_base_schema_state(profile=profile, app_key=app_key)
@@ -2076,7 +2122,7 @@ class AiBuilderFacade:
                 api_error,
                 normalized_args={"app_key": app_key},
                 details={"app_key": app_key},
-                suggested_next_call={"tool_name": "app_read_summary", "arguments": {"profile": profile, "app_key": app_key}},
+                suggested_next_call={"tool_name": "app_get", "arguments": {"profile": profile, "app_key": app_key}},
             )
         parsed = state["parsed"]
         response = AppFieldsReadResponse(
@@ -2112,7 +2158,7 @@ class AiBuilderFacade:
                 api_error,
                 normalized_args={"app_key": app_key},
                 details={"app_key": app_key},
-                suggested_next_call={"tool_name": "app_read_summary", "arguments": {"profile": profile, "app_key": app_key}},
+                suggested_next_call={"tool_name": "app_get", "arguments": {"profile": profile, "app_key": app_key}},
             )
         parsed = state["parsed"]
         layout = parsed["layout"]
@@ -2157,7 +2203,7 @@ class AiBuilderFacade:
                 api_error,
                 normalized_args={"app_key": app_key},
                 details={"app_key": app_key},
-                suggested_next_call={"tool_name": "app_read_summary", "arguments": {"profile": profile, "app_key": app_key}},
+                suggested_next_call={"tool_name": "app_get", "arguments": {"profile": profile, "app_key": app_key}},
             )
         summarized_views, config_read_errors = _summarize_views_with_config(self.views, profile=profile, views=views)
         response = AppViewsReadResponse(
@@ -2211,7 +2257,7 @@ class AiBuilderFacade:
                 api_error,
                 normalized_args={"app_key": app_key},
                 details={"app_key": app_key},
-                suggested_next_call={"tool_name": "app_read_summary", "arguments": {"profile": profile, "app_key": app_key}},
+                suggested_next_call={"tool_name": "app_get", "arguments": {"profile": profile, "app_key": app_key}},
             )
         response = AppFlowReadResponse(
             app_key=app_key,
@@ -2251,7 +2297,7 @@ class AiBuilderFacade:
                 api_error,
                 normalized_args={"app_key": app_key},
                 details={"app_key": app_key},
-                suggested_next_call={"tool_name": "app_read_summary", "arguments": {"profile": profile, "app_key": app_key}},
+                suggested_next_call={"tool_name": "app_get", "arguments": {"profile": profile, "app_key": app_key}},
             )
         charts = _summarize_charts(items)
         response = AppChartsReadResponse(
@@ -2289,13 +2335,54 @@ class AiBuilderFacade:
                 details={},
                 suggested_next_call=None,
             )
-        items = _normalize_portal_list_items(raw_items)
+        warnings: list[dict[str, Any]] = []
+        items: list[dict[str, Any]] = []
+        permission_verified = True
+        for raw_item in raw_items if isinstance(raw_items, list) else []:
+            if not isinstance(raw_item, dict):
+                continue
+            dash_key = str(raw_item.get("dashKey") or "").strip()
+            if not dash_key:
+                continue
+            try:
+                portal_result = self.portals.portal_get(profile=profile, dash_key=dash_key, being_draft=True).get("result") or {}
+            except (QingflowApiError, RuntimeError):
+                permission_verified = False
+                warnings.append(
+                    _warning(
+                        "PORTAL_PERMISSION_READ_UNAVAILABLE",
+                        f"builder portal_list skipped `{dash_key}` because portal detail readback was unavailable during permission verification",
+                        dash_key=dash_key,
+                    )
+                )
+                continue
+            permission_outcome = self._guard_portal_permission(
+                profile=profile,
+                dash_key=dash_key,
+                normalized_args={"dash_key": dash_key},
+                portal_result=portal_result if isinstance(portal_result, dict) else {},
+            )
+            if permission_outcome.block is not None:
+                error_code = str(permission_outcome.block.get("error_code") or "")
+                if error_code != "PORTAL_EDIT_UNAUTHORIZED":
+                    permission_verified = False
+                    warnings.append(
+                        _warning(
+                            "PORTAL_PERMISSION_UNVERIFIED",
+                            f"builder portal_list skipped `{dash_key}` because builder edit permission could not be verified",
+                            dash_key=dash_key,
+                        )
+                    )
+                continue
+            normalized = _normalize_portal_list_items([raw_item])
+            if normalized:
+                items.extend(normalized)
         response = PortalListResponse(items=items, total=len(items))
         return {
             "status": "success",
             "error_code": None,
             "recoverable": False,
-            "message": "list accessible portals",
+            "message": "list builder-configurable portals",
             "normalized_args": {},
             "missing_fields": [],
             "allowed_values": {},
@@ -2303,9 +2390,12 @@ class AiBuilderFacade:
             "request_id": None,
             "suggested_next_call": None,
             "noop": False,
-            "warnings": [],
-            "verification": {"portal_list_loaded": True},
-            "verified": True,
+            "warnings": warnings,
+            "verification": {
+                "portal_list_loaded": True,
+                "portal_permissions_verified": permission_verified,
+            },
+            "verified": permission_verified,
             **response.model_dump(mode="json"),
         }
 
@@ -2380,7 +2470,7 @@ class AiBuilderFacade:
                 api_error,
                 normalized_args={"dash_key": dash_key, "being_draft": being_draft},
                 details={"dash_key": dash_key, "being_draft": being_draft},
-                suggested_next_call={"tool_name": "portal_read_summary", "arguments": {"profile": profile, "dash_key": dash_key, "being_draft": being_draft}},
+                suggested_next_call={"tool_name": "portal_get", "arguments": {"profile": profile, "dash_key": dash_key, "being_draft": being_draft}},
             )
         response = PortalReadSummaryResponse(
             dash_key=dash_key,
@@ -2420,17 +2510,17 @@ class AiBuilderFacade:
             **response.model_dump(mode="json"),
         }
 
-    def view_get(self, *, profile: str, viewgraph_key: str) -> JSONObject:
+    def view_get(self, *, profile: str, view_key: str) -> JSONObject:
         try:
-            config = self.views.view_get_config(profile=profile, viewgraph_key=viewgraph_key).get("result") or {}
+            config = self.views.view_get_config(profile=profile, viewgraph_key=view_key).get("result") or {}
         except (QingflowApiError, RuntimeError) as error:
             api_error = _coerce_api_error(error)
             return _failed_from_api_error(
                 "VIEW_GET_FAILED",
                 api_error,
-                normalized_args={"viewgraph_key": viewgraph_key},
-                details={"viewgraph_key": viewgraph_key},
-                suggested_next_call={"tool_name": "view_get", "arguments": {"profile": profile, "viewgraph_key": viewgraph_key}},
+                normalized_args={"view_key": view_key},
+                details={"view_key": view_key},
+                suggested_next_call={"tool_name": "view_get", "arguments": {"profile": profile, "view_key": view_key}},
             )
 
         warnings: list[dict[str, Any]] = []
@@ -2443,7 +2533,7 @@ class AiBuilderFacade:
 
         base_info: dict[str, Any] = {}
         try:
-            base_info_payload = self.views.view_get_base_info(profile=profile, viewgraph_key=viewgraph_key, passcode=None).get("result") or {}
+            base_info_payload = self.views.view_get_base_info(profile=profile, viewgraph_key=view_key, passcode=None).get("result") or {}
             if isinstance(base_info_payload, dict):
                 base_info = deepcopy(base_info_payload)
         except (QingflowApiError, RuntimeError):
@@ -2452,7 +2542,7 @@ class AiBuilderFacade:
 
         questions: list[dict[str, Any]] = []
         try:
-            questions_payload = self.views.view_list_questions(profile=profile, viewgraph_key=viewgraph_key).get("result") or []
+            questions_payload = self.views.view_list_questions(profile=profile, viewgraph_key=view_key).get("result") or []
             if isinstance(questions_payload, list):
                 questions = [deepcopy(item) for item in questions_payload if isinstance(item, dict)]
         except (QingflowApiError, RuntimeError):
@@ -2461,7 +2551,7 @@ class AiBuilderFacade:
 
         associations: list[dict[str, Any]] = []
         try:
-            associations_payload = self.views.view_list_associations(profile=profile, viewgraph_key=viewgraph_key).get("result") or []
+            associations_payload = self.views.view_list_associations(profile=profile, viewgraph_key=view_key).get("result") or []
             if isinstance(associations_payload, list):
                 associations = [deepcopy(item) for item in associations_payload if isinstance(item, dict)]
         except (QingflowApiError, RuntimeError):
@@ -2469,7 +2559,7 @@ class AiBuilderFacade:
             warnings.append(_warning("VIEW_ASSOCIATIONS_UNAVAILABLE", "view association list readback is unavailable"))
 
         response = ViewGetResponse(
-            viewgraph_key=viewgraph_key,
+            view_key=view_key,
             base_info=base_info,
             config=deepcopy(config) if isinstance(config, dict) else {},
             questions=questions,
@@ -2480,7 +2570,7 @@ class AiBuilderFacade:
             "error_code": None,
             "recoverable": False,
             "message": "read view detail",
-            "normalized_args": {"viewgraph_key": viewgraph_key},
+            "normalized_args": {"view_key": view_key},
             "missing_fields": [],
             "allowed_values": {},
             "details": {},
@@ -2498,69 +2588,72 @@ class AiBuilderFacade:
         *,
         profile: str,
         chart_id: str,
-        data_payload: dict[str, Any] | None = None,
-        page_num: int | None = None,
-        page_size: int | None = None,
-        page_num_y: int | None = None,
-        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_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,
-                payload=normalized_payload,
-                page_num=page_num,
-                page_size=page_size,
-                page_num_y=page_num_y,
-                page_size_y=page_size_y,
-            ).get("result") or {}
         except (QingflowApiError, RuntimeError) as error:
             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,
-                },
+                normalized_args={"chart_id": chart_id},
                 details={"chart_id": chart_id},
                 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:
+            fallback_config: dict[str, Any] | None = None
+            try:
+                data_fallback = self.charts.qingbi_report_get_data(profile=profile, chart_id=chart_id, payload={}).get("result") or {}
+                config_from_data = data_fallback.get("config") if isinstance(data_fallback, dict) else None
+                if isinstance(config_from_data, dict):
+                    fallback_config = deepcopy(config_from_data)
+            except (QingflowApiError, RuntimeError):
+                fallback_config = None
+            if isinstance(fallback_config, dict):
+                config = fallback_config
+                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},
+                    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 {},
             config=deepcopy(config) if isinstance(config, dict) else {},
-            data=deepcopy(data) if isinstance(data, dict) else {"value": data},
         )
         return {
             "status": "success",
             "error_code": None,
             "recoverable": False,
-            "message": "read chart detail",
-            "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,
-            },
+            "message": "read chart config detail",
+            "normalized_args": {"chart_id": chart_id},
             "missing_fields": [],
             "allowed_values": {},
             "details": {},
             "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"),
         }
@@ -2664,7 +2757,7 @@ class AiBuilderFacade:
                     },
                     details={"unknown_selectors": missing_selectors},
                     missing_fields=[str(item) for item in missing_selectors],
-                    suggested_next_call={"tool_name": "app_read_fields", "arguments": {"profile": profile, "app_key": request.app_key}},
+                    suggested_next_call={"tool_name": "app_get_fields", "arguments": {"profile": profile, "app_key": request.app_key}},
                 )
         merged = _merge_layout(
             current_layout={
@@ -2764,7 +2857,7 @@ class AiBuilderFacade:
             elif first_issue.get("kind", "").startswith("member"):
                 suggested_call = {"tool_name": "member_search", "arguments": {"profile": profile, "query": first_issue.get("value") or ""}}
             elif first_issue.get("kind") in {"editable_fields", "condition_fields"}:
-                suggested_call = {"tool_name": "app_read_fields", "arguments": {"profile": profile, "app_key": request.app_key}}
+                suggested_call = {"tool_name": "app_get_fields", "arguments": {"profile": profile, "app_key": request.app_key}}
             return _failed(
                 first_issue.get("error_code") or "FLOW_ASSIGNEE_UNRESOLVED",
                 "workflow contains unresolved assignees or field permissions",
@@ -3152,7 +3245,7 @@ class AiBuilderFacade:
                     normalized_args=normalized_args,
                     allowed_values={"field_types": [item.value for item in PublicFieldType]},
                     details=_with_state_read_blocked_details({"app_key": target.app_key}, resource="schema", error=api_error),
-                    suggested_next_call={"tool_name": "app_read_summary", "arguments": {"profile": profile, "app_key": target.app_key}},
+                    suggested_next_call={"tool_name": "app_get", "arguments": {"profile": profile, "app_key": target.app_key}},
                 ))
             schema_result = _empty_schema_result(target.app_name)
             _schema_source = "synthetic_new_app"
@@ -3177,7 +3270,7 @@ class AiBuilderFacade:
                     f"field '{patch.name}' already exists",
                     normalized_args=normalized_args,
                     details={"field_name": patch.name},
-                    suggested_next_call={"tool_name": "app_read_fields", "arguments": {"profile": profile, "app_key": target.app_key}},
+                    suggested_next_call={"tool_name": "app_get_fields", "arguments": {"profile": profile, "app_key": target.app_key}},
                 )
             current_fields.append(field_dict)
             existing_index[field_dict["field_id"]] = len(current_fields) - 1
@@ -3193,7 +3286,7 @@ class AiBuilderFacade:
                     "field selector did not match any existing field",
                     normalized_args=normalized_args,
                     details={"selector": patch.selector.model_dump(mode="json")},
-                    suggested_next_call={"tool_name": "app_read_fields", "arguments": {"profile": profile, "app_key": target.app_key}},
+                    suggested_next_call={"tool_name": "app_get_fields", "arguments": {"profile": profile, "app_key": target.app_key}},
                 )
             field = current_fields[matched]
             previous_name = field["name"]
@@ -3211,7 +3304,7 @@ class AiBuilderFacade:
                     "remove selector did not match any existing field",
                     normalized_args=normalized_args,
                     details={"selector": patch.model_dump(mode="json")},
-                    suggested_next_call={"tool_name": "app_read_fields", "arguments": {"profile": profile, "app_key": target.app_key}},
+                    suggested_next_call={"tool_name": "app_get_fields", "arguments": {"profile": profile, "app_key": target.app_key}},
                 )
             field = current_fields.pop(matched)
             layout = _remove_field_from_layout(layout, field["name"])
@@ -3234,7 +3327,7 @@ class AiBuilderFacade:
                 api_error,
                 normalized_args=normalized_args,
                 details={"app_key": target.app_key},
-                suggested_next_call={"tool_name": "app_read_fields", "arguments": {"profile": profile, "app_key": target.app_key}},
+                suggested_next_call={"tool_name": "app_get_fields", "arguments": {"profile": profile, "app_key": target.app_key}},
             )
         except ValueError as error:
             return _failed(
@@ -3242,7 +3335,7 @@ class AiBuilderFacade:
                 str(error),
                 normalized_args=normalized_args,
                 details={"app_key": target.app_key},
-                suggested_next_call={"tool_name": "app_read_fields", "arguments": {"profile": profile, "app_key": target.app_key}},
+                suggested_next_call={"tool_name": "app_get_fields", "arguments": {"profile": profile, "app_key": target.app_key}},
             )
 
         try:
@@ -3256,7 +3349,7 @@ class AiBuilderFacade:
                 str(error),
                 normalized_args=normalized_args,
                 details={"app_key": target.app_key},
-                suggested_next_call={"tool_name": "app_read_fields", "arguments": {"profile": profile, "app_key": target.app_key}},
+                suggested_next_call={"tool_name": "app_get_fields", "arguments": {"profile": profile, "app_key": target.app_key}},
             )
 
         q_linker_schema_context = deepcopy(schema_result)
@@ -3272,7 +3365,7 @@ class AiBuilderFacade:
                 str(error),
                 normalized_args=normalized_args,
                 details={"app_key": target.app_key},
-                suggested_next_call={"tool_name": "app_read_fields", "arguments": {"profile": profile, "app_key": target.app_key}},
+                suggested_next_call={"tool_name": "app_get_fields", "arguments": {"profile": profile, "app_key": target.app_key}},
             )
 
         relation_field_count = _count_relation_fields(current_fields)
@@ -3351,7 +3444,7 @@ class AiBuilderFacade:
                     request_id=api_error.request_id,
                     backend_code=api_error.backend_code,
                     http_status=None if api_error.http_status == 404 else api_error.http_status,
-                    suggested_next_call={"tool_name": "app_read_fields", "arguments": {"profile": profile, "app_key": target.app_key}},
+                    suggested_next_call={"tool_name": "app_get_fields", "arguments": {"profile": profile, "app_key": target.app_key}},
                 )
             return _failed_from_api_error(
                 "SCHEMA_APPLY_FAILED",
@@ -3362,7 +3455,7 @@ class AiBuilderFacade:
                     "app_key": target.app_key,
                     "field_diff": {"added": added, "updated": updated, "removed": removed},
                 },
-                suggested_next_call={"tool_name": "app_read_fields", "arguments": {"profile": profile, "app_key": target.app_key}},
+                suggested_next_call={"tool_name": "app_get_fields", "arguments": {"profile": profile, "app_key": target.app_key}},
             )
         if _code_block_relations_need_source_rebind(compiled_question_relations) or _q_linker_relations_need_source_rebind(compiled_question_relations):
             try:
@@ -3396,7 +3489,7 @@ class AiBuilderFacade:
                     str(error),
                     normalized_args=normalized_args,
                     details={"app_key": target.app_key},
-                    suggested_next_call={"tool_name": "app_read_fields", "arguments": {"profile": profile, "app_key": target.app_key}},
+                    suggested_next_call={"tool_name": "app_get_fields", "arguments": {"profile": profile, "app_key": target.app_key}},
                 )
             except (QingflowApiError, RuntimeError) as error:
                 api_error = _coerce_api_error(error)
@@ -3409,7 +3502,7 @@ class AiBuilderFacade:
                         "app_key": target.app_key,
                         "field_diff": {"added": added, "updated": updated, "removed": removed},
                     },
-                    suggested_next_call={"tool_name": "app_read_fields", "arguments": {"profile": profile, "app_key": target.app_key}},
+                    suggested_next_call={"tool_name": "app_get_fields", "arguments": {"profile": profile, "app_key": target.app_key}},
                 )
             rebound_payload = _build_form_payload_from_fields(
                 title=rebound_schema.get("formTitle") or target.app_name,
@@ -3436,7 +3529,7 @@ class AiBuilderFacade:
                         "app_key": target.app_key,
                         "field_diff": {"added": added, "updated": updated, "removed": removed},
                     },
-                    suggested_next_call={"tool_name": "app_read_fields", "arguments": {"profile": profile, "app_key": target.app_key}},
+                    suggested_next_call={"tool_name": "app_get_fields", "arguments": {"profile": profile, "app_key": target.app_key}},
                 )
             current_fields = rebound_fields
         response = {
@@ -3588,7 +3681,7 @@ class AiBuilderFacade:
                 api_error,
                 normalized_args=normalized_args,
                 details=_with_state_read_blocked_details({"app_key": app_key}, resource="schema", error=api_error),
-                suggested_next_call={"tool_name": "app_read_layout_summary", "arguments": {"profile": profile, "app_key": app_key}},
+                suggested_next_call={"tool_name": "app_get_layout", "arguments": {"profile": profile, "app_key": app_key}},
             ))
         parsed = _parse_schema(schema_result)
         current_fields = parsed["fields"]
@@ -3601,7 +3694,7 @@ class AiBuilderFacade:
                 normalized_args=normalized_args,
                 details={"unknown_selectors": missing_selectors},
                 missing_fields=[str(item) for item in missing_selectors],
-                suggested_next_call={"tool_name": "app_read_fields", "arguments": {"profile": profile, "app_key": app_key}},
+                suggested_next_call={"tool_name": "app_get_fields", "arguments": {"profile": profile, "app_key": app_key}},
             )
         fields_by_name = {field["name"]: field for field in current_fields}
         seen: list[str] = []
@@ -3614,7 +3707,7 @@ class AiBuilderFacade:
                             f"layout references unknown field '{field_name}'",
                             normalized_args=normalized_args,
                             details={"field_name": field_name},
-                            suggested_next_call={"tool_name": "app_read_layout_summary", "arguments": {"profile": profile, "app_key": app_key}},
+                            suggested_next_call={"tool_name": "app_get_layout", "arguments": {"profile": profile, "app_key": app_key}},
                         )
                     if field_name in seen:
                         return _failed(
@@ -3755,7 +3848,7 @@ class AiBuilderFacade:
                 "allowed_values": {"modes": ["merge", "replace"]},
                 "details": {},
                 "request_id": None,
-                "suggested_next_call": {"tool_name": "app_read_layout_summary", "arguments": {"profile": profile, "app_key": app_key}},
+                "suggested_next_call": {"tool_name": "app_get_layout", "arguments": {"profile": profile, "app_key": app_key}},
                 "noop": False,
                 "warnings": [],
                 "verification": {"layout_verified": False, "layout_summary_verified": False, "layout_read_unavailable": True},
@@ -3876,7 +3969,7 @@ class AiBuilderFacade:
                 api_error,
                 normalized_args=normalized_args,
                 details=_with_state_read_blocked_details({"app_key": app_key}, resource="workflow", error=api_error),
-                suggested_next_call={"tool_name": "app_read_flow_summary", "arguments": {"profile": profile, "app_key": app_key}},
+                suggested_next_call={"tool_name": "app_get_flow", "arguments": {"profile": profile, "app_key": app_key}},
             ))
         entity = _entity_spec_from_app(base_info=base, schema=schema, views=None)
         current_fields = _parse_schema(schema)["fields"]
@@ -3904,7 +3997,7 @@ class AiBuilderFacade:
             elif first_issue.get("kind", "").startswith("member"):
                 suggested_call = {"tool_name": "member_search", "arguments": {"profile": profile, "query": first_issue.get("value") or ""}}
             elif first_issue.get("kind") in {"editable_fields", "condition_fields"}:
-                suggested_call = {"tool_name": "app_read_fields", "arguments": {"profile": profile, "app_key": app_key}}
+                suggested_call = {"tool_name": "app_get_fields", "arguments": {"profile": profile, "app_key": app_key}}
             return _failed(
                 first_issue.get("error_code") or "FLOW_ASSIGNEE_UNRESOLVED",
                 "workflow contains unresolved assignees or field permissions",
@@ -4006,7 +4099,7 @@ class AiBuilderFacade:
             error_code = "FLOW_READBACK_PENDING"
             recoverable = True
             message = "applied workflow patch; flow readback pending"
-            suggested_next_call = {"tool_name": "app_read_flow_summary", "arguments": {"profile": profile, "app_key": app_key}}
+            suggested_next_call = {"tool_name": "app_get_flow", "arguments": {"profile": profile, "app_key": app_key}}
         elif workflow_verified:
             status = "success"
             error_code = None
@@ -4022,7 +4115,7 @@ class AiBuilderFacade:
                 if workflow_structure_verified and not branch_structure_verified
                 else "applied workflow patch; flow readback did not confirm the requested workflow"
             )
-            suggested_next_call = {"tool_name": "app_read_flow_summary", "arguments": {"profile": profile, "app_key": app_key}}
+            suggested_next_call = {"tool_name": "app_get_flow", "arguments": {"profile": profile, "app_key": app_key}}
             if not branch_structure_verified:
                 warnings.append(_warning("WORKFLOW_BRANCH_STRUCTURE_UNVERIFIED", "branch or condition structure was written, but MCP could not fully verify downstream lane structure"))
         response = {
@@ -4110,7 +4203,7 @@ class AiBuilderFacade:
                 api_error,
                 normalized_args=normalized_args,
                 details=_with_state_read_blocked_details({"app_key": app_key}, resource="views", error=api_error),
-                suggested_next_call={"tool_name": "app_read_views_summary", "arguments": {"profile": profile, "app_key": app_key}},
+                suggested_next_call={"tool_name": "app_get_views", "arguments": {"profile": profile, "app_key": app_key}},
             ))
         existing_views = existing_views or []
         existing_by_key: dict[str, dict[str, Any]] = {}
@@ -4187,7 +4280,7 @@ class AiBuilderFacade:
             if len(matches) > 1:
                 return _failed(
                     "AMBIGUOUS_VIEW",
-                    "multiple views matched remove request; use app_read_views_summary and resolve duplicates before removing by name",
+                    "multiple views matched remove request; use app_get_views and resolve duplicates before removing by name",
                     normalized_args=normalized_args,
                     details={
                         "app_key": app_key,
@@ -4197,7 +4290,7 @@ class AiBuilderFacade:
                             for view in matches
                         ],
                     },
-                    suggested_next_call={"tool_name": "app_read_views_summary", "arguments": {"profile": profile, "app_key": app_key}},
+                    suggested_next_call={"tool_name": "app_get_views", "arguments": {"profile": profile, "app_key": app_key}},
                 )
             if len(matches) == 1:
                 key = _extract_view_key(matches[0])
@@ -4246,7 +4339,7 @@ class AiBuilderFacade:
                         "ignored_system_columns": ignored_system_columns,
                     },
                     missing_fields=missing_columns,
-                    suggested_next_call={"tool_name": "app_read_fields", "arguments": {"profile": profile, "app_key": app_key}},
+                    suggested_next_call={"tool_name": "app_get_fields", "arguments": {"profile": profile, "app_key": app_key}},
                 )
             if patch.group_by and patch.group_by not in field_names:
                 return _failed(
@@ -4259,7 +4352,7 @@ class AiBuilderFacade:
                         "missing_fields": [patch.group_by],
                     },
                     missing_fields=[patch.group_by],
-                    suggested_next_call={"tool_name": "app_read_fields", "arguments": {"profile": profile, "app_key": app_key}},
+                    suggested_next_call={"tool_name": "app_get_fields", "arguments": {"profile": profile, "app_key": app_key}},
                 )
             for gantt_field_name in (patch.start_field, patch.end_field, patch.title_field):
                 if gantt_field_name and gantt_field_name not in field_names:
@@ -4273,7 +4366,7 @@ class AiBuilderFacade:
                             "missing_fields": [gantt_field_name],
                         },
                         missing_fields=[gantt_field_name],
-                        suggested_next_call={"tool_name": "app_read_fields", "arguments": {"profile": profile, "app_key": app_key}},
+                        suggested_next_call={"tool_name": "app_get_fields", "arguments": {"profile": profile, "app_key": app_key}},
                     )
             translated_filters, filter_issues = _build_view_filter_groups(current_fields_by_name=current_fields_by_name, filters=patch.filters)
             if filter_issues:
@@ -4289,7 +4382,7 @@ class AiBuilderFacade:
                     },
                     missing_fields=list(first_issue.get("missing_fields") or []),
                     allowed_values=first_issue.get("allowed_values") or {"view_types": [member.value for member in PublicViewType], "view.filter.operator": [member.value for member in ViewFilterOperator]},
-                    suggested_next_call={"tool_name": "app_read_fields", "arguments": {"profile": profile, "app_key": app_key}},
+                    suggested_next_call={"tool_name": "app_get_fields", "arguments": {"profile": profile, "app_key": app_key}},
                 )
             explicit_button_dtos: list[dict[str, Any]] | None = None
             expected_button_summary: list[dict[str, Any]] | None = None
@@ -4328,7 +4421,7 @@ class AiBuilderFacade:
                         f"view_key '{patch.view_key}' does not exist on this app",
                         normalized_args=normalized_args,
                         details={"app_key": app_key, "view_key": patch.view_key, "view_name": patch.name},
-                        suggested_next_call={"tool_name": "app_read_views_summary", "arguments": {"profile": profile, "app_key": app_key}},
+                        suggested_next_call={"tool_name": "app_get_views", "arguments": {"profile": profile, "app_key": app_key}},
                     )
                 existing_key = patch.view_key
             else:
@@ -4346,7 +4439,7 @@ class AiBuilderFacade:
                                 for view in name_matches
                             ],
                         },
-                        suggested_next_call={"tool_name": "app_read_views_summary", "arguments": {"profile": profile, "app_key": app_key}},
+                        suggested_next_call={"tool_name": "app_get_views", "arguments": {"profile": profile, "app_key": app_key}},
                     )
                 if len(name_matches) == 1:
                     matched_existing_view = name_matches[0]
@@ -4638,7 +4731,7 @@ class AiBuilderFacade:
                 api_error,
                 normalized_args=normalized_args,
                 details=_with_state_read_blocked_details({"app_key": app_key}, resource="views", error=api_error),
-                suggested_next_call={"tool_name": "app_read_views_summary", "arguments": {"profile": profile, "app_key": app_key}},
+                suggested_next_call={"tool_name": "app_get_views", "arguments": {"profile": profile, "app_key": app_key}},
             ))
         verified_names = {
             _extract_view_name(item)
@@ -4858,7 +4951,7 @@ class AiBuilderFacade:
                     ),
                 },
                 "request_id": first_failure.get("request_id"),
-                "suggested_next_call": {"tool_name": "app_read_views_summary", "arguments": {"profile": profile, "app_key": app_key}},
+                "suggested_next_call": {"tool_name": "app_get_views", "arguments": {"profile": profile, "app_key": app_key}},
                 "backend_code": first_failure.get("backend_code"),
                 "http_status": first_failure.get("http_status"),
                 "noop": noop,
@@ -4931,7 +5024,7 @@ class AiBuilderFacade:
                 ),
             },
             "request_id": None,
-            "suggested_next_call": None if verified and view_filters_verified and view_buttons_verified else {"tool_name": "app_read_views_summary", "arguments": {"profile": profile, "app_key": app_key}},
+            "suggested_next_call": None if verified and view_filters_verified and view_buttons_verified else {"tool_name": "app_get_views", "arguments": {"profile": profile, "app_key": app_key}},
             "noop": noop,
             "warnings": warnings,
             "verification": {
@@ -4968,7 +5061,7 @@ class AiBuilderFacade:
                 api_error,
                 normalized_args=normalized_args,
                 details={"app_key": app_key},
-                suggested_next_call={"tool_name": "app_read_summary", "arguments": {"profile": profile, "app_key": app_key}},
+                suggested_next_call={"tool_name": "app_get", "arguments": {"profile": profile, "app_key": app_key}},
             )
         tag_ids_before = _coerce_int_list(base_before.get("tagIds"))
         already_published = bool(base_before.get("appPublishStatus") in {1, 2})
@@ -4982,7 +5075,7 @@ class AiBuilderFacade:
                 api_error,
                 normalized_args=normalized_args,
                 details={"app_key": app_key},
-                suggested_next_call={"tool_name": "app_read_views_summary", "arguments": {"profile": profile, "app_key": app_key}},
+                suggested_next_call={"tool_name": "app_get_views", "arguments": {"profile": profile, "app_key": app_key}},
             )
         views_before = views_before or []
         if already_published and package_already_attached is not False and isinstance(views_before, list) and not views_before_unavailable:
@@ -5019,7 +5112,7 @@ class AiBuilderFacade:
                 api_error,
                 normalized_args=normalized_args,
                 details={"app_key": app_key, "edit_version_no": edit_version_no},
-                suggested_next_call={"tool_name": "app_read_summary", "arguments": {"profile": profile, "app_key": app_key}},
+                suggested_next_call={"tool_name": "app_get", "arguments": {"profile": profile, "app_key": app_key}},
             )
         try:
             base = self.apps.app_get_base(profile=profile, app_key=app_key, include_raw=True).get("result") or {}
@@ -5030,7 +5123,7 @@ class AiBuilderFacade:
                 api_error,
                 normalized_args=normalized_args,
                 details={"app_key": app_key},
-                suggested_next_call={"tool_name": "app_read_summary", "arguments": {"profile": profile, "app_key": app_key}},
+                suggested_next_call={"tool_name": "app_get", "arguments": {"profile": profile, "app_key": app_key}},
             )
         tag_ids_after = _coerce_int_list(base.get("tagIds"))
         package_attached = None if not expected_package_tag_id else expected_package_tag_id in tag_ids_after
@@ -5043,7 +5136,7 @@ class AiBuilderFacade:
                 api_error,
                 normalized_args=normalized_args,
                 details={"app_key": app_key},
-                suggested_next_call={"tool_name": "app_read_views_summary", "arguments": {"profile": profile, "app_key": app_key}},
+                suggested_next_call={"tool_name": "app_get_views", "arguments": {"profile": profile, "app_key": app_key}},
             )
         views = views or []
         views_ok = isinstance(views, list) and not views_unavailable
@@ -5706,7 +5799,7 @@ class AiBuilderFacade:
                 api_error,
                 normalized_args=normalized_args,
                 details={"app_key": app_key, "phase": "prepare_edit_context"},
-                suggested_next_call={"tool_name": "app_read_summary", "arguments": {"profile": profile, "app_key": app_key}},
+                suggested_next_call={"tool_name": "app_get", "arguments": {"profile": profile, "app_key": app_key}},
             )
         edit_version_no = _coerce_positive_int(version_result.get("editVersionNo") or version_result.get("versionNo")) or 1
         return edit_version_no, None
@@ -5987,7 +6080,7 @@ class AiBuilderFacade:
                     "APP_CREATE_READBACK_FAILED",
                     api_error,
                     details={"app_key": new_app_key, "app_name": app_name, "package_tag_id": package_tag_id},
-                    suggested_next_call={"tool_name": "app_read_summary", "arguments": {"profile": profile, "app_key": new_app_key}},
+                    suggested_next_call={"tool_name": "app_get", "arguments": {"profile": profile, "app_key": new_app_key}},
                 )
             return {
                 "status": "success",
@@ -6060,7 +6153,7 @@ class AiBuilderFacade:
                 api_error,
                 normalized_args=normalized_args,
                 details={"app_key": app_key},
-                suggested_next_call={"tool_name": "app_read_summary", "arguments": {"profile": profile, "app_key": app_key}},
+                suggested_next_call={"tool_name": "app_get", "arguments": {"profile": profile, "app_key": app_key}},
             )
         raw_base = base_result.get("result") if isinstance(base_result.get("result"), dict) else {}
         effective_title = str(raw_base.get("formTitle") or fallback_title or "未命名应用").strip() or "未命名应用"
@@ -6086,7 +6179,7 @@ class AiBuilderFacade:
                 "app base info did not include editable auth payload required for icon update",
                 normalized_args=normalized_args,
                 details={"app_key": app_key},
-                suggested_next_call={"tool_name": "app_read_summary", "arguments": {"profile": profile, "app_key": app_key}},
+                suggested_next_call={"tool_name": "app_get", "arguments": {"profile": profile, "app_key": app_key}},
             )
         try:
             update_result = self.apps.app_update_base(profile=profile, app_key=app_key, payload=payload)
@@ -6097,7 +6190,7 @@ class AiBuilderFacade:
                 api_error,
                 normalized_args=normalized_args,
                 details={"app_key": app_key, "app_icon": desired_icon},
-                suggested_next_call={"tool_name": "app_read_summary", "arguments": {"profile": profile, "app_key": app_key}},
+                suggested_next_call={"tool_name": "app_get", "arguments": {"profile": profile, "app_key": app_key}},
             )
         return {
             "status": "success",
@@ -8075,11 +8168,15 @@ def _parse_schema(schema: dict[str, Any]) -> dict[str, Any]:
                     section_rows.append(labels)
             if section_rows:
                 parsed_section_id = _coerce_positive_int(section_question.get("queId"))
+                raw_section_id = None
                 if parsed_section_id is None:
-                    parsed_section_id = _coerce_positive_int(section_question.get("sectionId"))
+                    raw_section_id = str(section_question.get("sectionId") or "").strip() or None
+                    if raw_section_id is None:
+                        parsed_section_id = _coerce_positive_int(section_question.get("sectionId"))
                 sections.append(
                     {
-                        "section_id": str(parsed_section_id or _slugify(section_question.get("queTitle") or "section", default="section")),
+                        "section_id": raw_section_id
+                        or str(parsed_section_id or _slugify(section_question.get("queTitle") or "section", default="section")),
                         "title": section_question.get("queTitle") or "未命名分组",
                         "rows": section_rows,
                     }