fix: preserve explicit node routing under elevated auto exec#61739
Conversation
Greptile SummaryFixes a routing bug where an explicit Confidence Score: 5/5Safe to merge — targeted fix with no correctness issues found and good regression coverage. All changes are correct and well-tested. No P0/P1 issues found. The logic is sound: validation now runs before elevation, selectedTarget correctly incorporates the per-call requestedTarget override, and downstream callers of effectiveHost/selectedTarget are unaffected by the change in returned values. No files require special attention. Reviews (1): Last reviewed commit: "fix: preserve explicit node routing unde..." | Re-trigger Greptile |
275cb1e to
3222c63
Compare
Summary
host=noderouting when exec runs with elevated defaults undertools.exec.host=autoTesting
pnpm test src/agents/bash-tools.exec-runtime.test.ts src/agents/bash-tools.exec.approval-id.test.tsLinked Issue