Skip to content

接通 Rust 默认生命周期、固定原生复检入口与 32-grammar 运行时候选 - #92

Merged
loong10k merged 5 commits into
mainfrom
rust-lifecycle-default-and-grammar-runtime
Oct 7, 2026
Merged

loong10k merged 5 commits into
mainfrom
rust-lifecycle-default-and-grammar-runtime

Conversation

@loong10k

@loong10k loong10k commented Oct 7, 2026

Copy link
Copy Markdown
Contributor

把本地 main 上已完成的三个提交推送到远端,解除 codeguard 主仓 CI 的硬阻断。

为什么需要

codeguard 主仓 .github/workflows/ci.yml 的 gate 任务会克隆本仓并校验
源码审计源 dec5f9d 是否可达(git cat-file -e dec5f9d^{commit},失败即 exit 1)。
该提交此前只存在于本地 main,从未推送,导致 codeguard 侧 CI 连续 6 次全部失败
(最近一次 run 37573422912 失败于 "Check out corpus evidence source")。

包含的提交

  • 7bfb07a feat: expose pinned 32-grammar Rust runtime candidate(16 文件 +245/-33)
  • 5ef4ea6 合并远端名称与 MCP 更新并保留 Rust 候选发布版本(12 文件 +67/-10)
  • dec5f9d 接通 Rust 默认生命周期和固定原生复检入口,准备 0.20.0 发布(28 文件 +451/-76)

工作树干净,无未提交内容,未推送任何 tag(不触发公开发布流水线)。

风险提示

dec5f9d 的 plugin.json 含 0.20.0 发布准备。合并本 PR 只推送提交,不推送 tag;
若后续需要真正发布 0.20.0,请单独确认 tag 推送时机。

合并后 codeguard 主仓的 CI 才能取得该固定审计源;本 PR 合并不代表 CodeGuard
已具备生产资格——当前 57 语言 × 四核心的 228 项能力单元仍全部 blocked。

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 7, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-07T07:17:20.110105Z 0130198 New commits
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: dec5f9d136

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .agents/plugins/marketplace.json
Comment thread hooks/rust_runtime_dispatch.cjs Outdated
Comment thread runtime/codeguard_runtime.cjs
review 指出两个真实缺陷:

1) hooks 在用户 workspace 运行,而管理器位于已安装插件根目录,指引却给相对
   路径 `node runtime/codeguard_runtime.cjs`,照抄必然 MODULE_NOT_FOUND,
   新装用户会被永久卡在未完成。改为用已计算的绝对 pluginRoot 拼接。

2) 内容寻址的缓存目录存在但核验失败时(grammar 许可被改动或删除),
   重装对同一个损坏目录反复验证,用户无从判断该清理哪里。
   binary-distribution 规范要求「已存在但内容不同的目录 MUST 保留且拒绝,
   不得替换」,因此不静默重建覆盖;而是在拒绝时把确切的缓存目录写进原因,
   让用户据此移除后重试。新增 packageIntact 作非失败式复用判断,
   最终仍由 verifyPackage 复核。

本仓库测试 655 passed;node --check 通过。

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 79af622b33

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread com.github.copilot/hooks/rust_runtime_dispatch.cjs Outdated
Comment thread hooks/rust_runtime_dispatch.cjs
上一提交只改了 canonical hook,两个分发镜像仍是旧的相对路径,导致
validate_portable_plugin.py 的 mirror-drift 校验失败;现三副本一致。

同时转义恢复命令中的绝对路径:插件路径可能含空格(macOS 的 Application
Support)或单引号,未转义会被 shell 拆成多个参数,照抄即失败。
按 POSIX 规则单引号包裹,内部单引号转义;已实测普通路径、含空格路径、
含单引号路径三种情况经 sh 还原均与原值一致。

mirror-drift 校验 PASS;655 项测试通过;三副本 node --check 通过。
@loong10k
loong10k merged commit 7f67e34 into main Oct 7, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant