\\n

🚥 Pre-merge checks | ✅ 2 | ❌ 1

\\n\\n### ❌ Failed checks (1 warning)\\n\\n| Check name | Status | Explanation | Resolution |\\n| :----------------: | :--------- | :----------------------------------------------------------------------------------- | :--------------------------------------------------------------------------------- |\\n| Docstring Coverage | ⚠️ Warning | Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. | Write docstrings for the functions missing them to satisfy the coverage threshold. |\\n\\n

\\n

✅ Passed checks (2 passed)

\\n\\n| Check name | Status | Explanation |\\n| :---------------: | :------- | :---------------------------------------------------------- |\\n| Description Check | ✅ Passed | Check skipped - CodeRabbit’s high-level summary is enabled. |\\n| Title check | ✅ Passed | 标题清晰准确地反映了本次变更的主要目的：恢复Windows下--dev选项的源地址选择功能。 |\\n\\n

\\n\\n_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}\\n\\n

\\n\\n\\n\\n\\n\\n

\\n

✨ Finishing Touches

\\n\\n

\\n

🧪 Generate unit tests (beta)

\\n\\n- [ ] Create PR with unit tests\\n\\n

\\n\\n

\\n\\n\\n\\n\\n\\n---\\n\\nThanks for using [CodeRabbit](https://coderabbit.ai?utm_source=oss&utm_medium=github&utm_campaign=nxtrace/NTrace-core&utm_content=364)! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.\\n\\n

\\n

❤️ Share

\\n\\n- [X](https://twitter.com/intent/tweet?text=I%20just%20used%20%40coderabbitai%20for%20my%20code%20review%2C%20and%20it%27s%20fantastic%21%20It%27s%20free%20for%20OSS%20and%20offers%20a%20free%20trial%20for%20the%20proprietary%20code.%20Check%20it%20out%3A&url=https%3A//coderabbit.ai)\\n- [Mastodon](https://mastodon.social/share?text=I%20just%20used%20%40coderabbitai%20for%20my%20code%20review%2C%20and%20it%27s%20fantastic%21%20It%27s%20free%20for%20OSS%20and%20offers%20a%20free%20trial%20for%20the%20proprietary%20code.%20Check%20it%20out%3A%20https%3A%2F%2Fcoderabbit.ai)\\n- [Reddit](https://www.reddit.com/submit?title=Great%20tool%20for%20code%20review%20-%20CodeRabbit&text=I%20just%20used%20CodeRabbit%20for%20my%20code%20review%2C%20and%20it%27s%20fantastic%21%20It%27s%20free%20for%20OSS%20and%20offers%20a%20free%20trial%20for%20proprietary%20code.%20Check%20it%20out%3A%20https%3A//coderabbit.ai)\\n- [LinkedIn](https://www.linkedin.com/sharing/share-offsite/?url=https%3A%2F%2Fcoderabbit.ai&mini=true&title=Great%20tool%20for%20code%20review%20-%20CodeRabbit&summary=I%20just%20used%20CodeRabbit%20for%20my%20code%20review%2C%20and%20it%27s%20fantastic%21%20It%27s%20free%20for%20OSS%20and%20offers%20a%20free%20trial%20for%20proprietary%20code)\\n\\n

\\n\\n_{Comment `@coderabbitai help` to get the list of available commands and usage tips.}\\n\\n\\n\\n\\n\\n\\n\\n\\n\",\"path\":null,\"side\":null,\"line\":null}{\"type\":\"pull_request_comment\",\"body\":\"`NormalizeExplicitSourceConfig` no longer uses the `method` parameter, and the added `_ = method` is a code smell that can confuse readers. Prefer renaming the parameter to `_` in the function signature (keeps API compatibility without an in-body no-op), or add a brief comment explaining why the parameter is intentionally unused (e.g., kept for backward compatibility).\",\"path\":\"trace/source_config.go\",\"side\":\"RIGHT\",\"line\":null}{\"type\":\"pull_request_comment\",\"body\":\"## Pull request overview\\n\\nRestores Windows `--dev` behavior so TCP traces no longer fail early and instead derive a source IP from the selected device, aligning Windows behavior across ICMP/TCP/UDP and updating documentation to reflect that Windows `--dev` is source-address selection (not strict egress-interface binding).\\n\\n**Changes:**\\n- Remove the Windows+TCP `SourceDevice` rejection in `NormalizeExplicitSourceConfig`, allowing device→source-IP resolution.\\n- Update/extend tests to validate Windows TCP resolves `--dev` to `SrcAddr` and clears `SourceDevice`.\\n- Update CLI help text and READMEs to clarify Windows `--dev` and standalone `--mtu` semantics.\\n\\n### Reviewed changes\\n\\nCopilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.\\n\\n

\\n

Show a summary per file

\\n\\n| File | Description |\\\\n| ---- | ----------- |\\\\n| `trace/source_config.go` | Removes Windows TCP `--dev` rejection; normalizes Windows `--dev` into `SrcAddr` while clearing `SourceDevice`. |\\\\n| `trace/source_config_test.go` | Updates test to assert Windows TCP resolves device to a source IP instead of erroring. |\\\\n| `cmd/cmd.go` | Adjusts `--dev` help text and related comments to match restored Windows behavior. |\\\\n| `README.md` | Documents Windows `--dev` as source-IP selection for ICMP/TCP/UDP and clarifies `--mtu` behavior. |\\\\n| `README_zh_CN.md` | Same documentation updates as `README.md` (Chinese). |\\\\n| `AGENTS.md` | Records the Windows compatibility rule: resolve `--dev` to `SrcAddr` and clear `SourceDevice` (v1.6.2-compatible behavior). |\\n

\\n\\n\\n\\n\\n\\n\\n---\\n\\n💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.\",\"path\":null,\"side\":null,\"line\":null}{\"type\":\"pull_request_comment\",\"body\":\"\\n\\n

\\n

🧹 Nitpick comments (1)

\\n\\n
\\n
trace/source_config_test.go (1)
\\n\\n`303-330`: **建议把 Windows 协议覆盖做成表驱动。**\\n\\n当前新增用例覆盖了这次 TCP 回归；可以顺手把 `ICMPTrace` / `TCPTrace` / `UDPTrace` 都放进同一个表，锁住 PR 描述里的三种协议行为，避免后续重新引入 method-specific 分支时漏掉 UDP。\\n\\n
\\n
🧪 可选测试重构
\\n\\n```diff\\n-func TestNormalizeExplicitSourceConfigWindowsTCPResolvesDeviceToSourceAddress(t *testing.T) {\\n+func TestNormalizeExplicitSourceConfigWindowsResolvesDeviceToSourceAddress(t *testing.T) {\\n \\trestore := stubSourceDeviceResolver(t, func(device string) (*net.Interface, error) {\\n \\t\\tif device != \\\"Ethernet0\\\" {\\n \\t\\t\\tt.Fatalf(\\\"ResolveSourceDevice device = %q, want Ethernet0\\\", device)\\n \\t\\t}\\n \\t\\treturn &net.Interface{Name: device}, nil\\n \\t}, func(_ *net.Interface) ([]net.Addr, error) {\\n \\t\\treturn []net.Addr{&net.IPNet{IP: net.ParseIP(\\\"192.0.2.44\\\"), Mask: net.CIDRMask(24, 32)}}, nil\\n \\t})\\n \\tdefer restore()\\n \\n-\\tcfg := Config{\\n-\\t\\tOSType: osTypeWindows,\\n-\\t\\tDstIP: net.ParseIP(\\\"1.1.1.1\\\"),\\n-\\t\\tSourceDevice: \\\"Ethernet0\\\",\\n-\\t}\\n-\\n-\\tgot, err := NormalizeExplicitSourceConfig(TCPTrace, cfg)\\n-\\tif err != nil {\\n-\\t\\tt.Fatalf(\\\"NormalizeExplicitSourceConfig() error = %v\\\", err)\\n-\\t}\\n-\\tif got.SrcAddr != \\\"192.0.2.44\\\" {\\n-\\t\\tt.Fatalf(\\\"SrcAddr = %q, want 192.0.2.44\\\", got.SrcAddr)\\n-\\t}\\n-\\tif got.SourceDevice != \\\"\\\" {\\n-\\t\\tt.Fatalf(\\\"SourceDevice = %q, want empty\\\", got.SourceDevice)\\n+\\tfor _, method := range []Method{ICMPTrace, TCPTrace, UDPTrace} {\\n+\\t\\tt.Run(string(method), func(t *testing.T) {\\n+\\t\\t\\tcfg := Config{\\n+\\t\\t\\t\\tOSType: osTypeWindows,\\n+\\t\\t\\t\\tDstIP: net.ParseIP(\\\"1.1.1.1\\\"),\\n+\\t\\t\\t\\tSourceDevice: \\\"Ethernet0\\\",\\n+\\t\\t\\t}\\n+\\n+\\t\\t\\tgot, err := NormalizeExplicitSourceConfig(method, cfg)\\n+\\t\\t\\tif err != nil {\\n+\\t\\t\\t\\tt.Fatalf(\\\"NormalizeExplicitSourceConfig() error = %v\\\", err)\\n+\\t\\t\\t}\\n+\\t\\t\\tif got.SrcAddr != \\\"192.0.2.44\\\" {\\n+\\t\\t\\t\\tt.Fatalf(\\\"SrcAddr = %q, want 192.0.2.44\\\", got.SrcAddr)\\n+\\t\\t\\t}\\n+\\t\\t\\tif got.SourceDevice != \\\"\\\" {\\n+\\t\\t\\t\\tt.Fatalf(\\\"SourceDevice = %q, want empty\\\", got.SourceDevice)\\n+\\t\\t\\t}\\n+\\t\\t})\\n \\t}\\n }\\n```\\n
\\n\\nAs per coding guidelines, `**/cmd/cmd.go`: For Windows ICMP/TCP/UDP, use source IP to influence routing selection, not actual per-device binding; document in README that device specification may be inaccurate.\\n\\n
\\n
🤖 Prompt for AI Agents
\\n\\n```\\nVerify each finding against the current code and only fix it if needed.\\n\\nIn `@trace/source_config_test.go` around lines 303 - 330, The test\\nTestNormalizeExplicitSourceConfigWindowsTCPResolvesDeviceToSourceAddress\\ncurrently only covers TCP; change it to a table-driven test iterating over trace\\ntypes (use TCPTrace, ICMPTrace, UDPTrace) and assert the same expectations for\\nWindows: when SourceDevice resolves to an IP,\\nNormalizeExplicitSourceConfig(traceType, cfg) returns SrcAddr == resolved IP and\\nclears SourceDevice; update the test function name (e.g.\\nTestNormalizeExplicitSourceConfigWindowsProtocolResolvesDeviceToSourceAddress)\\nand loop over the trace types, calling NormalizeExplicitSourceConfig for each\\nand running the same assertions, and add a short README note where Windows\\nbehavior is documented (per-cmd behavior in cmd.go) that device spec affects\\nrouting via source IP rather than guaranteed device binding.\\n```\\n\\n
\\n\\n
\\n\\n

\\n\\n

\\n

🤖 Prompt for all review comments with AI agents

\\n\\n```\\nVerify each finding against the current code and only fix it if needed.\\n\\nNitpick comments:\\nIn `@trace/source_config_test.go`:\\n- Around line 303-330: The test\\nTestNormalizeExplicitSourceConfigWindowsTCPResolvesDeviceToSourceAddress\\ncurrently only covers TCP; change it to a table-driven test iterating over trace\\ntypes (use TCPTrace, ICMPTrace, UDPTrace) and assert the same expectations for\\nWindows: when SourceDevice resolves to an IP,\\nNormalizeExplicitSourceConfig(traceType, cfg) returns SrcAddr == resolved IP and\\nclears SourceDevice; update the test function name (e.g.\\nTestNormalizeExplicitSourceConfigWindowsProtocolResolvesDeviceToSourceAddress)\\nand loop over the trace types, calling NormalizeExplicitSourceConfig for each\\nand running the same assertions, and add a short README note where Windows\\nbehavior is documented (per-cmd behavior in cmd.go) that device spec affects\\nrouting via source IP rather than guaranteed device binding.\\n```\\n\\n

\\n\\n---\\n\\n

\\n

ℹ️ Review info

\\n\\n

\\n

⚙️ Run configuration

\\n\\n**Configuration used**: Organization UI\\n\\n**Review profile**: CHILL\\n\\n**Plan**: Pro\\n\\n**Run ID**: `ecef484f-d74c-43f5-982d-ca57aff80c09`\\n\\n

\\n\\n

\\n

📥 Commits

\\n\\nReviewing files that changed from the base of the PR and between e1115a2553e6b90d42ecaf5e70a8c3c50b60b1dc and e66e2663f6b4d2235fb3ab0e2db276bf91010746.\\n\\n

\\n\\n

\\n

📒 Files selected for processing (6)

\\n\\n* `AGENTS.md`\\n* `README.md`\\n* `README_zh_CN.md`\\n* `cmd/cmd.go`\\n* `trace/source_config.go`\\n* `trace/source_config_test.go`\\n\\n

\\n\\n

\\n\\n\",\"path\":null,\"side\":null,\"line\":null}{\"type\":\"pull_request_comment\",\"body\":\"@coderabbitai run pre-merge checks\",\"path\":null,\"side\":null,\"line\":null}{\"type\":\"pull_request_comment\",\"body\":\"\\n

\\n

✅ Actions performed

\\n\\nRunning Pre-merge checks. Results will be posted in the summary comment once complete.\\n\\n

\",\"path\":null,\"side\":null,\"line\":null}{\"type\":\"pull_request_comment\",\"body\":\"@codex run pre-merge checks\",\"path\":null,\"side\":null,\"line\":null}{\"type\":\"pull_request_comment\",\"body\":\"To use Codex here, [create an environment for this repo](https://chatgpt.com/codex/cloud/settings/environments).\",\"path\":null,\"side\":null,\"line\":null}","text_elements",[],{"_381":580,"_368":369,"_383":371},{"_384":581,"_386":-5,"_42":-5,"_387":-5,"_388":-5,"_96":307,"_92":390},[582],{"_46":576,"_92":583},"input_text","turn/started",{"_368":369,"_87":586},{"_103":-5,"_42":371,"_372":587,"_374":588},[],"inProgress",{"_374":590,"_368":369},{"_591":592,"_92":593},"activeFlags",[],"active",{"_381":595,"_368":369,"_383":596},{"_384":597,"_386":-5,"_42":-5,"_387":-5,"_388":-5,"_96":307,"_92":390},"auto-compact-0",[598,599],{"_46":601,"_92":583},{"_46":600,"_92":583},"\n /workspace/NTrace-core\n bash\n 2026-04-20\n Etc/UTC\n","# AGENTS.md instructions for /workspace/NTrace-core\n\n\n# NTrace-core 项目记忆文件（2026-03 快照，rev-3）\n\n# 供 LLM 在后续会话中快速加载上下文，减少重复分析。\n\n## 项目概览\n\n- 名称：NextTrace (NTrace-core)\n- 仓库：github.com/nxtrace/NTrace-core\n- 模块：`github.com/nxtrace/NTrace-core`\n- 语言：Go（`go 1.26.0`）\n- 入口：`main.go -> cmd.Execute()`\n- 核心能力：ICMP/TCP/UDP traceroute、GeoIP/RDNS、MTR 连续探测、Web/API、多平台构建\n\n## 构建与测试（必须遵守）\n\n- 常用命令：\n - 构建：`go build ./...`\n - 测试：`go test ./...`\n- 交叉编译脚本：`.cross_compile.sh`\n- Darwin 下 `trace/internal/icmp_darwin.go` 已不再使用 `//go:linkname`，改为\n `syscall.Socket` + `os.NewFile` + 自定义 `icmpPacketConn`（实现 `net.PacketConn` /\n `net.Conn` / `syscall.Conn` + `ReadMsgIP` 以满足 `x/net/internal/socket.ipConn`\n 接口），并在 `ReadFrom` 中调用 `stripIPv4Header` 剥离 macOS DGRAM ICMP socket\n 返回的外层 IP 头。\n\n## 当前 CLI 语义（重点）\n\n### 常规 traceroute 路径\n\n- `--table`：现在是\"最终汇总表\"模式（一次探测完成后输出汇总表），不再是旧的异步 table 刷新模式。\n- `--route-path`：仍由 `reporter.New(...).Print()` 负责（与 MTR report 无关）。\n\n### 独立 `--mtu` 路径\n\n- `--mtu`：独立 UDP path-MTU / tracepath 风格模式，不复用普通 `trace.Traceroute` / MTR / Web 路径。\n- flavor 可用性：仅 `nexttrace` / `nexttrace-tiny` 包含；`ntr` 不注册该 flag。\n- 输出：\n - `TTY`：当前 TTL 占位后原地更新，边探测边刷行。\n - `非 TTY`：TTL 定稿后逐行流式输出，不使用 renderer 自己的 ANSI 控制序列。\n - `--json`：输出独立 mtu schema；`hop.geo` 已存在。\n- 参数语义：\n - 复用 `--data-provider`、`--language`、`--no-rdns`、`--always-rdns`、`--dot-server`。\n - `--mtu` 仍只支持 UDP；显式 `--tcp` 冲突报错。\n- Geo/RDNS：\n - `trace/mtu` 自带独立 metadata helper，不依赖普通 `trace.Hop.fetchIPData`。\n - 流式事件会先输出基础 hop，再在同一 TTL 内补一条带 Geo/RDNS 的 update，最后 `ttl_final` 定稿。\n - macOS 上曾有 `Warning: macOS --mtu support is experimental.` 提示，现已删除；不要再假设 CLI 会打印这句。\n\n### `--psize` / `--tos` 语义与平台差异\n\n- `--psize` 现在统一对齐 `mtr -s/--psize`：\n - 用户输入语义是“含 IP + 当前探测协议头的总字节数”。\n - 内部 `trace.Config.PktSize` 仍保存 payload bytes。\n - 未显式传入时，不再固定默认 `52`，而是按协议/IP 族自动取最小合法值：\n - ICMPv4 / UDPv4 = `28`\n - TCPv4 = `44`\n - ICMPv6 = `48`\n - UDPv6 = `50`\n - TCPv6 = `64`\n - 负数 `--psize` 表示“每个 probe 独立随机”，CLI 允许 `--psize -84` 这种写法并会在解析前归一化。\n- `--tos` / `-Q`：\n - 范围固定 `0..255`。\n - `--mtu` 与 Globalping 显式传 `--psize` / `--tos` 会直接报不支持。\n- 平台发送路径差异（这是后续判断 bug 的关键记忆）：\n - Linux / 其他 Unix：\n - `ICMP/TCP/UDP` 的 IPv4/IPv6 都走原生 socket/raw socket 路径。\n - `--tos` 只是在现有路径上设置 `TOS/TrafficClass`，不会切换实现。\n - macOS：\n - 与 Linux 类似，`ICMP/TCP/UDP` 的 IPv4/IPv6 都走原生发送路径。\n - `--tos` 同样只是在现有路径上设置 `TOS/TrafficClass`。\n - Windows：\n - `TCP/UDP` 的 IPv4/IPv6 一直走 WinDivert raw send。\n - `ICMPv4` 一直走 socket path（`SetTOS` / `SetTTL`）。\n - `ICMPv6`：\n - 默认或 `--tos 0`：继续走原生 socket path，只设置 `HopLimit`，保持与 `v1.5.2` 一致。\n - 非零 `--tos`：切到 WinDivert raw send，直接发送完整 `IPv6 + ICMPv6` 报文，因为 Windows 的 `x/net/ipv6.PacketConn` 不能可靠设置 `TrafficClass`。\n - 因此，Windows 上只有“`ICMPv6` 且 `--tos != 0`”这个组合会额外依赖 WinDivert 发送能力；README 中英两份都已写明。\n\n### 间隔默认值（分层体系）\n\n- `-z/--send-time`：每包间隔，默认 `defaultPacketIntervalMs = 50` ms。\n- `-i/--ttl-time`：\n - **常规 traceroute**：TTL 分组间隔，默认 `defaultTracerouteTTLIntervalMs = 300` ms。\n - **MTR 模式**：`normalizeMTRTraceConfig()` 始终覆盖为 `defaultMTRInternalTTLIntervalMs = 0` ms（各 TTL 间不间隔）。\n - MTR 每跳探测间隔由 `-i` 显式传值或默认 1000ms 决定（见下文 `-q/-i` 语义）。`-z/--send-time` 在 MTR 模式下被忽略。\n\n### MTR 相关参数\n\n- `-t/--mtr`：开启 MTR 交互模式（TTY 全屏 TUI）。\n- `-r/--report`：MTR 报告模式（非交互），隐式开启 MTR。\n- `-w/--wide`：宽报告模式，隐式等价 `--mtr --report --wide`。\n- `--raw`：与 MTR 组合时进入 **MTR raw 流式模式**（`runMTRRaw`），不再与 MTR 冲突。\n- 有效 MTR 开关：`effectiveMTR = mtr || report || wide`。\n- MTR 三路分支（`chooseMTRRunMode`）：\n 1. `effectiveMTRRaw` → `runMTRRaw`（流式行输出，适合管道/脚本）\n 2. `effectiveReport` → `runMTRReport`（非交互报告表）\n 3. 默认 → `runMTRTUI`（全屏 TUI）\n- MTR 冲突参数（会直接报错退出）：`--table` `--classic` `--json` `--output` `--route-path` `--from` `--fast-trace` `--file` `--deploy`。\n - **注意**：`--raw` 不再是冲突参数。\n\n### MTR 中 `-q/-i/-y` 的新语义\n\n- `-q/--queries`：\n - 在 MTR report 下表示每跳探测次数，默认 10（仅当用户未显式传 `-q`）。\n - 在 MTR TUI 下表示每跳最大探测次数，未显式传时默认无限运行。\n- `-i/--ttl-time`：\n - 在 MTR 下表示每跳探测间隔毫秒，默认 1000ms（仅当用户未显式传 `-i`）。\n - 各 TTL 间内部扫描间隔固定 0ms（`normalizeMTRTraceConfig` 覆盖为 `defaultMTRInternalTTLIntervalMs = 0`）。\n - `-z/--send-time` 在 MTR 模式下被忽略。\n- `-y/--ipinfo <0..4>`：\n - TUI 初始 Host 显示模式，默认 0（IP/PTR only）。\n - 0=Base(IP/PTR) 1=ASN 2=City 3=Owner 4=Full\n - 仅 TUI 模式生效，report/raw 不受影响。\n\n### MTR report wide / non-wide 区别\n\n- **wide 模式**（`-w` 或 `--mtr --report --wide`）：\n - 查询 GeoIP，显示完整 host 信息（ASN + geo + MPLS）。\n- **非 wide 模式**（`-r` 或 `--mtr --report`）：\n - `normalizeMTRReportConfig` 设 `IPGeoSource=nil`（不查 geo）、`AlwaysWaitRDNS=true`。\n - 显示 `formatCompactReportHost`：仅 IP/PTR + ASN，无 geo 列。\n\n## MTR 运行链路（重要文件）\n\n- 入口与调度：`cmd/mtr_mode.go`（~315 行）\n - `runMTRTUI(...)` / `runMTRReport(...)` / `runMTRRaw(...)`\n - `normalizeMTRTraceConfig(conf)` / `normalizeMTRReportConfig(conf, wide)`\n - `buildAPIInfo(...)` / `buildRawAPIInfoLine(...)`\n - MTR CLI 现在统一使用 `signal.NotifyContext(...)` 管理 Ctrl-C / SIGTERM；不再保留额外的 `sigCh` + goroutine 等待器。\n- 交互控制：`cmd/mtr_ui.go`\n - alternate screen + raw mode\n - 输入状态机 `mtrInputParser`（字节流，吞掉 CSI/SS3/OSC/鼠标/焦点等序列）\n - Enter/Leave 显式关闭输入扩展模式：1000/1002/1003/1006/1015/1004/2004\n - Quit 路径会先判空 `cancel`，因此 `newMTRUI(nil, ...)` / 测试注入 nil 不会 panic。\n- 核心探测循环：`trace/mtr_runner.go`\n - `RunMTR` / `mtrLoop` / `RunMTRRaw`\n - 支持暂停、重置、流式预览（`ProgressThrottle` 默认 200ms）\n - ICMP 持久引擎 + TCP/UDP fallback\n- 统计聚合：`trace/mtr_stats.go`\n - `MTRAggregator` / `MTRHopStat`\n - unknown 合并策略：单路径时把 unknown 合并到唯一已知路径，避免同 TTL 分裂成 waiting + 真实 IP 两行\n- 输出层：\n - TUI：`printer/mtr_tui.go`\n - table/report：`printer/mtr_table.go`\n - raw 行格式化：`printer.FormatMTRRawLine(rec)`\n - TUI 颜色：`printer/mtr_tui_color.go`\n\n## MTR 交互行为（当前）\n\n- `q`/`Q`/`Ctrl-C`：退出\n- `p`：暂停\n- `SPACE`：恢复\n- `r`：重置统计\n- `y`：切换 Host 显示模式（IP/PTR → ASN → City → Owner → Full → 循环）\n- `n`：切换 Host 基名显示（PTR-or-IP / IP-only）\n- `e`：切换 MPLS 标签显示（toggle MPLS on/off）\n\n## MTR 显示与统计规则（当前）\n\n- Host 显示支持 5 种模式（`-y/--ipinfo` 设初始值，`y` 键运行时循环）：\n - `HostModeBase=0`：仅 IP/PTR，无 ASN 前缀\n - `HostModeASN=1` / `HostModeCity=2` / `HostModeOwner=3` / `HostModeFull=4`\n - `HostNamePTRorIP` / `HostNameIPOnly`\n- 默认语言：`cn`（`--language en` 才优先英文字段）\n- waiting 判定：`loss >= 99.95 && IP==\"\" && Host==\"\"`\n - 显示为 `(waiting for reply)`\n - 指标列（Loss/Snt/Last/Avg/Best/Wrst/StDev）留空\n- TUI Host 对齐（重要，已从 tab 改为手动空格）：\n - `buildTUIHostParts(stat, mode, nameMode, lang, showIPs)` 生成结构化 parts\n - `computeTUIASNWidth(stats, ...)` 扫描所有 hop 确定 ASN 列最大宽度\n - `formatTUIHost(parts, asnW)` 用 `padRight(asn, asnW)` + 空格拼接（不用 `\\t`）\n - ASN 为空但 IP 已知时填 `\"AS???\"` 占位符，保证列对齐（HostModeBase 除外，该模式不显示 ASN）\n - waiting hop 不填占位符\n- compact report host（非 wide report）：\n - `formatCompactReportHost(stat, nameMode, lang)` 仅输出 hostname/IP + ASN\n- TUI 其他特性：\n - 终端宽度自适应 + CJK 宽度计算（go-runewidth）\n - 窄屏右锚定指标区\n - 动态 hop 前缀宽度（覆盖 3 位/4 位 TTL）\n - MPLS 独立续行显示\n - 紧凑指标列宽度：Loss=5 Snt=3 RTT=7 RTTMin=5\n\n## MTR 目的地检测与高 TTL 丢弃\n\n- 当 `knownFinalTTL` 已确定后，所有 `TTL > knownFinalTTL` 的调度槽位被标记为 `disabled`。\n- disabled TTL 的探测回包（包括在途探测返回的 dst-ip 回复）**一律丢弃**，不折叠、不计入任何统计。\n- **MaxPerHop 上限检查**（`states[originTTL].completed + inFlightCount >= MaxPerHop`）：\n - 调度时使用 `completed + inFlightCount >= MaxPerHop` 防止超发。\n - 完成时仍检查 `completed >= MaxPerHop` 丢弃溢出结果。\n- `originTTL < curFinal` 时（更低 TTL 先到 dst-ip → 降低 `knownFinalTTL`）：\n - 保存 `oldFinal`，更新 `knownFinalTTL = originTTL`，disable 所有 `originTTL+1..maxHops`。\n - 调用 `agg.ClearHop(oldFinal)`：清除旧 finalTTL 的聚合数据（避免幽灵行），**不合并**到新 finalTTL。\n - 新 finalTTL 由独立的 per-hop 调度器自行积累新鲜探测数据，不存在 Snt 膨胀问题。\n- 调度状态（`inFlightCount`/`nextAt`/`consecutiveErrs`）更新在 `originTTL`。\n- 统计聚合（`completed++`/`agg.Update`/`onProbe`）均使用 `originTTL`（不再有 `accountTTL` 分离）。\n\n## MTR Per-Hop 调度器关键设计（当前）\n\n- **多 in-flight 探测**：每 TTL 允许最多 `MaxInFlightPerHop`（默认 3）个并发探测。\n - `mtrHopState.inFlightCount` 是计数器（非 bool）。\n - 这解决了高丢包 hop 因超时阻塞导致 Snt 积累速率远低于低丢包 hop 的问题。\n- **nextAt 基于发送时间**：`launchProbe` 时设 `nextAt = now + hopInterval`。\n - 不再等探测完成才设 nextAt，调度器可在超时探测还在飞行中时为同一 TTL 发射新探测。\n - 这保证了所有 TTL 的 Snt 积累速率大致相同，不受丢包率影响。\n- **全局并发限制**：`inFlight`（全局计数器）< `parallelism` 仍然有效。\n- **`MaxInFlightPerHop` 配置**：`mtrSchedulerConfig.MaxInFlightPerHop`，默认动态计算。\n - 动态默认 = `ceil(Timeout / HopInterval) + 1`（至少 1）。\n - 例：`Timeout=2s, HopInterval=1s` → 默认 3；`Timeout=2s, HopInterval=200ms` → 默认 11。\n - 用户显式设置 > 0 时优先使用用户值。\n\n## MTR 引擎关键机制（易踩坑）\n\n- 目的地提前停止：\n - `knownFinalTTL`（持久缓存）用于缩短后续探测的 TTL 上界；高 TTL 标记 disabled 后不再调度。\n- seq 16 位回卷处理：\n - `seqWillWrap(...)` 触发 `rotateEngine(...)`\n - 轮换 echoID 并重建 listener，协议层隔离新旧回包。\n- 额外安全网：\n - onICMP 中有 RTT 合理性检查（`<=0` 或 `>timeout` 丢弃）。\n- 流式预览：\n - 仅已发送 TTL 才会参与预览；未发送 TTL 保持 nil 槽位，避免提前计入 Snt/Loss。\n\n## Web Console / WebSocket（server/）\n\n### WS 架构（`server/ws_handler.go`，~451 行）\n\n- **异步写模型**：`wsTraceSession` 使用 `sendCh`（buffered channel，1024）+ `writeLoop` goroutine。\n - 调用方通过 `send(envelope)` 非阻塞投递；channel 满时返回 `errWSSlowConsumer`。\n - `writeLoop` 从 `sendCh` 取消息，`SetWriteDeadline` + `WriteJSON`。\n- **关闭路径**：\n - `closeWithCode(code, reason)`：异常关闭（slow consumer / write error），关 `stopCh` + 发 close frame。\n - `finish()`：正常结束，`sendMu` 下关 `sendCh`，等 `writerDone`，再关 conn。\n - 两者均幂等（`closeOnce` / `finishOnce`）。\n- **可测试性**：`wsConn` 接口 + `fakeWSConn` mock（`server/ws_handler_test.go`）。\n- **常量**：`wsSendQueueSize=1024`，`wsWriteTimeout=5s`。\n\n### Web MTR 调度模式（重要变更）\n\n- **已从 round-based 迁移到 per-hop 调度**。\n- `runMTRTrace()`：\n - 优先读 `HopIntervalMs`，fallback `IntervalMs`，再缺省 1000ms。\n - `MaxRounds` → `MaxPerHop`（0 = 无限运行直到客户端断开）。\n - 不再使用 legacy round-based 的 `Interval` / `RunRound`。\n- `executeMTRRaw()` 两路分支：\n - `HopInterval > 0`：per-hop 模式，仅在 LeoMoe/FastIP 初始化阶段短暂加锁；长期探测不再依赖 `SrcDev` / `DisableMPLS` 等进程级全局。\n - fallback：legacy round-based 模式（保留兼容），`RunRound` 回调内 per-round 锁定。\n - `trace/runMTRRawRoundBased()` 也会先做 `normalizeRuntimeConfig(&cfg)`，因此 legacy raw 路径同样能继承 `SourceDevice`；`DisableMPLS` 不再从全局反向覆盖会话配置。\n- `traceRequest` 新增 `HopIntervalMs` 字段（`json:\"hop_interval_ms\"`），与 `IntervalMs` 解耦。\n- 前端 MTR 请求现在发送 `hop_interval_ms=1000`，不再把旧的 `interval_ms=2000` 当默认值。\n- 前端 raw 聚合键现在按 TTL 折叠，避免同一 hop 的 timeout / success 被拆成两行。\n\n### 前端渲染节流（`server/web/assets/app.js`）\n\n- MTR raw 消息通过 `scheduleMTRRender()` 节流，最小间隔 100ms，优先 `requestAnimationFrame`。\n- `cancelScheduledMTRRender()` 在 `clearResult`、socket close/error 路径调用，避免孤儿回调。\n- `flushMTRRender()` 立即执行挂起渲染。\n\n### 其他 server 文件\n\n- `server/server.go`：Gin 路由注册\n- `server/handlers.go`：REST 接口\n- `server/mtr.go`：MTR 专用 handler 逻辑\n- `server/trace_handler.go`：traceroute handler\n- `server/cache_handler.go`：缓存\n\n## DoT 与 Geo DNS\n\n- `--dot-server` 不仅影响目标域名解析，也影响 GeoIP API / LeoMoe FastIP 的域名解析链路。\n- 关键文件：`util/dns_resolver.go`\n - `SetGeoDNSResolver(dotServer)`\n - `WithGeoDNSResolver(dotServer, fn)`：为 Web/API 请求提供作用域化的 resolver 切换；不同 resolver 串行切换，相同 resolver 允许安全嵌套，避免 `GetSourceWithGeoDNS` + 外层作用域组合时死锁。\n - `geoResolverOverride` 的读写现在也走 `geoMu`，避免测试覆盖 resolver 时的数据竞争。\n - `LookupHostForGeo(ctx, host)`：IP 字面量短路 -> DoT -> 失败时按配置 fallback 系统 DNS\n- `cmd/cmd.go` 在早期阶段（fast-trace / ws 初始化之前）注入 DoT 解析策略，避免早期分支绕过。\n- `server/trace_handler.go` 通过 `ipgeo.GetSourceWithGeoDNS(...)` + `WithGeoDNSResolver(...)` 让 Web/API 请求也遵守 `dot_server`，包括 LeoMoe/FastIP 初始化阶段。\n- Geo HTTP 请求统一走 `util.NewGeoHTTPClient(...)`（`util/http_client_geo.go`），其 Transport 现在从默认 Transport `Clone()` 而来，保留代理/HTTP2/连接池等标准行为。\n\n## LeoMoe FastIP 与 MTR 首行\n\n- `util/latency.go`：\n - `FastIPMetaCache` 缓存节点元数据（IP/Latency/NodeName）\n - `SuppressFastIPOutput` 可抑制彩色横幅\n- `GetFastIP(...)` 的 DNS 阶段现在显式受 `timeout` 限制；`FastIPMetaCache` 也改为在 fallback/default IP 决定后再写入，避免缓存空 IP。\n- MTR 模式在进入 TUI 前会设 `SuppressFastIPOutput=true`，避免污染主终端历史。\n- MTR TUI/report 首行 `APIInfo` 由 `cmd/mtr_mode.go` 的 `buildAPIInfo(...)` 生成（仅 LeoMoeAPI 且有元数据时显示）。\n- MTR raw 首行由 `buildRawAPIInfoLine(...)` 生成（格式略不同，包含延迟信息）。\n\n## `--source` / `--dev` 现状\n\n- `--dev` 在 `cmd/cmd.go` 先解析网卡并推导 `srcAddr`（已处理非 `*net.IPNet` 地址类型，避免 panic）。\n- `trace.Config` 现在显式携带 `SourceDevice` / `DisableMPLS`，Darwin TCP/UDP 抓包与 MPLS 解析优先走会话级配置，不再依赖 Web 侧临时改写全局变量。\n- `trace.Config` 也显式携带 `Context`；`TracerouteWithContext(...)` 通过把上游 ctx 传入各 tracer 的 `signal.NotifyContext(...)` 基底，让 TCP/UDP fallback MTR 可以响应取消。\n- Windows TCP 目前仍无法把 `SourceDevice` 映射到 WinDivert 接口选择；当前策略是显式报错拒绝，而不是静默忽略该字段。\n- MTR 标题显示源信息来自：\n - `--source`（最高优先）\n - `--dev` 推导\n - UDP dial fallback\n- 相关函数：`cmd/mtr_mode.go -> resolveSrcIP(...)`\n\n## CI 与工具链（当前）\n\n- `go.mod`: `go 1.26.0`\n- GitHub Actions：\n - `.github/workflows/build.yml` 使用 `setup-go@v6` + `go-version: 1.26.x`\n - `.github/workflows/test.yml` 使用 `setup-go@v6` + `go-version: 1.26.x`\n - test workflow 中 `GOTOOLCHAIN=go1.26.0+auto`\n - build matrix 已移除 `windows/arm`\n- `.cross_compile.sh` 与 workflow 里的 `go build` 现在都用数组构造 `-tags` 参数，避免 shell word-splitting；脚本也会把当前 `GOARM` 传给 `compress_with_upx`，使 linux/armv7 目标能命中对应压缩分支。\n- `ipgeo/ipdbone.go` 不再原地修改全局 `defaultClient.httpClient.Timeout`；超时覆盖会通过克隆 client（复用 token cache / token init，同步替换整个 HTTP client）实现，避免 dial timeout 与 client timeout 脱节。\n\n## 关键文件导航\n\n- CLI 主调度：`cmd/cmd.go`（~855 行）\n- MTR 参数/流程：`cmd/mtr_mode.go`（~315 行）\n- MTR 交互输入：`cmd/mtr_ui.go`\n- MTR 引擎：`trace/mtr_runner.go`\n- MTR 聚合：`trace/mtr_stats.go`\n- MTR TUI：`printer/mtr_tui.go`（~691 行）\n- MTR table/report：`printer/mtr_table.go`（~625 行）\n- MTR TUI 颜色：`printer/mtr_tui_color.go`\n- WS handler：`server/ws_handler.go`（~449 行）\n- 前端：`server/web/assets/app.js`\n- Geo DoT 解析：`util/dns_resolver.go`\n- Geo HTTP 客户端：`util/http_client_geo.go`\n- FastIP：`util/latency.go`\n\n## 2026-03 Gocyclo 重构快照\n\n- 第一波低风险重构已落地：\n - `ipgeo.Filter` 改为 CIDR 规则表驱动。\n - `util.DomainLookUp` 拆成 resolver / lookup / family filter / interactive select 四段。\n - `server.prepareTrace`、`normalizeTarget`、`trace.Traceroute`、`trace.Hop.fetchIPData` 已改成薄协调器。\n - `GetMTUByIPForDevice`、`GetICMPResponsePayload`、`parseIPDBOneResponse`、`GlobalpingFormatLocation` 已拆 helper。\n- 第二波主流程/输出层已部分落地：\n - `cmd.Execute` 已拆成 parser 注册 helper、启动模式 helper、运行时调度 helper。\n - `fast_trace.FastTest` / `testFile` 已拆成交互选择、源地址推导、文件目标解析、单目标执行。\n - `server.mtrAggregator.Update`、`wshandle.messageSendHandler` 已改成薄入口。\n - `printer.RealtimePrinter`、`RealtimePrinterWithRouter`、`tracelog.RealtimePrinter` 现在共用 `internal/hoprender` 的 hop attempt 分组逻辑。\n- 第三波协议层已开始落地：\n - `trace/internal/icmp_common.go`、`tcp_common.go`、`udp_common.go` 已改成“读包循环 + 共享解码 helper + 回调派发”结构。\n - 新增 `trace/internal/icmp_decode.go`，集中处理 ICMPv4/v6 解析、echo reply 匹配、内嵌目标 IP 校验、内嵌 ICMP seq 提取。\n - 新增 `trace/internal/icmp_decode_test.go`，覆盖 IPv4/IPv6 echo reply、error payload、目标 IP 校验、内嵌 seq 提取。\n - `trace/internal/udp_unix.go` 的 `SendUDP` 已拆成 IPv4/IPv6 独立发送 helper；`trace/udp_ipv4.go` 的 `send()` 也已拆成配额检查、构包、超时守护、发送记账四段。\n - Windows 协议层新增 `trace/internal/windivert_sniff_windows.go`，把 WinDivert sniff handle 打开、收包、ICMP/TCP 解码下沉为共享 helper；`icmp_windows.go` / `tcp_windows.go` / `udp_windows.go` 的 sniff 入口已变成薄协调器。\n - Darwin `trace/internal/icmp_darwin.go:ListenPacket` 已拆成 socket spec、接口绑定、bind sockaddr、finalize packet conn 四段；`trace/internal/tcp_darwin.go:ListenTCP` 也改成设备选择 + BPF + 共享 TCP reply 解码 helper。\n - 新增 `trace/internal/tcp_probe_decode.go` 与 `trace/internal/tcp_probe_decode_test.go`，集中处理 TCP probe reply 的 seq 还原、peer IP 提取与 IPv4/IPv6 解析，供 Darwin/Windows TCP sniff 共用。\n- 当前已知剩余高复杂度主要集中在：\n - `printer/mtr_*` 渲染层\n - `cmd/mtr_ui.go` 输入状态机\n - `trace/globalping.go` 的主流程函数\n - `trace/mtr_runner.go` 中仍未拆薄的 ICMP round handler（`probeRound` / `onICMP`）\n - 少量收尾函数：`fast_trace ipv6.go`、`reporter/reporter.go`、`trace/mtr_raw.go`\n\n## 2026-03 Gocyclo 重构快照（追加）\n\n- MTR 核心热点已完成一轮收敛：\n - `trace/mtr_scheduler.go:runMTRScheduler` 已改成薄入口，核心状态与分支移动到 `trace/mtr_scheduler_runtime.go`。\n - `trace/mtr_stats.go:Update` / `MigrateStats` 已拆成按 hop 分组、累加器合并、裁剪 helper；新增 `trace/mtr_stats_helpers.go`。\n - `trace/mtr_runner.go:mtrLoop` 已改成薄入口，取消/重置/暂停/预览/backoff 分支移动到 `trace/mtr_loop_runtime.go`。\n- MTR 输出层与输入层热点也已收敛：\n - `printer/mtr_tui.go:mtrTUIRenderWithWidth` 已拆成布局扫描、三行头部构建、host part 预构建、MPLS 续行渲染四段。\n - `printer/mtr_table.go` 的 host 组装和 `MTRReportPrint` 已改成共享 host-part 拼接 helper + report header/row helper。\n - `cmd/mtr_ui.go:(*mtrInputParser).Feed` 已拆成按状态分发的 parser helper。\n- 最后一批业务流程热点也已拆薄：\n - `trace/globalping.go:GlobalpingTraceroute` 已拆成 client 构建、measurement 请求、结果解码、hop limit 推导、结果组装五段。\n - `trace/mtr_runner.go:(*mtrICMPEngine).onICMP` / `probeRound` 已拆成 reply 校验、notify 清理、目的地 TTL 识别、round 准备、发包 sweep、等待回包、结果构建多个 helper。\n - `fast_trace/fast_trace ipv6.go:FastTestv6`、`reporter/reporter.go:generateRouteReportNode`、`trace/mtr_raw.go:buildMTRRawRecordFromProbe` 也已分别拆成选择分发、route-node 属性构建、raw record metadata 填充 helper。\n- 当前本地复杂度扫描结果：\n - `go run /tmp/checkcyclo.go .` 已无 `>15` 函数输出。\n - `go test ./...` 通过。\n\n## 仍需记住的残余风险（非阻断）\n\n- `closeWithCode` 中 `closed.Store(true)` 在 `closeOnce.Do` 外部，理论上有微小竞态窗口（实际无害，因 `sendMu` 保护；且无法简单移入 Once 内部，否则第二个调用者无法设置 closed）。\n\n\n## Skills\nA skill is a set of local instructions to follow that is stored in a `SKILL.md` file. Below is the list of skills that can be used. Each entry includes a name, description, and file path so you can open the source for full instructions when using a specific skill.\n### Available skills\n- skill-creator: Guide for creating effective skills. This skill should be used when users want to create a new skill (or update an existing skill) that extends Codex's capabilities with specialized knowledge, workflows, or tool integrations. (file: /opt/codex/skills/.system/skill-creator/SKILL.md)\n- skill-installer: Install Codex skills into $CODEX_HOME/skills from a curated list or a GitHub repo path. Use when a user asks to list installable skills, install a curated skill, or install a skill from another repo (including private repos). (file: /opt/codex/skills/.system/skill-installer/SKILL.md)\n### How to use skills\n- Discovery: The list above is the skills available in this session (name + description + file path). Skill bodies live on disk at the listed paths.\n- Trigger rules: If the user names a skill (with `$SkillName` or plain text) OR the task clearly matches a skill's description shown above, you must use that skill for that turn. Multiple mentions mean use them all. Do not carry skills across turns unless re-mentioned.\n- Missing/blocked: If a named skill isn't in the list or the path can't be read, say so briefly and continue with the best fallback.\n- How to use a skill (progressive disclosure):\n 1) After deciding to use a skill, open its `SKILL.md`. Read only enough to follow the workflow.\n 2) When `SKILL.md` references relative paths (e.g., `scripts/foo.py`), resolve them relative to the skill directory listed above first, and only consider other paths if needed.\n 3) If `SKILL.md` points to extra folders such as `references/`, load only the specific files needed for the request; don't bulk-load everything.\n 4) If `scripts/` exist, prefer running or patching them instead of retyping large code blocks.\n 5) If `assets/` or templates exist, reuse them instead of recreating from scratch.\n- Coordination and sequencing:\n - If multiple skills apply, choose the minimal set that covers the request and state the order you'll use them.\n - Announce which skill(s) you're using and why (one short line). If you skip an obvious skill, say why.\n- Context hygiene:\n - Keep context small: summarize long sections instead of pasting them; only load extra files when needed.\n - Avoid deep reference-chasing: prefer opening only files directly linked from `SKILL.md` unless you're blocked.\n - When variants exist (frameworks, providers, domains), pick only the relevant reference file(s) and note that choice.\n- Safety and fallback: If a skill can't be applied cleanly (missing files, unclear instructions), state the issue, pick the next-best approach, and continue.\n",{"_381":603,"_368":369,"_383":596},{"_384":604,"_386":-5,"_42":-5,"_387":-5,"_388":-5,"_96":605,"_92":390},[606],"developer",{"_46":607,"_92":583},"\nFilesystem sandboxing defines which files can be read or written. `sandbox_mode` is `danger-full-access`: No filesystem sandboxing - all commands are permitted. Network access is enabled.\nApproval policy is currently never. Do not provide the `sandbox_permissions` for any reason, commands will be rejected.\n","thread/started",{"_610":611},"thread",{"_612":-5,"_613":-5,"_614":615,"_616":617,"_425":426,"_618":15,"_619":-5,"_620":-5,"_42":369,"_621":622,"_167":-5,"_623":-5,"_157":624,"_625":181,"_626":-5,"_432":627,"_374":628,"_629":-5,"_77":630,"_631":617},"agentNickname","agentRole","cliVersion","0.114.0","createdAt",1776649526,"ephemeral","forkedFromId","gitInfo","modelProvider","openai","parentThreadId","/opt/codex/sessions/2026/04/20/rollout-2026-04-20T01-45-26-019da890-5cc8-7172-88d5-e99ab4ed5255.jsonl","preview","recencyAt","vscode",{"_92":378},"threadSource",[],"updatedAt",[],"messages",[635],{"_42":636,"_637":638,"_639":640,"_641":640,"_384":642,"_374":643,"_386":-5,"_644":32,"_645":646,"_647":281,"_648":-5},"2b88abfd-3b06-4849-9424-444329744354","author",{"_96":307,"_167":-5,"_645":657},"create_time",1776649526.032718,"update_time",{"_653":46,"_654":655},"finished_successfully","weight","metadata",{"_649":650},"recipient","channel","l1239dk1",{"_651":652},"inline_comments",[],"content_type","parts",[656],"@codex run pre-merge checks\n[codex] Restore Windows --dev source selection\n\n## Summary\n- restore Windows `--dev` handling so ICMP/TCP/UDP resolve the selected device to a source IP instead of rejecting TCP\n- document that Windows uses `--dev` as source-address selection, not real egress-interface binding\n- clarify that standalone `--mtu` follows the same source-address behavior and only uses the device name for local MTU lookup\n\n## Why\nA previous change made Windows `TCP + --dev` fail before reaching the tracer. That regressed the v1.6.2-compatible behavior where Windows derives a source IP from the selected device and lets Windows routing choose the actual egress path.\n\n## Validation\n- `go test ./trace ./cmd`\n- `go test ./...`\n\n\n## Summary by CodeRabbit\n\n* **Bug Fixes**\n * 修复 Windows 下使用 --dev 时的行为：TCP/ICMP/UDP 探测可通过指定网卡推导源 IP 并继续执行（不再因 TCP+--dev 而报错）。\n\n* **Documentation**\n * 更新 CLI 和中/英文文档，说明 Windows 下 --dev/--mtu 将基于设备推导源地址，但路由仍可能选择不同出口。\n\n* **Tests**\n * 更新并扩展相关单元测试以覆盖 Windows 上设备解析为源地址的行为。\n\n\n\n\n## Walkthrough\n\n本次变更移除了对 Windows 上 TCP + `--dev` 的显式拒绝，文档与 CLI 帮助文本同步更新；实现上 `NormalizeExplicitSourceConfig` 在 Windows 下会把指定设备解析成源 IP 写入 `SrcAddr` 并清空 `SourceDevice`，相关单元测试调整为断言成功解析该地址。\n\n## Changes\n\n|Cohort / File(s)|Summary|\n|---|---|\n|**文档更新**
`AGENTS.md`, `README.md`, `README_zh_CN.md`|更新 Windows 上 `--dev` 的描述：说明 `--dev` 会从指定网卡派生源 IP 并用于 ICMP/TCP/UDP 探测，但不等同于强制绑定出口接口；补充 `--mtu` 单独模式遵循相同源地址语义。|\n|**CLI 帮助与注释**
`cmd/cmd.go`|调整 `--dev/-D` 的帮助文本与两处内联注释措辞，移除“TCP + --dev 不支持/报错”表述，改为“选择设备源地址；路由可能仍选出口接口”。|\n|**核心逻辑**
`trace/source_config.go`|删除基于 `method` 的 Windows 拒绝分支（不再因 TCP 而报错）；参数名去掉未使用的 `method` 依赖，保持其他解析/清理逻辑不变。|\n|**单元测试**
`trace/source_config_test.go`|替换拒绝型测试为表驱动测试（ICMP/TCP/UDP），调整存根以返回 `192.0.2.44/24` 并断言 `SrcAddr==\"192.0.2.44\"` 且 `SourceDevice==\"\"`。|\n\n## Estimated code review effort\n\n🎯 3 (Moderate) | ⏱️ ~20 分钟\n\n## 诗句\n\n> 🐰 Windows 兔子跳一跳， \n> 网卡化作小小源地址， \n> TCP、UDP、ICMP 一同笑， \n> 不再拒绝，只把源记牢， \n> 路由自由去往远方跑。\n\n\n\n\n\n\n

🚥 Pre-merge checks | ✅ 2 | ❌ 1

\n\n### ❌ Failed checks (1 warning)\n\n| Check name | Status | Explanation | Resolution |\n| :----------------: | :--------- | :----------------------------------------------------------------------------------- | :--------------------------------------------------------------------------------- |\n| Docstring Coverage | ⚠️ Warning | Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. | Write docstrings for the functions missing them to satisfy the coverage threshold. |\n\n

✅ Passed checks (2 passed)

\n\n| Check name | Status | Explanation |\n| :---------------: | :------- | :---------------------------------------------------------- |\n| Description Check | ✅ Passed | Check skipped - CodeRabbit’s high-level summary is enabled. |\n| Title check | ✅ Passed | 标题清晰准确地反映了本次变更的主要目的：恢复Windows下--dev选项的源地址选择功能。 |\n\n

\n\n_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}\n\n

\n\n\n\n\n\n

✨ Finishing Touches

\n\n

🧪 Generate unit tests (beta)

\n\n- [ ] Create PR with unit tests\n\n

\n\n

\n\n\n\n\n\n---\n\nThanks for using [CodeRabbit](https://coderabbit.ai?utm_source=oss&utm_medium=github&utm_campaign=nxtrace/NTrace-core&utm_content=364)! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.\n\n

❤️ Share

\n\n- [X](https://twitter.com/intent/tweet?text=I%20just%20used%20%40coderabbitai%20for%20my%20code%20review%2C%20and%20it%27s%20fantastic%21%20It%27s%20free%20for%20OSS%20and%20offers%20a%20free%20trial%20for%20the%20proprietary%20code.%20Check%20it%20out%3A&url=https%3A//coderabbit.ai)\n- [Mastodon](https://mastodon.social/share?text=I%20just%20used%20%40coderabbitai%20for%20my%20code%20review%2C%20and%20it%27s%20fantastic%21%20It%27s%20free%20for%20OSS%20and%20offers%20a%20free%20trial%20for%20the%20proprietary%20code.%20Check%20it%20out%3A%20https%3A%2F%2Fcoderabbit.ai)\n- [Reddit](https://www.reddit.com/submit?title=Great%20tool%20for%20code%20review%20-%20CodeRabbit&text=I%20just%20used%20CodeRabbit%20for%20my%20code%20review%2C%20and%20it%27s%20fantastic%21%20It%27s%20free%20for%20OSS%20and%20offers%20a%20free%20trial%20for%20proprietary%20code.%20Check%20it%20out%3A%20https%3A//coderabbit.ai)\n- [LinkedIn](https://www.linkedin.com/sharing/share-offsite/?url=https%3A%2F%2Fcoderabbit.ai&mini=true&title=Great%20tool%20for%20code%20review%20-%20CodeRabbit&summary=I%20just%20used%20CodeRabbit%20for%20my%20code%20review%2C%20and%20it%27s%20fantastic%21%20It%27s%20free%20for%20OSS%20and%20offers%20a%20free%20trial%20for%20proprietary%20code)\n\n

\n\n_{Comment `@coderabbitai help` to get the list of available commands and usage tips.}\n\n\n\n\n\n\n\n\n\n`NormalizeExplicitSourceConfig` no longer uses the `method` parameter, and the added `_ = method` is a code smell that can confuse readers. Prefer renaming the parameter to `_` in the function signature (keeps API compatibility without an in-body no-op), or add a brief comment explaining why the parameter is intentionally unused (e.g., kept for backward compatibility).\n## Pull request overview\n\nRestores Windows `--dev` behavior so TCP traces no longer fail early and instead derive a source IP from the selected device, aligning Windows behavior across ICMP/TCP/UDP and updating documentation to reflect that Windows `--dev` is source-address selection (not strict egress-interface binding).\n\n**Changes:**\n- Remove the Windows+TCP `SourceDevice` rejection in `NormalizeExplicitSourceConfig`, allowing device→source-IP resolution.\n- Update/extend tests to validate Windows TCP resolves `--dev` to `SrcAddr` and clears `SourceDevice`.\n- Update CLI help text and READMEs to clarify Windows `--dev` and standalone `--mtu` semantics.\n\n### Reviewed changes\n\nCopilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.\n\n

Show a summary per file

\n\n| File | Description |\\n| ---- | ----------- |\\n| `trace/source_config.go` | Removes Windows TCP `--dev` rejection; normalizes Windows `--dev` into `SrcAddr` while clearing `SourceDevice`. |\\n| `trace/source_config_test.go` | Updates test to assert Windows TCP resolves device to a source IP instead of erroring. |\\n| `cmd/cmd.go` | Adjusts `--dev` help text and related comments to match restored Windows behavior. |\\n| `README.md` | Documents Windows `--dev` as source-IP selection for ICMP/TCP/UDP and clarifies `--mtu` behavior. |\\n| `README_zh_CN.md` | Same documentation updates as `README.md` (Chinese). |\\n| `AGENTS.md` | Records the Windows compatibility rule: resolve `--dev` to `SrcAddr` and clear `SourceDevice` (v1.6.2-compatible behavior). |\n

\n\n\n\n\n\n\n---\n\n💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.\n\n\n

🧹 Nitpick comments (1)

\n\n
\n
trace/source_config_test.go (1)
\n\n`303-330`: **建议把 Windows 协议覆盖做成表驱动。**\n\n当前新增用例覆盖了这次 TCP 回归；可以顺手把 `ICMPTrace` / `TCPTrace` / `UDPTrace` 都放进同一个表，锁住 PR 描述里的三种协议行为，避免后续重新引入 method-specific 分支时漏掉 UDP。\n\n
\n
🧪 可选测试重构
\n\n```diff\n-func TestNormalizeExplicitSourceConfigWindowsTCPResolvesDeviceToSourceAddress(t *testing.T) {\n+func TestNormalizeExplicitSourceConfigWindowsResolvesDeviceToSourceAddress(t *testing.T) {\n \trestore := stubSourceDeviceResolver(t, func(device string) (*net.Interface, error) {\n \t\tif device != \"Ethernet0\" {\n \t\t\tt.Fatalf(\"ResolveSourceDevice device = %q, want Ethernet0\", device)\n \t\t}\n \t\treturn &net.Interface{Name: device}, nil\n \t}, func(_ *net.Interface) ([]net.Addr, error) {\n \t\treturn []net.Addr{&net.IPNet{IP: net.ParseIP(\"192.0.2.44\"), Mask: net.CIDRMask(24, 32)}}, nil\n \t})\n \tdefer restore()\n \n-\tcfg := Config{\n-\t\tOSType: osTypeWindows,\n-\t\tDstIP: net.ParseIP(\"1.1.1.1\"),\n-\t\tSourceDevice: \"Ethernet0\",\n-\t}\n-\n-\tgot, err := NormalizeExplicitSourceConfig(TCPTrace, cfg)\n-\tif err != nil {\n-\t\tt.Fatalf(\"NormalizeExplicitSourceConfig() error = %v\", err)\n-\t}\n-\tif got.SrcAddr != \"192.0.2.44\" {\n-\t\tt.Fatalf(\"SrcAddr = %q, want 192.0.2.44\", got.SrcAddr)\n-\t}\n-\tif got.SourceDevice != \"\" {\n-\t\tt.Fatalf(\"SourceDevice = %q, want empty\", got.SourceDevice)\n+\tfor _, method := range []Method{ICMPTrace, TCPTrace, UDPTrace} {\n+\t\tt.Run(string(method), func(t *testing.T) {\n+\t\t\tcfg := Config{\n+\t\t\t\tOSType: osTypeWindows,\n+\t\t\t\tDstIP: net.ParseIP(\"1.1.1.1\"),\n+\t\t\t\tSourceDevice: \"Ethernet0\",\n+\t\t\t}\n+\n+\t\t\tgot, err := NormalizeExplicitSourceConfig(method, cfg)\n+\t\t\tif err != nil {\n+\t\t\t\tt.Fatalf(\"NormalizeExplicitSourceConfig() error = %v\", err)\n+\t\t\t}\n+\t\t\tif got.SrcAddr != \"192.0.2.44\" {\n+\t\t\t\tt.Fatalf(\"SrcAddr = %q, want 192.0.2.44\", got.SrcAddr)\n+\t\t\t}\n+\t\t\tif got.SourceDevice != \"\" {\n+\t\t\t\tt.Fatalf(\"SourceDevice = %q, want empty\", got.SourceDevice)\n+\t\t\t}\n+\t\t})\n \t}\n }\n```\n
\n\nAs per coding guidelines, `**/cmd/cmd.go`: For Windows ICMP/TCP/UDP, use source IP to influence routing selection, not actual per-device binding; document in README that device specification may be inaccurate.\n\n
\n
🤖 Prompt for AI Agents
\n\n```\nVerify each finding against the current code and only fix it if needed.\n\nIn `@trace/source_config_test.go` around lines 303 - 330, The test\nTestNormalizeExplicitSourceConfigWindowsTCPResolvesDeviceToSourceAddress\ncurrently only covers TCP; change it to a table-driven test iterating over trace\ntypes (use TCPTrace, ICMPTrace, UDPTrace) and assert the same expectations for\nWindows: when SourceDevice resolves to an IP,\nNormalizeExplicitSourceConfig(traceType, cfg) returns SrcAddr == resolved IP and\nclears SourceDevice; update the test function name (e.g.\nTestNormalizeExplicitSourceConfigWindowsProtocolResolvesDeviceToSourceAddress)\nand loop over the trace types, calling NormalizeExplicitSourceConfig for each\nand running the same assertions, and add a short README note where Windows\nbehavior is documented (per-cmd behavior in cmd.go) that device spec affects\nrouting via source IP rather than guaranteed device binding.\n```\n\n
\n\n
\n\n

\n\n

🤖 Prompt for all review comments with AI agents

\n\n```\nVerify each finding against the current code and only fix it if needed.\n\nNitpick comments:\nIn `@trace/source_config_test.go`:\n- Around line 303-330: The test\nTestNormalizeExplicitSourceConfigWindowsTCPResolvesDeviceToSourceAddress\ncurrently only covers TCP; change it to a table-driven test iterating over trace\ntypes (use TCPTrace, ICMPTrace, UDPTrace) and assert the same expectations for\nWindows: when SourceDevice resolves to an IP,\nNormalizeExplicitSourceConfig(traceType, cfg) returns SrcAddr == resolved IP and\nclears SourceDevice; update the test function name (e.g.\nTestNormalizeExplicitSourceConfigWindowsProtocolResolvesDeviceToSourceAddress)\nand loop over the trace types, calling NormalizeExplicitSourceConfig for each\nand running the same assertions, and add a short README note where Windows\nbehavior is documented (per-cmd behavior in cmd.go) that device spec affects\nrouting via source IP rather than guaranteed device binding.\n```\n\n

\n\n---\n\n

ℹ️ Review info

\n\n

⚙️ Run configuration

\n\n**Configuration used**: Organization UI\n\n**Review profile**: CHILL\n\n**Plan**: Pro\n\n**Run ID**: `ecef484f-d74c-43f5-982d-ca57aff80c09`\n\n

\n\n

📥 Commits

\n\nReviewing files that changed from the base of the PR and between e1115a2553e6b90d42ecaf5e70a8c3c50b60b1dc and e66e2663f6b4d2235fb3ab0e2db276bf91010746.\n\n

\n\n

📒 Files selected for processing (6)

\n\n* `AGENTS.md`\n* `README.md`\n* `README_zh_CN.md`\n* `cmd/cmd.go`\n* `trace/source_config.go`\n* `trace/source_config_test.go`\n\n

\n\n

\n\n\n@coderabbitai run pre-merge checks\n\n

✅ Actions performed

\n\nRunning Pre-merge checks. Results will be posted in the summary comment once complete.\n\n

\n@codex run pre-merge checks\nTo use Codex here, [create an environment for this repo](https://chatgpt.com/codex/cloud/settings/environments).",{},"created",1776649691.9291658,"Completing the task","event_type","info",{"_92":390,"_96":93,"_384":668},{"_92":665,"_666":667},"partial_repo_snapshot","files",[],[669],{"_653":46,"_46":392},{"_92":307,"_42":84,"_96":307,"_672":673,"_674":675,"_676":677,"_132":-5,"_119":678},[83],"input_items",[696,697,698,699,700,701,702,703,704,705],"external_post_config",{"_679":680,"_245":178,"_681":287,"_255":-5,"_682":239,"_683":-5,"_684":685,"_686":687,"_688":-5,"_689":-5,"_690":691,"_692":15,"_693":59,"_694":54,"_695":15},"user_model_slug","slug",1776649476.223077,"installation_id",116305806,"pr_number","repo_id","review_trigger","comment_id",4277372776,"comment_type","issue","pr_write_guard_token","pending_pr_reaction_id","pending_comment_reaction_id",348612027,"comment_if_no_findings","comment_on_error","share_url","is_parallel_review",{"_92":390,"_96":307,"_384":722},{"_294":295,"_296":297,"_719":129,"_720":302,"_303":-5,"_92":721},{"_92":706,"_296":718,"_157":-5,"_708":-5,"_709":-5},{"_92":706,"_296":715,"_157":716,"_708":717,"_709":-5},{"_92":706,"_296":714,"_157":-5,"_708":-5,"_709":-5},{"_92":706,"_296":713,"_157":-5,"_708":-5,"_709":-5},{"_92":706,"_296":712,"_157":-5,"_708":-5,"_709":-5},{"_92":706,"_296":711,"_157":-5,"_708":-5,"_709":-5},{"_92":706,"_296":710,"_157":-5,"_708":-5,"_709":-5},{"_92":706,"_296":707,"_157":-5,"_708":-5,"_709":-5},"pull_request_comment","To use Codex here, [create an environment for this repo](https://chatgpt.com/codex/cloud/settings/environments).","side","line","@codex run pre-merge checks","\n

✅ Actions performed

\n\nRunning Pre-merge checks. Results will be posted in the summary comment once complete.\n\n

","@coderabbitai run pre-merge checks","\n\n

🧹 Nitpick comments (1)

\n\n
\n
trace/source_config_test.go (1)
\n\n`303-330`: **建议把 Windows 协议覆盖做成表驱动。**\n\n当前新增用例覆盖了这次 TCP 回归；可以顺手把 `ICMPTrace` / `TCPTrace` / `UDPTrace` 都放进同一个表，锁住 PR 描述里的三种协议行为，避免后续重新引入 method-specific 分支时漏掉 UDP。\n\n
\n
🧪 可选测试重构
\n\n```diff\n-func TestNormalizeExplicitSourceConfigWindowsTCPResolvesDeviceToSourceAddress(t *testing.T) {\n+func TestNormalizeExplicitSourceConfigWindowsResolvesDeviceToSourceAddress(t *testing.T) {\n \trestore := stubSourceDeviceResolver(t, func(device string) (*net.Interface, error) {\n \t\tif device != \"Ethernet0\" {\n \t\t\tt.Fatalf(\"ResolveSourceDevice device = %q, want Ethernet0\", device)\n \t\t}\n \t\treturn &net.Interface{Name: device}, nil\n \t}, func(_ *net.Interface) ([]net.Addr, error) {\n \t\treturn []net.Addr{&net.IPNet{IP: net.ParseIP(\"192.0.2.44\"), Mask: net.CIDRMask(24, 32)}}, nil\n \t})\n \tdefer restore()\n \n-\tcfg := Config{\n-\t\tOSType: osTypeWindows,\n-\t\tDstIP: net.ParseIP(\"1.1.1.1\"),\n-\t\tSourceDevice: \"Ethernet0\",\n-\t}\n-\n-\tgot, err := NormalizeExplicitSourceConfig(TCPTrace, cfg)\n-\tif err != nil {\n-\t\tt.Fatalf(\"NormalizeExplicitSourceConfig() error = %v\", err)\n-\t}\n-\tif got.SrcAddr != \"192.0.2.44\" {\n-\t\tt.Fatalf(\"SrcAddr = %q, want 192.0.2.44\", got.SrcAddr)\n-\t}\n-\tif got.SourceDevice != \"\" {\n-\t\tt.Fatalf(\"SourceDevice = %q, want empty\", got.SourceDevice)\n+\tfor _, method := range []Method{ICMPTrace, TCPTrace, UDPTrace} {\n+\t\tt.Run(string(method), func(t *testing.T) {\n+\t\t\tcfg := Config{\n+\t\t\t\tOSType: osTypeWindows,\n+\t\t\t\tDstIP: net.ParseIP(\"1.1.1.1\"),\n+\t\t\t\tSourceDevice: \"Ethernet0\",\n+\t\t\t}\n+\n+\t\t\tgot, err := NormalizeExplicitSourceConfig(method, cfg)\n+\t\t\tif err != nil {\n+\t\t\t\tt.Fatalf(\"NormalizeExplicitSourceConfig() error = %v\", err)\n+\t\t\t}\n+\t\t\tif got.SrcAddr != \"192.0.2.44\" {\n+\t\t\t\tt.Fatalf(\"SrcAddr = %q, want 192.0.2.44\", got.SrcAddr)\n+\t\t\t}\n+\t\t\tif got.SourceDevice != \"\" {\n+\t\t\t\tt.Fatalf(\"SourceDevice = %q, want empty\", got.SourceDevice)\n+\t\t\t}\n+\t\t})\n \t}\n }\n```\n
\n\nAs per coding guidelines, `**/cmd/cmd.go`: For Windows ICMP/TCP/UDP, use source IP to influence routing selection, not actual per-device binding; document in README that device specification may be inaccurate.\n\n
\n
🤖 Prompt for AI Agents
\n\n```\nVerify each finding against the current code and only fix it if needed.\n\nIn `@trace/source_config_test.go` around lines 303 - 330, The test\nTestNormalizeExplicitSourceConfigWindowsTCPResolvesDeviceToSourceAddress\ncurrently only covers TCP; change it to a table-driven test iterating over trace\ntypes (use TCPTrace, ICMPTrace, UDPTrace) and assert the same expectations for\nWindows: when SourceDevice resolves to an IP,\nNormalizeExplicitSourceConfig(traceType, cfg) returns SrcAddr == resolved IP and\nclears SourceDevice; update the test function name (e.g.\nTestNormalizeExplicitSourceConfigWindowsProtocolResolvesDeviceToSourceAddress)\nand loop over the trace types, calling NormalizeExplicitSourceConfig for each\nand running the same assertions, and add a short README note where Windows\nbehavior is documented (per-cmd behavior in cmd.go) that device spec affects\nrouting via source IP rather than guaranteed device binding.\n```\n\n
\n\n
\n\n

\n\n

🤖 Prompt for all review comments with AI agents

\n\n---\n\n

ℹ️ Review info

\n\n

⚙️ Run configuration

\n\n**Configuration used**: Organization UI\n\n**Review profile**: CHILL\n\n**Plan**: Pro\n\n**Run ID**: `ecef484f-d74c-43f5-982d-ca57aff80c09`\n\n

\n\n

📥 Commits

\n\nReviewing files that changed from the base of the PR and between e1115a2553e6b90d42ecaf5e70a8c3c50b60b1dc and e66e2663f6b4d2235fb3ab0e2db276bf91010746.\n\n

\n\n

📒 Files selected for processing (6)

\n\n* `AGENTS.md`\n* `README.md`\n* `README_zh_CN.md`\n* `cmd/cmd.go`\n* `trace/source_config.go`\n* `trace/source_config_test.go`\n\n

\n\n

\n\n","## Pull request overview\n\nRestores Windows `--dev` behavior so TCP traces no longer fail early and instead derive a source IP from the selected device, aligning Windows behavior across ICMP/TCP/UDP and updating documentation to reflect that Windows `--dev` is source-address selection (not strict egress-interface binding).\n\n**Changes:**\n- Remove the Windows+TCP `SourceDevice` rejection in `NormalizeExplicitSourceConfig`, allowing device→source-IP resolution.\n- Update/extend tests to validate Windows TCP resolves `--dev` to `SrcAddr` and clears `SourceDevice`.\n- Update CLI help text and READMEs to clarify Windows `--dev` and standalone `--mtu` semantics.\n\n### Reviewed changes\n\nCopilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.\n\n

Show a summary per file

\n\n\n\n\n\n\n---\n\n💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.","`NormalizeExplicitSourceConfig` no longer uses the `method` parameter, and the added `_ = method` is a code smell that can confuse readers. Prefer renaming the parameter to `_` in the function signature (keeps API compatibility without an in-body no-op), or add a brief comment explaining why the parameter is intentionally unused (e.g., kept for backward compatibility).","trace/source_config.go","RIGHT","\n\n\n## Walkthrough\n\n本次变更移除了对 Windows 上 TCP + `--dev` 的显式拒绝，文档与 CLI 帮助文本同步更新；实现上 `NormalizeExplicitSourceConfig` 在 Windows 下会把指定设备解析成源 IP 写入 `SrcAddr` 并清空 `SourceDevice`，相关单元测试调整为断言成功解析该地址。\n\n## Changes\n\n|Cohort / File(s)|Summary|\n|---|---|\n|**文档更新**
`AGENTS.md`, `README.md`, `README_zh_CN.md`|更新 Windows 上 `--dev` 的描述：说明 `--dev` 会从指定网卡派生源 IP 并用于 ICMP/TCP/UDP 探测，但不等同于强制绑定出口接口；补充 `--mtu` 单独模式遵循相同源地址语义。|\n|**CLI 帮助与注释**
`cmd/cmd.go`|调整 `--dev/-D` 的帮助文本与两处内联注释措辞，移除“TCP + --dev 不支持/报错”表述，改为“选择设备源地址；路由可能仍选出口接口”。|\n|**核心逻辑**
`trace/source_config.go`|删除基于 `method` 的 Windows 拒绝分支（不再因 TCP 而报错）；参数名去掉未使用的 `method` 依赖，保持其他解析/清理逻辑不变。|\n|**单元测试**
`trace/source_config_test.go`|替换拒绝型测试为表驱动测试（ICMP/TCP/UDP），调整存根以返回 `192.0.2.44/24` 并断言 `SrcAddr==\"192.0.2.44\"` 且 `SourceDevice==\"\"`。|\n\n## Estimated code review effort\n\n🎯 3 (Moderate) | ⏱️ ~20 分钟\n\n## 诗句\n\n> 🐰 Windows 兔子跳一跳， \n> 网卡化作小小源地址， \n> TCP、UDP、ICMP 一同笑， \n> 不再拒绝，只把源记牢， \n> 路由自由去往远方跑。\n\n\n\n\n\n\n

🚥 Pre-merge checks | ✅ 2 | ❌ 1

✅ Passed checks (2 passed)

\n\n_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}\n\n

\n\n\n\n\n\n

✨ Finishing Touches

\n\n

🧪 Generate unit tests (beta)

\n\n- [ ] Create PR with unit tests\n\n

\n\n

❤️ Share

\n\n_{Comment `@coderabbitai help` to get the list of available commands and usage tips.}\n\n\n\n\n\n\n\n\n","base_ref","head_ref","pull_request_info",[723],{"_653":46,"_46":710},[940,941],[],{"_658":659,"_46":660,"_661":662},{"_633":931},{"_321":930},{"_321":755},{"_286":287,"_276":288,"_289":290,"_291":59,"_292":15,"_293":15,"_294":295,"_296":297,"_298":127,"_299":127,"_300":129,"_301":302,"_303":304,"_305":-5,"_306":-5,"_307":754,"_309":310,"_311":312,"_313":314,"_315":316},{"_42":123,"_175":176,"_177":178,"_179":15,"_180":181,"_182":183,"_184":741,"_186":-5,"_187":-5,"_188":742,"_190":743,"_192":744,"_194":745,"_196":746,"_198":747,"_200":-5,"_201":202,"_203":204,"_205":206,"_207":748,"_55":-5,"_119":209,"_210":749,"_212":750,"_214":215,"_216":15,"_217":15},{"_133":736},[734,735],{"_144":158,"_146":147,"_148":149,"_150":159,"_152":153,"_154":155,"_156":32,"_157":-5},{"_144":145,"_146":147,"_148":149,"_150":151,"_152":153,"_154":155,"_156":32,"_157":-5},[737,738],{"_163":740,"_165":172},{"_163":739,"_165":166},{"_167":168,"_92":169,"_119":170},{"_167":173,"_92":169,"_119":174},{"_278":279,"_280":281,"_282":181,"_283":-5,"_284":-5,"_285":-5},[239],[181],[181],{"_239":751},{"_221":222,"_223":224,"_225":226,"_227":228,"_229":230,"_231":232,"_233":234,"_235":236,"_237":238},{},[],{"_220":59},{"_218":59,"_219":181},{"_42":241,"_167":242,"_243":752,"_245":178,"_246":247,"_248":127,"_249":250,"_251":252,"_253":254,"_255":64,"_256":15,"_257":15,"_258":59,"_259":59,"_260":59,"_261":59,"_262":-5,"_156":263,"_264":-5,"_55":753},{"_270":271,"_42":272,"_92":273,"_274":275,"_276":277},{"_266":59,"_267":15,"_268":59,"_269":59},{"_270":317,"_167":317,"_318":-5,"_274":319,"_42":320},[756,757,758,759,760,761,762,763,764,765,766,767,768,769,770,771,772,773,774,775,776,777,778,779,780,781,782,783,784,785,786,787,788,789,790,791,792,793,794,795,796,797],{"_150":608,"_366":926},{"_150":379,"_366":922},{"_150":379,"_366":917},{"_150":375,"_366":914},{"_150":584,"_366":911},{"_150":379,"_366":907},{"_150":394,"_366":902},{"_150":394,"_366":898},{"_150":379,"_366":895},{"_150":379,"_366":893},{"_150":394,"_366":889},{"_150":379,"_366":887},{"_150":394,"_366":883},{"_150":379,"_366":880},{"_150":379,"_366":878},{"_150":394,"_366":874},{"_150":379,"_366":872},{"_150":394,"_366":868},{"_150":379,"_366":865},{"_150":379,"_366":863},{"_150":379,"_366":861},{"_150":394,"_366":857},{"_150":379,"_366":854},{"_150":379,"_366":852},{"_150":394,"_366":848},{"_150":379,"_366":846},{"_150":394,"_366":842},{"_150":379,"_366":839},{"_150":379,"_366":837},{"_150":394,"_366":833},{"_150":379,"_366":831},{"_150":394,"_366":827},{"_150":379,"_366":824},{"_150":379,"_366":822},{"_150":394,"_366":818},{"_150":379,"_366":816},{"_150":394,"_366":812},{"_150":379,"_366":809},{"_150":394,"_366":807},{"_150":379,"_366":803},{"_150":375,"_366":801},{"_150":365,"_366":798},{"_368":369,"_87":799},{"_103":-5,"_42":371,"_372":800,"_374":113},[],{"_374":802,"_368":369},{"_92":378},{"_381":804,"_368":369,"_383":371},{"_384":805,"_386":-5,"_42":-5,"_387":-5,"_388":389,"_96":93,"_92":390},[806],{"_46":392,"_92":393},{"_381":808,"_368":369,"_383":371},{"_42":397,"_388":389,"_46":392,"_92":398},{"_381":810,"_368":369,"_383":371},{"_384":-5,"_401":402,"_42":-5,"_387":-5,"_403":811,"_92":405},[],{"_381":813,"_368":369,"_383":371},{"_384":814,"_42":409,"_403":815,"_92":405},[],[],{"_381":817,"_368":369,"_383":371},{"_413":414,"_42":-5,"_387":-5,"_415":416,"_92":417},{"_381":819,"_368":369,"_383":371},{"_420":-5,"_421":422,"_423":820,"_425":426,"_427":428,"_429":32,"_42":414,"_430":431,"_432":433,"_374":113,"_92":434},[821],{"_421":436,"_92":437},{"_381":823,"_368":369,"_383":371},{"_440":441,"_413":414,"_42":-5,"_387":-5,"_167":442,"_443":-5,"_92":444},{"_381":825,"_368":369,"_383":371},{"_384":-5,"_401":447,"_42":-5,"_387":-5,"_403":826,"_92":405},[],{"_381":828,"_368":369,"_383":371},{"_384":829,"_42":452,"_403":830,"_92":405},[],[],{"_381":832,"_368":369,"_383":371},{"_413":456,"_42":-5,"_387":-5,"_415":457,"_92":417},{"_381":834,"_368":369,"_383":371},{"_420":460,"_421":461,"_423":835,"_425":426,"_427":463,"_429":32,"_42":456,"_430":464,"_432":433,"_374":113,"_92":434},[836],{"_421":466,"_92":437},{"_381":838,"_368":369,"_383":371},{"_440":469,"_413":456,"_42":-5,"_387":-5,"_167":442,"_443":-5,"_92":444},{"_381":840,"_368":369,"_383":371},{"_384":-5,"_401":472,"_42":-5,"_387":-5,"_403":841,"_92":405},[],{"_381":843,"_368":369,"_383":371},{"_384":844,"_42":477,"_403":845,"_92":405},[],[],{"_381":847,"_368":369,"_383":371},{"_413":481,"_42":-5,"_387":-5,"_415":482,"_92":417},{"_381":849,"_368":369,"_383":371},{"_420":485,"_421":486,"_423":850,"_425":426,"_427":488,"_429":32,"_42":489,"_430":490,"_432":433,"_374":113,"_92":434},[851],{"_421":492,"_92":437},{"_381":853,"_368":369,"_383":371},{"_440":495,"_413":481,"_42":-5,"_387":-5,"_167":496,"_443":-5,"_92":444},{"_381":855,"_368":369,"_383":371},{"_384":-5,"_401":499,"_42":-5,"_387":-5,"_403":856,"_92":405},[],{"_381":858,"_368":369,"_383":371},{"_384":859,"_42":504,"_403":860,"_92":405},[],[],{"_381":862,"_368":369,"_383":371},{"_413":489,"_42":-5,"_387":-5,"_415":508,"_92":417},{"_381":864,"_368":369,"_383":371},{"_440":511,"_413":489,"_42":-5,"_387":-5,"_167":442,"_443":-5,"_92":444},{"_381":866,"_368":369,"_383":371},{"_384":-5,"_401":514,"_42":-5,"_387":-5,"_403":867,"_92":405},[],{"_381":869,"_368":369,"_383":371},{"_384":870,"_42":519,"_403":871,"_92":405},[],[],{"_381":873,"_368":369,"_383":371},{"_413":523,"_42":-5,"_387":-5,"_415":524,"_92":417},{"_381":875,"_368":369,"_383":371},{"_420":527,"_421":528,"_423":876,"_425":426,"_427":32,"_429":32,"_42":523,"_430":530,"_432":433,"_374":113,"_92":434},[877],{"_421":532,"_92":437},{"_381":879,"_368":369,"_383":371},{"_440":535,"_413":523,"_42":-5,"_387":-5,"_167":442,"_443":-5,"_92":444},{"_381":881,"_368":369,"_383":371},{"_384":-5,"_401":538,"_42":-5,"_387":-5,"_403":882,"_92":405},[],{"_381":884,"_368":369,"_383":371},{"_384":885,"_42":543,"_403":886,"_92":405},[],[],{"_381":888,"_368":369,"_383":371},{"_413":547,"_42":-5,"_387":-5,"_415":548,"_92":417},{"_381":890,"_368":369,"_383":371},{"_420":551,"_421":552,"_423":891,"_425":426,"_427":32,"_429":32,"_42":547,"_430":554,"_432":433,"_374":113,"_92":434},[892],{"_421":556,"_92":437},{"_381":894,"_368":369,"_383":371},{"_440":559,"_413":547,"_42":-5,"_387":-5,"_167":442,"_443":-5,"_92":444},{"_381":896,"_368":369,"_383":371},{"_384":-5,"_401":562,"_42":-5,"_387":-5,"_403":897,"_92":405},[],{"_381":899,"_368":369,"_383":371},{"_384":900,"_42":567,"_403":901,"_92":405},[],[],{"_381":903,"_368":369,"_383":371},{"_571":-5,"_384":904,"_42":573,"_92":574},[905],{"_46":576,"_577":906,"_92":46},[],{"_381":908,"_368":369,"_383":371},{"_384":909,"_386":-5,"_42":-5,"_387":-5,"_388":-5,"_96":307,"_92":390},[910],{"_46":576,"_92":583},{"_368":369,"_87":912},{"_103":-5,"_42":371,"_372":913,"_374":588},[],{"_374":915,"_368":369},{"_591":916,"_92":593},[],{"_381":918,"_368":369,"_383":596},{"_384":919,"_386":-5,"_42":-5,"_387":-5,"_388":-5,"_96":307,"_92":390},[920,921],{"_46":601,"_92":583},{"_46":600,"_92":583},{"_381":923,"_368":369,"_383":596},{"_384":924,"_386":-5,"_42":-5,"_387":-5,"_388":-5,"_96":605,"_92":390},[925],{"_46":607,"_92":583},{"_610":927},{"_612":-5,"_613":-5,"_614":615,"_616":617,"_425":426,"_618":15,"_619":-5,"_620":-5,"_42":369,"_621":622,"_167":-5,"_623":-5,"_157":624,"_625":181,"_626":-5,"_432":627,"_374":928,"_629":-5,"_77":929,"_631":617},{"_92":378},[],[],[932],{"_42":636,"_637":933,"_639":640,"_641":640,"_384":934,"_374":643,"_386":-5,"_644":32,"_645":935,"_647":281,"_648":-5},{"_96":307,"_167":-5,"_645":939},{"_653":46,"_654":938},{"_649":936},{"_651":937},[],[656],{},{"_92":390,"_96":93,"_384":943},{"_92":665,"_666":942},[],[944],{"_653":46,"_46":392},[947,948,949,950,951,952,953,954,955,956],{"_679":680,"_245":178,"_681":287,"_255":-5,"_682":239,"_683":-5,"_684":685,"_686":687,"_688":-5,"_689":-5,"_690":691,"_692":15,"_693":59,"_694":54,"_695":15},{"_92":390,"_96":307,"_384":957},{"_294":295,"_296":297,"_719":129,"_720":302,"_303":-5,"_92":721},{"_92":706,"_296":718,"_157":-5,"_708":-5,"_709":-5},{"_92":706,"_296":715,"_157":716,"_708":717,"_709":-5},{"_92":706,"_296":714,"_157":-5,"_708":-5,"_709":-5},{"_92":706,"_296":713,"_157":-5,"_708":-5,"_709":-5},{"_92":706,"_296":712,"_157":-5,"_708":-5,"_709":-5},{"_92":706,"_296":711,"_157":-5,"_708":-5,"_709":-5},{"_92":706,"_296":710,"_157":-5,"_708":-5,"_709":-5},{"_92":706,"_296":707,"_157":-5,"_708":-5,"_709":-5},[958],{"_653":46,"_46":710},"task_e_69e585038da08324b55b43749171c098",1776649475.553342,"Github Mention: [codex] Restore Windows --dev source selection","has_generated_title","has_unread_turn","denormalized_metadata",{"_990":32,"_991":992,"_993":-5,"_994":-5,"_985":995,"_996":-5,"_997":-5,"_998":-5,"_999":-5},"task_status_display",{"_978":979,"_980":178,"_130":127,"_981":-5,"_982":983},"creator_workspace","external_pull_requests",[971],{"_42":972,"_973":83,"_974":975,"_976":302},"github-491101744-364","assistant_turn_id","pull_request",{"_286":287,"_276":288,"_289":290,"_291":59,"_292":15,"_293":15,"_294":295,"_296":297,"_298":127,"_299":127,"_300":129,"_301":302,"_303":304,"_305":-5,"_306":-5,"_307":977,"_309":310,"_311":312,"_313":314,"_315":316},"codex_updated_sha",{"_270":317,"_167":317,"_318":-5,"_274":319,"_42":320},"latest_turn_status_display",{"_984":83,"_112":113,"_985":-5,"_94":986,"_987":-5,"_988":-5,"_989":983,"_114":-5},"environment_label","fix_task_id","initial_intent","pr","turn_id","diff_stats",[],"image_asset_pointer_citations","issue_counts","intent","num_follow_ups","repo_name_map",{},"last_user_turn_created_at","last_turn_status",{"_1000":-5,"_1001":-5,"_1002":-5},"last_pushed_turn_id","start_branch_name","pushed_branch_name","cached_pull_request_data","files_modified","lines_added","lines_removed","dd",{"_1014":1015,"_1016":1017},"disablePrefetch","shouldPrefetchAccount","shouldPrefetchUser","shouldPrefetchSystemHints","promoteCss","disableSSR","stripModulepreloadImports","statsigGateEvaluationsPromise",["P",1013],"traceId","13594503948555703625","traceTime",1782994195877]