Merge pull request #4 from YspCoder/codex/conduct-robustness-check-on-code

fix: remove duplicate allowlist initialization in shell exec tool

pkg/tools/shell.go

@@ -39,11 +39,6 @@ func NewExecTool(cfg config.ShellConfig, workspace string) *ExecTool {
 		allowPatterns = append(allowPatterns, regexp.MustCompile(`(?i)\b`+regexp.QuoteMeta(p)+`\b`))
 	}
 
-	allowPatterns := make([]*regexp.Regexp, 0, len(cfg.AllowedCmds))
-	for _, p := range cfg.AllowedCmds {
-		allowPatterns = append(allowPatterns, regexp.MustCompile(`\b`+regexp.QuoteMeta(p)+`\b`))
-	}
-
 	return &ExecTool{
 		workingDir:          workspace,
 		timeout:             cfg.Timeout,