Skip to content

feat: --print-tcc-path 顯示執行 context(parent chain)與 context-dependence 警告 (#169) - #174

Merged
kiki830621 merged 5 commits into
mainfrom
idd/169-feature-print-tcc-path-context-parent-ch
Jul 10, 2026
Merged

feat: --print-tcc-path 顯示執行 context(parent chain)與 context-dependence 警告 (#169)#174
kiki830621 merged 5 commits into
mainfrom
idd/169-feature-print-tcc-path-context-parent-ch

Conversation

@kiki830621

Copy link
Copy Markdown
Member

Refs #169

Summary

--print-tcc-path 輸出尾端新增「Execution context (parent process chain)」段(self → … → launchd,單次 ps -A 快照 + in-memory walk,cycle guard / hop cap 15 / graceful degrade)+ context-dependence 警告行 — 使用者從此看得到「這次查詢是在哪個 host context 下跑的」,消除 #168 診斷的 context 誤判。

實作:新 ParentChainSource seam(<Domain>Source 慣例)+ 純邏輯 ParentChainWalker / ParentChainFormatter#117 抽出先例);--help 同步。

Verification

6-AI cross-model verification PASS(4 lens Agents + sequenced Devil's Advocate + Codex gpt-5.5 xhigh)。詳見 issue #169Verify master report

唯一實質 finding(security+Codex 跨模型同源:ps comm 未消毒直印 stdout,CWE-150)已 in-scope 修復於 37e9e32(三插補點全過 escapeForStderr + 2 消毒測試)。470 tests,僅 2 pre-existing 失敗(版本漂移 → #172,非本 diff)。

Checklist

  • Diagnose ✓(Simple / A_parallel_safe)
  • Implement(4 commits:ef991dd seam+walk、74f48f5 輸出段+警告、9154490 changelog、37e9e32 sanitizer fix)
  • Verify ✓(6-AI PASS,0 blocking)
  • Verify-gated: verify PASS — ready to merge → after merge, run /idd-close to finalize this issue (manual gate + closing summary; no auto-close trailer)

Related


🤖 Generated by /idd-all. Do NOT add a GitHub close trailer (Closes/Fixes/Resolves) — IDD discipline requires manual close via the skill after merge to enforce checklist gate + closing summary.

Verify finding (security lens + Codex cross-model, DA-confirmed real):
ps comm is ancestor-controlled and reached stdout unescaped — CWE-150
terminal escape injection, bypassing the repo's own escapeForStderr
discipline (#37/#73/#150). All three interpolated fields (hop.command,
selfPath, failureReason) now route through the sanitizer.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant