Merged
Conversation
Scope is informational context, not actionable items to check off. 1. Add stripCheckboxesFromScope() function that: - Converts '- [ ] text' to '- text' (plain bullets) - Filters out placeholder content entirely - Removes empty lines 2. Update PR_META_FALLBACK_PLACEHOLDERS.scope: - Change from '- [ ] Scope section missing...' to italicized text - Matches standard PLACEHOLDERS.scope pattern 3. Update consumer repo template with same placeholder change 4. Update test to match new placeholder format This ensures scope sections in: - PR Automated Status Summary (via issue_scope_parser.js) - Verifier follow-up issues (via verifier_issue_formatter.js) ...display as informational text rather than checkbox items.
Add extractSourceSection() to parse Source section from PR body and include it in buildTaskAppendix(). This gives Codex visibility into linked parent issues and original PRs for additional context. Changes: - Add extractSourceSection() function to extract Source links from PR body - Update buildTaskAppendix() to accept options.prBody parameter - Add 'Source Context' section to agent prompt when Source links present - Add CONTEXT TIP to keepalive-instruction.md template - Add 5 new tests for source section extraction When a PR has a Source section with links like: - Original PR: #123 - Parent issue: #456 The agent prompt now includes these as 'Source Context' so Codex knows to check linked issues/PRs for additional background.
The src/trend_analysis/ directory contains CI test fixtures used to exercise the autofix pipeline workflows - not production code. These files (e.g., _autofix_trigger_sample.py, _ci_probe_faults.py) were showing 0% coverage and artificially dragging down overall metrics. Add 'src/trend_analysis/*' to [tool.coverage.run].omit patterns.
These CI autofix test fixtures were at risk of being mistakenly deleted or flagged as orphaned code. The README explains: - Why these files exist (autofix pipeline testing) - Why they're excluded from coverage - Which tests use them
The Workflows repo was missing the caller workflow for the bot comment handler - only the reusable version existed. This meant the 'autofix:bot-comments' label had no effect. Copied from templates/consumer-repo/.github/workflows/ to enable: - Manual trigger via 'autofix:bot-comments' label - Automatic trigger after Gate completion for agent PRs - Manual dispatch via workflow_dispatch
…r bug Addresses 3 review comments from Copilot and Codex: 1. **Copilot (regex)**: Changed `\s+` to `\s*` in checkbox pattern to handle edge cases where space after bracket is missing (e.g., `- [ ]text`) 2. **Copilot (tests)**: Added 11 new tests for `stripCheckboxesFromScope()`: - Checkbox-to-bullet conversion (unchecked, x, X) - Preservation of indentation - Filtering of placeholder content - Handling of empty lines and null input - Edge case: no space after bracket 3. **Codex P1 (scope placeholder alignment)**: Fixed critical regression where `agents_pr_meta_update_body.js` was still emitting checkbox format for scope fallback while `issue_scope_parser.js` changed to italicized format. Scope is now treated as informational (plain text), not actionable (checkbox). Tests: 471 passing (up from 424)
Add instruction #5 and CRITICAL section to keepalive-instruction.md asking Codex to output '✅ Completed: [task text]' in final summary. This enables reliable reconciliation parsing for completed tasks.
Consumer repos had their own copies that weren't being updated. Now the template will sync automatically with other consumer files.
Contributor
Contributor
Automated Status SummaryHead SHA: 414f2ae
Coverage Overview
Coverage Trend
Updated automatically; will refresh on subsequent CI/Docker completions. Keepalive checklistScopeNo scope information available Tasks
Acceptance criteria
|
stranske
temporarily deployed
to
agent-standard
GitHub Actions
Inactive
Contributor
🤖 Keepalive Loop StatusPR #327 | Agent: Codex | Iteration 0/5 Current State
🔍 Failure Classification| Error type | infrastructure | |
Contributor
There was a problem hiding this comment.
Pull request overview
This PR adds a new keepalive instruction template to guide automated agents (Codex) in completing tasks with proper checkbox tracking. The template provides explicit directives about code implementation, checkbox updates, and summary formatting to prevent issues with scope checkbox stripping.
- Introduces a new keepalive-instruction.md template with detailed task completion guidelines
- Adds the template to the sync manifest for distribution to consumer repositories
- Provides explicit instructions for checkbox format updates and completion tracking
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| templates/consumer-repo/.github/templates/keepalive-instruction.md | New instruction template providing detailed guidelines for agents to implement code changes, update task checkboxes, and format completion summaries |
| .github/sync-manifest.yml | Adds new templates section to manifest for syncing the keepalive instruction template, but has a structural issue with section placement |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Owner
Author
|
@codex fix comments |
|
Summary
Testing
|
8 tasks
stranske
added a commit
that referenced
this pull request
Feb 7, 2026
Adds noqa: N806 comments to suppress false positive naming convention warnings where ChatAnthropic is set to None as an ImportError fallback. The variable name intentionally matches the class name for consistency. Fixes linting failures in sync PR #327 (stranske/Collab-Admin)
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.
No description provided.