Conversation
Automated sync from stranske/Workflows Template hash: 69a32b2a4174 Changes synced from sync-manifest.yml
🤖 Keepalive Loop StatusPR #140 | Agent: Codex | Iteration 0/5 Current State
🔍 Failure Classification| Error type | infrastructure | |
There was a problem hiding this comment.
Pull request overview
This PR syncs workflow templates, scripts, and dependencies from the stranske/Workflows repository. The main change introduces a centralized agent registry system that enables multi-agent support across workflow automation, replacing hardcoded 'codex' references with a configurable registry-based approach.
Changes:
- Introduces new agent registry system (
.github/agents/registry.ymland.github/scripts/agent_registry.js) for centralized agent configuration - Updates 11 workflow files to integrate with the agent registry, adding
agent_keyparameters throughout the workflow call chain - Updates langchain-related dependencies to newer versions (langchain 1.2.9→1.2.10, langchain-core 1.2.11→1.2.13, langchain-openai 1.1.7→1.1.9, langchain-anthropic 1.3.2→1.3.3)
- Removes deprecated black drift detection from sync_dev_dependencies.py
- Enhances autofix.yml with early lint failure detection via workflow_job trigger
- Minor formatting and whitespace cleanup in Python scripts
Reviewed changes
Copilot reviewed 19 out of 19 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
.github/agents/registry.yml |
New agent registry configuration defining codex agent with branch prefix, capabilities, and runner workflow |
.github/scripts/agent_registry.js |
New agent registry helper module with YAML parser and agent resolution functions |
tools/requirements-llm.txt |
Updated langchain dependency versions for workflow LLM steps |
scripts/sync_dev_dependencies.py |
Removed black drift detection logic, simplified exit code handling |
scripts/langchain/progress_reviewer.py |
F-string formatting changes (split for line-length compliance) |
scripts/langchain/pr_verifier.py |
Removed blank line between import groups |
scripts/langchain/issue_optimizer.py |
Removed blank line between import groups |
CLAUDE.md |
Removed CI Test Policy documentation section |
.github/workflows/autofix.yml |
Added workflow_job trigger for early lint failure detection, extracted helper functions, agent registry integration |
.github/workflows/agents-verify-to-new-pr.yml |
Integrated agent registry to resolve agent from PR labels when creating follow-up issues |
.github/workflows/agents-verify-to-issue.yml |
Updated documentation text to reference generic agent:* labels instead of hardcoded agent:codex |
.github/workflows/agents-verify-to-issue-v2.yml |
Integrated agent registry, added dynamic agent label resolution for follow-up issues |
.github/workflows/agents-autofix-loop.yml |
Added agent_type resolution via registry, added validation to only support codex agent for autofix loop |
.github/workflows/agents-auto-pilot.yml |
Integrated agent registry to dynamically resolve default agent, pass agent_key to dispatcher |
.github/workflows/agents-73-codex-belt-conveyor.yml |
Added agent_key parameter, dynamic concurrency groups, branch prefix resolution via registry |
.github/workflows/agents-72-codex-belt-worker.yml |
Added agent_key parameter, removed hardcoded branch prefix, dynamic concurrency groups, conditional assignee logic |
.github/workflows/agents-72-codex-belt-worker-dispatch.yml |
Added agent_key input parameter passthrough |
.github/workflows/agents-71-codex-belt-dispatcher.yml |
Added agent_key parameter, dynamic branch prefix resolution, agent-specific label filtering |
.github/scripts/keepalive_loop.js |
Integrated agent registry helper for resolving agent type from PR labels |
| 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 character class [\[\]\\] in the escapeRegex function is problematic. Inside a character class, the first [ after the opening bracket doesn't need to be escaped. The pattern should be /[.*+?^${}()|[\]\\]/g instead of /[.*+?^${}()|[\[\]\\]/g.
While this may work in practice due to how JavaScript handles malformed regex character classes, it's technically incorrect and could lead to unexpected behavior. The correct way to include [, ], and \ in a character class is [\]\\] (no need to escape the opening [ when it's the first character after [).
| const escapeRegex = (value) => String(value).replace(/[.*+?^${}()|[\[\]\\]/g, '\\$&'); | |
| const escapeRegex = (value) => String(value).replace(/[.*+?^${}()|[\]\\]/g, '\\$&'); |
|
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