Conversation
Automated sync from stranske/Workflows Template hash: fc69344fa9aa Changes synced from sync-manifest.yml
🤖 Keepalive Loop StatusPR #753 | Agent: Codex | Iteration 0/5 Current State
🔍 Failure Classification| Error type | infrastructure | |
Keepalive Work Log (click to expand)
|
There was a problem hiding this comment.
Pull request overview
Syncs workflow-template changes from the central Workflows repo, refining how the verifier identifies “follow-up” iterations and how the PR verification prompt instructs the LLM to scope its evaluation for chained/follow-up PRs.
Changes:
- Expanded follow-up PR evaluation guidance in
scripts/langchain/pr_verifier.py(prompt addendum for chained iterations). - Broadened chain-depth detection heuristics in
.github/scripts/agents_verifier_context.jsbeyond thefollow-uplabel to also consider title/body signals.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
scripts/langchain/pr_verifier.py |
Adds additional chain-depth prompt guidance to keep follow-up evaluations focused on the targeted verifier concerns. |
.github/scripts/agents_verifier_context.js |
Enhances follow-up chain-depth detection by checking label/title/body signals on linked issues. |
You can also share your feedback on Copilot code review. Take the survey.
| if (issue.labels && issue.labels.some((l) => l.name === 'follow-up')) { | ||
| const hasFollowupLabel = issue.labels && issue.labels.some((l) => l.name === 'follow-up'); | ||
| const hasFollowupTitle = /follow-?up/i.test(String(issue.title || '')); | ||
| const hasSourceRef = /source\s+(issue|pr)\s*:\s*#\d+/i.test(String(issue.body || '')); |
There was a problem hiding this comment.
hasSourceRef is unlikely to ever match the repo’s existing “Source” markers (e.g. > **Source:** Issue #384), since the regex requires the literal pattern source issue: or source pr:. As a result, this new follow-up detection signal will be ineffective. Consider updating the regex to also match the current **Source:** (Issue|PR) #<n> formatting (and any other formats you expect).
| const hasSourceRef = /source\s+(issue|pr)\s*:\s*#\d+/i.test(String(issue.body || '')); | |
| const hasSourceRef = | |
| /(?:source\s+(issue|pr)\s*:\s*#\d+|\*\*source:\*\*\s*(issue|pr)\s*#\d+)/i.test( | |
| String(issue.body || ''), | |
| ); |
Sync Summary
Files Updated
Files Skipped
Review Checklist
Source: stranske/Workflows
Manifest:
.github/sync-manifest.yml