Greptile Summary

This PR fixes a regression introduced in 2026.3.2 where every nodes run command using a shell wrapper (e.g., powershell, bash) would fail with INVALID_REQUEST: rawCommand does not match command. The fix is a targeted two-line removal: the rawCommand: formatExecCommand(command) parameter from the system.run.prepare call and its associated (now-unused) import.

Key changes:

• Removes rawCommand: formatExecCommand(command) from the system.run.prepare params in nodes-tool.ts.
• Removes the now-unused formatExecCommand import from system-run-command.js.

Why this works: validateSystemRunCommandConsistency (in system-run-command.ts) skips the raw-vs-inferred comparison when rawCommand is null/undefined (if (raw && raw !== inferred)). The server then infers the correct cmdText from the argv array. The subsequent system.run call still receives a valid rawCommand via prepared.plan.rawCommand ?? prepared.cmdText, so downstream approval and display logic is unaffected.

Confidence Score: 5/5

• This PR is safe to merge — it is a minimal, targeted removal of a parameter that was causing a hard failure for all shell-wrapper commands.
• The change is a two-line removal with a clearly diagnosed root cause. The validation logic in validateSystemRunCommandConsistency explicitly short-circuits when rawCommand is absent, and the downstream system.run call still receives rawCommand from the server-resolved prepared.plan.rawCommand ?? prepared.cmdText. No logic is altered beyond removing the mismatched input.
• No files require special attention.

_{Last reviewed commit: d6069a3}

Rebased on latest main. Ready for review! 🙏

Landed via temp rebase onto main.

• Gate: pnpm format:check; pnpm test -- --run src/agents/tools/nodes-tool.test.ts src/infra/system-run-command.test.ts src/infra/system-run-command.contract.test.ts
• Note: pnpm tsgo is currently red on unchanged extensions/feishu/src/monitor.test-mocks.ts
• Land commit: e959b74
• Merge commit: ac63f30

Thanks @madsmash!