1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
use serde_json::{Map, Value};
use crate::server::tool_trait::{get_str, get_str_array, ToolContext};
#[derive(Debug)]
pub struct ResolvedPaths {
pub roots: Vec<String>,
pub is_multi: bool,
}
/// Resolve tool paths with multi-root support.
///
/// Priority:
/// 0. `repo` argument (multi-repo alias → specific root)
/// 1. `paths` array argument (explicit multi-root)
/// 2. `path` string argument (single root, pre-resolved by dispatch)
/// 3. Session `extra_roots` (default multi-root from config/MCP)
/// 4. Fallback to `"."` (project root)
///
/// Returns `Err` when an **explicit** `path`/`paths` argument was supplied but
/// could not be resolved (outside the project root, secret-screened, or
/// non-existent). Silently falling back to the project root in that case made
/// `ctx_tree path=/outside/repo` return the whole project tree (#401).
pub fn resolve_tool_paths(
args: &Map<String, Value>,
ctx: &ToolContext,
) -> Result<ResolvedPaths, String> {
if let Some(repo) = get_str(args, "repo") {
if let Some(root) = crate::core::multi_repo::resolve_repo_root(&repo) {
return Ok(ResolvedPaths {
roots: vec![root],
is_multi: false,
});
}
}
if let Some(paths) = get_str_array(args, "paths") {
if !paths.is_empty() {
let resolved = resolve_paths_sync(ctx, &paths);
if !resolved.is_empty() {
return Ok(ResolvedPaths {
is_multi: resolved.len() > 1,
roots: resolved,
});
}
// The caller explicitly listed paths but none resolved — surface
// the failure instead of scanning the project root (#401).
return Err(format!(
"none of the requested paths could be resolved — they may not exist or are \
outside the project root: {}",
paths.join(", ")
));
}
}
if let Some(path) = ctx.resolved_path("path") {
return Ok(ResolvedPaths {
roots: vec![path.to_string()],
is_multi: false,
});
}
// An explicit `path` the dispatcher could not resolve lands in
// `path_errors` (not `resolved_paths`). Do NOT fall back to the project
// root — return the resolution error so the agent learns the path is out
// of scope rather than silently receiving an unrelated tree (#401).
if let Some(detail) = ctx.path_error("path") {
return Err(detail.to_string());
}
if let Some(session_lock) = ctx.session.as_ref() {
let (extra, jail_root) = tokio::task::block_in_place(|| {
let rt = tokio::runtime::Handle::current();
rt.block_on(async {
let session = session_lock.read().await;
let root = session
.project_root
.clone()
.unwrap_or_else(|| ".".to_string());
(session.extra_roots.clone(), root)
})
});
if !extra.is_empty() {
let jail = std::path::Path::new(&jail_root);
let mut roots = vec![ctx.project_root.clone()];
for r in &extra {
let p = std::path::Path::new(r);
if !p.is_dir() {
continue;
}
match crate::core::pathjail::jail_path(p, jail) {
Ok(_) => roots.push(r.clone()),
Err(e) => tracing::warn!("extra_root rejected by PathJail: {e}"),
}
}
if roots.len() > 1 {
return Ok(ResolvedPaths {
is_multi: true,
roots,
});
}
}
}
Ok(ResolvedPaths {
roots: vec![".".to_string()],
is_multi: false,
})
}
fn resolve_paths_sync(ctx: &ToolContext, raw: &[String]) -> Vec<String> {
let mut out = Vec::with_capacity(raw.len());
for p in raw {
match ctx.resolve_path_sync(p) {
Ok(resolved) => out.push(resolved),
Err(e) => {
tracing::warn!("multi-path resolve failed for {p}: {e}");
}
}
}
out
}
#[cfg(test)]
mod tests {
use super::*;
use serde_json::json;
fn test_ctx() -> ToolContext {
ToolContext {
project_root: "/test/project".to_string(),
minimal: false,
resolved_paths: std::collections::HashMap::new(),
crp_mode: crate::tools::CrpMode::Off,
cache: None,
session: None,
tool_calls: None,
agent_id: None,
workflow: None,
ledger: None,
client_name: None,
pipeline_stats: None,
call_count: None,
autonomy: None,
pressure_snapshot: None,
path_errors: std::collections::HashMap::new(),
bm25_cache: None,
progress_sender: None,
}
}
#[test]
fn fallback_to_dot_when_nothing_set() {
let args = Map::new();
let ctx = test_ctx();
let result = resolve_tool_paths(&args, &ctx).expect("no explicit path → default");
assert_eq!(result.roots, vec!["."]);
assert!(!result.is_multi);
}
#[test]
fn uses_resolved_path_when_present() {
let args = Map::new();
let mut ctx = test_ctx();
ctx.resolved_paths
.insert("path".to_string(), "/resolved/dir".to_string());
let result = resolve_tool_paths(&args, &ctx).expect("resolved path");
assert_eq!(result.roots, vec!["/resolved/dir"]);
assert!(!result.is_multi);
}
#[test]
fn empty_paths_array_falls_back() {
let mut args = Map::new();
args.insert("paths".to_string(), json!([]));
let mut ctx = test_ctx();
ctx.resolved_paths
.insert("path".to_string(), "/fallback".to_string());
let result = resolve_tool_paths(&args, &ctx).expect("empty paths → fallback");
assert_eq!(result.roots, vec!["/fallback"]);
assert!(!result.is_multi);
}
// #401: an explicit `path` the dispatcher could not resolve (out of jail,
// secret-screened, non-existent) must surface the error — NOT silently
// fall back to the project root and return an unrelated tree.
#[test]
fn explicit_unresolvable_path_errors_instead_of_root_fallback() {
let mut args = Map::new();
args.insert(
"path".to_string(),
json!("/home/jules/.claude/skills/mpm-config"),
);
let mut ctx = test_ctx();
// Dispatcher could not resolve it → recorded in path_errors, absent
// from resolved_paths (exactly what the daemon does for out-of-jail).
ctx.path_errors.insert(
"path".to_string(),
"path escapes project root: /home/jules/.claude/skills/mpm-config \
(root: /test/project)"
.to_string(),
);
let err = resolve_tool_paths(&args, &ctx)
.expect_err("out-of-jail explicit path must be an error");
assert!(
err.contains("escapes project root"),
"error must explain the path is out of scope: {err}"
);
}
// #401: an explicit `paths` array where nothing resolves must error too.
//
// The candidate is an absolute path that cannot exist; its only existing
// ancestor is the filesystem root `/`, which is never inside the project
// root or any allow-listed directory. PathJail therefore rejects it
// deterministically on every platform and regardless of allow-list env
// state another test may have left behind. (An earlier version used sibling
// temp dirs and flaked under `--test-threads=1`: a prior test had
// allow-listed the temp directory via `LEAN_CTX_*` env vars, so the
// out-of-jail sibling was accepted and the expected error never fired.)
//
// Gated on a live jail: `--all-features` (used by CI) enables `no-jail`,
// which compiles PathJail out so every path resolves — there is no
// out-of-scope path to reject. The same gate guards the PathJail unit
// tests in `core::pathjail`.
#[cfg(not(feature = "no-jail"))]
#[test]
fn explicit_unresolvable_paths_array_errors() {
let ctx = test_ctx();
let mut args = Map::new();
args.insert(
"paths".to_string(),
json!(["/lean-ctx-nonexistent-path/never/here"]),
);
let err = resolve_tool_paths(&args, &ctx)
.expect_err("a path outside the project root must be an error");
assert!(
err.contains("none of the requested paths"),
"error must report the unresolved paths: {err}"
);
}
}