@josephyan/qingflow-app-user-mcp 1.1.21 → 1.1.22

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

@@ -212,7 +212,6 @@ def register(subparsers: argparse._SubParsersAction[argparse.ArgumentParser]) ->
     schema_apply_apply.add_argument("--icon")
     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=None)
     schema_apply_apply.add_argument("--apps-file", help="多应用 schema JSON 对象；每项可带 client_key/app_name/form，支持 relation target_app_ref")
     schema_apply_apply.add_argument("--form-file", help="单应用表单结构 JSON 数组；form[].rows[][] 同时声明字段和表单分组布局")
@@ -386,6 +385,7 @@ def _handle_package_apply(args: argparse.Namespace, context: CliContext) -> dict
     config = load_object_arg(args.config_file, option_name="--config-file")
     if not isinstance(config, dict):
         raise_config_error("package apply --config-file must contain a JSON object.")
+    config.pop("create_if_missing", None)
     return context.builder.package_apply(profile=args.profile, **config)
 
 
@@ -561,17 +561,11 @@ def _handle_schema_apply(args: argparse.Namespace, context: CliContext) -> dict:
     if package_id is None and apps_payload.get("package_id") is not None:
         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=True,
-        )
         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(
@@ -581,7 +575,7 @@ def _handle_schema_apply(args: argparse.Namespace, context: CliContext) -> dict:
             )
         if args.app_key or args.app_name or args.app_title or args.form_file or args.add_fields_file or args.update_fields_file or args.remove_fields_file:
             raise_config_error(
-                "schema apply multi-app mode accepts --package-id plus --apps-file only; --create-if-missing is optional.",
+                "schema apply multi-app mode accepts --package-id plus --apps-file only.",
                 fix_hint="Use apps[].form inside --apps-file for batch mode, or remove --apps-file and use --form-file for a single app.",
             )
         if package_id is None:
@@ -593,7 +587,6 @@ 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=effective_create_if_missing,
             publish=effective_publish,
             apps=apps,
             form=None,
@@ -611,16 +604,16 @@ def _handle_schema_apply(args: argparse.Namespace, context: CliContext) -> dict:
     has_app_title = bool((args.app_title or "").strip())
     has_package_id = args.package_id is not None
     if has_app_key:
-        if args.create_if_missing or has_package_id:
+        if has_package_id:
             raise_config_error(
                 "schema apply edit mode accepts --app-key and optional --app-name only.",
-                fix_hint="For existing apps, use `--app-key` and optionally `--app-name`. For create mode, use `--package-id --app-name --create-if-missing`.",
+                fix_hint="For existing apps, use `--app-key` and optionally `--app-name`. For create mode, use `--package-id --app-name`.",
             )
     else:
