fix: import and arg collision

mockgen/internal/tests/import_collision/internalpackage/foo.go

@@ -0,0 +1,3 @@
+package internalpackage
+
+type FooExported struct{}

mockgen/internal/tests/import_collision/p2/p2.go

@@ -0,0 +1,12 @@
+package p2
+
+//go:generate mockgen -destination=mocks/mocks.go -package=internalpackage . Mything
+
+import (
+	"go.uber.org/mock/mockgen/internal/tests/import_collision/internalpackage"
+)
+
+type Mything interface {
+	// issue here, is that the variable has the same name as an imported package.
+	DoThat(internalpackage int) internalpackage.FooExported
+}

mockgen/mockgen.go

@@ -766,16 +766,30 @@ func (g *generator) getArgNames(m *model.Method, in bool) []string {
 		params = m.Out
 	}
 	argNames := make([]string, len(params))
+
+	// check if the name already exists as an imported package name
+	// which would cause a name collision
+	nameExistsAsPackage := func(name string) bool {
+		for _, symbolName := range g.packageMap {
+			if symbolName == name {
+				return true
+			}
+		}
+		return false
+	}
+
 	for i, p := range params {
 		name := p.Name
-		if name == "" || name == "_" {
+
+		if name == "" || name == "_" || nameExistsAsPackage(name) {
 			name = fmt.Sprintf("arg%d", i)
 		}
 		argNames[i] = name
 	}
 	if m.Variadic != nil && in {
 		name := m.Variadic.Name
-		if name == "" {
+
+		if name == "" || nameExistsAsPackage(name) {
 			name = fmt.Sprintf("arg%d", len(params))
 		}
 		argNames = append(argNames, name)