feat: add PR meta and autofix workflows, remove redundant scripts - #47
Conversation
- Add agents-pr-meta.yml for keepalive detection and dispatch - Add autofix.yml for automatic lint/format fixes - Remove .github/scripts/* (now fetched from stranske/Workflows via dual checkout) - Add README.md explaining the dual checkout pattern This enables full agent automation with centralized script maintenance.
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
Pull request overview
This PR implements centralized script maintenance by introducing two new agent automation workflows and removing local script files that are now fetched from the stranske/Workflows repository via a dual checkout pattern.
Key Changes:
- Adds
agents-pr-meta.ymlworkflow for detecting keepalive comments and dispatching agent continuation - Adds
autofix.ymlworkflow for automatic lint and format fixes on pull requests - Removes 9 local JavaScript helper scripts from
.github/scripts/that are now centrally maintained
Reviewed changes
Copilot reviewed 11 out of 11 changed files in this pull request and generated 9 comments.
Show a summary per file
| File | Description |
|---|---|
.github/workflows/agents-pr-meta.yml |
New workflow that delegates PR meta management (keepalive detection) to reusable workflow in centralized Workflows repo |
.github/workflows/autofix.yml |
New workflow that delegates autofix operations (lint/format) to reusable workflow, triggered on PR events and labels |
.github/scripts/keepalive_state.js |
Removed - state management utilities now fetched from central repo |
.github/scripts/keepalive_instruction_template.js |
Removed - instruction templating now fetched from central repo |
.github/scripts/keepalive_contract.js |
Removed - keepalive contract utilities now fetched from central repo |
.github/scripts/issue_scope_parser.js |
Removed - scope/task/acceptance parsing now fetched from central repo |
.github/scripts/issue_pr_locator.js |
Removed - PR locator logic now fetched from central repo |
.github/scripts/issue_context_utils.js |
Removed - context building utilities now fetched from central repo |
.github/scripts/comment-dedupe.js |
Removed - comment deduplication logic now fetched from central repo |
.github/scripts/api-helpers.js |
Removed - API helper utilities now fetched from central repo |
.github/scripts/README.md |
New file explaining the dual checkout pattern for script maintenance |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
731cdf7 to
503a95a
Compare
|
Status | ✅ no new diagnostics |
|
Status | ✅ no new diagnostics |
e350098 to
a858962
Compare
|
Status | ✅ no new diagnostics |
|
Status | ✅ no new diagnostics |
Changes applied: - Simplify concurrency group expression (agents-pr-meta.yml) - Alphabetize permissions for consistency (both workflows) - Add Base64 encoding documentation comment (agents-pr-meta.yml) - Use repository variable for allowed_keepalive_logins (agents-pr-meta.yml) - Add explicit null check for sameRepo (autofix.yml) - Fix literal \n characters in scripts/README.md Not applied (intentional): - @main pinning: First-party Workflows repo, intentional - Remove workflow_dispatch inputs: Useful for manual testing - pull_request_target warning: Necessary for fork autofix Related: stranske/Workflows#65
|
Status | ✅ no new diagnostics |
This enables full agent automation with centralized script maintenance.
Summary
One sentence.
Checklist
workflow docs, CI)
Labels
Add
stage:*,type:*, and size (XS/S/M/L).Add
stage:*,type:*, and size (XS/S/M/L).