Skip to content

feat: v0.3.23 ABI 契约修复与绑定同步(review #67,冲突取 PR 方) - #69

Merged
miaobyte merged 1 commit into
mainfrom
fix/review-v0.3.23-on-main
Nov 1, 2026
Merged

miaobyte merged 1 commit into
mainfrom
fix/review-v0.3.23-on-main

Conversation

@miaobyte

@miaobyte miaobyte commented Nov 1, 2026

Copy link
Copy Markdown
Contributor

把 review PR #67 落到当前 main,冲突一律取 PR 方(用户裁决)。

做法

main(bf26030,含 #68 原地写收紧)上应用 coding4 的解冲突整树 patch:
workspace/583/coding4/patches/kvspace-c-pr67-on-main.patch(29 文件 +6226/−762)。

冲突处理(全部取 PR 方):

  • CMakeLists.txt 自动合并,两边 test target 都保留
  • src/durable_abi.c:SetPart → PR 的 kvspaceShmSetPart
  • src/kvspace_shm.h:取 PR(KVSPACE_SHM_H、format 6、SetPart/ListAt 原型)
  • src/kvspace.c:ViewWrite → PR 的 head-len;SetPartByRef 两路 → PR 的物理容量 + u64
  • main 独有的 test_slot_view.c 保留;test_inplace_body_only.c 保留但删掉「SetPart(offset=0) 必须失败」一段(取 PR 口径后必红)

验证(本机隔离构建)

  • CMake Release 构建 rc=0
  • ctest 12/12 通过(作者报 12/12,一致)
  • 作者另报:Python 21/21、Rust 3 passed

⚠ 已知后果(必须随提交留档)

body-only 守卫让位给 PR 口径后:kvspace-c 的 SetPart / SetPartByPart 只受整值范围与物理容量约束(仅 ViewWrite 仍等长),没有任何入口阻止写 head 或 attr。详见 ci-followups ⑪。

🤖 Generated with Claude Code

按用户裁决把 review PR #67 落到当前 main:冲突一律取 PR 方。

- shm/ABI 契约修复、List 遍历缓存、DelTree、Watch(None)
- xvalue_head 有界 langtype 读取与构造失败契约
- Python/Rust 绑定对齐 #571 wire 格式
- make test + CI 跑测试套件,deps.json pin kvspace v0.3.23

已知后果(提交历史留档):body-only 守卫让位给 PR 口径后,kvspace-c 的
SetPart/SetPartByPart 仅受整值范围与物理容量约束(仅 ViewWrite 仍等长),
没有任何入口阻止写 head/attr —— 见 ci-followups ⑪。

Co-Authored-By: Claude Code <noreply@anthropic.com>
@miaobyte
miaobyte merged commit 1d72c7c into main Nov 1, 2026
0 of 4 checks passed
@miaobyte
miaobyte deleted the fix/review-v0.3.23-on-main branch November 2, 2026 01:34
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