@josephyan/qingflow-app-builder-mcp 0.2.0-beta.37 → 0.2.0-beta.39

package/skills/qingflow-app-builder/references/update-views.md

@@ -4,12 +4,11 @@ Use this when the task is only about table, card, board, or gantt views.
 
 ## Minimal sequence
 
-1. `builder_tool_contract(tool_name="app_views_plan")`
+1. `builder_tool_contract(tool_name="app_views_apply")`
 2. `app_read_fields`
 3. `app_read_views_summary`
-4. `app_views_plan`
-5. `app_views_apply`
-6. `app_read_views_summary` again whenever apply returns `failed` or `partial_success`
+4. `app_views_apply`
+5. `app_read_views_summary` again whenever apply returns `failed` or `partial_success`
 
 If you are unsure about keys or view types, call `builder_tool_contract(tool_name="app_views_apply")` before guessing.
 
@@ -30,28 +29,7 @@ Canonical rules before any example:
 - For gantt, use `start_field`, `end_field`, and optionally `title_field`
 - If `app_read_views_summary` shows duplicate view names, include `view_key` in `upsert_views[]` and update that exact target
 
-Plan a default table view:
-
-```json
-{
-  "tool_name": "app_views_plan",
-  "arguments": {
-    "profile": "default",
-    "app_key": "APP_123",
-    "upsert_views": [
-      {
-        "name": "全部订单",
-        "view_key": "VIEW_KEY_IF_DUPLICATE_NAMES_EXIST",
-        "type": "table",
-        "columns": ["订单编号", "客户名称", "订单金额", "状态"]
-      }
-    ],
-    "remove_views": []
-  }
-}
-```
-
-Apply it:
+Apply a default table view:
 
 ```json
 {
@@ -63,6 +41,7 @@ Apply it:
     "upsert_views": [
       {
         "name": "全部订单",
+        "view_key": "VIEW_KEY_IF_DUPLICATE_NAMES_EXIST",
         "type": "table",
         "columns": ["订单编号", "客户名称", "订单金额", "状态"]
       }
@@ -71,8 +50,7 @@ Apply it:
   }
 }
 ```
-
-After `app_views_plan` succeeds, prefer reusing its `suggested_next_call.arguments` directly. Do not rewrite aliases back into non-canonical keys such as `column_names`.
+After `app_views_apply` returns canonical arguments or blocking issues, prefer reusing its `suggested_next_call.arguments` directly. Do not rewrite aliases back into non-canonical keys such as `column_names`.
 
 Board example:
 

package/src/qingflow_mcp/__init__.py

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

package/src/qingflow_mcp/builder_facade/models.py

@@ -69,6 +69,14 @@ class PublicViewType(str, Enum):
     gantt = "gantt"
 
 
+class PublicChartType(str, Enum):
+    target = "target"
+    pie = "pie"
+    bar = "bar"
+    line = "line"
+    table = "table"
+
+
 class LayoutApplyMode(str, Enum):
     merge = "merge"
     replace = "replace"
@@ -491,6 +499,261 @@ class ViewUpsertPatch(StrictModel):
         return self
 
 
