-
Notifications
You must be signed in to change notification settings - Fork 42
Fix repository import checkout paths and add selective subfolder merging #12883
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
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>
pelikhan
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
-
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>
|
📰 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