@qingflow-tech/qingflow-app-builder-mcp 1.0.40 → 1.0.42

package/src/qingflow_mcp/cli/commands/builder.py

@@ -5,7 +5,7 @@ from copy import deepcopy
 
 from ..context import CliContext
 from ..json_io import load_json_value
-from .common import load_list_arg, load_object_arg, raise_config_error, require_list_arg
+from .common import load_list_arg, load_object_arg, parse_bool_text, raise_config_error, require_list_arg
 
 
 def register(subparsers: argparse._SubParsersAction[argparse.ArgumentParser]) -> None:
@@ -196,7 +196,7 @@ def register(subparsers: argparse._SubParsersAction[argparse.ArgumentParser]) ->
     schema_apply_apply.add_argument("--color")
     schema_apply_apply.add_argument("--visibility-file")
     schema_apply_apply.add_argument("--create-if-missing", action="store_true")
-    schema_apply_apply.add_argument("--publish", action=argparse.BooleanOptionalAction, default=True)
+    schema_apply_apply.add_argument("--publish", action=argparse.BooleanOptionalAction, default=None)
     schema_apply_apply.add_argument("--apps-file", help="多应用 schema JSON 数组；每项可带 client_key/app_name/add_fields，支持 relation target_app_ref")
     schema_apply_apply.add_argument("--add-fields-file", help="字段 JSON 数组；字段可用 as_data_title/as_data_cover 标记数据标题/封面")
     schema_apply_apply.add_argument("--update-fields-file", help="字段更新 JSON 数组；set 内可用 as_data_title/as_data_cover 标记数据标题/封面")
@@ -483,8 +483,20 @@ def _handle_schema_apply(args: argparse.Namespace, context: CliContext) -> dict:
     apps_warnings = apps_payload.get("warnings") or []
     package_id = args.package_id
     if package_id is None and apps_payload.get("package_id") is not None:
-        package_id = int(apps_payload["package_id"])
+        package_id = _coerce_apps_file_package_id(apps_payload["package_id"])
     if args.apps_file:
+        file_create_if_missing = _coerce_apps_file_bool(
+            apps_payload.get("create_if_missing"),
+            field_name="create_if_missing",
+            default=False,
+        )
+        file_publish = _coerce_apps_file_bool(
+            apps_payload.get("publish"),
+            field_name="publish",
+            default=True,
+        )
+        effective_create_if_missing = bool(args.create_if_missing or file_create_if_missing)
+        effective_publish = bool(args.publish if args.publish is not None else file_publish)
         if not apps:
             raise_config_error(
                 "schema apply multi-app mode requires a non-empty --apps-file.",
@@ -505,8 +517,8 @@ def _handle_schema_apply(args: argparse.Namespace, context: CliContext) -> dict:
             profile=args.profile,
             package_id=package_id,
             visibility=load_object_arg(args.visibility_file, option_name="--visibility-file"),
-            create_if_missing=bool(args.create_if_missing),
-            publish=bool(args.publish),
+            create_if_missing=effective_create_if_missing,
+            publish=effective_publish,
             apps=apps,
             add_fields=[],
             update_fields=[],
@@ -543,7 +555,7 @@ def _handle_schema_apply(args: argparse.Namespace, context: CliContext) -> dict:
         color=args.color,
         visibility=load_object_arg(args.visibility_file, option_name="--visibility-file"),
         create_if_missing=bool(args.create_if_missing),
-        publish=bool(args.publish),
+        publish=True if args.publish is None else bool(args.publish),
         add_fields=load_list_arg(args.add_fields_file, option_name="--add-fields-file"),
         update_fields=load_list_arg(args.update_fields_file, option_name="--update-fields-file"),
         remove_fields=load_list_arg(args.remove_fields_file, option_name="--remove-fields-file"),
@@ -595,6 +607,10 @@ def _load_apps_file_arg(path: str | None) -> dict[str, object]:
             }
             if wrapper.get("package_id") is not None:
                 result["package_id"] = wrapper.get("package_id")
