chore: sync workflow templates - #126
Conversation
Automated sync from stranske/Workflows Template hash: 7aa47830e06c Changes synced from sync-manifest.yml
stranske
added
sync
GitHub Actions
Inactive
GitHub Actions
Inactive
🤖 Keepalive Loop StatusPR #126 | Agent: Codex | Iteration 0/5 Current State
🔍 Failure Classification| Error type | infrastructure | |
|
Status | ✅ no new diagnostics |
|
Autofix updated these files:
|
There was a problem hiding this comment.
Pull request overview
This PR syncs workflow templates from the stranske/Workflows repository, updating the agents-issue-intake.yml workflow with improved formatting, debugging capabilities, and simplified secrets management.
Key Changes:
- Enhanced comments for better clarity on the two operational modes (agent_bridge and chatgpt_sync)
- Added debug statements for troubleshooting workflow execution
- Refactored inline regex pattern for better readability
- Simplified secrets passing using
secrets: inheritinstead of explicit secret parameters - Added explicit permissions block for the sync job
| echo "DEBUG: event_name=${{ github.event_name }}" | ||
| echo "DEBUG: inputs.mode=${{ inputs.mode }}" |
There was a problem hiding this comment.
Debug statements should not be left in production workflow files. These echo statements will pollute the workflow logs on every run. Consider removing them or protecting them behind a debug flag if debugging is needed.
| mode="agent_bridge" | ||
| fi | ||
|
|
||
| echo "DEBUG: final mode=${mode}" |
There was a problem hiding this comment.
Debug statement should not be left in production workflow files. This echo statement will pollute the workflow logs on every run. Consider removing it or protecting it behind a debug flag if debugging is needed.
| secrets: | ||
| service_bot_pat: ${{ secrets.SERVICE_BOT_PAT }} | ||
| owner_pr_pat: ${{ secrets.OWNER_PR_PAT }} | ||
| secrets: inherit |
There was a problem hiding this comment.
The documentation at the top of the file states that SERVICE_BOT_PAT and OWNER_PR_PAT are required secrets, but the sync job now uses secrets: inherit instead of explicitly passing these secrets. While secrets: inherit will pass all available secrets to the reusable workflow, the documentation should be updated to clarify that these secrets are passed via inheritance for the chatgpt_sync mode, or the documentation should mention that secrets: inherit is used to ensure consistency.
Sync Summary
Files Updated
Files Skipped
Review Checklist
Source: stranske/Workflows
Manifest:
.github/sync-manifest.yml