Skip to content

Rename workflow steps for clarity: "Print Safe Outputs" and "Upload Safe Outputs"#1034

Merged
pelikhan merged 4 commits intomainfrom
copilot/fix-9873c38e-859e-46b2-974f-15159972d156
Sep 26, 2025
Merged

Rename workflow steps for clarity: "Print Safe Outputs" and "Upload Safe Outputs"#1034
pelikhan merged 4 commits intomainfrom
copilot/fix-9873c38e-859e-46b2-974f-15159972d156

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Sep 26, 2025

This PR renames two workflow step names to better reflect their purpose in the safe outputs system:

  • "Print Agent output""Print Safe Outputs"
  • "Upload agentic output file""Upload Safe Outputs"

Background

The current step names were inconsistent and didn't clearly communicate their role in the safe outputs workflow system. The new names provide better clarity about what these steps actually do:

  • "Print Safe Outputs" clearly indicates this step displays the safe outputs (JSONL format) in the GitHub Actions step summary
  • "Upload Safe Outputs" clearly indicates this step uploads the safe outputs file as an artifact

Changes Made

Source Code

  • pkg/workflow/compiler.go: Updated the two lines that generate these step names in the workflow compilation process
  • pkg/workflow/agentic_output_test.go: Updated test assertions and comments to match the new step names

Generated Files

  • 28 workflow files: All .lock.yml files in .github/workflows/ and pkg/cli/workflows/ have been regenerated with the new step names

Example

Before:

- name: Print Agent output
  run: |
    echo "## Safe Outputs (JSONL)" >> $GITHUB_STEP_SUMMARY
    # ...

- name: Upload agentic output file
  uses: actions/upload-artifact@v4
  with:
    name: safe_output.jsonl
    # ...

After:

- name: Print Safe Outputs
  run: |
    echo "## Safe Outputs (JSONL)" >> $GITHUB_STEP_SUMMARY
    # ...

- name: Upload Safe Outputs
  uses: actions/upload-artifact@v4
  with:
    name: safe_output.jsonl
    # ...

Testing

  • All unit tests pass
  • Specific tests for agentic output functionality verified
  • Workflow compilation confirmed working with make recompile
  • CLI functionality remains intact

This is a minimal change that improves clarity without affecting any functionality.

Original prompt

rename steps "Print Agent Output" to "Print Safe Outputs", and "Upload agentic output file" to "Upload Safe Outputs"


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@pelikhan
Copy link
Copy Markdown
Collaborator

@copilot Rename "Ingest agent output" to "Ingest Safe Outputs"

Copilot AI and others added 3 commits September 26, 2025 11:13
… Outputs

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] rename steps "Print Agent Output" to "Print Safe Outputs", and "Upload agentic output file" to "Upload Safe Outputs" Rename workflow steps for clarity: "Print Safe Outputs" and "Upload Safe Outputs" Sep 26, 2025
Copilot AI requested a review from pelikhan September 26, 2025 11:19
@pelikhan pelikhan marked this pull request as ready for review September 26, 2025 11:19
@pelikhan pelikhan enabled auto-merge (squash) September 26, 2025 11:19
@pelikhan pelikhan disabled auto-merge September 26, 2025 11:19
@pelikhan pelikhan merged commit 45b8d18 into main Sep 26, 2025
13 of 15 checks passed
@pelikhan pelikhan deleted the copilot/fix-9873c38e-859e-46b2-974f-15159972d156 branch September 26, 2025 11:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants