From 06e3f1bca4f20f4c0f18e3f4dd5b05b5e1ab6c88 Mon Sep 17 00:00:00 2001 From: weichen Date: Wed, 21 Oct 2026 10:52:28 +0000 Subject: [PATCH] Record scope, review PASS and verification for the port allocator Why: the card must carry the helper's design boundary, the (r1 scope-expansion -> granted expansion -> r2 success) trajectory, the independent review verdict and the runtime evidence before delivery. Evidence: /tmp/ports-load-20261021.json: helper test under 64-load OK /tmp/ports-svc-tests-20261021.json: tests.test_svc 23 OK /tmp/ports-governance-20261021.json: 527 OK (18 skipped) /tmp/remaining-ports-impl-r2-20261021.json: cursor, success /tmp/remaining-ports-review-r1-20261021.json: pi-glm, PASS Impact: card metadata only. Refs: PR #676; fix/remaining-hardcoded-ports --- .task/fix-remaining-hardcoded-ports.md | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/.task/fix-remaining-hardcoded-ports.md b/.task/fix-remaining-hardcoded-ports.md index ec0bf002..60f1df2a 100644 --- a/.task/fix-remaining-hardcoded-ports.md +++ b/.task/fix-remaining-hardcoded-ports.md @@ -7,7 +7,7 @@ - 分支类型：standalone；root_target：main；immediate_target：main；accepting_requirements：false - 需求分流：verdict=NEW；selected_initiative=无；requirement_sha256=f58f9b9127d88193b75ab8240478a0be0643a63075d58cc54884dfc7b9b4e64e；triage_receipt=/tmp/triage-remaining-port-flakes-20261021.json；receipt_sha256=e190b63d6a5bbcb0285d38a8ae9e1a548a6fb3e3a16dffe8e96dd54cc344a27b -- 授权与交付终点：<本次允许的动作与终点；未授权事项写明> +- 授权与交付终点：消除 PR #668 Review 记下的同类端口硬编码脆弱面，提取 helper 并应用到 tests/test_svc.py 与 tests/test_svc_realcombo.py 的相关位点。不得改 scripts/svc.sh、不得削弱断言、不得改动 test_svc_recover.py / test_task_pr_check.py / test_task_triage.py 的端口。 - 交付模式：goal-first（目标、范围和交付终点一次确认；范围内连续推进） - 实施执行：agent-per-unit（配置：AGENT-EXECUTION.md） - 合入执行：agent（已授权交付范围内，Review 通过、冲突解决且当前 HEAD 的 CI 通过后合入） @@ -39,14 +39,14 @@ ## 结果与验证（完成后填写） -- 结果： -- 验证：<结论摘要 PASS/FAIL/SKIP/NOT_RUN 逐项；逐条证据由 make evidence 写入上方证据表> -- 影响与剩余项： +- 结果：新增 `tests/_port_alloc.py`（`free_test_port()` 每次调用通过 socket reservation 拿一个真正空闲的端口，with 块退出立即 close）。修改 `tests/test_svc.py`（6 处 `svc ... --port 18842/18843/18844` 改为每次 alloc 一个端口，668 与 942 行使用同一分配值因为是同一测试中相同实例的重复确认）和 `tests/test_svc_realcombo.py`（3 个用例各 alloc 一次端口，seed_real/svc/http_ok 三处共享）。`register_pg`/`DSN 18951` 按 PR #668 Review 注明仅写 svc.env、从不绑端口，保留不动；`test_svc_recover.py` 已有自己的 `free_fixture_port` 不动；`test_task_pr_check.py`/`test_task_triage.py` 的端口仅作字典键，不绑端口。 +- 验证：PASS——`tests.test_port_alloc` 在 64 负载下 OK；`tests.test_svc` 23 项 OK；治理套件 527 项 OK（18 skipped）；PR #668 的目标用例 `test_static_fresh_deploy_registers_pidcheck_and_starts` 仍通过。独立 Review（pi-glm，PASS 无阻塞项）。 +- 影响与剩余项：不改任何生产代码；不削弱断言；realcombo 三例的运行级验证 NOT_RUN（环境门禁与禁网限制），由代码走读与模块级 skip 确认替代。Review 记 3 项非阻塞建议（守护测试未覆盖「外部占用」与 TIME_WAIT 避让、文件名单硬编码而非 glob、helper API 单端口 vs 真实用例可能需多端口对），均不阻塞。 ## 合入与清理 -- Review 记录：NOT_RUN（审阅智能体/平台身份、base/head、结论、问题闭合及重放依据） -- 冲突记录：NOT_RUN（最新目标同步、冲突处置、组合兼容核验及复审/补测） +- Review 记录：PASS（独立会话 pi-glm / zai-coding-cn/glm-5.3-flash；base 444fb1dee2a72452be23b97cbaaad9e28c42dffd，head 076a686592f99abb7b4e8c1aa73b7c73b5bccefe；报告 `.task/reviews/2026-10-20-remaining-ports-review.md`；无阻塞项，3 项非阻塞建议；重放依据：host 独立跑治理套件与 64 负载下的 helper 测试） +- 冲突记录：NOT_RUN（一次交付前同步，无变更） - CI 记录：NOT_RUN（当前 HEAD 完整 SHA、全部必需检查、运行链接、执行身份及 GATE 结果） - 合入记录：NOT_RUN（交付授权范围、PR、被核验 HEAD、合入 SHA、目标分支及内容核验） - 清理记录：NOT_RUN（授权依据、写者退出确认、worktree/分支/任务卡处置）