chore: sync actions from gh-aw@v0.82.7#183
Merged
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Automated sync of the setup/ GitHub Actions helper scripts from gh-aw@v0.82.7, including new firewall sandbox cleanup/logging utilities and improved diagnostics/robustness in the JavaScript harness/parsers.
Changes:
- Add shell utilities (and tests) to pre-create/reclaim firewall sandbox directories and to emit a firewall log summary after runs.
- Improve repo-memory artifact filtering by validating/compiling
FILE_GLOB_FILTERpatterns once and enhancing scan reporting. - Enhance Claude/Copilot engine diagnostics and resilience (Claude startup diagnostics + bounded startup retries; Copilot SDK cleanup timeouts and shorter post-completion watchdog).
Show a summary per file
| File | Description |
|---|---|
| setup/sh/print_firewall_logs.sh | New script to reclaim firewall sandbox ownership/permissions and print an awf logs summary. |
| setup/sh/print_firewall_logs_test.sh | Adds a bash test harness for print_firewall_logs.sh. |
| setup/sh/create_gh_aw_tmp_dir.sh | Adds preflight cleanup for stale root-owned firewall sandbox dirs and pre-creates firewall subdirs. |
| setup/sh/create_gh_aw_tmp_dir_test.sh | Adds a bash test harness for create_gh_aw_tmp_dir.sh. |
| setup/js/push_repo_memory.cjs | Adds FILE_GLOB_FILTER validation, precompiles patterns, and expands scan logging. |
| setup/js/log_parser_bootstrap.cjs | Adds Claude startup diagnostics when no structured log entries are produced; sets additional outputs. |
| setup/js/glob_pattern_helpers.cjs | Extends glob-to-regex conversion with matchSubfolderRoot option for slashless patterns. |
| setup/js/copilot_sdk_session.cjs | Shortens the post-completion watchdog default and adds time-bounded cleanup operations. |
| setup/js/claude_harness.cjs | Adds bounded startup retries for zero-output startup failures via GH_AW_CLAUDE_STARTUP_RETRIES. |
Review details
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 9/9 changed files
- Comments generated: 4
- Review effort level: Low
Comment on lines
+28
to
+31
| # FIREWALL_DIR is the firewall sandbox root, derived as the parent of AWF_LOGS_DIR. | ||
| # The Go compiler sets AWF_LOGS_DIR="${RUNNER_TEMP}/gh-aw/sandbox/firewall/logs", | ||
| # so FIREWALL_DIR resolves to "${RUNNER_TEMP}/gh-aw/sandbox/firewall". | ||
| FIREWALL_DIR="$(dirname "${AWF_LOGS_DIR}")" |
| core.info(` [skip] ${normalizedRelPath} (${stats.size} bytes) — no pattern matched`); | ||
| filteredOutFiles.push({ path: normalizedRelPath, reason: "no pattern matched" }); | ||
| // Skip this file instead of failing - it may be from a previous run with different patterns | ||
| return; |
Comment on lines
+367
to
+370
| if (filteredOutFiles.length > 0) { | ||
| core.info(`Filtered-out files (${filteredOutFiles.length}):`); | ||
| filteredOutFiles.forEach(f => core.info(` - ${f.path} (${f.reason})`)); | ||
| } |
Comment on lines
+332
to
335
| const matchResults = compiledPatterns.map((pattern, idx) => { | ||
| const matches = pattern.test(normalizedRelPath); | ||
| core.debug(` Pattern ${idx + 1}: "${patternStrs[idx]}" -> ${pattern.source} -> ${matches ? "✓ MATCH" : "✗ NO MATCH"}`); | ||
| core.info(` [test] ${normalizedRelPath} pattern[${idx + 1}] "${patternStrs[idx]}" -> ${matches ? "✓ match" : "✗ no match"}`); | ||
| return matches; |
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.7.