Conversation
Automated sync from stranske/Workflows Template hash: fc69344fa9aa Changes synced from sync-manifest.yml
stranske
added
sync
🤖 Keepalive Loop StatusPR #321 | 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 verifier utilities to refine how follow-up PR iterations are evaluated and detected, improving the reliability of agent verification guidance and context generation.
Changes:
- Updated the follow-up chain prompt addendum to avoid re-grading already-satisfied upstream scope and to flag “external evidence required” for out-of-band metadata tasks.
- Expanded follow-up iteration detection heuristics to recognize follow-up indicators via linked issue title/body patterns in addition to labels.
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 | Refines verifier guidance for follow-up iterations and out-of-band acceptance criteria evidence. |
| .github/scripts/agents_verifier_context.js | Improves chain depth inference by checking additional follow-up signals on linked issues. |
You can also share your feedback on Copilot code review. Take the survey.
| // Also check for follow-up label on linked issues as a depth-1 indicator | ||
| if (chainDepth === 0) { | ||
| for (const issue of closingIssues) { | ||
| 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 || '')); |
There was a problem hiding this comment.
The inline comment says this block checks only for a follow-up label, but the logic now also uses the issue title and a source issue/pr: #... body reference. Update the comment to reflect all depth-1 indicators to avoid misleading future edits.
Sync Summary
Files Updated
Files Skipped
Review Checklist
Source: stranske/Workflows
Manifest:
.github/sync-manifest.yml