style: apply prettier formatting
Co-authored-by: Chris Lloyd <chrislloyd@users.noreply.github.com>
This commit is contained in:
parent
a328bf4b16
commit
e55fe60b4e
@ -101,11 +101,7 @@ export function buildRemoteAgentAllowedToolsString(
|
||||
);
|
||||
|
||||
// Add safe git tools only (read-only operations)
|
||||
baseTools.push(
|
||||
"Bash(git status:*)",
|
||||
"Bash(git diff:*)",
|
||||
"Bash(git log:*)",
|
||||
);
|
||||
baseTools.push("Bash(git status:*)", "Bash(git diff:*)", "Bash(git log:*)");
|
||||
|
||||
// Add GitHub Actions MCP tools if enabled
|
||||
if (includeActionsTools) {
|
||||
|
||||
@ -1248,7 +1248,10 @@ describe("buildRemoteAgentAllowedToolsString", () => {
|
||||
|
||||
test("should never include dangerous git tools regardless of parameters", () => {
|
||||
const dangerousCustomTools = ["Bash(git commit:*)", "Bash(git config:*)"];
|
||||
const result = buildRemoteAgentAllowedToolsString(dangerousCustomTools, true);
|
||||
const result = buildRemoteAgentAllowedToolsString(
|
||||
dangerousCustomTools,
|
||||
true,
|
||||
);
|
||||
|
||||
// The function should still include dangerous tools if explicitly provided in custom tools
|
||||
// This is by design - if someone explicitly adds them, they should be included
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user