dsh-origin-bridge 0.4.5 → 0.4.6

package/asciiio.py

@@ -10,6 +10,7 @@ This reader sniffs the delimiter, handles BOM and GBK, skips comment lines, and
 understands the common "long name / units" two-row header.
 """
 
+import codecs
 import csv
 import os
 import re
@@ -100,13 +101,72 @@ def read_metadata(head, k):
     return meta
 
 
+_BOMS = ((codecs.BOM_UTF32_LE, "utf-32-le"), (codecs.BOM_UTF32_BE, "utf-32-be"),
+         (codecs.BOM_UTF16_LE, "utf-16-le"), (codecs.BOM_UTF16_BE, "utf-16-be"),
+         (codecs.BOM_UTF8, "utf-8-sig"))
+
+# Some instrument software and Windows' "Unicode Text" save write UTF-16 with no
+# BOM at all. Half of every byte is then 0x00, which no other codec survives.
+_NUL_RATIO = 0.2
+
+
+def high_byte_runs(data):
+    """Lengths of maximal runs of bytes >= 0x80.
+
+    GBK-family text pairs them: one Chinese character is two high bytes. cp1252 /
+    latin-1 punctuation shows up as isolated singles — µ, °, ±, ·. That is the only
+    honest way to tell "温度/C" from "T/°C", because gb18030 will decode *both*
+    without raising and silently mojibake the second one into 祍/癈.
+    """
+    runs, cur = [], 0
+    for byte in data:
+        if byte >= 0x80:
+            cur += 1
+        elif cur:
+            runs.append(cur)
+            cur = 0
+    if cur:
+        runs.append(cur)
+    return runs
+
+
+def candidate_encodings(data):
+    order = list(ENCODINGS)
+    runs = high_byte_runs(data)
+    if runs and any(run % 2 for run in runs):
+        gb = order.index("gb18030")
+        singles = [enc for enc in ("cp1252", "latin-1") if enc in order]
+        for enc in singles:
+            order.remove(enc)
+        order[gb:gb] = singles
+    return order
+
+
 def read_text(path):
     """Decode a data file, trying the encodings that actually occur in labs."""
+    with open(path, "rb") as fh:
+        data = fh.read()
+    for bom, enc in _BOMS:
+        if data.startswith(bom):
+            body = data[len(bom):]
+            if enc == "utf-8-sig":
+                return body.decode("utf-8"), enc
+            try:
+                return body.decode(enc), enc
+            except UnicodeDecodeError as exc:
+                raise ValueError("文件带 %s 字节头但解不开：%s（%s）" % (enc, path, exc))
+    if len(data) >= 8 and data.count(b"\x00") / float(len(data)) > _NUL_RATIO:
+        for enc in ("utf-16-le", "utf-16-be"):
+            try:
+                text = data.decode(enc)
+            except UnicodeDecodeError:
+                continue
+            if "\x00" not in text:
+                return text, "%s（无 BOM，按 NUL 分布判断）" % enc
     last = None
-    for enc in ENCODINGS:
+    for enc in candidate_encodings(data):
         try:
-            with open(path, "r", encoding=enc, newline="") as fh:
-                return fh.read(), enc
+            return data.decode(enc), enc
         except (UnicodeDecodeError, LookupError) as exc:
             last = exc
     raise ValueError("无法解码 %s（试过 %s），最后错误：%s" % (path, "/".join(ENCODINGS), last))
@@ -126,13 +186,21 @@ def sniff_delimiter(rows):
         width = max(widths)
         if width < 2:
             continue
-        # score: how many columns are numeric in >80% of rows, tie-broken by width
+        # score: numeric columns first, then how many rows actually have that width,
+        # then FEWER columns. Ranking by width alone let a wrong comma split of a
+        # European "0,0;1,5" file beat the correct ';' because it made three columns.
+        consistent = sum(1 for g in grid if len(g) == width) / float(len(grid))
+        # The first row is a header candidate, so keep it out of the numeric ratio —
+        # otherwise a European "Zeit;Signal / 0,0;1,5" file scores every delimiter as
+        # non-numeric and the widest wrong split wins by default.
+        data = grid[1:] if len(grid) >= 2 else grid
         hits = 0
         for col in range(width):
-            vals = [g[col] for g in grid if len(g) > col]
-            if vals and sum(1 for v in vals if _is_number(v)) >= 0.8 * len(vals):
+            vals = [g[col] for g in data if len(g) > col]
+            if vals and sum(1 for v in vals if _is_number(v) or _CELL_DECCOMA.match(v.strip())) \
+                    >= 0.8 * len(vals):
                 hits += 1
-        score = (hits, width)
+        score = (hits, round(consistent, 3), -width)
         if score > best_score:
             best, best_score = delim, score
     if best is not None:
@@ -149,6 +217,92 @@ def sniff_delimiter(rows):
     raise ValueError("识别不出分隔符，也没有多列空白对齐的数据")
 
 
+_NUM_SRC = r"[+-]?(?:\d+\.?\d*|\.\d+)(?:[eEdD][+-]?\d+)?"
+# Lab exports decorate their numbers. Dropping a whole column because one cell is
+# decorated silently loses real data, so each shape below is recognised, unwrapped,
+# and reported as a warning instead of quietly skipped.
+_CELL_DL = re.compile(r"^(?:[<>]=?|≥|≤)\s*(%s)$" % _NUM_SRC)          # <0.001, >1e6
+_CELL_PM = re.compile(r"^(%s)\s*(?:±|\+/-|\+-)\s*\d*\.?\d+$" % _NUM_SRC)  # 2.5±0.1
+_CELL_UNIT = re.compile(r"^(%s)\s+\S+$" % _NUM_SRC)                   # "25 mV"
+# Two numbers sharing one cell (a space-aligned dump read with the wrong delimiter)
+# must not be dressed up as "number + unit", or the second one disappears quietly.
+_CELL_TWO = re.compile(r"^(%s)\s+(%s)\s*$" % (_NUM_SRC, _NUM_SRC))
+_CELL_FLAG = re.compile(r"^(%s)[*†‡°\s]+$" % _NUM_SRC)                # "2.5*"
+_CELL_THOUSANDS = re.compile(r"^[+-]?\d{1,3}(?:,\d{3})+(?:\.\d+)?$")  # "1,234.5"
+# European exports use the comma as the decimal mark, which only parses when the
+# field separator is something else (usually ';').
+_CELL_DECCOMA = re.compile(r"^[+-]?\d+,\d+$")                          # "0,5"
+_CELL_PAREN = re.compile(r"^\((%s)\)$" % _NUM_SRC)                     # "(500)"
+_MISSING_TOKENS = ("NA", "NAN", "-", "--", "NIL", "N/A", "NULL", "#N/A", "#DIV/0!")
+
+CELL_KIND_NOTES = {
+    "检测限": "第 %d 列 %r 有 %d 个检测限标记（例如 <0.001），已按边界值代入",
+    "不确定度": "第 %d 列 %r 有 %d 个 ± 写法（例如 2.5±0.1），已取中心值",
+    "带单位的数": "第 %d 列 %r 有 %d 个数值把单位写在同一格（例如 25 mV），已取数值",
+    "一格两个数": "第 %d 列 %r 有 %d 个格子里挤了两个数（空格分隔），已取第一个；"
+                  "若那本是两列，请把分隔符设为空格或改用制表符/CSV 再导",
+    "千分位": "第 %d 列 %r 有 %d 个千分位写法（例如 1,234.5），已去掉分隔逗号",
+    "小数逗号": "第 %d 列 %r 有 %d 个用逗号当小数点的写法（例如 0,5），已按小数解析",
+    "标记后缀": "第 %d 列 %r 有 %d 个带 * / † 标记的值，已按数值本身代入",
+    "括号负数": "第 %d 列 %r 有 %d 个用括号表示负数的写法（例如 (500)），已取负值",
+    "溢出": "第 %d 列 %r 有 %d 个数值超出浮点范围（例如 1e400），已按缺值处理",
+}
+
+
+def _to_float(text):
+    return float(text.replace("d", "E").replace("D", "E"))
+
+
+def _finite(value):
+    return value == value and value not in (float("inf"), float("-inf"))
+
+
+def cell_number(cell, decimal_comma=False):
+    """One cell -> (value or None, kind or None). kind is what we had to forgive.
+
+    Lab exports decorate their numbers: detection limits, uncertainties, a unit in
+    the same cell, thousands separators, outlier flags. Dropping the whole column
+    because one cell is decorated silently loses real data, so each shape is
+    recognised, unwrapped, and reported as a warning instead of quietly skipped.
+    """
+    text = str(cell).strip().strip('"').strip("'")
+    if not text or text.upper() in _MISSING_TOKENS:
+        return None, None
+    if decimal_comma and _CELL_DECCOMA.match(text):
+        value = _to_float(text.replace(",", "."))
+        return (value, "小数逗号") if _finite(value) else (None, "溢出")
+    try:
+        value = _to_float(text)
+    except ValueError:
+        pass
+    else:
+        return (value, None) if _finite(value) else (None, "溢出")
+    if _CELL_THOUSANDS.match(text):
+        value = _to_float(text.replace(",", ""))
+        return (value, "千分位") if _finite(value) else (None, "溢出")
+    m = _CELL_PAREN.match(text)
+    if m:
+        value = -abs(_to_float(m.group(1)))
+        return (value, "括号负数") if _finite(value) else (None, "溢出")
+    m = _CELL_TWO.match(text)
+    if m:
+        # Checked before "number + unit": "2.5 3.5" is a mis-split column, not a
+        # measurement with a unit, and calling it a unit loses the second number
+        # without ever telling anyone.
+        value = _to_float(m.group(1))
+        return (value, "一格两个数") if _finite(value) else (None, "溢出")
+    for rx, kind in ((_CELL_DL, "检测限"), (_CELL_PM, "不确定度"),
+                     (_CELL_UNIT, "带单位的数"), (_CELL_FLAG, "标记后缀")):
+        m = rx.match(text)
+        if m:
+            try:
+                value = _to_float(m.group(1))
+            except ValueError:
+                return None, None
+            return (value, kind) if _finite(value) else (None, "溢出")
+    return None, None
+
+
 def parse(path, comment=None):
     """Return {columns:[{name,units,values}], rows, delimiters, encoding, warnings}."""
     text, enc = read_text(path)
@@ -167,6 +321,10 @@ def parse(path, comment=None):
         grid = [next(csv.reader([ln], delimiter=delim)) for ln in kept]
 
     width = max(len(r) for r in grid)
+    # Deciding this here matters twice: the header/data split below and the cell
+    # reader both need to know whether a bare "0,5" is a European decimal number
+    # rather than two comma-separated fields.
+    decimal_comma = delim not in (",", None)
     if len({len(r) for r in grid}) > 1:
         warnings.append("行长度不一致，已按最宽 %d 列补齐" % width)
     grid = [r + [""] * (width - len(r)) for r in grid]
@@ -174,7 +332,8 @@ def parse(path, comment=None):
     # split leading header rows (rows that are mostly non-numeric) from data
     head, body = [], []
     for i, row in enumerate(grid):
-        numeric = sum(1 for c in row if _is_number(c))
+        numeric = sum(1 for c in row if _is_number(c)
+                      or (decimal_comma and _CELL_DECCOMA.match(c.strip())))
         if numeric >= max(1, int(0.6 * width)):
             body = grid[i:]
             break
@@ -205,22 +364,30 @@ def parse(path, comment=None):
     columns = []
     for c in range(width):
         vals = []
+        kinds = {}
         for row in body:
             cell = row[c].strip()
-            if not cell or cell.upper() in ("NA", "NAN", "-", "--"):
-                vals.append(None)
-                continue
-            try:
-                vals.append(float(cell.replace("d", "E").replace("D", "E")))
-            except ValueError:
-                vals.append(None)
+            value, kind = cell_number(cell, decimal_comma=decimal_comma)
+            if kind:
+                kinds[kind] = kinds.get(kind, 0) + 1
+            vals.append(value)
         live = sum(1 for v in vals if v is not None)
+        name = raw_names[c] or "col%d" % (c + 1)
+        # Report what we forgave even when the column ends up dropped: "全为空" alone
+        # hides that every cell was text (a row-label column) or overflowed.
+        for kind, count in sorted(kinds.items()):
+            warnings.append(CELL_KIND_NOTES[kind] % (c + 1, name, count))
         if live == 0:
-            warnings.append("第 %d 列 %r 解析后全为空，已跳过" % (c + 1, raw_names[c] or "col%d" % (c + 1)))
+            blanks = sum(1 for row in body if not row[c].strip())
+            if blanks == len(body):
+                continue  # a genuinely empty trailing column, nothing to explain
+            warnings.append("第 %d 列 %r 没有一个能读成数值（%d 行），已跳过；"
+                            "如果它是行标签/文本列属正常，要画它请选别的列"
+                            % (c + 1, name, len(body)))
             continue
         columns.append({
             "index": c + 1,
-            "name": raw_names[c] or "col%d" % (c + 1),
+            "name": name,
             "units": units[c] or "",
             "values": vals,
             "numeric_rows": live,

package/fixtures/l_cp1252_micro.dat

@@ -0,0 +1,3 @@
+T/V	s/�s
+1	2
+3	4

package/fixtures/m_cp1252_degree.dat

@@ -0,0 +1,3 @@
+T/�C	y
+25	1.2
+30	1.4

package/fixtures/make_fixtures.py

@@ -20,6 +20,21 @@ FILES = {
     "g_na_and_D.dat": ("utf-8", "X\tY\tZ\n0\t1.5D+00\tNA\n1\t2.5d+00\t-\n2\tNA\t3.1\n"),
     "h_gbk_units.dat": ("gbk", "温度\t电导\n25.0\t1.23\n30.0\t1.45\n"),
     "i_bom_utf8.csv": ("utf-8-sig", "波长,吸光度\n200,0.12\n210,0.34\n"),
+    # Encodings and decorated numbers that come out of real instruments.
+    "j_utf16_bom.dat": ("utf-16", "X\tY\n0\t1.5\n1\t2.5\n"),
+    "k_utf16_nobom.dat": ("utf-16-le", "X\tY\n0\t1.5\n1\t2.5\n"),
+    "l_cp1252_micro.dat": ("cp1252", "T/V\ts/µs\n1\t2\n3\t4\n"),
+    "m_cp1252_degree.dat": ("cp1252", "T/°C\ty\n25\t1.2\n30\t1.4\n"),
+    "n_detection_limit.dat": ("utf-8", "x\ty\n1\t<0.001\n2\t0.5\n3\t>1e6\n"),
+    "o_plusminus.dat": ("utf-8", "x\ty\n1\t2.5±0.1\n2\t3.5 +/- 0.2\n"),
+    "p_unit_in_cell.dat": ("utf-8", "x\ty\n1\t25 mV\n2\t30 mV\n"),
+    "q_thousands.csv": ("utf-8", "time,signal\n0,\"1,234.5\"\n1,\"2,345.6\"\n"),
+    "r_decimal_comma.txt": ("utf-8", "Zeit;Signal\n0,0;1,5\n1,0;2,5\n"),
+    "s_star_flag.dat": ("utf-8", "x\ty\n1\t2.5*\n2\t3.5\n"),
+    "t_paren_negative.dat": ("utf-8", "x\ty\n1\t(500)\n2\t250\n"),
+    "u_overflow.dat": ("utf-8", "x\ty\n1\t1e400\n2\t1.2e308\n"),
+    "v_row_labels.dat": ("utf-8", "ch\tV\tI\nCh1\t1.0\t2.0\nCh2\t3.0\t4.0\n"),
+    "w_two_numbers_cell.csv": ("utf-8", "x,y\n1,\"2.5 3.5\"\n2,\"4.0 5.0\"\n"),
 }
 
 

package/fixtures/n_detection_limit.dat

@@ -0,0 +1,4 @@
+x	y
+1	<0.001
+2	0.5
+3	>1e6

package/fixtures/o_plusminus.dat

@@ -0,0 +1,3 @@
+x	y
+1	2.5±0.1
+2	3.5 +/- 0.2

package/fixtures/p_unit_in_cell.dat

@@ -0,0 +1,3 @@
+x	y
+1	25 mV
+2	30 mV

package/fixtures/q_thousands.csv

@@ -0,0 +1,3 @@
+time,signal
+0,"1,234.5"
+1,"2,345.6"

package/fixtures/r_decimal_comma.txt

@@ -0,0 +1,3 @@
+Zeit;Signal
+0,0;1,5
+1,0;2,5

package/fixtures/s_star_flag.dat

@@ -0,0 +1,3 @@
+x	y
+1	2.5*
+2	3.5

package/fixtures/t_paren_negative.dat

@@ -0,0 +1,3 @@
+x	y
+1	(500)
+2	250

package/fixtures/u_overflow.dat

@@ -0,0 +1,3 @@
+x	y
+1	1e400
+2	1.2e308

package/fixtures/v_row_labels.dat

@@ -0,0 +1,3 @@
+ch	V	I
+Ch1	1.0	2.0
+Ch2	3.0	4.0

package/fixtures/w_two_numbers_cell.csv

@@ -0,0 +1,3 @@
+x,y
+1,"2.5 3.5"
+2,"4.0 5.0"

package/originlab.py

@@ -24,6 +24,7 @@ import asciiio
 import binio
 
 MISSING_SENTINEL = -1.23456789e-300
+_INF = float("inf")
 # Verified against OriginPro 2024 SR1 by exporting each code and looking at the
 # image: 200 line, 201 symbol, 202 line+symbol, 203 column.
 PLOT_CODES = {"line": 200, "scatter": 201, "line_symbol": 202, "column": 203}
@@ -396,13 +397,20 @@ def _graph_of(op, ref):
 
 
 def sanitize(obj):
-    """Origin encodes missing numbers as -1.23456789e-300; never leak that upstream."""
+    """Origin encodes missing numbers as -1.23456789e-300; never leak that upstream.
+
+    +-Infinity belongs here too. json.dumps happily writes a bare ``Infinity``,
+    which is not valid JSON, so a single overflowed fit parameter or a ``1e400``
+    read out of a file would make the whole tool response unparseable for a strict
+    client — the failure looks like "the bridge is broken", not like one bad number.
+    """
     if isinstance(obj, dict):
         return {k: sanitize(v) for k, v in obj.items()}
     if isinstance(obj, (list, tuple)):
         return [sanitize(v) for v in obj]
-    if isinstance(obj, float) and (obj != obj or abs(obj - MISSING_SENTINEL) < 1e-310):
-        return None
+    if isinstance(obj, float):
+        if obj != obj or obj in (_INF, -_INF) or abs(obj - MISSING_SENTINEL) < 1e-310:
+            return None
     return obj
 
 
@@ -510,6 +518,127 @@ def _verify_file(path, must_be_image=True, min_bytes=1024):
     return {"ok": ok, "path": path, "bytes": size, "dimensions": dims, "head": head[:8].hex()}
 
 
+# Measured 2026-10-12 against the live bridge: Origin's export filter treats '#'
+# in the filename as a numbering token — asking for 100%_a#b[1].png produced
+# 100%_a1b[1].png (the '#' became "1"), while '%', '[]', spaces and 中文 survive.
+# A trailing space or dot in any component is stripped by Win32 before creation,
+# so isfile() on the literal path we asked for says False while the bytes really
+# are on disk. And a path that carries no real filename (".", "   ", ".png")
+# used to resolve into the process CWD and come back as a dotfile named ".png" —
+# littering the plugin install folder.
+_HASH_RUN = re.compile(r"#+")
+
+
+def _clean_component(part):
+    stripped = part.rstrip(" .")
+    return stripped if stripped else part
+
+
+def _norm_target(path, fmt=None):
+    """Absolute, expanduser'd, and normalised the way Windows will do it anyway."""
+    raw = str(path or "").strip()
+    if not raw:
+        return ""
+    target = os.path.abspath(os.path.expanduser(raw))
+    drive, rest = os.path.splitdrive(target)
+    target = drive + os.sep.join(_clean_component(p) for p in rest.split(os.sep))
+    base = os.path.basename(target)
+    if not base.split(".")[0].strip(" ."):
+        raise OriginError("bad_path", "路径里没有真正的文件名：%r" % path,
+                          ["给完整的文件名，例如 D:\\out\\图.png",
+                           "只想用默认目录就别传 path"])
+    if os.path.isdir(target):
+        raise OriginError("bad_path", "path 指向一个目录，不是文件：%s" % target,
+                          ["把文件名补上，例如 %s\\figure.png" % target])
+    stem, ext = os.path.splitext(base)
+    if fmt and ext.lower().lstrip(".") != fmt:
+        target = os.path.splitext(target)[0] + "." + fmt
+    return target
+
+
+def _prepare_write(target):
+    """Make the directory and refuse up front what Origin cannot write anyway."""
+    folder = os.path.dirname(target) or "."
+    try:
+        os.makedirs(folder, exist_ok=True)
+    except OSError as exc:
+        raise OriginError("bad_path", "输出目录建不出来：%s（%s）" % (folder, exc.strerror or exc),
+                          ["盘符存在吗（例如 Q: 未映射）",
+                           "网络路径要用 \\服务器\\共享\\… 且当前身份有写权限",
+                           "路径里去掉尾部空格或点，Windows 会静默截掉"])
+    if os.path.isdir(target):
+        raise OriginError("bad_path", "path 是一个目录，不是文件：%s" % target,
+                          ["把文件名补上，例如 %s\\figure.png" % target])
+    if os.path.exists(target) and not os.access(target, os.W_OK):
+        raise OriginError("target_locked", "目标文件已存在且不可写（只读或被占用）：%s" % target,
+                          ["去掉只读属性或换文件名", "文件可能正被 Origin/图片查看器占用，先关掉"])
+
+
+def _rel_note(path):
+    """A relative path resolves against the process CWD, which under dsh is the
+    plugin's own install folder — worth saying so instead of littering it."""
+    raw = str(path or "").strip()
+    if raw and not os.path.isabs(os.path.expanduser(raw)):
+        return ("path 是相对路径，已解析到进程工作目录 %s；dsh 下那是插件安装目录，"
+                "交付文件请给绝对路径" % os.path.abspath(raw))
+    return None
+
+
+def _lift_warnings(out):
+    """Nested export/project notes belong at the top of the result too.
+
+    A model reads the head of a tool result first; if Origin rewrote a filename
+    and the note only lives under out["export"]["warnings"], the delivery looks
+    clean when it is not.
+    """
+    lifted = list(out.get("warnings") or [])
+    for part in ("export", "project"):
+        sub = out.get(part) or {}
+        for note in (sub.get("warnings") or []):
+            if note not in lifted:
+                lifted.append(note)
+    if lifted:
+        out["warnings"] = lifted
+    return out
+
+
+def _written_or_mangled(target):
+    """Locate what Origin actually wrote; rename it back when its filter rewrote '#'."""
+    if os.path.isfile(target):
+        return target, None
+    folder = os.path.dirname(target) or "."
+    stem, ext = os.path.splitext(os.path.basename(target))
+    if not _HASH_RUN.search(stem):
+        return None, None
+    # Origin replaced every run of '#' with digits; turn the requested name into a
+    # pattern and accept a single unambiguous match, then rename it back.
+    pattern = []
+    i = 0
+    while i < len(stem):
+        m = _HASH_RUN.match(stem, i)
+        if m:
+            pattern.append(r"\d+")
+            i = m.end()
+        else:
+            pattern.append(re.escape(stem[i]))
+            i += 1
+    rx = re.compile("^%s%s$" % ("".join(pattern), re.escape(ext)))
+    try:
+        hits = [n for n in os.listdir(folder) if rx.match(n)]
+    except OSError:
+        hits = []
+    if len(hits) != 1:
+        return None, None
+    cand = os.path.join(folder, hits[0])
+    try:
+        os.replace(cand, target)
+        return target, ("Origin 的导出滤镜把文件名里的 # 换成了编号（实际写成 %s），已改名回你要求的路径；"
+                        "要避免就别说文件名里带 #" % hits[0])
+    except OSError as exc:
+        return cand, ("Origin 把文件名写成了 %s（# 被换成编号），改名回原路径失败：%s"
+                      % (hits[0], exc.strerror or exc))
+
+
 # ---------------------------------------------------------------- operations
 
 
