Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,12 @@
也要过这道门禁,所以清单没法在合并时被悄悄放宽。工具 `tools/check-pr.py`,
自测 `tests/test_check_pr.py`(18 例);机器人 PR(dependabot)跳过。

### 变更
- Codex skill 迁移到独立仓库 [cadywang/skills](https://github.com/cadywang/skills) 的
`skills/serial-debug-bridge/`:本仓库不再保留 `skill/` 副本,避免 skill 与
`devctl`/`agent` 在同一仓库里重复维护(两份实现容易各改各的)。README / CONTRIBUTING /
REVIEW_POLICY 里的引用已改指向新仓库;`scripts/sdb` 仍使用本仓库的 `cli/devctl.py`。

### 修复
- CI 的提交规范门禁误拒 GitHub 网页端 squash 合并产生的提交:作者被改写成
`<ID>+<login>@users.noreply.github.com`、自动追加 `Co-authored-by:`,正文还可能夹在
Expand Down
2 changes: 1 addition & 1 deletion CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -141,7 +141,7 @@ git config core.hooksPath tools/git-hooks
3. required status checks 未通过不可合入;单人维护模式下不设强制批准数(GitHub 不允许自批),由维护者审阅后合并——加了协作者后按 [REVIEW_POLICY.md](REVIEW_POLICY.md) 提升为「至少 1 位他人批准」。
4. 由 maintainer 合入 `main`:**Rebase and merge**(线性历史,分支自动删除)。仓库只允许这一种合并方式——**不要 Squash**:它会把整个 PR 压成一个提交,提交标题变成 PR 标题、正文被拼成一坨,每个提交的 `Signed-off-by:` 也不再是原文。rebase 保留 PR 里每个提交的作者与信息,所以提交信息必须一开始就写对;PR 标题同样按内核规范 `<subsystem>: <祈使句摘要>`(检索入口,且万一有人改回 squash 就会写进历史)。
5. 发布:打 tag `vX.Y.Z`,同步更新 `CHANGELOG.md`。
6. 补丁必须保持同步:`agent/serial_agent.ps1` ↔ `agent/serial_agent.py`、`cli/devctl.ps1` ↔ `cli/devctl.py`,以及 `PROTOCOL.md` / `README.md` / `CHANGELOG.md` / `skill/` 随行为变化一起改,清单见 [REVIEW_POLICY.md](REVIEW_POLICY.md)。
6. 补丁必须保持同步:`agent/serial_agent.ps1` ↔ `agent/serial_agent.py`、`cli/devctl.ps1` ↔ `cli/devctl.py`,以及 `PROTOCOL.md` / `README.md` / `CHANGELOG.md` 随行为变化一起改,清单见 [REVIEW_POLICY.md](REVIEW_POLICY.md)。Codex skill 已迁到 [cadywang/skills](https://github.com/cadywang/skills),跨仓库的改动请两边分别提 PR。

## 7. 许可

Expand Down
15 changes: 9 additions & 6 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -443,8 +443,9 @@ journalctl -u serial-agent -f
(agent 会一直持有串口,所以一旦进入 shell,后续 `exec` 都有效。)
* 设备**重启后会停在 `login:`**:直接以 **root** 登录(这些开发板 root 通常无密码),
**不要反复用 `exec` 试** —— 它会把命令当用户名、探针行当密码,每次白等一个 30 秒超时。
登录方式(`devctl ... shell` 手动登录,或 raw 透传模式发 `root`)见 skill 的
`references/troubleshooting.md` 第 10 节。
登录方式(`devctl ... shell` 手动登录,或 raw 透传模式发 `root`)见
[skills 仓库的 `troubleshooting.md`](https://github.com/cadywang/skills/blob/main/skills/serial-debug-bridge/references/troubleshooting.md)
第 10 节。
* 波特率/串口名不对:核对 `-Port` `-Baud` / `--port` `--baud`,确认设备实际参数。
* 换行/回车不对:极少数设备只认 `\r`,可在设备侧 `stty` 调整,或先 `shell` 手动敲一次。

Expand Down Expand Up @@ -562,10 +563,12 @@ squash 合并,它就会变成 `main` 上的提交标题)与 **模板里「
* 安全问题**不要**开公开 Issue,见 [`SECURITY.md`](SECURITY.md)。
* 提问前先看 [`SUPPORT.md`](SUPPORT.md),Issue/PR 模板在 `.github/`。
* 依赖更新由 `.github/dependabot.yml` 发起 PR(同样要过 CI 五个 job;bot 生成的提交按 `--skip-bots` 豁免 commit 风格检查,见 [`REVIEW_POLICY.md`](REVIEW_POLICY.md))。
* 面向 server 的 Codex skill 在 `skill/serial-debug-bridge/`,安装到 server 的
`~/.codex/skills/` 后即可让 server 侧的 Codex 直接调用(`scripts/sdb` 为统一入口,
`scripts/sdb-check.sh` 做链路与设备身份自检):
`scp -r skill/serial-debug-bridge <server>:~/.codex/skills/`
* 面向 server 的 Codex skill 已迁到 [cadywang/skills](https://github.com/cadywang/skills) 的
`skills/serial-debug-bridge/`(本仓库不再保留副本);装到 server 的 `~/.codex/skills/`
后即可让 server 侧的 Codex 直接调用(`scripts/sdb` 为统一入口,`scripts/sdb-check.sh`
做链路与设备身份自检):在 skills 仓库里执行
`scp -r skills/serial-debug-bridge <server>:~/.codex/skills/`。
`scripts/sdb` 仍使用本仓库的 `cli/devctl.py`(安装方式见 skills 仓库 README)。
* 重新施加/校验保护规则:
`pwsh -File tools/create-github-repo.ps1 -Owner <you>`(幂等,支持 `-DryRun`)。

Expand Down
2 changes: 1 addition & 1 deletion REVIEW_POLICY.md
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ git push -u origin agent/hex-retry
| `cli/devctl.ps1` | `cli/devctl.py` |
| 协议行为(帧格式、编码、超时语义) | `PROTOCOL.md` + 两套实现 + 两套 mock(`tests/*`) |
| 面向用户的行为 | `README.md`、`CHANGELOG.md` |
| skill 用到的入口/参数(`scripts/sdb`、环境变量) | `skill/serial-debug-bridge/SKILL.md` 与 `references/` |
| skill 用到的入口/参数(`scripts/sdb`、环境变量) | [cadywang/skills](https://github.com/cadywang/skills) 的 `skills/serial-debug-bridge/SKILL.md` 与 `references/`(不在本仓库;跨仓库改动两边分别提 PR) |

CI 的 `Cross implementation check` 只能验证「ps 的 CLI 连 py 的 agent」这类组合,
**覆盖不了语义是否一致**——所以人工审查时要专门盯这一项。
Expand Down
146 changes: 0 additions & 146 deletions skill/serial-debug-bridge/SKILL.md

This file was deleted.

7 changes: 0 additions & 7 deletions skill/serial-debug-bridge/agents/openai.yaml

This file was deleted.

Loading
Loading