Preserve native PowerShell alias collisions

Author: SqlRush

docs/cc-100-roadmap.md

@@ -211,6 +211,8 @@ M5 补充：PowerShell read-only classifier 现在覆盖 `Get-FileHash` 的安
 
 M5 补充：PowerShell read-only classifier 继续覆盖 Windows `certutil -hashfile <path> [algorithm]` 的安全相对路径 checksum 形态，同时拒绝 `certutil` 其它子命令、pass-through、越界路径、动态算法和额外参数。
 
+M5 补充：PowerShell command canonicalization 现在保留与 PowerShell alias 碰撞的 `.exe`/`.cmd`/`.bat`/`.com` 原生命令名，避免 `fc.exe`、`sort.exe`、`sc.exe` 等被误当成 `Format-Custom`、`Sort-Object` 或 `Set-Content` 分类。
+
 M5 补充：Bash 前台输出和 `BashOutput` 现在都走统一 tool-result budget 截断/落盘路径；`BashOutput` 增加 100k 最大结果限制，大后台输出会保存完整内容并返回 `full_output_path` 元数据。
 
 M5 补充：Bash/PowerShell 后台任务现在会发 `*_background_started` 和 `*_background_finished` tool progress 事件，包含后台 ID、shell/status、exit/timed_out/cancelled、duration、时间戳和输出字节数，不携带 command 文本；completed、timed_out、cancelled 终态均已覆盖测试。

docs/claude-code-go-rewrite-plan.md

@@ -217,6 +217,7 @@ test/parity/                 # golden tests against TS/official behavior
 - 本轮补充：Bash read-only classifier 现在覆盖常见 checksum 命令 `md5sum`/`sha*sum`/`b2sum`/`shasum`/`cksum`/`sum` 的普通文件读取形态，同时阻断 `-c`/`--check` 校验清单模式，避免清单内路径绕过路径权限。
 - 本轮补充：PowerShell read-only classifier 现在覆盖 `Get-FileHash` 的安全路径读取和常见 native checksum 命令 `md5sum`/`sha*sum`/`b2sum`/`shasum`/`cksum`/`sum`，同时拒绝 `Get-FileHash -InputStream` 表达式和 `-c`/`--check` 校验清单模式。
 - 本轮补充：PowerShell read-only classifier 继续覆盖 Windows `certutil -hashfile <path> [algorithm]` 的安全相对路径 checksum 形态，同时拒绝 `certutil` 其它子命令、pass-through、越界路径、动态算法和额外参数。
+- 本轮补充：PowerShell command canonicalization 现在保留与 PowerShell alias 碰撞的 `.exe`/`.cmd`/`.bat`/`.com` 原生命令名，避免 `fc.exe`、`sort.exe`、`sc.exe` 等被误当成 `Format-Custom`、`Sort-Object` 或 `Set-Content` 分类。
 - 本轮补充：`BashOutput` 现在设置 100k 最大结果大小，和前台 `Bash` 一起覆盖大输出 tool-result preview 截断、完整输出落盘及 `full_output_path` 元数据。
 - 本轮补充：Bash/PowerShell 后台命令现在会通过 tool progress 通道发 started/finished 事件，记录后台 ID、shell/status、exit/timed_out/cancelled、duration、时间戳和 stdout/stderr byte count，且 completed、timed_out、cancelled 终态测试确保 progress 不携带 command 文本。
 - 本轮补充：WebSearch domain filters 现在在 schema 层声明 array `items:string`，通用 tool schema validator 同步支持 `items` 校验；`allowed_domains`/`blocked_domains` 会拒绝空字符串、URL/port、非法 wildcard 和非域名 label。

internal/tools/powershell/tools.go

@@ -2026,17 +2026,30 @@ func safeRelativePowerShellPath(path string) bool {
 
 func canonicalCommand(command string) string {
 	name := strings.ToLower(strings.Trim(strings.TrimSpace(command), `"'`))
-	if name == "where.exe" {
-		return name
-	}
 	if !strings.ContainsAny(name, `/\`) {
-		for _, suffix := range []string{".exe", ".cmd", ".bat", ".com"} {
-			if strings.HasSuffix(name, suffix) {
-				name = strings.TrimSuffix(name, suffix)
-				break
+		if stem, ok := stripPowerShellExecutableSuffix(name); ok {
+			if powerShellAliasTarget(stem) != "" {
+				return name
 			}
+			name = stem
 		}
 	}
+	if target := powerShellAliasTarget(name); target != "" {
+		return target
+	}
+	return name
+}
+
+func stripPowerShellExecutableSuffix(name string) (string, bool) {
+	for _, suffix := range []string{".exe", ".cmd", ".bat", ".com"} {
+		if strings.HasSuffix(name, suffix) {
+			return strings.TrimSuffix(name, suffix), true
+		}
+	}
+	return name, false
+}
+
+func powerShellAliasTarget(name string) string {
 	switch name {
 	case "cat", "gc":
 		return "get-content"
@@ -2105,6 +2118,6 @@ func canonicalCommand(command string) string {
 	case "measure":
 		return "measure-object"
 	default:
-		return name
+		return ""
 	}
 }

internal/tools/powershell/tools_test.go

@@ -350,6 +350,8 @@ func TestPowerShellCommandClassification(t *testing.T) {
 		"certutil -hashfile README.md SHA256 extra",
 		"certutil --% -hashfile README.md SHA256",
 		"certutil -urlcache -split -f https://example.com/file out.bin",
+		"fc.exe README.md /etc/passwd",
+		"sort.exe /etc/passwd",
 		"dotnet build",
 		"dotnet --info $env:SECRET",
 		"docker run alpine",
@@ -421,6 +423,9 @@ func TestPowerShellCommandClassification(t *testing.T) {
 			t.Fatalf("%q should be destructive", command)
 		}
 	}
+	if IsDestructiveCommand("sc.exe query") {
+		t.Fatalf("native sc.exe should not be classified as Set-Content alias")
+	}
 }
 
 func TestPowerShellToolDynamicSafetyFlags(t *testing.T) {