feat: define interface for event recorder

[bakito]

This PR introduces an interface for the newly introduced capsule event recorder.

Using the interface helps to hide the implementing struct from the user and allows better testing.

[Copilot]

Pull request overview

This PR refactors the Capsule runtime event recorder to expose an EventRecorder interface (instead of a concrete struct) and keeps the implementation type unexported, aiming to hide implementation details and improve testability.

Changes:

• Replaced the exported EventRecorder struct with an exported EventRecorder interface and introduced an unexported eventRecorder implementation.
• Updated NewEventRecorder to return the interface type and adjusted internal receivers accordingly.
• Updated controller wiring to pass the returned interface value (removing the previous dereference).

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File	Description
pkg/runtime/events/recorder.go	Introduces the exported EventRecorder interface and unexported eventRecorder implementation; updates constructor and receivers.
pkg/runtime/events/event_types.go	Adjusts LabeledEvent to reference the new unexported recorder implementation and updates the LabeledEvent factory receiver.
cmd/controller/main.go	Updates webhook.Register call to pass the interface returned by NewEventRecorder (no dereference).

Comments suppressed due to low confidence (1)

pkg/runtime/events/event_types.go:38

• EventRecorder.LabeledEvent() returns *LabeledEvent, but LabeledEvent has only unexported fields and relies on internal initialization (labels/annotations maps). This makes it effectively impossible for code outside the events package to implement EventRecorder without risking nil-map panics in WithLabels/WithAnnotations or returning a non-functional event, which undermines the stated testing goal.

func (r *eventRecorder) LabeledEvent(
	regarding runtime.Object,
	eventType string,
	reason string,
	action string,
	note string,
) *LabeledEvent {

[Copilot]

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

[Copilot]

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

[Copilot]

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.