Conversation
…r dependency links Co-authored-by: nevstop <8196752+nevstop@users.noreply.github.com>
module-template.md
Outdated
| | CSM INI Static Variable Support | 可选 | [GitHub](https://github.com/NEVSTOP-LAB/CSM-INI-Static-Variable-Support) | | ||
| | CSM Mermaid Plugin | 可选 | [GitHub](https://github.com/NEVSTOP-LAB/CSM-Mermaid-Plugin) | | ||
| | CSM TCP Router | 可选 | [GitHub](https://github.com/NEVSTOP-LAB/CSM-TCP-Router-App) | | ||
| | Communicable State Machine (CSM) | 必须 | [Communicable-State-Machine](https://github.com/NEVSTOP-LAB/Communicable-State-Machine) | |
There was a problem hiding this comment.
已简化为分组项目列表,去掉冗余的表格列(链接文字本身就是仓库名,"依赖名"列完全多余):
**必须**
- [Communicable-State-Machine](https://github.com/NEVSTOP-LAB/Communicable-State-Machine)
**可选**(根据所用功能选择)
- [CSM-API-String-Arguments-Support](...)提交:c8b6443
There was a problem hiding this comment.
也要增加一些 模块的 meta 信息,如支持的系统版本,labview版本, 是否支持 RT 等
There was a problem hiding this comment.
在 ## 功能简述 之后新增了 ## 模块信息 章节:
## 模块信息
| 属性 | 值 |
| --- | --- |
| LabVIEW 版本 | ≥ [版本号,例如 2019] |
| 支持的操作系统 | [Windows / Linux / macOS] |
| 支持 RT | [✅ 支持 / ❌ 不支持] |
| 支持 64-bit | [✅ 支持 / ❌ 不支持] |docs/csm-module-skill.md 中也同步更新了章节列表和 AI 检查清单。提交:c8b6443
…ndency list Co-authored-by: nevstop <8196752+nevstop@users.noreply.github.com>
module-template.md
Outdated
| | CSM INI Static Variable Support | 可选 | [GitHub](https://github.com/NEVSTOP-LAB/CSM-INI-Static-Variable-Support) | | ||
| | CSM Mermaid Plugin | 可选 | [GitHub](https://github.com/NEVSTOP-LAB/CSM-Mermaid-Plugin) | | ||
| | CSM TCP Router | 可选 | [GitHub](https://github.com/NEVSTOP-LAB/CSM-TCP-Router-App) | | ||
| **必须** |
There was a problem hiding this comment.
已恢复为 2 列表格,链接嵌入依赖名称列,移除单独的链接列(提交:70174d6):
| 依赖 | 类型 |
| --- | --- |
| [Communicable-State-Machine](https://github.com/NEVSTOP-LAB/Communicable-State-Machine) | 必须 |
| [CSM-API-String-Arguments-Support](...) | 可选 |There was a problem hiding this comment.
Pull request overview
This PR updates the CSM module documentation conventions and aligns both the canonical template and the AI skill guide to a more extensible, per-entry sub-paragraph structure.
Changes:
- Replaces API/status interface tables with
###sub-paragraph entries (per message) and standardizes fields (调用方式/参数/响应). - Simplifies the dependency section into grouped bullet lists (必须 / 可选) using repository names as link text.
- Adds a required
## 模块信息metadata table (LabVIEW version, OS, RT, 64-bit).
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| module-template.md | Updates the module doc template to include 模块信息, grouped dependencies, and ###-based API/status entry formatting. |
| docs/csm-module-skill.md | Updates the AI skill guide to specify and enforce the new 模块信息/依赖项 formats and the new per-entry API/status documentation structure. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
docs/csm-module-skill.md
Outdated
| | **API** | **描述** | **参数数据类型** | **参数描述** | **响应数据类型** | **响应描述** | | ||
| | --- | --- | --- | --- | --- | --- | | ||
| | 完整消息字符串,包含前缀,例如 `` `API: Start` `` | 一句话说明该 API 的作用 | 参数的数据类型(例如 `APIString`、`MassData`、`${FilePath}` 等)。无参数时写 `N/A`。 | `>>` 之后传递的数据含义。无参数时写 `N/A`。 | 响应的数据类型。无响应或异步调用时写 `N/A`。 | 模块返回内容的含义。无响应或异步调用时写 `N/A`。 | | ||
| - **调用方式**:异步 (`->`) / 同步 (`-@`) / 无应答 (`->|`) |
There was a problem hiding this comment.
不需要调用方式,因为任何消息都可以被当作同步和异步调用
There was a problem hiding this comment.
已从所有 API 段落和 skill 文档规范中移除 **调用方式** 字段(提交:70174d6)。
module-template.md
Outdated
| | CSM INI Static Variable Support | 可选 | [GitHub](https://github.com/NEVSTOP-LAB/CSM-INI-Static-Variable-Support) | | ||
| | CSM Mermaid Plugin | 可选 | [GitHub](https://github.com/NEVSTOP-LAB/CSM-Mermaid-Plugin) | | ||
| | CSM TCP Router | 可选 | [GitHub](https://github.com/NEVSTOP-LAB/CSM-TCP-Router-App) | | ||
| **必须** |
module-template.md
Outdated
| 初始化内部资源。必须在其他 API 之前调用。 | ||
|
|
||
| - **调用方式**:异步 (`->`) | ||
| - **参数**:`APIString` — 配置文件路径 |
There was a problem hiding this comment.
参数/响应都应该要有 CSM参数类型(APIString/MassData/HexStr等)和数据描述。
CSM参数类型:APIString
数据类型:Cluster
数据描述:
Flag:Boolean
Level:DBL
There was a problem hiding this comment.
已更新参数/响应格式,同时标注 CSM 参数类型和数据类型+描述(提交:70174d6)。简单值:
- **参数**:`APIString` — `String`:配置文件路径复合数据(Cluster)展开字段:
- **参数**:`HexStr` — `Cluster`:
- `Flag`:Boolean
- `Level`:DBL|
@copilot 修改以上的 comments |
…ram format Co-authored-by: nevstop <8196752+nevstop@users.noreply.github.com>
Three doc convention changes to
module-template.mdanddocs/csm-module-skill.md:###sub-paragraphs — drop flat tables in favour of one sub-section per message, making individual entries easier to read and extend.## 模块信息table placed after## 功能简述captures LabVIEW version, supported OS, RT support, and 64-bit support.**参数**and**响应**entry now shows both the CSM serialization type (APIString,HexStr,MassData, …) and the actual LabVIEW data type + description. Cluster-type parameters enumerate fields as a sub-list.New API entry format (
module-template.md)For compound data (Cluster):
New status broadcast format
Dependency table (2 columns)
New module info section
docs/csm-module-skill.mdupdated accordingly: new sub-sections 3.1 and 3.2 document the module-info table and dependency table formats; section 4.2 specifies theCSM type — data type: descriptionparameter pattern with Cluster sub-list example; section 4.3 updated with matching examples; the AI checklist (§10) and common-errors table (§11) updated to reflect all new conventions.🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.