@josephyan/qingflow-app-user-mcp 0.2.0-beta.81 → 0.2.0-beta.82

package/README.md

@@ -3,13 +3,13 @@
 Install:
 
 ```bash
-npm install @josephyan/qingflow-app-user-mcp@0.2.0-beta.81
+npm install @josephyan/qingflow-app-user-mcp@0.2.0-beta.82
 ```
 
 Run:
 
 ```bash
-npx -y -p @josephyan/qingflow-app-user-mcp@0.2.0-beta.81 qingflow-app-user-mcp
+npx -y -p @josephyan/qingflow-app-user-mcp@0.2.0-beta.82 qingflow-app-user-mcp
 ```
 
 Environment:

package/package.json

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

package/src/qingflow_mcp/__init__.py

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

package/src/qingflow_mcp/builder_facade/service.py

@@ -542,7 +542,10 @@ class AiBuilderFacade:
             "warnings": [],
             "verification": {
                 "package_exists": True,
-                "visibility_verified": verification.get("visibility") == _public_visibility_from_member_auth(desired_auth),
+                "visibility_verified": _visibility_matches_expected(
+                    verification.get("visibility"),
+                    _public_visibility_from_member_auth(desired_auth),
+                ),
             },
             "verified": True,
             **{
@@ -8688,6 +8691,50 @@ def _visibility_summary(visibility: dict[str, Any]) -> dict[str, Any]:
     }
 
 
+def _visibility_matches_expected(actual: Any, expected: Any) -> bool:
+    if not isinstance(actual, dict) or not isinstance(expected, dict):
+        return False
+    if str(actual.get("mode") or "") != str(expected.get("mode") or ""):
+        return False
+    if str(actual.get("external_mode") or "") != str(expected.get("external_mode") or ""):
+        return False
+
+    actual_selectors = actual.get("selectors") if isinstance(actual.get("selectors"), dict) else {}
+    expected_selectors = expected.get("selectors") if isinstance(expected.get("selectors"), dict) else {}
+    actual_external = actual.get("external_selectors") if isinstance(actual.get("external_selectors"), dict) else {}
+    expected_external = expected.get("external_selectors") if isinstance(expected.get("external_selectors"), dict) else {}
+
+    def sorted_values(source: dict[str, Any], key: str) -> list[str]:
+        return sorted(str(item) for item in (source.get(key) or []) if item is not None and str(item).strip())
+
+    for key in ("member_uids", "dept_ids", "role_ids"):
+        if sorted_values(actual_selectors, key) != sorted_values(expected_selectors, key):
+            return False
+    for key in ("member_ids", "dept_ids"):
+        if sorted_values(actual_external, key) != sorted_values(expected_external, key):
+            return False
+
+    # Backends commonly enrich id-based selectors with names/emails on readback; require
+    # caller-specified text selectors only when no id selector anchors that subject.
+    text_selector_pairs = (
+        (actual_selectors, expected_selectors, "member_uids", "member_emails"),
+        (actual_selectors, expected_selectors, "member_uids", "member_names"),
+        (actual_selectors, expected_selectors, "dept_ids", "dept_names"),
+        (actual_selectors, expected_selectors, "role_ids", "role_names"),
+        (actual_external, expected_external, "member_ids", "member_emails"),
+    )
+    for actual_group, expected_group, id_key, text_key in text_selector_pairs:
+        if sorted_values(expected_group, id_key):
+            continue
+        expected_text = sorted_values(expected_group, text_key)
+        if expected_text and sorted_values(actual_group, text_key) != expected_text:
+            return False
+
+    if "include_sub_departs" in expected_selectors and actual_selectors.get("include_sub_departs") != expected_selectors.get("include_sub_departs"):
+        return False
+    return True
+
+
 def _mapping_contains(actual: Any, expected: Any) -> bool:
     if isinstance(expected, dict):
         if not isinstance(actual, dict):