Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 31, 2026

actions/checkout@v6's path parameter requires relative paths, but the compiler was generating absolute paths (/tmp/gh-aw/repo-imports/...), causing checkout failures for workflows with repository imports.

Changes

  • Compiler: Generate relative checkout paths (.github/aw/imports/... instead of /tmp/gh-aw/repo-imports/...)
  • Compiler: Removed if: ${{ always() }} condition from the merge remote .github folder step
  • Merge script: Resolve relative paths at runtime using path.join(workspace, relativePath)
  • Merge script: Filter copied files to only include specific subfolders: agents/, skills/, prompts/, instructions/, plugins/
  • Tests: Update assertions to expect relative paths

Before/After

# Before (fails)
- uses: actions/checkout@v6
  with:
    path: /tmp/gh-aw/repo-imports/github-repo-main

# After (works)
- uses: actions/checkout@v6
  with:
    path: .github/aw/imports/github-repo-main
# Before (merge step with always condition)
- name: Merge remote .github folder
  if: ${{ always() }}
  uses: actions/github-script@...

# After (merge step without condition)
- name: Merge remote .github folder
  uses: actions/github-script@...

The merge script constructs the absolute path at runtime: $GITHUB_WORKSPACE/.github/aw/imports/github-repo-main.

Repository imports now checkout to .github/aw/imports/ which aligns with the existing .github/aw/ directory structure used for workflow-related files. The selective subfolder merging prevents unnecessary files (like workflows/) from being copied from imported repositories, keeping workspaces clean and organized. The removal of if: always() ensures the merge step follows standard GitHub Actions conditional execution patterns.

Original prompt

Reference: https://github.com/githubnext/gh-aw/actions/runs/21540699673/job/62074653233#step:5:1

Fix checkout location. See logs.


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


Changeset

  • Type: patch
  • Description: Added the github-deep-research-agent import to smoke-claude and limited the merge script to only copy agents/skills/prompts/instructions/plugins subfolders when importing agents

Ahoy! This treasure was crafted by 🏴‍☠️ Changeset Generator


Changeset

  • Type: patch
  • Description: Document the two-file agentic workflow structure and update compiler safe-output dispatch_workflow handling to validate referenced workflows and build dispatch tools from compiled/yml files.

Ahoy! This treasure was crafted by 🏴‍☠️ Changeset Generator

- Changed checkout path from absolute `/tmp/gh-aw/repo-imports/...` to relative `tmp/gh-aw/repo-imports/...`
- Updated merge script to resolve relative path using GITHUB_WORKSPACE
- Updated tests to expect relative paths
- Recompiled all workflows with new path format

This fixes the issue where actions/checkout@v6 was failing because it requires relative paths, not absolute paths.

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix checkout location based on logs Fix repository import checkout paths for actions/checkout@v6 compatibility Jan 31, 2026
Copilot AI requested a review from pelikhan January 31, 2026 07:35
@github-actions
Copy link
Contributor

🔍 PR Triage Results

Category: bug | Risk: high | Priority: 65/100

Scores Breakdown

  • Impact: 40/50 - Bug with high risk level
  • Urgency: 10/30 - Recent PR (0 days), CI pending
  • Quality: 15/20 - CI pending, Draft status

📋 Recommended Action: Batch Review

This PR is recommended for batch review with similar PRs.

📦 Batch Processing

This PR is part of batch-bug-001 with 1 other PR(s): #12664

Consider reviewing these PRs together for consistency and efficiency.


Triaged by PR Triage Agent on 2026-01-31 12:17 UTC
See full report: PR Triage Discussion

AI generated by PR Triage Agent

@pelikhan
Copy link
Contributor

@copilot use .github/aw/imports/... as the folder to checkout

Changed checkout path from tmp/gh-aw/repo-imports/ to .github/aw/imports/ as requested by @pelikhan. This aligns with the existing .github/aw/ directory structure used for workflow-related files.

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

Copilot AI commented Jan 31, 2026

@copilot use .github/aw/imports/... as the folder to checkout

Done. Changed to .github/aw/imports/ in commit 16ebe2f.

Copy link
Contributor

@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

  • add github copilot deep research agent import to smoke-Claude

  • update the copy the following subfolders only: agents, skills, prompts, instructions, plugins

- Added github/github-deep-research-agent@main import to smoke-claude workflow
- Modified merge script to only copy specific subfolders: agents, skills, prompts, instructions, plugins
- This prevents copying unnecessary files like workflows/ from imported repositories

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

Copilot AI commented Jan 31, 2026

@copilot

  • add github copilot deep research agent import to smoke-Claude ...

Done in commit edd50aa:

  1. Added github/github-deep-research-agent@main import to smoke-claude.md
  2. Modified merge script to only copy files from: agents, skills, prompts, instructions, plugins subfolders

Copilot AI changed the title Fix repository import checkout paths for actions/checkout@v6 compatibility Fix repository import checkout paths and add selective subfolder merging Jan 31, 2026
Copilot AI requested a review from pelikhan January 31, 2026 13:49
@pelikhan pelikhan added the smoke label Jan 31, 2026
@github-actions
Copy link
Contributor

📰 BREAKING: Smoke Copilot is now investigating this pull request. Sources say the story is developing...

@github-actions
Copy link
Contributor

github-actions bot commented Jan 31, 2026

✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟

@github-actions
Copy link
Contributor

github-actions bot commented Jan 31, 2026

🎉 Yo ho ho! Changeset Generator found the treasure and completed successfully! ⚓💰

@github-actions
Copy link
Contributor

github-actions bot commented Jan 31, 2026

💫 TO BE CONTINUED... Smoke Claude failed! Our hero faces unexpected challenges...

@pelikhan
Copy link
Contributor

@copilot

  • do not generate if: always expression for the merge step.
  • add import of GitHub deep research agent to smoke copilot

Copilot AI changed the title Fix repository import checkout paths and add selective subfolder merging Fix repository import checkout paths, add selective subfolder merging, and enhance smoke test workflows Jan 31, 2026
@pelikhan
Copy link
Contributor

@copilot remove GitHub/github deep research from smoke Claude, smoke copilot and scout

…ilot, and scout

- Removed github/github-deep-research-agent@main from smoke-claude.md
- Removed github/github-deep-research-agent@main from smoke-copilot.md
- Removed github/github-deep-research-agent@main from scout.md
- Recompiled all three workflows without repository import steps

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

Copilot AI commented Jan 31, 2026

@copilot remove GitHub/github deep research from smoke Claude, smoke copilot and scout

Done in commit 6a9c9e2. Removed github/github-deep-research-agent@main import from smoke-claude.md, smoke-copilot.md, and scout.md.

Copilot AI changed the title Fix repository import checkout paths, add selective subfolder merging, and enhance smoke test workflows Fix repository import checkout paths and add selective subfolder merging Jan 31, 2026
@pelikhan pelikhan added the smoke label Jan 31, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Jan 31, 2026

🎬 THE ENDSmoke Claude MISSION: ACCOMPLISHED! The hero saves the day! ✨

@github-actions
Copy link
Contributor

github-actions bot commented Jan 31, 2026

🎉 Yo ho ho! Changeset Generator found the treasure and completed successfully! ⚓💰

@github-actions
Copy link
Contributor

📰 BREAKING: Smoke Copilot is now investigating this pull request. Sources say the story is developing...

@github-actions
Copy link
Contributor

github-actions bot commented Jan 31, 2026

✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟

@github-actions
Copy link
Contributor

Agent Container Tool Check

Tool Status Version
bash 5.2.21
sh available
git 2.52.0
jq 1.7
yq 4.50.1
curl 8.5.0
gh 2.86.0
node 24.13.0
python3 3.13.11
go 1.24.12
java not found
dotnet not found

Result: 10/12 tools available ⚠️

Missing tools: java, dotnet

AI generated by Agent Container Smoke Test

@github-actions
Copy link
Contributor

Smoke Test Results

Status: ✅ PASS
Run: §21545715892

@pelikhan

AI generated by Smoke Copilot

@github-actions
Copy link
Contributor

📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤

@github-actions
Copy link
Contributor

Smoke Test Results ✅

PRs Reviewed:

Test Results:

  • ✅ GitHub MCP
  • ✅ Safe Inputs GH CLI
  • ✅ Serena MCP
  • ✅ Make Build
  • ✅ Playwright
  • ✅ Tavily Web Search
  • ✅ File Writing
  • ✅ Bash Tool
  • ✅ Discussion Interaction

Overall: PASS

AI generated by Smoke Claude

@github-actions
Copy link
Contributor

Smoke test results
PR titles: chore: refine default campaign orchestrator rules | Remove noop from handler registry and clarify skip reason logging
Merged PR titles: Remove two-file structure from workflow documentation | [docs] Consolidate developer specifications - Remove marketing language
1 GitHub MCP merged PRs ✅; 2 safeinputs-gh PR list ✅; 3 Serena activate ✅
4 Playwright GitHub title ✅; 5 Tavily search ✅; 6 File write ✅
7 Bash cat ✅; 8 Discussion comment ✅; 9 Build ✅
Overall: PASS

AI generated by Smoke Codex

@pelikhan
Copy link
Contributor

Copy link
Contributor Author

Copilot AI commented Jan 31, 2026

@copilot fix tests Reference: https://github.com/githubnext/gh-aw/actions/runs/21545786433/job/62087458233#step:6:1

I investigated the test failure. The CI logs show the test package took 20.001 seconds with no specific test failure - all individual tests passed. This appears to be a test framework timeout rather than a failure caused by my changes.

All repository import tests pass locally:

  • TestRepositoryImportCheckout ✓
  • TestMultipleRepositoryImportCheckouts ✓
  • TestRefSanitization ✓

The full unit test suite for pkg/workflow also passes locally in 16.2 seconds.

This looks like a transient CI issue or resource constraint. Would you like me to rerun the CI, or is there a specific test you'd like me to investigate?

@pelikhan pelikhan marked this pull request as ready for review January 31, 2026 14:58
@pelikhan pelikhan merged commit d9258ee into main Jan 31, 2026
180 of 186 checks passed
@pelikhan pelikhan deleted the copilot/fix-checkout-location branch January 31, 2026 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants