Skip to content

docs: 在阶段 C 冻结 graph 表生成 API、索引与校验规范 - #47

Merged
0x5143 merged 2 commits into
masterfrom
2lezo4-codex/-graph-issue
Jul 7, 2026
Merged

docs: 在阶段 C 冻结 graph 表生成 API、索引与校验规范#47
0x5143 merged 2 commits into
masterfrom
2lezo4-codex/-graph-issue

Conversation

@0x5143

@0x5143 0x5143 commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Motivation

  • 在进入实现前需要冻结 graph 表的生成 API、内建索引约束与校验规则,以避免 parservalidatortemplate 与文档在后续实现中各自演进。
  • 目标是确保文档、生成器与运行时行为一致,并禁止在 DataTableProcessor 主流程中追加 graph 专用分支。

Description

  • 修改了 docs/optimization-plan-2026-07.md,在阶段 C(Milestone 3)新增 C4 条目以提出并冻结 graph 表规范要点。
  • 在新增条目中列出需要确认的生成 API(节点集合查询、出/入/关联边查询、前驱/后继/邻居查询、两点边查询、HasEdge/HasPathFindPath、BFS、TryGetEdge/edge id 查询)。
  • 指明需要确认的索引约束(EdgeId 内建唯一索引、From/To 分组索引、节点集合是否由 From/To 自动推导)和校验规则(EdgeId/From/To 非空、EdgeId 唯一、Weight 必须可解析为数字、是否允许自环/重边/孤立节点/无向语义)。
  • 增加验收标准,要求在实现前冻结 API 命名与返回类型,确保文档、parservalidatortemplate 行为一致,并禁止在 DataTableProcessor 主流程继续添加 graph 专用分支,且引用了 docs/graph-table-design.md 作为背景设计。

Testing

  • 运行了自动检查 git diff --check,结果通过(无格式/空白错误)。
  • 使用 git status --short 验证工作区状态为已提交变更。

Codex Task

@0x5143
0x5143 merged commit b217f5b into master Jul 7, 2026
2 of 3 checks passed
@0x5143
0x5143 deleted the 2lezo4-codex/-graph-issue branch July 7, 2026 01:49

@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: 1e7f89a677

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

#### C4. 冻结 `graph` 表生成 API、索引约束与校验规范

背景设计参考:`docs/tree-table-design.md`。
背景设计参考:`docs/graph-table-design.md`。在进入实现前,需要先冻结 `graph` 表的生成 API、索引约束与校验规范,避免 parser、validator、template 与文档在后续实现中各自演进

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 Update graph freeze task to match implemented status

The referenced design doc already says graph is implemented and lists GraphTableParser/the graph template as added (docs/graph-table-design.md lines 3 and 126-130), so adding this as work to freeze "before implementation" makes the phase plan stale as soon as it is introduced. In this context, readers following the plan would treat completed parser/template/validation work as not started; please reword this as reconciling or validating the existing graph implementation and its remaining policy gaps.

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant