ht32-code-mcp 0.4.2__tar.gz → 0.4.4__tar.gz

{ht32_code_mcp-0.4.2 → ht32_code_mcp-0.4.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: ht32-code-mcp
-Version: 0.4.2
+Version: 0.4.4
 Summary: HT32 Code MCP Server - 为 AI Agent 提供 HT32 函数库/例程/模板检索与工程导出
 License-Expression: MIT
 Keywords: firmware,holtek,ht32,mcp,mcu

{ht32_code_mcp-0.4.2 → ht32_code_mcp-0.4.4}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "ht32-code-mcp"
-version = "0.4.2"
+version = "0.4.4"
 description = "HT32 Code MCP Server - 为 AI Agent 提供 HT32 函数库/例程/模板检索与工程导出"
 requires-python = ">=3.11"
 license = "MIT"

{ht32_code_mcp-0.4.2 → ht32_code_mcp-0.4.4}/src/ht32_code_mcp/materializer.py

@@ -4,7 +4,7 @@
 1. 先校验（schema/来源 sha256/授权/唯一 recipe/配置闭包）再创建 staging；
 2. XML 只做限定 target/节点路径/预期命中数的声明式改写，保留原字节其余部分；
 3. 内存一处裁决（recipe memory model）、多处一致：Cpu/IROM/IRAM/OCR_RVCT4/OCR_RVCT9/
-   umfTarg/FlashDriverDll -FL 全部由同一模型派生；
+   FlashDriverDll -FL 由同一模型派生；umfTarg 保持官方生成工程口径（0）；
 4. manifest 四件套（project-lock/files/deviations/validation.json）随产物发布；
 5. staging -> 同文件系统原子重命名；目标已存在则失败，不合并覆盖；
 6. ZIP 写 .part 校验后原子发布；失败不留公开半成品。
@@ -622,6 +622,14 @@ class Materializer:
         lib_src = pkg / ("libraries" if kernel == "m4" else "library")
         lib_dst = stage / "library"
         self._copy_official_tree(lib_src, lib_dst, manifest_files, origin["package"])
+        # 1b) middlewares：仅 M4 官方包自带（USB drivers/classes、FatFS、I2C app library）。
+        #     官方工程 IncludePath 直接引用 middlewares\usb_drivers\inc，必须随工程携带；
+        #     M0+/M3 官方包无此目录。
+        if kernel == "m4":
+            mid_src = pkg / "middlewares"
+            if mid_src.is_dir():
+                self._copy_official_tree(mid_src, stage / "middlewares",
+                                         manifest_files, origin["package"])
         # 2) utilities：官方构建资产；host 工具（gsar/eLink32pro）默认随包
         if kernel == "m4":
             board_asset = r.recipe.get("board_asset_dir") or board_dir
@@ -698,6 +706,15 @@ class Materializer:
                 deviations.append(dict(kind="example-doc-excluded", detail=sf["file_name"],
                                        reason="官方示例文档，非构建资产"))
                 continue
+            # 工程自有资产（system 时钟/看门狗配置、startup 栈配置）必须落在 project/ 内。
+            # 写进 library/ 会在用户整体更新库时被官方原档覆盖，工程配置静默失效。
+            if sf["kind"] in ("cmsis", "mdk") and \
+                    sf["target_path"].replace("/", "\\").startswith("..\\..\\library\\"):
+                raise ContractError(
+                    RECIPE_MISMATCH,
+                    f"{cfg['config_id']}: 工程自有文件 {sf['file_name']} 指向 library/"
+                    f"（{sf['target_path']}）；system/startup 必须随工程放在 "
+                    f"project/Src_USER 或 project/MDK_ARMv5")
             target_file = _resolve_target(stage, sf["target_path"])
             in_tree = target_file is None
             base_name = Path(sf["origin"]).name
@@ -831,6 +848,28 @@ class Materializer:
                               _remap_includepath, text)
         if n_inc:
             edits.append(f"IncludePath remapped ({n_inc} nodes scanned)")
+        # b2) M4 官方 middlewares：官方板级模板本身不含 middlewares，官方逐 IC 生成工程
+        #     会追加 usb_drivers/inc；随工程携带 middlewares/ 后同步补进 IncludePath
+        #     （F493x5 的目录名是 usbd_drivers）。
+        mw_incs: list[str] = []
+        if kernel == "m4":
+            mw_root = pkg / "middlewares"
+            for drv in ("usb_drivers", "usbd_drivers"):
+                if (mw_root / drv / "inc").is_dir():
+                    mw_incs.append(f"..\\..\\middlewares\\{drv}\\inc")
+        if mw_incs:
+            def _add_middleware_incs(m):
+                body = m.group(2)
+                if not body.strip():
+                    return m.group(0)
+                entries = body.split(";")
+                for inc in mw_incs:
+                    if inc not in entries:
+                        entries.append(inc)
+                        edits.append(f"IncludePath += {inc} (official middlewares)")
+                return m.group(1) + ";".join(entries) + m.group(3)
+            text = re.sub(r"(<IncludePath>)([^<]*)(</IncludePath>)",
+                          _add_middleware_incs, text)
         # c) Device / Cpu / 内存节点（一处裁决多处一致）
         mem = cfg["memory"]
         flash_base = int(mem["flash"][0]["base"], 16)
@@ -851,10 +890,9 @@ class Materializer:
             if f"<{tag}>" in text:
                 text = rewrite_memory_node(text, tag, f"0x{base:08X}", f"0x{size:X}", what)
         edits.append("memory nodes (Cpu/IROM/IRAM/OCR_RVCT4/OCR_RVCT9) from recipe model")
-        # umfTarg：使用 Target Dialog 内存布局生成容量受限 scatter（报告 §4.4-6）
-        if "<umfTarg>0</umfTarg>" in text:
-            text = text.replace("<umfTarg>0</umfTarg>", "<umfTarg>1</umfTarg>", 1)
-            edits.append("umfTarg 0->1 (Use Memory Layout from Target Dialog)")
+        # umfTarg：与官方逐 IC 生成工程一致（0 = 不使用 Target Dialog 内存布局）；
+        # M4 板级模板原值是 1，由 recipe 的 project_edits 声明式改为 0；
+        # M0+/M3 官方原值已是 0，这里不再强制改写。
         # FlashDriverDll：-FL 与裁决容量一致（官方逐 IC 值优先；仅裁决覆盖时改写）
         # 注意 UL2CM3 参数是**按算法块分块**的：
         #   UL2CM3(... -FF0HT32F -FS00 -FL<主 Flash 容量> -FF1HT32F_OPT -FS11FF00000 -FL<OPT 区长度>)
@@ -904,6 +942,28 @@ class Materializer:
         # e2) 声明式 uvprojx 选项编辑（如 M4 统一 -O2；命中数必须精确）
         for e in cfg.get("project_edits", []):
             tag = e["tag"]
+            if e.get("op") == "ensure":
+                # 节点存在则改值，不存在则在指定锚点后插入（M0+/M3 的 uC99）
+                m0 = re.search(rf"<{tag}>([^<]*)</{tag}>", text)
+                old_val = m0.group(1) if m0 else None
+                if m0:
+                    text = rewrite_tag_value(text, tag, e["new_value"], what)
+                    edits.append(f"{tag} ensure -> {e['new_value']}")
+                else:
+                    anchor = e["insert_after_tag"]
+                    m = re.search(rf"([ \t]*)<{anchor}>[^<]*</{anchor}>(\r?\n)", text)
+                    if not m:
+                        raise ContractError(
+                            RECIPE_MISMATCH,
+                            f"{what}: <{tag}> 不存在且找不到插入锚点 <{anchor}>")
+                    ins = f"{m.group(1)}<{tag}>{e['new_value']}</{tag}>{m.group(2)}"
+                    text = text[:m.end()] + ins + text[m.end():]
+                    edits.append(f"{tag} inserted -> {e['new_value']}")
+                deviations.append(dict(
+                    kind="project-option-edit", file=f"{stem}.uvprojx", tag=tag,
+                    old=old_val, new=e["new_value"],
+                    reason=e.get("reason") or "声明式项目选项编辑"))
+                continue
             old = f"<{tag}>{e['old_value']}</{tag}>"
             new = f"<{tag}>{e['new_value']}</{tag}>"
             hits = text.count(old)
@@ -1139,19 +1199,36 @@ class Materializer:
             lines.append("  - printf 经 C 库 + 官方 retarget 钩子输出；波特率 115200")
         else:
             lines.append("  - 官方 `_RETARGET=0`：printf/getchar 退化为空操作，不占用 UART")
