Skip to content

kb: 支持保留未知项的基础知识发布与深化交接 - #319

Merged
tzhouam merged 7 commits into
mainfrom
prep/kb-foundation-explicit-partial-20261006
Oct 6, 2026
Merged

tzhouam merged 7 commits into
mainfrom
prep/kb-foundation-explicit-partial-20261006

Conversation

@tzhouam

@tzhouam tzhouam commented Oct 6, 2026 •

Copy link
Copy Markdown
Collaborator

基础知识批次完成全部初始任务后,六维中的真实未知项会阻止进入深化。本变更增加显式 --foundation-mode partial:只发布已有源码引用及原生独立评审认可的内容,保留未知项、修正次数和原始生成身份,并绑定发布收据。默认严格模式保持原行为;后续深化仍须满足全部功能认可、七维各超过 90% 和生产文件覆盖目标。

部分发布使用有界 PR 描述,完整模型轨迹和诊断保留在 Git 外。已准备发布的恢复通过官方 API 仅更新描述,保持文件、基线、分支和提交身份。覆盖工具区分结构覆盖与六维基础完整度,深化、组装和审计共同验证真实已合并的基础发布收据。

验证:新增恢复、描述大小、模式隔离、收据/源码/计数篡改和分片完成状态回归;原生基础回归 78 项、覆盖/深度回归 68 项、接口/组合回归 119 项通过。独立评审覆盖原实现 27 个不同案例及最终描述修复 7 个案例,均通过。最终提交的完整 CI 单独执行。

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 6, 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-06T00:24:08.712764Z 7a77724 PR opened
ℹ️ 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: 7a777247ff

ℹ️ 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".

@@ -0,0 +1,34 @@
# Explicit partial foundation publication

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Index the partial-foundation documentation

This new page is not linked from doc/knowledge/README.md or another documentation index—repository-wide search for partial-foundation finds only this page—so readers entering through the knowledge documentation cannot discover it. Add the page to the directory's existing documentation index in the same change.

AGENTS.md reference: AGENTS.md:L7-L11

Useful? React with 👍 / 👎.

Comment on lines +143 to +144
if self.STAGE == "knowledge" and self.foundation_mode == "partial":
return False # Reconstruct a truthful publication-only scope from the saved native tasks.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Bind foundation mode before reusing a publication

When a knowledge run has already been published, this new partial-mode branch is unreachable because the method returns at line 140, while _input_options() does not include foundation_mode; strict and partial invocations therefore compute the same input digest. After publishing a partial foundation, rerunning the default strict command in the same state silently returns the partial record as a successful published result instead of rejecting the mode change. Include the mode in the input identity or compare it before the published-record shortcut.

Useful? React with 👍 / 👎.

@tzhouam tzhouam changed the title fix(kb): separate partial foundation publication from depth acceptance kb: 支持保留未知项的基础知识发布与深化交接 Oct 6, 2026
@tzhouam
tzhouam merged commit 1766559 into main Oct 6, 2026
1 of 2 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