@qingflow-tech/qingflow-cli 1.1.2 → 1.1.34

package/skills/qingflow-cli/scripts/validate_system_build_summary.py

@@ -0,0 +1,124 @@
+#!/usr/bin/env python3
+from __future__ import annotations
+
+import argparse
+import json
+import sys
+from pathlib import Path
+from typing import Any
+
+
+VALID_PORTAL_STATUSES = {"verified", "unverified", "not_created"}
+VALID_APP_VERIFY_STATUSES = {"verified", "unverified", "failed"}
+
+
+def main(argv: list[str] | None = None) -> int:
+    parser = argparse.ArgumentParser(description="Validate a Qingflow complete-system delivery summary JSON file.")
+    parser.add_argument("summary_file", nargs="?", default="tmp/qingflow_system_build_summary.json")
+    args = parser.parse_args(argv)
+    path = Path(args.summary_file)
+    issues = validate_summary_file(path)
+    if issues:
+        _emit({"status": "failed", "error_code": "SYSTEM_BUILD_SUMMARY_INVALID", "path": str(path), "issues": issues})
+        return 1
+    payload = json.loads(path.read_text(encoding="utf-8"))
+    _emit(
+        {
+            "status": "success",
+            "path": str(path),
+            "summary": {
+                "package_id": payload.get("package_id"),
+                "package_name": payload.get("package_name"),
+                "portal_dash_key": payload.get("portal_dash_key"),
+                "portal_live_status": payload.get("portal_live_status"),
+                "app_count": len(payload.get("apps") or []),
+                "partial_count": len(payload.get("partial_items") or []),
+                "needs_followup_count": len(payload.get("needs_followup") or []),
+            },
+        }
+    )
+    return 0
+
+
+def validate_summary_file(path: Path) -> list[dict[str, Any]]:
+    if not path.exists():
+        return [_issue("MISSING_FILE", "$", f"summary file does not exist: {path}")]
+    try:
+        payload = json.loads(path.read_text(encoding="utf-8"))
+    except OSError as exc:
+        return [_issue("READ_FAILED", "$", str(exc))]
+    except json.JSONDecodeError as exc:
+        return [_issue("INVALID_JSON", "$", exc.msg)]
+    if not isinstance(payload, dict):
+        return [_issue("INVALID_ROOT", "$", "summary root must be a JSON object")]
+
+    issues: list[dict[str, Any]] = []
+    _require_positive_int(payload, "package_id", issues)
+    _require_nonempty_string(payload, "package_name", issues)
+    _require_bool(payload, "front_end_visible", issues)
+    _require_string_enum(payload, "portal_live_status", VALID_PORTAL_STATUSES, issues)
+    if payload.get("portal_live_status") == "verified":
+        _require_nonempty_string(payload, "portal_dash_key", issues)
+    elif "portal_dash_key" in payload and payload.get("portal_dash_key") is not None and not isinstance(payload.get("portal_dash_key"), str):
+        issues.append(_issue("INVALID_TYPE", "$.portal_dash_key", "portal_dash_key must be a string when present"))
+
+    apps = payload.get("apps")
+    if not isinstance(apps, list) or not apps:
+        issues.append(_issue("INVALID_APPS", "$.apps", "apps must be a non-empty array"))
+    else:
+        for index, app in enumerate(apps):
+            prefix = f"$.apps[{index}]"
+            if not isinstance(app, dict):
+                issues.append(_issue("INVALID_APP_ITEM", prefix, "app item must be an object"))
+                continue
+            _require_nonempty_string(app, "app_key", issues, prefix)
+            _require_nonempty_string(app, "app_name", issues, prefix)
+            for key in ("fields_count", "views_count", "flows_count", "charts_count"):
+                _require_nonnegative_int(app, key, issues, prefix)
+            _require_string_enum(app, "publish_verify_status", VALID_APP_VERIFY_STATUSES, issues, prefix)
+
+    for key in ("warnings", "partial_items", "needs_followup"):
+        if not isinstance(payload.get(key), list):
+            issues.append(_issue("INVALID_TYPE", f"$.{key}", f"{key} must be an array"))
+    return issues
+
+
+def _require_positive_int(payload: dict[str, Any], key: str, issues: list[dict[str, Any]], prefix: str = "$") -> None:
+    value = payload.get(key)
+    if isinstance(value, bool) or not isinstance(value, int) or value <= 0:
+        issues.append(_issue("INVALID_TYPE", f"{prefix}.{key}", f"{key} must be a positive integer"))
+
+
+def _require_nonnegative_int(payload: dict[str, Any], key: str, issues: list[dict[str, Any]], prefix: str = "$") -> None:
+    value = payload.get(key)
+    if isinstance(value, bool) or not isinstance(value, int) or value < 0:
+        issues.append(_issue("INVALID_TYPE", f"{prefix}.{key}", f"{key} must be a non-negative integer"))
+
+
+def _require_nonempty_string(payload: dict[str, Any], key: str, issues: list[dict[str, Any]], prefix: str = "$") -> None:
+    value = payload.get(key)
+    if not isinstance(value, str) or not value.strip():
+        issues.append(_issue("INVALID_TYPE", f"{prefix}.{key}", f"{key} must be a non-empty string"))
+
+
+def _require_bool(payload: dict[str, Any], key: str, issues: list[dict[str, Any]], prefix: str = "$") -> None:
+    if not isinstance(payload.get(key), bool):
+        issues.append(_issue("INVALID_TYPE", f"{prefix}.{key}", f"{key} must be a boolean"))
+
+
+def _require_string_enum(payload: dict[str, Any], key: str, allowed: set[str], issues: list[dict[str, Any]], prefix: str = "$") -> None:
+    value = payload.get(key)
+    if value not in allowed:
+        issues.append(_issue("INVALID_VALUE", f"{prefix}.{key}", f"{key} must be one of {sorted(allowed)}"))
+
+
+def _issue(code: str, path: str, message: str) -> dict[str, Any]:
+    return {"code": code, "path": path, "message": message}
+
+
+def _emit(payload: dict[str, Any]) -> None:
+    print(json.dumps(payload, ensure_ascii=False, indent=2))
+
+
+if __name__ == "__main__":
+    sys.exit(main())

package/skills/qingflow-cli/scripts/workflow/diff_flow_spec.py

@@ -0,0 +1,275 @@
+#!/usr/bin/env python3
+"""
+对比两个工作流 spec 的差异，输出新增/删除/修改的节点和边。
+用于更新模式下辅助 agent 判断是否遵循最小修改原则。
+
+用法:
+  python3 diff_flow_spec.py <old_spec.json> <new_spec.json>
+  python3 diff_flow_spec.py <old_spec.json> <new_spec.json> --json   # JSON 输出
+"""
+
+import json
+import sys
+
+
+def load_json(path):
+    with open(path, 'r', encoding='utf-8') as f:
+        return json.load(f)
+
+
+def extract_edges(spec):
+    """从 spec 中提取边列表，兼容 edges 在不同层级的情况"""
+    e = spec.get('edges', [])
+    if isinstance(e, dict) and 'edges' in e:
+        return e['edges']
+    if isinstance(e, list):
+        return e
+    return []
+
+
+def node_key(n):
+    """节点的唯一标识"""
+    return n.get('id', '')
+
+
+def edge_key(e):
+    """边的唯一标识：from→to"""
+    return (e.get('from', ''), e.get('to', ''))
+
+
+def node_identity(n):
+    """节点的完整内容（不含 id，用于判断是否修改）"""
+    return {
+        'type': n.get('type', ''),
+        'name': n.get('name', ''),
+        'attrs': n.get('attrs', {}),
+    }
+
+
+def edge_identity(e):
+    """边的完整内容（不含 from/to，用于判断是否修改）"""
+    return {
+        'label': e.get('label', ''),
+        'condition': e.get('condition', {}),
+    }
+
+
+def diff_nodes(old_nodes, new_nodes):
+    """对比节点差异"""
+    old_by_id = {node_key(n): n for n in old_nodes}
+    new_by_id = {node_key(n): n for n in new_nodes}
+
+    old_ids = set(old_by_id.keys())
+    new_ids = set(new_by_id.keys())
+
+    deleted_ids = sorted(old_ids - new_ids)
+    added_ids = sorted(new_ids - old_ids)
+    common_ids = sorted(old_ids & new_ids)
+
+    modified = []
+    id_changes = []
+    for nid in common_ids:
+        old_ident = node_identity(old_by_id[nid])
+        new_ident = node_identity(new_by_id[nid])
+        if old_ident != new_ident:
+            modified.append({
+                'id': nid,
+                'old': {'type': old_ident['type'], 'name': old_ident['name']},
+                'new': {'type': new_ident['type'], 'name': new_ident['name']},
+                'attrs_changed': old_ident['attrs'] != new_ident['attrs'],
+                'type_changed': old_ident['type'] != new_ident['type'],
+                'name_changed': old_ident['name'] != new_ident['name'],
+            })
+
+    # 检测可能的不必要 ID 变更（内容相同但 ID 不同）
+    old_by_identity = {}
+    for n in old_nodes:
+        ident = json.dumps(node_identity(n), sort_keys=True, ensure_ascii=False)
+        old_by_identity.setdefault(ident, []).append(n)
+
+    for n in new_nodes:
+        if node_key(n) in added_ids:
+            ident = json.dumps(node_identity(n), sort_keys=True, ensure_ascii=False)
+            if ident in old_by_identity and old_by_identity[ident]:
+                old_match = old_by_identity[ident][0]
+                if node_key(old_match) in deleted_ids:
+                    id_changes.append({
+                        'old_id': node_key(old_match),
+                        'new_id': node_key(n),
+                        'identity': node_identity(n),
+                        'warning': '节点内容未变但 ID 已变更，可能导致后端不支持配置丢失',
+                    })
+
+    return {
+        'deleted': [{'id': nid, 'name': old_by_id[nid].get('name', ''), 'type': old_by_id[nid].get('type', '')} for nid in deleted_ids],
+        'added': [{'id': nid, 'name': new_by_id[nid].get('name', ''), 'type': new_by_id[nid].get('type', '')} for nid in added_ids],
+        'modified': modified,
+        'id_changes': id_changes,
+        'unchanged': len(common_ids) - len(modified),
+    }
+
+
+def diff_edges(old_edges, new_edges):
+    """对比边差异"""
+    old_by_key = {edge_key(e): e for e in old_edges}
+    new_by_key = {edge_key(e): e for e in new_edges}
+
+    old_keys = set(old_by_key.keys())
+    new_keys = set(new_by_key.keys())
+
+    deleted_keys = sorted(old_keys - new_keys)
+    added_keys = sorted(new_keys - old_keys)
+    common_keys = sorted(old_keys & new_keys)
+
+    modified = []
+    for key in common_keys:
+        old_ident = edge_identity(old_by_key[key])
+        new_ident = edge_identity(new_by_key[key])
+        if old_ident != new_ident:
+            modified.append({
+                'from': key[0],
+                'to': key[1],
+                'old_condition': old_ident['condition'],
+                'new_condition': new_ident['condition'],
+                'label_changed': old_ident['label'] != new_ident['label'],
+                'condition_changed': old_ident['condition'] != new_ident['condition'],
+            })
+
+    return {
+        'deleted': [{'from': k[0], 'to': k[1]} for k in deleted_keys],
+        'added': [{'from': k[0], 'to': k[1]} for k in added_keys],
+        'modified': modified,
+        'unchanged': len(common_keys) - len(modified),
+    }
+
+
+def print_human(result):
+    """人类可读输出"""
+    nodes = result['nodes']
+    edges = result['edges']
+
+    print("=" * 60)
+    print("工作流 Spec 差异分析")
+    print("=" * 60)
+
+    # 节点汇总
+    print(f"\n📊 节点汇总:")
+    print(f"  删除: {len(nodes['deleted'])} | 新增: {len(nodes['added'])} | "
+          f"修改: {len(nodes['modified'])} | 未变: {nodes['unchanged']}")
+
+    if nodes['id_changes']:
+        print(f"\n⚠️  检测到 {len(nodes['id_changes'])} 个可能的 ID 变更（内容相同但 ID 不同）:")
+        for ic in nodes['id_changes']:
+            print(f"  {ic['old_id']} → {ic['new_id']} ({ic['identity']['name']})")
+            print(f"    ⚠ {ic['warning']}")
+
+    if nodes['deleted']:
+        print(f"\n🗑  删除的节点 ({len(nodes['deleted'])}):")
+        for d in nodes['deleted']:
+            print(f"  - [{d['type']}] {d['id']} ({d['name']})")
+
+    if nodes['added']:
+        print(f"\n➕ 新增的节点 ({len(nodes['added'])}):")
+        for a in nodes['added']:
+            print(f"  - [{a['type']}] {a['id']} ({a['name']})")
+
+    if nodes['modified']:
+        print(f"\n✏️  修改的节点 ({len(nodes['modified'])}):")
+        for m in nodes['modified']:
+            changes = []
+            if m['type_changed']:
+                changes.append(f"type: {m['old']['type']} → {m['new']['type']}")
+            if m['name_changed']:
+                changes.append(f"name: {m['old']['name']} → {m['new']['name']}")
+            if m['attrs_changed']:
+                changes.append("attrs 已变更")
+            print(f"  - {m['id']}: {', '.join(changes) if changes else '无实质变更'}")
+
+    # 边汇总
+    print(f"\n📊 边汇总:")
+    print(f"  删除: {len(edges['deleted'])} | 新增: {len(edges['added'])} | "
+          f"修改: {len(edges['modified'])} | 未变: {edges['unchanged']}")
+
+    if edges['deleted']:
+        print(f"\n🗑  删除的边 ({len(edges['deleted'])}):")
+        for d in edges['deleted']:
+            print(f"  - {d['from']} → {d['to']}")
+
+    if edges['added']:
+        print(f"\n➕ 新增的边 ({len(edges['added'])}):")
+        for a in edges['added']:
+            print(f"  - {a['from']} → {a['to']}")
+
+    if edges['modified']:
+        print(f"\n✏️  修改的边 ({len(edges['modified'])}):")
+        for m in edges['modified']:
+            changes = []
+            if m['label_changed']:
+                changes.append("label 已变更")
+            if m['condition_changed']:
+                changes.append("condition 已变更")
+            if changes:
+                print(f"  - {m['from']} → {m['to']}: {', '.join(changes)}")
+            else:
+                print(f"  - {m['from']} → {m['to']}: 无实质变更")
+
+    # 最小修改原则评估
+    print(f"\n📋 最小修改原则评估:")
+    issues = 0
+    if nodes['id_changes']:
+        print(f"  ❌ 存在 ID 变更（{len(nodes['id_changes'])} 个节点），可能导致后端不支持配置丢失")
+        issues += 1
+    if nodes['deleted']:
+        print(f"  ⚠️  删除了 {len(nodes['deleted'])} 个节点，请确认是否为业务需要")
+        issues += 1
+    if not nodes['deleted'] and not nodes['id_changes'] and nodes['modified']:
+        print(f"  ✅ 仅修改已有节点，未删除或变更 ID，符合最小修改原则")
+    if not nodes['deleted'] and not nodes['modified'] and not nodes['added'] and not nodes['id_changes']:
+        print(f"  ✅ 无任何变更")
+
+    print(f"\n总结: 错误 {issues} 项")
+
+
+def main():
+    if len(sys.argv) < 3:
+        print("用法: python3 diff_flow_spec.py <old_spec.json> <new_spec.json> [--json]")
+        sys.exit(1)
+
+    old_file = sys.argv[1]
+    new_file = sys.argv[2]
+    output_json = '--json' in sys.argv
+
+    try:
+        old_spec = load_json(old_file)
+    except Exception as e:
+        print(f"FATAL: 无法读取旧 spec 文件 {old_file}: {e}")
+        sys.exit(1)
+
+    try:
+        new_spec = load_json(new_file)
+    except Exception as e:
+        print(f"FATAL: 无法读取新 spec 文件 {new_file}: {e}")
+        sys.exit(1)
+
+    old_nodes = old_spec.get('nodes', [])
+    new_nodes = new_spec.get('nodes', [])
+    old_edges = extract_edges(old_spec)
+    new_edges = extract_edges(new_spec)
+
+    result = {
+        'nodes': diff_nodes(old_nodes, new_nodes),
+        'edges': diff_edges(old_edges, new_edges),
+    }
+
+    if output_json:
+        print(json.dumps(result, indent=2, ensure_ascii=False))
+    else:
+        print_human(result)
+
+    # 如果有 ID 变更，返回非零以便脚本判断
+    has_id_changes = len(result['nodes']['id_changes']) > 0
+    sys.exit(1 if has_id_changes else 0)
+
+
+if __name__ == '__main__':
+    main()