+class ChartFilterRulePatch(StrictModel):
+    field_name: str = Field(validation_alias=AliasChoices("field_name", "fieldName", "field", "name"))
+    operator: ViewFilterOperator = Field(validation_alias=AliasChoices("operator", "op"))
+    values: list[Any] = Field(default_factory=list)
+
+    @model_validator(mode="before")
+    @classmethod
+    def normalize_aliases(cls, value: Any) -> Any:
+        if not isinstance(value, dict):
+            return value
+        payload = dict(value)
+        if "value" in payload and "values" not in payload:
+            payload["values"] = [payload.pop("value")]
+        raw_operator = payload.get("operator", payload.get("op"))
+        if isinstance(raw_operator, str):
+            normalized = raw_operator.strip().lower()
+            operator_aliases = {
+                "equals": ViewFilterOperator.eq.value,
+                "equal": ViewFilterOperator.eq.value,
+                "=": ViewFilterOperator.eq.value,
+                "not_equals": ViewFilterOperator.neq.value,
+                "not_equal": ViewFilterOperator.neq.value,
+                "!=": ViewFilterOperator.neq.value,
+                ">=": ViewFilterOperator.gte.value,
+                "<=": ViewFilterOperator.lte.value,
+                "any_of": ViewFilterOperator.in_.value,
+                "one_of": ViewFilterOperator.in_.value,
+                "between_any": ViewFilterOperator.in_.value,
+                "empty": ViewFilterOperator.is_empty.value,
+                "is blank": ViewFilterOperator.is_empty.value,
+                "blank": ViewFilterOperator.is_empty.value,
+                "not_empty": ViewFilterOperator.not_empty.value,
+                "not blank": ViewFilterOperator.not_empty.value,
+            }
+            if normalized in operator_aliases:
+                payload["operator"] = operator_aliases[normalized]
+            elif "operator" not in payload:
+                payload["operator"] = normalized
+        payload.pop("op", None)
+        return payload
+
+    @model_validator(mode="after")
+    def validate_shape(self) -> "ChartFilterRulePatch":
+        if self.operator in {ViewFilterOperator.is_empty, ViewFilterOperator.not_empty}:
+            self.values = []
+            return self
+        if not self.values:
+            raise ValueError("chart filter rule requires values")
+        return self
+
+
+class ChartUpsertPatch(StrictModel):
+    chart_id: str | None = None
+    name: str
+    chart_type: PublicChartType
+    dimension_field_ids: list[str] = Field(default_factory=list)
+    indicator_field_ids: list[str] = Field(default_factory=list)
+    filters: list[ChartFilterRulePatch] = Field(default_factory=list)
+    question_config: list[dict[str, Any]] = Field(default_factory=list)
+    user_config: list[dict[str, Any]] = Field(default_factory=list)
+    config: dict[str, Any] = Field(default_factory=dict)
+
+    @model_validator(mode="before")
+    @classmethod
+    def normalize_aliases(cls, value: Any) -> Any:
+        if not isinstance(value, dict):
+            return value
+        payload = dict(value)
+        if "id" in payload and "chart_id" not in payload:
+            payload["chart_id"] = payload.pop("id")
+        if "type" in payload and "chart_type" not in payload:
+            payload["chart_type"] = payload.pop("type")
+        if "dimension_fields" in payload and "dimension_field_ids" not in payload:
+            payload["dimension_field_ids"] = payload.pop("dimension_fields")
+        if "indicator_fields" in payload and "indicator_field_ids" not in payload:
+            payload["indicator_field_ids"] = payload.pop("indicator_fields")
+        if "metric_field_ids" in payload and "indicator_field_ids" not in payload:
+            payload["indicator_field_ids"] = payload.pop("metric_field_ids")
+        raw_type = payload.get("chart_type")
+        if isinstance(raw_type, str):
+            normalized = raw_type.strip().lower()
+            aliases = {
+                "targetchart": PublicChartType.target.value,
+                "piechart": PublicChartType.pie.value,
+                "barchart": PublicChartType.bar.value,
+                "linechart": PublicChartType.line.value,
+                "tablechart": PublicChartType.table.value,
+            }
+            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
+
+
+class ChartApplyRequest(StrictModel):
+    app_key: str
+    upsert_charts: list[ChartUpsertPatch] = Field(default_factory=list)
+    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:
+            raise ValueError("chart apply requires at least one upsert, remove, or reorder operation")
+        return self
+
+
+class PortalComponentPositionPatch(StrictModel):
+    pc_x: int = Field(default=0, validation_alias=AliasChoices("pc_x", "pcX", "x"))
+    pc_y: int = Field(default=0, validation_alias=AliasChoices("pc_y", "pcY", "y"))
+    pc_w: int = Field(default=12, validation_alias=AliasChoices("pc_w", "pcW", "w"))
+    pc_h: int = Field(default=8, validation_alias=AliasChoices("pc_h", "pcH", "h"))
+    mobile_x: int = Field(default=0, validation_alias=AliasChoices("mobile_x", "mobileX"))
+    mobile_y: int = Field(default=0, validation_alias=AliasChoices("mobile_y", "mobileY"))
+    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
+    chart_id: str | None = None
+    chart_name: str | None = None
+
+    @model_validator(mode="after")
+    def validate_target(self) -> "PortalChartRefPatch":
+        if not (self.chart_id or self.chart_name):
+            raise ValueError("chart_ref requires chart_id or chart_name")
+        return self
+
+
+class PortalViewRefPatch(StrictModel):
+    app_key: str
+    view_key: str | None = None
+    view_name: str | None = None
+
+    @model_validator(mode="after")
+    def validate_target(self) -> "PortalViewRefPatch":
+        if not (self.view_key or self.view_name):
+            raise ValueError("view_ref requires view_key or view_name")
+        return self
+
+
+class PortalSectionPatch(StrictModel):
+    title: str
+    source_type: str = Field(validation_alias=AliasChoices("source_type", "sourceType"))
+    position: PortalComponentPositionPatch | None = None
+    dash_style_config: dict[str, Any] | None = Field(default=None, validation_alias=AliasChoices("dash_style_config", "dashStyleConfigBO"))
+    config: dict[str, Any] = Field(default_factory=dict)
+    chart_ref: PortalChartRefPatch | None = None
+    view_ref: PortalViewRefPatch | None = None
+    text: str | None = None
+    url: str | None = None
+
+    @model_validator(mode="before")
+    @classmethod
+    def normalize_aliases(cls, value: Any) -> Any:
+        if not isinstance(value, dict):
+            return value
+        payload = dict(value)
+        raw_type = payload.get("source_type", payload.get("sourceType"))
+        if isinstance(raw_type, str):
+            payload["source_type"] = raw_type.strip().lower()
+        if "chartRef" in payload and "chart_ref" not in payload:
+            payload["chart_ref"] = payload.pop("chartRef")
+        if "viewRef" in payload and "view_ref" not in payload:
+            payload["view_ref"] = payload.pop("viewRef")
+        if "dashStyleConfigBO" in payload and "dash_style_config" not in payload:
+            payload["dash_style_config"] = payload.pop("dashStyleConfigBO")
+        return payload
+
+    @model_validator(mode="after")
+    def validate_shape(self) -> "PortalSectionPatch":
+        supported = {"chart", "view", "grid", "filter", "text", "link"}
+        if self.source_type not in supported:
+            raise ValueError(f"unsupported portal source_type '{self.source_type}'")
+        if self.source_type == "chart" and self.chart_ref is None:
+            raise ValueError("chart section requires chart_ref")
+        if self.source_type == "view" and self.view_ref is None:
+            raise ValueError("view section requires view_ref")
+        if self.source_type == "text" and self.text is None:
+            raise ValueError("text section requires text")
+        if self.source_type == "link" and self.url is None:
+            raise ValueError("link section requires url")
+        return self
+
+
+class PortalApplyRequest(StrictModel):
+    dash_key: str | None = None
+    dash_name: str | None = None
+    package_tag_id: int | None = None
+    publish: bool = True
+    sections: list[PortalSectionPatch] = Field(default_factory=list)
+    auth: dict[str, Any] | None = None
+    icon: str | None = None
+    color: str | None = None
+    hide_copyright: bool | None = Field(default=None, validation_alias=AliasChoices("hide_copyright", "hideCopyright"))
+    dash_global_config: dict[str, Any] | None = Field(default=None, validation_alias=AliasChoices("dash_global_config", "dashGlobalConfig"))
+    config: dict[str, Any] = Field(default_factory=dict)
+
+    @model_validator(mode="after")
+    def validate_shape(self) -> "PortalApplyRequest":
+        if not self.dash_key and not self.package_tag_id:
+            raise ValueError("package_tag_id is required when dash_key is empty")
+        if not self.dash_key and not self.dash_name:
+            raise ValueError("dash_name is required when creating a portal")
+        if not self.sections:
+            raise ValueError("portal apply requires a non-empty sections list")
+        return self
+
+
 FieldPatch.model_rebuild()