cleaning up the last few stragglers

Signed-off-by: quobix <dave@quobix.com>

datamodel/low/extraction_functions.go

@@ -530,10 +530,6 @@ func ExtractMapNoLookupExtensions[PT Buildable[N], N any](
 					}
 				}
 			}
-			if foundIndex == nil {
-				foundIndex = idx
-			}
-
 			var n PT = new(N)
 			err := BuildModel(node, n)
 			if err != nil {
@@ -553,7 +549,6 @@ func ExtractMapNoLookupExtensions[PT Buildable[N], N any](
 				}] = ValueReference[PT]{
 					Value:     n,
 					ValueNode: node,
-					//IsReference: isReference,
 					Reference: referenceValue,
 				}
 			}

document.go

@@ -370,11 +370,7 @@ func CompareDocuments(original, updated Document) (*model.DocumentChanges, []err
 		if len(uErrs) > 0 {
 			errs = append(errs, uErrs...)
 		}
-		if v2ModelLeft != nil && v2ModelRight != nil {
-			return what_changed.CompareSwaggerDocuments(v2ModelLeft.Model.GoLow(), v2ModelRight.Model.GoLow()), errs
-		} else {
-			return nil, errs
-		}
+		return what_changed.CompareSwaggerDocuments(v2ModelLeft.Model.GoLow(), v2ModelRight.Model.GoLow()), errs
 	}
 	return nil, []error{fmt.Errorf("unable to compare documents, one or both documents are not of the same version")}
 }

index/resolver.go

@@ -335,10 +335,6 @@ func (resolver *Resolver) isInfiniteCircularDependency(ref *Reference, visitedDe
 		if initialRef != nil && initialRef.Definition == r.Definition {
 			return true, visitedDefinitions
 		}
-
-		if visitedDefinitions[r.Definition] {
-			continue
-		}
 		visitedDefinitions[r.Definition] = true
 
 		ir := initialRef