-        if not args.create_if_missing or not has_package_id or not (has_app_name or has_app_title):
+        if not has_package_id or not (has_app_name or has_app_title):
             raise_config_error(
-                "schema apply create mode requires --package-id, --app-name, and --create-if-missing.",
-                fix_hint="Use `--app-key` for existing apps, or pass `--package-id --app-name --create-if-missing` to create a new app.",
+                "schema apply create mode requires --package-id and --app-name.",
+                fix_hint="Use `--app-key` for existing apps, or pass `--package-id --app-name` to create a new app.",
             )
     return context.builder.app_schema_apply(
         profile=args.profile,
@@ -631,7 +624,6 @@ def _handle_schema_apply(args: argparse.Namespace, context: CliContext) -> dict:
         icon=args.icon,
         color=args.color,
         visibility=load_object_arg(args.visibility_file, option_name="--visibility-file"),
-        create_if_missing=bool(args.create_if_missing),
         publish=True if args.publish is None else bool(args.publish),
         form=load_list_arg(args.form_file, option_name="--form-file") if args.form_file else [],
         add_fields=load_list_arg(args.add_fields_file, option_name="--add-fields-file"),
@@ -685,8 +677,6 @@ 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
@@ -710,8 +700,6 @@ 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

package/src/qingflow_mcp/server_app_builder.py

@@ -39,7 +39,7 @@ def build_builder_server() -> FastMCP:
             "Use workspace_icon_catalog_get before creating app packages, apps, or portals when supported icon/color candidates are needed; new workspace resources require explicit non-template icon + color, and the CLI validates choices without inferring business defaults. "
             "app_get as the default app map read, then app_get_fields/app_repair_code_blocks/app_get_layout/app_get_views/app_get_flow/app_get_charts/portal_list/portal_get/view_get/chart_get for focused configuration reads, "
             "member_search/role_search/role_create when workflow assignees must come from the directory or role catalog, preferring roles over explicit members unless the user explicitly names members, "
-            "then app_schema_apply/app_layout_apply/app_flow_apply/app_views_apply/app_custom_buttons_apply/app_associated_resources_apply/app_charts_apply/portal_apply/portal_delete to execute normalized patches; these apply/delete tools perform planning, normalization, and dependency checks internally where applicable. Schema/layout/views noop requests skip publish, app_custom_buttons_apply and app_associated_resources_apply publish after at least one write succeeds and expose no draft-only parameter, charts are immediate-live without publish and resolve targets by chart_id first then exact unique chart name, portal updates use replace semantics only when sections are supplied and edit-mode base-info-only updates may omit sections, portal delete separates DELETE execution from readback verification, portal pc layout is a 24-column grid and mobile is a 6-column grid so omit position or use layout_preset when unsure, publish=false only guarantees draft/base-info updates for tools that still expose that parameter, and flow should use publish=false whenever you only want draft/precheck behavior. "
+            "then app_schema_apply/app_layout_apply/app_flow_apply/app_views_apply/app_custom_buttons_apply/app_associated_resources_apply/app_charts_apply/portal_apply/portal_delete to execute normalized patches; these apply/delete tools perform basic validation, normalization, writes, and readback verification where applicable. Schema/layout/views noop requests skip publish, app_custom_buttons_apply and app_associated_resources_apply publish after at least one write succeeds and expose no draft-only parameter, charts are immediate-live without publish and resolve targets by chart_id first then exact unique chart name, portal updates use replace semantics only when sections are supplied and edit-mode base-info-only updates may omit sections, portal delete separates DELETE execution from readback verification, portal pc layout is a 24-column grid and mobile is a 6-column grid so omit position or use layout_preset when unsure, publish=false only guarantees draft/base-info updates for tools that still expose that parameter, and flow should use publish=false whenever you only want draft/precheck behavior. "
             "Builder apply/write outputs include schema_version, operation, summary, and resources[]; use resources[].id/key/name/ids/parent as the stable UI and agent display entry, and keep legacy fields such as field_diff/views_diff/chart_results only for compatibility or troubleshooting. "
             "For existing object parameter replacement, prefer patch_views, patch_buttons, patch_resources, and patch_charts with set/unset; the tool reads current config and full-saves internally, while upsert_* is for creation or full target configuration and should not be used as an incomplete partial update. "
             "For builder delete/remove apply results, separate delete execution from readback verification: after DELETE is sent, resources expose delete_executed, readback_status, and safe_to_retry_delete=false. If readback_status is unavailable or still_exists, do not blindly repeat the delete; confirm later with app_get/view_get/chart_get or the relevant apply readback. Views/buttons use single-item readback; associated resources use one app-level resource-pool readback because there is no confirmed single-item GET. "
@@ -197,7 +197,6 @@ def build_builder_server() -> FastMCP:
         profile: str = DEFAULT_PROFILE,
         package_id: int | None = None,
         package_name: str | None = None,
-        create_if_missing: bool | None = None,
         icon: str | None = None,
         color: str | None = None,
         visibility: dict | None = None,
@@ -208,7 +207,6 @@ def build_builder_server() -> FastMCP:
             profile=profile,
             package_id=package_id,
             package_name=package_name,
-            create_if_missing=create_if_missing,
             icon=icon,
             color=color,
             visibility=visibility,
@@ -426,7 +424,6 @@ def build_builder_server() -> FastMCP:
         icon: str = "",
         color: str = "",
         visibility: dict | None = None,
-        create_if_missing: bool | None = None,
         publish: bool = True,
         form: list[dict] | None = None,
         add_fields: list[dict] | None = None,
@@ -437,7 +434,7 @@ def build_builder_server() -> FastMCP:
         if apps:
             if app_key or app_name or app_title or form or add_fields or update_fields or remove_fields:
                 return _config_failure(
-                    "app_schema_apply multi-app mode accepts package_id plus apps only; create_if_missing is optional.",
+                    "app_schema_apply multi-app mode accepts package_id plus apps only.",
                     fix_hint="Use apps[].form for batch mode, or use the single-app arguments without apps.",
                 )
             if package_id is None:
@@ -449,7 +446,6 @@ def build_builder_server() -> FastMCP:
                 profile=profile,
                 package_id=package_id,
                 visibility=visibility,
-                create_if_missing=True if create_if_missing is None else bool(create_if_missing),
                 publish=publish,
                 form=None,
                 add_fields=[],
@@ -461,17 +457,17 @@ def build_builder_server() -> FastMCP:
         has_app_name = bool((app_name or "").strip())
         has_app_title = bool((app_title or "").strip())
         has_package_id = package_id is not None
-        effective_create_if_missing = bool(create_if_missing)
+        create_by_shape = has_package_id and (has_app_name or has_app_title)
         if has_app_key:
-            if effective_create_if_missing or has_package_id:
+            if has_package_id:
                 return _config_failure(
                     "app_schema_apply edit mode accepts app_key and optional app_name rename only.",
-                    fix_hint="For existing apps, use `app_key` and optionally `app_name`. For create mode, use `package_id + app_name + create_if_missing=true`.",
+                    fix_hint="For existing apps, use `app_key` and optionally `app_name`. For create mode, use `package_id + app_name`.",
                 )
-        elif not (effective_create_if_missing and has_package_id and (has_app_name or has_app_title)):
+        elif not create_by_shape:
             return _config_failure(
-                "app_schema_apply create mode requires package_id, app_name, and create_if_missing=true.",
-                fix_hint="Use `app_key` for existing apps, or pass `package_id + app_name + create_if_missing=true` to create a new app.",
+                "app_schema_apply create mode requires package_id and app_name.",
+                fix_hint="Use `app_key` for existing apps, or pass `package_id + app_name` to create a new app.",
             )
         return ai_builder.app_schema_apply(
             profile=profile,
@@ -482,7 +478,6 @@ def build_builder_server() -> FastMCP:
             icon=icon,
             color=color,
             visibility=visibility,
-            create_if_missing=effective_create_if_missing,
             publish=publish,
             form=form or [],
             add_fields=add_fields or [],