Fimeg/letta-code
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

fix: command analyzer uses first command in pipelines (#438)

Browse Source 
Co-authored-by: Letta <noreply@letta.com>
This commit is contained in:
Charles Packer
2025-12-31 15:56:31 -08:00
committed by GitHub
parent 19ecc2af1a
commit d63e1e01bb
1 changed files with 8 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
src/permissions/analyzer.ts
View File

@@ -349,14 +349,19 @@ function analyzeBashApproval(
}
// Handle complex piped/chained commands (cd /path && git diff | head)
// Strip out cd commands and extract the actual command
// For pipes (|), the FIRST command is the main one
// For && and ;, we skip cd prefixes and use the actual command
if (
command.includes("&&") ||
command.includes("|") ||
command.includes(";")
) {
// Split on these delimiters and analyze each part
const segments = command.split(/\s*(?:&&|\||;)\s*/);
// First, strip everything after the first pipe - the piped-to command is secondary
// e.g., "curl --version | head -1" -> analyze "curl --version"
const beforePipe = (command.split("|")[0] ?? command).trim();
// Now split on && and ; to handle cd prefixes
const segments = beforePipe.split(/\s*(?:&&|;)\s*/);
for (const segment of segments) {
const segmentParts = segment.trim().split(/\s+/);