Skip to content

Implement runtime support for issue remediation#6578

Open
codejasleen wants to merge 3 commits into
mindersec:mainfrom
codejasleen:issue-remediation-runtime
Open

Implement runtime support for issue remediation#6578
codejasleen wants to merge 3 commits into
mindersec:mainfrom
codejasleen:issue-remediation-runtime

Conversation

@codejasleen

Copy link
Copy Markdown
Contributor

Summary

This PR completes the runtime implementation for Issue Remediation, building on the previously introduced provider interface and remediation configuration.

Changes

  • Add the Issue remediation engine.

  • Implement runtime support for:

    • creating GitHub issues when remediation is enabled
    • closing previously created issues when remediation is disabled
  • Persist issue metadata to avoid creating duplicate issues while a remediation remains active.

  • Wire the issue remediation type into the remediation engine.

  • Enable type: issue in remediation validation.

  • Add validation for IssueRemediation configuration.

  • Add unit tests covering:

    • issue creation
    • issue creation failures
    • existing issue reuse
    • issue closure
    • issue closure without stored metadata
  • Regenerate protobuf, OpenAPI, and documentation after enabling the new remediation type.

This makes issue a functional remediation type end-to-end.

Fixes #(related issue)

Testing

Verified with:

go test ./internal/engine/actions/remediate/issue
go test ./internal/engine/actions/remediate/...
go build ./...

Future work

Potential follow-up improvements include:

  • configurable issue labels

  • configurable issue assignees

  • optional comment when automatically closing an issue

  • support for reopening previously closed remediation issues

@codejasleen codejasleen requested a review from a team as a code owner July 7, 2026 14:00
@coveralls

coveralls commented Jul 7, 2026

Copy link
Copy Markdown

Coverage Status

coverage: 60.844% (-0.03%) from 60.872% — codejasleen:issue-remediation-runtime into mindersec:main

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.

2 participants