@josephyan/qingflow-app-user-mcp 1.1.18 → 1.1.19

package/README.md

@@ -3,13 +3,13 @@
 Install:
 
 ```bash
-npm install @josephyan/qingflow-app-user-mcp@1.1.18
+npm install @josephyan/qingflow-app-user-mcp@1.1.19
 ```
 
 Run:
 
 ```bash
-npx -y -p @josephyan/qingflow-app-user-mcp@1.1.18 qingflow-app-user-mcp
+npx -y -p @josephyan/qingflow-app-user-mcp@1.1.19 qingflow-app-user-mcp
 ```
 
 Environment:

package/package.json

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

package/src/qingflow_mcp/builder_facade/service.py

@@ -3155,7 +3155,7 @@ class AiBuilderFacade:
         return expanded, issues, results
 
     def app_custom_buttons_apply(self, *, profile: str, request: CustomButtonsApplyRequest) -> JSONObject:
-        normalized_args = request.model_dump(mode="json")
+        normalized_args = request.model_dump(mode="json", exclude_none=True, exclude_defaults=True)
         app_key = request.app_key
         permission_outcomes: list[PermissionCheckOutcome] = []
         button_write_intent = bool(request.upsert_buttons or request.patch_buttons or request.remove_buttons)
@@ -3239,7 +3239,10 @@ class AiBuilderFacade:
                     )
                 )
             upsert_buttons.extend(expanded_buttons)
-            normalized_args["upsert_buttons"] = [patch.model_dump(mode="json") for patch in upsert_buttons]
+            normalized_args["upsert_buttons"] = [
+                patch.model_dump(mode="json", exclude_none=True, exclude_defaults=True)
+                for patch in upsert_buttons
+            ]
             normalized_args["patch_results"] = patch_results
 
         compiled_add_data_configs, add_data_issues = self._compile_custom_button_semantic_add_data_configs(
@@ -10261,8 +10264,14 @@ class AiBuilderFacade:
             return CustomButtonsApplyRequest.model_validate(
                 {
                     "app_key": app_key,
-                    "upsert_buttons": [item.model_dump(mode="json") for item in upsert_buttons],
-                    "view_configs": [item.model_dump(mode="json") for item in view_configs],
+                    "upsert_buttons": [
+                        item.model_dump(mode="json", exclude_none=True, exclude_defaults=True)
+                        for item in upsert_buttons
+                    ],
+                    "view_configs": [
+                        item.model_dump(mode="json", exclude_none=True, exclude_defaults=True)
+                        for item in view_configs
+                    ],
                 }
             ), []
         except Exception as error:

package/src/qingflow_mcp/tools/ai_builder_tools.py

@@ -220,9 +220,10 @@ class AiBuilderTools(ToolBase):
                 wrapped["error"] = error
             app_results.append(wrapped)
 
+        partial = sum(1 for item in app_results if str(item.get("status") or "") == "partial_success")
         succeeded = sum(1 for item in app_results if str(item.get("status") or "") in {"success", "partial_success"})
         failed = len(app_results) - succeeded
-        status = "success" if failed == 0 else "failed" if succeeded == 0 else "partial_success"
+        status = "failed" if succeeded == 0 else "partial_success" if failed > 0 or partial > 0 else "success"
         write_executed = any(
             isinstance(item.get("result"), dict) and bool(item["result"].get("write_executed"))
             for item in app_results
@@ -230,7 +231,7 @@ class AiBuilderTools(ToolBase):
         payload: JSONObject = {
             "status": status,
             "error_code": None if status == "success" else f"{tool_name.upper()}_BATCH_PARTIAL" if succeeded else f"{tool_name.upper()}_BATCH_FAILED",
-            "recoverable": failed > 0,
+            "recoverable": failed > 0 or partial > 0,
             "message": (
                 f"applied {tool_name} to {succeeded}/{len(app_results)} apps"
                 if status != "success"
@@ -240,12 +241,13 @@ class AiBuilderTools(ToolBase):
             "apps": app_results,
             "errors": errors,
             "verification": {
-                "batch_verified": failed == 0,
+                "batch_verified": failed == 0 and partial == 0,
                 "succeeded": succeeded,
+                "partial": partial,
                 "failed": failed,
             },
             "write_executed": write_executed,
-            "write_succeeded": write_executed and failed == 0,
+            "write_succeeded": write_executed and failed == 0 and partial == 0,
             "safe_to_retry": not write_executed,
         }
         return _attach_builder_apply_envelope(tool_name, payload)
@@ -295,18 +297,20 @@ class AiBuilderTools(ToolBase):
                 app_item["error"] = error
                 errors.append(error)
             apps.append(app_item)
+        partial = sum(1 for item in apps if str(item.get("status") or "") == "partial_success")
         succeeded = sum(1 for item in apps if str(item.get("status") or "") in {"success", "partial_success"})
         failed = len(apps) - succeeded
+        status = "failed" if succeeded == 0 else "partial_success" if failed > 0 or partial > 0 else "success"
         return {
-            "status": "success" if failed == 0 else "failed" if succeeded == 0 else "partial_success",
-            "error_code": None if failed == 0 else f"{tool_name.upper()}_BATCH_PARTIAL" if succeeded else f"{tool_name.upper()}_BATCH_FAILED",
-            "recoverable": failed > 0,
-            "message": f"read {tool_name} for {succeeded}/{len(apps)} apps" if failed else f"read {tool_name} for {succeeded} apps",
+            "status": status,
+            "error_code": None if status == "success" else f"{tool_name.upper()}_BATCH_PARTIAL" if succeeded else f"{tool_name.upper()}_BATCH_FAILED",
+            "recoverable": failed > 0 or partial > 0,
+            "message": f"read {tool_name} for {succeeded}/{len(apps)} apps" if status != "success" else f"read {tool_name} for {succeeded} apps",
             "normalized_args": {"app_keys": normalized_keys},
             "app_keys": normalized_keys,
             "apps": apps,
             "errors": errors,
-            "verification": {"batch_verified": failed == 0, "succeeded": succeeded, "failed": failed},
+            "verification": {"batch_verified": failed == 0 and partial == 0, "succeeded": succeeded, "partial": partial, "failed": failed},
         }
 
     def register(self, mcp) -> None: