Repository navigation
composeStacks silently drops every non-array top-level key — api: today, server: as of #4910 #5005
Description
发现于 #4910 开发过程(入站 rateLimit seam),与本单修复无关,按 Prime Directive #10 单独立单,未认领。
事实
composeStacks(packages/spec/src/stack.zod.ts)只合并三类东西:
manifest—— 按manifest策略择一;i18n—— last-wins;objects+CONCAT_ARRAY_FIELDS里列出的数组集合 —— 拼接。
其余一律不进 composed。它从一个空对象 {} 开始逐项填充,所以任何既不是 manifest/i18n、也不在数组清单里的顶层键,合成后直接消失,没有告警。
今天受影响的顶层键:
| 键 | 谁消费它 | 合成后 |
|---|---|---|
api(enableProjectScoping / projectResolution / enforceProjectMembership) |
objectstack serve(serve.ts),转发给 REST + dispatcher |
丢 |
server(security.rateLimit / trustProxy,#4910 新增) |
objectstack serve → dispatcher 的入站限流器 |
丢 |
datasourceMapping |
数据源路由 | 丢 |
datasourceMapping 尤其值得一看:它是数组,只是没被列进 CONCAT_ARRAY_FIELDS(需实测确认)。
为什么这是 declared ≠ enforced 的形状
作者写下 server.security.rateLimit,单栈下真限流;把同一个栈丢进 composeStacks([base, addon]),限流静默消失,defineStack 不报错、validate 不报错、启动日志也不会说少了什么 —— 因为消费方看到的就是 undefined,与「没写过」无法区分。这正是 #4686 那一类缺陷换了个入口。
api.enforceProjectMembership 走同一条路径,后果是每环境成员 403 闸门在合成栈里悄悄关掉,安全影响比限流更直接。
复现(未跑,依据是源码 —— 请开工时先实测确认)
const a = defineStack({ manifest, server: { security: { rateLimit: { enabled: true, maxRequests: 5 } } } });
const b = defineStack({ manifest });
composeStacks([a, b]).server // → undefined注意 composeStacks 的短路:stacks.length === 1 时原样返回,所以单元素合成看起来是好的,只有真正 ≥2 个栈才丢 —— 这也解释了为什么至今没被发现。
待裁决(不要猜)
顶层标量/对象键的合成语义没有先例可循,需要维护者定一次,而不是每个键各定各的:
- A. last-wins(与
i18n一致)—— 最省事,但api/server是安全配置,后一个包静默覆盖前一个包的限流预算是危险的默认。 - B. 深合并 + 冲突报错 —— 两个栈都声明
server.security.rateLimit且值不同 →composeStacks抛错,和objectConflict: 'error'的既有姿态一致。 - C. 显式策略参数(
ComposeStacksOptions加一项,默认 B) —— 最贵,但把选择权交给作者。
另有一个与语义无关、无论选哪个都该做的:合成时丢弃任何未处理的顶层键,应该至少 warn 一次并点名。今天它是完全静默的,这才是真正让人查不出来的部分 —— 定了语义之后,凡是新增顶层键忘了接进 composeStacks 的,也会立刻自曝,而不是等下一次有人做 #4910 这样的活儿时偶然撞见。
关联:#4910(引入 server: 的单)、#4686(三份 RateLimitConfig 零 reader)、Prime Directive #10 / #12。
Activity
xuyushun441-sys commentedon Aug 4, 2026
维护者裁决(2026-08-04):冲突即报错 + 未处理键必警
api:被丢的孪生形态)、不做 deep-merge(制造新的静默歧义类)。needs-user-decision摘除,入队。Generated by Claude Code
xuyushun441-sys commentedon Aug 4, 2026
🔒 认领:PM 循环派发(批 15 验收后空位,裁决执行队列首位——活 bug)
会话:
session_01Ehu85kbvMcrNTUJjwxvLJ9分支:
claude/issue-5005-compose-stacks-merge-semanticsWorktree:
objectstack-issue-5005域:
domain:engine文件面:
composeStacks实现及其测试(objectql/runtime,按实测定位)、changeset。⛔ 不碰 packages/spec 源(若发现需要 spec 侧配合的新契约问题,needs_decision 不猜)。按 2026-08-04 裁决执行:同值放行、冲突报错带处方、未处理顶层键必警;⛔ 不做 last-wins / deep-merge;显式覆盖机制不预支。与在飞批 14(spec/ui)、#4956(spec scripts)文件面不相交。
Generated by Claude Code
xuyushun441-sys commentedon Aug 4, 2026
✅ 验收通过 —— PR #5053(已挂轨)
复核实况(3 文件;
stack.zod.ts零 zod schema 定义变更实查;authorable/api-surface 零漂移;⛔ releases/ 零触碰):composeStacks实住packages/spec/src/stack.zod.ts:1413而非派发词假设的引擎侧。dev 编辑 spec 源但保持契约字节级中性并显著申明偏差——fix(spec): make the bulk-action option item's openness deliberate, not accidental (#4001) #4909 诚实偏差先例,比空转一轮 needs_decision 正确。Record<keyof ObjectStackDefinition, ComposeDisposition>全覆盖,新顶层键不声明组合语义即 TS2741(删表项实证)。CONCAT_ARRAY_FIELDS由表推导,手工白名单退役。functions被丢意味着组合栈的全部声明式 handler 启动静默失败——比立单点名的限流丢失更重;七个声明数组集合(datasets/jobs/emailTemplates/docs/books/tiers/datasourceMapping)同样在漏。ADR-0109 曾按键打补丁(tools)正是白名单模式的病历。functions按名合并、重名报错、map/array 各保形态不互转。composeStacks的i18n仍是 last-wins —— #5005 裁决否掉的那个形状,只剩这一个键还在用 #5051(i18n last-wins 是仅存不一致键)已标 v18 地图——翻译叠加可能正是设计意图,无活伤害不进急件区。Generated by Claude Code
fix(spec)!: composeStacks 不再静默丢弃顶层键 —— 同值放行 / 冲突报错 / 未声明必警 (objectsta…
fix(spec): ADR-0087 别名转换按值处置被遮蔽的旧拼法(等值删除、异值双留) (objectstack-ai#4923) (o…
fix(spec): composeStacks refuses a non-array `objects` with an ADR-01…
test(spec): the first root-level test file group's titles state each …
test(spec): the second root-level test file group's titles state each…