Repository navigation
NODE_ENV 未设置时 /discovery 广播 environment=development,而 os start 默认 NODE_ENV=production、CLI doctor 也按 production 解析 #5673
Description
发现于 #4828 的实施(界外发现,查重无命中故新开;未认领)。
现象
同一个「未设置 NODE_ENV」的事实,仓里有两套相反的默认:
| 位置 | 未设置 NODE_ENV 时的解读 |
|---|---|
packages/cli/src/commands/ 的 doctorNodeEnv() |
production(见 doctor-env-provenance.test.ts:expect(doctorNodeEnv({})).toBe('production')) |
seed-loader.ts 的 resolveSeedEnvFromNodeEnv() |
undefined —— 显式表示「宿主没说自己在哪」,由调用方决定含义 |
| discovery 生产者 | development(#4828 之前是 getEnv('NODE_ENV', 'development') 直塞,之后是同一默认经映射) |
seed-loader.ts 的注释还写着 os start 会把 NODE_ENV 默认成 production。
为什么值得单开
environment 是机器可读面上的字段,客户端拿它回答「我在不在生产环境」。一个忘记设 NODE_ENV 的生产部署,/discovery 会宣称 development —— 这个方向的错报正是危险的那个方向(客户端可能因此不显示生产警示、放宽破坏性操作的确认)。
#4828 的映射函数刻意保留了这个既有默认,没有顺手改:改默认是行为变更,会影响所有本地开发流程(本地开发通常不设 NODE_ENV,一改就全都变成 production),不属于该单裁定范围。所以那里只保证「识别不了的拼法绝不宣称 production」,没有解决「未设置时该信谁」。
可能的处置(需要维护者定)
- 三处统一到
production(最保守的安全默认),代价是本地开发不设NODE_ENV时 discovery 会说 production; - 三处统一到「未知」这个显式状态 —— 但
DiscoverySchema.environment是必填枚举,没有「未知」成员,需要先改契约; - 维持现状,但把差异写进文档,并要求生产部署显式设置
NODE_ENV(可以加一条doctor检查)。
第 2 条会改动公开契约形状,故不猜。
参考:#4828(discovery 面的枚举映射),Prime Directive #9(NODE_ENV 是第三方边界上的既有例外)。
Activity
os-zhuang commentedon Aug 5, 2026
分诊:决策类(
needs-user-decision)+ 预挂domain:cli。production改变全部本地开发流程的行为默认;② 需先给DiscoverySchema.environment增加「未知」成员,是公共契约形状变更;③ 维持现状 + 文档 + doctor 检查是产品取舍。错报方向(生产环境自称 development)是安全相关语义,不属分诊可代裁的范围。origin/main——doctor 侧在packages/cli/src/commands/doctor.ts,discovery 生产者在packages/runtime/src/http-dispatcher.ts:1277(getEnv('NODE_ENV', 'development')),两者均属 cli 车道;seed-loader.ts在packages/metadata-protocol(engine-core),但它只是对照面,三个方向的主要落点都不在它身上,故预挂domain:cli。pm:dispatched,同一 discovery 生产者文件——本单裁定后实施应等它落地再动,避免撞面);finding 「环境类别」这一个概念在 spec 里有两个枚举:DiscoverySchema.environment(3 成员)与 EnvironmentTypeSchema(7 成员) #5676(DiscoverySchema.environment与EnvironmentTypeSchema的枚举二元性——若走方向 2,两单应一并考虑)。本评论来自分诊座位 Routine(#5474 试点),不构成认领。
Generated by Claude Code
baozhoutao commentedon Aug 6, 2026
维护者裁定(2026-08-06,经 cli 车道 PM
session_01DWUR56YsttL5sTF72Q75TQ誊写):方案 1 —— 三处统一到production(保守安全默认),并入方案 3 的 doctor 半边;方案 2(契约加「未知」成员)否,不动DiscoverySchema.environment枚举。实施口径:
packages/runtime/src/http-dispatcher.ts的getEnv('NODE_ENV', 'development'))默认翻为production—— 撞面约束已消(两个 discovery 生产者都在线上返回 schema 未声明的顶层字段(scoping / features / endpoints),且 REST 形状永远无法通过 DiscoverySchema #4828 已于 08-06 00:24Z 落地);NODE_ENV未设置时提示「已按 production 处理,请显式设置」,让缺省状态响亮而非仅文档化;pnpm dev类脚本显式设NODE_ENV=development(实现细节,dev 处理并在 PR 正文说明影响面);seed-loader.ts(packages/metadata-protocol,engine-core 域)是对照面 —— 若实施发现必须改它才能满足「三处统一」,停手报告转跨域处理,⛔ 不越面直改。标签:摘
needs-user-decision→pm:queue(domain:cli已在),归 cli 车道排程(runtime 半边排 #5085 → #5613 之后,doctor 半边与 cli 包串行序协调)。Generated by Claude Code
baozhoutao commentedon Aug 6, 2026
认领:PM 循环第 4 轮(cli 车道)
会话:
session_01DWUR56YsttL5sTF72Q75TQ分支:
claude/issue-5673-node-env-unified-productionWorktree:
objectstack-issue-5673域:
domain:cli文件面:
packages/runtime/src/http-dispatcher.ts(discovery 生产者默认)+packages/cli/src/commands/doctor.ts(新检查)+ 对应测试 + changeset(+ 如需,本地 dev 脚本显式NODE_ENV=development及文档一处);⛔ 不动packages/metadata-protocol/seed-loader.ts(engine-core 域,对照面;若实施证明必须改它才能统一,停手报告)执行依据:维护者 2026-08-06 已拍方案 1 + doctor(裁决评论在本 thread):三处统一
production保守安全默认,doctor 新增「NODE_ENV 未设,已按 production 处理」提示,⛔ 不改DiscoverySchema.environment枚举(方案 2 否)。串行注记:#5429(同落 doctor.ts)押后至本单落地;churn:PR #5819(已入队)动过environment-variables.mdx与 serve.ts,合 main 后再动文档;#4828 已落地,discovery 生产者文件撞面约束已消。Generated by Claude Code
baozhoutao commentedon Aug 6, 2026
认领解除(cli 车道 PM,
session_01DWUR56YsttL5sTF72Q75TQ,2026-08-06 07:4xZ):06:5xZ 派发的 dev 被维护者手动中止(宿主明确回报,零推送、无远端分支),配合车道收尾指示,摘 assignee 与pm:dispatched,回pm:queue。裁决不受影响:维护者已拍「方案 1 + doctor」(本 thread 04:0xZ 裁决评论),下次派发直接按裁决 + 07:0xZ 认领评论的执行注记执行,无需重新分析;唯一需刷新的是 churn 基线(#5819/#5821 已合入,environment-variables.mdx与serve.ts以合并后现状为准)。Generated by Claude Code
baozhoutao commentedon Aug 6, 2026
恢复认领 + 重派:PM 循环第 6 轮(cli 车道)
会话:
session_01DWUR56YsttL5sTF72Q75TQ分支:
claude/issue-5673-node-env-unified-production(不变)Worktree:
objectstack-issue-5673(本会话容器,dev 自建)域:
domain:cli;文件面同 07:0xZ 认领评论(runtime http-dispatcher 的 discovery 默认 + cli doctor 新检查 + 测试 + changeset;⛔ seed-loader/spec 不动)07:4xZ 中止门已随维护者「继续循环」示意解除。执行依据不变:裁决「方案 1 + doctor」。churn 刷新:以最新 origin/main(
d9cac6019)为准——#5895 刚动过packages/runtime/src/domains/meta.ts(与本单 http-dispatcher.ts 不相交)、#5863 加了check:dev-prereqs(与 dev 脚本影响面盘点相关,如实纳入)、#5819 动过environment-variables.mdx(触该文档以现状为准)。Generated by Claude Code
baozhoutao commentedon Aug 6, 2026
验收:ACCEPT(cli 车道 PM,
session_01DWUR56YsttL5sTF72Q75TQ,2026-08-06 13:1xZ)→ PR #5951production(只动getEnv第二参,spec 映射函数与枚举零触碰);两个 discovery 生产者都在线上返回 schema 未声明的顶层字段(scoping / features / endpoints),且 REST 形状永远无法通过 DiscoverySchema #4828「认不出的拼法绝不宣称 production」独立钉住不回退;doctor 新增nodeEnvCheck()(warning 档、不改退出码、--verbose 给设置命令),已设置零噪音。os dev→serve --dev在 runtime import 前就地设NODE_ENV=development,feat(devx): 根 dev 入口新增构建完整性前置 check:dev-prereqs —— 一个前置条件一句修法 (#5795) #5863 的check:dev-prereqs不读不设;受影响面收敛为裸os serve与自建入口,changeset 迁移说明如实。/discovery有第二个生产者(metadata-protocolgetDiscovery(),经 rest 暴露)未设时仍报 development——跨域边界正确停手,四处留档(调用点注释/docs Callout/changeset/PR 正文)+ 立 NODE_ENV 未设置时,第二个 /discovery 生产者(metadata-protocol 的 getDiscovery,经 @objectstack/rest)仍广播 environment=development #5936(已被分诊送 spec 决策箱:首选修法在共享映射函数,单点决策)。本单验收判据①在 runtime 生产者面上成立,rest 面的收敛随 NODE_ENV 未设置时,第二个 /discovery 生产者(metadata-protocol 的 getDiscovery,经 @objectstack/rest)仍广播 environment=development #5936 定案。success;反向验证方向一致(撤翻转 → 2 红皆新钉,8 条拼法用例绿)。转 ready + 挂 auto-merge,两读数跟到 MERGED;落地后 doctor.ts 与 runtime 解锁,接续派 #5429 + #5613 并批。
Generated by Claude Code
fix(cli): os doctor reads an unset NODE_ENV as development in the sou…
fix(cli): os doctor reads an unset NODE_ENV as development in the sou…