fix(cosh-ng): accept quoted dev null as sink

[jfeng18]

Why

A quoted redirect target such as 2>"/dev/null" was classified as a filesystem write (redirection-write) instead of a null sink, so the approval card showed a misleading reason even though the unquoted form was already classified correctly (#1667).

What changed

• Fully quoted SAFE_OUTPUT_SINKS targets are now recognized as null redirections.
• Every other quoted form stays fail-closed (unchanged).
• apply_null_redirection_policy still resolves to AskUser — noted in a code comment. Layer 2 (auto-approving null redirections) is intentionally out of scope and remains a separate discussion.

Related issue

Refs #1752 (not closing): this PR only fixes the Layer 1 misclassification; the issue's full auto-approve request is not resolved here.

User / Agent impact

The approval-card reason/impact for quoted null redirections now reads as a null redirection instead of redirection-write, aligning with the unquoted behavior from #1667. The execution decision (AskUser) is unchanged.

Risk and compatibility

• Public CLI, API, configuration, or documented behavior changed
• Privileged or security-sensitive behavior changed
• Cross-component contract changed
• Migration or rollback guidance is needed

Execution decision is identical before and after (AskUser); only the approval-card reason/impact changes. Low risk.

Validation

• ECS discriminative two-direction test: misclassification reproduces on pre-fix code, correct classification on fixed code.
• ECS cargo test: 1342 passed (baseline 1340 + 2 new).
• cargo fmt and clippy clean with zero warnings.

Documentation and rollback

Single commit — revert it directly.

[qoderai]

本次审查范围内未发现需要修改的问题。

---

🤖 Generated by Qoder • View workflow run

[SunnyQjm]

PR number: #2710
head_sha: 9bf29dc
reviewed_at: 2026-08-20T08:32:56Z

Findings

• [P1] CI 实测与 PR 验证声明冲突：Test cosh-ng（08:19:25Z）与 Test cosh-ng fast checks（08:20:04Z）在 head_sha 9bf29dc1 上均为 FAILURE，而 PR body 声称 ECS cargo test 1342 passed。静态评审无法判断失败原因（可能是慢 PTY 集成测试抖动，也可能是本 patch 引入的回归），但 Build cosh-ng release 成功、失败恰好落在被改 crate 的测试门上，合并前必须由作者给出失败日志解释或重跑转绿。
• [P2] command_risk_parser.rs 本次新增约 74 行（按 diff 增量估算）。该文件未登记于 large-file-inventory，输入中无其基线行数，无法确认叠加后是否越过 700 行阈值。按规则行数判定以 CI gate 实测为准，此处仅为估算提示：若实测越 700，需补 owner note 或拆分计划。

结构评审结论

未发现 blocking package/module/public API 组织问题。逐项核对：

• Owner 归属：三个文件全部落在 tools/（command risk 分类/解析/测试），符合 standard.md §2 中 tools/ 承担 command risk 的职责定义，无跨 owner 混杂。
• Public API：SAFE_OUTPUT_SINKS 仍为 private const，parse_command 维持 pub(super)，apply_null_redirection_policy 仅加注释；lib.rs 未触碰，inventory 分类计数不受影响。
• Root src / 入口文件：未新增 root implementation 文件，未触碰 lib.rs/main.rs。
• Forbidden dependency：未引入新 import，tools 未新增对 runtime/agent/ui 等方向的依赖。
• Self-crate path：未新增 cosh_shell::...。
• 测试位置：新增用例放入既有 tools/command_risk_tests.rs 决策矩阵，与 [cosh-ng] feat(cosh-ng): distinguish null redirection from filesystem writes #1667 边界测试职责一致。
• 范围：仅 cosh-shell crate 内变更，无跨 crate 混入。

安全语义复核（非 blocking）

本 PR 收窄 fail-closed 规则（整词带引号的 /dev/null 改判 null sink），属安全敏感分类器变更。复核认为方向保守可控：开引号须紧邻重定向目标位置、内容逐字节等于 allowlist 条目、闭引号后必须为词边界（刻意排除 {}）；未闭合引号、转义、后缀拼接（'/dev/null'x）、相邻引号续接、$/通配符、空引号、引号内空白全部落回 RedirectionWrite。正反两向测试齐备（含 &>'/dev/null'、argv 不泄漏断言、AskUser 不变的 V-M10），执行决策未变。map_or + #[allow(clippy::unnecessary_map_or)] 为 MSRV 1.74 兼容写法，已有注释说明。

剩余风险

• 内层内容扫描与 word_ends 各维护一份分隔符集合（{} 差异为刻意保守且有注释），后续修改需两处同步。
• PR body 声称 1342 passed（基线 1340 + 新增 2），与 diff 中两个新 #[test] 相符，但与上述 CI FAILURE 矛盾，以 CI gate 为准。

未跑验证

按评审约束未执行任何工具与构建命令；cargo test、check-layout.sh、inventory-public-api.sh、fmt/clippy 均未本地复验，全部以 CI gate 实测为准——当前 Test cosh-ng 两门为 FAILURE，是合并前必须清零的首要事项。

[jfeng18]

Both findings addressed in 7c77cb7 (single commit, force-pushed):

P1 (CI vs body) — two distinct failures, both explained:

• Test cosh-ng fast checks: the audit gate failed on the test-inventory overlap ratchet — the two new tests had landed in tools/command_risk_tests.rs, which is compiled into both the lib and bin test targets (main.rs re-declares lib modules), pushing the lib/bin overlap to 696 > 694 ceiling. They were moved to tools/command_risk_quoted_tests.rs, declared only from lib.rs via the existing wrap_tests pattern (the parser internals are pub(super), so a tests/ integration file cannot reach the assessment API either), restoring overlap = 694 = ceiling — the audit gate now passes locally.
• Test cosh-ng: provider_lifecycle::cosh_core_sync_drains_child_output_while_writing_large_prompt timed out after its 3s budget (71/72 pass on that job; the same code is green on main; zero interaction with this diff — it does not touch provider lifecycle). Will re-run the failed job off-peak to confirm the environmental attribution.

The quoted-sink behavior itself is unchanged; full ECS re-verification on the amended commit is green (fmt/clippy clean, --lib 1344 passed / 0 failed = baseline 1342 + the 2 relocated tests).

P2 (parser line-count registration) — Deferred with data: command_risk_parser.rs is 517 lines after this change (+99) and command_risk_build.rs is 691 (+5), both under the 700 threshold. Registration belongs to the large-file-inventory maintenance flow and will follow if and when an actual crossing occurs.

[SunnyQjm]

PR number: #2710
head_sha: 7c77cb7
reviewed_at: 2026-08-21T06:20:14Z

Findings

• [P3][advisory] src/cosh-ng/crates/cosh-shell/src/tools/command_risk_build.rs:510 本次 +5 行（纯注释），作者声明当前 691 行，距 700 行 owner-note 阈值仅剩约 9 行余量。行数判定以 CI gate 实测为准（本轮 fast checks 已 SUCCESS，未越线），此处仅提示后续增长需提前登记或拆分。
• 无 blocking findings。

结构评审结论

未发现 blocking package/module/public API 组织问题。逐项核对：

• Owner 归属：全部改动落在 tools/（command risk 解析/构建/测试），符合 standard.md §2 中 tools/ 承担 command classification/risk 的职责定义，无跨 owner 混杂。
• Public API：lib.rs 仅新增 #[cfg(test)] #[path] mod command_risk_quoted_tests;（私有、test-only），不是 pub mod/pub use，inventory 分类计数不受影响；SAFE_OUTPUT_SINKS 仍为 private const，parse_command 维持 pub(super)，未为测试新增 production pub（作者说明因 pub(super) 不可达，测试经 assessment API 断言可观察效果，此边界成立）。
• Root src / 入口文件：新测试文件位于 tools/ 下，未新增 root implementation 文件。
• Forbidden dependency / self-crate path：未引入新 import，未新增 cosh_shell::...。
• 范围：仅 cosh-shell crate 内变更，无跨 crate 混入，单一 commit。

上轮 findings 复核

• 上轮 P1（CI FAILURE 与 body 声明冲突）：当前 head 7c77cb727 上 Test cosh-ng（04:06:02Z）与 Test cosh-ng fast checks（04:04:49Z）均为 SUCCESS；作者对 overlap ratchet（696>694 回落至 694）与 provider_lifecycle 超时的解释合理，本 diff 不触碰 provider lifecycle。关闭。
• 上轮 P2（parser 行数登记）：作者给出实测 parser 517 行（+99）、build 691 行（+5），均低于 700 阈值，且 layout audit 已随 fast checks 转绿。关闭，保留上方 P3 余量提示。

安全语义复核（非 blocking）

收窄 fail-closed 规则的方向保守：开引号须紧邻重定向目标位置、内容逐字节等于 allowlist 条目、闭引号后必须为词边界（刻意排除 {} 且有注释）；未闭合引号、转义、后缀拼接（'/dev/null'x）、$/通配符、空引号、引号内空白均落回 RedirectionWrite。正反两向测试齐备（含 &>'/dev/null'、AskUser 不变断言），执行决策未变。#[allow(clippy::unnecessary_map_or)] 为 MSRV 1.74 兼容写法，已有注释。

剩余风险

• 内层内容扫描与 word_ends 的分隔符集合各维护一份（{} 差异为刻意保守），后续修改需两处同步。
• 测试文件通过 lib.rs #[path] 声明而非 tools/ owner 目录挂载，作者声明为既有 wrap_tests pattern 且服务于 overlap ratchet；测试位置最终口径以测试 SDD 与 CI 的 test-inventory gate 为准。

未跑验证

按评审约束未执行任何工具；cargo test、check-layout.sh、inventory-public-api.sh、fmt/clippy 均未本地复验，全部以 CI gate 实测为准——当前 head 上相关检查已全绿。

[SunnyQjm]

PR number: #2710
head_sha: 26035a8
reviewed_at: 2026-08-28T03:05:54Z

Findings

• 无 blocking findings。
• [P3][advisory] src/cosh-ng/crates/cosh-shell/src/tools/command_risk_build.rs:510 本次 +5 行纯注释，作者声明该文件当前 691 行，距 700 行 owner-note 阈值余量约 9 行。行数判定以 CI gate 实测为准（当前 head 上 fast checks 已 SUCCESS，未越线），仅提示后续增长需提前登记或拆分。

结构评审结论

未发现 blocking package/module/public API 组织问题。逐项核对：

• Owner 归属：全部改动落在 tools/（command risk 解析/构建/测试），符合 standard.md §2 中 tools/ 承担 command classification/risk 的职责定义，无跨 owner 混杂。
• Public API：lib.rs 仅新增 #[cfg(test)] #[path] mod command_risk_quoted_tests;（私有、test-only），不是 pub mod/pub use，inventory 分类计数不受影响；SAFE_OUTPUT_SINKS 仍为 private const，parse_command、apply_null_redirection_policy 维持 pub(super)，未为测试新增 production pub。
• Root src / 入口文件：新测试文件物理位于 tools/ 下，未新增 root implementation 文件；lib.rs/main.rs 无 implementation 混入。
• Forbidden dependency / self-crate path：未引入新 import；新测试模块使用 crate::tools::command_risk::...，未新增 cosh_shell::... self-crate path。
• 范围：仅 cosh-shell crate 内变更，无跨 crate 混入，作者声明单 commit 可直接 revert。

与上轮评审的关系

本轮 head_sha（26035a80）与上轮已评审 head 完全相同，patch 文件集与改动内容未见任何变化；上轮 findings 已全部关闭。PR updated_at 推进仅来自作者 2026-08-28 的 ping 评论，非代码变更。当前 head 上 Test cosh-ng、Test cosh-ng fast checks、Build cosh-ng release 均 SUCCESS，早期 CI FAILURE 已清零。

安全语义复核（非 blocking）

本 PR 收窄 fail-closed 规则（整词带引号的 /dev/null 改判 null sink），属安全敏感分类器变更。方向保守可控：

• 开引号须紧邻重定向目标位置；扫描基于 lookahead.clone()，豁免不成立时原迭代器状态不变，完整落回既有 fail-closed 路径。
• 去引号内容须逐字节等于 allowlist 条目；SAFE_OUTPUT_SINKS 条目不含 $/反引号/反斜杠，引号不可能引入展开。
• 闭引号后必须为词边界（刻意排除 {/} 且有注释）；未闭合引号、空引号、引号内空白、后缀拼接（'/dev/null'x）、&>'/dev/null'、$/通配符均保持 RedirectionWrite。
• 正反两向测试齐备（含 AskUser 不变断言与 argv 不泄漏的可观察效果钉住），执行决策未变；#[allow(clippy::unnecessary_map_or)] 为 MSRV 1.74 兼容写法，已有注释。

剩余风险

• 内层内容扫描的分隔符集合（含 {}）与 word_ends 的集合（不含 {}）各维护一份，差异为刻意保守且有注释，后续修改需两处同步。
• 测试模块经 lib.rs #[path] 声明（wrap_tests pattern）以规避 lib/bin overlap ratchet，而非 tools/ owner 目录挂载；测试位置最终口径以测试 SDD 与 CI test-inventory gate 为准（当前已转绿）。
• 新豁免挂在 !guarded 门控内，guarded 模式下与 unquoted 路径的一致性依赖 diff 之外的既有代码结构；即便不一致方向也是 fail-closed（RedirectionWrite → AskUser），执行决策不变。
• command_risk_build.rs 行数余量小（见 P3）。

未跑验证

按评审约束未执行任何工具；cargo test、check-layout.sh、inventory-public-api.sh、fmt/clippy 均未本地复验，全部以 CI gate 实测为准——当前 head 上相关检查已全绿。

[jfeng18]

@SunnyQjm friendly ping — this is the sibling fix from the same batch as #2708 (same readonly hardening series). Would appreciate your review when you have a cycle.

[SunnyQjm]

PR number: #2710
head_sha: 26035a8
verified_at: 2026-08-28T03:25:10Z
tier: T1

PR #2710 验证报告：fix(cosh-ng): accept quoted dev null as sink

1. 验证范围

• 现场核对：git rev-parse HEAD = 26035a80bd9ca7422955015eed04b43d06e0e0a2，与作业 head_sha 一致。
• Doctor：ok=true，contended=false（load_1m 2.5 / 12 核）；container runtime 探测为 Apple container，但按既有验证环境惯例（宿主机 Apple container 缺 images 插件）profile 固定 runtime = "docker"。
• 环境：container（docker），RUN_ID 20260828T031825Z-13e754a9，镜像 alinux3/alinux3:latest（本机已有），arm64。示例 profile 含占位符，本次基于 container.example.toml 生成临时 profile（去 auth 段、cache_path 指向既有 runtime/cosh-lab-cache/container-arm64），用后已删除。
• scope 校验：作业 scope cargo test -p cosh-shell --lib command_risk 与 test plan 推荐（cargo test --locked --package cosh-shell，crate 级）一致且为其聚焦子集，采用作业 scope。
• 已运行：
  • cosh-lab test run --on 20260828T031825Z-13e754a9 --scope unit --filter command_risk
  • 原生命令：cargo test --locked --package cosh-shell --lib command_risk
  • 测试 RUN_DIR：.cosh-lab/runs/20260828T032152Z-321b230d

2. 结果分类

通过（focused green），首跑即成功，无需复跑：

• exit_code = 0，编译 30.1s，测试 0.01s
• test result: ok. 44 passed; 0 failed; 0 ignored; 1305 filtered out
• 本 PR 新增的两个测试均执行并通过：
  • command_risk_quoted_tests::quoted_safe_output_sink_redirection_is_null_suppression ... ok
  • command_risk_quoted_tests::quoted_non_sink_redirection_targets_stay_fail_closed ... ok
• 既有决策矩阵测试（redirection_fail_closed_paths_stay_high、null_redirection_*、fd_duplication_* 等 42 个）全部通过，确认收窄未破坏既有 fail-closed 语义。

3. Findings 验证

[P3][advisory] command_risk_build.rs:510 行数余量 —— 证实（数值准确，当前未越线）：

• 实测 wc -l = 691 行，与作者声明一致；按 cosh-ng AGENTS.md 的 700 行 owner-note 阈值，余量恰为 9 行。
• diff 确认该文件本次 +5 行（507 行后）确为纯 doc 注释（issue [TestAgent][cosh-ng] bug: 只读搜索命令 find ... 2>/dev/null 被 readonly_rules 误判为 redirection-write high risk,触发 Approval panel(应 Auto-approved) #1752 政策说明），无代码变更。
• finding 中「CI fast checks 已 SUCCESS」属 CI 侧事实，本地不复验 CI；但行数阈值本身（691 < 700）已本地证实未越线。advisory 提示成立。

结构评审与安全语义复核关键点抽查（非 findings 条目，顺带核验，均与静态评审结论一致）：

• lib.rs 仅新增 #[cfg(test)] #[path = "tools/command_risk_quoted_tests.rs"] mod command_risk_quoted_tests;（test-only 私有模块，diff 确认）。
• 新豁免确实挂在 if !guarded 门控内；扫描基于 lookahead.clone()，豁免不成立时原迭代器状态不变，落回既有 fail-closed 路径（command_risk_parser.rs 新增块）。
• 去引号内容经 SAFE_OUTPUT_SINKS.contains(&quoted_target.as_str()) 逐字节匹配；word_ends 边界集刻意不含 {/}（内层扫描 break 集含），与注释声明一致。
• 反向用例（2>'/dev/null'x、&>'/dev/null'、2>"$F"、2>' /dev/null' 等）在测试中保持 High/redirection-write，且容器内实测通过。

4. 补充测试

无。本作业为 T1 且无 repro_hint；PR 自带正反两向测试（+98 行新测试文件）已在容器内全部执行通过，无需补写回归测试。

5. 证据索引

无截图证据（T1 作业，不涉及 PTY/渲染验证，evidence_dir 未产生文件）。

6. 未运行 scope

• cargo test --package cosh-shell 全量（logic / protocol / raw-cli / shell-host / 全量 lib 中其余 1305 个被过滤测试）
• cargo test --workspace（cosh-ng 其余 crate）
• fmt / clippy / check-layout.sh / check-test-inventory.sh 等静态 gate（以 CI 实测为准，本地未复验）

7. 结论

Go —— focused scope（cargo test --locked --package cosh-shell --lib command_risk，44/44 通过，含 2 个新增引号 sink 测试）在隔离 Linux 容器内首跑通过；唯一 advisory finding（行数余量 691/700）经本地实测证实但未越线，静态评审结论无需修正。focused green 不构成全量/CI 通过，合入门禁仍以 CI 为准。

[jfeng18]

@SunnyQjm @KaiLongZhou 求 review。小修复：2>"/dev/null" 引号形式之前被误分类为 redirection-write。只动 Layer 1 分类，Layer 2（自动审批）明确留到 #1752 另议。CI 全绿。

[SunnyQjm]

PR number: #2710
head_sha: 2634e78
reviewed_at: 2026-08-31T08:07:56Z

Findings

• 无 blocking findings。
• [P3][advisory] src/cosh-ng/crates/cosh-shell/src/tools/command_risk_build.rs:510 本次 +5 行纯注释（issue [TestAgent][cosh-ng] bug: 只读搜索命令 find ... 2>/dev/null 被 readonly_rules 误判为 redirection-write high risk,触发 Approval panel(应 Auto-approved) #1752 政策说明），作者声明该文件 691 行、距 700 行 owner-note 阈值余量约 9 行；此前 T1 验证已在同一 patch 内容上实测 691 行证实。行数判定以 CI gate 实测为准——当前 head 上 Test cosh-ng fast checks 已 SUCCESS，未越线。仅提示后续增长需提前登记或拆分。

与上轮评审的关系

本轮 head（2634e78e）与上一轮已评审 head 完全相同，diff 文件集与内容未见任何变化；PR updated_at（2026-08-31T07:22:23Z）恰为上轮评审评论的发布时间，判断为评论发布触发的无代码变化重触发。上轮评审结论与 findings 原样成立，本轮无新增问题。

结构评审结论

未发现 blocking package/module/public API 组织问题。逐项核对：

• Owner 归属：全部改动落在 tools/（command risk 解析/构建/测试），符合 standard.md §2 中 tools/ 承担 command classification/risk 的职责定义，无跨 owner 混杂。
• Public API：lib.rs 仅新增 #[cfg(test)] #[path = "tools/command_risk_quoted_tests.rs"] mod command_risk_quoted_tests;（私有、test-only），不是 pub mod/pub use，inventory 分类计数不受影响；SAFE_OUTPUT_SINKS 仍为 private const，parse_command、apply_null_redirection_policy 维持 pub(super)，未为测试新增 production pub。
• Root src / 入口文件：新测试文件物理位于 tools/ 下，未新增 root implementation 文件；lib.rs/main.rs 无 implementation 混入。
• Forbidden dependency / self-crate path：未引入新 import；新测试模块使用 crate::tools::command_risk::...，未新增 cosh_shell::... self-crate path。
• 大文件：command_risk_build.rs +5 行纯注释（实测 691 行）、command_risk_parser.rs 作者声明 517 行，均低于 700 行登记阈值；两文件未出现在 large-file-inventory 不构成 finding（规则：台账文件未出现在 changed files 不得判"未登记"）。
• 范围：仅 cosh-shell crate 内变更，无跨 crate 混入，作者声明单 commit 可直接 revert。

安全语义复核（非 blocking）

本 PR 收窄 fail-closed 规则（整词带引号的 /dev/null 改判 null sink），属安全敏感分类器变更。方向保守可控：

• 开引号须紧邻重定向目标位置；扫描基于 quoted_scan = lookahead.clone()，豁免不成立时原迭代器状态不变，完整落回既有 fail-closed 路径。
• 去引号内容须逐字节等于 allowlist 条目；SAFE_OUTPUT_SINKS 条目不含 $/反引号/反斜杠，引号不可能引入展开。
• 闭引号后必须为词边界（word_ends 刻意排除 {/} 且有注释，防止 '/dev/null'{a,b} 花括号拼接逃逸）；未闭合引号、空引号、引号内空白、后缀拼接（'/dev/null'x）、&>'/dev/null'、$/通配符均保持 RedirectionWrite。
• 正反两向测试齐备（含 AskUser 不变断言与 argv 不泄漏的可观察效果钉住），执行决策未变；#[allow(clippy::unnecessary_map_or)] 为 MSRV 1.74 兼容写法，已有注释。
• 豁免成功后的迭代器同步（consumed + quoted_consumed）与 fd token 清理（token.clear(); token_quoted = false;）同既有 unquoted sink 路径同构，行为已被 CI 及上一轮 T1 focused 实测覆盖。

剩余风险

• 内层内容扫描的分隔符集合（含 {}）与 word_ends 的集合（不含 {}）各维护一份，差异为刻意保守且有注释，后续修改需两处同步。
• 测试模块经 lib.rs #[path] 声明（wrap_tests pattern）以规避 lib/bin overlap ratchet，而非 tools/ owner 目录挂载；测试位置最终口径以测试 SDD 与 CI test-inventory gate 为准（当前已转绿）。
• 新豁免挂在 !guarded 门控内，guarded 模式下与 unquoted 路径的一致性依赖 diff 之外的既有代码结构；即便不一致方向也是 fail-closed（RedirectionWrite → AskUser），执行决策不变。
• command_risk_build.rs 行数余量小（见 P3）。

未跑验证

按评审约束未执行任何工具；cargo test、check-layout.sh、inventory-public-api.sh、fmt/clippy 均未本地复验，全部以 CI gate 实测为准——当前 head 2634e78e 上相关检查已全绿。上一轮 T1 focused 验证（cargo test -p cosh-shell --lib command_risk，44/44 通过）在与本 patch 内容一致的 head 26035a80 上完成，建议 gate 按同 scope 对当前 head 复验一次。

[SunnyQjm]

PR number: #2710
head_sha: 2634e78
verified_at: 2026-08-31T14:33:35Z
tier: T1

自动验证未完成。原因：验证会话超时（3000s），可能是验证范围过大或环境阻塞

静态评审结论不受影响；CI 仍是合入门禁权威。