Skip to content

Rename agent logs to agent-stdio with simpler naming#1038

Merged
pelikhan merged 6 commits intomainfrom
copilot/fix-a407a993-946c-4cfc-9eba-121584f6aa3b
Sep 26, 2025
Merged

Rename agent logs to agent-stdio with simpler naming#1038
pelikhan merged 6 commits intomainfrom
copilot/fix-a407a993-946c-4cfc-9eba-121584f6aa3b

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Sep 26, 2025

This PR simplifies the agent log file naming convention and updates the upload step name for better clarity and consistency.

Changes Made

Step Name Update:

  • Changed "Upload agent logs" to "Upload Agent Stdio" to better reflect the purpose of capturing standard input/output

Simplified File Naming:

  • Replaced dynamic workflow-name-based log file naming with a consistent fixed pattern
  • Changed from generateSafeFileName(data.Name) to hardcoded "agent-stdio"
  • Set file path to fixed /tmp/agent-stdio.log instead of /tmp/{workflow-name}.log
  • Updated artifact name from {workflow-name}.log to agent-stdio.log

Workflow Recompilation:

  • Recompiled all 41 existing workflow files to use the new step name consistently across the repository

Before vs After

Before:

- name: Upload agent logs
  uses: actions/upload-artifact@v4
  with:
    name: my-workflow-name.log
    path: /tmp/my-workflow-name.log

After:

- name: Upload Agent Stdio
  uses: actions/upload-artifact@v4
  with:
    name: agent-stdio.log
    path: /tmp/agent-stdio.log

Benefits

  • Consistency: All workflows now use the same predictable log file name
  • Simplicity: Removes the complexity of generating safe filenames from workflow names
  • Clarity: The step name "Upload Agent Stdio" clearly indicates it captures standard input/output from agent execution
  • Unified Terminology: Both step name and file naming consistently use "agent-stdio" terminology

Testing

  • All existing unit tests pass
  • Updated tests to expect the new file naming convention and step name
  • Verified workflow compilation produces correct output with sample workflows
  • Recompiled all 41 existing workflow files to use new step name
  • No breaking changes to existing functionality
Original prompt

in compiler.go

  • rename "Upload agent logs" to "Upload Agent Output"
  • simplify the creation of logFile, set to "agent-output.log"
  • remove logic to generate logFile name from workflow name
  • set full file path to "/tmp/agent-output.log"

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
@pelikhan
Copy link
Copy Markdown
Collaborator

@copilot rename from "agent-output" to "agent-stdio"

Copilot AI changed the title [WIP] in compiler.go - rename "Upload agent logs" to "Upload Agent Output" - simplify the creation of logFile, set to "agent-output.log" - remove logic to generate logFile name from workflow name - set full file path to "/tmp/agent-output.log" Rename agent logs to Agent Output and simplify file naming Sep 26, 2025
Copilot AI requested a review from pelikhan September 26, 2025 12:15
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Sep 26, 2025

@copilot rename from "agent-output" to "agent-stdio"

Done! Renamed "agent-output" to "agent-stdio" in all references. Commit aba4ac3.

@pelikhan pelikhan changed the title Rename agent logs to Agent Output and simplify file naming Rename agent logs to agent-stdio with simpler naming Sep 26, 2025
Copy link
Copy Markdown
Collaborator

@pelikhan pelikhan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot recompile

Comment thread pkg/workflow/compiler.go Outdated
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Sep 26, 2025

@copilot recompile

Done! Recompiled all 41 workflow files with the updated step name. Commit 55247ec.

@pelikhan pelikhan marked this pull request as ready for review September 26, 2025 12:46
@pelikhan pelikhan merged commit f93383e into main Sep 26, 2025
13 of 14 checks passed
@pelikhan pelikhan deleted the copilot/fix-a407a993-946c-4cfc-9eba-121584f6aa3b branch September 26, 2025 12:46
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