Handle gemini review and lint issue

Signed-off-by: GUAN MING <guanmingchiu@gmail.com>

Author: guan404ming

utils/component/openapi_generator.go

@@ -204,132 +204,147 @@ func getResolvedManifest(manifest string) (string, error) {
 	return string(resolved), nil
 }
 
-// clearDocRefs clears $ref strings across the entire OpenAPI document,
-// including paths, components/parameters, components/headers,
-// components/requestBodies, components/responses, and components/callbacks.
+// clearDocRefs clears $ref strings across the entire OpenAPI document.
 func clearDocRefs(doc *openapi3.T) {
 	stack := make(map[*openapi3.Schema]bool)
+	visited := make(map[*openapi3.PathItem]bool)
 
 	if doc.Components != nil {
 		for _, sr := range doc.Components.Schemas {
 			clearSchemaRefs(sr, stack)
 		}
 		for _, pr := range doc.Components.Parameters {
-			clearParameterRefs(pr, stack)
+			clearParameterRefs(pr, stack, visited)
 		}
 		for _, hr := range doc.Components.Headers {
-			clearHeaderRefs(hr, stack)
+			clearHeaderRefs(hr, stack, visited)
 		}
 		for _, rb := range doc.Components.RequestBodies {
-			clearRequestBodyRefs(rb, stack)
+			clearRequestBodyRefs(rb, stack, visited)
 		}
 		for _, rr := range doc.Components.Responses {
-			clearResponseRefs(rr, stack)
+			clearResponseRefs(rr, stack, visited)
 		}
 		for _, cr := range doc.Components.Callbacks {
-			clearCallbackRefs(cr, stack)
+			clearCallbackRefs(cr, stack, visited)
+		}
+		for _, er := range doc.Components.Examples {
+			if er != nil {
+				er.Ref = ""
+			}
+		}
+		for _, lr := range doc.Components.Links {
+			if lr != nil {
+				lr.Ref = ""
+			}
+		}
+		for _, ssr := range doc.Components.SecuritySchemes {
+			if ssr != nil {
+				ssr.Ref = ""
+			}
 		}
 	}
 
 	if doc.Paths != nil {
 		for _, pathItem := range doc.Paths.Map() {
-			clearPathItemRefs(pathItem, stack)
+			clearPathItemRefs(pathItem, stack, visited)
 		}
 	}
 }
 
-func clearContentRefs(content openapi3.Content, stack map[*openapi3.Schema]bool) {
+func clearContentRefs(content openapi3.Content, stack map[*openapi3.Schema]bool, visited map[*openapi3.PathItem]bool) {
 	for _, mt := range content {
 		if mt != nil {
 			clearSchemaRefs(mt.Schema, stack)
 		}
 	}
 }
 
-func clearParameterRefs(pr *openapi3.ParameterRef, stack map[*openapi3.Schema]bool) {
+func clearParameterRefs(pr *openapi3.ParameterRef, stack map[*openapi3.Schema]bool, visited map[*openapi3.PathItem]bool) {
 	if pr == nil {
 		return
 	}
 	pr.Ref = ""
 	if pr.Value != nil {
 		clearSchemaRefs(pr.Value.Schema, stack)
-		clearContentRefs(pr.Value.Content, stack)
+		clearContentRefs(pr.Value.Content, stack, visited)
 	}
 }
 
-func clearHeaderRefs(hr *openapi3.HeaderRef, stack map[*openapi3.Schema]bool) {
+func clearHeaderRefs(hr *openapi3.HeaderRef, stack map[*openapi3.Schema]bool, visited map[*openapi3.PathItem]bool) {
 	if hr == nil {
 		return
 	}
 	hr.Ref = ""
 	if hr.Value != nil {
 		clearSchemaRefs(hr.Value.Schema, stack)
-		clearContentRefs(hr.Value.Content, stack)
+		clearContentRefs(hr.Value.Content, stack, visited)
 	}
 }
 
-func clearRequestBodyRefs(rb *openapi3.RequestBodyRef, stack map[*openapi3.Schema]bool) {
+func clearRequestBodyRefs(rb *openapi3.RequestBodyRef, stack map[*openapi3.Schema]bool, visited map[*openapi3.PathItem]bool) {
 	if rb == nil {
 		return
 	}
 	rb.Ref = ""
 	if rb.Value != nil {
-		clearContentRefs(rb.Value.Content, stack)
+		clearContentRefs(rb.Value.Content, stack, visited)
 	}
 }
 
-func clearResponseRefs(rr *openapi3.ResponseRef, stack map[*openapi3.Schema]bool) {
+func clearResponseRefs(rr *openapi3.ResponseRef, stack map[*openapi3.Schema]bool, visited map[*openapi3.PathItem]bool) {
 	if rr == nil {
 		return
 	}
 	rr.Ref = ""
 	if rr.Value != nil {
-		clearContentRefs(rr.Value.Content, stack)
+		clearContentRefs(rr.Value.Content, stack, visited)
 		for _, hr := range rr.Value.Headers {
-			clearHeaderRefs(hr, stack)
+			clearHeaderRefs(hr, stack, visited)
 		}
 	}
 }
 
-func clearCallbackRefs(cr *openapi3.CallbackRef, stack map[*openapi3.Schema]bool) {
+func clearCallbackRefs(cr *openapi3.CallbackRef, stack map[*openapi3.Schema]bool, visited map[*openapi3.PathItem]bool) {
 	if cr == nil {
 		return
 	}
 	cr.Ref = ""
 	if cr.Value != nil {
 		for _, pathItem := range cr.Value.Map() {
-			clearPathItemRefs(pathItem, stack)
+			clearPathItemRefs(pathItem, stack, visited)
 		}
 	}
 }
 
-func clearPathItemRefs(pathItem *openapi3.PathItem, stack map[*openapi3.Schema]bool) {
-	if pathItem == nil {
+func clearPathItemRefs(pathItem *openapi3.PathItem, stack map[*openapi3.Schema]bool, visited map[*openapi3.PathItem]bool) {
+	if pathItem == nil || visited[pathItem] {
 		return
 	}
+	visited[pathItem] = true
 	for _, pr := range pathItem.Parameters {
-		clearParameterRefs(pr, stack)
+		clearParameterRefs(pr, stack, visited)
 	}
 	for _, op := range pathItem.Operations() {
-		clearOperationRefs(op, stack)
+		clearOperationRefs(op, stack, visited)
 	}
 }
 
-func clearOperationRefs(op *openapi3.Operation, stack map[*openapi3.Schema]bool) {
+func clearOperationRefs(op *openapi3.Operation, stack map[*openapi3.Schema]bool, visited map[*openapi3.PathItem]bool) {
 	if op == nil {
 		return
 	}
 	for _, pr := range op.Parameters {
-		clearParameterRefs(pr, stack)
+		clearParameterRefs(pr, stack, visited)
 	}
-	clearRequestBodyRefs(op.RequestBody, stack)
+	clearRequestBodyRefs(op.RequestBody, stack, visited)
 	if op.Responses != nil {
 		for _, rr := range op.Responses.Map() {
-			clearResponseRefs(rr, stack)
+			clearResponseRefs(rr, stack, visited)
 		}
 	}
 	for _, cr := range op.Callbacks {
-		clearCallbackRefs(cr, stack)
+		clearCallbackRefs(cr, stack, visited)
 	}
 }