Fix repository import checkout paths and add selective subfolder merging#12883
Fix repository import checkout paths and add selective subfolder merging#12883
Conversation
- 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>
🔍 PR Triage ResultsCategory: bug | Risk: high | Priority: 65/100 Scores Breakdown
📋 Recommended Action: Batch ReviewThis PR is recommended for batch review with similar PRs. 📦 Batch ProcessingThis 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
|
|
@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>
- 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>
|
📰 BREAKING: Smoke Copilot is now investigating this pull request. Sources say the story is developing... |
|
✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟 |
|
🎉 Yo ho ho! Changeset Generator found the treasure and completed successfully! ⚓💰 |
|
💫 TO BE CONTINUED... Smoke Claude failed! Our hero faces unexpected challenges... |
|
|
@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>
|
🎬 THE END — Smoke Claude MISSION: ACCOMPLISHED! The hero saves the day! ✨ |
|
🎉 Yo ho ho! Changeset Generator found the treasure and completed successfully! ⚓💰 |
|
📰 BREAKING: Smoke Copilot is now investigating this pull request. Sources say the story is developing... |
|
✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟 |
Agent Container Tool Check
Result: 10/12 tools available Missing tools: java, dotnet
|
Smoke Test Results
Status: ✅ PASS
|
|
📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤 |
Smoke Test Results ✅PRs Reviewed:
Test Results:
Overall: PASS ✅
|
|
Smoke test results
|
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:
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? |
actions/checkout@v6'spathparameter requires relative paths, but the compiler was generating absolute paths (/tmp/gh-aw/repo-imports/...), causing checkout failures for workflows with repository imports.Changes
.github/aw/imports/...instead of/tmp/gh-aw/repo-imports/...)if: ${{ always() }}condition from the merge remote .github folder steppath.join(workspace, relativePath)agents/,skills/,prompts/,instructions/,plugins/Before/After
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 (likeworkflows/) from being copied from imported repositories, keeping workspaces clean and organized. The removal ofif: always()ensures the merge step follows standard GitHub Actions conditional execution patterns.Original prompt
✨ 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
Changeset