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
4 changes: 2 additions & 2 deletions .github/workflows/copilot-pr-merged-report.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/daily-choice-test.lock.yml

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

4 changes: 2 additions & 2 deletions .github/workflows/daily-cli-performance.lock.yml

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

4 changes: 2 additions & 2 deletions .github/workflows/daily-performance-summary.lock.yml

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

4 changes: 2 additions & 2 deletions .github/workflows/daily-regulatory.lock.yml

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

4 changes: 2 additions & 2 deletions .github/workflows/go-logger.lock.yml

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

4 changes: 2 additions & 2 deletions .github/workflows/mcp-inspector.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/notion-issue-summary.lock.yml

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

6 changes: 3 additions & 3 deletions .github/workflows/smoke-claude.lock.yml

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

4 changes: 2 additions & 2 deletions .github/workflows/smoke-codex.lock.yml

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

6 changes: 3 additions & 3 deletions .github/workflows/smoke-copilot-arm.lock.yml

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

6 changes: 3 additions & 3 deletions .github/workflows/smoke-copilot.lock.yml

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

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.

2 changes: 1 addition & 1 deletion pkg/workflow/compiler_safe_outputs_job.go
Original file line number Diff line number Diff line change
Expand Up @@ -734,7 +734,7 @@ func buildCustomScriptFilesStep(scripts map[string]*SafeScriptConfig, frontmatte
sort.Strings(scriptNames)

var steps []string
steps = append(steps, " - name: Setup Safe Outputs Custom Scripts\n")
steps = append(steps, " - name: Configure Safe Outputs Custom Scripts\n")
steps = append(steps, " run: |\n")

for _, scriptName := range scriptNames {
Expand Down
4 changes: 2 additions & 2 deletions pkg/workflow/mcp_setup_generator.go
Original file line number Diff line number Diff line change
Expand Up @@ -357,7 +357,7 @@ func (c *Compiler) generateMCPSetup(yaml *strings.Builder, tools map[string]any,
// For stdio mode, we only write the files but don't start the HTTP server
if IsMCPScriptsEnabled(workflowData.MCPScripts, workflowData) {
// Step 1: Write config files (JavaScript files are now copied by actions/setup)
yaml.WriteString(" - name: Setup MCP Scripts Config\n")
yaml.WriteString(" - name: Write MCP Scripts Config\n")
yaml.WriteString(" run: |\n")
yaml.WriteString(" mkdir -p ${RUNNER_TEMP}/gh-aw/mcp-scripts/logs\n")

Expand All @@ -382,7 +382,7 @@ func (c *Compiler) generateMCPSetup(yaml *strings.Builder, tools map[string]any,
yaml.WriteString(" \n")

// Step 2: Generate tool files (js/py/sh)
yaml.WriteString(" - name: Setup MCP Scripts Tool Files\n")
yaml.WriteString(" - name: Write MCP Scripts Tool Files\n")
yaml.WriteString(" run: |\n")

// Generate individual tool files (sorted by name for stable code generation)
Expand Down
4 changes: 2 additions & 2 deletions pkg/workflow/safe_jobs.go
Original file line number Diff line number Diff line change
Expand Up @@ -241,7 +241,7 @@ func (c *Compiler) buildSafeJobs(data *WorkflowData, threatDetectionEnabled bool
agentOutputArtifactFilename := "${RUNNER_TEMP}/gh-aw/safe-jobs/" + constants.AgentOutputFilename

// Add environment variables step with GH_AW_AGENT_OUTPUT and job-specific env vars
steps = append(steps, " - name: Setup Safe Job Environment Variables\n")
steps = append(steps, " - name: Configure Safe Job Environment Variables\n")
steps = append(steps, " id: setup-safe-job-env\n")
steps = append(steps, " run: |\n")
steps = append(steps, " find \"${RUNNER_TEMP}/gh-aw/safe-jobs/\" -type f -print\n")
Expand All @@ -256,7 +256,7 @@ func (c *Compiler) buildSafeJobs(data *WorkflowData, threatDetectionEnabled bool
}

// Add custom steps from the job configuration, injecting env vars from the
// "Setup Safe Job Environment Variables" step output so user steps can access them.
// "Configure Safe Job Environment Variables" step output so user steps can access them.
if len(jobConfig.Steps) > 0 {
// Build the env vars that were set in the setup step so we can inject them.
setupEnvVars := map[string]string{
Expand Down
2 changes: 1 addition & 1 deletion pkg/workflow/safe_scripts_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -265,7 +265,7 @@ func TestBuildCustomScriptFilesStep(t *testing.T) {

fullYAML := strings.Join(steps, "")

assert.Contains(t, fullYAML, "Setup Safe Outputs Custom Scripts", "Should have setup step name")
assert.Contains(t, fullYAML, "Configure Safe Outputs Custom Scripts", "Should have configure step name")
assert.Contains(t, fullYAML, "safe_output_script_my_handler.cjs", "Should reference the output filename")
// Verify heredoc delimiter follows the randomized GH_AW_SAFE_OUTPUT_SCRIPT_MY_HANDLER_<hex>_EOF format
delimRE := regexp.MustCompile(`GH_AW_SAFE_OUTPUT_SCRIPT_MY_HANDLER_[0-9a-f]{16}_EOF`)
Expand Down
Loading