chore: sync actions from gh-aw@v0.82.0#176
Merged
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Syncs setup/ scripts and supporting assets from gh-aw@v0.82.0, improving proxy upstream environment derivation, cleanup robustness, and diagnostics/UX around agent failures and workflow dispatch.
Changes:
- Add shared proxy upstream env derivation (
GH_HOST,GITHUB_API_URL,GITHUB_GRAPHQL_URL,GITHUB_COPILOT_BASE_URL) and pass these through to proxy containers. - Harden cleanup of stale AWF chroot-home directories to prevent EACCES / “engine terminated unexpectedly” failures.
- Improve diagnostics and UX: HTTP 400 failure context, safer markdown fencing in log summaries, better tool-result extraction, and richer workflow-dispatch metadata for status comments.
Show a summary per file
| File | Description |
|---|---|
| setup/sh/start_difc_proxy.sh | Source shared proxy env lib, derive upstream env, and pass additional GitHub routing vars into DIFC proxy container. |
| setup/sh/start_cli_proxy.sh | Source shared proxy env lib, derive upstream env, and pass additional GitHub routing vars into CLI proxy container. |
| setup/sh/proxy_env_lib.sh | New shared Bash library to normalize GitHub host and derive upstream API/Copilot URLs for proxies. |
| setup/sh/proxy_env_lib_test.sh | New shell test script covering host normalization and upstream env derivation cases. |
| setup/sh/install_copilot_cli.sh | Pre-install cleanup of stale AWF chroot-home dirs to avoid Copilot CLI cleanup EACCES failures. |
| setup/post.js | Post-step cleanup now also removes stale AWF chroot-home dirs under /tmp. |
| setup/clean.sh | Script-mode cleanup now also removes stale AWF chroot-home dirs under /tmp. |
| setup/md/workflow_install_note.md | Fix summary phrasing/formatting for the install note. |
| setup/md/http_400_response_error.md | New troubleshooting snippet for generic HTTP 400 bad-request failures. |
| setup/md/agent_failure_issue.md | Include HTTP 400 context placeholder in failure issue template. |
| setup/md/agent_failure_comment.md | Include HTTP 400 context placeholder in failure comment template. |
| setup/js/unified_timeline.cjs | Refactor event tallying to aggregate counts via bySource/byKind. |
| setup/js/templatable.cjs | Make boolean templating more robust (trim().toLowerCase() before comparing to "false"). |
| setup/js/start_mcp_gateway.cjs | Add JSON.parse error context (line/column/key) to aid debugging invalid gateway config JSON. |
| setup/js/set_issue_type.cjs | Switch issue-type discovery to repository scope and adjust GraphQL issue-type update payload shape. |
| setup/js/safe_outputs_tools.json | Add x-synonyms for PR review decision event (“action”). |
| setup/js/run_operation_update_upgrade.cjs | Expand upgrade allowlist and stage allowlisted files with git add -A -- <file> to capture deletions. |
| setup/js/route_slash_command.cjs | Dispatch workflow via REST request supporting optional run details; update status comment with dispatched run URL when available. |
| setup/js/push_to_pull_request_branch.cjs | Capture git push stderr and surface a typed error when rejected due to missing workflows scope. |
| setup/js/messages_footer.cjs | Add label-command “run again” hint support and refactor command-hint parsing. |
| setup/js/log_parser_shared.cjs | Improve extraction of structured tool results (arrays/json/object fallbacks) into legacy log entries. |
| setup/js/log_parser_format.cjs | Generate outer markdown fences safely even when content includes nested backtick runs. |
| setup/js/log_parser_bootstrap.cjs | Adjust Claude fallback messaging and downgrade “no entries” to warning when safe outputs exist. |
| setup/js/handle_agent_failure.cjs | Add HTTP 400 failure category/context and use parseBoolTemplatable for GH_AW_FAILURE_REPORT_AS_ISSUE. |
| setup/js/generate_aw_info.cjs | Optionally include parsed features map from GH_AW_INFO_FEATURES in aw_info.json. |
| setup/js/detect_agent_errors.cjs | Detect and output a generic HTTP 400 response error signal. |
| setup/js/copilot_harness.cjs | Detect/classify generic HTTP 400 response failures and adjust retry behavior accordingly. |
| setup/js/chroot_home_cleanup.test.js | New vitest coverage for the chroot-home cleanup behavior in post.js and clean.sh. |
| setup/js/apply_samples.cjs | Improve repo resolution order for PR head ref derivation (incl. configured safe-outputs target repo). |
| setup/js/ai_credits_context.cjs | Refactor AI credits provenance logging into a helper function; simplify boolean return. |
| setup/js/add_workflow_run_comment.cjs | Allow dispatched workflow name/run URL override via aw_context when updating reusable status comments. |
Review details
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 31/31 changed files
- Comments generated: 2
- Review effort level: Low
Comment on lines
+8
to
+10
| # This script exports GH_HOST (and related vars) within the script for use when | ||
| # launching the proxy container, but does NOT write to $GITHUB_ENV and the | ||
| # exports do not persist beyond this script. |
Comment on lines
+345
to
+347
| core.info(`Fetching issue types for repo ${owner}/${repo}`); | ||
| const issueTypes = await fetchIssueTypesForRepo(githubClient, owner, repo); | ||
| core.info(`Found ${issueTypes.length} issue type(s) for repo ${owner}/${repo}`); |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Automated sync of actions from gh-aw at
v0.82.0.