Explain the matching root cause instead of citing the duplicate's report ID - #26
Open
manuelconcari-ai wants to merge 1 commit into
Conversation
manuelconcari-ai
force-pushed
the
codex/duplicate-comment-standalone-explanation
branch
from
September 10, 2026 22:31
3162dbe to
28f25b1
Compare
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.
The Duplicate template in
references/triage-comment-templates.mdputs the other report's identifier in the comment body:This issue matches an existing report with the same root cause and impact: **<report-id>**.The repository forbids that elsewhere. The comment-templates skill states "Never reference other report IDs in the comment body — reporters cannot see them"; its own Duplicate template asks instead for one sentence "without referencing the other report's ID"; the handoff output rules repeat it for client summaries; and the recorded triage mistakes list the same thing as a mistake to avoid.
Replace the identifier with a short standalone explanation of the confirmed matching root cause, taken only from facts already verified in the current report. That also satisfies the recorded rule that a duplicate comment must address the specific finding in the report it answers rather than a generic reason.
Only the Duplicate section of that file changes: 1 file, +6/−3. The draft comment goes from three sentences to two, which is what the bulk-triage skill asks for and the current template does not meet. The acknowledgment, the Duplicate decision, the other templates, and the triage skill's confirmation requirement before
add_commentare untouched. Internaldup-{id}labels are unaffected: they are not part of the comment body.Validation: the patch applies and reverses at main 60d8813 with
--whitespace=error-all, changes only that file, and the cited sources were checked against that commit. No agent run, deployed template, platform access control, or actual disclosure was tested.