Add sequential numbering and suggested fixes to again skill output#9
Merged
Conversation
Issues now get a single global ID across must_fix/should_fix/suggestion tables so users can reference fixes by number (e.g. 'apply fix 1, 3, 5'). Each table gains a # column and a Suggested Fix column, and aggregate.md mirrors the same columns with full untruncated fix text.
There was a problem hiding this comment.
Pull request overview
Updates the again skill specification so aggregated issues receive a single global sequential identifier and both the conversational summary and aggregate.md include a # reference column plus a Suggested Fix column, enabling users to apply fixes by number.
Changes:
- Specify global sequential issue IDs (
1..N) assigned during aggregation and stored asidinaggregate.json. - Update Phase 3 reporting format to include
#andSuggested Fixcolumns and add guidance about preserving fullsuggested_fixcontent inaggregate.md. - Add user-facing guidance for referencing fixes by number (e.g., “apply fix 1, 3, 5”).
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Make Phase 2's "no restart" wording explicit and pull the user-reference
syntax ("apply fix 1, 3, 5") out of a parenthetical into its own step,
and call out the three-table layout in the Phase 3 intro. Add a
test_again_numbering_format suite to scripts/test.sh that greps for the
numbering, Suggested Fix column, '#' ID column, full-text aggregate
rule, and pipe-escape rule so future edits cannot silently drop them.
Plans for the again skill now exceed 2048 tokens because Phase 2 was expanded with explicit "no restart" wording and a separate Reference-by-ID step. Tests 8-10 were truncating mid-sentence before the summary-format section and failing rubric checks. 4096 leaves headroom for the full plan.
HartBrook
approved these changes
May 8, 2026
Owner
HartBrook
left a comment
HartBrook
left a comment
There was a problem hiding this comment.
Looks good, thanks for submitting!
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.
Summary
idinaggregate.json.aggregate.mdgain a#column and aSuggested Fixcolumn so users can reference fixes by number (e.g. "apply fix 1, 3, 5").aggregate.mdis specified to keep the fullsuggested_fixtext untruncated.