harness: anchor 'update allow-list' reminder on both #[tool_router] impls (argus nit on #511)
This commit is contained in:
parent
e02b279785
commit
c62c73e546
1 changed files with 12 additions and 0 deletions
|
|
@ -467,6 +467,12 @@ impl AgentServer {
|
|||
}
|
||||
}
|
||||
|
||||
// IMPORTANT: when adding a new `#[tool]` fn to this impl, also add
|
||||
// its name to `allowed_mcp_tools(Flavor::Agent)` below. Claude
|
||||
// Code's permission gate refuses uninlisted MCP tools in
|
||||
// non-interactive `--print` mode with "permissions not granted yet"
|
||||
// — same failure mode #511 cleaned up on the manager side. Keep the
|
||||
// two lists in lockstep.
|
||||
#[tool_router]
|
||||
impl AgentServer {
|
||||
#[tool(
|
||||
|
|
@ -1058,6 +1064,12 @@ impl ManagerServer {
|
|||
}
|
||||
}
|
||||
|
||||
// IMPORTANT: when adding a new `#[tool]` fn to this impl, also add
|
||||
// its name to `allowed_mcp_tools(Flavor::Manager)` below. Claude
|
||||
// Code's permission gate refuses uninlisted MCP tools in
|
||||
// non-interactive `--print` mode with "permissions not granted yet"
|
||||
// — exactly the failure mode PR #511 cleaned up. Keep the two lists
|
||||
// in lockstep.
|
||||
#[tool_router]
|
||||
impl ManagerServer {
|
||||
#[tool(
|
||||
|
|
|
|||
Loading…
Reference in a new issue