Skip to content

Add resolving findings skill - #4

Draft
QuentinTorg wants to merge 5 commits into
feat/reviewing-code-skillfrom
feat/resolving-findings-skill
Draft

Add resolving findings skill#4
QuentinTorg wants to merge 5 commits into
feat/reviewing-code-skillfrom
feat/resolving-findings-skill

Conversation

@QuentinTorg

@QuentinTorg QuentinTorg commented Jul 28, 2026

Copy link
Copy Markdown
Owner

Summary

  • define the contract for a focused resolving-findings skill
  • preserve the original author/reviewer loop while bounding fixer authority to selected findings
  • separate local code correction from GitHub, review publication, and lifecycle actions
  • add normalized finding intake, risk-based resolution policy, verification and rereview handoff guidance
  • add a proportional result template and behavioral evaluation plan

Stack

Verification

  • scoped git diff --check -- skills/resolving-findings
  • frontmatter parsed as YAML; description is 335 characters
  • all relative reference and asset targets verified
  • no exact long-line duplication across the package
  • trigger and behavioral assertions documented in skills/resolving-findings/evals/evals.md

The repository-wide whitespace check still reports a trailing space in an unrelated pre-existing edit to skills/addressing-review-feedback/SKILL.md; this branch does not include or modify that work.

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