Skip to content
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions .github/workflows/smoke-gemini.lock.yml

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 4 additions & 0 deletions pkg/workflow/gemini_engine.go
Original file line number Diff line number Diff line change
Expand Up @@ -176,6 +176,10 @@ func (e *GeminiEngine) GetExecutionSteps(workflowData *WorkflowData, logFile str
// The conversion script (convert_gateway_config_gemini.sh) writes settings.json
// during the MCP setup step, so no --mcp-config flag is needed here.

// Auto-approve all tool executions (equivalent to Codex's --dangerously-bypass-approvals-and-sandbox)
Comment thread
pelikhan marked this conversation as resolved.
// Without this, Gemini CLI's default approval mode rejects tool calls with "Tool execution denied by policy"
geminiArgs = append(geminiArgs, "--yolo")

// Add headless mode with JSON output
geminiArgs = append(geminiArgs, "--output-format", "json")

Expand Down
1 change: 1 addition & 0 deletions pkg/workflow/gemini_engine_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -151,6 +151,7 @@ func TestGeminiEngineExecution(t *testing.T) {
assert.Contains(t, stepContent, "name: Run Gemini", "Should have correct step name")
assert.Contains(t, stepContent, "id: agentic_execution", "Should have agentic_execution ID")
assert.Contains(t, stepContent, "gemini", "Should invoke gemini command")
assert.Contains(t, stepContent, "--yolo", "Should include --yolo flag for auto-approving tool executions")
Comment thread
pelikhan marked this conversation as resolved.
assert.Contains(t, stepContent, "--output-format json", "Should use JSON output format")
assert.Contains(t, stepContent, `--prompt "$(cat /tmp/gh-aw/aw-prompts/prompt.txt)"`, "Should include prompt argument with correct shell quoting")
assert.Contains(t, stepContent, "/tmp/test.log", "Should include log file")
Expand Down
Loading