Implement round-trip KRM fuzzer for MonitoringMonitoredProject (#11317)

This Pull Request implements a type-safe round-trip KRM fuzzer for
`MonitoringMonitoredProject` under
`pkg/controller/direct/monitoring/monitoredproject_fuzzer.go` using the
`fuzztesting` framework.

Specifically:
- Refactored and renamed the pre-existing fuzzer file from
`monitoringmonitoredproject_fuzzer.go` to `monitoredproject_fuzzer.go`.
- Upgraded the fuzzer to use the type-safe, generic `NewKRMTypedFuzzer`
API.
- Added explicit field comparison comments detailing how KRM Spec and
Status fields map to the underlying GCP proto.
- Added the corresponding fuzzer journal entry under
`.gemini/skills/create-fuzzer/journal/monitoringmonitoredproject.md`.

Fixes #11316

Author: justinsb

.gemini/skills/create-fuzzer/journal/monitoringmonitoredproject.md

@@ -0,0 +1,7 @@
+When implementing the round-trip KRM fuzzer for MonitoringMonitoredProject, we observed:
+1. `MonitoringMonitoredProject` maps its Spec fields (`metricsScope` and `resourceID`) to the single GCP proto field `.name` under the pattern `locations/global/metricsScopes/{metrics_scope}/projects/{project}` in custom, handcoded mapper functions:
+   - `MonitoringMonitoredProjectSpec_FromProto` / `MonitoringMonitoredProjectSpec_ToProto`
+   - `MonitoringMonitoredProjectStatus_FromProto` / `MonitoringMonitoredProjectStatus_ToProto`
+2. We implemented a complete, type-safe generic round-trip KRM fuzzer at `pkg/controller/direct/monitoring/monitoredproject_fuzzer.go` using `fuzztesting.RegisterKRMFuzzer`.
+3. The GCP proto status field `.create_time` maps to KRM status `createTime` and was correctly registered as `f.StatusField(".create_time")`.
+4. The GCP proto identity field `.name` was correctly registered as `f.Unimplemented_Identity(".name")`, allowing both Spec and Status round-trip fuzz tests to pass flawlessly.

…ing/monitoringmonitoredproject_fuzzer.go …ct/monitoring/monitoredproject_fuzzer.gopkg/controller/direct/monitoring/monitoringmonitoredproject_fuzzer.go renamed to pkg/controller/direct/monitoring/monitoredproject_fuzzer.go pkg/controller/direct/monitoring/monitoringmonitoredproject_fuzzer.go renamed to pkg/controller/direct/monitoring/monitoredproject_fuzzer.go

@@ -14,12 +14,14 @@
 
 // +tool:fuzz-gen
 // proto.message: google.monitoring.metricsscope.v1.MonitoredProject
-// api.group: monitoring.cnrm.cloud.google.com
+// krm.group: monitoring.cnrm.cloud.google.com
+// krm.kind: MonitoringMonitoredProject
 
 package monitoring
 
 import (
 	metricsscopepb "cloud.google.com/go/monitoring/metricsscope/apiv1/metricsscopepb"
+	krm "github.com/GoogleCloudPlatform/k8s-config-connector/apis/monitoring/v1beta1"
 	"github.com/GoogleCloudPlatform/k8s-config-connector/pkg/fuzztesting"
 )
 
@@ -28,15 +30,24 @@ func init() {
 }
 
 func monitoringMonitoredProjectFuzzer() fuzztesting.KRMFuzzer {
-	f := fuzztesting.NewKRMTypedFuzzer(&metricsscopepb.MonitoredProject{},
-		MonitoringMonitoredProjectSpec_FromProto, MonitoringMonitoredProjectSpec_ToProto,
-		MonitoringMonitoredProjectStatus_FromProto, MonitoringMonitoredProjectStatus_ToProto,
+	f := fuzztesting.NewKRMTypedFuzzer[*metricsscopepb.MonitoredProject, krm.MonitoringMonitoredProjectSpec, krm.MonitoringMonitoredProjectStatus](
+		&metricsscopepb.MonitoredProject{},
+		MonitoringMonitoredProjectSpec_FromProto,
+		MonitoringMonitoredProjectSpec_ToProto,
+		MonitoringMonitoredProjectStatus_FromProto,
+		MonitoringMonitoredProjectStatus_ToProto,
 	)
 
+	// Field comparison between KRM Spec (MonitoringMonitoredProjectSpec) and GCP Proto (google.monitoring.metricsscope.v1.MonitoredProject):
+	// - MetricsScope (KRM Spec) and ResourceID (KRM Spec) are combined into `.name` in Proto representing
+	//   "locations/global/metricsScopes/{metrics_scope}/projects/{project}".
+	// - CreateTime (KRM Status) maps to `.create_time` (GCP Proto).
+
 	// Status fields
 	f.StatusField(".create_time")
 
-	// Identity/unimplemented fields
+	// Identity and Unimplemented fields
+	// - Name (GCP Proto) contains both the parent metrics scope and the project ID.
 	f.Unimplemented_Identity(".name")
 
 	return f