Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
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
2 changes: 1 addition & 1 deletion .github/workflows/agentics/shared/job-summary.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ tools:

### Output Report implemented via GitHub Action Job Summary

You will use the Job Summary for GitHub Actions run ${{ github.run_id }} in ${{ github.repository }} to report progess. This means writing to the special file $GITHUB_STEP_SUMMARY. You can write the file using "echo" or the "Write" tool. GITHUB_STEP_SUMMARY is an environment variable set by GitHub Actions which you can use to write the report. You can read this environment variable using the bash command "echo $GITHUB_STEP_SUMMARY".
You will use the Job Summary for GitHub Actions run ${{ github.run_id }} in ${{ github.repository }} to report progess. This means writing to the special file that is stored in the environment variable GITHUB_STEP_SUMMARY. You can write the file using "echo" or the "Write" tool. GITHUB_STEP_SUMMARY is an environment variable set by GitHub Actions which you can use to write the report. You can read this environment variable using the bash command "echo $GITHUB_STEP_SUMMARY".

At the end of the workflow, finalize the job summry with a very, very succinct summary in note form of
- the steps you took
Expand Down
5 changes: 3 additions & 2 deletions .github/workflows/test-codex.lock.yml

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

2 changes: 1 addition & 1 deletion .github/workflows/weekly-research.lock.yml

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

9 changes: 5 additions & 4 deletions pkg/workflow/codex_engine.go
Original file line number Diff line number Diff line change
Expand Up @@ -44,8 +44,8 @@ func (e *CodexEngine) GetInstallationSteps(engineConfig *EngineConfig) []GitHubA
}

func (e *CodexEngine) GetExecutionConfig(workflowName string, logFile string, engineConfig *EngineConfig) ExecutionConfig {
// Use model from engineConfig if available, otherwise default to gpt-4o
model := "gpt-4o"
// Use model from engineConfig if available, otherwise default to o4-mini
model := "o4-mini"
if engineConfig != nil && engineConfig.Model != "" {
model = engineConfig.Model
}
Expand All @@ -59,13 +59,14 @@ mkdir -p /tmp/aw-logs
# Run codex with log capture
codex exec \
-c model=%s \
--full-auto "$INSTRUCTION" 2>&1 | tee /tmp/aw-logs/%s.log`, model, logFile)
--full-auto "$INSTRUCTION" 2>&1 | tee %s`, model, logFile)

return ExecutionConfig{
StepName: "Run Codex",
Command: command,
Environment: map[string]string{
"OPENAI_API_KEY": "${{ secrets.OPENAI_API_KEY }}",
"OPENAI_API_KEY": "${{ secrets.OPENAI_API_KEY }}",
"GITHUB_STEP_SUMMARY": "${{ env.GITHUB_STEP_SUMMARY }}",
},
}
}
Expand Down
2 changes: 1 addition & 1 deletion pkg/workflow/codex_engine_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ func TestCodexEngine(t *testing.T) {
t.Errorf("Expected command to contain 'codex exec', got '%s'", config.Command)
}

if !strings.Contains(config.Command, "test-log.log") {
if !strings.Contains(config.Command, "test-log") {
t.Errorf("Expected command to contain log file name, got '%s'", config.Command)
}

Expand Down
4 changes: 2 additions & 2 deletions pkg/workflow/codex_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -122,8 +122,8 @@ This is a test workflow.
t.Errorf("Expected lock file to contain 'codex exec' command but it didn't.\nContent:\n%s", lockContent)
}
// Check for correct model based on AI setting
if !strings.Contains(lockContent, "model=gpt-4o") {
t.Errorf("Expected lock file to contain 'model=gpt-4o' for codex but it didn't.\nContent:\n%s", lockContent)
if !strings.Contains(lockContent, "model=o4-mini") {
t.Errorf("Expected lock file to contain 'model=o4-mini' for codex but it didn't.\nContent:\n%s", lockContent)
}
if !strings.Contains(lockContent, "OPENAI_API_KEY") {
t.Errorf("Expected lock file to contain 'OPENAI_API_KEY' for codex but it didn't.\nContent:\n%s", lockContent)
Expand Down