+            if wrapper.get("create_if_missing") is not None:
+                result["create_if_missing"] = wrapper.get("create_if_missing")
+            if wrapper.get("publish") is not None:
+                result["publish"] = wrapper.get("publish")
             return result
         if any(isinstance(item, dict) and "apps" in item for item in payload):
             raise_config_error(
@@ -616,6 +632,10 @@ def _load_apps_file_arg(path: str | None) -> dict[str, object]:
         result: dict[str, object] = {"apps": apps}
         if payload.get("package_id") is not None:
             result["package_id"] = payload.get("package_id")
+        if payload.get("create_if_missing") is not None:
+            result["create_if_missing"] = payload.get("create_if_missing")
+        if payload.get("publish") is not None:
+            result["publish"] = payload.get("publish")
         return result
     raise_config_error(
         "--apps-file must be a JSON array or an object containing apps.",
@@ -625,6 +645,60 @@ def _load_apps_file_arg(path: str | None) -> dict[str, object]:
     )
 
 
+def _coerce_apps_file_package_id(value: object) -> int:
+    package_id: int
+    if isinstance(value, bool):
+        _raise_apps_file_package_id_invalid(value)
+    if isinstance(value, int):
+        package_id = value
+    elif isinstance(value, str):
+        stripped = value.strip()
+        try:
+            package_id = int(stripped)
+        except ValueError:
+            _raise_apps_file_package_id_invalid(value)
+    else:
+        _raise_apps_file_package_id_invalid(value)
+    if package_id <= 0:
+        _raise_apps_file_package_id_invalid(value)
+    return package_id
+
+
+def _raise_apps_file_package_id_invalid(value: object) -> None:
+    raise_config_error(
+        "--apps-file package_id must be a positive integer.",
+        fix_hint='Use a numeric package_id, for example {"package_id":1001,"apps":[...]}.',
+        error_code="APPS_FILE_PACKAGE_ID_INVALID",
+        details={
+            "field": "package_id",
+            "value": value,
+            "expected": "positive integer or numeric string",
+        },
+    )
+
+
+def _coerce_apps_file_bool(value: object, *, field_name: str, default: bool) -> bool:
+    if value is None:
+        return default
+    if isinstance(value, bool):
+        return value
+    if isinstance(value, str):
+        try:
+            return parse_bool_text(value)
+        except argparse.ArgumentTypeError:
+            pass
+    raise_config_error(
+        f"--apps-file {field_name} must be a boolean.",
+        fix_hint=f'Use "{field_name}": true or "{field_name}": false in --apps-file.',
+        error_code="APPS_FILE_BOOLEAN_INVALID",
+        details={
+            "field": field_name,
+            "value": value,
+            "expected": "boolean true/false or string true/false/1/0/yes/no",
+        },
+    )
+
+
 def _handle_layout_apply(args: argparse.Namespace, context: CliContext) -> dict:
     return context.builder.app_layout_apply(
         profile=args.profile,

package/src/qingflow_mcp/cli/formatters.py

@@ -36,6 +36,7 @@ def _is_executed_nonfatal_result(result: dict[str, Any]) -> bool:
     status = str(result.get("status") or "").lower()
     executed = bool(
         result.get("write_executed")
+        or result.get("write_may_have_succeeded")
         or result.get("delete_executed")
         or result.get("action_executed")
         or result.get("export_executed")

package/src/qingflow_mcp/cli/main.py

@@ -423,6 +423,7 @@ def _is_executed_nonfatal_result(result: dict[str, Any]) -> bool:
     status = str(result.get("status") or "").lower()
     executed = bool(
         result.get("write_executed")
+        or result.get("write_may_have_succeeded")
         or result.get("delete_executed")
         or result.get("action_executed")
         or result.get("export_executed")
@@ -441,6 +442,7 @@ def _has_readback_unavailable_verification(result: dict[str, Any]) -> bool:
             "readback_pending",
             "metadata_unverified",
             "views_read_unavailable",
+            "readback_before_retry",
         )
     )
 

package/src/qingflow_mcp/response_trim.py

@@ -100,7 +100,7 @@ def trim_error_response(payload: dict[str, Any]) -> dict[str, Any]:
     details = trimmed.get("details")
     if isinstance(details, dict):
         preserved = {}
-        for key in ("blocking_issues", "compiled_match_rules"):
+        for key in ("blocking_issues", "compiled_match_rules", "issues", "expected_shape"):
             if key in details:
                 preserved[key] = details.get(key)
         compact_details = _compact_scalar_dict(details)
@@ -169,6 +169,7 @@ def _is_executed_nonfatal_payload(payload: dict[str, Any]) -> bool:
     status = str(payload.get("status") or "").lower()
     executed = bool(
         payload.get("write_executed")
+        or payload.get("write_may_have_succeeded")
         or payload.get("delete_executed")
         or payload.get("action_executed")
         or payload.get("export_executed")
@@ -183,7 +184,7 @@ def _trim_returned_failure(payload: dict[str, Any]) -> dict[str, Any]:
     details = trimmed.get("details")
     if isinstance(details, dict):
         preserved = {}
-        for key in ("blocking_issues", "compiled_match_rules"):
+        for key in ("blocking_issues", "compiled_match_rules", "issues", "expected_shape"):
             if key in details:
                 preserved[key] = details.get(key)
         compact_details = _compact_scalar_dict(details)
@@ -1070,8 +1071,9 @@ def _trim_builder_envelope(payload: JSONObject) -> None:
         if isinstance(details, dict):
             _drop_deep_keys(details, {"request_route", "base_url", "normalized_args", "suggested_next_call", "transport", "response", "body", "raw"})
             preserved = {}
-            if isinstance(details.get("compiled_match_rules"), dict):
-                preserved["compiled_match_rules"] = details.get("compiled_match_rules")
+            for key in ("compiled_match_rules", "issues", "expected_shape"):
+                if key in details:
+                    preserved[key] = details.get(key)
             compact = _compact_scalar_dict(details)
             compact.update(preserved)
             if compact: