Skip to content

Explain the matching root cause instead of citing the duplicate's report ID - #26

Open
manuelconcari-ai wants to merge 1 commit into
hackenproof-public:mainfrom
manuelconcari-ai:codex/duplicate-comment-standalone-explanation
Open

manuelconcari-ai wants to merge 1 commit into
hackenproof-public:mainfrom
manuelconcari-ai:codex/duplicate-comment-standalone-explanation

Conversation

@manuelconcari-ai

@manuelconcari-ai manuelconcari-ai commented Sep 10, 2026

Copy link
Copy Markdown

The Duplicate template in references/triage-comment-templates.md puts 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_comment are untouched. Internal dup-{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.

@manuelconcari-ai
manuelconcari-ai force-pushed the codex/duplicate-comment-standalone-explanation branch from 3162dbe to 28f25b1 Compare September 10, 2026 22:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant