Go 1.21previous case (typecheck) on first case of a switch statement

[codeliger]

My CI won't pass and I cannot reproduce it by pulling code out into a smaller file.
My local golangci-lint does not raise the error.
If i install golangci-lint manually on my CI it raises the error.
If i install the github action with the latest version it raises the error.
I checked and all versions are 1.54.2 (latest)
No matter what I do I cannot reproduce it locally.
I tried clearing cache in go and golangci-lint.

Here is an anonymized version of my source code that does not throw the error, but the original throws an error at the first case statement error previous case (typecheck).

func SeedDatabase(config *viper.Viper, db *bun.DB) (*dbfixture.Fixture, error) {
    // choose which fields to do update conflict resolution on
    fixture := dbfixture.New(db, dbfixture.WithBeforeInsert(
        func(ctx context.Context, data *dbfixture.BeforeInsertData) error {
            columns := []string{}
            switch t := data.Model.(type) {
            case *models.Object1:
                columns = append(columns, "id")
            case *models.Object2:
                columns = append(columns, "field1", "field2")
            case *models.Object3:
                columns = append(columns, "id")
            case *models.Object4:
                columns = append(columns, "id")
            case *models.Object5:
                columns = append(columns, "field1", "field2")
            case *models.Object6:
                columns = append(columns, "field1", "field2")
            case *models.Object7:
                columns = append(columns, "id")
            case *models.Object8:
                columns = append(columns, "id")
            case *models.Object9:
                columns = append(columns, "id")
            default:
                panic(t)
            }

            if len(columns) > 0 {
                fields := strings.Join(columns, ",")
                data.Query.Apply(func(q *bun.InsertQuery) *bun.InsertQuery {
                    onConflict := fmt.Sprintf("CONFLICT (%s) DO UPDATE", fields)
                    return q.On(onConflict)
                })
            }

            return nil
        }))

    ctx := context.Background()

    // Register models before loading fixtures.
    db.RegisterModel((*models.Object1)(nil))
    db.RegisterModel((*models.Object2)(nil))
    db.RegisterModel((*models.Object3)(nil))
    db.RegisterModel((*models.Object4)(nil))
    db.RegisterModel((*models.Object5)(nil))
    db.RegisterModel((*models.Object6)(nil))
    db.RegisterModel((*models.Object7)(nil))
    db.RegisterModel((*models.Object8)(nil))
    db.RegisterModel((*models.Object9)(nil))

    seedDir := filepath.Join(util.FindNearestWdParentFolder("app-backend"), "database", "seeds")
    err := fixture.Load(ctx, os.DirFS(seedDir),
        "object1.yaml",
        "object2.yaml",
        "object3.yaml",
        "object4.yaml",
        "object5.yaml",
        "object6.yaml",
        "object7.yaml",
        "object8.yaml",
        "object9.yaml",
    )
    if err != nil {
        return nil, err
    }

    return fixture, nil
}

[ldez]

Hello,

it's hard to diagnose this problem.
previous case (typecheck) means that a case inside the switch has 2 times the same element.
But as it's a compilation error, the problem can be somewhere else.

I recommend trying to check if you have a compilation error inside your CI by following the process here: https://golangci-lint.run/usage/faq/#why-do-you-have-typecheck-errors

[shelldandy]

Im having pretty much that same issue as well

[ldez]

Do you have a reproducible example?

[markus-wa]

I have the same issue but also can't create a minimal reproduction example as it's in a big private project and all small examples work fine