-
Notifications
You must be signed in to change notification settings - Fork 1
Fix/scope checkbox stripping #327
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
11 commits
Select commit
Hold shift + click to select a range
6fd45e5
fix: strip checkboxes from scope section in verifier follow-up issues
stranske 6615ecb
feat(keepalive): include Source section links in agent prompt
stranske c4f03f9
fix(coverage): exclude src/trend_analysis from coverage metrics
stranske dd2570e
docs: add README explaining src/trend_analysis test fixtures
stranske 4957a0e
Merge branch 'main' into fix/scope-checkbox-stripping
stranske 8474df2
feat: add agents-bot-comment-handler workflow to Workflows repo
stranske 6c28afe
chore(codex-autofix): apply updates (PR #322)
github-actions[bot] 19da1ca
fix: address bot review comments - regex, tests, and scope placeholde…
stranske b9228c3
feat: ask Codex to post explicit task completion markers
stranske a4604e2
sync: add keepalive-instruction.md template to sync manifest
stranske 36e1f81
Merge branch 'main' into fix/scope-checkbox-stripping
stranske File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
34 changes: 34 additions & 0 deletions
34
templates/consumer-repo/.github/templates/keepalive-instruction.md
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,34 @@ | ||
| Your objective is to satisfy the **Acceptance Criteria** by completing each **Task** within the defined **Scope**. | ||
|
|
||
| **This round you MUST:** | ||
| 1. Implement actual code or test changes that advance at least one incomplete task toward acceptance. | ||
| 2. Commit meaningful source code (.py, .yml, .js, etc.)—not just status/docs updates. | ||
| 3. **UPDATE THE CHECKBOXES** in the Tasks and Acceptance Criteria sections below to mark completed items. | ||
| 4. Change `- [ ]` to `- [x]` for items you have completed and verified. | ||
| 5. **In your final summary**, list completed tasks using the format: `✅ Completed: [exact task text]` | ||
|
|
||
| **CRITICAL - Checkbox Updates:** | ||
| When you complete a task or acceptance criterion, update its checkbox directly in this prompt file. Change the `[ ]` to `[x]` for completed items. The automation will read these checkboxes and update the PR's status summary. | ||
|
stranske marked this conversation as resolved.
|
||
|
|
||
| **CRITICAL - Summary Format:** | ||
| At the end of your work, include explicit completion markers for each task you finished: | ||
| ``` | ||
| ✅ Completed: Add validation for user input | ||
| ✅ Completed: Write unit tests for validator module | ||
| ``` | ||
| This helps the automation accurately track which tasks were addressed in this round. | ||
|
|
||
| **Example:** | ||
| Before: `- [ ] Add validation for user input` | ||
| After: `- [x] Add validation for user input` | ||
|
|
||
| **DO NOT:** | ||
| - Commit only status files, markdown summaries, or documentation when tasks require code. | ||
| - Mark checkboxes complete without actually implementing and verifying the work. | ||
| - Close the round without source-code changes when acceptance criteria require them. | ||
| - Change the text of checkboxes—only change `[ ]` to `[x]`. | ||
|
stranske marked this conversation as resolved.
|
||
|
|
||
| **CONTEXT TIP:** | ||
| If the PR body includes a **Source** section with links to a parent issue or original PR, those contain additional context about the problem being solved. Check the linked issue/PR for background information, related discussions, or details not captured in the Scope section. | ||
|
|
||
| Review the Scope/Tasks/Acceptance below, identify the next incomplete task that requires code, implement it, then **update the checkboxes** to mark completed items. | ||
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.