hap-cli 0.8.25__tar.gz → 0.8.26__tar.gz

{hap_cli-0.8.25 → hap_cli-0.8.26}/PKG-INFO

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

{hap_cli-0.8.25 → hap_cli-0.8.26}/hap_cli/__init__.py

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

{hap_cli-0.8.25 → hap_cli-0.8.26}/hap_cli/commands/chart_cmd.py

@@ -123,7 +123,8 @@ def _resolve_app_id_for_filter(session, worksheet_id: str, extra: dict) -> str:
     return (data or {}).get("appId", "") if isinstance(data, dict) else ""
 
 
-@chart.command("create")
+@chart.command("create", help="Create a new chart on a worksheet. "
+                              "Chart spec shapes are in `hap guide chart`.")
 @click.argument("worksheet_id")
 @click.option("--name", required=True, help="Chart display name.")
 @click.option("--report-type", "report_type", type=int, required=True,
@@ -163,7 +164,9 @@ def chart_create(ctx, worksheet_id, name, report_type, app_type, app_id,
         ctx.handle_error(e)
 
 
-@chart.command("update")
+@chart.command("update", help="Update an existing chart's configuration. "
+                              "Read the config with `chart get`, edit it, submit "
+                              "it back — see `hap guide chart`.")
 @click.argument("report_id")
 @click.option("--app-id", "-a", required=True, help="Worksheet ID the chart attaches to.")
 @click.option("--name", required=True, help="Chart display name.")

{hap_cli-0.8.25 → hap_cli-0.8.26}/hap_cli/commands/record_cmd.py

@@ -374,16 +374,28 @@ def record_discussions(ctx, worksheet_id, row_id, page_size, page):
 @click.option("--app-id", default="", help="Application ID (auto-detected from the worksheet if omitted)")
 @click.option("--view-id", default="", help="View ID the discussion originates from")
 @click.option("--reply-id", default="", help="ID of the discussion being replied to")
+@click.option("--file", "-f", "files", multiple=True,
+              help="Attach a file to the comment (repeatable). Accepts a local "
+                   "path or an http(s) URL.")
 @pass_context
-def record_add_discussion(ctx, worksheet_id, row_id, message, app_id, view_id, reply_id):
+def record_add_discussion(ctx, worksheet_id, row_id, message, app_id, view_id, reply_id, files):
     """Add a discussion/comment to a record."""
     try:
         session = ctx.get_session()
+        attachments = None
+        if files:
+            from hap_cli.core import upload as up_mod
+            descriptors = up_mod.source_descriptors(list(files))
+            attachments = up_mod.upload_files(
+                session, descriptors,
+                worksheet_id=worksheet_id, app_id=app_id,
+            )
         data = rec_mod.add_discussion(
             session, worksheet_id, row_id, message,
             app_id=app_id, view_id=view_id, reply_id=reply_id,
+            attachments=attachments,
         )
-        ctx.output(data, lambda d: click.echo(f"Discussion added."))
+        ctx.output(data, lambda d: click.echo("Discussion added."))
     except Exception as e:
         ctx.handle_error(e)
 

{hap_cli-0.8.25 → hap_cli-0.8.26}/hap_cli/core/chart.py

@@ -44,6 +44,30 @@ from hap_cli.core.session import Session
 
 logger = logging.getLogger("hap_cli.chart")
 
+# A chart that is saved without a ``filter`` block saves successfully but
+# never renders: the statistics backend cannot scope the query and
+# ``report/getData`` answers status -1/0, which the UI paints as
+# "无法形成图表 / 构成要素不存在或已删除". The chart looks fine in the editor
+# (which supplies its own default scope) and only breaks after save — the
+# single most common way a hand-written ``chart create`` spec silently
+# produces a blank chart. The frontend's ``getNewReport`` always injects a
+# default time filter for a new worksheet chart; we mirror that here, but
+# default to ``rangeType: 0`` (全部时间 / all-time) instead of the UI's
+# "last 365 days" so the chart is never blank merely because the default
+# window excluded older data. Callers that supply their own ``filter``
+# (any non-empty dict) keep it untouched.
+_DEFAULT_CHART_FILTER: dict[str, Any] = {
+    "filterRangeId": "ctime",
+    "filterRangeName": "创建时间",
+    "rangeType": 0,
+    "rangeValue": None,
+    "today": False,
+    "dynamicFilter": {
+        "startType": 1, "startCount": 1, "startUnit": 1,
+        "endType": 1, "endCount": 1, "endUnit": 1,
+    },
+}
+
 
 def list_charts(
     session: Session,
@@ -218,6 +242,12 @@ def save_chart(
         split_control_id = body["split"].get("controlId")
         if split_control_id:
             body["splitId"] = split_control_id
+    # A chart with no filter block saves but never renders (blank after
+    # save — see _DEFAULT_CHART_FILTER). Inject an all-time default when the
+    # caller omitted it; a caller-supplied filter (any non-empty value) is
+    # left untouched so an explicit scope always wins.
+    if not body.get("filter"):
+        body["filter"] = dict(_DEFAULT_CHART_FILTER)
     # Resolve the chart's filter block before sending saveReportConfig.
     if isinstance(body.get("filter"), dict):
         body["filter"] = _ensure_filter_id(

{hap_cli-0.8.25 → hap_cli-0.8.26}/hap_cli/core/record.py

@@ -1089,6 +1089,7 @@ def add_discussion(
     app_id: str = "",
     view_id: str = "",
     reply_id: str = "",
+    attachments: Optional[list[dict[str, Any]]] = None,
 ) -> dict[str, Any]:
     """Add a discussion/comment to a record (sourceType 8 / worksheet-row).
 
@@ -1117,6 +1118,12 @@ def add_discussion(
     }
     if reply_id:
         data["replyId"] = reply_id
+    # ``attachments`` carries uploaded-file descriptors (from
+    # hap_cli.core.upload). The frontend WorkSheetCommenter sends them as a
+    # JSON-encoded string under ``attachments`` (see pd-openweb
+    # components/comment/commenter.jsx), so match that shape.
+    if attachments:
+        data["attachments"] = json.dumps(attachments, ensure_ascii=False)
     result = session.api_call("Discussion", "AddDiscussion", data)
     _ensure_discussion_ok(result, "add discussion")
     return result

hap_cli-0.8.26/hap_cli/guide/en/chart.md

@@ -0,0 +1,150 @@
+# Charts: create, update, place on a custom page
+
+`hap worksheet chart` builds a chart (report) on a worksheet; `hap custom-page`
+places charts into a custom page to form a dashboard. This topic covers how to
+shape a chart spec, the right way to edit a chart, and the most common trap —
+"saved fine, blank on the page".
+
+> 🚨 Core risk of this topic: **an under-specified chart still saves
+> successfully but renders blank**, showing "无法形成图表 / 构成要素不存在或已删除"
+> (cannot draw chart / component missing or deleted). It looks right in the
+> editor and only breaks after save. Root cause is in section 3 — the CLI now
+> backfills the missing filter scope, but you still need the model.
+
+## 1. What a chart is made of
+
+A chart hangs off **one worksheet** and needs at least three things:
+
+- **Data source**: the worksheet ID (`WORKSHEET_ID` in the command, `appId` in
+  the spec). Note HAP's chart spec calls the worksheet `appId` — it is NOT the
+  application ID. When editing a chart, do not fill this with the app ID.
+- **Dimension `xaxes`**: the grouping, e.g. by date (year) or by status. A
+  number chart (`report_type=10`) may have no dimension — just a value.
+- **Value `yaxisList`**: what to aggregate, e.g. sum of amount, record count.
+
+Plus a **required filter scope `filter`** (section 3).
+
+## 2. Chart types (reportType integer)
+
+Pass an integer to `--report-type`. **There is no 4** — an old API doc labels 4
+as the number chart, which is wrong; the number chart is **10**.
+
+| # | type | # | type |
+|---|------|---|------|
+| 1 | column | 10 | number |
+| 2 | line | 11 | bidirectional bar |
+| 3 | pie | 12 | scatter |
+| 5 | radar | 13 | word cloud |
+| 6 | funnel | 14 | gauge |
+| 7 | dual-axis | 15 | progress |
+| 8 | pivot | 16 | ranking |
+| 9 | region map | 17 | world map |
+
+## 3. Why "saved but blank": a missing filter
+
+A chart **saves successfully even with no `filter` block** (`create` still
+returns a `reportId`), but at render time the backend cannot scope the query and
+`report/getData` returns status -1/0, so the page shows "cannot draw chart". This
+is exactly the "fine in editor, blank after save" symptom: the editor supplies
+its own default scope, the stored spec does not.
+
+**The CLI now injects a default all-time filter when you omit `filter`**, so a
+plain `chart create` no longer goes blank. You only need to write `filter`
+yourself to **restrict the time range**:
+
+```jsonc
+"filter": {
+  "filterRangeId": "ctime",          // which date field to scope by; "ctime"=create time
+  "filterRangeName": "创建时间",
+  "rangeType": 0,                     // 0=all time; 8=this month; 18=last N days (with rangeValue)
+  "rangeValue": null,
+  "today": false
+}
+```
+
+Common `rangeType`: `0` all, `4` this week, `8` this month, `11` this quarter,
+`15` this year, `16` last year, `18` last N days (`rangeValue` = day count, e.g. 365).
+
+## 4. Create: `hap worksheet chart create`
+
+Pass the spec via `-j/--spec-json` (inline) or `-f/--spec-file` (file). Reference
+fields by **controlId**; read fields with `hap worksheet fields WS_ID`.
+
+**Column chart (amount by status):**
+
+```bash
+hap worksheet chart create WS_ID --name "Amount by status" --report-type 1 -j '{
+  "xaxes": {"controlId": "<status field>", "controlType": 11},
+  "yaxisList": [{"controlId": "<amount field>", "controlType": 6, "normType": 1}]
+}'
+```
+
+**Number chart (no dimension, one value):** `--report-type 10`, spec with only `yaxisList`.
+
+`yaxisList[].normType` aggregation: `1` sum, `2` max, `3` min, `4` avg, `5` count,
+`6` distinct count. Numeric fields default to `1`, text/option fields use `5`.
+Record count uses the special column
+`{"controlId": "record_count", "controlName": "记录数量", "controlType": 10000000, "normType": 5}`.
+
+**Date dimension**: add `particleSizeType` to `xaxes`: `5` year, `4` quarter,
+`3` month, `2` week, `1` day.
+
+**Condition filter** (not a time range but "status = done"): put a high-level
+`items` array in `filter`; the CLI stores it as a saved filter and backfills the id:
+
+```jsonc
+"filter": {"items": [{"controlId": "<status field>", "dataType": 11, "spliceType": 1,
+                      "filterType": 2, "values": ["<option key>"]}]}
+```
+
+When using `items`, pass `--app-id <application id>` (the app, not the worksheet)
+so the CLI can store the filter.
+
+## 5. Update: read the config, change a few params, submit back
+
+You don't rebuild a spec from scratch. **Read the current config, change what you
+need, submit the whole thing back:**
+
+```bash
+# 1. read the current config
+hap --json worksheet chart get REPORT_ID --app-id WS_ID > chart.json
+# 2. edit chart.json (name / normType / xaxes / add a filter, etc.)
+# 3. submit it back (update needs --app-id=worksheetId, --name, --report-type)
+hap worksheet chart update REPORT_ID --app-id WS_ID --name "New name" --report-type 1 -f chart.json
+```
+
+Extra fields `get` returns (`controls`, `account`, `createdDate`) can be passed
+back as-is; the server ignores them. `update` replaces the whole config, so edit
+on top of the read config — don't send only the fields you changed.
+
+## 6. Place on a custom page (dashboard)
+
+Once the chart exists and you have its `reportId`, add it as a component with
+`custom-page save`.
+
+```bash
+# page version: read it first with hap --json custom-page info PAGE_ID, pass the same value
+hap custom-page save PAGE_ID --version <current version> --components '[
+  {"type": 1, "value": "<reportId>", "worksheetId": "<WS_ID>",
+   "name": "Amount by status", "reportType": 1,
+   "config": {"objectId": "<32 hex chars>"},
+   "web": {"title": "", "titleVisible": false, "visible": true,
+           "layout": {"x": 0, "y": 0, "w": 24, "h": 10, "minW": 2, "minH": 4}},
+   "mobile": {"title": "", "titleVisible": false, "visible": true, "layout": null}}
+]'
+```
+
+Key points:
+
+- A chart component has `type` **1**, `value` = `reportId`, plus `worksheetId`
+  and `reportType`.
+- `config.objectId` is a unique 32-hex string per component (used by linked filters).
+- Layout is a **48-column grid**: `w` max 48; two charts per row at `w:24` each.
+- `--version` must equal the page's current version, else "自定义页面保存失败";
+  a successful save bumps version by 1.
+- `save` replaces the whole page: to keep existing components, read them via
+  `custom-page info` first and append the new one before submitting.
+
+For building a whole app with dashboards, prefer the `hap-cli-app-creator` skill:
+it generates full chart specs and page layouts from a friendly config. Use the
+commands above to hand-edit a single chart or add one chart to an existing page.

{hap_cli-0.8.25 → hap_cli-0.8.26}/hap_cli/guide/index.json

@@ -24,6 +24,14 @@
         "zh-Hans": "顺着通知或讨论找到它所属的那条应用记录。"
       }
     },
+    {
+      "name": "chart",
+      "title": {"en": "Charts and dashboards", "zh-Hans": "统计图与看板"},
+      "summary": {
+        "en": "Chart spec shape, editing a chart, placing it on a custom page. Miss a field and it saves fine but renders blank.",
+        "zh-Hans": "图表规格怎么写、怎么改图、怎么放进自定义页。写少了照样保存成功，页面却空白。"
+      }
+    },
     {
       "name": "skills",
       "title": {"en": "When to hand off", "zh-Hans": "什么时候转交专项 skill"},

hap_cli-0.8.26/hap_cli/guide/zh-Hans/chart.md

@@ -0,0 +1,133 @@
+# 统计图：创建、修改、放进自定义页
+
+`hap worksheet chart` 在工作表上建统计图（报表），`hap custom-page` 把图放进自定义页
+组成看板。本主题讲清楚图表规格该怎么写、改图的正确姿势，以及最容易踩的「保存成功、
+页面空白」这个坑。
+
+> 🚨 本主题核心风险：**图表规格写少了照样保存成功，但页面上一片空白**，提示
+> 「无法形成图表 / 构成要素不存在或已删除」。编辑界面能正常显示，一保存就没了。
+> 根因见下面第 3 节——现在 CLI 会自动补上缺失的筛选范围，但你仍要理解这个模型。
+
+## 1. 一张图由什么组成
+
+统计图挂在**一张工作表**上，最少需要三件东西：
+
+- **数据源**：工作表 ID（就是命令里的 `WORKSHEET_ID` / 规格里的 `appId`）。
+  注意 HAP 的图表规格里 `appId` 指的是**工作表**，不是应用；改图时不要填成应用 ID。
+- **维度 `xaxes`**：分组依据，比如按「日期(年)」「状态」分。数值图（`report_type=10`）
+  可以没有维度，只放数值。
+- **数值 `yaxisList`**：要统计的量，比如「金额求和」「记录数量」。
+
+此外还有一个**必须存在的筛选范围 `filter`**（见第 3 节）。
+
+## 2. 图表类型（reportType 整数）
+
+`--report-type` 传整数。**没有 4**——老接口文档把 4 标成数值图是错的，数值图是 **10**。
+
+| 值 | 类型 | 值 | 类型 |
+|----|------|----|------|
+| 1 | 柱状图 | 10 | 数值图 |
+| 2 | 折线图 | 11 | 双向条形图 |
+| 3 | 饼图 | 12 | 散点图 |
+| 5 | 雷达图 | 13 | 词云 |
+| 6 | 漏斗图 | 14 | 仪表盘 |
+| 7 | 双轴图 | 15 | 进度图 |
+| 8 | 透视表 | 16 | 排行榜 |
+| 9 | 行政区域地图 | 17 | 世界地图 |
+
+## 3. 「保存成功却空白」的根因：缺 filter
+
+一张图**没有 `filter` 块也能保存成功**（`create` 照样返回 `reportId`），但渲染时后端
+无法给查询定范围，`report/getData` 返回状态 -1/0，页面就显示「无法形成图表」。这正是
+「编辑正常、保存后空白」的原因：编辑器有自己的默认范围，存下来的规格却没有。
+
+**现在 CLI 会在你没写 `filter` 时自动补一个「全部时间」的默认范围**，所以直接
+`chart create` 已经不会空白。你只在想**限定时间范围**时才需要自己写 `filter`：
+
+```jsonc
+"filter": {
+  "filterRangeId": "ctime",          // 按哪个日期字段限定；"ctime"=创建时间
+  "filterRangeName": "创建时间",
+  "rangeType": 0,                     // 0=全部时间；8=本月；18=过去N天(配 rangeValue)
+  "rangeValue": null,
+  "today": false
+}
+```
+
+`rangeType` 常用值：`0` 全部、`4` 本周、`8` 本月、`11` 本季度、`15` 本年、
+`16` 上一年、`18` 今天之前 N 天（`rangeValue` 给天数，如 365）。
+
+## 4. 创建：`hap worksheet chart create`
+
+规格用 `-j/--spec-json`（内联）或 `-f/--spec-file`（文件）。字段引用一律用 **controlId**，
+取字段用 `hap worksheet fields WS_ID`。
+
+**柱图（按状态统计金额）：**
+
+```bash
+hap worksheet chart create WS_ID --name "各状态金额" --report-type 1 -j '{
+  "xaxes": {"controlId": "<状态字段>", "controlType": 11},
+  "yaxisList": [{"controlId": "<金额字段>", "controlType": 6, "normType": 1}]
+}'
+```
+
+**数值图（无维度，只放一个值）：** `--report-type 10`，规格只给 `yaxisList`。
+
+`yaxisList[].normType` 聚合方式：`1` 求和、`2` 最大、`3` 最小、`4` 平均、`5` 计数、
+`6` 去重计数。数值字段默认 `1`，文本/选项字段用 `5`。记录数量用特殊列
+`{"controlId": "record_count", "controlName": "记录数量", "controlType": 10000000, "normType": 5}`。
+
+**日期维度**在 `xaxes` 里加 `particleSizeType`：`5` 年、`4` 季、`3` 月、`2` 周、`1` 日。
+
+**带条件筛选**（不是时间范围，是「状态=已完成」这类）：在 `filter` 里放 `items`
+高层条件数组，CLI 会自动存成筛选器并回填 id：
+
+```jsonc
+"filter": {"items": [{"controlId": "<状态字段>", "dataType": 11, "spliceType": 1,
+                       "filterType": 2, "values": ["<选项key>"]}]}
+```
+
+用 `items` 时给 `--app-id 应用ID`（应用，不是工作表），CLI 才能存筛选器。
+
+## 5. 修改：先取原配置，改几个参数，再提交
+
+改图不用从头拼规格，**把原图配置读出来、改动要改的、整体提交回去**：
+
+```bash
+# 1. 取出当前配置
+hap --json worksheet chart get REPORT_ID --app-id WS_ID > chart.json
+# 2. 编辑 chart.json（改 name / normType / xaxes / 加 filter 等）
+# 3. 提交回去（update 必须带 --app-id=工作表ID、--name、--report-type）
+hap worksheet chart update REPORT_ID --app-id WS_ID --name "新名字" --report-type 1 -f chart.json
+```
+
+`get` 返回的 `controls` / `account` / `createdDate` 等多余字段可以原样带回，服务端会忽略。
+`update` 会整体替换这张图的配置，所以要在原配置基础上改，别只发你改的那几个字段。
+
+## 6. 放进自定义页（看板）
+
+图建好后拿到 `reportId`，再用 `custom-page save` 把它作为一个组件放到页面上。
+
+```bash
+# 页面版本号：先 hap --json custom-page info PAGE_ID 看当前 version，save 时传相同值
+hap custom-page save PAGE_ID --version <当前version> --components '[
+  {"type": 1, "value": "<reportId>", "worksheetId": "<WS_ID>",
+   "name": "各状态金额", "reportType": 1,
+   "config": {"objectId": "<32位十六进制随机串>"},
+   "web": {"title": "", "titleVisible": false, "visible": true,
+           "layout": {"x": 0, "y": 0, "w": 24, "h": 10, "minW": 2, "minH": 4}},
+   "mobile": {"title": "", "titleVisible": false, "visible": true, "layout": null}}
+]'
+```
+
+要点：
+
+- 图表组件 `type` 是 **1**，`value` 放 `reportId`，还要带 `worksheetId` 和 `reportType`。
+- `config.objectId` 每个组件一个唯一 32 位十六进制串（联动筛选靠它定位）。
+- 布局是 **48 栅格**：`w` 最大 48，一行放两张图各 `w:24`。
+- `--version` 必须等于页面当前 version，否则报「自定义页面保存失败」；存成功后 version +1。
+- `save` 是**整页替换**：要保留页上已有组件，先 `custom-page info` 读出 `components`
+  再把新组件追加进去一起提交。
+
+整体建应用带看板时优先用 `hap-cli-app-creator` skill，它按友好配置自动生成完整图表规格
+和页面布局，比手写规格省心。手改单张图、或往已有页加一张图时，用上面的命令即可。

{hap_cli-0.8.25 → hap_cli-0.8.26}/hap_cli/locale/messages.json

@@ -3930,8 +3930,8 @@
       "zh-Hans": "工作表统计图管理"
     },
     "worksheet chart create": {
-      "en": "Create a new chart on a worksheet.",
-      "zh-Hans": "在工作表上创建新的统计图"
+      "en": "Create a new chart on a worksheet. Chart spec shapes are in `hap guide chart`.",
+      "zh-Hans": "在工作表上创建新的统计图。图表规格的写法见 `hap guide chart`。"
     },
     "worksheet chart create --app-id": {
       "en": "Application id the worksheet lives under. Required only when the chart spec carries inline filter conditions (filter.items) — the CLI then mints a stored filter on the user's behalf. Auto-resolved via the worksheet info when omitted.",
@@ -4006,8 +4006,8 @@
       "zh-Hans": "每页条数（可选）"
     },
     "worksheet chart update": {
-      "en": "Update an existing chart's configuration.",
-      "zh-Hans": "更新已有统计图的配置"
+      "en": "Update an existing chart's configuration. Read the config with `chart get`, edit it, submit it back — see `hap guide chart`.",
+      "zh-Hans": "更新已有统计图的配置。用 `chart get` 读出配置、修改后整体提交回去——见 `hap guide chart`。"
     },
     "worksheet chart update --app-id": {
       "en": "Worksheet ID the chart attaches to.",
@@ -4245,6 +4245,10 @@
       "en": "Application ID (auto-detected from the worksheet if omitted)",
       "zh-Hans": "应用 ID（省略时自动从工作表检测）"
     },
+    "worksheet record add-discussion --file": {
+      "en": "Attach a file to the comment (repeatable). Accepts a local path or an http(s) URL.",
+      "zh-Hans": "为评论附加文件（可重复）。接受本地路径或 http(s) URL。"
+    },
     "worksheet record add-discussion --message": {
       "en": "Discussion message",
       "zh-Hans": "讨论内容"
@@ -4814,4 +4818,4 @@
       "zh-Hans": "应用到现有视图的高层视图 JSON（格式与 'view create --view-spec' 相同）。规格中表达的内容全部更新；其他设置保持不变。字段引用可为名称、别名或 controlId。提供时忽略其他选项"
     }
   }
-}
+}

{hap_cli-0.8.25 → hap_cli-0.8.26}/hap_cli/tests/test_chart.py

@@ -199,6 +199,39 @@ class TestSaveChart:
 
         assert "splitId" not in mock_post.call_args[1]["json"]
 
+    @patch("hap_cli.core.session.requests.post")
+    def test_default_filter_block_injected_when_absent(self, mock_post):
+        # A chart saved without a `filter` block saves fine but never
+        # renders (blank after save). save_chart must inject an all-time
+        # default filter so a hand-written spec with only axes still works.
+        sess = Session(server_url="https://t.com", auth_token="tok")
+        mock_post.return_value = _resp({"reportId": "r1"})
+
+        chart_mod.save_chart(
+            sess, app_id="ws1", name="Bare", report_type=1,
+            extra={"yaxisList": [{"controlId": "m1", "normType": 1}]},
+        )
+
+        flt = mock_post.call_args[1]["json"]["filter"]
+        assert flt["filterRangeId"] == "ctime"
+        assert flt["rangeType"] == 0  # all-time, never blank by default
+        assert "dynamicFilter" in flt
+
+    @patch("hap_cli.core.session.requests.post")
+    def test_caller_supplied_filter_is_not_overwritten(self, mock_post):
+        # An explicit filter scope always wins over the injected default.
+        sess = Session(server_url="https://t.com", auth_token="tok")
+        mock_post.return_value = _resp({"reportId": "r1"})
+
+        chart_mod.save_chart(
+            sess, app_id="ws1", name="Scoped", report_type=1,
+            extra={"filter": {"filterRangeId": "6a1", "rangeType": 8}},
+        )
+
+        flt = mock_post.call_args[1]["json"]["filter"]
+        assert flt["filterRangeId"] == "6a1"
+        assert flt["rangeType"] == 8
+
     @patch("hap_cli.core.session.requests.post")
     def test_extra_dict_is_merged_under_required_fields(self, mock_post):
         sess = Session(server_url="https://t.com", auth_token="tok")

{hap_cli-0.8.25 → hap_cli-0.8.26}/hap_cli/tests/test_core.py

@@ -10042,6 +10042,30 @@ class TestRecordDiscussions:
         with pytest.raises(APIError):
             rec_mod.add_discussion(session, "ws1", "row1", "hi", app_id="app1")
 
+    def test_add_discussion_encodes_attachments_as_json_string(self):
+        # The frontend sends attachments as a JSON-encoded string under
+        # `attachments`; a caller passing uploaded-file descriptors must
+        # get that exact wire shape (and no key when none are supplied).
+        session = MagicMock()
+        session.get_global_meta.return_value = self._META
+        session.api_call.return_value = {"code": 1, "data": {"discussionId": "d1"}}
+        descriptor = {"fileID": "f1", "originalFileName": "shot", "fileExt": ".png"}
+        rec_mod.add_discussion(
+            session, "ws1", "row1", "see attached", app_id="app1",
+            attachments=[descriptor],
+        )
+        _, _, payload = session.api_call.call_args[0][:3]
+        assert isinstance(payload["attachments"], str)
+        assert json.loads(payload["attachments"]) == [descriptor]
+
+    def test_add_discussion_omits_attachments_when_none(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", "no files", app_id="app1")
+        _, _, payload = session.api_call.call_args[0][:3]
+        assert "attachments" not in payload
+
     def test_remove_discussion_defaults_to_row_source_type(self):
         session = MagicMock()
         session.api_call.return_value = {"code": 1}

{hap_cli-0.8.25 → hap_cli-0.8.26}/hap_cli.egg-info/PKG-INFO

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

{hap_cli-0.8.25 → hap_cli-0.8.26}/hap_cli.egg-info/SOURCES.txt

@@ -146,11 +146,13 @@ hap_cli/core/v3/dispatcher.py
 hap_cli/core/v3/render.py
 hap_cli/core/v3/schema.py
 hap_cli/guide/index.json
+hap_cli/guide/en/chart.md
 hap_cli/guide/en/messages.md
 hap_cli/guide/en/overview.md
 hap_cli/guide/en/record.md
 hap_cli/guide/en/setup.md
 hap_cli/guide/en/skills.md
+hap_cli/guide/zh-Hans/chart.md
 hap_cli/guide/zh-Hans/messages.md
 hap_cli/guide/zh-Hans/overview.md
 hap_cli/guide/zh-Hans/record.md