Closed
Conversation
Automated sync from stranske/Workflows Template hash: 66aa94ed5aa4 Changes synced from sync-manifest.yml
Contributor
Contributor
🤖 Keepalive Loop StatusPR #136 | Agent: Codex | Iteration 0/5 Current State
🔍 Failure Classification| Error type | infrastructure | |
Contributor
There was a problem hiding this comment.
Pull request overview
Syncs workflow-template-related utilities and configuration from stranske/Workflows, updating LLM dependency pins, adding an agent registry + helper, and applying small formatting/cleanup changes in supporting scripts/docs.
Changes:
- Bump pinned LLM workflow dependencies in
tools/requirements-llm.txt. - Add agent registry source-of-truth (
.github/agents/registry.yml) and a Node helper to load/resolve it (.github/scripts/agent_registry.js). - Minor cleanups/format-only tweaks in sync/langchain scripts and a documentation update in
CLAUDE.md.
Reviewed changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| tools/requirements-llm.txt | Updates strict pins used by workflow LLM steps for reproducible installs. |
| scripts/sync_dev_dependencies.py | Removes a Black-specific drift message and slightly restructures check/apply output flow. |
| scripts/langchain/progress_reviewer.py | Splits long f-strings via implicit concatenation (formatting-only). |
| scripts/langchain/pr_verifier.py | Removes a stray whitespace-only line in imports. |
| scripts/langchain/issue_optimizer.py | Removes a stray whitespace-only line in imports. |
| CLAUDE.md | Removes the “CI Test Policy” section under Common Issues (doc content reduction). |
| .github/scripts/agent_registry.js | Adds a lightweight registry YAML loader + label-to-agent resolution helpers. |
| .github/agents/registry.yml | Introduces registry config for agents (default agent, workflow mapping, capabilities). |
Comments suppressed due to low confidence (1)
CLAUDE.md:56
- The CI Test Policy section was removed, but it still matches the current workflows (Gate uses coverage: false + ignores integration dirs +
pytest_markers: "not release"; main CI enforces coverage). Removing this guidance makes it harder to understand why PR Gate and main CI behave differently. Consider restoring this section (or relocating it to a repo docs file) to preserve the repo-specific CI policy explanation.
## Common Issues
### Workflow fails with "workflow file issue"
- A reusable workflow is being called that doesn't exist
- Check Workflows repo has the required `reusable-*.yml` file
- Consumer workflows call INTO Workflows repo, not local files
Contributor
|
Autofix updated these files:
|
Owner
Author
|
Superseded by newer sync PR #143 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Sync Summary
Files Updated
Files Skipped
Review Checklist
Source: stranske/Workflows
Manifest:
.github/sync-manifest.yml