Conversation
Automated sync from stranske/Workflows Template hash: 7e563c99409c Changes synced from sync-manifest.yml
🤖 Keepalive Loop StatusPR #139 | Agent: Codex | Iteration 0/5 Current State
🔍 Failure Classification| Error type | infrastructure | |
There was a problem hiding this comment.
Pull request overview
Syncs workflow templates and related automation scripts from stranske/Workflows, primarily to generalize “Codex belt” automation to support registry-driven agent selection (agent labels, branch prefixes, runner workflows), and to update LLM/runtime pins used by workflow steps.
Changes:
- Introduces an agent registry (
.github/agents/registry.yml) and loader/helpers (.github/scripts/agent_registry.js) and wires workflows to useagent_key. - Expands automation workflows (autofix + belt dispatcher/worker/conveyor + auto-pilot) to accept/propagate
agent_keyand derive branch prefixes from the registry. - Updates workflow LLM dependency pins and minor formatting/cleanup in a few Python scripts/docs.
Reviewed changes
Copilot reviewed 16 out of 16 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| tools/requirements-llm.txt | Bumps pinned LangChain-related versions used by workflow LLM steps. |
| scripts/sync_dev_dependencies.py | Removes Black-drift special-casing; minor control-flow tidy. |
| scripts/langchain/progress_reviewer.py | Minor string formatting change (line wrapping). |
| scripts/langchain/pr_verifier.py | Import whitespace cleanup. |
| scripts/langchain/issue_optimizer.py | Import whitespace cleanup. |
| CLAUDE.md | Removes the CI test policy section from the assistant context doc. |
| .github/workflows/autofix.yml | Adds workflow_job trigger and refactors PR-context resolution/helpers for autofix. |
| .github/workflows/agents-autofix-loop.yml | Adds agent registry awareness and gates Codex-only behavior. |
| .github/workflows/agents-auto-pilot.yml | Labels/issues/dispatch now use registry-derived default agent + branch prefix. |
| .github/workflows/agents-71-codex-belt-dispatcher.yml | Adds agent_key input/output and uses registry branch prefix. |
| .github/workflows/agents-72-codex-belt-worker.yml | Propagates agent_key, adjusts concurrency/labels/step-branch naming. |
| .github/workflows/agents-72-codex-belt-worker-dispatch.yml | Adds agent_key workflow_dispatch input and forwards it to the worker. |
| .github/workflows/agents-73-codex-belt-conveyor.yml | Adds agent_key, updates concurrency + branch-prefix inference + re-dispatch inputs. |
| .github/scripts/keepalive_loop.js | Resolves agent type via the registry helper when agent labels are present. |
| .github/scripts/agent_registry.js | Adds minimal registry YAML parsing + agent resolution/config helpers. |
| .github/agents/registry.yml | Adds initial registry config for codex (prefix, runner workflow, capabilities). |
| core.warning(`Could not load agent registry; defaulting branch prefix: ${error.message}`); | ||
| } | ||
|
|
||
| const escapeRegex = (value) => String(value).replace(/[.*+?^${}()|[\[\]\\]/g, '\\$&'); |
There was a problem hiding this comment.
The regex literal used in escapeRegex is missing the closing ] for the character class (/[.*+?^${}()|[\[\]\\]/g), which will throw a JavaScript syntax error and break the conveyor when it runs. Fix the regex to a valid escape-regex pattern (or reuse a known-good escapeRegExp helper) so the branch-prefix matching works.
| const escapeRegex = (value) => String(value).replace(/[.*+?^${}()|[\[\]\\]/g, '\\$&'); | |
| const escapeRegex = (value) => String(value).replace(/[.*+?^${}()|[\]\\]/g, '\\$&'); |
| // Branch prefix validation is performed later after checkout (needs registry). | ||
|
|
There was a problem hiding this comment.
This comment says branch prefix validation happens later after checkout, but there is no subsequent validation of the branch name/prefix in this workflow. Either add the intended validation step (e.g., using the registry-configured prefix) or update/remove this comment to avoid misleading future maintainers.
| // Branch prefix validation is performed later after checkout (needs registry). |
|
Autofix updated these files:
|
|
Superseded by newer sync PR #143 |
Sync Summary
Files Updated
Files Skipped
Review Checklist
Source: stranske/Workflows
Manifest:
.github/sync-manifest.yml