Skip to content

Fix missing log directory for copilot CLI execution#5693

Merged
pelikhan merged 3 commits into
mainfrom
copilot/investigate-review-failure
Dec 6, 2025
Merged

Fix missing log directory for copilot CLI execution#5693
pelikhan merged 3 commits into
mainfrom
copilot/investigate-review-failure

Conversation

Copilot AI commented Dec 6, 2025

Copy link
Copy Markdown
Contributor

The validate_errors script expects logs at /tmp/gh-aw/sandbox/agent/logs/ but the directory wasn't being created before copilot CLI execution with --log-dir, causing validation to silently skip.

Changes

  • pkg/workflow/sh/create_gh_aw_tmp_dir.sh: Added mkdir -p /tmp/gh-aw/sandbox/agent/logs to create the full log directory structure
  • All workflow .lock.yml files: Recompiled to propagate the embedded shell script change

The copilot CLI was already configured to write logs to /tmp/gh-aw/sandbox/agent/logs/, but only /tmp/gh-aw/agent was being created, resulting in:

Log path not found: /tmp/gh-aw/sandbox/agent/logs/
No logs to validate - skipping error validation
Original prompt

Review failure and investigate why it can be failing. Review shell script for hints.

Reference: https://github.com/githubnext/gh-aw/actions/runs/19994130753/job/57339143235#step:15:1


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI and others added 2 commits December 6, 2025 21:03
…tion

The validate_errors script was failing because it expected logs at
/tmp/gh-aw/sandbox/agent/logs/ but this directory wasn't being created
before the copilot CLI execution. Updated create_gh_aw_tmp_dir.sh to
create this directory and recompiled all workflows.

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Review failure and investigate potential causes Fix missing log directory for copilot CLI execution Dec 6, 2025
Copilot AI requested a review from pelikhan December 6, 2025 21:15
@pelikhan pelikhan marked this pull request as ready for review December 6, 2025 21:19
@pelikhan pelikhan merged commit b687d5c into main Dec 6, 2025
5 checks passed
@pelikhan pelikhan deleted the copilot/investigate-review-failure branch December 6, 2025 21: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