hap-cli 0.8.28__tar.gz → 0.8.30__tar.gz

{hap_cli-0.8.28 → hap_cli-0.8.30}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: hap-cli
-Version: 0.8.28
+Version: 0.8.30
 Summary: CLI harness for Mingdao HAP - Enterprise no-code platform
 Author: hap-cli
 License: Apache-2.0

{hap_cli-0.8.28 → hap_cli-0.8.30}/hap_cli/__init__.py

@@ -1,3 +1,3 @@
 """CLI harness for Mingdao HAP (hap-cli)."""
 
-__version__ = "0.8.28"
+__version__ = "0.8.30"

{hap_cli-0.8.28 → hap_cli-0.8.30}/hap_cli/commands/worksheet_cmd.py

@@ -919,24 +919,143 @@ def worksheet_delete_button(ctx, worksheet_id, btn_id, app_id, view_id, yes):
 
 @worksheet.command("switches")
 @click.argument("worksheet_id")
+@click.option("--raw", is_flag=True,
+              help="Return the server's switch list unannotated.")
 @pass_context
-def worksheet_switches(ctx, worksheet_id):
-    """Get feature switches for a worksheet."""
+def worksheet_switches(ctx, worksheet_id, raw):
+    """Get feature switches for a worksheet.
+
+    Each switch is listed with the name and group it has on the feature
+    switch page, because several numbers share a name and only the group
+    tells them apart.
+    """
     try:
         session = ctx.get_session()
         data = ws_mod.get_switches(session, worksheet_id)
+        if not raw:
+            data = ws_mod.annotate_switches(data)
         ctx.output(data, lambda d: output_json(d))
     except Exception as e:
         ctx.handle_error(e)
 
 
+@worksheet.command("switch-types")
+@pass_context
+def worksheet_switch_types(ctx):
+    """Show the feature switch numbers with their names and groups."""
+    rows = [ws_mod.describe_switch(t) for t in sorted(ws_mod.SWITCH_TYPES)]
+    ctx.output(
+        rows,
+        lambda d: output_table(
+            d, ["type", "group", "name", "path"],
+            ["Type", "Group", "Name", "Path"],
+        ),
+    )
+
+
+def _parse_switch_changes(pairs):
+    changes = {}
+    for raw in pairs:
+        if "=" not in raw:
+            raise click.UsageError(
+                f"Invalid --set value {raw!r}. Use --set TYPE=true|false.")
+        key, value = raw.split("=", 1)
+        try:
+            stype = int(key.strip())
+        except ValueError:
+            raise click.UsageError(
+                f"Invalid switch number {key!r}. Run `hap worksheet "
+                "switch-types` for the list.")
+        val = value.strip().lower()
+        if val not in ("true", "false", "1", "0", "on", "off"):
+            raise click.UsageError(
+                f"Invalid state {value!r} for switch {stype}. Use true or false.")
+        changes[stype] = val in ("true", "1", "on")
+    return changes
+
+
+@worksheet.command("batch-edit-switch")
+@click.argument("worksheet_ids", nargs=-1, required=True)
+@click.option("--set", "pairs", multiple=True, required=True,
+              help="Switch to change as TYPE=true|false (repeatable). "
+                   "Numbers come from `hap worksheet switch-types`.")
+@click.option("--dry-run", is_flag=True,
+              help="Show what would change on each worksheet, without saving.")
+@pass_context
+def worksheet_batch_edit_switch(ctx, worksheet_ids, pairs, dry_run):
+    """Change several feature switches on one or more worksheets.
+
+    The switches you do not name keep their current values. Turning the
+    批量操作 or 统计 parent on or off carries its children with it. Each
+    worksheet is read back after saving, and any switch that did not take
+    is reported rather than counted as success.
+    """
+    try:
+        changes = _parse_switch_changes(pairs)
+        session = ctx.get_session()
+        if dry_run:
+            preview = [
+                {**ws_mod.describe_switch(t), "state": v}
+                for t, v in changes.items()
+            ]
+            ctx.output(
+                {"worksheetIds": list(worksheet_ids), "changes": preview,
+                 "saved": False},
+                lambda d: click.echo(
+                    f"Would set {len(preview)} switch(es) on "
+                    f"{len(worksheet_ids)} worksheet(s): "
+                    + ", ".join(f"{c['path'] or c['type']}={c['state']}"
+                                for c in preview)),
+            )
+            return
+        results, failed_sheets = [], []
+        for wid in worksheet_ids:
+            try:
+                res = ws_mod.batch_edit_switch(session, wid, changes)
+            except Exception as exc:          # keep going across worksheets
+                results.append({"worksheetId": wid, "error": str(exc)})
+                failed_sheets.append(wid)
+                continue
+            results.append(res)
+            if res.get("failed"):
+                failed_sheets.append(wid)
+        payload = {"results": results,
+                   "worksheetsChanged": len(worksheet_ids) - len(failed_sheets),
+                   "worksheetsFailed": failed_sheets}
+
+        def _render(d):
+            click.echo(f"Changed {d['worksheetsChanged']} of "
+                       f"{len(worksheet_ids)} worksheet(s).")
+            for r in d["results"]:
+                if r.get("error"):
+                    click.echo(f"  {r['worksheetId']}: {r['error']}")
+                elif r.get("failed"):
+                    for t, info in r["failed"].items():
+                        click.echo(f"  {r['worksheetId']}: switch {t} is "
+                                   f"{info['actual']!r}, wanted "
+                                   f"{info['expected']!r}")
+        ctx.output(payload, _render)
+        if failed_sheets:
+            raise click.ClickException(
+                f"{len(failed_sheets)} worksheet(s) did not end up in the "
+                "requested state: " + ", ".join(failed_sheets))
+    except Exception as e:
+        ctx.handle_error(e)
+
+
 @worksheet.command("edit-switch")
 @click.argument("worksheet_id")
 @click.argument("switch_id", type=int)
 @click.argument("value", type=bool)
 @pass_context
 def worksheet_edit_switch(ctx, worksheet_id, switch_id, value):
-    """Edit a single feature switch."""
+    """Turn one feature switch on or off.
+
+    SWITCH_ID is a switch number from `hap worksheet switch-types`. The
+    new state is read back, so a change that did not take is reported as
+    an error instead of success. To change several switches, or the same
+    ones across many worksheets, use `worksheet batch-edit-switch`.
+    """
     try:
         session = ctx.get_session()
         data = ws_mod.edit_switch(session, worksheet_id, switch_id, value)

{hap_cli-0.8.28 → hap_cli-0.8.30}/hap_cli/core/app_creator/seed/executor.py

@@ -141,18 +141,12 @@ class _Resolver:
 
 def _record_attachment_cell(descriptors: list[dict[str, Any]]) -> dict[str, Any]:
     """Wrap general uploaded-file descriptors into a worksheet Attachment
-    (type 14) cell value (mirrors core upload.record_attachment_cell)."""
-    attachments = []
-    for i, d in enumerate(descriptors):
-        attachments.append({
-            "fileID": d.get("fileID"), "fileSize": d.get("fileSize"),
-            "serverName": d.get("serverName"), "filePath": d.get("filePath"),
-            "fileName": d.get("fileName"), "fileExt": d.get("fileExt"),
-            "originalFileName": d.get("originalFileName"), "key": d.get("key"),
-            "url": d.get("url"), "oldOriginalFileName": d.get("originalFileName"),
-            "index": i, "isEdit": False,
-        })
-    return {"attachmentData": [], "attachments": attachments, "knowledgeAtts": []}
+    (type 14) cell value. Delegates to the core wrapper so the seeded rows
+    carry exactly what any other CLI upload does — the local copy this
+    replaced silently dropped each file's download permission."""
+    from hap_cli.core.upload import record_attachment_cell
+
+    return record_attachment_cell(descriptors)
 
 
 def _upload_row_attachments(

{hap_cli-0.8.28 → hap_cli-0.8.30}/hap_cli/core/upload.py

@@ -209,6 +209,23 @@ def _iso(deadline: Optional[int]) -> Optional[str]:
     return datetime.fromtimestamp(deadline, tz=timezone.utc).astimezone().isoformat()
 
 
+def _download_permission(ext: str) -> dict[str, Any]:
+    """The per-file download permission the holder of the file is sent.
+
+    Whoever stores an attachment decides, per file, whether anyone but
+    the uploader may download it — an omitted flag used to read as "no",
+    so a file attached by the CLI answered 权限不足 for every colleague.
+    pd-openweb sets it in ``formatResponseData``
+    (src/components/UploadFiles/utils.js:202-206): non-pictures get
+    ``allowDown: true`` plus an empty ``docVersionID``, pictures get
+    neither because the viewer already treats a picture as downloadable
+    (UploadFiles/File.js:724). Mirrored verbatim, empty dict included.
+    """
+    if ext.lower() in _PICTURE_EXTS:
+        return {}
+    return {"allowDown": True, "docVersionID": ""}
+
+
 def upload_bytes(
     session: Session,
     *,
@@ -239,7 +256,7 @@ def upload_bytes(
     up = _post_to_store(upload_host(session), token, content,
                         original_name=original, ext=ext,
                         server_name=server_name)
-    return {
+    descriptor = {
         "fileID": generate_file_oid(),
         "fileSize": len(content),
         "etag": up.get("etag"),
@@ -256,6 +273,8 @@ def upload_bytes(
         # (public-bucket) address, which does not expire.
         "urlExpiresAt": _iso(url_expires_at(token.get("url"))),
     }
+    descriptor.update(_download_permission(ext))
+    return descriptor
 
 
 def source_descriptors(sources: "list[str] | tuple[str, ...]") -> list[dict[str, str]]:
@@ -324,4 +343,9 @@ def record_attachment_cell(descriptors: list[dict[str, Any]]) -> dict[str, Any]:
             "index": i,
             "isEdit": False,
         })
+        # Carried, not recomputed: the descriptor already decided this at
+        # upload time — see _download_permission().
+        for key in ("allowDown", "docVersionID"):
+            if key in d:
+                attachments[-1][key] = d[key]
     return {"attachmentData": [], "attachments": attachments, "knowledgeAtts": []}

{hap_cli-0.8.28 → hap_cli-0.8.30}/hap_cli/core/worksheet.py

@@ -4,7 +4,7 @@ import json
 import logging
 from typing import Any, Optional
 
-from hap_cli.core.session import Session
+from hap_cli.core.session import Session, APIError
 from hap_cli.core import worksheet_templates as wt
 
 logger = logging.getLogger("hap_cli.worksheet")
@@ -1681,14 +1681,237 @@ def get_switches(
     )
 
 
+# ── Feature switch catalog ───────────────────────────────────────────────
+#
+# GetSwitch answers bare numbers, which say nothing about what they
+# control, and several numbers share a label: 26 and 36 are both "复制",
+# 28 and 39 are both "删除", 23 and 32 are both "系统默认打印". Only the
+# group tells them apart, so the group travels with the name.
+#
+# Names, grouping and parent/child structure come from pd-openweb
+# ``src/pages/FormSet/containers/FunctionalSwitch/config.js``
+# (``listConfigStr``, ``worksheetSwitch`` / ``viewSwitch`` /
+# ``recordSwitch`` / ``approveSwitch``, ``batch``, ``statistics``).
+
+SWITCH_PARENT_BATCH = 25        # 视图 > 批量操作
+SWITCH_PARENT_STATISTICS = 500  # 工作表 > 统计
+
+_SWITCH_BATCH_CHILDREN = (24, 26, 23, 27, 28, 29)
+_SWITCH_STATISTICS_CHILDREN = (50, 51)
+
+SWITCH_TYPES: dict[int, dict[str, Any]] = {
+    # 工作表
+    10: {"name": "显示创建按钮", "group": "工作表"},
+    11: {"name": "讨论", "group": "工作表"},
+    12: {"name": "日志", "group": "工作表"},
+    13: {"name": "导入", "group": "工作表"},
+    14: {"name": "回收站", "group": "工作表"},
+    500: {"name": "统计", "group": "工作表"},
+    50: {"name": "公共", "group": "工作表", "parent": SWITCH_PARENT_STATISTICS},
+    51: {"name": "个人", "group": "工作表", "parent": SWITCH_PARENT_STATISTICS},
+    # 视图
+    20: {"name": "公开分享", "group": "视图"},
+    38: {"name": "嵌入链接", "group": "视图"},
+    21: {"name": "导出", "group": "视图"},
+    22: {"name": "快捷编辑", "group": "视图"},
+    25: {"name": "批量操作", "group": "视图"},
+    24: {"name": "编辑", "group": "视图", "parent": SWITCH_PARENT_BATCH},
+    26: {"name": "复制", "group": "视图", "parent": SWITCH_PARENT_BATCH},
+    23: {"name": "系统默认打印", "group": "视图", "parent": SWITCH_PARENT_BATCH},
+    27: {"name": "导出", "group": "视图", "parent": SWITCH_PARENT_BATCH},
+    28: {"name": "删除", "group": "视图", "parent": SWITCH_PARENT_BATCH},
+    29: {"name": "执行自定义动作", "group": "视图", "parent": SWITCH_PARENT_BATCH},
+    # 记录
+    30: {"name": "公开分享", "group": "记录"},
+    52: {"name": "内部访问链接", "group": "记录"},
+    32: {"name": "系统默认打印", "group": "记录"},
+    33: {"name": "附件下载", "group": "记录"},
+    34: {"name": "日志", "group": "记录"},
+    35: {"name": "讨论", "group": "记录"},
+    36: {"name": "复制", "group": "记录"},
+    39: {"name": "删除", "group": "记录"},
+    37: {"name": "重新创建", "group": "记录"},
+    # 审批
+    40: {"name": "系统字段", "group": "审批"},
+    41: {"name": "审批流转详情", "group": "审批"},
+}
+
+
+def describe_switch(switch_type: int) -> dict[str, Any]:
+    """Name, group and parent of a switch number; empty-ish when unknown."""
+    meta = SWITCH_TYPES.get(switch_type)
+    if not meta:
+        return {"type": switch_type, "name": "", "group": "", "path": ""}
+    parent = meta.get("parent")
+    path = " > ".join(
+        [meta["group"]]
+        + ([SWITCH_TYPES[parent]["name"]] if parent in SWITCH_TYPES else [])
+        + [meta["name"]]
+    )
+    out = {"type": switch_type, "name": meta["name"], "group": meta["group"],
+           "path": path}
+    if parent:
+        out["parent"] = parent
+    return out
+
+
+def annotate_switches(switches: Any) -> list[dict[str, Any]]:
+    """Return the switch list with name/group/path added to each entry."""
+    out = []
+    for entry in switches or []:
+        if not isinstance(entry, dict):
+            continue
+        merged = dict(entry)
+        merged.update(describe_switch(entry.get("type")))
+        out.append(merged)
+    return out
+
+
+def _switch_wire_fields(entry: dict[str, Any]) -> dict[str, Any]:
+    """The subset of a switch the page sends back on save.
+
+    ``FunctionalSwitch/index.jsx`` picks ``roleType/type/state/viewIds``;
+    ``displayFlowChart`` rides along where the switch carries one.
+    """
+    out = {k: entry[k] for k in ("roleType", "type", "state", "viewIds")
+           if k in entry}
+    if "displayFlowChart" in entry:
+        out["displayFlowChart"] = entry["displayFlowChart"]
+    out.setdefault("type", entry.get("type"))
+    out.setdefault("roleType", 0)
+    return out
+
+
+def _apply_switch_linkage(
+    current: list[dict[str, Any]], changes: dict[int, bool],
+) -> dict[int, bool]:
+    """Expand ``changes`` with the parent/child rules the page applies.
+
+    * Turning the 批量操作 (25) or 统计 (500) parent on or off carries all
+      of its children with it.
+    * When every 统计 child ends up off, the parent goes off too. The
+      批量操作 children deliberately do **not** close their parent — the
+      page only does this for 统计 (``index.jsx:108-123``; its comment
+      says 批量操作 but the code targets the statistics parent).
+    """
+    resolved = dict(changes)
+    for parent, children in (
+        (SWITCH_PARENT_BATCH, _SWITCH_BATCH_CHILDREN),
+        (SWITCH_PARENT_STATISTICS, _SWITCH_STATISTICS_CHILDREN),
+    ):
+        if parent in resolved:
+            for child in children:
+                resolved.setdefault(child, resolved[parent])
+    states = {e.get("type"): e.get("state") for e in current
+              if isinstance(e, dict)}
+    states.update(resolved)
+    if any(c in resolved for c in _SWITCH_STATISTICS_CHILDREN):
+        if not any(states.get(c) for c in _SWITCH_STATISTICS_CHILDREN):
+            resolved.setdefault(SWITCH_PARENT_STATISTICS, False)
+    return resolved
+
+
+def batch_edit_switch(
+    session: Session,
+    worksheet_id: str,
+    changes: dict[int, bool],
+    *,
+    verify: bool = True,
+) -> dict[str, Any]:
+    """Apply several switch changes to one worksheet in a single call.
+
+    This is what the feature-switch page itself uses
+    (``Worksheet/BatchEditSwitch``, ``src/api/worksheet.js:2346-2356``):
+    it submits the **whole** ``switchList``, so the untouched switches
+    have to be read first and sent back unchanged or they are lost.
+
+    Parent/child linkage is applied for you (see
+    :func:`_apply_switch_linkage`). With ``verify`` the new state is read
+    back and any switch that did not take is reported, because the
+    endpoint answers ``true`` regardless.
+
+    Returns ``{"worksheetId", "applied": {type: bool}, "failed": {type:
+    {"expected", "actual"}}}``.
+    """
+    current = get_switches(session, worksheet_id) or []
+    if not isinstance(current, list):
+        raise APIError(
+            f"could not read the switches of worksheet {worksheet_id}")
+    resolved = _apply_switch_linkage(current, {int(k): bool(v)
+                                               for k, v in changes.items()})
+    switch_list = []
+    for entry in current:
+        if not isinstance(entry, dict):
+            continue
+        wire = _switch_wire_fields(entry)
+        if entry.get("type") in resolved:
+            wire["state"] = resolved[entry["type"]]
+        switch_list.append(wire)
+    known = {e.get("type") for e in current if isinstance(e, dict)}
+    for stype, state in resolved.items():
+        if stype not in known:
+            switch_list.append({"type": stype, "state": state, "roleType": 0})
+    session.api_call(
+        "Worksheet", "BatchEditSwitch",
+        {"worksheetId": worksheet_id, "switchList": switch_list},
+    )
+    result: dict[str, Any] = {"worksheetId": worksheet_id,
+                              "applied": dict(resolved), "failed": {}}
+    if verify:
+        after = {e.get("type"): e.get("state")
+                 for e in (get_switches(session, worksheet_id) or [])
+                 if isinstance(e, dict)}
+        for stype, want in resolved.items():
+            if bool(after.get(stype)) != bool(want):
+                result["failed"][stype] = {"expected": want,
+                                           "actual": after.get(stype)}
+    return result
+
+
 def edit_switch(
     session: Session, worksheet_id: str, switch_id: int, value: bool,
 ) -> dict[str, Any]:
-    """Edit a single feature switch."""
-    return session.api_call(
-        "Worksheet", "EditSwitch",
-        {"worksheetId": worksheet_id, "state": switch_id, "value": value},
-    )
+    """Turn one feature switch on or off, and confirm it stuck.
+
+    ``EditSwitch`` names the boolean ``state`` and the switch number
+    ``type`` (pd-openweb ``src/api/worksheet.js:2330-2343``). This wrapper
+    used to send ``{state: <switch number>, value: <bool>}`` — the two
+    swapped, plus a ``value`` field the API does not define. The server
+    accepts that quietly and answers ``{"data": true}`` without changing
+    anything, so every edit looked like it worked and none of them did.
+
+    The switch's remaining fields are sent back as they are, the way the
+    page does (``_.pick(o, ['roleType', 'type', 'state', 'viewIds'])`` in
+    ``FormSet/containers/FunctionalSwitch/index.jsx``); dropping
+    ``viewIds`` would reset a view-scoped switch.
+
+    Because a stale payload fails silently rather than loudly, the new
+    state is read back and a write that did not land raises instead of
+    reporting success.
+    """
+    current = {}
+    for entry in get_switches(session, worksheet_id) or []:
+        if isinstance(entry, dict) and entry.get("type") == switch_id:
+            current = entry
+            break
+    payload: dict[str, Any] = {"worksheetId": worksheet_id,
+                               "type": switch_id, "state": value}
+    for field in ("roleType", "viewIds", "displayFlowChart"):
+        if field in current:
+            payload[field] = current[field]
+    payload.setdefault("roleType", 0)
+    result = session.api_call("Worksheet", "EditSwitch", payload)
+    applied = None
+    for entry in get_switches(session, worksheet_id) or []:
+        if isinstance(entry, dict) and entry.get("type") == switch_id:
+            applied = entry.get("state")
+            break
+    if bool(applied) != bool(value):
+        raise APIError(
+            f"switch {switch_id} is still {applied!r} after asking for "
+            f"{value!r} — the server reported success but did not apply it"
+        )
+    return result
 
 
 # ── Logs ─────────────────────────────────────────────────────────────────

{hap_cli-0.8.28 → hap_cli-0.8.30}/hap_cli/locale/messages.json

@@ -3969,6 +3969,18 @@
       "en": "Raw JSON array of new controls to append. Each item matches the AddWorksheetControls `controls` schema (same shape as `worksheet fields --raw` output).",
       "zh-Hans": "要追加的新字段原始 JSON 数组。每一项遵循 AddWorksheetControls 的 `controls` 结构（与 `worksheet fields --raw` 输出形态相同）"
     },
+    "worksheet batch-edit-switch": {
+      "en": "Change several feature switches on one or more worksheets.\n\nThe switches you do not name keep their current values. Turning the\n批量操作 or 统计 parent on or off carries its children with it. Each\nworksheet is read back after saving, and any switch that did not take\nis reported rather than counted as success.",
+      "zh-Hans": "批量修改一张或多张工作表的功能开关。\n\n未指定的开关保持原值。开启或关闭「批量操作」「统计」父开关时，其子项会随之联动。每张表保存后都会回读校验，未生效的开关会被报出而不是计为成功。"
+    },
+    "worksheet batch-edit-switch --dry-run": {
+      "en": "Show what would change on each worksheet, without saving.",
+      "zh-Hans": "仅显示每张工作表将要修改的内容，不实际保存。"
+    },
+    "worksheet batch-edit-switch --set": {
+      "en": "Switch to change as TYPE=true|false (repeatable). Numbers come from `hap worksheet switch-types`.",
+      "zh-Hans": "要修改的开关，格式为 TYPE=true|false（可重复指定）。开关号见 `hap worksheet switch-types`。"
+    },
     "worksheet chart": {
       "en": "Worksheet chart (统计图) management.",
       "zh-Hans": "工作表统计图管理"
@@ -4198,8 +4210,8 @@
       "zh-Hans": "跳过确认"
     },
     "worksheet edit-switch": {
-      "en": "Edit a single feature switch.",
-      "zh-Hans": "编辑单个功能开关"
+      "en": "Turn one feature switch on or off.\n\nSWITCH_ID is a switch number from `hap worksheet switch-types`. The\nnew state is read back, so a change that did not take is reported as\nan error instead of success. To change several switches, or the same\nones across many worksheets, use `worksheet batch-edit-switch`.",
+      "zh-Hans": "开启或关闭单个功能开关。\n\nSWITCH_ID 为开关号，可用 `hap worksheet switch-types` 查看。修改后会回读校验，未生效将报错而不是报成功。要一次改多个开关、或对多张工作表改同一批开关，请用 `worksheet batch-edit-switch`。"
     },
     "worksheet field-types": {
       "en": "Show field type constants as a reference table.\n\nLists the builder-style CODE names (Text/Number/Relation/...) you\npass via 'update-fields --fields' alongside the wire ControlTypeId\nand the legacy SCREAMING_SNAKE alias still accepted for backward\ncompatibility. Region maps to three ControlTypeIds — pass\n'regionLevel' in the field spec's `extra` to pick one.",
@@ -4705,9 +4717,17 @@
       "en": "Rule type: 0=interact 1=validate 2=lock",
       "zh-Hans": "规则类型：0=交互 1=校验 2=锁定"
     },
+    "worksheet switch-types": {
+      "en": "Show the feature switch numbers with their names and groups.",
+      "zh-Hans": "显示功能开关号及其对应的名称和分组。"
+    },
     "worksheet switches": {
-      "en": "Get feature switches for a worksheet.",
-      "zh-Hans": "获取工作表的功能开关"
+      "en": "Get feature switches for a worksheet.\n\nEach switch is listed with the name and group it has on the feature\nswitch page, because several numbers share a name and only the group\ntells them apart.",
+      "zh-Hans": "获取工作表的功能开关。每个开关会附带它在功能开关页面上的名称和分组——因为多个开关号共用同一名称，只有分组能区分它们。"
+    },
+    "worksheet switches --raw": {
+      "en": "Return the server's switch list unannotated.",
+      "zh-Hans": "返回服务端原始开关列表，不附加名称和分组。"
     },
     "worksheet update": {
       "en": "Update worksheet alias, description, sidebar name, or sidebar icon.\n\n`--name` and `--icon` change how the worksheet appears in the app\nsidebar; both require --app-id so the section can be resolved.",

hap_cli-0.8.30/hap_cli/tests/test_upload_allow_down.py

@@ -0,0 +1,118 @@
+"""Uploaded files must carry their download permission.
+
+Reported as "日程/讨论里别人上传的文件下载不了": a colleague opening a file
+attached through the CLI got *您权限不足，无法下载，请联系管理员或文件上传者*,
+while the same file attached from the web UI downloaded fine.
+
+The file store does not decide this — the *client* does, per file, when
+it hands the upload result to whatever will hold it. pd-openweb does it
+in ``formatResponseData`` (src/components/UploadFiles/utils.js:202-206):
+every non-picture upload gets ``allowDown: true`` and an empty
+``docVersionID``; pictures are left alone because the viewer treats them
+as downloadable anyway (UploadFiles/File.js:724). Every attachment
+payload the frontend sends — record comments, record attachment cells —
+is that same object, stringified.
+
+The CLI's uploaded-file descriptor never had the field, so anything it
+attached was stored with the permission unset and only the uploader
+could get the bytes back.
+"""
+
+import json
+from unittest.mock import MagicMock, patch
+
+import pytest
+
+from hap_cli.core import record as rec_mod
+from hap_cli.core import upload as up_mod
+from hap_cli.core.session import Session
+
+
+@pytest.fixture
+def session():
+    return Session(server_url="https://test.mingdao.com", auth_token="tok123")
+
+
+class _store:
+    """Stand in for the credential + store round trip of an upload."""
+
+    def __init__(self, ext=".pdf"):
+        self.ext = ext
+
+    def __enter__(self):
+        token = {"uptoken": "t", "key": f"doc/20260909/k{self.ext}",
+                 "serverName": "https://doc.mingdao.com/",
+                 "fileName": f"k{self.ext}",
+                 "url": f"https://doc.mingdao.com/doc/k{self.ext}"}
+        self._patches = [
+            patch.object(up_mod, "get_upload_token", return_value=token),
+            patch.object(up_mod, "upload_host",
+                         return_value="https://upload.qiniup.com/"),
+            patch.object(up_mod, "_post_to_store", return_value={
+                "key": f"doc/20260909/k{self.ext}", "etag": "e",
+                "bucket": "mdoc", "filePath": "doc/20260909/", "fileName": "k",
+                "fileExt": self.ext, "originalFileName": "spec",
+            }),
+        ]
+        for p in self._patches:
+            p.start()
+        return self
+
+    def __exit__(self, *exc):
+        for p in self._patches:
+            p.stop()
+        return False
+
+
+class TestDescriptorCarriesDownloadPermission:
+
+    def test_document_upload_is_downloadable(self, session):
+        with _store(".pdf"):
+            d = up_mod.upload_bytes(session, content=b"x", name="spec.pdf")
+        assert d["allowDown"] is True
+        assert d["docVersionID"] == ""
+
+    def test_archive_upload_is_downloadable(self, session):
+        # The reported file was a 13 MB .zip attached to a calendar comment.
+        with _store(".zip"):
+            d = up_mod.upload_bytes(session, content=b"x", name="proto.zip")
+        assert d["allowDown"] is True
+
+    def test_picture_upload_leaves_the_field_alone(self, session):
+        # Mirrors formatResponseData: pictures get neither field.
+        with _store(".png"):
+            d = up_mod.upload_bytes(session, content=b"x", name="cover.png")
+        assert "allowDown" not in d
+        assert "docVersionID" not in d
+
+
+class TestConsumersKeepThePermission:
+
+    _META = {"md.global": {"APPInfo": {"worksheetRowAppID": "row-app-id"}}}
+
+    def test_record_comment_attachment_keeps_it(self):
+        session = MagicMock()
+        session.get_global_meta.return_value = self._META
+        session.api_call.return_value = {"code": 1, "data": {"discussionId": "d1"}}
+        rec_mod.add_discussion(
+            session, "ws1", "row1", "here you go", app_id="app1",
+            attachments=[{"fileID": "o_1", "fileExt": ".zip", "allowDown": True,
+                          "docVersionID": ""}],
+        )
+        _, _, payload = session.api_call.call_args[0][:3]
+        assert json.loads(payload["attachments"])[0]["allowDown"] is True
+
+    def test_attachment_cell_keeps_it(self):
+        cell = up_mod.record_attachment_cell([
+            {"fileID": "o_1", "fileExt": ".zip", "originalFileName": "proto",
+             "allowDown": True, "docVersionID": ""},
+        ])
+        assert cell["attachments"][0]["allowDown"] is True
+        assert cell["attachments"][0]["docVersionID"] == ""
+
+    def test_attachment_cell_leaves_a_picture_alone(self):
+        cell = up_mod.record_attachment_cell([
+            {"fileID": "o_2", "fileExt": ".png", "originalFileName": "cover"},
+        ])
+        assert "allowDown" not in cell["attachments"][0]
+        assert "docVersionID" not in cell["attachments"][0]