Skip to content

E 系列收尾本体:摘除已失真的「structurally unreachable」注释族 + 多租户 decline 分支补零覆盖(#5231 的实施子单) #5399

Closed

Description

@baozhoutao

Part of #5231(父单为协调节点不直派,按 PM 规程拆此实施子单;另一子单 #5244 已随 PR #5395 落地)。

工作内容照父单正文两节,不重复展开:

1. 失真注释摘除(E7 翻转 d21c001 + E8 回迁后措辞已不成立)

台账纪律同 #5078:注释必须与实际一致。纯注释/台账修真,零行为变更。⚠️ 行号取自父单立案时,以 origin/main 现状为准逐处核对(#5385 等已重排过 runtime 文件)。

2. 多租户 decline 分支零覆盖

dispatcher-plugin.ts 的「多租户解析不出环境 → 弃权 + warn」分支(E5b)全仓无测试引用(kernel-resolver 提供方在 cloud)。补法照父单:packages/runtime 内以桩 kernelResolver 驱动集成测试,覆盖「按请求环境解析」与「解析不出 → 404 + warn」两条。范本:PR #5385 的 http-dispatcher.multi-tenant-concurrency.test.ts 已有桩 kernel-resolver 姿态可参照。

关联:#5231(父单)、#5040(E 系列)、#5112、PR #5230 / #5385。

Activity

  1. baozhoutao commented on Aug 5, 2026

    @baozhoutao
    ContributorAuthor

    🔒 认领:PM 循环第 5 轮(cli 车道)
    会话:session_01VkPSGsX9o17MsGv3Lbxu2w
    分支:claude/issue-5399-e-series-comment-truth
    Worktree:objectstack-issue-5399
    域:domain:cli
    文件面:issue 列出的 7 处注释(runtime 5 文件 + metadata 1 + rest 1,纯注释)+ dispatcher-plugin.ts 的 decline 分支集成测试 + changeset(若仓规要求)。与同批 #5397(doctor.ts)不相交。⚠️ 行号以 origin/main 现状逐处核对(#5385 重排过 runtime)。


    Generated by Claude Code

  2. baozhoutao commented on Aug 5, 2026

    @baozhoutao
    ContributorAuthor

    ✅ 验收通过 —— PR #5404,转 ready 挂合并队列

    对实况复核(cli 车道 PM,session_01VkPSGsX9o17MsGv3Lbxu2w):

    • 家族扫荡超出清单:7 处列出全部失真已改(行号全漂,含 metadata-manager 1551→2163 的大位移),grep 又抓出 6 处同族一并闭环(13 处);2 处核过判「过去时,本来就真」如实留档。issue 里 d21c001 在本仓不可解析 —— dev 以 endpoint-publish-gate.ts 的自述为权威另行确证翻转,处置正确。
    • E5b decline 补测:LiteKernel + 真 Hono + 真 socket(该分支活在 setFallbackHandler 闭包内无接缝可交出,上移一层的理由成立);反向验证抓出自己初版的伪绿(宿主无 metadata 槽位时分支因无关原因再次弃权,断言恒真)—— 改为宿主同路径声明后,删分支红得正是该分支要挡的事(无法归属的请求被宿主答成 200),修正过程写进测试抬头防复发。
    • 边界纪律:else 分支注释的条件句已触发(「翻转落地后此处必须响亮」),debug→warn 是行为变更 —— 不夹带,只把注释改真话并立 E7 翻转后,「适配器无 setFallbackHandler」仍只记 debug —— 声明式端点静默不可达,而代码注释自己写明此时应转 warn #5400,PM 已分诊入队(见该单)。
    • 零行为变更(全 diff 唯一非注释行是 ledger 的 note: 字符串,无断言引用);三包测试 + client 消费方 17 文件 / 222 用例全绿;skip-changeset 按 docs(examples): showcase 的 cacheTtl 注释写成 private —— 运行时发的就是它,且它是安全规则 (#5244) #5395 口径已挂。

    转 ready 挂 auto-merge,跟到 MERGED。落地后父单 #5231 两子单齐关,由 PM 收口关父单。


    Generated by Claude Code

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

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions