Conversation
Summary
Testing
|
🤖 Augment PR SummarySummary: This PR migrates the repository’s GitHub AI automation from the shared Claude workflow to local Codex-backed workflows. Changes:
Technical Notes: Codex runs with 🤖 Was this summary useful? React with 👍 or 👎 |
|
|
||
| case "$EVENT_NAME" in | ||
| issue_comment) | ||
| target_number="$(jq -r '.issue.number' "$EVENT_PATH")" |
There was a problem hiding this comment.
jq -r '.issue.number' can yield the literal string null if the expected field is missing, which won’t trip the -z "$target_number" guard and can later break gh api .../issues/$TARGET_NUMBER / --argjson targetNumber "$TARGET_NUMBER". Consider treating null as empty (e.g., via // empty) so the workflow reliably skips instead of failing on unexpected payload shapes.
Severity: medium
🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #854 +/- ##
==========================================
- Coverage 14.41% 14.40% -0.01%
==========================================
Files 634 634
Lines 39006 39006
==========================================
- Hits 5621 5619 -2
- Misses 33385 33387 +2 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Summary
@claudeworkflow with an@codexworkflow backed byopenai/codex-action@v1.OPENAI_API_KEYis unavailable.Safety shape
OPENAI_API_KEYinstead ofCLAUDE_CODE_OAUTH_TOKEN.sandbox: read-onlyandsafety-strategy: drop-sudo.github-scriptor shell steps that validate Codex output first.Validation
ruby -ryaml -e 'ARGV.each { |f| YAML.load_file(f); puts "yaml ok #{f}" }' .github/workflows/codex.yml .github/workflows/issue-triage.yml .github/workflows/pr-triage.yml .github/workflows/release_notes.yml/tmp/actionlint-1.7.12/actionlint -color=false .github/workflows/codex.yml .github/workflows/issue-triage.yml .github/workflows/pr-triage.yml .github/workflows/release_notes.ymlgit diff --checkgh/jq.