@@ -1441,18 +1570,24 @@ def export(graph, path="", fmt="png", width=1600):
         # No path is not an invitation to write into the process CWD: under dsh
         # that is the plugin's node_modules folder. Origin's User Files wins.
         folder = _safe(lambda: op.path("u"), "") or os.getcwd()
-        target_path = os.path.abspath(path or os.path.join(folder, "%s.%s" % (name, fmt)))
-        if os.path.splitext(target_path)[1].lower().lstrip(".") != fmt:
-            target_path = os.path.splitext(target_path)[0] + "." + fmt
-        os.makedirs(os.path.dirname(target_path) or ".", exist_ok=True)
+        raw_path = str(path or "").strip()
+        target_path = _norm_target(raw_path or os.path.join(folder, "%s.%s" % (name, fmt)), fmt)
+        _prepare_write(target_path)
         _safe(lambda: page.save_fig(target_path))
-        if not os.path.isfile(target_path):
+        written, note = _written_or_mangled(target_path)
+        if not written:
             raise OriginError("export_failed", "Origin 说导出了但文件不存在：%s" % target_path,
-                              ["路径必须可写且最好是 ASCII", "先 origin_inspect 确认图页非空"])
-        info = _verify_file(target_path)
+                              ["路径必须可写；UNC/未映射盘先排除",
+                               "先 origin_inspect 确认图页非空"])
+        info = _verify_file(written)
         if not info["ok"]:
             raise OriginError("export_suspect", "导出文件过小或文件头不对：%s" % info,
                               ["图页可能是空的：origin_inspect 看 plots 数"])
