Skip to content

Make add-comment risk dynamic based on content and confidence#3545

Open
mattleibow wants to merge 1 commit intomainfrom
dev/dynamic-comment-risk
Open

Make add-comment risk dynamic based on content and confidence#3545
mattleibow wants to merge 1 commit intomainfrom
dev/dynamic-comment-risk

Conversation

@mattleibow
Copy link
Contributor

Summary

Previously, all add-comment actions in triage and repro JSON outputs were hardcoded as high risk regardless of content. This made the risk field meaningless for comments — every comment required human review equally.

Changes

Updated the add-comment risk from a static mapping to a dynamic calculation based on content type and confidence score:

Condition Risk Rationale
Factual only (repro findings, version matrix) AND confidence ≥ 0.85 low Reporting observed facts — minimal reputation exposure
Includes workaround or technical suggestion AND confidence ≥ 0.85 medium Advice could be wrong, but evidence is strong
Suggests closing, rejects the issue, or states "by-design" (any confidence) high Always needs human review
Any content AND confidence < 0.70 high Not confident enough
Default medium

Files changed

  • .github/skills/issue-triage/references/schema-cheatsheet.md — replaced static table with decision table
  • .github/skills/issue-repro/references/schema-cheatsheet.md — added decision table
  • .github/skills/issue-triage/references/triage-schema.json — updated actionRisk description
  • .github/skills/issue-repro/references/repro-schema.json — updated actionRisk description

Impact on existing cached JSON

Applying this to all 114 existing cached JSON files:

  • 29 actions drop from high → medium/low (less noise for human review)
  • 5 actions elevate from low/medium → high (close-paired comments that were under-rated)
  • 27 unchanged (already correct)

Previously, all add-comment actions were hardcoded as 'high' risk regardless
of content. This change makes the risk level dynamic:

- low: factual content (repro findings, version matrix) with confidence >= 0.85
- medium: workaround/suggestion content with confidence >= 0.85
- high: close/reject content, or confidence < 0.70

Updated in both triage and repro schema cheatsheets and schema definitions.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
mattleibow added a commit that referenced this pull request Mar 4, 2026
Re-evaluated all 64 add-comment actions across 114 cached JSON files
using the new dynamic risk calculation from PR #3545:

- 29 triage comments: high -> medium (workaround/suggestion content)
- 5 triage comments: high -> low (factual content, high confidence)
- 5 repro comments: low/medium -> high (paired with close actions)
- 2 repro comments: low -> medium (non-factual, moderate confidence)
- 1 repro comment: medium -> low (factual, high confidence)
- 27 unchanged (correctly high: close-paired or reject language)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

1 participant