Make TryInsert functions within the packages module use INSERT ... ON CONFLICT

models/packages/package.go

@@ -128,14 +128,11 @@ func TryInsertPackage(ctx context.Context, p *Package) (*Package, error) {
 		LowerName: p.LowerName,
 	}
 
-	has, err := e.Get(key)
-	if err != nil {
-		return nil, err
-	}
-	if has {
-		return key, ErrDuplicatePackage
-	}
-	if _, err = e.Insert(p); err != nil {
+	if _, err := e.Insert(p); err != nil {
+		// Try to get the key again
+		if has, _ := e.Get(key); has {
+			return key, ErrDuplicatePackage
+		}
 		return nil, err
 	}
 	return p, nil

models/packages/package_file.go

@@ -53,14 +53,10 @@ func TryInsertFile(ctx context.Context, pf *PackageFile) (*PackageFile, error) {
 		CompositeKey: pf.CompositeKey,
 	}
 
-	has, err := e.Get(key)
-	if err != nil {
-		return nil, err
-	}
-	if has {
-		return pf, ErrDuplicatePackageFile
-	}
-	if _, err = e.Insert(pf); err != nil {
+	if _, err := e.Insert(pf); err != nil {
+		if has, _ := e.Get(key); has {
+			return pf, ErrDuplicatePackageFile
+		}
 		return nil, err
 	}
 	return pf, nil

models/packages/package_version.go

@@ -46,14 +46,10 @@ func GetOrInsertVersion(ctx context.Context, pv *PackageVersion) (*PackageVersio
 		LowerVersion: pv.LowerVersion,
 	}
 
-	has, err := e.Get(key)
-	if err != nil {
-		return nil, err
-	}
-	if has {
-		return key, ErrDuplicatePackageVersion
-	}
-	if _, err = e.Insert(pv); err != nil {
+	if _, err := e.Insert(pv); err != nil {
+		if has, _ := e.Get(key); has {
+			return key, ErrDuplicatePackageVersion
+		}
 		return nil, err
 	}
 	return pv, nil