+        if note:
+            info["warnings"] = [note]
+        rel = _rel_note(path)
+        if rel:
+            info["warnings"] = (info.get("warnings") or []) + [rel]
         return sanitize(dict(info, channel="originpro.save_fig", graph=name))
 
     return call(run)
@@ -1463,8 +1598,8 @@ def _save_project(path=""):
     def run():
         global _project_file
         op = origin()
-        if path:
-            target = os.path.abspath(path)
+        if str(path).strip():
+            target = _norm_target(path)
         elif _project_file:
             target = _project_file
         else:
@@ -1474,7 +1609,7 @@ def _save_project(path=""):
                                   time.strftime("origin_bridge_%Y%m%d_%H%M%S.opju"))
         if not target.lower().endswith((".opju", ".opj")):
             target += ".opju"
-        os.makedirs(os.path.dirname(target) or ".", exist_ok=True)
+        _prepare_write(target)
 
         # op.save(file) is Origin's Save-As: once the project is bound to that
         # file, saving the same name again returns False without meaning failure.
@@ -1484,10 +1619,12 @@ def _save_project(path=""):
         mtime_before = os.path.getmtime(target) if os.path.isfile(target) else 0
         ok = bool(op.save()) if same_file else bool(op.save(target))
 
-        if not os.path.isfile(target):
+        written, _note = _written_or_mangled(target)
+        if not written:
             raise OriginError("save_failed", "保存工程失败：%s (ok=%s)" % (target, ok),
-                              ["确认目录可写、路径为 ASCII",
+                              ["确认目录可写；中文、空格、超长路径实测都能存，UNC/未映射盘不行",
                                "换一个不冲突的文件名，或省略 path 用自动时间戳命名"])
+        target = written
         fresh = os.path.getmtime(target) > mtime_before
         if not ok and not (same_file and not fresh):
             raise OriginError("save_failed", "保存工程失败：%s (ok=%s)" % (target, ok),
@@ -1647,7 +1784,7 @@ def figure(source="", columns=None, x=1, y=1, plot_type="line", title="",
     out["proof_level"] = "verified" if deliver.get("image") else "readback_only"
     out["steps"] = steps
     out["duration_ms"] = int((time.time() - t0) * 1000)
-    return out
+    return _lift_warnings(out)
 
 
 # ------------------------------------------------------- matrix / 3D / heat maps
@@ -1711,22 +1848,27 @@ def matrix_plot(rows, template="heat_map", book_name="", graph_name="",
             out["export"] = _export_on_com_thread(gr, export_path)
         if project_path:
             out["project"] = _save_project(project_path)
-        return sanitize(out)
+        return sanitize(_lift_warnings(out))
 
     return call(run)
 
 
 def _export_on_com_thread(page, path):
     """export() body, for reuse inside another run() that is already on the thread."""
-    ext = os.path.splitext(path)[1].lower() or ".png"
-    target = os.path.abspath(path)
-    os.makedirs(os.path.dirname(target) or ".", exist_ok=True)
+    target = _norm_target(path)
+    if not os.path.splitext(target)[1]:
+        target += ".png"
+    ext = os.path.splitext(target)[1].lower()
+    _prepare_write(target)
     _safe(lambda: page.save_fig(target))
-    if not os.path.isfile(target):
+    written, note = _written_or_mangled(target)
+    if not written:
         raise OriginError("export_failed", "矩阵图导出没落盘：%s" % target)
-    info = _verify_file(target, must_be_image=ext in IMAGE_MAGIC)
+    info = _verify_file(written, must_be_image=ext in IMAGE_MAGIC)
     if not info["ok"]:
         raise OriginError("export_suspect", "导出文件异常：%s" % info)
+    if note:
+        info["warnings"] = [note]
     return info
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-origin-bridge",
-  "version": "0.4.5",
+  "version": "0.4.6",
   "description": "在对话里驱动本机 Origin / OriginPro：导入 .dat/.csv/.txt 与电化学工作站 .bin，绘图、拟合、改样式，导出图片并保存可在 Origin 里继续编辑的 .opju 工程。dsh 插件，27 个 MCP 工具。第三方工具，与 OriginLab Corporation 无关联、未获认可。",
   "license": "MIT",
   "type": "module",
@@ -14,9 +14,6 @@
   "homepage": "https://github.com/caob23/dsh-origin-bridge#readme",
   "main": "./index.js",
   "icon": "./icon.svg",
-  "dependencies": {
-    "@deepseek-ai/dsh-mcp-client": ">=0.1.0-rc.1"
-  },
   "bin": {
     "origin-bridge": "bin/origin-bridge.mjs"
   },
@@ -53,6 +50,8 @@
     "fixtures/*.txt",
     "smoke_test.py",
     "edge_checks.py",
+    "test_paths.py",
+    "test_json_safety.py",
     "test_asciiio.py",
     "test_binio.py",
     "test_lifecycle.py",
@@ -78,5 +77,13 @@
     "opju",
     "scientific-plotting",
     "electrochemistry"
-  ]
+  ],
+  "peerDependencies": {
+    "@deepseek-ai/dsh-mcp-client": "0.2.0-rc.2 || >=0.1.0-rc.1 <1"
+  },
+  "peerDependenciesMeta": {
+    "@deepseek-ai/dsh-mcp-client": {
+      "optional": true
+    }
+  }
 }