-        lines += [
-            "",
-            "## 目录",
-            "",
-            "| 目录 | 内容 |",
-            "|---|---|",
+        dir_rows = [
             "| `library/` | 完整官方库（字节冻结，manifest/files.json 可逐文件核对） |",
+        ]
+        if kernel == "m4":
+            dir_rows.append(
+                "| `middlewares/` | 官方中间件：USB drivers/classes、FatFS、"
+                "I2C application library（官方式 IncludePath 引用 "
+                "`middlewares\\usb_drivers\\inc`） |")
+        dir_rows += [
             "| `utilities/` | 官方板级/公共构建资产 + host 工具（`gsar/`、`elink32pro/`）"
             "+ DFP 安装包（`pack/`，见上「前置条件」） |",
-            "| `project/MDK_ARMv5/` | 工程文件（由官方逐 IC 工程声明式物化） |",
-            "| `project/Src_USER/` | 用户源码：自有 `main.c` + 官方配置/中断文件 |",
+            "| `project/MDK_ARMv5/` | 工程文件（官方工程声明式物化）"
+            " + 工程自有 `startup_*.s`（栈配置） |",
+            "| `project/Src_USER/` | 用户源码：自有 `main.c` + 官方配置/中断文件"
+            " + 工程自有 `system_*.c`（时钟/看门狗配置） |",
             "| `licenses/` | 官方许可文件 |",
             "| `manifest/` | project-lock / files / deviations / validation 四件套 |",
+        ]
+        lines += ["", "## 目录", "", "| 目录 | 内容 |", "|---|---|", *dir_rows]
+        lines += [
+            "",
+            "## 更新官方库",
+            "",
+            "`library/`、`utilities/`"
+            + ("、`middlewares/`" if kernel == "m4" else "")
+            + " 是官方资产，可整体替换为同版本或更新的官方包；"
+              "工程自己的配置都在 `project/` 下（`system_*.c` 时钟/看门狗、"
+              "`startup_*.s` 栈、`<series>_conf.h`、`main.c`），**不会被库更新覆盖**。"
+              "替换库后如需合并官方改动，请只 diff `system_*.c` / `startup_*.s`，"
+              "不要直接用官方原档覆盖工程自有副本。",
             "",
             "## 自校验",
             "",

{ht32_code_mcp-0.4.2 → ht32_code_mcp-0.4.4}/src/ht32_code_mcp/server.py

@@ -98,7 +98,8 @@ M0+（HT32F5 / L5）、M3（HT32F1 / F175x-275x）、M4（F490x1 / F491x3 / F493
 - 知识库缺了用户需要的内容，你只能用推测回答
 - 导出工程有缺陷（编译失败、型号参数错、缺文件）
 - **ht32-skills（编排层：流程/脚本/模板/环境检测）本身有缺陷**，用 issue_type=skill_defect
-  上报（本渠道是 skill 问题的统一入口），details 里附 ht32-skills 版本号与现象
+  上报（本渠道是 skill 问题的统一入口），`skill_version` 传本机 `ht32.py version` 的版本号，
+  details 里附现象与日志路径
 务必附 model_id；submit_feedback 之外的调用已由服务端自动记入用量日志，无需你重复上报。
 """
 
@@ -561,6 +562,7 @@ def submit_feedback(
     details: str = "",
     model_id: str = "",
     tools_called: list[str] | None = None,
+    skill_version: str = "",
 ) -> str:
     """提交纠正反馈，改进 HT32 代码知识库 / ht32-skills 编排层。
 
@@ -576,9 +578,12 @@ def submit_feedback(
         issue_type: 问题类型，枚举上述其一（skill_defect = ht32-skills 编排层缺陷）
         answer: 你给出的回答原文
         correction: 你认为正确的信息或用户指出的正确内容
-        details: 补充上下文（如编译报错原文；skill_defect 请附 ht32-skills 版本号与现象）
+        details: 补充上下文（如编译报错原文、复现步骤、日志路径）
         model_id: 你的模型标识，如 deepseek-v4-flash
         tools_called: 本次调过的查询工具列表（不含 submit_feedback 自身）
+        skill_version: 本机 ht32-skills 版本，取 `ht32.py version` 的 skill_version
+            （如 "v0.1.19"）。服务端跨进程拿不到客户端 skill 版本，**skill_defect 必填**，
+            否则无从判断缺陷属于哪个版本
     """
     if not question or not question.strip():
         return "错误：question 参数不能为空，请提供用户原始需求。"
@@ -595,6 +600,7 @@ def submit_feedback(
         tools_called=tools_called,
         mcp_version=_MCP_VERSION,
         kb_version=index.data.get("code_version", ""),
+        skill_version=skill_version,
     )
     if path is None:
         return ("错误：反馈未能写入。服务端可能未配置 FEEDBACK_ROOT，"

{ht32_code_mcp-0.4.2 → ht32_code_mcp-0.4.4}/src/ht32_code_mcp/usage.py

@@ -9,7 +9,8 @@
 - **tool_call**（`UsageMiddleware`，服务端自动）：拦截所有工具调用，
   记录工具名、参数原文、耗时、成败。不依赖 Agent 配合，数据完整。
 - **feedback**（`record_feedback`，由各 server 的 `submit_feedback` 工具调用）：
-  记录 Agent 主动上报的语义性问题，middleware 无法判断这些。
+  记录 Agent 主动上报的语义性问题，middleware 无法判断这些。其中 `skill_version`
+  由 Agent 自报（服务端跨进程拿不到客户端 skill 版本），用于定位 ht32-skills 缺陷版本。
 
 目录布局（同根 + 服务子目录）：
 
@@ -38,7 +39,7 @@ from typing import Any
 from fastmcp.server.middleware import Middleware, MiddlewareContext
 
 # 实现版本号：两仓库副本必须一致（各仓测试会校验，防止只改一边造成漂移）
-_IMPL_VERSION = "1.1.0"
+_IMPL_VERSION = "1.2.0"
 
 # 服务名 → 子目录名。新增服务在此登记。
 SERVICE_WIKI = "wiki"
@@ -203,6 +204,7 @@ def record_feedback(
     tools_called: list[str] | None = None,
     mcp_version: str = "",
     kb_version: str = "",
+    skill_version: str = "",
 ) -> Path | None:
     """记录一条 Agent 主动提交的反馈。"""
     record = {
@@ -219,5 +221,6 @@ def record_feedback(
         "tools_called": [t for t in (tools_called or []) if t and t != "submit_feedback"],
         "mcp_version": mcp_version,
         "kb_version": kb_version,
+        "skill_version": skill_version.strip(),
     }
     return write_record(log_dir, FEEDBACK_PREFIX, record)

ht32_code_mcp-0.4.4/tests/test_export_cloud_mode.py

@@ -0,0 +1,176 @@
+"""回归测试：导出侧实测暴露的三个缺陷。
+
+1) 云端 zip 模式下响应格式化用了未定义的 `_fmt_time` → 线上 `export_project_bundle`
+   100% 失败（本地模式与直接调 `exporter.export_zip` 都覆盖不到这行）。
+2) 物化器的单次调用状态（`_stage` / `_project_rel`）曾挂在进程级单例实例上，
+   并发导出互相覆盖 → "…is not in the subpath of…" 串台错误。
+3) 默认时钟 profile `hsi-safe` 曾整体失效：官方 system 文件只存在于 `library/`，由完整
+   官方树按字节拷贝，而选项编辑只作用于 `_resolve_target` 能定位到 staging 的文件 →
+   声明式编辑从未落到 system 文件，导出工程仍依赖外部晶振（HSE）。现在的口径是：
+   system/startup 属于**工程自有资产**，导出到 `project/Src_USER` 与
+   `project/MDK_ARMv5`；`library/` 保持官方原档字节，用户整体更新库不会覆盖工程配置。
+"""
+from __future__ import annotations
+
+import concurrent.futures
+import json
+import re
+
+from ht32_code_mcp import server as srv
+
+CFG_F52352 = "ht32f52352.ht32f52352-sk.ac5.hsi-safe.retarget"
+CFG_F50241 = "ht32f50241.ht32f50241-sk.ac5.hsi-safe.retarget"
+CFG_L52241 = "ht32l52241.ht32l52241-sk.ac5.hsi-safe.retarget"
+CFG_F49163 = "ht32f49163-100lqfp.ht32f49163-sk.ac5.hick-safe.retarget"
+
+
+def test_cloud_zip_response_formats_expiry(tmp_path, monkeypatch):
+    """云端模式必须能给出下载 URL 与有效期（不得因未定义的 helper 直接抛 NameError）。"""
+    monkeypatch.setattr(srv, "CLOUD_BUNDLE", True)
+    monkeypatch.setattr(srv, "BUNDLE_DIR", str(tmp_path / "bundles"))
+    monkeypatch.setattr(srv, "BUNDLE_BASE_URL", "http://example.test/ht32-bundles")
+    text = srv.export_project_bundle(config_id=CFG_F52352, project_name="CloudMode")
+    assert "导出失败" not in text, text
+    assert "下载 URL" in text and "http://example.test/ht32-bundles/" in text
+    assert "有效期" in text
+    assert list(tmp_path.joinpath("bundles").glob("*.zip")), "未生成 zip 产物"
+
+
+def test_concurrent_exports_do_not_cross_talk(tmp_path):
+    """并发导出必须各自落到自己的目录（不得因共享实例状态串台）。"""
+    jobs = [(CFG_F50241, "ConcA"), (CFG_L52241, "ConcB")]
+
+    def run(item):
+        idx, (cid, name) = item
+        return srv.export_project_bundle(
+            config_id=cid, project_name=name, out_dir=str(tmp_path / f"out{idx}"))
+
+    with concurrent.futures.ThreadPoolExecutor(max_workers=2) as pool:
+        texts = list(pool.map(run, enumerate(jobs)))
+
+    for text, (_, name) in zip(texts, jobs):
+        assert "导出失败" not in text, text
+    for idx, (_, name) in enumerate(jobs):
+        proj = tmp_path / f"out{idx}" / "project" / "MDK_ARMv5" / f"{name}.uvprojx"
+        assert proj.is_file(), f"并发导出产物缺失：{proj}"
+
+
+def test_hsi_safe_profile_edits_project_owned_system_file(tmp_path):
+    """hsi-safe 改写工程自有 system 副本，library/ 必须保持官方字节。
+
+    F50241 的官方 system 文件属 hse-direct 家族（HSE_ENABLE=1 / HCLK_SRC=2），
+    导出后 `project/Src_USER` 下的副本应改为 HSE_ENABLE=0 + HCLK_SRC=3（HSI，
+    20 MHz 满频），并在 deviations 留 option-edit 记录；`library/` 下的同名
+    官方文件保持原样，这样用户整体更新库时不会把工程配置清掉。
+    """
+    out = tmp_path / "out"
+    text = srv.export_project_bundle(config_id=CFG_F50241, project_name="HsiSafe",
+                                     out_dir=str(out))
+    assert "导出失败" not in text, text
+
+    dev = json.loads((out / "manifest" / "deviations.json").read_text(encoding="utf-8"))
+    edits = [d for d in dev["deviations"] if d["kind"] == "option-edit"]
+    assert {d["selector"] for d in edits} >= {"HSE_ENABLE", "HCLK_SRC"}, f"缺 option-edit 记录：{edits}"
+    assert len({d["file"] for d in edits}) == 1, f"option-edit 跨了多个文件：{edits}"
+    sys_name = edits[0]["file"]
+
+    owned = out / "project" / "Src_USER" / sys_name
+    assert owned.is_file(), f"工程自有副本缺失：{owned}"
+    code = owned.read_text(encoding="utf-8", errors="replace")
+    assert re.search(r"#define\s+HSE_ENABLE\s+\(0\)", code), "HSE_ENABLE 未被改写为 0"
+    assert re.search(r"#define\s+HCLK_SRC\s+\(3\)", code), "HCLK_SRC 未被改写为 3（HSI）"
+
+    library_copy = next((p for p in out.glob("library/Device/**/system_*.c")
+                         if p.name == sys_name), None)
+    assert library_copy is not None, f"完整官方库缺少 {sys_name}"
+    lib_code = library_copy.read_text(encoding="utf-8", errors="replace")
+    assert re.search(r"#define\s+HSE_ENABLE\s+\(1\)", lib_code), \
+        "library/ 下的官方 system 文件被改写（更新库会覆盖工程配置）"
+    assert re.search(r"#define\s+HCLK_SRC\s+\(2\)", lib_code), \
+        "library/ 下的官方 system 文件被改写（更新库会覆盖工程配置）"
+
+    files = json.loads((out / "manifest" / "files.json").read_text(encoding="utf-8"))
+    paths = [f["path"] for f in files["files"]]
+    assert len(paths) == len(set(paths)), "同一路径被重复登记（树拷贝 + 源集各一次）"
+    owned_entry = next(f for f in files["files"]
+                       if f["path"] == f"project/Src_USER/{sys_name}")
+    assert owned_entry.get("edited") is True, "工程自有 system 副本未标记 edited"
+    lib_entry = next(f for f in files["files"]
+                     if f["path"].startswith("library/") and f["path"].endswith("/" + sys_name))
+    assert not lib_entry.get("edited"), "library/ 官方文件不应被标记 edited"
+
+
+def test_m4_system_startup_are_project_owned(tmp_path):
+    """M4 与 M0+/M3 同口径：system/startup 放 project/，library 保持官方字节。
+
+    官方 M4 生成工程把这两个文件放在 libraries/cmsis 下；若导出仍引用库内副本，
+    用户整体更新 library/ 会覆盖工程配置（与 M0+/M3 修复前同一个问题）。
+    """
+    out = tmp_path / "m4"
+    text = srv.export_project_bundle(config_id=CFG_F49163, project_name="M4Owned",
+                                     out_dir=str(out))
+    assert "导出失败" not in text, text
+
+    sys_name = "system_ht32f491x3.c"
+    startup_name = "startup_ht32f491x3.s"
+    owned_sys = out / "project" / "Src_USER" / sys_name
+    owned_startup = out / "project" / "MDK_ARMv5" / startup_name
+    assert owned_sys.is_file(), f"缺工程自有 system 副本：{owned_sys}"
+    assert owned_startup.is_file(), f"缺工程自有 startup 副本：{owned_startup}"
+
+    # 完整官方库仍保留同名文件（作为官方资产），但构建引用的是 project/ 副本
+    assert next(out.glob(f"library/**/{sys_name}"), None) is not None
+    assert next(out.glob(f"library/**/{startup_name}"), None) is not None
+    uv = (out / "project" / "MDK_ARMv5" / "M4Owned.uvprojx").read_text(
+        encoding="utf-8", errors="replace")
+    assert f"..\\Src_USER\\{sys_name}" in uv, "uvprojx 未引用工程自有 system 副本"
+    assert f".\\{startup_name}" in uv, "uvprojx 未引用工程自有 startup 副本"
+
+    files = json.loads((out / "manifest" / "files.json").read_text(encoding="utf-8"))
+    paths = [f["path"] for f in files["files"]]
+    assert len(paths) == len(set(paths)), "同一路径被重复登记"
+    entry = next(f for f in files["files"] if f["path"] == f"project/Src_USER/{sys_name}")
+    assert not entry.get("edited"), "M4 默认 profile 不应改写 system 文件"
+
+
+def test_m4_export_carries_official_middlewares(tmp_path):
+    """M4 官方包自带 middlewares/（USB/FatFS/I2C），导出必须携带、登记并补 IncludePath。"""
+    out = tmp_path / "m4mw"
+    text = srv.export_project_bundle(config_id=CFG_F49163, project_name="M4Mid",
+                                     out_dir=str(out))
+    assert "导出失败" not in text, text
+
+    usb_inc = out / "middlewares" / "usb_drivers" / "inc" / "usbd_core.h"
+    assert usb_inc.is_file(), "缺少 middlewares/usb_drivers/inc（官方 M4 包自带）"
+
+    files = json.loads((out / "manifest" / "files.json").read_text(encoding="utf-8"))
+    paths = {f["path"] for f in files["files"]}
+    assert "middlewares/usb_drivers/inc/usbd_core.h" in paths, \
+        "middlewares 未登记在 manifest/files.json"
+
+    uv = (out / "project" / "MDK_ARMv5" / "M4Mid.uvprojx").read_text(encoding="utf-8")
+    assert "..\\..\\middlewares\\usb_drivers\\inc" in uv, \
+        "IncludePath 未指向 middlewares/usb_drivers/inc"
+    assert "<uC99>1</uC99>" in uv, "M4 未启用 C99（应与官方生成工程对齐）"
+    assert "<Misc>--entry Reset_Handler</Misc>" in uv, \
+        "M4 链接入口未与官方生成工程对齐"
+    assert "<umfTarg>0</umfTarg>" in uv, "M4 umfTarg 应与官方生成工程一致（0）"
+
+    # M0+ 官方包没有 middlewares，不应凭空出现
+    out2 = tmp_path / "m0nomid"
+    srv.export_project_bundle(config_id=CFG_F52352, project_name="M0NoMid",
+                              out_dir=str(out2))
+    assert not (out2 / "middlewares").exists(), "M0+ 工程不应携带 middlewares"
+
+
+def test_c99_enabled_for_all_series(tmp_path):
+    """全系列统一开启 C99；M0+/M3 由 ensure 编辑插入 uC99 节点。"""
+    cases = [(CFG_F52352, "F5C99"), (CFG_L52241, "L5C99"), (CFG_F49163, "M4C99")]
+    for cfg, name in cases:
+        out = tmp_path / name
+        text = srv.export_project_bundle(config_id=cfg, project_name=name,
+                                         out_dir=str(out))
+        assert "导出失败" not in text, text
+        uv = next((out / "project" / "MDK_ARMv5").glob("*.uvprojx"))
+        xml = uv.read_text(encoding="utf-8")
+        assert "<uC99>1</uC99>" in xml, f"{cfg} 未开启 C99"

{ht32_code_mcp-0.4.2 → ht32_code_mcp-0.4.4}/tests/test_tools.py

@@ -273,7 +273,7 @@ def _bundle_checks_v2(out_dir: Path, device: str, config_id: str = ""):
     assert m and int(m.group(1), 16) == flash_size, "IROM 与锁定 memory 不一致"
     m = re.search(r"<IRAM>.*?<Size>(0x[0-9A-Fa-f]+)</Size>", text, re.S)
     assert m and int(m.group(1), 16) == ram_size, "IRAM 与锁定 memory 不一致"
-    assert "<umfTarg>1</umfTarg>" in text, "未启用 Use Memory Layout from Target Dialog"
+    assert "<umfTarg>0</umfTarg>" in text, "umfTarg 应与官方生成工程一致（0）"
     # 模板自用文件不进用户工程；README 属于用户工程
     assert not (out_dir / "devices.json").exists()
     assert not (out_dir / "TEMPLATE.md").exists()
@@ -381,7 +381,9 @@ def test_export_positive_default_chip(tmp_path):
     out = tmp_path / "bundle_default"
     result = srv.export_project_bundle("HT32F52352", str(out))
     assert "已导出" in result and "contract_version: 2" in result
-    assert "config_id: ht32f52352." in result and "recipe_revision: 2.0.0" in result
+    rev = json.loads((REPO / ".work/template-recipes/recipes/ht32f5.json")
+                     .read_text(encoding="utf-8"))["recipe_revision"]
+    assert "config_id: ht32f52352." in result and f"recipe_revision: {rev}" in result
     assert "v1 chip 参数已弃用" in result          # 旧参数走只读适配层
     assert "flash" not in result.lower() or "Flash" in result
     _bundle_checks_v2(out, "HT32F52352")
@@ -677,7 +679,7 @@ def test_version_reports_skill_latest(monkeypatch):
 def test_usage_impl_version_pinned():
     """两仓库的 usage.py 是逐字节相同副本，实现版本号用于防止只改一边。"""
     from ht32_code_mcp import usage
-    assert usage._IMPL_VERSION == "1.1.0"
+    assert usage._IMPL_VERSION == "1.2.0"
     assert usage.SERVICE_CODE == "code"
     assert usage.SERVICE_WIKI == "wiki"
     # 噪声参数含 out_dir（code 特有：可能含用户本地路径）
@@ -735,6 +737,7 @@ def test_submit_feedback_writes_jsonl(tmp_path, monkeypatch):
         details="编译报错 too few arguments",
         model_id="deepseek-v4-flash",
         tools_called=["search_code_knowledge", "submit_feedback", "get_file"],
+        skill_version="v0.1.19",
     )
     assert "反馈已记录" in out
     f = next((tmp_path / "logs").glob("feedback_*.jsonl"))
@@ -747,6 +750,8 @@ def test_submit_feedback_writes_jsonl(tmp_path, monkeypatch):
     # submit_feedback 自身被过滤，只留查询类工具
     assert rec["tools_called"] == ["search_code_knowledge", "get_file"]
     assert rec["mcp_version"] and rec["kb_version"]
+    # 客户端 skill 版本是结构化字段（不再靠 details 散文），未传时为空串而非缺键
+    assert rec["skill_version"] == "v0.1.19"
 
 
 def test_submit_feedback_accepts_skill_defect(tmp_path, monkeypatch):
@@ -757,11 +762,22 @@ def test_submit_feedback_accepts_skill_defect(tmp_path, monkeypatch):
         issue_type="skill_defect",
         details="ht32-skills v0.1.0；导出后 uvprojx 未按工程名重命名",
         model_id="deepseek-v4-flash",
+        skill_version="v0.1.0",
     )
     assert "skill_defect" in out
     f = next((tmp_path / "logs").glob("feedback_*.jsonl"))
     rec = json.loads(f.read_text(encoding="utf-8").splitlines()[0])
     assert rec["issue_type"] == "skill_defect"
+    assert rec["skill_version"] == "v0.1.0"
+
+
+def test_feedback_skill_version_key_always_present(tmp_path, monkeypatch):
+    """没传 skill_version 时字段仍存在（空串），日志消费方不必分支判键。"""
+    monkeypatch.setattr(srv, "LOG_DIR", str(tmp_path / "logs"))
+    srv.submit_feedback(question="x", issue_type="other")
+    f = next((tmp_path / "logs").glob("feedback_*.jsonl"))
+    rec = json.loads(f.read_text(encoding="utf-8").splitlines()[0])
+    assert rec["skill_version"] == ""
 
 
 def test_submit_feedback_empty_question(tmp_path, monkeypatch):

{ht32_code_mcp-0.4.2 → ht32_code_mcp-0.4.4}/uv.lock

@@ -440,7 +440,7 @@ wheels = [
 
 [[package]]
 name = "ht32-code-mcp"
-version = "0.4.2"
+version = "0.4.4"
 source = { editable = "." }
 dependencies = [
     { name = "fastmcp" },

ht32_code_mcp-0.4.2/tests/test_export_cloud_mode.py

@@ -1,83 +0,0 @@
-"""回归测试：导出侧实测暴露的三个缺陷。
-
-1) 云端 zip 模式下响应格式化用了未定义的 `_fmt_time` → 线上 `export_project_bundle`
-   100% 失败（本地模式与直接调 `exporter.export_zip` 都覆盖不到这行）。
-2) 物化器的单次调用状态（`_stage` / `_project_rel`）曾挂在进程级单例实例上，
-   并发导出互相覆盖 → "…is not in the subpath of…" 串台错误。
-3) 默认时钟 profile `hsi-safe` 曾整体失效：官方 system 文件在 `library/` 下，由完整官方树
-   按字节拷贝，而选项编辑只作用于 `_resolve_target` 能定位到 staging 的文件 → 声明式编辑
-   从未落到 system 文件，导出工程仍依赖外部晶振（HSE）。
-"""
-from __future__ import annotations
-
-import concurrent.futures
-import json
-import re
-
-from ht32_code_mcp import server as srv
-
-CFG_F52352 = "ht32f52352.ht32f52352-sk.ac5.hsi-safe.retarget"
-CFG_F50241 = "ht32f50241.ht32f50241-sk.ac5.hsi-safe.retarget"
-CFG_L52241 = "ht32l52241.ht32l52241-sk.ac5.hsi-safe.retarget"
-
-
-def test_cloud_zip_response_formats_expiry(tmp_path, monkeypatch):
-    """云端模式必须能给出下载 URL 与有效期（不得因未定义的 helper 直接抛 NameError）。"""
-    monkeypatch.setattr(srv, "CLOUD_BUNDLE", True)
-    monkeypatch.setattr(srv, "BUNDLE_DIR", str(tmp_path / "bundles"))
-    monkeypatch.setattr(srv, "BUNDLE_BASE_URL", "http://example.test/ht32-bundles")
-    text = srv.export_project_bundle(config_id=CFG_F52352, project_name="CloudMode")
-    assert "导出失败" not in text, text
-    assert "下载 URL" in text and "http://example.test/ht32-bundles/" in text
-    assert "有效期" in text
-    assert list(tmp_path.joinpath("bundles").glob("*.zip")), "未生成 zip 产物"
-
-
-def test_concurrent_exports_do_not_cross_talk(tmp_path):
-    """并发导出必须各自落到自己的目录（不得因共享实例状态串台）。"""
-    jobs = [(CFG_F50241, "ConcA"), (CFG_L52241, "ConcB")]
-
-    def run(item):
-        idx, (cid, name) = item
-        return srv.export_project_bundle(
-            config_id=cid, project_name=name, out_dir=str(tmp_path / f"out{idx}"))
-
-    with concurrent.futures.ThreadPoolExecutor(max_workers=2) as pool:
-        texts = list(pool.map(run, enumerate(jobs)))
-
-    for text, (_, name) in zip(texts, jobs):
-        assert "导出失败" not in text, text
-    for idx, (_, name) in enumerate(jobs):
-        proj = tmp_path / f"out{idx}" / "project" / "MDK_ARMv5" / f"{name}.uvprojx"
-        assert proj.is_file(), f"并发导出产物缺失：{proj}"
-
-
-def test_hsi_safe_profile_edits_system_file_in_library_tree(tmp_path):
-    """hsi-safe 默认 profile 必须真正改写官方 system 文件（不得因 tree 拷贝路径整条跳过）。
-
-    F50241 的官方 system 文件属 hse-direct 家族（HCLK_SRC=2/HSE），编辑器应把它改成
-    HSE_ENABLE=0 + HCLK_SRC=3（HSI，20 MHz 满频），并在 deviations 留 option-edit 记录。
-    """
-    out = tmp_path / "out"
-    text = srv.export_project_bundle(config_id=CFG_F50241, project_name="HsiSafe",
-                                     out_dir=str(out))
-    assert "导出失败" not in text, text
-
-    # 完整官方树里有 ~20 个 system_*.c；只有本配置选中的那个应被改写
-    dev = json.loads((out / "manifest" / "deviations.json").read_text(encoding="utf-8"))
-    edits = [d for d in dev["deviations"] if d["kind"] == "option-edit"]
-    assert {d["selector"] for d in edits} >= {"HSE_ENABLE", "HCLK_SRC"}, f"缺 option-edit 记录：{edits}"
-    assert len({d["file"] for d in edits}) == 1, f"option-edit 跨了多个文件：{edits}"
-    sys_name = edits[0]["file"]
-
-    target = next((p for p in out.glob("library/Device/**/system_*.c") if p.name == sys_name), None)
-    assert target is not None, f"导出物里找不到 {sys_name}"
-    code = target.read_text(encoding="utf-8", errors="replace")
-    assert re.search(r"#define\s+HSE_ENABLE\s+\(0\)", code), "HSE_ENABLE 未被改写为 0"
-    assert re.search(r"#define\s+HCLK_SRC\s+\(3\)", code), "HCLK_SRC 未被改写为 3（HSI）"
-
-    files = json.loads((out / "manifest" / "files.json").read_text(encoding="utf-8"))
-    paths = [f["path"] for f in files["files"]]
-    assert len(paths) == len(set(paths)), "同一路径被重复登记（树拷贝 + 源集各一次）"
-    entry = next(f for f in files["files"] if f["path"].endswith("/" + sys_name))
-    assert entry.get("edited") is True, "system 文件的 manifest 条目未标记 edited"